fix(deps): replace dependency framer-motion with motion ^12.0.0#164
Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe project replaces its ChangesMotion dependency update
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@package.json`:
- Line 19: Update the imports in About, Contact, Experience, Hero, Works, and
SectionWrapper to use motion/react instead of framer-motion, preserving their
existing imported symbols. Regenerate the lockfile so the direct motion
dependency is reflected without the remaining transitive framer-motion coupling.
🪄 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: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 5e77d620-5e6d-45bd-b66c-f31052b0cc62
⛔ Files ignored due to path filters (1)
package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (1)
package.json
📜 Review details
⏰ Context from checks skipped due to timeout. (3)
- GitHub Check: Lint Code Base
- GitHub Check: Analyze (javascript-typescript)
- GitHub Check: Analyze (actions)
| "@react-three/drei": "^9.121.2", | ||
| "@react-three/fiber": "^8.17.12", | ||
| "framer-motion": "^12.0.0", | ||
| "motion": "^12.0.0", |
There was a problem hiding this comment.
🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '\n== package.json ==\n'
sed -n '1,120p' package.json
printf '\n== package-lock.json references ==\n'
rg -n '"framer-motion"|"motion/react"|"motion"' package-lock.json package.json src || true
printf '\n== import sites ==\n'
rg -n 'from\s+[\'"]framer-motion[\'"]|from\s+[\'"]motion/react[\'"]|import\s+.*motion.*from' src || trueRepository: SamuelIVX/SamuelIVX.github.io
Length of output: 2164
Switch the remaining imports to motion/react.
src/components/About.jsx, src/components/Contact.jsx, src/components/Experience.jsx, src/components/Hero.jsx, src/components/Works.jsx, and src/hoc/SectionWrapper.jsx still import from framer-motion even though motion is now the direct dependency. Updating these imports and regenerating the lockfile removes the transitive dependency coupling.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@package.json` at line 19, Update the imports in About, Contact, Experience,
Hero, Works, and SectionWrapper to use motion/react instead of framer-motion,
preserving their existing imported symbols. Regenerate the lockfile so the
direct motion dependency is reflected without the remaining transitive
framer-motion coupling.
This PR contains the following updates:
^12.0.0→^12.0.0This is a special PR that replaces
framer-motionwith the community suggested minimal stable replacement version.Configuration
📅 Schedule: (in timezone America/New_York)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate.