Skip to content

fix: make CI case results reliable and restore ARM64 validation - #630

Merged
LunaStev merged 6 commits into
wavefnd:masterfrom
LunaStev:fix/ci-case-results-and-arm64
Sep 12, 2026
Merged

fix: make CI case results reliable and restore ARM64 validation#630
LunaStev merged 6 commits into
wavefnd:masterfrom
LunaStev:fix/ci-case-results-and-arm64

Conversation

@LunaStev

@LunaStev LunaStev commented Sep 12, 2026

Copy link
Copy Markdown
Member

Case execution currently lets a compiler error satisfy an expected nonzero program exit, while the all-target precheck stops at the first error and prevents native results from being collected. Windows ARM64 also fails before meaningful runtime validation: implicit-target Cargo builds misclassify SDK import libraries, and debug module resolution exhausts a small process stack.

This PR contains one commit per issue:

Validation:

  • Rust workspace/all-target tests: 196 passed; fmt, Clippy with denied warnings, rustdoc with denied warnings, and std policy passed.
  • Python discovery: 63 tests, 62 passed and one Windows-only test skipped on Linux.
  • CI-matching Rust 1.89: all 20 frontend/runtime tests passed, including the 1 MiB stack regression.
  • Wave corpus: 277 passed; std examples: 26 passed.
  • Source precheck: all 605 sources passed after removing the stale import.
  • The repaired linux/amd64/test13.wave executes successfully locally. Full-suite testing on this machine is affected by the existing server-port isolation issue Require the server case to validate its own generated process #580 (another service uses port 8080).
  • Native Windows ARM64 build job: passed before the isolated Linux import correction, including Clippy, LLVM allocator ownership, MSVC companion publication and actual executor restart execution (sockets_survive_executor_restart_and_release_their_completion_port).
  • Native ARM64 cases job: before the isolated Linux import correction, compiler build and architecture verification passed, confirming that the psapi failure is resolved; all 135 native cases passed with zero failures/timeouts and the report was uploaded.
  • CI evidence for Collect all case precheck failures without suppressing native execution and reports #610: the preceding head ran all 148 native Linux cases despite the precheck failure and uploaded both reports. Its only failure was the stale import now repaired by Repair the stale process helper import in the Linux amd64 case suite #607; the new head reruns this required job.

Fixes #577
Fixes #610
Fixes #608
Fixes #609

Fixes #607

CI follow-up: the macOS ARM64 Python process-tree timeout test once returned a failure classification instead of the expected timeout, but the assertion discarded the captured subprocess output. The assertion now includes that output and the classification detail without changing timeout or pass/fail behavior. The macOS ARM64 job, including Python tooling and Rust validation, passes on rerun; the original intermittent failure's cause has not been identified, so this is diagnostic hardening rather than a claimed fix for a proven subprocess cause.

Fixes wavefnd#577

Signed-off-by: LunaStev <luna@lunastev.org>
Fixes wavefnd#610

Signed-off-by: LunaStev <luna@lunastev.org>
Use the same explicit Cargo target as the working native build and release paths. Preserve static CRT allocator ownership and stage the verified compiler for the existing case runner.

Fixes wavefnd#608

Signed-off-by: LunaStev <luna@lunastev.org>
Replace recursive owned expression reconstruction with an in-place worklist while preserving traversal and diagnostic order. Regress the executor import graph under a 1 MiB process stack and retain native crash inputs.

Fixes wavefnd#609

Signed-off-by: LunaStev <luna@lunastev.org>
@LunaStev LunaStev added bug A problem that causes incorrect behavior or crashes. ci GitHub CI or CI labels Sep 12, 2026
Fixes wavefnd#607

Signed-off-by: LunaStev <luna@lunastev.org>
Signed-off-by: LunaStev <luna@lunastev.org>
@LunaStev
LunaStev merged commit 291ec53 into wavefnd:master Sep 12, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug A problem that causes incorrect behavior or crashes. ci GitHub CI or CI

Projects

None yet

1 participant