Skip to content

docs: split claude.md into docs/ and fix stale claims - #83

Merged
KOliver94 merged 8 commits into
mainfrom
agent-files
Nov 13, 2026
Merged

KOliver94 merged 8 commits into
mainfrom
agent-files

Conversation

@KOliver94

@KOliver94 KOliver94 commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

What

CLAUDE.md had 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 in CLAUDE.md only what applies to every task and moves the reference into docs/, one file per topic.

  • CLAUDE.md now has four parts: what the repo is plus an index of docs/ 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.
  • Folder CLAUDE.md files, which Claude Code loads only when working in that folder: components/ui/ (registry output and the LOCAL MODIFICATIONS rule), components/ (AvatarContext / PortalShell), and a new workstation-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

  • Commit subjects: dropped "≤72 chars". config-conventional allows 100, and the squash commit uses the PR title.
  • Actions are SHA-pinned with the version in a comment, not "pinned to a patch version".
  • The main ruleset requires five checks: ci.yml's three plus Checks / Workflows and Checks / Dependencies, as well as code owner review. checks.yml is now in the workflows table.
  • The Renovate paragraph described the old in-repo config (weekend schedule, Monday automerge window, platformAutomerge off). It now points to BSStudio/renovate-config (weekly + approve-merges) and explains what CODEOWNERS leaves unowned.
  • structure.md claimed docker-publish.yml uses the setup composite; only ci.yml and authentik-contract.yml do.
  • README: the intro names every sync target, and the CI line names every required check.

Also

  • New entry in decisions.md: Better Auth runs without a database, and why. This was only recorded in local agent memory before. auth.md notes that an update.before hook returning false would not protect /update-user.
  • skills-lock.json and its ignore lines are removed. The skills were never in .claude/skills, so they never loaded.
  • README.md and workstation-agent/README.md are 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 lint and commitlint pass, and the ruleset, merge settings and CODEOWNERS claims were checked against the live repository settings.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@codecov

codecov Bot commented Nov 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (3b61be9) to head (6c8ea2a).
✅ All tests successful. No failed tests found.

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.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KOliver94
KOliver94 merged commit 88e1824 into main Nov 13, 2026
12 checks passed
@KOliver94
KOliver94 deleted the agent-files branch November 13, 2026 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant