-
Notifications
You must be signed in to change notification settings - Fork 52
Open
Description
Description
The src/runtime/executor.rs file has grown too large and handles too many responsibilities. Refactor it into focused sub-modules so it is easier for contributors to navigate and extend.
Requirements
- Split into:
loader.rs(WASM loading),invoker.rs(function invocation),parser.rs(arg parsing),result.rs(result formatting) - Keep public API identical so no other code breaks
- Add module-level doc comments explaining each sub-module's responsibility
- All existing tests must still pass after refactor
Suggested Execution
- Branch:
git checkout -b refactor/executor-split - Create
src/runtime/subdirectory with new files - Move relevant code into each new file
- Update
src/runtime/mod.rsto re-export public items - Run
cargo testto confirm nothing broke - Commit:
refactor: split executor into focused sub-modules
Acceptance Criteria
- Four new sub-modules created
- Public API unchanged
- All existing tests pass
- Each module has a doc comment
- No dead code warnings introduced
Reactions are currently unavailable