fix(components): move react-syntax-highlighter types to dev deps

[iamandrewluca]

Discussion: vuejs/language-tools#592

Using @storybook/vue JSX Runtime types and Volar extension conflicts with @types/react from Storybook

❯ npm ls @types/react
[email protected] /Users/iamandrewluca/Projects/my-project
└─┬ @storybook/[email protected]
  └─┬ @storybook/[email protected]
    └─┬ @storybook/[email protected]
      └─┬ @types/[email protected]
        └── @types/[email protected]

Closes #12505

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit ad02c64. 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 1 target

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

---

Sent with 💌 from NxCloud.

[yannbf]

Hey @iamandrewluca thanks a lot for your contribution! This is a quite tricky one, @ndelangen could you assist? Thanks!

[ndelangen]

This seems the same as #18425

[iamandrewluca]

@ndelangen cool! Then this PR can be closed?! 🙂

[shilman]

@iamandrewluca thanks so much for your patience on this & the original contribution. it looks like the contents of this PR are already contained in next so i'm closing it now.

[mockingjet]

Hi, @types/react is still installed, and btw I use pnpm

"devDependencies": {
    "@babel/core": "^7.18.10",
    "@storybook/addon-actions": "^6.5.12",
    "@storybook/addon-essentials": "^6.5.12",
    "@storybook/addon-interactions": "^6.5.12",
    "@storybook/addon-links": "^6.5.12",
    "@storybook/builder-vite": "^0.2.2",
    "@storybook/mdx2-csf": "^0.0.3",
    "@storybook/testing-library": "^0.0.13",
    "@storybook/vue3": "^6.5.12",
    "babel-loader": "^8.2.5",
    "vue-loader": "^16.8.3"
}

[iamandrewluca]

@mockingjet please read the conversation above

[mockingjet]

I read it, but I don't get it. v6.5.12 has this fix, right? I still get @types/react installed

[iamandrewluca]

It was fixed in v7 only. I think v6 won't get the fix 👀

[mockingjet]

Cool. I thought this fix was merged in May, it should be in the current version. Never thought of next branch means v7. Thanks

[Timakra]

Can we cherry pick this fix to have it fixed in 6.5.* too and not only v7

[noamori25]

I think it will be great to cherry-pick this fix also to 6.5 version.

[mockingjet]

I pernoally hope the v7 released beta as soon as possible.
And I do feel like the team take a negative attitude towards v6, which I think it's a good thing meaning that they can put more efforts on v7-beta

[yuval-agritask]

Hi all,
Thanks to everyone for their contribution, and this library, my team and I enjoy using it mostly.
Quite recently we upgraded to version 6 of Storybook since we upgraded to Vue 3.
We were very sad to see that we've tons of issues in our code due to React types appearing in normal dependencies. We see a lot of warnings and it really distracts us from our main efforts.
There are a lot of other people who also ran into the same issues, see: #12505
vuejs/language-tools#592

It's quite unfortunate that using Storybook, hurts so much the development experience on your actual code, not even the Storybook files, while it supposes to be the opposite to improve your Developer experience.

Since the PR is just moving the dependency from a normal to dev one, can't we just cherry pick it and release a hotfix? @shilman @ndelangen

I really hope you consider it. Since version 7 is in Alpha which means it's still very raw, and the previous upgrade we did for Storybook was quite painful, hopefully, we won't need to wait so much time and go through a major upgrade just for this small fix.

[shilman]

@yuval-agritask Sorry to hear about the upgrade headaches. I'll discuss with @ndelangen and see whether we can do anything here.

[alexandrkrucheniuk]

also faced the issue described by @yuval-agritask