Skip to content

Refactor temporary execution paths - #9

Merged
timzhong1024 merged 3 commits into
mainfrom
refactor/reclaim-launch-paths
Aug 22, 2026
Merged

Refactor temporary execution paths#9
timzhong1024 merged 3 commits into
mainfrom
refactor/reclaim-launch-paths

Conversation

@timzhong1024

Copy link
Copy Markdown
Owner

Summary

  • unify temporary project launch paths across Node, Python, Rust, and .NET
  • collapse duplicate process result and error representations
  • remove forwarding-only backend constructors and test-only display helpers
  • remove the serde_json runtime dependency used only for a static manifest
  • move CommonJS validation to the CLI boundary

Impact

  • 132 additions and 360 deletions relative to main
  • no user capability removed
  • Node now displays its explicit vp env exec launch command
  • invalid --commonjs combinations fail before source input is read

Validation

  • cargo fmt --all -- --check
  • cargo test --locked (30 passed)
  • cargo clippy --locked --all-targets -- -D warnings
  • cargo package --locked --allow-dirty
  • node --check bin/run-code.js
  • npm pack --dry-run
  • real dependency smoke tests for Node, Python, Rust, Go, and .NET
  • cwd, env-file, quiet mode, argument forwarding, exit codes, and missing-tool hints verified

@timzhong1024
timzhong1024 merged commit a34904d into main Aug 22, 2026
4 checks passed
@timzhong1024
timzhong1024 deleted the refactor/reclaim-launch-paths branch August 22, 2026 16:03
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