Wiki: split the monolith, and cut CLAUDE.md to steering and rules - #30
Merged
Conversation
docs/wiki/index.md was 133,893 chars in one file and @-ref'd as auto-loaded, so all of it entered every session — 101,856 of that was `## Domain detail` alone. It is now a 9,427-char router with the twelve domains in their own lazily-read files. CLAUDE.md had the same shape: 23,353 chars, 20,222 of it one lookup table, now at docs/wiki/feature-map.md. Both drifted because docs/wiki/CLAUDE.md was still the NestJS/pnpm scaffold and had never been filled in, so every refresh re-derived domains and build commands from placeholder examples. Also fixes what the refreshed scan surfaced. The modeling skill cited none of the four canonical docs its design requires, so a rule change had nothing to update against — and the design's prescribed fix was itself broken, since `npx skills add` copies only skills/, making every repo path inside it a dead link on a user's machine. Eight such citations removed, with a check that now fails the build on any repo path in skills/. /api/flow also returned a different shape for models with no flows/ dir, breaking any consumer reading entityModel.
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.
Single squashed commit. Supersedes #29 — identical tree, same content.
docs/wiki/index.mdwas 133,893 chars in a single file and@-ref'd as auto-loaded, so all of it entered every session;## Domain detailalone was 101,856 of that. It is now a 9,427-char router, with the twelve domains in their own files that load only when read.CLAUDE.mdhad the same shape — 23,353 chars, 20,222 of it one lookup table, now atdocs/wiki/feature-map.md.Both drifted for the same reason:
docs/wiki/CLAUDE.mdwas still the NestJS/pnpm scaffold and had never been filled in, so every refresh re-derived domains and build commands from placeholder examples.Two fixes from the refreshed scan were worth doing rather than filing. The modeling skill restated linter rules, key derivation, and the
ignatius.ymlschema while citing none of them — and the mechanism its design prescribed was itself broken, becausenpx skills addcopies onlyskills/, so every repo path written inside it is a dead link on a user's machine. Eight such citations are gone, four of which predated this branch, and a check now fails the build on any repo path underskills/. Separately,/api/flowreturned a different shape when a model had noflows/directory, so a consumer readingentityModelbroke only on those models.CLAUDE.mdalso drops its Testing and Frontend sections:bun testfinds 0 files and exits 1 here, anddocs/wiki/index.mdalready documented the real harness correctly.