Repository navigation
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The narrowly scoped dependency updates are internally consistent, preserve supported Node runtimes, and introduce no identified defects.
Review effort: Balanced
Findings: None
What changed in this PR
Updates audited dependencies while preserving package versions and dependency ranges.
Changes:
- Adds overrides for patched
brace-expansionandmarkdown-it. - Resolves affected lockfiles to
undici6.29.0.
| File | Description |
|---|---|
packages/artifact/package.json |
Adds security-related dependency overrides. |
packages/artifact/package-lock.json |
Locks patched transitive dependencies. |
packages/core/package-lock.json |
Updates locked Undici version. |
packages/glob/package.json |
Overrides vulnerable brace-expansion versions. |
packages/glob/package-lock.json |
Locks patched brace-expansion and Undici. |
packages/http-client/package-lock.json |
Updates locked Undici version. |
packages/tool-cache/package-lock.json |
Updates locked Undici version. |
Files not reviewed (5)
- packages/artifact/package-lock.json: Generated file
- packages/core/package-lock.json: Generated file
- packages/glob/package-lock.json: Generated file
- packages/http-client/package-lock.json: Generated file
- packages/tool-cache/package-lock.json: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Scope
Audit-only patch, independently based on upstream
main(dc752c75f8868e7e83761116161c952938b8ddae), with exactly seven files changed:All package release versions and dependency ranges are preserved, including HTTP-client 4.0.1 / Undici
^6.23.0and consumer HTTP-client minima. No release notes, version bumps, HTTP-client 4.0.2 intent, retry changes, dependency majors, or other layers.Generated-file provenance
Only the two source
package.jsonfiles were manually edited. npm 11.19.0 generated all five locks with targetednpm update ... --package-lock-only --ignore-scripts --no-fund --no-auditcommands. No copied or hand-edited lock entries, URLs, integrity values, or metadata. All seven resulting files byte-match closed historical reference #2517 (4eb240c03b7ce94f9c302c927071cd9ae3de24eb); it is not a prerequisite.Local validation
npm ciin all five affected packages; CI-equivalent rootnpm installandnpm run bootstrap; lockfiles remained unchanged.npm audit --audit-level=moderate --omit dev: 0 vulnerabilities.npm run audit-all: all nine audited packages have 0 vulnerabilities.The existing root dev-tool audit baseline (32 findings) is outside the CI production audit scope and unchanged. Remote CI is not yet verified; keep this PR draft pending checks on its exact head commit.