Skip to content

ci: fmt + clippy under stable 1.96 to make rust-ci green (follow-up)#78

Merged
hyperpolymath merged 10 commits into
mainfrom
claude/new-session-znxgm7
Jun 28, 2026
Merged

ci: fmt + clippy under stable 1.96 to make rust-ci green (follow-up)#78
hyperpolymath merged 10 commits into
mainfrom
claude/new-session-znxgm7

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Follow-up to the merged CI-green PR (#77). With rust-ci now actually running, main is red under the CI toolchain (stable 1.96) on both cargo fmt --all -- --check and cargo clippy --locked --all-targets -- -D warnings. This brings the Rust sources to fmt + clippy clean.

Changes

  • clippy: criterion::black_boxstd::hint::black_box (criterion's re-export is deprecated); cmp_owned (compare PathBuf against Path::new(...) instead of an owned PathBuf); empty-format-string lint.
  • Rust hygiene: cargo fmt across the crate so cargo fmt --all -- --check passes under stable 1.96.

RSR Quality Checklist

Required

  • Tests pass (cargo test --locked --all-targets)
  • Code is formatted (cargo fmt --all -- --check)
  • Linter is clean (cargo clippy --locked --all-targets -- -D warnings)
  • No banned language patterns
  • No banned functions (believe_me, sorry, etc.)
  • SPDX license headers present on modified files
  • No secrets, credentials, or .env files included

Testing

Verified locally with the CI toolchain (rustc/clippy/rustfmt 1.96.0): cargo fmt --check, clippy -D warnings, cargo check --locked, cargo test --locked all pass.

🤖 Generated with Claude Code


Generated by Claude Code

claude and others added 9 commits June 27, 2026 19:31
Add Iseriser.ABI.Semantics proving the headline domain property: a
generated -iser scaffold is Conformant exactly when all five required
components (Manifest, Idris2 ABI, Zig FFI, Codegen, Rust CLI) are
present. Conformant is built from genuine Data.List.Elem membership
obligations with no catch-all constructor, so a scaffold missing any
component has no witness.

Includes a sound+complete decConformant : (s) -> Dec (Conformant s),
a soundness fact certifyConformantSound, a positive control
(completeIsConformant via explicit Elem positions), and a negative
control (ffiMissingNotConformant : Not (Conformant ffiMissing)).
Non-vacuity confirmed: a deliberately-false Has Ffi witness for the
FFI-missing scaffold is rejected by idris2.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
Add Iseriser.ABI.Invariants over the existing Layer-2 Semantics model
(Component/Scaffold/Has/Conformant). Two new, deeper, distinct theorems:

  1. Generation soundness (correct-by-construction): genScaffold provably
     emits (s ** Conformant s) for any LanguageModel, with a corollary
     tying it back to the Layer-2 certifier (conformantCertifies).
  2. Upward-closure / monotonicity: conformantStable proves Conformance is
     preserved under extension, via a genuine Elem-weakening lemma
     (elemAppendRight) — an algebraic closure law, not the Layer-2 decision.

Includes a sound+complete Dec (decExtendConformant), a positive control
(extendedGeneratedConformant) and a non-vacuity negative control
(extendedBrokenNotConformant : Not ...). Builds clean with zero warnings;
adversarial false proof rejected. No believe_me/postulate/assert_total.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
Add Iseriser.ABI.FfiSeam proving the resultToInt encoding is sound:
- intToResult decoder + resultRoundTrip (lossless/faithful encoding)
- resultToIntInjective derived from round-trip (distinct outcomes never
  collide on the wire)
- positive controls (concrete decodes by Refl) and a machine-checked
  non-vacuity control (resultToInt Ok /= resultToInt Error)

Genuine total proof: no believe_me/postulate/assert_total/etc. Registered
in iseriser-abi.ipkg; package builds clean with zero warnings.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
Assemble the existing Layer-2/3/4 ABI proofs into one inhabited value,
Iseriser.ABI.Capstone.abiContractDischarged : ABISound, with fields:
  - flagship     : Conformant Semantics.completeScaffold
                   (reuses Semantics.completeIsConformant)
  - invariant    : Conformant (extend [Manifest] generatedScaffold)
                   (reuses Invariants.extendedGeneratedConformant)
  - ffiInjective : resultToInt injectivity
                   (reuses FfiSeam.resultToIntInjective)

Pure composition of already-exported witnesses: if any prior layer were
unsound the capstone would not typecheck. Adversarial check confirms a
wrong-type witness in the flagship slot is rejected. %default total, SPDX,
zero warnings. ipkg updated (Capstone listed last).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
…ble fix); port ABI-FFI gate Python->Bash (Python is estate-banned)

Resolves the standing baseline CI reds (rust-ci toolchain error, governance
Language/anti-pattern, governance workflow-lint) without altering the proven
ABI. The Bash gate reproduces the former Python gate's verdict verbatim
(validated across all -iser repos) and catches the same drift classes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
…x->std::hint, cmp_owned, empty-format; make rust-ci green on main
@hyperpolymath hyperpolymath marked this pull request as ready for review June 28, 2026 09:23
@hyperpolymath hyperpolymath merged commit 356017a into main Jun 28, 2026
4 checks passed
@hyperpolymath hyperpolymath deleted the claude/new-session-znxgm7 branch June 28, 2026 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants