Skip to content

feat: Verus proofs, code quality, decoder fixes, coverage --html#140

Merged
avrabe merged 1 commit intomainfrom
feat/parallel-improvements
Mar 1, 2026
Merged

feat: Verus proofs, code quality, decoder fixes, coverage --html#140
avrabe merged 1 commit intomainfrom
feat/parallel-improvements

Conversation

@avrabe
Copy link
Collaborator

@avrabe avrabe commented Mar 1, 2026

Summary

  • Verus StaticString proofs: 20 verified proofs for BoundedString (total: 50 across StaticVec + StaticQueue + StaticString)
  • Code quality: panic! macros reduced 87→24 (72%), .unwrap() reduced 486→435, via try_new() + ? propagation
  • Decoder bug fixes: Fixed 3 real bugs (lazy detection layer encoding, unified loader component detection, streaming validator config) + 2 test fixes
  • Coverage --html: Full llvm-cov pipeline implementation (profraw→profdata→lcov→HTML)
  • SIMD NaN canonicalization: Correct WebAssembly canonical NaN handling for f32x4/f64x2 ops
  • Multi-memory: Enable WebAssembly 3.0 multi-memory proposal support
  • Custom section validation: UTF-8 name validation per WebAssembly spec

Test plan

  • cargo build --workspace --lib --bins — clean build
  • cargo test --lib across all modified crates — 99 passed, 0 regressions
  • bazel test //kiln-foundation/src/verus_proofs:static_string_verify — 20 verified, 0 errors
  • All 3 Verus proof suites pass (static_vec: 13, static_queue: 17, static_string: 20)
  • CI green

🤖 Generated with Claude Code

…ixes, coverage --html

- Add 20 verified Verus proofs for StaticString (total: 50 across 3 structures)
- Reduce panic! macros from 87 to 24 (72% reduction) via expect() and try_new()
- Reduce .unwrap() calls from 486 to 435 via proper error propagation with ?
- Fix 5 kiln-decoder test failures (3 real bugs in lazy_detection/unified_loader)
- Implement cargo-kiln coverage --html with full llvm-cov pipeline
- Add NaN canonicalization for SIMD f32x4/f64x2 operations
- Enable multi-memory proposal support (WebAssembly 3.0)
- Add custom section UTF-8 name validation per spec

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Mar 1, 2026

🔍 Build Diagnostics Report

Summary

Metric Base Branch This PR Change
Errors 0 0 0
Warnings 2 2 0

🎯 Impact Analysis

Issues in Files You Modified

  • 0 new errors introduced by your changes
  • 0 new warnings introduced by your changes
  • 0 total errors in modified files
  • 0 total warnings in modified files
  • 0 files you modified

Cascading Issues (Your Changes Breaking Other Files)

  • 0 new errors in unchanged files
  • 0 new warnings in unchanged files
  • 0 unchanged files now affected

Note: "Cascading issues" are errors in files you didn't modify, caused by your changes (e.g., breaking API changes, dependency issues).

✅ No Issues Detected

Perfect! Your changes don't introduce any new errors or warnings, and don't break any existing code.


📊 Full diagnostic data available in workflow artifacts

🔧 To reproduce locally:

# Install cargo-kiln
cargo install --path cargo-kiln

# Analyze your changes
cargo-kiln build --output json --filter-severity error
cargo-kiln check --output json --filter-severity warning

@avrabe avrabe merged commit ada840a into main Mar 1, 2026
16 checks passed
@avrabe avrabe deleted the feat/parallel-improvements branch March 1, 2026 15:14
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.

1 participant