docs: weekly review 2026-06-25 - #135
Merged
Merged
Conversation
MCP_STANDARDS.md quoted a section title "Optimistic Concurrency Control (Inform Half)" that doesn't exist verbatim in SPECIFICATION.md — the actual structure is a level-3 heading "Optimistic Concurrency Control" containing a separate level-4 subsection "Inform half". Updated the cross-reference to match.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Owner
Author
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates maintainer-facing documentation to fix a stale cross-reference in the MCP standards doc so it matches the actual heading structure in docs/dev/SPECIFICATION.md.
Changes:
- Updated the SPECIFICATION.md cross-reference from a non-existent quoted heading to the correct
"Optimistic Concurrency Control" → "Inform half"structure.
|
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
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.



Summary
Weekly documentation review. Inventoried user-facing docs (README.md, docs/user/NPM.md, FAQ/InstallGuide/FeatureGrid website copy) and maintainer-facing docs (CLAUDE.md, docs/dev/SPECIFICATION.md, SECURITY.md, MCP_STANDARDS.md, CODE_STYLE.md, BEAR_DATABASE_SCHEMA.md, evals/README.md, CHANGELOG.md, CI/release workflows) and cross-checked every specific claim — env vars, tool counts, file/line references, Z_OPT bump behavior per URL action, OCC inform/enforce mechanics, install commands, license — against the actual source. The docs in this repo are unusually well-maintained; the review surfaced exactly one stale cross-reference, fixed below.
Changes
"Optimistic Concurrency Control (Inform Half)", that doesn't exist verbatim. The actual structure is a level-3 heading### Optimistic Concurrency Controlcontaining a separate level-4 subsection#### Inform half. Updated the quoted reference to"Optimistic Concurrency Control" → "Inform half"so it matches the real heading structure.No Issues Found
These checklist areas were verified clean — listed to show the check was performed, not skipped:
database.ts:59forPRAGMA busy_timeout = 3000) and BEAR_DATABASE_SCHEMA.md's per-URL-actionZ_OPTbump claims (/add-text,/create,/add-tags,/add-file,/archive) all verified againstsrc/infra/database.ts,src/tools/note-tools.ts, andsrc/tools/tag-tools.ts.UI_DEBUG_TOGGLE,UI_ENABLE_NEW_NOTE_CONVENTION,UI_ENABLE_CONTENT_REPLACEMENTenv vars and their defaults match across README.md, docs/user/NPM.md, manifest.jsonuser_config, andsrc/config.ts.checkRevisionGate,awaitRevisionIncrement, timeout sentinels) in SPECIFICATION.md matchsrc/tools/responses.tsandsrc/operations/notes.tsexactly, including the documented+2first-edit-after-creation jump. Tool gating counts (4 read-only, 1 off-only, 8 write) matchtests/system/registration-gate.test.ts.website/src/data/site.tsare consistent; Node engine requirement (^24.13.0) matches package.json/manifest.json across all docs and website copy.src/infra/fts-index.ts,src/infra/database.ts, andsrc/infra/bear-schema.ts.🤖 Generated with Claude Code
Generated by Claude Code