Add CLAUDE.md and correct the stale ADR references - #1
Merged
Merged
Conversation
The source comments cited ADR numbers from before the documents were carried over, so every reference pointed at a file that does not exist: 0008 for the self-repair loop and guard (0005), 0007 for the two-hop chain (0004), 0003 for the chaining label writes (0002). `0007` was also doing double duty for the reply-not-resolve dedup marker, which no ADR argues. Those two citations are dropped rather than pointed somewhere wrong — the surrounding comments already carry the reasoning, and the decision can get its own ADR later. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
casperpanduro
force-pushed
the
docs/claude-md-and-adr-refs
branch
from
August 11, 2026 06:00
36a4ccd to
61f9dce
Compare
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
1 participant
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.
Summary
CLAUDE.md: commands, the one-binary/one-command-per-step design, the pure-core vs IO-shell split, the lifecycle table's two invariants, the self-repair loop and its guard, and the repo's conventions.src/shared/— they were numbered from beforedocs/adr/was carried over, so each pointed at a file that does not exist:0008→0005,0007→0004,0003→0002.0007citations on the reply-not-resolve dedup marker. That number was doing double duty for two unrelated decisions, and no ADR argues this one; the surrounding comments already carry the reasoning.Test plan
npm run typechecknpm test— 92 passeddocs/adr/NNNNreference insrc/resolves to a file indocs/adr/🤖 Generated with Claude Code