Skip to content

chore: upgrade toolchain and dependencies#421

Merged
bzp2010 merged 6 commits into
mainfrom
bzp/chore-upgrade-deps-260330
Mar 30, 2026
Merged

chore: upgrade toolchain and dependencies#421
bzp2010 merged 6 commits into
mainfrom
bzp/chore-upgrade-deps-260330

Conversation

@bzp2010

@bzp2010 bzp2010 commented Mar 30, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Collaborator

Description

  • Upgrade toolchain and dependencies.
  • Use datum-diff instead of deep-diff, which has been deprecated.
  • Resolve some security issues related to dependencies

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible

Summary by CodeRabbit

  • Chores

    • Bumped numerous dev tooling packages and workspace catalog entries.
    • Replaced the project's diff dependency with a new alternative and updated workspace catalog.
    • Updated package manifests and dependency catalog; added repository ignore patterns.
    • Disabled analytics collection in project configuration.
    • Added an eslint rule exception for a runtime helper dependency in JSON files.

  • Refactor

    • Tightened the typing of emitted diff payloads for more precise event shapes.

@coderabbitai

coderabbitai Bot commented Mar 30, 2026 •
edited
Loading

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7564cc04-e166-4879-82ce-b28e4ea435e0

📥 Commits

Reviewing files that changed from the base of the PR and between 515c681 and 66fc8f1.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (2)
  • libs/sdk/eslint.config.ts
  • libs/sdk/package.json
✅ Files skipped from review due to trivial changes (1)
  • libs/sdk/package.json
📝 Walkthrough

Walkthrough

Replaced deep-diff with datum-diff and updated related types/imports; bumped multiple devDependency versions and pnpm catalog entries; added IDE/tooling ignores to .gitignore; added analytics: false to nx.json; added tslib to ignoredDependencies in ESLint dependency-checks config.

Changes

Cohort / File(s) Summary
Diff Library Migration
libs/differ/package.json, libs/sdk/package.json, libs/differ/src/differv3.ts, libs/sdk/src/core/differ.ts, pnpm-workspace.yaml		 Removed deep-diff and @types/deep-diff; added datum-diff (catalog + deps). Updated imports to datum-diff and narrowed diff types to Array<Diff<ResourceFor<T>, object>>.
Tooling & Dev Dependencies
package.json, pnpm-workspace.yaml		 Bumped many devDependency versions (Nx, SWC, Vitest, Prettier plugin, @types/node, eslint/parser, etc.) and adjusted pnpm catalog entries (added/updated versions, removed deep-diff entries).
Config & Ignore Files
.gitignore, nx.json		 Added ignore patterns for .cursor/rules/nx-rules.mdc, .github/instructions/nx.instructions.md, .claude/*; normalized EOL in .gitignore. Added top-level analytics: false to nx.json.
ESLint Dependency Checks
libs/sdk/eslint.config.ts		 Added ignoredDependencies: ['tslib'] to the @nx/dependency-checks rule for JSON files.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Poem

🐰 I hopped through deps and swapped a thread,
From deep to datum, new types ahead.
Ignores tucked snug, nx set to hush,
Versions refreshed in a tidy rush.
🥕 Code burrowed cozy — I thumped with a blush.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title accurately describes the main changes: upgrading toolchain dependencies and replacing deep-diff with datum-diff.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bzp/chore-upgrade-deps-260330

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

@bzp2010
bzp2010 merged commit d9ccec5 into main Mar 30, 2026
24 of 26 checks passed
@bzp2010
bzp2010 deleted the bzp/chore-upgrade-deps-260330 branch March 30, 2026 05:20
@coderabbitai coderabbitai Bot mentioned this pull request Mar 30, 2026
Merged
5 tasks
This was referenced Apr 20, 2026
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juzhiyuan juzhiyuan juzhiyuan approved these changes
@LiteSun LiteSun Awaiting requested review from LiteSun
@guoqqqi guoqqqi Awaiting requested review from guoqqqi

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bzp2010 @juzhiyuan