-
Notifications
You must be signed in to change notification settings - Fork 559
build(build-tools): Update dependencies and document limitations #25928
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build(build-tools): Update dependencies and document limitations #25928
Conversation
Keep updated dependency versions from bt-deps-major-updates branch: - @commitlint/* packages: ^17 → ^20 - chai: ^5 → ^6 - mocha: ^10 → ^11 - rimraf: ^4 → ^6 - date-fns: ^2 → ^3 - inquirer: ^8 → ^9 (required for @commitlint/cz-commitlint peer dep) - azure-devops-node-api: ^11 → ^15 - change-case: ^3 → ^5 - cosmiconfig: ^8 → ^9 - strip-ansi: ^6 → ^7 - picomatch: ^2 → ^4 - @inquirer/prompts: ^7 → ^8 - @fluid-tools/api-markdown-documenter: ^0.17 → ^0.23 - @microsoft/api-extractor: ^7.52 → ^7.55 - typedoc: ^0.26 → ^0.28 - eslint-plugin-chai-friendly: ~1.0 → ~1.1 - sharp override: ^0.33 → ^0.34 - @types/rimraf: ^2 → ^3 Also added @typescript-eslint/parser to peerDependencyRules allowedVersions.
Run syncpack fix-mismatches to align versions: - @fluidframework/eslint-config-fluid: ^6.1.0 → ^8.1.0 - @microsoft/api-extractor: ^7.52.11 → ^7.55.1 - cosmiconfig: ^8.3.6 → ^9.0.0 - type-fest: ^2.19.0 → ^5.2.0
- Revert type-fest from v5 to v2 (v5+ is ESM-only, incompatible with CommonJS) - Update ESLint configs to use import-x/ rules (plugin was renamed from import/) - Update inline eslint-disable comments from import/ to import-x/ - Expand internal modules allow list in build-cli - Update DEV.md to document type-fest limitation
- Restore pnpm packageManager integrity hash - Remove Migration Path section from DEV.md - ESLint: replace import/* with import-x/* where applicable Committed on 2025-11-26T19:37:46.621Z
Josmithr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, save for a comment that I re-opened.
alexvy86
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haven't looked at the DEV.md file yet, but left several comments.
build-tools/packages/bundle-size-tools/api-report/bundle-size-tools.api.md
Outdated
Show resolved
Hide resolved
alexvy86
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Updates build-tools dependencies with the following changes:
Successfully Upgraded:
ESM-Only Dependencies (Cannot Upgrade):
Multiple dependencies are pinned to older versions because newer versions are ESM-only and incompatible with the CommonJS codebase:
ESLint Plugin Migration:
All changes are documented in DEV.md for future reference.