Repository navigation
chore(deps): consolidate Dependabot updates and npm audit fixes - #274
Merged
Jackson Weber (JacksonWeber) merged 1 commit intoOct 26, 2026
Conversation
Update approved dependency trees and Actions, remove vulnerable MCP dependencies with OpenAI Agents 0.15.0, upgrade Vitest and MSAL, and lock the Fabric sample. Document proxy-blocked Vite and body-parser updates. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
The MSAL major upgrade and substantial OpenAI Agents dependency changes warrant final human compatibility review.
No open issues 🎉
4 files not reviewed
samples/agent365-s2s/package-lock.json: Generated filesamples/aks-langchain/package-lock.json: Generated filesamples/fabric/package-lock.json: Generated filesamples/package-lock.json: Generated file
What changed in this PR
Consolidates Dependabot updates and security fixes across the distro’s development dependencies, samples, and GitHub Actions, while preserving root production dependency ranges and the required Microsoft package proxy.
Changes:
- Updates OpenAI Agents, Vitest, MSAL Node, and affected transitive dependencies.
- Refreshes SHA-pinned GitHub Actions versions.
- Adds a Fabric sample lockfile and documents
npm ci, with proxy-blocked fixes recorded in the changelog.
| File | Description |
|---|---|
samples/package.json |
Raises OpenAI Agents to ^0.15.0. |
samples/package-lock.json |
Updates Agents and Zod; removes the old MCP/Express dependency chain. |
samples/fabric/README.md |
Switches installation instructions to npm ci. |
samples/fabric/package-lock.json |
Adds reproducible dependency resolutions for Fabric. |
samples/aks-langchain/package-lock.json |
Updates Zod to 4.6.7. |
samples/agent365-s2s/package.json |
Raises MSAL Node to ^7.1.1. |
samples/agent365-s2s/package-lock.json |
Locks updated MSAL packages and refreshes root metadata. |
package.json |
Updates Agents, Vitest, and Istanbul coverage development dependencies. |
CHANGELOG.md |
Records dependency updates and proxy-blocked fixes. |
.github/workflows/pr-validation.yml |
Updates the pinned setup-node action. |
.github/workflows/performance.yml |
Updates setup-node and upload-artifact pins. |
.github/workflows/docs.yml |
Updates the pinned setup-node action. |
Review effort: Balanced
Radhika Gupta (rads-1996)
approved these changes
Oct 26, 2026
Jackson Weber (JacksonWeber)
merged commit Oct 26, 2026
e8c7dae
into
microsoft:main
10 checks passed
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
Consolidate the available updates from 10 open Dependabot PRs, plus additional npm audit fixes, into one dependency update. Keep the required Microsoft package proxy and leave production dependency ranges unchanged.
^0.15.0in the root and main samples. This removes the vulnerable MCP SDK / Express / body-parser / Ajv / fast-uri dependency chain rather than forcing incompatible transitive overrides.^4.1.12and update Vite to the proxy-approved8.3.4.14.3.3, brace-expansion to5.0.12, source-map-js to1.2.2, and Zod to4.6.7across affected lockfiles, including the AKS sample.^7.1.1for GHSA-xcx6-jx8p-x3h5.actions/setup-nodeto7.1.0andactions/upload-artifactto7.0.2.npm ci.Dependabot coverage
4.6.7in samples.14.3.3.5.0.12.1.2.2.0.15.0.Leave #266 open. Its exact Vite
8.3.5target is unavailable through the required Microsoft proxy. This PR uses the latest proxy-approved8.3.4, which clears the reported Vite audit findings. The requester approved proceeding with this exception.npm audit results and remaining blocker
Final
npm audit --json --prefer-onlineresults, including development dependencies:Fabric is not audit-clean: GHSA-27r3-jxmh-p828 requires body-parser
1.20.9or2.4.0. Both patched versions are unavailable through the required proxy (exact-version lookups return E404). Its lockfile currently resolves Express4.22.3and body-parser1.20.8. Upgrading Express to the proxy's latest5.3.0would still pull a vulnerable body-parser2.xversion, so this PR avoids an ineffective major upgrade. The requester approved proceeding while documenting this blocked fix; it must be revisited when a patched version is proxy-approved.Verification
npm cisucceeds for the root and all four sample package trees.npm test -- --maxWorkers=4: 1,293 passed, 16 skipped, 5 todo. The first unbounded parallel run hit one durable-export replay timeout; that test passed in isolation and the complete bounded-concurrency rerun passed.0.15.0.7.1.1.Full main-samples build limitation:
npm run build --prefix samplesreports four errors in the unchangedsamples/src/a365HostingMiddleware.ts: an unusedInferenceScopeimport (line 35),Promise<unknown[]>typing (line 114), and missinggetParentSpanRefcalls (lines 205 and 306). No unrelated sample source changes are included.