Skip to content

docs: true up the doc set for local-writes (compat/roadmap/skill-lint)#134

Closed
lance0 wants to merge 1 commit into
masterfrom
docs/local-writes-doc-sweep
Closed

docs: true up the doc set for local-writes (compat/roadmap/skill-lint)#134
lance0 wants to merge 1 commit into
masterfrom
docs/local-writes-doc-sweep

Conversation

@lance0

@lance0 lance0 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Follow-up to the ADR-0002 local-writes work: that feature changed the write story everywhere, so this sweep re-aligns the docs and fixes confirmed drift. Reviewed/verified before landing.

Changes

  • MCP write framing made consistent across README, docs/MCP.md, SECURITY.md, KNOWN_ISSUES.md, AGENTS.md, skills, comparison, CONFIG — local stdio --local-writes (profile token) vs shared HTTP/OIDC --allow-writes + nbox:write + [serve.vault].
  • ROADMAP — read-first (not read-only) framing, fixed the Writes section heading/anchor, added the live MCP E2E coverage note.
  • COMPATIBILITY — split 4.5 vs 4.6 so ETag/If-Match is attributed only to 4.6 (the old combined column was self-contradictory); allocate-write note expanded to all three reserve commands.
  • ARCHITECTURE — mutation.rs, write_audit.rs, the write flow.
  • BENCHMARKS — numbers marked historical (0.13.0-era), not current 0.14.
  • Skill lint — now covers the root SKILL.md (lint_skills.sh + the skills-lint workflow trigger).
  • release.yml — drop the stale keyring build comment (keyring removed in 0.8.0).

Verified

lint_skills.sh passes (root SKILL.md covered); COMPATIBILITY split is factually correct; MCP-write framing uniform with no local/shared conflation; no residual flat "read-only server" claims; ROADMAP anchors resolve; docs + workflows only, no Rust.

…lint drift

The ADR-0002 local stdio write mode re-introduced drift across the doc set; this
sweep makes the MCP write story consistent and corrects stale/broken docs.

- MCP writes: every surface (README, MCP.md, SECURITY, KNOWN_ISSUES, AGENTS,
  skills, comparison) now distinguishes local stdio `--local-writes` (profile
  token) from shared HTTP/OIDC `--allow-writes` + `nbox:write` + `[serve.vault]`.
- ROADMAP: drop stale read-only framing (read-first product/UX), fix the Writes
  section heading + anchor, note the live MCP E2E CI coverage.
- COMPATIBILITY: split 4.5 vs 4.6 so ETag/If-Match is attributed only to 4.6,
  and expand the allocate-write note to all three reserve commands.
- ARCHITECTURE: document mutation.rs, write_audit.rs, and the write flow.
- BENCHMARKS: mark the numbers historical (0.13.0-era), not current 0.14.
- Skill lint now covers the root SKILL.md (lint_skills.sh + skills-lint trigger).
- release.yml: drop the stale keyring build comment (keyring removed in 0.8.0).

Docs + workflows only — no Rust change.
@lance0

lance0 commented Jun 29, 2026

Copy link
Copy Markdown
Owner Author

Merged as ac222fd (squash) and shipped in v0.14.0 — the PR status was left open by a transient 502 during the merge, so closing it out. The changes are on master and in the release.

@lance0 lance0 closed this Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant