docs: record kernel id-immutability contract (ids/refs never reused) - #494
Conversation
📦 Built Package ArtifactBranch: Option A — Install from npm PR tag (recommended)npm install -g billion-context@pr-494Each push to this PR publishes a new version under the Option B — Download artifact
tar xzf billion-context-pr494.tgz
npm install -g packageThis comment is automatically updated on each push. |
[bot] 🏷 Reviewed — all claims check out. Verdict below. What I verified1. Kernel cursor mechanism (the load-bearing claim) — confirmed against installed acp-kernel 0.0.47 ( 2. 3. No-bump guard vs 0.0.48/0.0.49 — current pin is 0.0.47 (
AssessmentDocs-only (+22 lines, AGENTS.md §2), CI artifact already posted, typecheck/test/build untouched by markdown. Layering judgment: this records a host-side duty (don't shrink the map in ways that lower the kernel's derived cursor) plus a dependency constraint (skip 0.0.48/0.0.49) — both correctly attributed, neither overclaiming what the kernel guarantees. One optional nit, your call: since the revert release now exists, the final line could say "resume bumping at 0.0.50 (first post-revert release, 2026-09-03)" to make the doc self-contained without re-investigation. Leaving it version-free is also fine since the line gets deleted at bump time anyway. LGTM. Merging stays human-side per repo rules — ready when you are: #494 |
|
Review vs master 3cd85b6: Still accurate (verified on master):
Stale and must be removed/rewritten before merging:
Master is on acp-kernel 0.0.56 (PR#607) — the reclamation was reverted long ago and five kernel releases have shipped since. This paragraph, if merged, would directly contradict the current pin and mislead the next kernel bump. Suggested replacement: one line noting the ref-slot reclamation experiment was reverted in kernel #191 and the guard is historical. (tracking: #600) |
Closes #600 — tracking issue for this PR.
Adds a "Kernel Contract" subsection to AGENTS.md §2, paired with kernel revert ranxianglei/acp-kernel#191:
Records for this repo:
messageRefsin ways that let a freed number be re-issued (kernel cursor =highestUsedIndex(map)+1, so a shrinking map can re-issue).applyCompactionArchive(fix: archive blocks across omp native compaction instead of silently deactivating them (#395) #421) map-prune has a theoretical re-issue window — drop it once kernel ref-space widening lands.acp-kernelto 0.0.48/0.0.49 (contain the reverted reclamation); resume bumping at the revert release.Docs-only change; no code touched.