feat: new libraries/framework pages#872
Conversation
✅ Deploy Preview for tanstack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis pull request adds Alpine framework support to the TanStack library browser. Changes include introducing Alpine as a new framework option, creating a dynamic route for framework-specific library browsing, implementing utility functions for library filtering and ordering, updating the LibraryCard component to support framework prefixes, and extending framework metadata. Changes
Sequence DiagramsequenceDiagram
actor User
participant Router
participant FrameworkPage as Framework Page<br/>(libraries_.$framework)
participant Utils as Library Utils
participant UI as Component Render
User->>Router: Click framework link<br/>(/libraries/alpine)
Router->>FrameworkPage: Navigate & trigger loader
FrameworkPage->>FrameworkPage: Validate framework<br/>against frameworkOptions
FrameworkPage->>Utils: getVisibleLibraries()
Utils-->>FrameworkPage: Filtered library list
FrameworkPage->>Utils: getFrameworkLibraryCounts()
Utils-->>FrameworkPage: Count per framework
FrameworkPage->>Utils: orderLibrariesForBrowse()
Utils-->>FrameworkPage: Ordered libraries
FrameworkPage->>FrameworkPage: Filter by active<br/>framework
FrameworkPage->>UI: Render hero + nav +<br/>library grid
UI-->>User: Display framework page<br/>with filtered libraries
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
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 `@src/libraries/libraries.ts`:
- Around line 322-331: The Table description string is out of sync with the
frameworks list: you added 'alpine' to the frameworks array but didn't include
"Alpine" in the user-facing Table description; update the Table description
string (the card/table copy that references frameworks) to include "Alpine"
alongside Angular, React, Solid, Svelte, Vue, Qwik, Lit and Vanilla so the
displayed text matches the frameworks array (reference the frameworks array and
the Table description string in src/libraries/libraries.ts).
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 94758330-af8e-4fd1-8334-bcd7ba756a49
⛔ Files ignored due to path filters (1)
src/images/alpine-logo.svgis excluded by!**/*.svg
📒 Files selected for processing (9)
src/components/LibraryCard.tsxsrc/libraries/frameworks.tsxsrc/libraries/libraries.tssrc/libraries/types.tssrc/routeTree.gen.tssrc/routes/-libraries-utils.tssrc/routes/libraries.tsxsrc/routes/libraries_.$framework.tsxsrc/utils/npm-packages.ts

Summary by CodeRabbit
New Features
Chores