Repository navigation
fix(deps): resolve open Dependabot alerts (undici, msal-node, prismjs, esbuild, octokit, uuid) - #423
Merged
Merged
Conversation
Bump transitive dependencies via root resolutions to patched versions: - undici 7.29.x -> ^7.30.1 (Dependabot alerts 518, 523-526) - prismjs ~1.27.0 -> ^1.30.0 (alert 81) - @octokit/request ^6 -> ^8.4.1 (alert 96) - @octokit/request-error ^3 -> ^5.1.1 (alert 94) Refs #403 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: Nithya Subramanian <nithyasu@microsoft.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF ScorecardScorecard details
Scanned Files
|
- refresh @azure/identity to 4.14.1, which depends on @azure/msal-node ^7.1.0 (alert 521) - bump esbuild resolution to ^0.28.1 (alert 336) - remap @octokit/plugin-paginate-rest ^6.1.2 -> ^9.2.2 (alert 95) - remap uuid ^8.3.2 and ^9.0.x -> ^11.1.1 (alert 319, partial) Refs #403 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: Nithya Subramanian <nithyasu@microsoft.com>
4 tasks
Strip __archiveUrl references to the internal npm mirror and align the uuid bin path with the public registry metadata. Refs #403 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: Nithya Subramanian <nithyasu@microsoft.com>
lakshmimsft
approved these changes
Nov 3, 2026
brooke-hamilton
previously approved these changes
Nov 3, 2026
kachawla
previously approved these changes
Nov 3, 2026
…bot-alerts-403 Resolve conflicts with #416: - package.json: keep this branch's resolutions and add main's "uuid@npm:^10.0.0": "11.1.1" resolution. - yarn.lock: take main's version and regenerate it with yarn install (plus yarn up -R @azure/identity) so the resolutions apply. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: Nithya Subramanian <nithyasu@microsoft.com>
nithyatsu
dismissed stale reviews from kachawla and brooke-hamilton
via
November 3, 2026 17:28
a7d3b11
nithyatsu
enabled auto-merge (squash)
November 3, 2026 18:53
willtsai
approved these changes
Nov 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds root
resolutionsthat move vulnerable transitive dependencies to patched versions, refreshes@azure/identitywithin its existing range, and updatesyarn.lock:@module-federation/dts-plugin)@azure/identity@4.14.0)@azure/identity@4.14.1, lockfile refresh within^4.0.0, no resolution)refractor@3)0.27.7devDependency that #416 added toplugins/plugin-radius)@octokit/rest@19,@octokit/core@4,@octokit/auth-*)@octokit/rest@19).github/dependabot.ymlis not modified because it's synced from radius-project/.github.After merging main (#416)
package.json: kept this PR's resolutions and main's new"uuid@npm:^10.0.0": "11.1.1"resolution.yarn.lock: took main's version, then regenerated it withyarn installandyarn up -R @azure/identity. It wasn't hand-merged.testcontainers→ undici 7.30.1, anddockerode→ uuid 11.1.1.esbuild: 0.27.7inplugins/plugin-radiusfor the packaging qualification test andscripts/qualify-plugin-host.ts. Like every other esbuild range, the rootesbuildresolution overrides that pin, so they now use 0.28.2.yarn test:package(23/23) andyarn test:e2e:standalone(10/10) pass on 0.28.2.Reason for change
Refs #403
The react-router (372/373/390/392), deepmerge (527) and elliptic (154) alerts were dismissed as tolerable risk and are tracked in #422.
Alert not fully fixed
^8.3.2and^9.0.xranges now resolve to 11.1.1. Their consumers only userequire("uuid").v4, and I checked that each one loads uuid 11.1.1 and gets av4function.uuid@^3.4.0remains, pulled in by@material-table/core(via@backstage/core-components). It calls_interopRequireDefault(require("uuid")).default.v4(). That breaks with uuid 11, whose__esModulebuild has no default export, so I left it at 3.4.0. material-table only callsv4()without a buffer argument, and the advisory doesn't concern that call. The alert will stay open until Backstage drops material-table/uuid@3. This is now tracked in Track Dependabot alerts with no fix available yet (react-router v7, deepmerge, elliptic, uuid@3) #422.Compatibility of the @octokit remaps
The breaking changes in
@octokit/requestv7/v8 are Node ≥ 18, nativefetch, and the removal of http(s)agentand custom fetch options. For@octokit/request-errorv4/v5 they are Node ≥ 18 and an@octokit/typesbump. All three packages stay CommonJS. Backstage (SingleInstanceGithubCredentialsProvider,GithubUrlReader) only usesnew Octokit({ authStrategy: createAppAuth }),paginate(...),apps.*anderror.status, and doesn't pass agents or custom request options.@octokit/plugin-paginate-rest@9.2.2declares the peer dependency@octokit/core: "5", while@octokit/rest@19provides core 4. This peer mismatch is expected. The plugin only usesoctokit.requestandrequest.endpoint, which core 4 provides, and the smoke test below covers pagination at runtime.How to test
yarn install --immutable, which should succeed with no lockfile changes.yarn whyforundici,prismjs,esbuild,@azure/msal-node,@octokit/request,@octokit/request-error,@octokit/plugin-paginate-restanduuid. Apart fromuuid@3.4.0, only patched versions should remain.yarn tsc,yarn lint:all,yarn run format:check,yarn workspaces foreach -A run build:all,yarn test:all,yarn test:packageandNODE_ENV=production yarn test:e2e:standaloneshould all pass.Results after merging main (#416):
yarn install --immutable(checksums validated): passedyarn tsc: passedyarn lint:all: passedyarn run format:check: passedyarn workspaces foreach -A run build:all(esbuild 0.28.2): passedyarn test:all: passed (58 suites passed, 1 skipped; 514 tests passed, 23 skipped). The skipped tests include the gated packaging qualification, whichtest:packageruns.yarn test:package: passed (23/23, including real-browser journeys in both packed hosts). Run locally withTMPDIRresolved through macOS's/private/varsymlink.NODE_ENV=production yarn test:e2e:standalone: passed (10/10)fetch(before the merge): passed. It used@octokit/rest@19+@octokit/auth-app@4on request 8.4.1 / request-error 5.1.1 / paginate-rest 9.2.2 and covered app JWT pagination across 2 pages, installation token creation, installation repo pagination, and a 404 surfacing asRequestErrorwithstatus: 404.File change summary
esbuildresolution to^0.28.1. Retarget theundici7.29.0/7.29.1 resolutions to^7.30.1. Add resolutions forprismjs,@octokit/request,@octokit/request-error,@octokit/plugin-paginate-restanduuid(^8.3.2/^9.0.xonly).@azure/identity4.14.1 →@azure/msal-node7.1.2.