Skip to content

plan(sprint-13/cleanup): CSI-19 D-id reconciliation + CSI-20 stale-claim sweep post PR #391 #392

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AdaWorldAPI merged 1 commit into from
May 16, 2026
+12 −9
Merged

plan(sprint-13/cleanup): CSI-19 D-id reconciliation + CSI-20 stale-claim sweep post PR #391 #392

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .claude/plans/cognitive-substrate-convergence-v3.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,12 @@

| D-id | PR | Commit | Outcome |
|---|---|---|---|
| D-CSV-11 (W-F4) | #388 | `77f2d26` (merge) | `InferenceStream` shipped in ndarray `hpc/stream/inference.rs` + registered in `mod.rs` (W-F5 over-delivered: also registered `qualia` + `splat_field` mod entries). Sibling `QualiaStream` + `SplatFieldStream` files shipped (W-F4/W-F6) — CSI-9 cross-repo registration was resolved via the d4e5bbc aggregation commit. `par_*` rayon variants NOT YET SHIPPED — carries to D-CSV-17 (sprint-13). |
| D-CSV-11 (W-F4) | #388 | `77f2d26` (merge) | `InferenceStream` shipped in ndarray `hpc/stream/inference.rs` + registered in `mod.rs` (W-F5 over-delivered: also registered `qualia` + `splat_field` mod entries). Sibling `QualiaStream` + `SplatFieldStream` files shipped (W-F4/W-F6). **CSI-7/CSI-8** (lance-graph-side lib.rs + workspace orphan) **resolved via lance-graph `d4e5bbc` aggregation commit.** **CSI-9** (ndarray-side `pub mod` registration) **resolved via ndarray PR #147 merge** — ndarray master HEAD = `e956e9d9` (verified 2026-05-16 post-PR-#391-merge). `par_*` rayon variants NOT YET SHIPPED — carries to D-CSV-17 (sprint-13). |
| D-CSV-12 (W-F7) | #388 | `77f2d26` (merge) | `splat_ops` module in thinking-engine: `splat_gaussian`, `score_hole_closure`, `replay_coherence`, `emit_if_epiphany` as **free functions** on i4 substrate. On-Think carrier methods deferred — carries to D-CSV-14 (sprint-13). |

**Wave F aggregation + codex P2 follow-up — COMPLETE:**

- `d4e5bbc` — W-Meta-Opus honest review committed; CSI-7 / CSI-8 / CSI-9 integration fixes landed in single aggregation commit.
- `d4e5bbc` — W-Meta-Opus honest review committed; CSI-7 / CSI-8 lance-graph-side integration fixes landed in single aggregation commit. **CSI-9 (cross-repo ndarray-side `pub mod` registration) was a separate ndarray-repo fix shipped via PR #147 merge (ndarray master `e956e9d9`), NOT via this lance-graph commit.**
- PR #389 (`b526485` merge) — codex P2: `AttentionMaskBackend` trait impl for `AttentionMaskSoA` + canonical `MailboxId` import (CSI-10 closed). 14/14 attention-mask tests pass.

### §0.2 What shipped in sprint-12 Wave G (post-Wave-F continuation) 🆕 v3
Expand Down Expand Up @@ -395,7 +395,7 @@ D-CSV-5b sprint-13 cutover requires consumer audit. Sibling approach minimized r

### 13.8 D-CSV-11 ndarray PR coordination (HIGH → MED) [UPDATED v3]

**Risk REDUCED.** Sprint-12 Wave F shipped the ndarray `hpc/stream/mod.rs` registration via the d4e5bbc aggregation commit (CSI-9 fix). Cross-repo coordination is now established; sprint-13 D-CSV-17 follows the same pattern with lower coordination cost.
**Risk REDUCED.** Sprint-12 Wave F shipped the ndarray `hpc/stream/mod.rs` registration via the **ndarray PR #147 merge** (CSI-9 fix; ndarray master `e956e9d9`). The lance-graph-side d4e5bbc aggregation commit closed CSI-7 (workspace member) + CSI-8 (lance-graph lib.rs orphan), NOT CSI-9. Cross-repo coordination is now established; sprint-13 D-CSV-17 follows the same pattern with lower coordination cost.

### 13.9 Subagent isolation — workspace build conflicts (MED — UNCHANGED) [carries from v2]

Expand All @@ -415,7 +415,7 @@ ISSUE-X3. Sprint-13 fleets should `cargo clean` at sprint-start.

### 13.13 Aggregation gap (CSI-13 → CSI-18) (HIGH → LOW) [UPDATED v3] 🆕 v3

**Risk REDUCED via Wave G + CSI-18.** Sprint-12 Wave F shipped CSI-7/8/9 registration gaps; the d4e5bbc aggregation commit + the PR #389 codex P2 follow-up closed them. Sprint-13 worker template v2 (PP-8) bakes the I-AGGREGATION-DISCIPLINE iron-rule candidate into the prompt template. Mechanism: every worker prompt includes "your lib.rs/mod.rs hunk OR an explicit aggregation deliverable as the wave's final worker."
**Risk REDUCED via Wave G + CSI-18.** Sprint-12 Wave F shipped CSI-7 / CSI-8 (lance-graph-side) via the d4e5bbc aggregation commit + the PR #389 codex P2 follow-up; CSI-9 (cross-repo ndarray-side) shipped via ndarray PR #147 merge (master `e956e9d9`). Sprint-13 worker template v2 (PP-8) bakes the I-AGGREGATION-DISCIPLINE iron-rule candidate into the prompt template. Mechanism: every worker prompt includes "your lib.rs/mod.rs hunk OR an explicit aggregation deliverable as the wave's final worker."

### 13.14 Plan-git drift (CSI-11) (LOW — UNCHANGED, mitigated via I-PLAN-GIT-RECONCILE candidate) [carries from v2]

Expand Down
13 changes: 8 additions & 5 deletions .claude/specs/pr-sprint-13-rayon-streams.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,11 @@ with the `rayon = ["dep:rayon", "std"]` feature flag at line 148. Sprint-12
left it unused inside `src/hpc/stream/`; sprint-13 wires it in.

**New deliverable id:** D-CSV-17 — chosen to not collide with the in-flight
D-CSV-13/14/15 entries in convergence-v2 §11, and to follow the D-CSV-16
slot reserved by PP-2 (sprint-13 splat on-Think method migration).
D-CSV-13/14/15 entries in convergence-v2 §11, and to follow the W-Meta-Opus
canonical sprint-13 D-id assignment: **D-CSV-13b → PP-6 (SIMD), D-CSV-14 →
PP-4 (splat on-Think), D-CSV-16 → PP-5 (WitnessIndexCamPq), D-CSV-17 → PP-3
(this spec, rayon par_*)**. See `.claude/board/sprint-log-13/preflight-meta-
review-opus.md` §2 for the reconciled per-planner table (CSI-19 cleanup).

**Cross-ref table:**

Expand All @@ -44,7 +47,7 @@ slot reserved by PP-2 (sprint-13 splat on-Think method migration).
| `splat_field.rs:7` | "par_splat_stream rayon variant is sprint-13+" | Same |
| `Cargo.toml:47,148` | rayon optional dep + feature flag | Feature gate already exists |
| convergence-v2.md §11 D-CSV-11 row | "QualiaStream, InferenceStream, SplatFieldStream + par_* rayon variants" | Parent deliverable |
| convergence-v2.md §11 D-CSV-14 row | "on-Think method migration for D-CSV-12 splat ops" | Sibling sprint-13+ deliverable (PP-2 spec) |
| convergence-v2.md §11 D-CSV-14 row | "on-Think method migration for D-CSV-12 splat ops" | Sibling sprint-13+ deliverable (PP-4 spec) |
| ndarray CLAUDE.md "Data-Flow Invariants" | `&[u8]` slices, no `&mut self` during compute | Hard constraint this spec respects |

---
Expand Down Expand Up @@ -577,7 +580,7 @@ a single-thread pool gracefully on uniprocessor builds.
**Sibling preflight specs (sprint-13, planned):**

- PP-1: sprint-13 execution plan v3 — introduces D-CSV-17 row in §11.
- PP-2: D-CSV-14 on-Think method migration spec — co-evolves with this PR;
- PP-4: D-CSV-14 on-Think method migration spec — co-evolves with this PR;
the splat on-Think methods will internally call `par_splat_field_stream`
for the fleet-evaluation hot path.
- PP-4: CI matrix update — adds `--features rayon` to the ndarray test job
Comment on lines +583 to 586
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot May 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Give the CI-matrix follow-up a unique owner

This CSI-19 cleanup now assigns PP-4 to the D-CSV-14 Think-method spec, but the very next sibling entry still assigns PP-4 to the separate CI-matrix update. The canonical table you added earlier in this same file only maps PP-4 to splat on-Think, so sprint-13 workers reading this section still have an ID/ownership collision for the required --features rayon CI coverage and may not know which planner/spec owns it.

Useful? React with 👍 / 👎.

Expand All @@ -587,7 +590,7 @@ a single-thread pool gracefully on uniprocessor builds.

- `.claude/knowledge/splat-shader-rayon-struct-method-vision.md` — original
vision for rayon + struct-method surface; this PR is the
rayon half (struct-method half is D-CSV-14 / PP-2).
rayon half (struct-method half is D-CSV-14 / PP-4).
- `.claude/knowledge/vsa-switchboard-architecture.md` — VSA bundle
associativity-in-expectation; underwrites the §6 pattern C safety
argument for `Vsa16kF32`.
Expand Down