Skip to content

docs: extend CLAUDE.md with migrated agent notes#104

Merged
rubenhensen merged 3 commits into
mainfrom
chore/add-claude-md
Jul 15, 2026
Merged

docs: extend CLAUDE.md with migrated agent notes#104
rubenhensen merged 3 commits into
mainfrom
chore/add-claude-md

Conversation

@rubenhensen

Copy link
Copy Markdown
Contributor

Extends the existing CLAUDE.md with agent notes migrated from the private dobby memory repo (encryption4all/dobby, formerly repos/<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.

@dobby-coder dobby-coder Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs-only PR migrating agent notes into CLAUDE.md. Rule check (writing-rules, conventional-commit PR title) passes and the prose is clean, terse, and free of AI-slop tells. One accuracy nit on the migrated CI note — see inline. Not blocking.

Comment thread CLAUDE.md Outdated
@rubenhensen

Copy link
Copy Markdown
Contributor Author

/dobby fix all comments you made

dobby-coder Bot and others added 2 commits July 15, 2026 06:55
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.

@dobby-coder dobby-coder Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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:watchgenerate-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.

@rubenhensen
rubenhensen merged commit c4a26c3 into main Jul 15, 2026
5 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dobby-coder
dobby-coder Bot deleted the chore/add-claude-md branch July 16, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant