plan(sprint-13/cleanup): CSI-19 D-id reconciliation + CSI-20 stale-claim sweep post PR #391#392
Merged
Conversation
…aim sweep post PR #391 merge W-Meta-Opus spawn-checklist items 3 + 4: doc-only post-merge cleanup of two drift classes surfaced during the Wave H preflight audit. Item 5 (CSI-9 cross-repo ndarray PR) resolved externally — ndarray PR #147 merged to master at HEAD e956e9d9. CSI-19 fixes — PP-3 rayon spec ID-attribution (3 sentences, three errors each): - "D-CSV-16 slot reserved by PP-2 (sprint-13 splat on-Think method migration)" → "W-Meta-Opus canonical: D-CSV-13b → PP-6, D-CSV-14 → PP-4, D-CSV-16 → PP-5, D-CSV-17 → PP-3 (this spec)" - "Sibling sprint-13+ deliverable (PP-2 spec)" → "(PP-4 spec)" [×2] - "(struct-method half is D-CSV-14 / PP-2)" → "(D-CSV-14 / PP-4)" CSI-20/CSI-9 fixes — plan v3 disentangles lance-graph-side d4e5bbc fix (CSI-7/8 only) from ndarray-side PR #147 fix (CSI-9). Four sites updated in convergence-v3.md: - §0 D-CSV-11 row: CSI-9 attribution now points to ndarray PR #147 / e956e9d9 - §0 d4e5bbc bullet: scope corrected to CSI-7 + CSI-8 only - §13.x first risk-table cell: CSI-9 attribution corrected - §13.13 aggregation gap: lance-graph d4e5bbc vs ndarray PR #147 disambiguated All four CSI-9 mentions now cite ndarray master HEAD e956e9d9 as verification anchor (per preflight-drift-auditor PD1 stale-claim verification pattern). Spawn-checklist gates remaining for sprint-13 worker fleet dispatch: ✅ 1. PR #390 merge (lance-graph main 961baf8) ✅ 2. Wave H preflight PR merge (PR #391, lance-graph main 33eb812) ✅ 3. CSI-19 ID consolidation (this commit) ✅ 4. CSI-20 stale-claim sweep (this commit) ✅ 5. CSI-9 cross-repo ndarray PR (ndarray PR #147 merged externally) ⏳ 6. OQ-CSV-7..16 user ratification (10 sprint-spawn blockers, user-side) ✅ 7. PP-8 worker-template-v2 → canonical (via PR #391 merge) ✅ 8. PP-13 brutally-honest-tester → probation-tier (via PR #391 BOOT.md) Only OQ ratification remains. Sprint-13 worker fleet dispatch unblocks immediately after user ratifies OQ-CSV-7..16 per PP-11 recommendations.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 516b0624f3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Comment on lines
+583
to
586
| - 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 |
There was a problem hiding this comment.
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 👍 / 👎.
Merged
5 tasks
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
2 participants
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
Doc-only post-merge cleanup closing W-Meta-Opus spawn-checklist items 3 + 4 from PR #391. Zero source-code touches.
Item 5 (CSI-9 cross-repo ndarray PR) resolved externally — ndarray PR #147 merged to master at HEAD
e956e9d9. After this PR merges, only OQ-CSV-7..16 user ratification (gate 6) remains before sprint-13 worker fleet dispatch.Fixes
CSI-19 — PP-3 rayon spec D-id attribution (4 edits)
PP-3 spec contained a three-error sentence: "
D-CSV-16slot reserved by PP-2 (sprint-13 splat on-Think method migration)" — PP-2 is iron-rules-doctrine, PP-4 owns splat-on-Think, D-CSV-16 is CAM-PQ per PP-5. Replaced with the W-Meta-Opus canonical assignment table (D-CSV-13b → PP-6,D-CSV-14 → PP-4,D-CSV-16 → PP-5,D-CSV-17 → PP-3). Three other "PP-2" → "PP-4" attribution fixes in the same spec.CSI-20 — plan v3 lance-graph d4e5bbc vs ndarray PR #147 disambiguation (4 sites)
PP-1 plan v3 conflated two repos' fixes by claiming "CSI-9 resolved via d4e5bbc aggregation commit." Reality:
d4e5bbcis a lance-graph commit that resolved CSI-7 (workspace member) + CSI-8 (lance-graph lib.rs orphan). CSI-9 (ndarray-sidepub modregistration) was fixed by ndarray PR #147 merge (mastere956e9d9). All four sites now cite ndarray master HEAD as the verification anchor (per preflight-drift-auditor PD1 stale-claim verification pattern).Spawn-checklist status
961baf833eb812e956e9d9oq-catalog.md)Test plan
grep "PP-2" .claude/specs/pr-sprint-13-rayon-streams.mdreturns zero hitshttps://claude.ai/code/session_01UwJuKqP828qyX1VkLgGJFS
Generated by Claude Code