Fix storybook configuration issue preventing it from running locally

[brentswisher]

This change: (check at least one)

• Adds a new feature
• Fixes a bug
• Improves maintainability
• Improves documentation
• Is a release activity

Is this a breaking change? (check one)

• Yes
• No

Is the: (complete all)

• Title of this pull request clear, concise, and indicative of the issue number it addresses, if any?
• Test suite(s) passing?
• Code coverage maximal?
• Changeset added?

What does this change address?
Closes #1220

How does this change work?
We had added @vitejs/plugin-react in the optimizeDeps > include block of the storybook vite config when we updated to Storybook v8.

Vite OptimzeDeps is a dev only setting which lists dependencies to pre-bundle.

It looks like @vitejs/plugin-react 5.x introduced a dynamic import call in its source to vite/internal. When @vitejs/plugin-react is listed in optimizeDeps.include, Vite's pre-bundler attempts to statically analyze and bundle the plugin itself, hitting this unresolvable specifier and crashing.

Removing @vitejs/plugin-react form the optimizeDeps config resolves this. If I remember correctly, the only reason it was added was that Storybook wasn't rendering locally without it at the time, so I don't see any reason we shouldn't remove it.

[changeset-bot]

⚠️ No Changeset found

Latest commit: d4720f9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

size-limit report 📦

Path	Size
packages/pharos/lib/index.js	1.04 MB (0%)