Skip to content

fix(core): remove multimatch dependency and legacy-core internals#4314

Merged
JamesHenry merged 2 commits intolerna:mainfrom
AI-JamesHenry-Org:dep-analysis
Mar 13, 2026
Merged

fix(core): remove multimatch dependency and legacy-core internals#4314
JamesHenry merged 2 commits intolerna:mainfrom
AI-JamesHenry-Org:dep-analysis

Conversation

@AI-JamesHenry
Copy link
Copy Markdown
Collaborator

Note

🤖 This PR was created by @AI-JamesHenry, an AI assistant account guided and overseen by @JamesHenry.

Summary

  • Replace multimatch dependency with internal implementation using existing minimatch
  • Remove @lerna/legacy-core package entirely, migrating its 3 remaining consumers (add-caching, clean, create) to @lerna/core's Command class and project graph APIs
  • Remove @ts-nocheck from the clean command

Test plan

  • All unit tests pass (nx run-many -t test)
  • All lint checks pass (nx run-many -t lint)
  • Build passes (nx run-many -t build)
  • Formatting passes (npm run format:check)

Replace the multimatch dependency with an internal implementation using
the existing minimatch dependency. Remove the legacy-core package
entirely by migrating its three remaining consumers (add-caching, clean,
create) to use core's Command class and project graph APIs. Also removes
@ts-nocheck from the clean command.
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Mar 13, 2026

View your CI Pipeline Execution ↗ for commit 5ebcccf

Command Status Duration Result
nx run-many --t e2e --parallel=1 ✅ Succeeded 12m 7s View ↗
nx run-many -t build --parallel=3 ✅ Succeeded 38s View ↗
nx run-many -t lint --parallel=3 ✅ Succeeded 37s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded 35s View ↗
nx run integration:integration --ci --maxWorkers=2 ✅ Succeeded 12s View ↗
nx run-many -t build ✅ Succeeded <1s View ↗
nx-cloud record -- npx nx format:check ✅ Succeeded 3s View ↗
nx run-many -t test --parallel=3 --ci --maxWork... ✅ Succeeded 3m 2s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-13 19:18:18 UTC

Previously provided transitively by multimatch, needed for minimatch v3
type declarations used in core build.
nx-cloud[bot]

This comment was marked as resolved.

@JamesHenry JamesHenry marked this pull request as ready for review March 13, 2026 19:19
@JamesHenry JamesHenry merged commit ec01462 into lerna:main Mar 13, 2026
15 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