v1.46.1.0 docs: trim CLAUDE.md ~46% by relocating reference to docs#1719
Open
Nickmcm1 wants to merge 9 commits into
Open
v1.46.1.0 docs: trim CLAUDE.md ~46% by relocating reference to docs#1719Nickmcm1 wants to merge 9 commits into
Nickmcm1 wants to merge 9 commits into
Conversation
CLAUDE.md kept a ~20-line summary of the load-bearing invariants (branch-scoped versioning, diff-not-narrative, release-summary format) and points to the new canonical doc for the detail. Trims ~165 lines from the always-loaded CLAUDE.md without dropping an invariant.
Moved the unique PTY/sidebar deep-dives (WebSocket Sec-WebSocket-Protocol auth, ownsTerminalAgent teardown, cross-pane injection, /health token rule) into a new ARCHITECTURE.md 'Sidebar & PTY lifecycle' section. The tunnel/Unicode/prompt-injection blocks were already duplicated there, so CLAUDE.md now keeps only the claude-in-chrome tripwire, the setup-symlink invariant, and a 'before editing X read Y' pointer list. Every CI tripwire and the onnxruntime-node constraint stay push-loaded.
CLAUDE.md keeps the philosophy, commands, and the load-bearing error-handling utilities table inline; the fix-vs-leave essays and score history move to the new doc with a pointer.
Replaced the 70-line directory listing with a compact top-level map + 'use Glob or gbrain to discover sub-files' note. Kept the two single-source-of-truth pointers (commands.ts, snapshot.ts) referenced by the SKILL.md workflow section.
Detailed steps for re-targeting garrytan-agents fork PRs now live next to the Community PR triage section in CONTRIBUTING.md. CLAUDE.md keeps a trigger-preserving pointer (the 'check out <PR link>' gotcha + the empty-env-auth headline) so the agent still knows to look.
The per-skill clawhub publish commands now live in docs/OPENCLAW.md next to the native methodology skills they publish. CLAUDE.md keeps a one-line pointer noting the clawhub-publish-not-skill-publish gotcha.
The fetch/reset/rebuild and binary-copy steps now live with the global-install workflows in CONTRIBUTING.md. CLAUDE.md keeps a one-line pointer.
Trim CLAUDE.md from 882 to 500 lines by relocating reference detail to docs (CHANGELOG_STYLE.md, SLOP_SCAN.md, ARCHITECTURE.md, CONTRIBUTING.md, OPENCLAW.md). No invariants dropped.
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
Trims
CLAUDE.mdfrom 882 → 500 lines (~12.2K → ~6.6K tokens, -46%) by relocating reference-only detail into docs that load on demand, leaving a tight set of always-on rules plus pointers. No invariant dropped: every section header, every "before editing X read Y" tripwire, the CI-test names, and the gbrain managed-block markers stay put. 7 commits, one per relocated section, plus the version/CHANGELOG commit.Relocations:
docs/CHANGELOG_STYLE.md(the operational checklist stays in the/shipskill)ARCHITECTURE.md(new "Sidebar & PTY lifecycle" section; the tunnel / Unicode / prompt-injection blocks were already duplicated there, so they were pointer-linked, not re-added)docs/SLOP_SCAN.md(the error-handling utilities table stays inline)CONTRIBUTING.mddocs/OPENCLAW.mdTest Coverage
Documentation only — no application code paths changed, no tests added.
Pre-Landing Review
Clean. Verified all 25 section headers preserved, gbrain managed-block markers intact, every
[doc](#anchor)target resolves, and no unique content lost (the 3 deepest browser blocks were already in ARCHITECTURE.md, so they were linked rather than re-added).Eval Results
No prompt / template / resolver files changed — evals skipped.
Documentation
This change is documentation. CHANGELOG updated with a
1.46.1.0release-summary entry; all relocated content lives in the docs listed above with pointers from CLAUDE.md.Test plan
bun run test:windowsshard 1: 51/51 pass (Windows-runnable suite)bun test: bashbin/scripts can't be exec'd on Windowstest:windowsshard 2:browse/test/server-pty-lease-routes.test.tshits aC:\C:\doubled-drive-letter path bugorigin/mainis 6 markdown files; the failing test and thebrowse/src/server.tsit reads are byte-identical toorigin/main, so they fail identically on main. CI on Linux is the authoritative gate.🤖 Generated with Claude Code