Skip to content

harness: correct Wasmtime instantiation verdicts and diagnostics - #379

Merged
lannbot merged 1 commit into
mainfrom
investigate/wasmtime-test-failures
Sep 15, 2026
Merged

lannbot merged 1 commit into
mainfrom
investigate/wasmtime-test-failures

Conversation

@lannbot

@lannbot lannbot commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Correct instantiation verdicts: recognize native start traps and narrowly typed host-resource import mismatches; reject wrong-phase errors and check link diagnostics.
  • Report actual variant discriminants and case counts in memory/flat lifting. The Wasmtime assertion passes through ordinary matching, with no file/revision-specific exception.
  • Recognize verified resource-borrow and string-alignment diagnostic equivalents; remove seven stale failure expectations.
  • Record declared-import checking constraints in the plan contract's open items. Full declared-import checking remains follow-up work under testing: triage classified Wasmtime supplementary coverage gaps #372.

Verification

  • just gates < /dev/null passed, including browser realms, consumer smokes, packaged npm tests, and scheduler seeds.
  • Wasmtime FIFO and seed 1: 614 passes, 62 known failures, 174 excluded commands, zero unexpected failures (previously 607 passes / 69 known failures).
  • Official conformance: 1,468 passes, 38 expected failures, five unsupported directives; unchanged.
  • Regression coverage rejects wrong-phase native/translation failures and distinguishes discriminant 2 of 2 cases from 3 of 3 cases.
  • Consumer checkout statuses unchanged; independent review passed.

Automerge will be armed after opening this PR.

@lannbot
lannbot enabled auto-merge September 15, 2026 00:21
@lannbot
lannbot merged commit 403910e into main Sep 15, 2026
4 checks passed
@lannbot
lannbot deleted the investigate/wasmtime-test-failures branch September 15, 2026 00:28
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