-
-
Notifications
You must be signed in to change notification settings - Fork 35
Fix fumadocs theme issues #1347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates fumadocs packages to fix theme-related issues by migrating from version 15.x to 16.x. The changes include dependency updates, API migrations, and CSS restructuring to align with the new fumadocs architecture.
Key changes:
- Upgraded fumadocs packages (core, mdx, ui, twoslash, typescript, docgen) from v15.x/v12.x to v16.x/v14.x
- Updated ESLint from 9.39.1 to 9.39.2 and Effect packages across the monorepo
- Migrated from
tailwindcss-animatedtotailwindcss-animatefor Tailwind v4 compatibility - Refactored CSS structure by consolidating theme.css into app.css with fumadocs neutral theme
Reviewed changes
Copilot reviewed 21 out of 22 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml | Updated dependency versions and lock file for all package updates |
| apps/website/package.json | Updated fumadocs packages, replaced tailwindcss-animated with tailwindcss-animate, moved tailwindcss to dependencies |
| apps/website/tsconfig.json | Added TypeScript path mapping for fumadocs-mdx collections |
| apps/website/lib/source.ts | Migrated to new fumadocs-core API with multiple collections and lucide icons plugin |
| apps/website/app/app.css | Consolidated theme CSS, imported fumadocs neutral theme, changed plugin syntax |
| apps/website/app/layout.tsx | Updated RootProvider import path from fumadocs-ui/provider to fumadocs-ui/provider/next |
| apps/website/app/(home)/layout.tsx | Updated import path for LinkItemType to fumadocs-ui/layouts/shared |
| package.json | Updated effect and eslint versions, removed fumadocs version overrides |
| Multiple package.json files | Reordered peerDependencies to come after devDependencies (style change) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
Checklist
fix: remove a typo, closes #___, #___)Other information