fix(core): target defaults from specifier should not be clobbered by name based target defaults

[AgentEnder]

Current Behavior

Its possible for a target default defined as "build" to overwrite the defaults defined by "@nx/js:tsc".

Expected Behavior

Executor based defaults have priority

Related Issue(s)

Fixes #21526
Fixes #21477

[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	Feb 2, 2024 4:03pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 83ccbf1. 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

---

✅ Successfully ran 6 targets

• nx affected --target=e2e --base=7c231e34fb700ed07f348a4af4b4066d4060443c --head=83ccbf12106e968bfa9e02cdddb8b677362d70b6 --parallel=1
• nx affected --targets=lint,test,build --base=7c231e34fb700ed07f348a4af4b4066d4060443c --head=83ccbf12106e968bfa9e02cdddb8b677362d70b6 --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=7c231e34fb700ed07f348a4af4b4066d4060443c --head=83ccbf12106e968bfa9e02cdddb8b677362d70b6
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=7c231e34fb700ed07f348a4af4b4066d4060443c --head=83ccbf12106e968bfa9e02cdddb8b677362d70b6

---

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.