Skip to content

Commit f3eabb2

Browse files
committed
docs(agents): a claim pre-check hit is a hint, not a claim
AGENTS.md rule 2 handed the dev `git ls-remote --heads origin | grep issue-<n>` as a one-command pre-check and said only what the command IS, never what a HIT means. The reading rule stood only in the fact layer (pm-dispatch references), which the dev definition never points at, so the layer holding the rule had no reader and the layer with the reader had no rule. The failure direction is the silent one: a stale head reads as "already claimed" and nothing goes red. One clause on the existing sentence states the reading. Paid in-file: the Skills section's restatement of the governed-surface queue mechanism retires, since Prime Directive #14 already carries it. Co-authored-by: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Gqi43smmqjJ5sUrhfoPeKu
1 parent a7bafc2 commit f3eabb2

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

AGENTS.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -460,7 +460,8 @@ Even inside your own worktree, operate defensively:
460460
straight to `main`.** Name the branch after the issue it fixes: `claude/issue-<n>-<slug>`.
461461
The issue number is what makes in-flight work *discoverable* — `git ls-remote --heads
462462
origin | grep issue-<n>` is a one-command pre-check, and the Duplicate Fix Guard
463-
workflow warns on fix PRs whose branch names no declared issue.
463+
workflow warns on fix PRs whose branch names no declared issue. A hit is a hint, not a
464+
claim — most heads carry no PR: ⛔ never skip a card on one, the `Claim:` comment decides.
464465

465466
**Off `main` is literal — a stacked series, each PR branched off the one below, is NOT a supported form.** No
466467
tooling represents it: squash landing destroys the ancestry link, so every descendant rewinds behind what landed and
@@ -796,8 +797,7 @@ working in its domain — browse the directory, never a hand-written list here:
796797
- `.claude/skills/` — repo-internal agent playbooks; every entry must carry
797798
`metadata.internal: true`.
798799

799-
**Both roots are governed surfaces** — human-merge only, or queued under **Prime Directive #14**'s pinned-approval
800-
path; no per-PR check holds it: the queue guard refuses an unpinned governed diff at queue time.
800+
**Both roots are governed surfaces** — human-merge only, or **Prime Directive #14**'s pinned-approval path.
801801

802802
---
803803

0 commit comments

Comments
 (0)