do not ship @types/react when production to avoid type conflicts

[upupming]

Issue: do not ship @types/react when production, see #16629 (comment)

What I did

How to test

• Is this testable with Jest or Chromatic screenshots?
• Does this need a new example in the kitchen sink apps?
• Does this need an update to the documentation?

If your answer is yes to any of these, please make sure to include it in your PR.

[nx-cloud]

☁️ Nx Cloud Report

CI ran the following commands for commit bba91d3. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 1 target

• nx run-many --target=prepare --all --parallel --max-parallel=15

---

Sent with 💌 from NxCloud.

[shilman]

More discussion here vuejs/core#1033

[ndelangen]

We are working towards this goal in 7.0 in the future/base branch.

There's really not much we can do in 6.5 because removing the types make it unsound, and will break another set of users, unfortunately.

@upupming Thank you for the PR and helping work out what needs to happen.
As mentioned this will all be addressed and fixed in 7.0.

I'm sorry, but I have to close this PR, because to actually address this we can't simply move some items independencies to devDependencies, instead we need to change a lot of architecture (which is happening in the future/base branch!