docs(claude-md): route CLAUDE.md by loading-discipline (ADR-0081) - #148
Merged
Conversation
Contributor
Skill Audit — PR #148Recommended skills (based on diff analysis)
Generated by skill-audit |
Contributor
|
Preview deployment ready |
Contributor
|
Preview deployment ready |
Contributor
Browser Automation Test Results
Screenshot Baselines — 20/20 ✅ Complete
📸 Screenshots (20) ⏭ 20 no actual⏭ no actual
|
| Baseline |
|---|
 |
⏭ no actual sidebar-expanded-desktop
| Baseline |
|---|
 |
⏭ no actual bio-tab-desktop
| Baseline |
|---|
 |
⏭ no actual bio-chat-open-desktop
| Baseline |
|---|
 |
⏭ no actual bio-form-empty-desktop
| Baseline |
|---|
 |
⏭ no actual bio-file-browser-desktop
| Baseline |
|---|
 |
⏭ no actual jobs-tab-desktop
| Baseline |
|---|
 |
⏭ no actual jobs-sidebar-chat-desktop
| Baseline |
|---|
 |
⏭ no actual outputs-sidebar-desktop
| Baseline |
|---|
 |
⏭ no actual outputs-tab-desktop
| Baseline |
|---|
 |
⏭ no actual research-tab-desktop
| Baseline |
|---|
 |
⏭ no actual pipelines-tab-desktop
| Baseline |
|---|
 |
⏭ no actual toolbox-tab-desktop
| Baseline |
|---|
 |
⏭ no actual jobs-chat-open-desktop
| Baseline |
|---|
 |
⏭ no actual sidebar-collapsed-desktop
| Baseline |
|---|
 |
⏭ no actual chat-component-desktop
| Baseline |
|---|
 |
⏭ no actual theme-light-desktop
| Baseline |
|---|
 |
⏭ no actual settings-modal-open-desktop
| Baseline |
|---|
 |
⏭ no actual chat-input-focused-desktop
| Baseline |
|---|
 |
⏭ no actual chat-help-badge-desktop
| Baseline |
|---|
 |
Apply the ADR-0081 loading-discipline to cv-builder's 366-line CLAUDE.md. Always-loaded footprint: ~4151 -> ~941 tokens (366 -> 72 lines), ~77% drop. Relocate-only (0 deletes, 0 @import); all distinctive strings verified present. Routing: - Layer 1: packages/agent-core/CLAUDE.md (nested) <- add-an-agent procedure + models note; loads when editing that package. - Layer 1: .claude/rules/mf-and-deploy.md (paths: browser-app + vercel.json) <- Module Federation surface, shared singletons, local MF dev, Vercel deploy + cache-header rules. - Layer 2: docs/claude-reference.md <- prerequisites, ESLint custom-rule catalog, Claude Code hook wiring, security commands, env setup, data dirs, architecture detail (multi-agent, monorepo tree, data models, package imports), agent system + .agents/ catalog, key technologies. - Layer 0 (stays): identity + server-side-only invariant, pnpm, command quick-ref, security/secrets invariant, testing philosophy, privacy, deployment-safety invariant. Fix: the broken root pointer 'see ARCHITECTURE.md' (CLAUDE.md:15) now targets docs/ARCHITECTURE.md (the real path). Verified docs/ARCHITECTURE.md is NOT a superset of the CLAUDE.md architecture block (missing Skills Gap Analyzer, Interview Coach, container-presenter, ResearchEntry, @frame/eslint-plugin, three-tier storage) -> relocated those to Layer 2 rather than deleting. L2 lives in repo-native docs/ (domain-knowledge/ is a gitignored symlink farm). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ojfbot
force-pushed
the
docs/claude-md-routing
branch
from
June 4, 2026 22:51
927fb5f to
70d7bf5
Compare
Contributor
|
Preview deployment ready |
Contributor
|
Preview deployment ready |
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.
What
Applies the ADR-0081 loading-discipline to cv-builder's 366-line root
CLAUDE.md.Always-loaded footprint (M1): ~4151 → ~941 tokens (366 → 72 lines), ~77% drop. Relocate-only — 0 deletes, 0
@import.Routing
packages/agent-core/CLAUDE.md(nested, loads when editing that package): add-an-agent procedure + models note..claude/rules/mf-and-deploy.md(paths:browser-app +vercel.json): Module Federation surface, shared singletons, local MF dev, Vercel deploy + cache-header rules.docs/claude-reference.md: prerequisites, ESLint custom-rule catalog, Claude Code hook wiring, security commands, env setup, data dirs, architecture detail (multi-agent, monorepo tree, data models, package imports), agent system +.agents/catalog, key technologies.Fix: broken pointer
CLAUDE.md:15pointed at a rootARCHITECTURE.mdthat doesn't exist — corrected todocs/ARCHITECTURE.md(the real path). Verifieddocs/ARCHITECTURE.mdis not a superset of the CLAUDE.md architecture block (missing Skills Gap Analyzer, Interview Coach, container-presenter, ResearchEntry, @frame/eslint-plugin, three-tier storage) → those were relocated to Layer 2, not deleted.L2 lives in repo-native
docs/(domain-knowledge/is a gitignored symlink farm into core).How to test
node core/scripts/claude-md/footprint.mjs cv-builder→ always ~941 / conditional ~1061;@import: 0.Skills Gap Analyzer,container-presenter,@frame/eslint-plugin,remoteEntry.js,9b84f80, …) verified present across the new fileset.packages/agent-core/**loads the nested guidance; editingpackages/browser-app/**orvercel.jsonloads the MF/deploy rule.🤖 Generated with Claude Code