Repository navigation
Claude / Claude Code Review
completed
Jul 25, 2026 in 32m 11s
Code review found 1 important issue
Found 2 candidates, confirmed 5. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 1 |
| 🟡 Nit | 4 |
| 🟣 Pre-existing | 0 |
| Severity | File:Line | Issue |
|---|---|---|
| 🟡 Nit | src/runtime/cli/mod.rs:894 |
npm --prefix / --pack-destination / --workspace need translation, not just drop or pass-through |
Annotations
Check warning on line 894 in src/runtime/cli/mod.rs
claude / Claude Code Review
npm --prefix / --pack-destination / --workspace need translation, not just drop or pass-through
Related to the `tag/access/otp/message` comment above but with a different fix shape: `prefix`, `pack-destination`, and `workspace` also have bun equivalents, but bun spells them differently (`--cwd`, `--destination`, `--filter`), so simply removing them from the drop list does *not* work — `npm --prefix ./client install` would then pass `./client` as a positional and try to `bun add ./client` into the current project. As-is, dropping them means `npm --prefix ./client install` silently runs `bun
Loading