Skip to content

fix(cli): resolve runner from prebuilt installs - #532

Merged
pgherveou merged 1 commit into
mainfrom
fix/prebuilt-runner
Aug 27, 2026
Merged

fix(cli): resolve runner from prebuilt installs#532
pgherveou merged 1 commit into
mainfrom
fix/prebuilt-runner

Conversation

@pgherveou

Copy link
Copy Markdown
Collaborator

Summary

  • resolve the packaged product-script runner through the installer's PATH and current symlinks
  • keep a running binary paired with its matching runner if a background update moves current
  • exercise a real product script through the installed entrypoint in the release E2E

Root cause

The release archive contained runner.js, but macOS reported the symlink used to launch truapi-host. Runner lookup checked beside that PATH entry, found no bundle, and fell back to the GitHub Actions build path compiled through CARGO_MANIFEST_DIR.

Verification

  • make e2e-cli-update
  • cargo test -p truapi-host-cli script_runner::tests
  • cargo test --workspace --all-features --all-targets
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo build --workspace
  • npm run test:scripts
  • cargo +nightly fmt --all -- --check
  • npx prettier --check scripts/e2e-cli-update.mjs

Created with assistance from OpenAI Codex.

@pgherveou
pgherveou marked this pull request as ready for review August 27, 2026 17:44
@pgherveou
pgherveou requested a review from a team August 27, 2026 17:44
@pgherveou
pgherveou added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit a0e308d Aug 27, 2026
17 checks passed
@pgherveou
pgherveou deleted the fix/prebuilt-runner branch August 27, 2026 21:23
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