Repository navigation
docs: cut restated rules and inferable reasons from merged instructions - #461
Merged
Merged
Conversation
davidvornholt
force-pushed
the
docs/cut-merged-redundancy
branch
from
October 22, 2026 18:18
de4d926 to
a102999
Compare
davidvornholt
marked this pull request as ready for review
October 22, 2026 18:18
davidvornholt
marked this pull request as draft
October 22, 2026 18:23
davidvornholt
marked this pull request as ready for review
October 22, 2026 18:23
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 free
to 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.
Before. Merged skills and instructions still contained text an agent loading
AGENTS.mddoesn't need. The issues skill pointed to theAGENTS.mdchange policy, and the prs skill restated its "state each fact once" rule. Several reasons explained what a capable model infers anyway, and the screenshots skill carried a history anecdote. Every agent pays for those words on every read.Now. The rules are unchanged, and only the restatements, inferable reasons, and history are gone.
Implementation
Cuts, each separable:
AGENTS.mddescribes". Every agent already loadsAGENTS.md.AGENTS.md. Also "because running it on every intermediate push was costly" and the sentence about the decider relying on the check lines. The verification line keeps its substance: what ran, how much passed, partial runs and failures.Kept, because they're cheap to state and costly to rediscover or get wrong:
gh pr readyandgh pr ready --undocommands;frontend-designis unchanged. It's Anthropic's skill, vendored with its license.Words:
prs477 → 445,screenshots-in-prs241 → 212,issues177 → 166,AGENTS.md522 → 512,AGENTS.local.md269 → 261,docs/operations.md353 → 345.Verification
Completed checks
bun run checkpassed: structure, source text, Dependabot, and GitHub settings checks, then 7 of 7 Turbo tasks.git diff --check: clean.