Skip to content

chore(dev): update agent skills - #414

Merged
alexey-igrychev merged 1 commit into
mainfrom
chore/dev/sync-werf-skills
Aug 5, 2026
Merged

chore(dev): update agent skills#414
alexey-igrychev merged 1 commit into
mainfrom
chore/dev/sync-werf-skills

Conversation

@alexey-igrychev

@alexey-igrychev alexey-igrychev commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

Agent guidance committed with trdl now uses the current workflow, review, test-verification, and session-retro procedures, adapted to this repository. This changes how agents prepare and assess trdl changes; it does not change trdl runtime behavior.

What

  • Agents creating branches, commits, and pull requests follow current workflow conventions; branch/base rules name main and release-please release-v* branches, not another project's version branches.
  • Agents reviewing a diff use the current review skill; the obsolete rigorous-review skill is removed.
  • The review skill checks against trdl facts: multi-module layout (server/, client/, release/, e2e/), the Linux-only CGO Vault plugin, GPG/TUF security paths, task server:*/client:*/release:*/e2e:* build and test commands, TRDL_* (and TRDL_VAULT_*) env counterparts, task docs:gen for reference pages, release-please ownership of CHANGELOG.md, samber/lo server-only usage, and AI-test naming (*_ai_test.go, //go:build ai_tests, TestAI_).
  • Agents changing tests follow the current test-falsifiability procedure; agents receiving review feedback apply its updated response guidance.
  • Agents can run the current session-retro procedure after work completes.
  • No instruction references or assumes another project's tools, config, release branches, or dependencies.
  • trdl binaries, CLI behavior, Task targets, and release artifacts do not change.

Why

The committed copies had diverged from the current shared skill procedures, and the imported text carried assumptions specific to another project (release branches, deploy engine, container backends, giterminism). Left as-is, agents would receive guidance that is wrong for trdl. Adapting the procedures to trdl's own layout, security model, and task commands keeps the skills useful here without a project-specific fork of unrelated rules.

@alexey-igrychev

alexey-igrychev commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Verification

  • Confirmed .agents/skills contains no werf reference and no leftover foreign-project terms (nelm, helm, buildah, buildx, giterminism, stapel, kubedog, netavark, WERF_).
  • Cross-review by an independent reviewer agent informed the trdl adaptation of the review skill; findings were folded back into this branch.
  • Ran git diff --check.

Review focus

  • .agents/skills/review/SKILL.md — verify the trdl gotchas (Vault/TUF security paths, Linux-only CGO server plugin, multi-module coordination) and task/TRDL_* claims match the repository.

@alexey-igrychev
alexey-igrychev force-pushed the chore/dev/sync-werf-skills branch from a15fcf0 to 9e0dd09 Compare August 5, 2026 17:53
@alexey-igrychev alexey-igrychev changed the title chore(dev): sync agent skills from werf chore(dev): update agent skills Aug 5, 2026
Update committed agent workflow, review, test-verification, and session-retro instructions and adapt them to trdl: multi-module layout, the Vault plugin and TUF security paths, `task` build/test commands, `TRDL_*` env conventions, release-please ownership of CHANGELOG, and AI-test naming. Drop assumptions carried over from another project.

Signed-off-by: Aleksei Igrychev <aleksei.igrychev@palark.com>
@alexey-igrychev
alexey-igrychev force-pushed the chore/dev/sync-werf-skills branch from 9e0dd09 to fdb75ae Compare August 5, 2026 18:15
@alexey-igrychev
alexey-igrychev marked this pull request as ready for review August 5, 2026 18:25
@alexey-igrychev
alexey-igrychev merged commit 287b96c into main Aug 5, 2026
22 checks passed
@alexey-igrychev
alexey-igrychev deleted the chore/dev/sync-werf-skills branch August 5, 2026 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant