Repository navigation
docs: split claude.md into docs/ and fix stale claims - #83
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #83 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 90 90
Lines 1832 1832
Branches 457 457
=========================================
Hits 1832 1832 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
What
CLAUDE.mdhad grown to ~1,600 lines, most of it reference material an agent reads on every task whether it needs it or not. This PR keeps inCLAUDE.mdonly what applies to every task and moves the reference intodocs/, one file per topic.CLAUDE.mdnow has four parts: what the repo is plus an index ofdocs/with a line on when to read each file; what to run before calling something done (and what CI additionally requires); commits and branches; rules and conventions.docs/(10 files: structure, data model, auth, API, integrations, observability, releases, testing, decisions, dev data) holds the former reference sections verbatim. Headings moved up one level and cross-references now point at other files and their heading anchors.CLAUDE.mdfiles, which Claude Code loads only when working in that folder:components/ui/(registry output and theLOCAL MODIFICATIONSrule),components/(AvatarContext/PortalShell), and a newworkstation-agent/one: the scripts must stay Windows PowerShell 5.1, and none of the root checks cover.ps1, so it gives a 5.1 parse check.Stale claims fixed
config-conventionalallows 100, and the squash commit uses the PR title.mainruleset requires five checks:ci.yml's three plusChecks / WorkflowsandChecks / Dependencies, as well as code owner review.checks.ymlis now in the workflows table.platformAutomergeoff). It now points to BSStudio/renovate-config (weekly+approve-merges) and explains whatCODEOWNERSleaves unowned.structure.mdclaimeddocker-publish.ymluses the setup composite; onlyci.ymlandauthentik-contract.ymldo.Also
decisions.md: Better Auth runs without a database, and why. This was only recorded in local agent memory before.auth.mdnotes that anupdate.beforehook returningfalsewould not protect/update-user.skills-lock.jsonand its ignore lines are removed. The skills were never in.claude/skills, so they never loaded.README.mdandworkstation-agent/README.mdare unwrapped to one line per paragraph, with no wording changes.Reviewing
No code changes. Each commit is one kind of change; the move commit is easiest to read with
git diff --color-moved=dimmed-zebra. Links and heading anchors resolve,pnpm lintand commitlint pass, and the ruleset, merge settings andCODEOWNERSclaims were checked against the live repository settings.