Uh oh!
There was an error while loading. Please reload this page.
docs(agents): scope the attribution-footer clause to PR bodies, and record the comment channel's measured append - #10430
Draft
qq9340100 wants to merge 1 commit into
Draft
Conversation
…cord the comment channel's measured append The clause read as surface-independent and ended "Comments are a different path and unaffected". Re-measured 2026-08-20 through two write paths (MCP add_issue_comment and direct REST POST) x both footer forms, on this card's own thread: issue comments are NOT unaffected, and the mechanism is not the rewrite the finding assumed. The platform APPENDS a bare footer to every issue comment. A comment already ending in the bare form absorbs it idempotently (one footer). A comment ending in the session-URL form keeps that form verbatim and gets a bare one appended underneath -- two footers. Reading only the tail is what makes it look like a session->bare downgrade; the original sample carries both. PR bodies and issue bodies get no append, so the session-URL rule stands there and is now stated as PR-body-scoped. Net zero lines: AGENTS.md stays at its 961-line ceiling. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019T1sSZbQTnLhrK9HhNdNiB
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes#10207
Ruled scope was C then A: re-measure through a second write path before writing
documentation, then scope the clause to what the measurements support.
C — the re-measurement (2026-08-20)
Four probes on this card's own thread, two write paths x two footer forms, each read back
byte-for-byte over REST. The card's original sample was re-read (read-only, never edited).
5360236506add_issue_comment5360236672add_issue_comment536023857053602387785356851695create_pull_requestissue_writeThe premise fork
The card reports the comment channel as a rewrite running "in the opposite direction"
from
create_pull_request— session-URL downgraded to bare. The measurements do notsupport that, and the card's own sample disproves it: comment
5356851695carries twofooters, the session-URL one written by its author intact, with a bare one underneath.
The actual mechanism is an append. Every issue comment gets a bare footer block
appended on write. When the body already ends in the bare form that append is idempotent
(the existing block is consumed and re-emitted, net one footer plus a blank line). When it
ends in the session-URL form the platform does not recognise it as the footer, so it
survives verbatim and a bare one lands under it — two footers. Reading only the tail of the
comment is what makes an append look like a downgrade.
Byte-level confirmation for M1 (written vs read back), the appended block being lines 7-10:
Two further findings the card left open: the behaviour is per-surface, not per-tool —
MCP and direct REST are indistinguishable — and PR bodies and issue bodies get no
append, which is why the session-URL rule is correct there and only there.
A — the clause
AGENTS.mdnow states the rule per surface, keeps both literal footer forms, and recordsthe comment-channel behaviour with its measurement date beside the PR-body rule. The old
closing sentence "Comments are a different path and unaffected" is gone; the practical
guidance for comments is bare (or no footer), and the anomaly-reading guard is inverted to
match the mechanism: a tail bare footer on a comment is the platform's, not your form
downgraded.
AGENTS.mdis at its shrink-only ceiling, so this is paid by dedup inside the clause's ownparagraph: 10 insertions, 10 deletions, net zero — the file stays at 961 lines
(
check-skill-line-ratchet: AGENTS.md is 961 lines (ceiling 961; headroom 0)).Verification
Gate union re-derived on the actual diff with
node scripts/pm/dispatch-gates.mjs(onepath,
AGENTS.md), then re-run at the final commit50e76aa4e— all green:The re-derivation named four gates the dispatch clue did not (
check:docs-audit-scope,check:pm-skill-id-lint,check:pm-skill-ratchet,check:required-contexts); all fourare included above.
Instruction-surface change only — no published package is affected, so
skip-changeset.Governed surface: draft, human merge, never ready/queued/auto-merge.
Generated by Claude Code