Skip to content

feat: upgrade dependencies - #153

Open
codfish wants to merge 1 commit into
mainfrom
deps
Open

feat: upgrade dependencies#153
codfish wants to merge 1 commit into
mainfrom
deps

Conversation

@codfish

@codfish codfish commented May 2, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Updated development tool dependencies including ESLint plugins and parsers, TypeScript ESLint, CommitLint, TanStack ESLint Plugin Query, Vitest ESLint Plugin, and related packages
    • Pinned Prettier from a loose range to a specific caret version
    • Removed Prettier as a required peer dependency while keeping it optional

@coderabbitai

coderabbitai Bot commented May 2, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

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

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 33488d13-807a-41d6-980b-9a5e70bf4980

📥 Commits

Reviewing files that changed from the base of the PR and between 87c657a and 9038c94.

📒 Files selected for processing (1)
  • package.json
📝 Walkthrough

Walkthrough

Updated dependency and devDependency versions across the ESLint configuration package, including ESLint plugins, parsers, formatters, and utilities. Pinned prettier to ^3.8.3 and removed it from peerDependencies, leaving only ESLint as a required peer.

Changes

Dependency Version Updates

Layer / File(s) Summary
Dependency Manifest
package.json (lines 57–83)
Dependencies bumped for commitlint, ESLint core, ESLint plugins (HTML, Next, TanStack, Vitest), config/plugin packages, globals, prettier (now pinned to ^3.8.3), and typescript-eslint.
Development Dependency Manifest
package.json (lines 85–92)
DevDependencies bumped for @types/node, Vitest tools, doctoc, ESLint, lint-staged, and testing utilities.
Peer Dependency Restructuring
package.json (line 108)
peerDependencies now requires only eslint >= 10; prettier peer requirement removed (already marked optional via peerDependenciesMeta).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A hop, a skip, a version bump spree,
Eslint and friends dance merrily!
Prettier's pinned, the peers now lean,
Cleaner dependencies, a maintainer's dream.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: upgrade dependencies' accurately describes the primary change—updating multiple dependency and devDependency versions in package.json, including a peer dependency modification.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch deps

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
Review rate limit: 0/1 reviews remaining, refill in 59 minutes and 35 seconds.

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

@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown

PR package published successfully!

Install:

pnpm add -D @codfish/eslint-config@0.0.0-PR-153--9038c94
npm install -D @codfish/eslint-config@0.0.0-PR-153--9038c94
yarn add -D @codfish/eslint-config@0.0.0-PR-153--9038c94
bun add -d @codfish/eslint-config@0.0.0-PR-153--9038c94

View on npm: https://www.npmjs.com/package/@codfish/eslint-config/v/0.0.0-PR-153--9038c94

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
package.json (1)

110-113: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Remove the now-orphaned peerDependenciesMeta.prettier entry.

prettier was removed from peerDependencies, so the peerDependenciesMeta.prettier block no longer has any effect — npm only processes metadata for packages that are actually listed in peerDependencies. Leaving it in place is misleading to consumers and tooling that generates install-time hints.

🛠️ Proposed fix
  "peerDependencies": {
    "eslint": ">= 10"
  },
- "peerDependenciesMeta": {
-   "prettier": {
-     "optional": true
-   }
- },
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@package.json` around lines 110 - 113, Remove the now-orphaned
peerDependenciesMeta.prettier entry: delete the "prettier" object from the
peerDependenciesMeta block in package.json, and if that leaves
peerDependenciesMeta empty remove the entire peerDependenciesMeta key so there
are no misleading metadata entries; reference the peerDependenciesMeta and
prettier keys when locating the entry to remove.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Line 64: The package.json currently declares "eslint": ">= 10" as a peer
requirement while depending on eslint-plugin-react (v7.37.5) which does not list
ESLint 10 in its peerDependencies; update package.json to either pin
eslint-plugin-react to a version that explicitly supports ESLint 10 or change
the peer eslint requirement to a range compatible with the plugin, and before
release verify that eslint-plugin-react has published a release declaring ESLint
10 support; refer to the dependency token "eslint-plugin-react" and the peer
field "eslint" in package.json when making the change.

---

Outside diff comments:
In `@package.json`:
- Around line 110-113: Remove the now-orphaned peerDependenciesMeta.prettier
entry: delete the "prettier" object from the peerDependenciesMeta block in
package.json, and if that leaves peerDependenciesMeta empty remove the entire
peerDependenciesMeta key so there are no misleading metadata entries; reference
the peerDependenciesMeta and prettier keys when locating the entry to remove.
🪄 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: 83d07cd8-56af-4a9a-b83e-acd965ad7e65

📥 Commits

Reviewing files that changed from the base of the PR and between 65b0faf and 87c657a.

📒 Files selected for processing (1)
  • package.json

Comment thread package.json
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