Dependency issue#360
Conversation
fix: peer dependency conflicts
|
@iamsyg is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Caution Review failedThe pull request is closed. WalkthroughProject-wide animation library migration from motion/react to framer-motion across app pages and components. Minor typing tweak in Navbar. Toaster (Sonner) component simplified to rely on style props instead of per-toast classNames. package.json switches dev script and swaps dependencies (add framer-motion, remove motion, bump sonner). A couple of newline-only formatting changes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (52)
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 |
There was a problem hiding this comment.
Review by RecurseML
🔍 Review performed on cd41038..38b2570
✨ No bugs found, your code is sparkling clean
✅ Files analyzed, no issues (5)
• surfsense_web/components/ui/sonner.tsx
• surfsense_web/components/Navbar.tsx
• surfsense_web/app/contact/page.tsx
• surfsense_web/app/dashboard/[search_space_id]/chats/chats-client.tsx
• surfsense_web/app/dashboard/[search_space_id]/connectors/(manage)/page.tsx
⏭️ Files skipped (49)
| Locations |
|---|
surfsense_web/app/dashboard/[search_space_id]/connectors/[connector_id]/edit/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/[connector_id]/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/airtable-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/confluence-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/discord-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/github-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/google-calendar-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/google-gmail-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/jira-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/linear-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/linkup-api/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/luma-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/notion-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/serper-api/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/slack-connector/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/connectors/add/tavily-api/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsFilters.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/DocumentsTableShell.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/components/PaginationControls.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/(manage)/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/upload/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/documents/youtube/page.tsx |
surfsense_web/app/dashboard/[search_space_id]/logs/(manage)/page.tsx |
surfsense_web/app/dashboard/api-key/api-key-client.tsx |
surfsense_web/app/dashboard/page.tsx |
surfsense_web/app/dashboard/searchspaces/page.tsx |
surfsense_web/app/login/GoogleLoginButton.tsx |
surfsense_web/app/login/LocalLoginForm.tsx |
surfsense_web/app/login/page.tsx |
surfsense_web/app/onboard/page.tsx |
surfsense_web/app/pricing/page.tsx |
surfsense_web/app/register/page.tsx |
surfsense_web/components/ModernHeroWithGradients.tsx |
surfsense_web/components/chat/AnimatedEmptyState.tsx |
surfsense_web/components/contact/contact-form.tsx |
surfsense_web/components/onboard/add-provider-step.tsx |
surfsense_web/components/onboard/assign-roles-step.tsx |
surfsense_web/components/onboard/completion-step.tsx |
surfsense_web/components/pricing.tsx |
surfsense_web/components/search-space-form.tsx |
surfsense_web/components/settings/llm-role-manager.tsx |
surfsense_web/components/settings/model-config-manager.tsx |
surfsense_web/components/theme/theme-toggle.tsx |
surfsense_web/components/ui/spotlight.tsx |
surfsense_web/components/ui/tilt.tsx |
surfsense_web/package-lock.json |
surfsense_web/package.json |
surfsense_web/pnpm-lock.yaml |
Description
Motivation and Context
FIX #
Changes Overview
Screenshots
API Changes
Types of changes
Testing
Checklist:
High-level PR Summary
This Pull Request addresses a dependency issue by replacing the
motionpackage withframer-motion. The changes include updating import statements across multiple files frommotion/reacttoframer-motion, updating package dependencies in package.json, and making minor adjustments to accommodate the new package. Additionally, the PR updates the Sonner toast component to a newer version (2.0.7) and modifies its implementation. The primary focus of this PR is to fix dependency issues rather than introducing new features or functionality.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
surfsense_web/package.jsonsurfsense_web/package-lock.jsonsurfsense_web/pnpm-lock.yamlsurfsense_web/app/dashboard/[search_space_id]/chats/chats-client.tsxsurfsense_web/components/ui/sonner.tsxsurfsense_web/app/dashboard/[search_space_id]/connectors/(manage)/page.tsxsurfsense_web/components/Navbar.tsxsurfsense_web/components/ModernHeroWithGradients.tsx