Skip to content

feat(skills): preview by prefix, planned evidence - #22

Merged
fredericrous merged 4 commits into
mainfrom
feat/approve-only-what-needs-judging
Oct 8, 2026
Merged

fredericrous merged 4 commits into
mainfrom
feat/approve-only-what-needs-judging

Conversation

@fredericrous

Copy link
Copy Markdown
Owner

Phase 3 of decisions docs/plans/2026-10-08-approve-only-what-needs-judging.md (ADR-0028; amont-agent v2.30.0 is released and installed).

Commits

  1. Pointer plan.
  2. The source catches up with the deployed skill. ~/.claude/skills/worktree-task/SKILL.md had gained F9 Teardown and the merge/release lines that the chezmoi source lacked, so the next chezmoi apply would have reverted them.
  3. worktree-task:
    • F7 asks with the register's question_prefix, verbatim.
    • New F7 step 0: the planned-evidence path, with the cases the hook still holds.
    • F4b's Delta may resume its reviewer with SendMessage.
    • plan-template.md gets an optional ## Preview section.
  4. Review fixes: the evidence: line comes first in the template. Without that, a copied plan never passes. F7 also names the held cases, cites ADR-0028, and allows a register chained last.

Verification

  • Implementation review: round 1 approve-with-changes (1 medium, 3 low), all fixed; delta approve.
  • The behaviour these steps describe was live-checked today on amont-agent 2.30.0. A register chained after curl bound on the first try, and one question built from question_prefix approved it. An approved plan with ## Preview / evidence: let a UI push through with the journal line evidence.
  • Pending until after merge: chezmoi apply, then chezmoi diff should be empty.

🤖 Generated with Claude Code

fredericrous and others added 4 commits October 8, 2026 16:13
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The deployed SKILL.md had gained F9 Teardown and the merge and release
lines, and the chezmoi source had not, so the next apply would have
reverted them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
F7 asks with the register's question_prefix and gains the ADR-0028
evidence path; a Delta review may resume its reviewer.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
From the implementation review: the template put prose before the
evidence line, so a copied plan never passed the gate. F7 names the
cases
the hook still holds, cites ADR-0028, and allows a chained register.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@fredericrous
fredericrous merged commit b2cf649 into main Oct 8, 2026
2 checks passed
@fredericrous
fredericrous deleted the feat/approve-only-what-needs-judging branch October 8, 2026 15:23
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