Repository navigation
fix(build): restore compilation; add squeakwell route plan (proof + engineering ladders) - #76
Conversation
…session_doc main.rs redeclared abi/engine/ingest/manifest as private modules but not verisim, so engine's use of crate::verisim failed with E0432 and the binary did not build. The binary now imports engine and ingest from the squeakwell library crate, which already exports all five modules. session_doc built an unused as_object_mut() binding that triggered an unused-variable warning and did nothing; removed. cargo build, cargo test (23 tests) and cargo clippy are clean. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J4QvDizuhJtQDcxHxM2wj8
The committed lockfile predated the VeriSimDB client dependencies declared in Cargo.toml; cargo build regenerated it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01J4QvDizuhJtQDcxHxM2wj8
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
| Layer / File(s) | Summary |
|---|---|
Update CLI module imports src/main.rs |
The entry point imports engine and ingest from the squeakwell crate instead of declaring local modules for abi, engine, ingest, and manifest. |
Session Document Cleanup
| Layer / File(s) | Summary |
|---|---|
Remove state-processing code src/verisim.rs |
session_doc retains its existing signature. The function no longer clones state or performs a mutable-object lookup. The related comment is also removed. |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Bug fix
Merge Risk: 
Minimal · previous review 7ba8a
-
No merge-blocking behavior or regression is established by this change.
Pre-merge checks: 
5
-
Full details: Description check
-
Explanation
The description includes the required Summary, Changes, checklist, and Testing sections. It reports the build and test results and identifies the pre-existing clippy warning. The stated planning document is described as future work rather than a change included in this pull request.
Full details: Docstring Coverage
-
Explanation
No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files.
-
Finishing Touches: 💡 1
-
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
📝 Generate docstrings
-
- 🤖 Coding Agent task started for docstring generation.
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
CI status on
No re-run is warranted: the failures are deterministic file-content checks, not flakes. Generated by Claude Code |
|
✅ Coding Agent task started: View task and status The task will inspect the CI failures, validate its fix, and commit the fix to this branch automatically.
⏭️ Checks skipped: 2 · already failing on `main` (not caused by this PR)
|
|
🤖 Completed: Generate docstrings for PR #76 — View commit |
… checklist, and verify pinned just downloads
|
…) and honest baseline (#78) ## Summary Follow-up to [#76](#76), which merged the compile fix but not the planning document it promised. This PR lands `docs/ROUTE-PLAN.adoc` and brings the repository's claim surface in line with it. The plan answers: the fastest honest route down the proof pathways and down the engineering route to squeakwell with VeriSimDB as the required underlying database; what else must be created in verisimdb, vcl-ut, echo-types, tropical-types, standards and mathesis; the architecture (four options judged under three lenses; the compiler route, D with A's pure kernel in front, is the labelled default arm); the README's five invariants restated after adversarial refutation (drift is not monotone under fill, two are vacuous as written, one belongs to vcl-ut); an 18-rung proof ladder with prover homes per estate PROOF-POLICY; the empirical gate E1–E7; the task graph for the ultracode engineering session; the decisions only the owner can take; and the cross-repo action list. It rests on a 28-agent survey (six readers, four designers, one ladder designer, three judges, fourteen refuters) over the six repositories. No prover ran: every proof row stays OPEN. ## Changes - `docs/ROUTE-PLAN.adoc` (new): the plan. - `container/deploy.k9.ncl`: `K9!` magic line and an inline pedigree so the Dogfood Gate K9 validator passes; placeholders replaced; described as a batch job. - `setup.sh`: `curl --proto "=https" --tlsv1.2` on the pinned just download (the one SonarCloud vulnerability on PR 76). - `src/interface/ffi/*.zig`: `{{PROJECT}}`/`{{project}}` placeholders replaced. - `src/abi/mod.rs`: invariants restated in a mechanisable form; records that no Idris2 proofs exist yet. - `README.adoc`: title names the repository; "data that passes all 10 levels" becomes the gate-admissible reintroduction statement at level ≤ 6; "Idris2 type kernel with proofs" becomes invariants with no proofs yet; TypedQLiser becomes the vclt-gate binary; Status points at the plan. - `PROOF-NEEDS.adoc`: phantom ~3,470 LOC and template-ABI-proof claims removed; prover homes per policy recorded. - `ROADMAP.adoc`, `TEST-NEEDS.adoc`, `EXPLAINME.adoc`, `.machine_readable/descriptiles/STATE.a2ml`: describe the recovery tool and its real state (23 tests, all against stubs; audio-processing template leak removed; phantom file references removed). - `tests/e2e_test.rs`: unused import dropped (the one clippy warning). ## RSR Quality Checklist ### Required - [x] Tests pass (`cargo test`: 23 passed) - [x] Code is formatted - [x] Linter is clean (`cargo clippy --all-targets`: no warnings) - [x] No banned language patterns - [x] No `unsafe` blocks (`#[forbid(unsafe_code)]`) - [x] No banned functions - [x] SPDX license headers present on all new/modified source files - [x] No secrets, credentials, or `.env` files included ### As Applicable - [x] `.machine_readable/descriptiles/STATE.a2ml` updated - [ ] `.machine_readable/descriptiles/ECOSYSTEM.a2ml` updated - [ ] `.machine_readable/descriptiles/META.a2ml` updated - [x] Documentation updated for user-facing changes - [ ] `TOPOLOGY.md` updated - [ ] `CHANGELOG` or release notes updated - [x] New dependencies reviewed for license compatibility (none added) - [x] ABI/FFI changes validated (placeholder rendering only; the Zig tree remains unbuilt, as the plan records) ## Testing ``` cargo build cargo test # 8 e2e + 5 property + 10 unit = 23 passed cargo clippy --all-targets # no warnings ``` CI expectations: Dogfood Gate K9 should now pass (`container/deploy.k9.ncl` carries the magic line and pedigree name); SonarCloud's new-code security rating should return to A (curl hardened). --- _Generated by [Claude Code](https://claude.ai/code/session_01J4QvDizuhJtQDcxHxM2wj8)_ --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>




Summary
The squeakwell binary did not build on
main:src/main.rsredeclaredabi/engine/ingest/manifestas private modules but notverisim, soengine'suse crate::verisimfailed with E0432. This PR makes the binary consume the library crate instead, drops a dead binding, and refreshes the stale lockfile.A planning document (architecture, what else must be created across verisimdb / vcl-ut / echo-types / tropical-types, the proof ladder, and the ultracode engineering session plan) follows on this branch once the multi-agent survey completes.
Changes
src/main.rs: replace module redeclarations withuse squeakwell::{engine, ingest};src/verisim.rs: remove unusedobjbinding insession_doc(was an unused-variable warning that did nothing)Cargo.lock: regenerated; the committed lock predated theureq/urlencoding/criteriondependencies inCargo.tomlRSR Quality Checklist
Required
cargo test: 23 passed across e2e/property/unit)cargo clippy --all-targets: one pre-existing unused-import warning intests/e2e_test.rs, not introduced here)unsafeblocks (#[forbid(unsafe_code)]already in place).envfiles includedAs Applicable
.machine_readable/STATE.a2mlupdated.machine_readable/ECOSYSTEM.a2mlupdated.machine_readable/META.a2mlupdatedTOPOLOGY.mdupdatedCHANGELOGor release notes updatedTesting
Generated by Claude Code