Repository navigation
Move the module tour out of CLAUDE.md to free up context - #393
Merged
Merged
Conversation
CLAUDE.md had grown to ~168 KB, every byte of it loaded into every session. The per-module narrative — what each module holds, the decisions inside it, the traps — moves verbatim to docs/codebase-tour.md, one `##` section per module, and CLAUDE.md keeps a one-line-per-module map with a pointer to read the module's section before changing it. The long ADR paragraphs under Binding rules become one bullet pointing at docs/adr/README.md plus short reminders of the rules most often hit; the paragraphs themselves are kept as the tour's appendix. Orientation, Commands, the emulator pitfalls, the non-ADR binding rules and Agent skills are unchanged. Also fixes a wrapped line in the offline section that began with `#245` and so rendered as a heading. Co-Authored-By: Claude Opus 5.5 <[email protected]>
The conflict was resolved by redoing the move rather than merging text: docs/codebase-tour.md is rebuilt from main's CLAUDE.md in whole line ranges (adding superplayer-effects and the android/ paths), and CLAUDE.md keeps the slim shape with the current orientation, commands and rules. Two comments that cited the moved prose now name the tour. Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01MJAhQmFgdRRAKUaMHibQLe
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.
CLAUDE.md had grown to ~168 KB (~42k tokens), all of it loaded into every agent session. This cuts it to ~18 KB without deleting anything.
What moved
benchmark/) moves verbatim to the newdocs/codebase-tour.md, one##section per module, so an agent cangrep -n '^## 'and read only the slice it needs.What CLAUDE.md now carries
PlaybackPolicy.docs/adr/README.md, with short reminders of the rules most often hit.Also
docs/README.mdgains a row for the tour.#245and rendered as a heading; fixed in the move.Not done
PlaybackHarness.kt:1607's comment points at a CLAUDE.md warning that now lives in the tour's core section. Comment-only, left as is.checklocally; CI on this PR is the gate.🤖 Generated with Claude Code
Updated 2026-10-14: the branch is merged with
mainand the split redone on today'sCLAUDE.md(the iOS port,superplayer-effectsand theandroid/layout had landed since).docs/codebase-tour.mdis rebuilt frommain's text in whole line ranges, so no prose was retyped; the stale "every other module is a placeholder" paragraph is dropped, as before. Spotless and the license-header check pass locally.