Skip to content

perf: move nuxt/kit to peer deps#379

Merged
danielroe merged 2 commits intomainfrom
ci/kit-peer
Feb 7, 2026
Merged

perf: move nuxt/kit to peer deps#379
danielroe merged 2 commits intomainfrom
ci/kit-peer

Conversation

@danielroe
Copy link
Member

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

this allows us to avoid adding nuxt kit v3 to a nuxt v4 project

@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

Warning

Rate limit exceeded

@danielroe has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 1 minutes and 39 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

The change modifies the package.json manifest to restructure how the @nuxt/kit dependency is declared. The package moves @nuxt/kit from the dependencies section to devDependencies, and adds two new sections: peerDependencies declaring @nuxt/kit with a minimum version constraint, and peerDependenciesMeta marking the peer dependency as optional. These are manifest-only changes with no modifications to code logic or runtime behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: moving @nuxt/kit to peer dependencies, which aligns with the changeset modifications to package.json.
Description check ✅ Passed The description is directly related to the changeset, explaining the rationale for moving nuxt/kit to peer dependencies to prevent version conflicts with Nuxt v4 projects.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ci/kit-peer

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielroe danielroe merged commit 2c4ee4e into main Feb 7, 2026
5 checks passed
@danielroe danielroe deleted the ci/kit-peer branch February 7, 2026 14:06
This was referenced Feb 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant