chore: update safe npm dependencies - #7983
Conversation
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Several minor-version upgrades conflict with the linked issue’s patch-only scope.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates core npm dependencies and pins installs to the trusted Azure Artifacts registry.
Changes:
- Updates YAML, linting, and Jest dependencies.
- Regenerates the dependency lockfile.
- Configures the project npm registry.
File summaries
| File | Description |
|---|---|
.npmrc |
Sets the Azure Artifacts registry. |
package.json |
Updates dependency versions. |
package-lock.json |
Locks updated dependency trees and registry URLs. |
Review details
- Files reviewed: 2/3 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "commander": "^12.1.0", | ||
| "execa": "^5.1.1", | ||
| "js-yaml": "5.3.0" | ||
| "js-yaml": "5.4.1" |
|
@copilot address review feedback |
|
✅ Copilot review passed with no inline comments. @copilot Add the |
✅ Coverage Check PassedOverall Coverage
📁 Per-file Coverage Changes (2 files)
✨ New Files (4 files)
Coverage comparison generated by |
|
✅ Contribution Check completed successfully! Code review complete: the PR follows CONTRIBUTING.md for style, documentation, file organization, and PR description. No new functionality was added, so the dependency-only update does not require additional tests beyond the existing lockfile/package refresh.
|
|
✅ Smoke Copilot BYOK AOAI (api-key) completed. Copilot AOAI BYOK (api-key) mode operational. 🔓
|
|
🔌 Smoke Services — All services reachable! ✅
|
|
❌ Smoke Gemini reports failed. Facets need polishing...
|
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤
|
|
✅ Build Test Suite completed successfully!
|
|
Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.
|
|
✅ Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓
|
|
📰 VERDICT: Smoke Docker Sbx has concluded. All systems operational. This is a developing story. 🎤
|
|
✅ Smoke Claude passed Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "api.anthropic.com"See Network Configuration for more information.
|
|
✅ Smoke Copilot BYOK AOAI (Entra) completed. Copilot AOAI BYOK (Entra) mode operational. 🔓
|
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "msfeed25.pkgs.visualstudio.com"See Network Configuration for more information.
|
|
🚀 Security Guard has started processing this pull request |
Smoke Test: Claude Engine Validation
Overall result: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "api.anthropic.com"See Network Configuration for more information.
|
|
Copilot Network Isolation Smoke Test EGRESS_RESULT allow=pass deny=pass
Overall: PASS Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "example.com"See Network Configuration for more information.
|
Smoke Test: Copilot BYOK (Direct) Mode ✅Test Results:
Status: PASS Running in direct BYOK mode (
|
|
Smoke Test: Services Connectivity
Overall: FAIL —
|
|
Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw)
|
|
Smoke Test: Docker Sbx —
Overall: PASS
|
Chroot Version Comparison
Not all runtime versions matched between host and chroot (Node.js differs), so the
|
|
Recent merged PRs: GitHub MCP review: ❌ Overall: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "msfeed25.pkgs.visualstudio.com"See Network Configuration for more information.
|
|
Smoke Test: Copilot Engine —
Overall: PASS
|
|
ci: recompile Playwright smoke workflows: ❌ Running in direct BYOK mode (AWF_AUTH_TYPE=github-oidc + AWF_AUTH_AZURE_* + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw) authenticated via Microsoft Entra Overall status: FAIL
|
Smoke Test Results: API Proxy OTel Tracing
Overall: 4/5 scenarios fully passing, 1 expected-pending (no OTLP export activity, no credentials configured in this run). No code regressions detected.
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS Notes:
|
Updates the requested npm dependencies to current safe versions and ensures installs resolve through the trusted Microsoft Azure Artifacts registry.
Dependency updates
js-yaml5.4.1eslint10.9.1Registry pinning
registry=https://ms-feed-25.pkgs.visualstudio.com/1es-public/_packaging/npm-public/npm/registry/