Skip to content

Conversation

@mohanson
Copy link
Collaborator

No description provided.

@mohanson mohanson requested a review from Copilot September 25, 2025 03:43
Copy link

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

This pull request adds a main entry point for the pywasm package to enable execution via python -m pywasm, consolidating functionality from separate scripts and adding comprehensive command-line interface features.

  • Adds __main__.py module with support for both WASI and regular WebAssembly function execution
  • Consolidates and removes standalone WASI execution scripts
  • Updates type annotations to support bytearray arguments in the Runtime.invocate method

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
pywasm/main.py Main entry point with argument parsing for WASI and function execution modes
pywasm/core.py Updates type annotations to include bytearray support in invocate method
test/main.py Test script demonstrating the new main entry functionality
script/wasi_testsuite_adapter.py Removed - functionality consolidated into main module
script/wasi.py Removed - functionality consolidated into main module
README.md Updates documentation to include new test command
.github/workflows/develop.yml Adds new test to CI workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mohanson mohanson merged commit ee4bc0e into master Sep 25, 2025
6 of 9 checks passed
@mohanson mohanson deleted the update branch September 25, 2025 04:29
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