microsoft / microsoft/fluentui-contrib
chore: remediate vulnerable transitive dependencies
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 56
- Forks
- 62
- Avg merge
- 3d 2h
- Merged PRs (30d)
- 1
Description
Problem
The repository currently pins or resolves several vulnerable transitive
dependencies even though patched versions are available within the existing
dependency majors. The open Nx Dependabot PR updates only nx, while the
repository keeps the rest of the @nx/* family at 22.1.3.
Related existing work:
- #666 updates only
nxfrom 22.1.3 to 22.7.7 and does not align the complete
Nx package family. - #604 targets Nx 22.3, is conflicting, and is below the desired 22.7.7 floor.
Requested change
Create one focused dependency-security PR from current main:
- Align
nxand every direct@nx/*dependency in the rootpackage.json
from 22.1.3 to 22.7.7. - Raise the root
axiosresolution from^1.12.0to at least^1.16.0. - Raise the root
node-forgeresolution from1.3.2to1.4.0. - Refresh and deduplicate
yarn.lockso compatible transitive dependencies
resolve at or above these patched floors:tar7.5.22minimatch3.1.4, 5.1.8, and 10.2.3 as applicable@babel/plugin-transform-modules-systemjs7.29.4@isaacs/brace-expansion5.0.1brace-expansion1.1.18 and 2.1.4browserslist4.28.7form-data4.0.6ip-address10.3.1js-yaml3.15.1nanoid3.3.18picomatch4.0.4postcss8.5.18rollup4.59.0tmp0.2.6vite6.4.3
- Apply an Nx migration file only if the Nx migration command generates a
necessary repository change.
Keep the change limited to dependency remediation. Do not introduce unrelated
toolchain or application refactors.
Acceptance criteria
- The root manifest uses one consistent 22.7.7 version for
nxand all direct
@nx/*dependencies. yarn.lockno longer resolves the vulnerable versions replaced above.yarn dedupe --checkpasses.- Existing repository lint, type-check, unit, and representative React
17/18/19 validation used by CI pass. - The pull request explains any dependency that cannot reach the requested
patched floor. - The pull request states whether it supersedes #666; do not leave two
overlapping dependency PRs intended to merge.
Notes
Current main no longer contains react-router 7.0.0 in manifests or
yarn.lock. Do not add a code change for that package unless a current
dependency path is found.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the root package.json and compare its nx/@nx/*, axios, and node-forge resolutions against the requested floors, then inspect yarn.lock for the listed vulnerable transitive versions. Refresh the lockfile without unrelated changes, run yarn dedupe --check, and run the repository lint, type-check, unit, and representative React 17/18/19 validation used by CI.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- build-system, security
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 66/100