fix(agents): point a trimming workflow to its audit #224
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "fix/trim-opening-step"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Why
Agent Foundry gives coding agents their rules in
AGENTS.mdand theirprocedures as skills, and routes each request to one workflow skill. A
change that shortens those rules, a trim, should first get a
documentation audit, which the
implement-featureworkflow's first stepsaves. When the request goes to another workflow, such as
fix-bugfora defect in a skill, the agent read nothing on its way that sent it to
that step, so the trim could go through its work unaudited. The one
pointer left, since
612d6b6moved the rule out ofAGENTS.md, was thelast clause of
implement-feature's description.Pointers in the opening steps of
fix-bug,document-feature, andrefactor-codewere set aside, as three places to keep alike. A step inrun-changewas set aside, since it runs after the opening step theaudit must precede.
Closes #220
What changed
Before, an agent routed to
fix-bugwith a change that trims a skillwent straight to the diagnosis. Now the routing rule of
AGENTS.mdtells it to keep its route and first run
implement-featurestep 1. Achange that trims nothing meets no new duty, and the rule's body stays
in
implement-feature.Before, the upgrade migration that moved the old rule applied while
AGENTS.mdheld "the Workflows bullets on a trim or upgrade", which thenew sentence could be taken for at every later upgrade. Now it names
the old bullet as a bullet of its own that the routing sentence
replaces, and sends a repository's own additions to the kinds a trim
covers to that sentence too.
The changes are in the first bullet of
AGENTS.md§ Workflows and inthe migration Workflow start points in skills of
UPGRADING.md, whoseparagraph Prettier rewrapped;
git diff --word-diffshows the wordingalone.
How to verify
Have Git and a clone of this repository, with
git fetch origin.The problem. On
main, neither the rules nor the defect workflowname a trim:
Both print nothing.
The solution. On the branch,
prints line 327 of
AGENTS.md, "skill, a template, or a subagentbrief keeps its route and first runs", the sentence to read with the
lines around it.
The checks of
docs/MAINTENANCE.mdpass on the tip. No test reads therouting rule, so the change adds none.
Not tried: an agent in a trial repository asked to fix a defect in a
skill by trimming it, on
mainand on the branch, and the behavioralevals that
docs/MAINTENANCE.mdhas the maintainer run before merginga change to
AGENTS.md; both need billed sessions outside the sandbox.Considerations
The sentence repeats the kinds a trim covers, which
implement-featurealso holds, so that an agent can tell at routing time whether the rule
concerns its change.
AGENTS.mdstill leaves the word trim undefined,as the old bullet did; a reviewer's finding on it was set aside as a
decision of its own.