This repository was archived by the owner on Aug 7, 2026. It is now read-only.
docs: extend CLAUDE.md with migrated agent notes - #122
Merged
Conversation
![dobby-coder[bot]](https://avatars.githubusercontent.com/in/3365457?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Rule check + review for the CLAUDE.md agent-notes migration.
Title docs: is conventional-commit compliant, and the prose reads as terse technical notes (no writing-rule slop). Two non-blocking style/nit items below — the migration itself is accurate and fine to merge.
Contributor
Author
|
/dobby fix all comments you made |
- Demote migrated subsections to ### so they nest under the ## Agent notes wrapper, matching the rest of the file's hierarchy. - Correct the size-limit note: the warning is on the taskpane.js (~1.03 MiB) and yivi-dialog.js (~1 MiB) entrypoint bundles that embed the WASM, not a standalone .wasm asset (none is emitted). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Rule check + review — cycle 2, HEAD a078792.
Both non-blocking nits from cycle 1 are confirmed fixed in a078792:
- Migrated subsections demoted from
##to###so they nest under## Agent notes(matches the file hierarchy). - Size-limit wording corrected — attributed to the
taskpane.js(~1.03 MiB) /yivi-dialog.js(~1 MiB) entrypoint bundles that embed the WASM; no standalone.wasmis emitted todist/.
Rule sweep (conventional-commit title, writing/design slop, docs hygiene) is clean, and Review Dobby 2's re-review of the delta returned no findings. Docs-only change to CLAUDE.md, accurate and merge-ready. Approving.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
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.