Skip to content

fix(pnpm): allow flag-first args to pass through#681

Open
TechWizard9999 wants to merge 1 commit intortk-ai:developfrom
TechWizard9999:codex/fix-pnpm-filter-args
Open

fix(pnpm): allow flag-first args to pass through#681
TechWizard9999 wants to merge 1 commit intortk-ai:developfrom
TechWizard9999:codex/fix-pnpm-filter-args

Conversation

@TechWizard9999
Copy link

Description

This fixes rtk pnpm argument parsing for flag-first invocations so RTK no longer fails before passthrough can run.
Commands like rtk pnpm --filter mymodule test are now routed to native pnpm safely.

Related Issue

Closes #259

Changes Made

  • Reworked rtk pnpm CLI parsing to accept raw pnpm args (instead of strict clap subcommands)
  • Added runtime routing for known optimized paths (list, outdated, install, typecheck)
  • Added explicit passthrough for flag-first pnpm forms and unsupported subcommands
  • Added regression tests for pnpm flag-first parsing and list-depth extraction
  • Updated changelog unreleased bug fixes

Testing

  • cargo fmt --all --check
  • rtk cargo clippy --all-targets
  • rtk cargo test --all
  • Manual reproduction:
    • rtk cargo run -- pnpm --filter mymodule test --help
    • Result now reaches pnpm (no RTK clap usage error)

@aeppling please review when you have a moment.

@TechWizard9999 TechWizard9999 force-pushed the codex/fix-pnpm-filter-args branch from fa9f944 to a60dba5 Compare March 19, 2026 04: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.

1 participant