refactor: remove unused dependencies#7541
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughRemoves multiple unused dependencies/devDependencies, reorders dependency lists in several package.json files, updates react-router build/export entries, and deletes LLM support docs and the LLM generation script. ChangesMonorepo Dependency Cleanup
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
|
View your CI Pipeline Execution ↗ for commit 80197a5
☁️ Nx Cloud last updated this comment at |
Merging this PR will degrade performance by 5.42%
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
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 `@packages/react-router/package.json`:
- Line 41: The package.json "build" script was changed to "vite build" which no
longer produces the llms artifacts referenced by exports["./llms"]
(dist/llms/index.{js,d.ts}); fix by either restoring the llms generation step
into the "build" script (e.g., run the llms compile/generation task before or
after vite build) so dist/llms/* is produced, or remove the "./llms" entry from
exports to avoid a broken subpath export; update the package.json "build" script
or the exports mapping accordingly and ensure any related build helper (the llms
generator task referenced in the repo/Vite config) is invoked when building.
🪄 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: d533e583-df76-43af-bd8c-93b183a2dd8a
📒 Files selected for processing (1)
packages/react-router/package.json
There was a problem hiding this comment.
Nx Cloud is proposing a fix for your failed CI:
We removed the "./llms" export entry from packages/react-router/package.json to fix the failing publint --strict check. The PR deleted scripts/llms-generate.mjs and the vibe-rules dependency, which meant the declared ./dist/llms/index.d.ts and ./dist/llms/index.js export targets were no longer being generated. This change aligns the package exports with the actual build artifacts.
Tip
✅ We verified this fix by re-running @tanstack/react-router:test:build.
Warning
The suggested diff is too large to display here, but you can view it on Nx Cloud ↗
Or Apply changes locally with:
npx nx-cloud apply-locally 9bf5-9bmG
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
vibe-rulespnpm-lock.yamlI hope I didn't break anything.
Summary by CodeRabbit