Skip to content

ci: add build-wasm job to verify WebAssembly compilation#16305

Merged
Mossaka merged 1 commit intomainfrom
ci/build-wasm
Feb 17, 2026
Merged

ci: add build-wasm job to verify WebAssembly compilation#16305
Mossaka merged 1 commit intomainfrom
ci/build-wasm

Conversation

@Mossaka
Copy link
Collaborator

@Mossaka Mossaka commented Feb 17, 2026

Summary

  • Adds a new build-wasm job to the CI pipeline that verifies the WebAssembly compilation target (make build-wasm) succeeds
  • Reports the resulting .wasm binary size to GITHUB_STEP_SUMMARY for visibility
  • Follows the same patterns as existing CI jobs (pinned action SHAs, retry logic for module downloads, concurrency groups)

Test plan

  • Verify the build-wasm job appears in the CI workflow run
  • Confirm make build-wasm completes successfully in CI
  • Check that the binary size is reported in the GitHub Actions step summary

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 17, 2026 07:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds CI coverage for the project’s WebAssembly build target so CI will fail if make build-wasm stops compiling, and surfaces the produced .wasm size in the run summary.

Changes:

  • Introduces a new build-wasm CI job that runs make build-wasm on Ubuntu.
  • Adds Go cache reporting and Go module download/verify steps consistent with other Go jobs.
  • Appends the resulting gh-aw.wasm size to GITHUB_STEP_SUMMARY.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +654 to +663
if ls gh-aw.wasm 1>/dev/null 2>&1; then
SIZE=$(stat --format="%s" gh-aw.wasm)
SIZE_MB=$(awk "BEGIN {printf \"%.2f\", $SIZE/1048576}")
echo "✅ Build succeeded" >> $GITHUB_STEP_SUMMARY
echo "- **Binary:** gh-aw.wasm" >> $GITHUB_STEP_SUMMARY
echo "- **Size:** ${SIZE_MB} MB (${SIZE} bytes)" >> $GITHUB_STEP_SUMMARY
else
echo "❌ No .wasm binary found" >> $GITHUB_STEP_SUMMARY
ls -la *.wasm 2>/dev/null || echo "No wasm files in working directory" >> $GITHUB_STEP_SUMMARY
exit 1
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the failure branch, ls -la *.wasm writes to the job logs, not to $GITHUB_STEP_SUMMARY, so the step summary won’t include the directory listing when a .wasm exists but gh-aw.wasm is missing. Consider redirecting the listing output into the step summary as well (and using a file test like [ -f gh-aw.wasm ] instead of ls for existence checks).

Copilot uses AI. Check for mistakes.
@Mossaka Mossaka merged commit 672539b into main Feb 17, 2026
100 checks passed
@Mossaka Mossaka deleted the ci/build-wasm branch February 17, 2026 08:38
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