chore: remove AUDIT.md, update docs for Storybook as core#2
Merged
Conversation
added 14 commits
March 7, 2026 01:54
- Remove AUDIT.md (served its purpose) - README: add Storybook to stack table, commands, project structure, and a new 'What's Included' section with story example - README: remove stale recipes/storybook/ reference - CLAUDE.md: update recipe list (storybook → storybook-deploy) - recipes/README.md: same - skills/recipe-install: update available recipes table
- Avatar, List, Table, Timestamp → src/components/ui/ (from recipes/data-display) - Card, Header, Main, Section, Sidebar → src/components/ui/ (from recipes/layout) - Forms recipe → docs/forms.md (was just a usage pattern, deps already core) - All new components exported from ui barrel - 23 core components total (was 14) - Updated recipe-install skill
Pickers (ColorPicker, IconPicker) stay as a recipe — they pull external deps and are domain-specific. DangerZone is a generic settings pattern.
Remove forwardRef from template and rules. React 19 passes ref as a regular prop — forwardRef is legacy and will be removed in a future React version.
- AGENTS.md: forwardRef → ref-as-prop in component checklist - recipe-install skill: remove forms, add pickers, fix blank table row - CLAUDE.md: add pickers to recipe list - recipes/README.md: add pickers - README.md: update project structure recipe list
Fix DangerZone.stories.tsx import: @storybook/test → storybook/test (Storybook 10 moved test utils to storybook/test subpath export).
anulman
force-pushed
the
chore/docs-cleanup
branch
from
69506e4 to
73b5d74
Compare
Include test/ in tsconfig.test.json so jest-dom type augmentation is picked up for .toBeInTheDocument() etc.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Changes
Removed
AUDIT.md— served its purposerecipes/markdown/— duplicate ofrecipes/authoring/recipes/posthog/— duplicate ofrecipes/analytics/recipes/forms/— moved todocs/forms.md(deps were already core)createAPIFileRouteTODOs from API routesPromoted to core UI (14 → 24 components)
recipes/data-display/)recipes/layout/)recipes/pickers/— misplaced, generic settings pattern)Storybook
styles.cssmove@storybook/testas explicit devDepDocs updated across the board
forwardRef→ React 19 ref-as-propHousekeeping
src/styles/global.css→src/styles.csssrc/test/setup.ts→test/setup.ts