Uh oh!
There was an error while loading. Please reload this page.
docs(ui): move non-config files out of .storybook directory - #2597
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughStorybook config updated to reference a specific welcome MDX file; several page stories now import MSW handlers from Changes
Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
cylewaitforit
commented
Apr 20, 2026
@coderabbitai review |
✅ Actions performedReview triggered.
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.storybook/main.ts:
- Around line 8-9: The explicit welcome entry
'../app/design-docs/storybook-welcome.mdx' is being matched again by the
catch-all '../app/**/*.@(mdx|stories.@(js|ts))'; update the catch-all glob to
exclude that specific file using picomatch negation so the file is only indexed
once—for example replace the second pattern with a negated form that omits
'design-docs/storybook-welcome' (i.e. change
'../app/**/*.@(mdx|stories.@(js|ts))' to a pattern using !(...):
"../app/**/!(design-docs/storybook-welcome).@(mdx|stories.@(js|ts))" while
keeping the explicit '../app/design-docs/storybook-welcome.mdx' entry for
sidebar ordering.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 05983184-3b2e-4faa-b43b-2f122b410089
📒 Files selected for processing (10)
.storybook/main.tsapp/design-docs/colors.mdxapp/design-docs/guidelines.mdxapp/design-docs/storybook-welcome.mdxapp/design-docs/typography.mdxapp/mocks/handlers.tsapp/pages/about.stories.tsapp/pages/pds.stories.tsapp/pages/recharging.stories.tsapp/pages/settings.stories.ts
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
🔗 Linked issue
🧭 Context
This PR moves non-config files that could change more frequently out of the
.storybookdirectory.📚 Description
Moved files that aren't part of the global storybook configs into
app/storybookdirectory.Moved the MSW handlers into
mocksdirectory more closely following the suggestions from https://mswjs.io/docs/best-practices/structuring-handlers/.