Docs: Remove STORYBOOK_REACT_CLASSES and global/globals.ts

[IanVS]

Issue:

What I did

I could be wrong, but it seems like STORYBOOK_REACT_CLASSES isn't actually used anywhere. Though, I found where it was added: #557, and it didn't really seem used there either, so maybe I'm missing something. But, it's adding a bit of extra complexity since this is injected into config via

storybook/code/lib/core-server/src/presets/common-preset.ts

Line 96 in 1041236

return [...config, require.resolve('@storybook/core-client/dist/esm/globals/globals'), ...base];

. The vite builder's docgen plugin does not use this global, and things seem to be working fine.

In the name of simplifying things, I figured I'd try removing it, and see what happens. But maybe users are relying on this, somehow?

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?

Maybe an update to the docs?

[shilman]

This feels good to me. It only existed for back compat in the early days of docs. If we make this change, we'll need to add a note to MIGRATION.md

[github-actions]

	Fails
🚫	PR is not labeled with one of: ["cleanup","BREAKING CHANGE","feature request","bug","documentation","maintenance","dependencies","other"]

Generated by 🚫 dangerJS against bba5c8c