docs: extend CLAUDE.md with migrated agent notes#104
Conversation
|
/dobby fix all comments you made |
The migrated 'Repo layout' note described integration.yml as running 'typecheck, build, test on Node 24', which under-states the actual matrix (Node 22 and 24, plus Bun and Deno lanes) and every lane's smoke step. It also contradicted the accurate description already present under 'Releases and CI'. Align the two. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The pre* hooks run three generators (wasm-base64, yivi-css, version), not two; version.ts is itself a gitignored build-time source.
There was a problem hiding this comment.
Rules Dobby 2 — sign-off ✅
Docs-only PR migrating agent notes into CLAUDE.md. Rule check (writing-rules prose patterns, conventional-commit PR title, docs-accuracy) passes and the prose is terse and slop-free.
One docs-accuracy nit surfaced by the prior review: the build-pipeline note said the pre* hooks run "both generators" (two), but package.json wires three into prebuild/pretypecheck/pretest/pretest:watch — generate-wasm-base64.mjs, generate-yivi-css.mjs, and generate-version.mjs. The third emits src/util/version.ts, itself a gitignored build-time source (confirmed via git check-ignore) imported through util/client-version.ts, so it belongs in this note.
Per your /dobby fix all comments request I fixed both occurrences (lines 94 and 103, "both" → "all three", adding the version generator) directly in commit c457be4. No blocking issues remain — approving.
|
🎉 This PR is included in version 2.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Extends the existing
CLAUDE.mdwith agent notes migrated from the private dobby memory repo (encryption4all/dobby, formerlyrepos/<this-repo>/), verified against the current state of the code before migration.Going forward this file is the single home for durable repo knowledge: dobby updates it in the same PR whenever it learns something worth keeping.