Skip to content

Fallback to npx wrapper script on windows#77

Open
Perlmint wants to merge 1 commit into
dodok8:mainfrom
Perlmint:fix/npx_on_windows
Open

Fallback to npx wrapper script on windows#77
Perlmint wants to merge 1 commit into
dodok8:mainfrom
Perlmint:fix/npx_on_windows

Conversation

@Perlmint

Copy link
Copy Markdown

Summary

On Windows, npx may not be invocable directly — only npx.ps1 or npx.cmd may be on PATH. This PR probes all three in order on Windows and caches the result. Non-Windows uses "npx" as a compile-time constant.

Changes

  • src/builder.rs: Added #[cfg(windows)] / #[cfg(not(windows))] get_npx() helper; refactored execute_workflow_npx to use it
  • README.md, docs/reference/cli.md, docs/ko/reference/cli.md: Note Windows fallback behavior

Related Issues

Fixes #74

Checklist

  • cargo test --all-features passes
  • cargo clippy --all-targets --all-features -- -D warnings passes
  • cargo fmt --all --check passes
  • Updated documentation if needed
  • Added tests for new functionality

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.

Windows npx command resolution failure breaks tsx/ts-node fallback

1 participant