docs(review): deep review 2026-06-11 + targeted July-1 plan#120
Merged
Conversation
Full fresh review of the Rust core, Elixir orchestration, VQL/VCL surface, docs and CI, with build/test ground truth (295/295 Rust tests green), a claims-vs-reality audit, and a three-week wiring-not-invention plan: make the drift->normalise loop real, truth-sweep the outward claims, connect VCL-total via a minimal admissibility gate, tag v0.2.0. https://claude.ai/code/session_01E8BpV19yxhf67UrCrvkfTr
🔍 Hypatia Security ScanFindings: 190 issues detected
View findings[
{
"reason": "Action actions/checkout@v6.0.3 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action erlef/setup-beam@v1 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action actions/cache@v5 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action decov/codecov-action@v7\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
…entity endpoint
- verisim-octad: expose graph_store()/temporal_store() accessors
(mirrors the existing provenance/spatial accessors)
- verisim-api: new drift_compute module measuring per-entity cross-modal
drift from live modality data: graph-document reflection, temporal
consistency over version history, tensor shape/data coherence, schema
baseline comparison, provenance chain integrity + staleness, spatial
coordinate validity. Semantic-vector drift is honestly marked
unmeasurable until a type-embedding registry exists.
- create/update handlers record measured drift into the shared
DriftDetector (best-effort; never fails a write). Updates use the
pre-update modality set as the schema baseline so silent modality
loss registers as schema drift.
- /drift/entity/{id} now measures THIS entity on demand with a
per-component breakdown, replacing the global-zeros shim.
- Non-computable components are excluded from detector recording, so
/drift/status aggregates reflect real measurements only.
- Tests: 4 unit + 3 integration (drift recorded on create; per-entity
component breakdown; graph-document drift falls to zero after a
consonant update). Workspace: 617 passed / 0 failed; clippy clean
under -D warnings.
Closes Week-1 items 1-2 of docs/reviews/2026-06-11-deep-review-and-july-1-plan.adoc
https://claude.ai/code/session_01E8BpV19yxhf67UrCrvkfTr
🔍 Hypatia Security ScanFindings: 190 issues detected
View findings[
{
"reason": "Action actions/checkout@v6.0.3 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action erlef/setup-beam@v1 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action actions/cache@v5 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action decov/codecov-action@v7\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
hyperpolymath
marked this pull request as ready for review
June 11, 2026 22:05
hyperpolymath
enabled auto-merge (rebase)
June 11, 2026 22:06
…endpoint
- verisim-api regenerator module: StoreRegenerator implements the
normalizer's ModalityRegenerator against the live octad store.
Supported repairs (all real write-backs through OctadStore::update,
so WAL-logged, versioned, and recorded on the provenance chain as
'normalized' events):
Graph -> Document (rebuild relations section from actual edges)
Document -> Vector (deterministic feature-hashing embedding)
Document/Vector -> Tensor (1-D feature tensor)
Unsupported pairs return honest errors instead of fake success.
measure_drift re-reads persisted state and scores via drift_compute.
- AppState gains a RegenerationEngine wired to StoreRegenerator, with a
content-modality authority order (provenance/temporal excluded as
regeneration sources: audit records, not content).
- POST /normalizer/trigger/{id} replaces the 202 no-op: measures drift,
picks the worst repairable component, runs the engine, re-measures
from persisted state, feeds the aggregates, and returns
action/before/after + the full post-repair component report.
- Tests: hash-embedding determinism, relations-section idempotency, and
an end-to-end integration test (drift 0.5 -> repaired -> 0.0,
provenance chain extended, second call no_action_needed).
Workspace: 620 passed / 0 failed; clippy clean under -D warnings.
Closes Week-1 items 3-4 of docs/reviews/2026-06-11-deep-review-and-july-1-plan.adoc
https://claude.ai/code/session_01E8BpV19yxhf67UrCrvkfTr
🔍 Hypatia Security ScanFindings: 190 issues detected
View findings[
{
"reason": "Action actions/checkout@v6.0.3 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action erlef/setup-beam@v1 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action actions/cache@v5 needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action decov/codecov-action@v7\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/checkout@v6.0.3\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action lef/setup-beam@v1\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
},
{
"reason": "Action tions/cache@v5\n needs attention",
"type": "unpinned_action",
"file": "elixir-ci.yml",
"action": "pin_sha",
"rule_module": "workflow_audit",
"severity": "medium"
}
]Powered by Hypatia Neurosymbolic CI/CD Intelligence |
hyperpolymath
disabled auto-merge
June 11, 2026 22:13
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.
Full fresh review of VeriSimDB (Rust core, Elixir orchestration, VQL/VCL surface, docs, CI) conducted 2026-06-11, plus a targeted three-week plan to 2026-07-01.
Ground truth verified this session:
cargo check --workspace --all-targetsclean (78 s);cargo test --workspace295 passed / 0 failed.Key findings:
/drift/statusreports initialized zeros; per-entity endpoint returns global zeros), and normalisation derives repairs without write-back (API endpoint is a 202 no-op).at_timereturns current state, search scores fabricated from rank, "ZKP" naming oversells sha2 commitments/Merkle/R1CS checking, orphaned-but-correct HNSW, unusedburndep, duplicate CHANGELOGs.The plan (in
docs/reviews/2026-06-11-deep-review-and-july-1-plan.adoc): week 1 wires the detect→repair loop for real + a scripted consonance demo; week 2 is the truth sweep + cheap wins (HNSW swap, real scores, honest endpoints); week 3 connects VCL-total via a minimal admissibility gate and tags v0.2.0. Acceptance criteria and explicit non-goals included.https://claude.ai/code/session_01E8BpV19yxhf67UrCrvkfTr
Generated by Claude Code