Skip to content

chore: update dependencies and pin eslint to latest v9#34

Merged
jobenjada merged 1 commit intomainfrom
codex/deps-update-eslint9
Mar 3, 2026
Merged

chore: update dependencies and pin eslint to latest v9#34
jobenjada merged 1 commit intomainfrom
codex/deps-update-eslint9

Conversation

@mattinannt
Copy link
Member

@mattinannt mattinannt commented Mar 3, 2026

Summary

  • run ncu -u --deep to update dependencies across the workspace
  • pin eslint and @eslint/js to the latest ESLint 9 release (9.39.3) instead of ESLint 10
  • refresh lockfile via pnpm install

Validation

  • pnpm build
  • pnpm test
  • pnpm lint

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

Walkthrough

This pull request updates various dependency versions across the monorepo structure. The root package.json bumps turbo from 2.8.8 to 2.8.12, rimraf from 6.1.2 to 6.1.3, and updates the package manager from pnpm@10.29.3 to pnpm@10.30.3. The playground application updates lucide-react, tailwindcss, postcss plugin, types/node, and eslint to their latest patch versions. The packages/js directory updates eslint-related development dependencies, vitest eslint plugin, and happy-dom to newer versions. All changes are dependency version updates with no modifications to application code or public exports.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main changes: dependency updates and pinning ESLint to v9, which matches the changeset.
Description check ✅ Passed The description is directly related to the changeset, explaining the update process, validation steps, and the rationale for pinning ESLint to v9.
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.

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

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)

24-30: ⚠️ Potential issue | 🟠 Major

Add root pnpm.overrides for ESLint v9 to prevent workspace-wide resolution drift.

Recent commits show ESLint versioning was an issue (commits 57fc3e3, 03996b4). Adding workspace-root overrides for eslint and @eslint/js at version 9.39.3 will enforce the constraint across all transitive dependencies and prevent accidental upgrades to ESLint 10.

Proposed fix
   "pnpm": {
     "overrides": {
-      "tar": "7.5.4"
+      "tar": "7.5.4",
+      "eslint": "9.39.3",
+      "@eslint/js": "9.39.3"
     },

After applying this change, regenerate the lockfile with pnpm install --frozen-lockfile=false.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@package.json` around lines 24 - 30, Add workspace-root pnpm.overrides to pin
eslint and `@eslint/js` to 9.39.3 to prevent workspace-wide resolution drift:
update package.json's "pnpm.overrides" (alongside existing "tar" override) to
include "eslint": "9.39.3" and "@eslint/js": "9.39.3", then run pnpm install
--frozen-lockfile=false to regenerate the lockfile so the override is applied
across the monorepo.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@package.json`:
- Around line 24-30: Add workspace-root pnpm.overrides to pin eslint and
`@eslint/js` to 9.39.3 to prevent workspace-wide resolution drift: update
package.json's "pnpm.overrides" (alongside existing "tar" override) to include
"eslint": "9.39.3" and "@eslint/js": "9.39.3", then run pnpm install
--frozen-lockfile=false to regenerate the lockfile so the override is applied
across the monorepo.

ℹ️ Review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 57fc3e3 and 03996b4.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (3)
  • apps/playground/package.json
  • package.json
  • packages/js/package.json

@mattinannt mattinannt requested a review from jobenjada March 3, 2026 10:49
@jobenjada jobenjada added this pull request to the merge queue Mar 3, 2026
Merged via the queue into main with commit 3dbfa03 Mar 3, 2026
9 checks passed
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.

2 participants