SoAReview agent + first sweep findings (4-angle transcode audit)

[AdaWorldAPI]

Summary

Ships the SoAReview multi-angle transcode review agent and the findings from its first run (2026-04-24). The agent audits how Python/legacy subsystems integrate into the BindSpace SoA + Arrow-scalar DTO discipline across 7 canonical angles plus 1 optional. Encodes the three-role-taxonomy (grammar × persona × thinking), semantic-kernel framing (Markov + CAM-PQ), and reusability-across-BBB doctrine.

What's in the commit

• .claude/agents/soa-review.md (370 lines) — agent card with full iron-rule reference block, 6-section deliverable template, verdict taxonomy, knowledge bootload list, anti-patterns, and four reference-run reports.
• .claude/plans/unified-integration-v1.md § 6 (appended, ~110 lines) — first SoAReview sweep findings with cross-cutting verdict, angle-by-angle verdict table, three-role-taxonomy check, ranked 10-item expansion list, semantic-kernel definition, reusability inside/outside BBB framing.
• .claude/board/TECH_DEBT.md 2026-04-24 ghost-columns entry — documents rationale_phase + dialect as CognitiveEventRow ghost fields; minimal Phase B wiring for each. rationale_phase already wired in commit 564aac4; dialect remains open.

First sweep — cross-cutting verdict

FingerprintColumns.cycle in bindspace.rs is Box<[u64]> (Binary16K), not Box<[f32; 16_384]> (Vsa16kF32) as CLAUDE.md §The Click mandates.

This single DRIFT blocks DU-3 (RoleDB VSA UDFs need the multiply+add carrier), blocks the semantic-kernel (Markov + CAM-PQ), and leaves MarkovBundler / Trajectory / vsa_permute as doc-referenced-but-unimplemented. Substrate precondition for most remaining deliverables.

Angle-by-angle verdicts

Angle	Verdict	One-liner
Callcenter	PARTIAL	BBB spine live; dialect/scent/subscribe() ghosts; 3 vsa_udfs.rs ops broken
Archetype	LOCKED-MAPPING-INCOMPLETE	ADR-0001 locked; crate not scaffolded; AsyncProcessor/CommandBroker/Component undefined
Persona	DRIFTING-BUT-MANAGEABLE	Contract clean; thinking-engine PersonaProfile carries content-in-register; persona/role_keys.rs missing
Grammar-Markov	SCATTERED-NOT-UNIFIED	Substrate format drift above; global_context absent; son/father/grandfather has no method

Three-role-taxonomy status

Taxonomy	Catalogue file	Status
Grammatical	contract::grammar::role_keys	LIVE
User/Agent/Persona	contract::persona::role_keys	MISSING
Thinking-style	contract::thinking_styles::role_keys	MISSING

Ranked expansion list (substrate → semantic kernel → surface ghosts)

1. Vsa16kF32 substrate migration (FingerprintColumns.cycle)
2. persona/role_keys.rs
3. thinking_styles/role_keys.rs
4. Slice allocation map with const_assert non-overlap
5. MarkovBundler + Trajectory + vsa_permute implementation
6. global_context: Box<[f32; 16_384]> field on BindSpace
7. persona/content_store.rs + YAML migration
8. dialect + scent ghost-column wiring
9. lance-graph-archetype crate scaffold
10. subscribe() live implementation

Items 1-4 unify the substrate; 5-7 ship the semantic kernel; 8-10 resolve the surface ghosts.

Test plan

• cargo test -p lance-graph-callcenter — confirm no regression (11 tests currently pass).
• cargo test -p lance-graph-contract — confirm no regression.
• cargo run --manifest-path crates/jc/Cargo.toml --release --example prove_it — six-pillar proof stays green (3 pass + 1 Pearl 2³ new + 2 deferred).
• Review agent card for completeness — can a fresh session spawn a SoAReview sweep using only this card?
• Review §6 against the four source agent reports — verdicts match.

https://claude.ai/code/session_01SbYsmmbPf9YQuYbHZN52Zh

---

Generated by Claude Code

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c58029013c

ℹ️ 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".

[chatgpt-codex-connector]

Mark rationale_phase debt as resolved

This row states that rationale_phase is a Phase-A ghost that is always false, but the code already wires it through LanceMembrane (current_rationale_phase, set_faculty_context(..., rationale_phase, ...), and projection into CognitiveEventRow) in commit 564aac4. Leaving this as an open ghost-debt item creates contradictory guidance for follow-up sweeps and can trigger redundant implementation work on an already-shipped path; this entry should be updated to track only unresolved fields (for example dialect) or mark the rationale_phase portion as paid.

Useful? React with 👍 / 👎.