Skip to content

chore: node22 dep cleanup#4681

Draft
escapedcat wants to merge 5 commits intochore/node-v22from
chore/node22-dep-cleanup
Draft

chore: node22 dep cleanup#4681
escapedcat wants to merge 5 commits intochore/node-v22from
chore/node22-dep-cleanup

Conversation

@escapedcat
Copy link
Copy Markdown
Member

No description provided.

escapedcat and others added 5 commits April 3, 2026 12:26
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR cleans up dependencies now that the repo targets Node.js 22+, replacing several third-party utilities with Node built-ins and updating lockfiles accordingly.

Changes:

  • Replaced glob / fast-glob usage with Node’s fs.glob* APIs in configs and tests.
  • Removed fs-extra from test tooling and updated filesystem helpers to use node:fs / node:fs/promises.
  • Removed minimist and switched git arg parsing to node:util.parseArgs; removed import-meta-resolve and updated module resolution logic.

Reviewed changes

Copilot reviewed 22 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
yarn.lock Removes transitive entries made unnecessary by dropping glob, fast-glob, fs-extra, minimist, import-meta-resolve.
@packages/test/src/npm.ts Replaces fs-extra helpers with node:fs/promises equivalents for manifest detection, JSON reading, and directory creation.
@packages/test/src/index.test.ts Switches from fs-extra to node:fs for sync existence checks.
@packages/test/src/fix.ts Replaces fs-extra.copy with fs.cp (recursive).
@packages/test/package.json Removes fs-extra and @types/fs-extra deps.
@commitlint/rules/src/index.test.ts Replaces glob with node:fs globSync and manual filtering logic.
@commitlint/rules/package.json Drops glob devDependency.
@commitlint/resolve-extends/src/index.ts Removes import-meta-resolve and switches to createRequire(...).resolve(...) for resolution.
@commitlint/resolve-extends/package.json Drops import-meta-resolve dependency.
@commitlint/read/src/read.ts Replaces minimist with node:util.parseArgs for gitLogArgs.
@commitlint/read/package.json Removes minimist and @types/minimist.
@commitlint/ensure/src/index.test.ts Replaces glob with node:fs globSync and manual filtering logic.
@commitlint/ensure/package.json Drops glob devDependency.
@commitlint/config-workspace-scopes/package.json Removes glob dependency and related types.
@commitlint/config-workspace-scopes/index.js Replaces glob usage with node:fs globSync.
@commitlint/config-pnpm-scopes/package.json Drops fast-glob dependency.
@commitlint/config-pnpm-scopes/index.ts Replaces fast-glob with node:fs/promises glob iteration.
@commitlint/config-patternplate/package.json Drops glob dependency.
@commitlint/config-patternplate/index.js Replaces glob package usage with node:fs/promises glob iteration.
@commitlint/config-lerna-scopes/package.json Drops fast-glob dependency.
@commitlint/config-lerna-scopes/index.js Replaces fast-glob with node:fs/promises glob iteration.
@commitlint/cli/src/cli.test.ts Replaces fs-extra with node:fs/promises.
@commitlint/cli/package.json Removes fs-extra from devDependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants