Skip to content

fix(npx): use npx instead of npm for generic passthrough#819

Open
soof-golan wants to merge 1 commit intortk-ai:developfrom
soof-golan:fix/npx-generic-passthrough-runs-npm
Open

fix(npx): use npx instead of npm for generic passthrough#819
soof-golan wants to merge 1 commit intortk-ai:developfrom
soof-golan:fix/npx-generic-passthrough-runs-npm

Conversation

@soof-golan
Copy link

Summary

  • rtk npx fallback for unrecognized commands called npm_cmd::run(), which executes npm instead of npx
  • Replaced with a direct npx passthrough, matching the existing prisma passthrough pattern

Fixes #815

The Npx command's default match arm incorrectly called npm_cmd::run(),
which executes `npm` instead of `npx`. This caused all non-routed npx
commands to fail (e.g. `rtk npx cowsay hello` → "Missing script").

Fixes rtk-ai#815
@CLAassistant
Copy link

CLAassistant commented Mar 25, 2026

CLA assistant check
All committers have signed the CLA.

@aeppling
Copy link
Contributor

Hey

We are cleaning up the codebase and improving the project structure for better onboarding. As part of this effort, PR #826 reorganizes src/ from a flat layout into subfolders.

No logic changes — only file moves and import path updates.

What you need to do

Rebase your branch on develop when receiving this comment:

git fetch origin && git rebase origin/develop

Git detects renames automatically. If you get import conflicts, update the paths:

use crate::git;        // now: use crate::cmds::git::git;
use crate::tracking;   // now: use crate::core::tracking;
use crate::config;     // now: use crate::core::config;
use crate::init;       // now: use crate::hooks::init;
use crate::gain;       // now: use crate::analytics::gain;

Need help rebasing? Tag @aeppling

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