docs: backfill W1.5 board entries and correct the stale W1.5 contract section - #296
Merged
Merged
Conversation
… section Two hygiene gaps, both found while closing out the #293/#294/#295 arc. 1. Board backfill. `.claude/blackboard.md` and `.claude/board/AGENT_LOG.md` contained ZERO mentions of the W1.5 signature primitives — all three PRs landed with no board entry, in violation of CLAUDE.md's Agent Protocol ("write decisions to the blackboard, not just to chat"). A session reading the blackboard would not have known signature_pde or randomized_signature existed. Entries added for #293, #294 and #295, carrying the decisions, the measured numbers, and the loose ends. 2. Contract-section correction. The W1.5 section of `.claude/knowledge/vertical-simd-consumer-contract.md` still described all three primitives as future work "gated on jc Pillar 11 activation". That gate opened 2026-05-07 (lance-graph jc/src/lib.rs:26), and #6 and #7 have shipped. The section is regraded in place with the workspace's existing ⊘-blockquote convention: original sketches kept verbatim, each followed by a correction block, rather than silently rewritten. The correction worth carrying forward: this doc's API sketches have now been wrong the same way TWICE. Both #6 and #7 were sketched as f32/F32x16 while the real consumer (lance-graph crates/sigker) is f64/Vec<f64>, so both shipped on F64x8. #294 additionally found the sketch wrong about buffer ownership (projections are materialized once per encoder and reused, not re-derived per step) and about shape (k is runtime, so the hot path is a k×k GEMV, not a single-register update). W1.5-#8's `I16x16` sketch is therefore marked UNVERIFIED — 2-for-2 miss rate — and must be read off crates/sigker/src/log_signature.rs before any implementation starts. Also recorded: W1.5-#8 is unblocked, not deferred; and #295 merged ~5s after leaving draft, so its 100% docstring figure is a manual /// scan rather than bot-confirmed. Markdown only — no source, tests, or build files touched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016WkNBjHc2e3zuyz9i8qJEv
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueComment |
Bugbot couldn't run - usage limit reachedBugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit. A user or team admin can review and increase usage limits in the Cursor dashboard. (requestId: serverGenReqId_34f1c47b-c2b9-40e2-b87c-b3a672b98012) |
AdaWorldAPI
marked this pull request as ready for review
September 4, 2026 03:55
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
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
Two hygiene gaps found while closing out the #293/#294/#295 arc. Markdown only — no source, tests, or build files touched.
1. The board had no record of any of it
.claude/blackboard.mdand.claude/board/AGENT_LOG.mdcontained zero mentions of the W1.5 signature primitives. All three PRs landed with no board entry, againstCLAUDE.md's Agent Protocol ("write decisions to the blackboard, not just to chat"). A session reading the blackboard would not have knownsignature_pdeorrandomized_signatureexisted at all.Entries added for #293, #294 and #295 carrying the decisions, the measured numbers, and the loose ends — including the two that matter most: the
debug_assert_eq!-compiles-out-in-release bug #294's follow-up fixed, and the caveat that #295's 100% docstring figure is a manual///scan rather than bot-confirmed (it merged ~5s after leaving draft, so CodeRabbit never re-ran).2. The W1.5 contract section was describing a world that no longer exists
.claude/knowledge/vertical-simd-consumer-contract.mdstill presented all three W1.5 primitives as future work "gated on jc Pillar 11 activation". That gate opened 2026-05-07 (lance-graph:crates/jc/src/lib.rs:26), and #6 and #7 have shipped.Regraded in place using the workspace's existing
⊘-blockquote convention — original sketches kept verbatim, each followed by a correction block, rather than silently rewritten.The correction worth carrying forward
This doc's API sketches have now been wrong the same way twice. Both #6 and #7 were sketched as
f32/F32x16while the real consumer (lance-graph crates/sigker) isf64/Vec<f64>— both shipped onF64x8. #294 additionally found the sketch wrong about buffer ownership (projections are materialized once per encoder and reused, not re-derived per step) and about shape (kis runtime, so the hot path is ak×kGEMV, not a single-register update).W1.5-#8's
I16x16sketch is therefore marked UNVERIFIED — a 2-for-2 miss rate — and must be read offcrates/sigker/src/log_signature.rsbefore any implementation starts.Also recorded: W1.5-#8 is unblocked, not deferred.
Test plan
cargostate is unaffected by this diff⊘correction convention matched against existing usage in the same doc family🤖 Generated with Claude Code
https://claude.ai/code/session_016WkNBjHc2e3zuyz9i8qJEv
Generated by Claude Code