feat: complex control flow tests + platform/CM/Zephyr rivet artifacts#54
Merged
feat: complex control flow tests + platform/CM/Zephyr rivet artifacts#54
Conversation
Complex test case (#36): 17 end-to-end control flow tests covering nested blocks, loops with br_if exit, if/else, br_table dispatch, fibonacci, factorial, deeply nested blocks, loop re-entry, call in loop, early return, unreachable traps, and sequential loops. Rivet artifacts for full traceability: - component-model.yaml: 14 artifacts (CM-001..005 system-reqs, CM-TR-001..006 sw-reqs, CM-VER-001..003 verification) tracing WASI Component Model integration with kiln via BA RFC #46 - target-platforms.yaml: 8 artifacts (TP-001..008) capturing Cortex-M4F/M7/M33 targets, STM32F407/nRF52840 board profiles, vector tables, MPU, and Renode emulation platform - zephyr-integration.yaml: 10 artifacts (ZI-001..010) for Zephyr RTOS integration including CMake build, AAPCS compliance, memory domains, calculator app, PID demo, and linker scripts Closes #36 Implements: FR-002 Implements: FR-005 Trace: skip Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
component-model.yaml): canonical ABI compat with kiln, BA RFC safety: complete STPA steps 2-4 (control structure, UCAs, loss scenarios) #46 lowering, host intrinsic ABI, WASI P2 dispatch, async intrinsics — cross-linked tokiln:artifactstarget-platforms.yaml): Cortex-M4F/M7/M33 targets, STM32F407/nRF52840 board profiles, vector tables, MPU allocation, Renode emulation platformzephyr-integration.yaml): CMake build, AAPCS compliance, memory domains, calculator app (Create Zephyr calculator application #21), hardware testing (Test calculator on real hardware #22), PID demo, linker scripts, meld runtime integrationRivet coverage: 89.7% overall, 100% on all STPA/verification chains. All 18 remaining errors are external cross-refs (kiln:/sigil:) that resolve when externals are synced.
Test plan
cargo test --workspace— all tests pass (0 failures)cargo clippy --workspace --all-targets -- -D warnings— cleancargo fmt --check— cleanrivet validate— 0 local errorsrivet coverage— 89.7%🤖 Generated with Claude Code