docs(board): OQ-MBX-7..14 — #418 follow-up findings catalogue#421
Conversation
Parallel-session FINDINGS/INPUT review of PR #418's `bindspace-singleton-to-mailbox-soa-v1` plan. Standalone catalogue at `.claude/board/oq-catalog-mbx.md`, mirroring `sprint-log-13/oq-catalog.md` shape; numbering continues the plan's §8 (OQ-1..4) into OQ-MBX-7..14. Each entry is framed as a question for the owning sessions (plan-author session and D-MBX-1..6 worker sessions), with three valid resolutions (covered-elsewhere / real-gap-queue-as-D-MBX-7+ / wrong-framing-because). Survey gate run before authorship: open-PR overlap clean (only #261 draft), prior-art grep per OQ cited inline. Highest-value finding is OQ-MBX-8 — `crates/cognitive-shader-driver/REFACTOR_NOTES.md:129` states the disk-write path is unbuilt and `driver.rs:458` records the in-RAM top-k row, so plan §2.5's "`persisted_row` is a pointer to the same SoA row" is forward-looking; row-identity semantics under Lance versioning/compaction need to be pinned before D-MBX-4 picks a type. Out of scope (deferred): Libet wall-clock budget — already explicitly rejected by `.claude/surreal/cognitive-substrate.md:49-51`; and cross-server baton replication. TD-RESONANCEDTO-DUP-1 severity re-rate closed before authorship (verified P3 is correct; separate modules, no crate-root collision). Docs only; no source touched, no plan-doc edited, no STATUS_BOARD modification. https://claude.ai/code/session_018FqA8TfgRBHy8zF92kx4TW
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR adds a new OQ catalogue markdown file documenting eight interconnected survey questions about MailboxSoA design, storage semantics, concurrency discipline, and eviction policies. The document establishes conventions for routing resolutions and provides indexed entries with detailed motivation, cross-references, and plan-block ownership for each question. ChangesOQ Catalogue Survey
🎯 1 (Trivial) | ⏱️ ~5 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
docs(board): OQ-MBX-7..14 — #418 follow-up findings catalogue
Summary
Parallel-session FINDINGS/INPUT review of #418
(
bindspace-singleton-to-mailbox-soa-v1). Single new file.claude/board/oq-catalog-mbx.md(304 lines); no source, no plan-doc, noSTATUS_BOARD edits.
Each entry is a question for the owning sessions (plan-author session;
D-MBX-1..6 worker sessions). Three valid resolutions per OQ, all useful as
calibration: covered-elsewhere / real-gap-queue / wrong-framing. Numbering
continues the plan's §8 (
OQ-1..4) intoOQ-MBX-7..14, so plan-§8 + thisfile form one logical catalogue.
Highest-value finding
OQ-MBX-8 —
persisted_rowsemantics under Lance versioning. The plan§2.5 claim "
ShaderCrystal.persisted_rowis a pointer to the same SoA rowlaid down in Lance" is forward-looking:
REFACTOR_NOTES.md:129states"BindSpace is in-memory only.
EmitMode::Persistsetspersisted_rowinthe crystal but doesn't write to disk.", and
driver.rs:458records thein-RAM
top_k[0].row. D-MBX-4 (death → SPO-G quad + Lance tombstone-witness)is the deliverable that has to make §2.5 true; row-identity semantics
under Lance MVCC / compaction need to be pinned before D-MBX-4 picks
a type, or the tombstone back-pointer breaks across the first compaction.
The 8 OQs (one-line each)
OQ-MBX-7#[repr(C)]byte-pin onMailboxSoA<N>itself?OQ-MBX-8persisted_rowsemantics under Lance versioningOQ-MBX-9ThinkingStyleI4_32Dplacement (column / projection / param)OQ-MBX-10Send, container shape, access modelOQ-MBX-11OQ-MBX-12OQ-MBX-13OQ-MBX-14u16↔ HHTL cascade-address encoding?Survey-gate provenance (run before authorship)
#[repr(C)]precedent onSoaContainerHeader/RungState/OntologyDelta;persisted_rowwrite-path-unbuilt evidence at
REFACTOR_NOTES.md:129; ThinkingStyleI4_32Dbasis already settled in EPIPHANIES.md:234+ but unplaced in §3; etc.).
authorship —
.claude/surreal/cognitive-substrate.md:49-51already rules"Keep the phase structure; reject the neuro-numerology."
TD-RESONANCEDTO-DUP-1 severity re-rate closed before authorship —
verified P3 stands (separate modules
dto::vsawareness_dto::, nocollision; crate compiles).
Test plan
.rs/Cargo.toml/build.rs/data/changes;no edits to
bindspace-singleton-to-mailbox-soa-v1.md(owned by theplan-author session); no STATUS_BOARD modification.
run before authorship; results cited in the catalogue preamble + per OQ.
D-MBX-7+items proposed by this PR — those arethe owning session's calls. Findings are input; deliverables are output.
https://claude.ai/code/session_018FqA8TfgRBHy8zF92kx4TW
Generated by Claude Code
Summary by CodeRabbit
Note: This update is internal documentation only and does not affect product functionality or user experience.