Skip to content

Completing task#1843

Merged
lpcox merged 2 commits intoclaude/go-fan-review-wazero-modulefrom
claude/66973980945-fix-github-actions-workflow-lint
Mar 13, 2026
Merged

Completing task#1843
lpcox merged 2 commits intoclaude/go-fan-review-wazero-modulefrom
claude/66973980945-fix-github-actions-workflow-lint

Conversation

@Claude
Copy link
Contributor

@Claude Claude AI commented Mar 13, 2026

Pull request created by AI Agent

Claude AI added 2 commits March 13, 2026 15:12
Replace deprecated InstantiateModuleFromBinary with InstantiateWithConfig
to fix lint error in GitHub Actions workflow.

The wazero API changed and InstantiateModuleFromBinary no longer exists.
Updated test to use runtime.InstantiateWithConfig with a ModuleConfig,
matching the pattern used in the production code (wasm.go:111).

Fixes: internal/guard/wasm_test.go:108
@Claude Claude AI assigned Claude and lpcox Mar 13, 2026
@lpcox lpcox marked this pull request as ready for review March 13, 2026 15:17
Copilot AI review requested due to automatic review settings March 13, 2026 15:17
@lpcox lpcox merged commit 85f78ca into claude/go-fan-review-wazero-module Mar 13, 2026
@lpcox lpcox deleted the claude/66973980945-fix-github-actions-workflow-lint branch March 13, 2026 15:18
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

Updates the WASM guard test suite to use the current wazero instantiation API, aligning tests with the runtime/module configuration pattern used elsewhere in the codebase.

Changes:

  • Replaces deprecated runtime.InstantiateModuleFromBinary usage with runtime.InstantiateWithConfig.
  • Introduces a wazero.ModuleConfig with an explicit module name for the blocking guard module used in context-cancellation testing.

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

You can also share your feedback on Copilot code review. Take the survey.

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.

3 participants