docs: sync AGENTS.md coding-output rules from distillery proof-2 #483

Merged

igor merged 1 commit from agent/481-docs-sync-agents-md-coding-output-rules-from into master

2026-08-09 06:00:16 +00:00

igor commented

2026-08-09 05:10:07 +00:00

Collaborator

What this PR does

docs: sync AGENTS.md coding-output rules from distillery proof-2
Replace the "Keep changes focused" bullet with the minimal-touch wording
Replace the "Comment discipline" bullet with the two-variety-comments wording

Test plan

make test passes (check-sync's OUTCOME sentinel contract is untouched)
git diff confirmed to show exactly two hunks in exactly one file (AGENTS.md)

Closes #481

## What this PR does - [x] docs: sync AGENTS.md coding-output rules from distillery proof-2 - [x] Replace the "Keep changes focused" bullet with the minimal-touch wording - [x] Replace the "Comment discipline" bullet with the two-variety-comments wording ## Test plan - [x] `make test` passes (check-sync's OUTCOME sentinel contract is untouched) - [x] `git diff` confirmed to show exactly two hunks in exactly one file (AGENTS.md) Closes #481

igor added 1 commit

2026-08-09 05:10:07 +00:00

docs: sync AGENTS.md coding-output rules from distillery proof-2

Lint / check-sync (pull_request) Successful in 6s

Details

Lint / check-sync (push) Successful in 6s

Details

767b13463d

igor commented

2026-08-09 05:11:21 +00:00

Author

Collaborator

🤖 Review — `APPROVE` (automated)

CI for 767b1346: success

Docs-only sync of two bullets in AGENTS.md to the distillery proof-2 wording. Verified verbatim against the linked issue; approving.

Checks performed

Both replacement blocks match issue #481's specified text character-for-character (indentation, -- dashes, emphasis markers included).
Exactly two hunks, exactly one file (AGENTS.md) — matches the stated scope and the "Out of scope" constraint. No sentinel/OUTCOME text touched; nothing else in the diff.
Hunk line accounting is consistent (-58,7 +58,11 and -78,13 +82,19), so nothing was silently dropped adjacent to the edits.
Description checklist is honest: all four checked items correspond to what's actually in the diff (or are verification claims consistent with CI success).
No code, no CI config, no security surface.

Non-blocking notes

AGENTS.md:61-65 (new bullet) and the immediately following Size the diff to what the task honestly requires. bullet now overlap substantially — both prohibit drive-by refactors and unrelated cleanup. That's the upstream wording, so it's correct to sync as-is, but if the two bullets diverge later in distillery it may be worth collapsing them there rather than in igor.
The old "Shorter is better" clause is gone from the comment-discipline bullet. Intentional per the issue's replacement text, just flagging it since it wasn't called out in the description.

No test changes needed for a prose-only doc sync; CI is green.

_{Independent review by the harness on claude-opus-5 (effort: high). The human reviewer is requested once Igor has reviewed; a human still merges.}

### 🤖 Review — `APPROVE` _(automated)_ CI for `767b1346`: **success** Docs-only sync of two bullets in `AGENTS.md` to the distillery proof-2 wording. Verified verbatim against the linked issue; approving. **Checks performed** - Both replacement blocks match issue #481's specified text character-for-character (indentation, `--` dashes, emphasis markers included). - Exactly two hunks, exactly one file (`AGENTS.md`) — matches the stated scope and the "Out of scope" constraint. No sentinel/OUTCOME text touched; nothing else in the diff. - Hunk line accounting is consistent (`-58,7 +58,11` and `-78,13 +82,19`), so nothing was silently dropped adjacent to the edits. - Description checklist is honest: all four checked items correspond to what's actually in the diff (or are verification claims consistent with CI `success`). - No code, no CI config, no security surface. **Non-blocking notes** - `AGENTS.md:61-65` (new bullet) and the immediately following `Size the diff to what the task honestly requires.` bullet now overlap substantially — both prohibit drive-by refactors and unrelated cleanup. That's the upstream wording, so it's correct to sync as-is, but if the two bullets diverge later in distillery it may be worth collapsing them there rather than in igor. - The old "Shorter is better" clause is gone from the comment-discipline bullet. Intentional per the issue's replacement text, just flagging it since it wasn't called out in the description. No test changes needed for a prose-only doc sync; CI is green. --- <sub>Independent review by the harness on `claude-opus-5` (effort: high). The human reviewer is requested once Igor has reviewed; a human still merges.</sub>

igor added spent time

2026-08-09 05:11:22 +00:00

21 seconds

igor merged commit dc455f185a into master

2026-08-09 06:00:16 +00:00

igor deleted branch agent/481-docs-sync-agents-md-coding-output-rules-from

2026-08-09 06:00:16 +00:00

igor referenced this pull request from a commit

2026-08-09 06:00:16 +00:00

Merge pull request 'docs: sync AGENTS.md coding-output rules from distillery proof-2' (#483) from agent/481-docs-sync-agents-md-coding-output-rules-from into master

~~igor referenced this pull request 2026-08-09 06:27:56 +00:00~~

chore: retarget check-sync to the sourced worker contract; delete unconsumed in-repo prompt copies #487