docs: trim derivable content from CLAUDE.md - #21
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Reduces the size of CLAUDE.md by removing command/script details that can be re-derived from the repository, while keeping non-derivable operational facts (CI gate order and the multi-pass typecheck detail) to minimize recurring context cost in agent sessions.
Changes:
- Removed the “Commands” shell snippet and other derivable operational details (script list, common vitest invocations, Node/pnpm pin note).
- Kept and reformulated the two non-derivable facts: CI gate order (including
typecheck’s three passes) and the “run single test from insidepackages/entity” note. - Updated a staleness reference from
addtocomputedin the type-level tests bullet.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
From a
/doctorpass.CLAUDE.mdloads into every session, so anything asession could reconstruct with a couple of tool calls is paid for on every
turn and gives nothing back.
6,867 → 6,457 chars (~1,716 → ~1,614 est. tokens).
Cut — derivable from the repo
The
pnpm build|test|typecheck|test:types|lint|format|kniptable (those arepackage.jsonscripts), the standardpnpm vitest run <file>/-tinvocations, and the Node/pnpm pin line (
.node-versionandpackageManageralready say it).
Kept, because it is not derivable
The two facts that block came with which nothing in the repo states:
btravstack/config, not here,and
typecheckbeing three passes (maintsc,.test-d.ts, and theconsumer declaration-emit pass from fix!: let a consumer emitting declarations compile against the seal #13) is not visible from the script name
packages/entity, not the rootNot cut
The Architecture section stays whole despite reading like a repo tour. Its
value is rationale rather than inventory — why only two zod slots and never
the full
ZodType, why the union dispatches instead of trying each branch,why
computedre-derives — and none of that is recoverable from the code. TheBinding conventions stay for the same reason.
Also
A staleness fix, not a trim: the type-level-tests bullet still said
add'scontextual typing.
addbecamecomputedin #10.Gate
format --checkclean. No source touched.