fix(angular): ensure generated editor tsconfig in apps only include runtime files

[leosvelperez]

Current Behavior

Having an application with Jest and Cypress Component Tests setup can result in type conflicts surfaced by the IDE. There's no actual runtime issue.

This happens due to a couple of reasons:

• the tsconfig.editor.json includes src/**/*.ts, which would include runtime code, Jest spec test files, and Cypres CT test files
• the tsconfig.editor.json project reference in the project tsconfig.json is added after the other (more specific) tsconfig files, and therefore, it takes precedence over them

Expected Behavior

Having an application with Jest and Cypress Component Tests setup should show now type conflicts in the IDE.

The tsconfig.editor.json file's purpose is to be a sort of catch-all tsconfig file to include new runtime files added to the application source code. This is needed because Angular application normally don't include all runtime code. They instead have the entry point file (e.g. main.ts) in the files option of the tsconfig.app.json. That leaves out of the IDE program any new file that's not yet imported by other files part of the program. It's only meant to be used by the IDE, no task actually uses it.

Related Issue(s)

Fixes #19132

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

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 203e75a. 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=5d6abe437d15e71f8a58ce40f591216b1eb5ace6 --head=203e75a5379b145f513b9c5a9f492d7f6a0587cf --parallel=1
• nx affected --targets=lint,test,build --base=5d6abe437d15e71f8a58ce40f591216b1eb5ace6 --head=203e75a5379b145f513b9c5a9f492d7f6a0587cf --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=5d6abe437d15e71f8a58ce40f591216b1eb5ace6 --head=203e75a5379b145f513b9c5a9f492d7f6a0587cf
• nx documentation --no-dte
• nx affected -t e2e-macos --parallel=1 --base=5d6abe437d15e71f8a58ce40f591216b1eb5ace6 --head=203e75a5379b145f513b9c5a9f492d7f6a0587cf

---

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.