Skip to content

ci(go): validate WASM routing bindings - #1131

Draft
afourniernv wants to merge 2 commits into
codex/go-backend-reportingfrom
codex/go-review-06b-ci-wasm
Draft

afourniernv wants to merge 2 commits into
codex/go-backend-reportingfrom
codex/go-review-06b-ci-wasm

Conversation

@afourniernv

@afourniernv afourniernv commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

What

Add a separate WASM job to the Go validation workflow. Build the guest, run tests and vet with CGO_ENABLED=0, then run the WASM tests with Go race detection.

Why

The default WASM backend needs explicit coverage without cgo. Separate native and WASM jobs make failures easier to identify.

How tested

Checked the refreshed stack locally with Go 1.26.2 on macOS arm64:

  • Locked native release and WASI guest builds passed using the direct Cargo commands.
  • Go tests and vet passed for all four cgo/build-tag combinations. Cgo-enabled tests used race detection; native tests also used GOEXPERIMENT=cgocheck2 and a forced rebuild.
  • Both the missing-startup-export and cancellation-before-outcome regression tests passed.
  • Native builds exclude wazero and the embedded guest.
  • Actionlint, Ruff, mypy, and 177 Python tests passed. Python tests excluded integration tests and reused a native extension with matching Python/PyO3 sources.

Native links emitted local macOS deployment-target warnings; tests passed. No live provider calls or publishing. GitHub checks run separately.

Notes for reviewers

Stacked on #1147, which adds Go backend reporting. Still a draft.

The guest build commands are inlined in this workflow and documented in the README. Validation only: no artifact uploads, releases, or publishing. The Go version comes from go.mod.

Adapted from Nick DiZazzo's dual-engine contribution. His original PR and branch are unchanged.

@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from f5047f3 to 3c2f009 Compare October 23, 2026 17:14
@afourniernv
afourniernv force-pushed the codex/go-review-04b2-wasm-routing branch from 61db4a5 to beb73e4 Compare October 23, 2026 17:14
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from 3c2f009 to 8efff51 Compare October 26, 2026 16:36
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from 8efff51 to b8ddcf9 Compare October 26, 2026 17:07
@afourniernv
afourniernv changed the base branch from codex/go-review-04b2-wasm-routing to codex/go-backend-reporting October 26, 2026 17:08
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch from 56939c4 to c0874d1 Compare October 26, 2026 19:05
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch 2 times, most recently from 52441fa to 29fa359 Compare October 26, 2026 20:12
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch from c0874d1 to 25393da Compare October 26, 2026 20:12
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from 4cc16f3 to 62eaeaf Compare October 27, 2026 20:26
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch from 00b7f64 to cd21af7 Compare October 27, 2026 23:44
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from 62eaeaf to 7088a95 Compare October 27, 2026 23:44
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch from cd21af7 to 8d5347f Compare October 28, 2026 00:09
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch 2 times, most recently from 20b5a1b to 47fdd43 Compare October 28, 2026 00:40
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch 2 times, most recently from 2000106 to 39a777b Compare October 29, 2026 19:25
Signed-off-by: Alex Fournier <afournier@nvidia.com>
Signed-off-by: Alex Fournier <afournier@nvidia.com>
@afourniernv
afourniernv force-pushed the codex/go-backend-reporting branch from 39a777b to 420c27c Compare October 30, 2026 16:53
@afourniernv
afourniernv force-pushed the codex/go-review-06b-ci-wasm branch from 47fdd43 to 9157514 Compare October 30, 2026 16:53

This branch has not been deployed

No deployments
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