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.
📝 変更情報
なぜこれをやるのか
現行の Storybook は CSF3.0 を推奨している。Storybook の恩恵を最大化するためにもこれに準拠するのが望ましい。
何を変更したのか
Docs
ページに表示されるよう修正した。技術的にはどこがポイントか
CSF3.0
default export
するオブジェクトにはMeta
型を明記する。StoryObj
型を明記する。-
args
-
argTypes
-
render
render
に渡す引数は無名関数のため、そのままだと ESLint ルールに抵触してフックが使えない。これを解消するため、*.stories.tsx
においてはreact-hooks/rules-of-hooks
ルールを無効化した。JSDoc
monorepo 配下にある全パッケージの UI コンポーネントを1つの Storybook に集約しようとすると、各 UI コンポーネントの JSDoc が適切に読み込まれず
Docs
ページに表示されない。これを解消するため、.storybook/main.js
に以下の設定を追加した。これにより現状の monorepo 構成であっても JSDoc が Storybook にて適切に表示されるようになる。
どこまで動作確認したか
備考
🏛️ 参考文献
pnpm
+vite
Mono-Repository Test Rig — Cases of No JSDocs, No new file HMR, Recursion, Breaking · Issue #21399 · storybookjs/storybook🚧 TODO リスト / 未完了タスク
📸 スクリーンショットや Movie ( 画面遷移、導線変更など ) 🎥