Release: Prerelease 10.2.0-alpha.6 #33332
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The third party action used in the documentation just provides a very thin wrapper around github's own actions, adding risk while providing little to no value.
…n types Fixes #12641 When TypeScript properties are optional (prop?: Type), TypeScript adds | undefined to the union type. The enum detection logic was failing because undefined is not a literal type, causing the union to not be recognized as an enum. This resulted in controls displaying as "object" type instead of "select" dropdown for optional properties with union types, regardless of the number of items in the union. The fix filters out undefined elements before checking if all remaining elements are literals, allowing optional unions to be correctly recognized as enums and display as select controls.
- Add type guard functions (isLiteral, isUndefined) for better type narrowing - Add 'undefined' to TSScalarType union in types.ts - Make elements non-optional in TSCombinationType (unions always have elements) - Wrap case block in braces to fix Biome linting error - Use proper type assertion instead of @ts-expect-error
- Separate TSLiteralType from TSScalarType to include 'value' property - Use proper TypeScript type guard syntax (type is X) for narrowing - Filter with type guard to get properly typed literal elements - Removes need for explicit type casting
- Add getDependencyVersion mock to package manager in tests - Split test into two scenarios: with and without vite installed - Test that vite@^7.0.0 is added when vite is not installed - Test that vite is not added when already installed Co-authored-by: valentinpalkovic <[email protected]>
- Add VITE_DEFAULT_VERSION constant in both implementation and tests - Add comment explaining explicit mock setup in test for clarity - Update test to use constant instead of hardcoded version Co-authored-by: valentinpalkovic <[email protected]>
- Export constant from main file to maintain single source of truth - Import constant in test file instead of duplicating - Remove redundant mock setup that was already done in beforeEach Co-authored-by: valentinpalkovic <[email protected]>
Add unit tests
…inks Docs: Fix migration guide links
Docs: Remove external action from Github pages documentation
…create-commands Docs: Add missing copyEvent on a create command
Controls: Force object control JSON mode to reset
github-actions
bot
added
the
release
…add-vite Nextjs-Vite: Install `vite` during migration if not installed yet
storybook-bot
force-pushed
the
version-non-patch-from-10.2.0-alpha.5
branch
from
5a79134 to
4ca6476
Compare
UI: Make vertical alignment of TestStatusIcon more robust
…-enum Controls: Fix displaying as object instead of select for optional union types
storybook-bot
force-pushed
the
version-non-patch-from-10.2.0-alpha.5
branch
from
4ca6476 to
5babe74
Compare
Docs-Blocks: Fix broken tooltip in ArgValue details
Manager: Ensure reset item only appears in globals toolbar when specified
storybook-bot
force-pushed
the
version-non-patch-from-10.2.0-alpha.5
branch
from
5babe74 to
9932826
Compare
valentinpalkovic
added
freeze
Package BenchmarksCommit: The following packages have significant changes to their size or dependencies:
|
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 191 | 191 | 0 |
| Self size | 75 KB | 75 KB | 0 B |
| Dependency size | 32.18 MB | 32.03 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/angular
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 191 | 191 | 0 |
| Self size | 118 KB | 118 KB | 0 B |
| Dependency size | 30.30 MB | 30.14 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/ember
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 195 | 195 | 0 |
| Self size | 15 KB | 15 KB | 🎉 -18 B 🎉 |
| Dependency size | 28.91 MB | 28.75 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/nextjs
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 537 | 537 | 0 |
| Self size | 645 KB | 645 KB | 🚨 +120 B 🚨 |
| Dependency size | 59.12 MB | 58.96 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/react-webpack5
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 277 | 277 | 0 |
| Self size | 24 KB | 24 KB | 0 B |
| Dependency size | 44.07 MB | 43.91 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/server-webpack5
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 203 | 203 | 0 |
| Self size | 16 KB | 16 KB | 0 B |
| Dependency size | 33.44 MB | 33.28 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/cli
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 173 | 173 | 0 |
| Self size | 774 KB | 775 KB | 🚨 +844 B 🚨 |
| Dependency size | 67.61 MB | 67.46 MB | 🎉 -156 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/codemod
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 166 | 166 | 0 |
| Self size | 30 KB | 30 KB | 🚨 +36 B 🚨 |
| Dependency size | 66.18 MB | 66.03 MB | 🎉 -156 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
@storybook/preset-react-webpack
| Before | After | Difference | |
|---|---|---|---|
| Dependency count | 170 | 170 | 0 |
| Self size | 18 KB | 18 KB | 🚨 +24 B 🚨 |
| Dependency size | 31.31 MB | 31.15 MB | 🎉 -160 KB 🎉 |
| Bundle Size Analyzer | Link | Link |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an automated pull request that bumps the version from
10.2.0-alpha.5to10.2.0-alpha.6.Once this pull request is merged, it will trigger a new release of version
10.2.0-alpha.6.If you're not a core maintainer with permissions to release you can ignore this pull request.
To do
Before merging the PR, there are a few QA steps to go through:
And for each change below:
This is a list of all the PRs merged and commits pushed directly to
next, that will be part of this release:viteduring migration if not installed yet #33316 (will also be patched)If you've made any changes doing the above QA (change PR titles, revert PRs), manually trigger a re-generation of this PR with this workflow and wait for it to finish. It will wipe your progress in this to do, which is expected.
Feel free to manually commit any changes necessary to this branch after you've done the last re-generation, following the Make Manual Changes section in the docs, especially if you're making changes to the changelog.
When everything above is done:
Generated changelog
10.2.0-alpha.6
viteduring migration if not installed yet - #33316, thanks ghengeveld!