implement-feature step 2 renames a handed-in audit #197

Closed
opened 2026-11-05 11:00:02 +01:00 by motoko · 0 comments
Collaborator

Defect

implement-feature step 2 and AGENTS.md § Work folders disagree on
the name of an audit the user hands in for a trim, and step 2 does not
say whether an audit kept under another name counts as held.

Location and evidence

Seen at 58ba061.

  • AGENTS.md:342-345: "audits/ holds only the audits a work starts
    from: one the user hands in, under its own name, and otherwise a
    documentation audit as docs.md".
  • .agents/skills/implement-feature/SKILL.md:24-28: "For a trim whose
    work folder holds no documentation audit, save document-reviewer's
    audit, or the one the user hands in, as audits/docs.md".

Scenario

The user hands in ux-review.md for a trim of a skill. If the agent
follows AGENTS.md, it keeps the file as audits/ux-review.md. Step 2
then has to decide whether the folder "holds no documentation audit",
and may run document-reviewer for a second audit. If the agent follows
step 2, it renames the file to audits/docs.md, against AGENTS.md.

Smallest fix

In step 2, save only document-reviewer's audit as audits/docs.md.
Keep an audit the user hands in under its own name, and count any audit
in audits/ as held.

Found in review of
#196, outside
that change.

https://claude.ai/code/session_012CBsgjdkRDt2uVUmT7bj64

## Defect `implement-feature` step 2 and `AGENTS.md` § Work folders disagree on the name of an audit the user hands in for a trim, and step 2 does not say whether an audit kept under another name counts as held. ## Location and evidence Seen at 58ba061. - `AGENTS.md:342-345`: "`audits/` holds only the audits a work starts from: one the user hands in, under its own name, and otherwise a documentation audit as `docs.md`". - `.agents/skills/implement-feature/SKILL.md:24-28`: "For a trim whose work folder holds no documentation audit, save `document-reviewer`'s audit, or the one the user hands in, as `audits/docs.md`". ## Scenario The user hands in `ux-review.md` for a trim of a skill. If the agent follows `AGENTS.md`, it keeps the file as `audits/ux-review.md`. Step 2 then has to decide whether the folder "holds no documentation audit", and may run `document-reviewer` for a second audit. If the agent follows step 2, it renames the file to `audits/docs.md`, against `AGENTS.md`. ## Smallest fix In step 2, save only `document-reviewer`'s audit as `audits/docs.md`. Keep an audit the user hands in under its own name, and count any audit in `audits/` as held. Found in review of https://git.unitoo.it/claudiomaradonna/agent-foundry/pulls/196, outside that change. https://claude.ai/code/session_012CBsgjdkRDt2uVUmT7bj64
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
claudiomaradonna/agent-foundry#197
No description provided.