fix(core): nx build --graph file.json should work in subdirectories

[AgentEnder]

Current Behavior

We rewrite argv to support "infix" target notation (e.g. nx build my-app).

Expected Behavior

We don't rewrite infix notation before it reaches yargs, as yargs handles parsing things in a more thorough manner. This eliminates some messy areas in our codebase, but also makes sure that flags passed into commands are interpreted correctly in the presence of positionals. Specifically, this fixes the following cases:

nx build --graph stdout runs the build script for the project that you are inside of instead of looking for a project named stdout

nx build --graph stdout mypkg would be odd to run, but would run the build script for mypkg

nx run-many -t test,build runs test + build

nx run-many -t test build runs test + build

Related Issue(s)

Fixes #

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
nx-dev	⬜️ Ignored (Inspect)	Visit Preview	Jan 30, 2024 9:57pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 630ce89. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

🟥 Failed Commands
nx affected --target=e2e --base=8ec589d0b90c13fe02dc6f09497af3cefde9e650 --head=630ce89113894c22ab0a78c998e9783a612acb4d --parallel=1

✅ Successfully ran 5 targets

• nx affected --targets=lint,test,build --base=8ec589d0b90c13fe02dc6f09497af3cefde9e650 --head=630ce89113894c22ab0a78c998e9783a612acb4d --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=8ec589d0b90c13fe02dc6f09497af3cefde9e650 --head=630ce89113894c22ab0a78c998e9783a612acb4d
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=8ec589d0b90c13fe02dc6f09497af3cefde9e650 --head=630ce89113894c22ab0a78c998e9783a612acb4d

---

Sent with 💌 from NxCloud.

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7d50aa2. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

🟥 Failed Commands
nx affected -t e2e-macos --parallel=1 --base=deb0f1a49294eafe18c407a96c95e40af15d3738 --head=7d50aa2c9e3cc5ec347409877b5be9be41edf5d9

✅ Successfully ran 5 targets

• nx affected --target=e2e --base=deb0f1a49294eafe18c407a96c95e40af15d3738 --head=7d50aa2c9e3cc5ec347409877b5be9be41edf5d9 --parallel=1
• nx affected --targets=lint,test,build --base=deb0f1a49294eafe18c407a96c95e40af15d3738 --head=7d50aa2c9e3cc5ec347409877b5be9be41edf5d9 --parallel=3
• nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=deb0f1a49294eafe18c407a96c95e40af15d3738 --head=7d50aa2c9e3cc5ec347409877b5be9be41edf5d9
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.