fix(core): run-commands should not parse strings as numbers

[Coly010]

Current Behavior

When passing command line args to a target using nx:run-commands whose value resembles a number, it is being parsed as a number.

This can cause erroneous behaviour when the intention of the value is to be a string.

The main potential of erroneous behaviour here comes from decimal numbers ending in 0, such as 1234.10. When this is parsed as a number then converted back to a string for interpolation, this value becomes 1234.1.

Expected Behavior

Args that are passed to the nx:run-commands via command line should not be parsed as numbers.

We should leave support args that are passed as options in the project.json

Related Issue(s)

Fixes #17575

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 19, 2023 2:48pm

[nx-cloud]

☁️ Nx Cloud Report

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

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• nx affected -t e2e-macos --parallel=1 --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222
• nx affected --target=test --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222 --parallel=1
• nx affected --target=e2e --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222 --parallel=1
• nx run-many -t check-imports check-commit check-lock-files depcheck check-codeowners documentation --parallel=1 --no-dte
• nx affected --target=build --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222 --parallel=3
• nx affected --target=lint --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222 --parallel=3
• nx-cloud record -- nx format:check --base=1fda800b332af491e14f2d5d213d63af952405aa --head=dd830951ad52880ca8d8f86bb1f0054fdbe47222
• 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.