Skip to content

ci: format parser.rs under stable 1.96 to make rust-ci green (follow-up)#44

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

ci: format parser.rs under stable 1.96 to make rust-ci green (follow-up)#44
hyperpolymath merged 8 commits into
mainfrom
claude/new-session-znxgm7

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Follow-up to the merged CI-green PR (#43). With rust-ci now actually running, cargo fmt --all -- --check fails on main under the CI toolchain (stable rustfmt 1.96) — the toolchain-pin PR landed before the formatting fix did. This brings src/codegen/parser.rs to rustfmt-clean so rust-ci goes green.

Changes

  • Rust hygiene: cargo fmt (src/codegen/parser.rs) 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
  • 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 added 7 commits June 27, 2026 19:24
…ition

Add Dafniser.ABI.Semantics raising the Idris2 ABI to Layer 2 with a genuine,
machine-checked semantic proof of the repo's headline ("correct-by-construction
code generation").

Model: a Dafny spec as the postcondition MaxPost a b r — a real proposition
inhabited only when r dominates both inputs and equals one of them. The
generated body genMax returns its result paired with a MaxPost proof
(correct-by-construction). genMaxCorrect derives all three ensures conjuncts
(result>=a, result>=b, result==a||result==b) for all inputs; noUndershootLeft
refutes the bad case. A sound certifier (certifyMax/certifyMaxSound) emits the
ABI's VerificationResult. Positive control maxPositive : MaxPost 7 3 7 and
negative control maxNegative : Not (MaxPost 7 3 2) are machine-checked; an
adversarial false proof (MaxPost 7 3 2) is rejected by idris2, confirming
non-vacuity.

Builds clean (idris2 0.7.0, exit 0, zero warnings).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
…max duality

Adds Dafniser.ABI.Invariants over the existing Layer-2 model
(Dafniser.ABI.Semantics, reused unchanged). Three distinct, deeper
machine-checked properties:

1. Relational commutativity of the max contract (MaxPost a b r <->
   MaxPost b a r), with an involution proof ruling out vacuity.
2. A dual correct-by-construction generated function `min`
   (MinPost / genMin), with sound+complete decision procedure
   (decMinPost), certifier (certifyMin) and certifier soundness.
3. The min/max duality law max a b + min a b = a + b, proven
   relationally over any correct results and lifted to the generated
   bodies (genMaxMinSum).

Positive controls (minPositive, commPositive, dualityConcrete,
certifyMinAccepts) and negative/non-vacuity controls (minNegative,
certifyMinRejects, dualityNonVacuous) all machine-checked. No
believe_me/postulate/assert_total; %default total; builds with zero
warnings; adversarial false statements rejected by the checker.

Registers Dafniser.ABI.Invariants last in dafniser-abi.ipkg.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A6PSzJWpRxtzGDjUCEh7Mx
…fiSeam)

Prove the FFI result-code encoding is sound:
- intToResult decoder + resultRoundTrip (lossless/faithful encoding)
- resultToIntInjective derived from the round-trip (no collisions on the wire)
- positive controls (decode 0=Ok, 4=NullPointer, 5=Nothing)
- machine-checked negative/non-vacuity control (Ok and Error differ)

Genuine total proof: no believe_me / postulate / assert_total / %hint.
Wires Dafniser.ABI.FfiSeam into dafniser-abi.ipkg. Builds clean, 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
end-to-end soundness certificate (Dafniser.ABI.Capstone):

- ABISound record whose fields are the key proven facts of each layer:
  flagship postcondition (maxPositive), algebraic commutativity invariant
  (commPositive), min/max duality (genMaxMinSum), and FFI-seam injectivity
  (resultToIntInjective).
- abiContractDischarged : ABISound, built solely from those exported
  witnesses — if any prior layer were unsound it would not typecheck.

Ties manifest -> ABI proofs (flagship + invariants) -> FFI seam into a
single end-to-end statement. Genuine composition only: no believe_me,
postulate, assert_total, sorry, or %hint hacks. %default total, SPDX
header, zero warnings. Appended to dafniser-abi.ipkg (last module).
Adversarially verified: a false certificate (bogus flagship field) is
rejected by the type checker.

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
@hyperpolymath hyperpolymath marked this pull request as ready for review June 28, 2026 09:23
@hyperpolymath hyperpolymath merged commit f5bc8f9 into main Jun 28, 2026
20 of 21 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