docs: fold session-memory lessons into committed docs - #262
Merged
Conversation
Extracted from persistent agent memory (ICM + auto-memory) so the committed docs are self-sufficient: - README: Node 22 requirement for local dev (better-sqlite3 prebuilds) - pitfalls: getAlbumList2 inline-SQL exception to #130 hoisting rule - pitfalls: merge-worker must stay dependency-light - pitfalls: never proxy the Navidrome native API to peers - pitfalls (Process): concurrent agent sessions need git worktrees; live hub is not a test bed (no DB resets, verify uptime + logs) - frontend-testing: duplicate img alt-text collision, scope with within(dialog) - CLAUDE.md: Write-tool control-character gotcha (Claude Code only) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SwwPcrohddTUN7kmQVedQC
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Closes #261.
Audited all persistent agent memories (ICM topics + auto-memory) for lessons that existed only in session memory, and folded the durable ones into committed docs. Most memory content was already covered (art-cache #145 docs, login-loop rule in authentication.md, merge-worker dual-path row) or is ephemeral/historical — this PR adds only the gaps:
README.mdbetter-sqlite3prebuild (NODE_MODULE_VERSIONmismatch)docs/pitfalls.mdgetAlbumList2inline-SQL is a deliberate exception to the #130 hoisting ruledocs/pitfalls.mddocs/pitfalls.mddocs/pitfalls.mdgit worktree; live hub is not a test beddocs/frontend-testing.mdwithin(dialog)CLAUDE.mdDocs-only change — no runtime surface.
🤖 Generated with Claude Code
https://claude.ai/code/session_01SwwPcrohddTUN7kmQVedQC