Repository navigation
Bump dependency overrides to fix Trivy vulnerabilities - #2481
Conversation
Update overrides and refresh lockfiles for dompurify, hono, fast-uri, follow-redirects, ip-address, markdown-it, undici, body-parser, @azure/msal-node and zod. Move wso2-platform zod deps to 4.6.7, since the zod 3 line has no fix for CVE-2023-54404. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Shared dependency overrides common/autoinstallers/rush-plugins/package.json, common/config/rush/pnpm-config.json |
The Rush plugin undici override changed to 6.29.1. Global overrides updated versions for hono, fast-uri, and dompurify, and added overrides for follow-redirects, ip-address, markdown-it, undici, body-parser, @azure/msal-node, and zod. Comments describe the @azure/msal-node dependency path and compatibility constraint. |
WSO2 zod dependency declarations workspaces/wso2-platform/wso2-platform-extension/package.json, workspaces/wso2-platform/wso2-platform-webviews/package.json |
Both packages now declare zod version 4.6.7. |
Priority:
Estimated code review effort: 2 (Simple) | ~10 minutes
Change: Other
Merge Risk: 
Minimal · up to c16ab
-
This dependency update bumps overrides and moves zod to 4.6.7 in the two WSO2 packages. No concrete compatibility problem was found, and it is ready to merge under normal checks.
Pre-merge checks: 
5
-
Full details: Title check
-
Explanation
The title names the main change: bumping dependency overrides to fix Trivy-reported vulnerabilities. The changeset updates the overrides in pnpm-config.json and the related package.json files to address the scan findings, so the title matches the primary change.
Full details: Description check
-
Explanation
The description follows the template structure. It includes Purpose, Goals, Approach, Release note, Documentation, Security checks, and Test environment, with substantive content in each. Sections not relevant to a dependency-only change (UI Component Development, Manage Icons, User stories, Certification, Marketing, Training, Samples, Related PRs, Migrations, Learning) are omitted. The Purpose section gives the Trivy scan link, the counts, and the affected lockfiles. Automation tests and Learning are not filled in, but the Test environment section documents the validation performed.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Warning
Some tools did not complete. Review the errors below.
🔧 Biome (2.5.15)
-
common/autoinstallers/rush-plugins/package.json
-
Biome could not lint this file: configuration resulted in errors. Check the repository's Biome configuration and plugins.
common/config/rush/pnpm-config.json
-
Biome could not lint this file: configuration resulted in errors. Check the repository's Biome configuration and plugins.
workspaces/wso2-platform/wso2-platform-extension/package.json
-
Biome could not lint this file: configuration resulted in errors. Check the repository's Biome configuration and plugins.
- 1 others
-
Comment @coderabbitai help to get the list of available commands.
Purpose
The Trivy scan (e.g. this run) reports 24 fixable vulnerabilities across the two lockfiles: 1 critical, 12 high, 7 medium and 4 low.
Goals
Both
common/config/rush/pnpm-lock.yamlandcommon/autoinstallers/rush-plugins/pnpm-lock.yamlscan clean, without adding entries to.trivyignore.Approach
Overrides in
common/config/rush/pnpm-config.json:Also:
undici6.28.1 -> 6.29.1 in the rush-plugins autoinstaller.zod4.6.7 inwso2-platform-extension(was 3.22.4) andwso2-platform-webviews(was 4.1.11).Two major-version jumps need reviewer attention:
zod/v4entry point, and the inspector UI is a prebuilt bundle with its own zod. The inspector proxy runs MCP SDK 1.32.1, which supports zod 4.@vscode/vsce->@azure/identity, which vsce uses only for--azure-credential. Our publish workflow uses a PAT (-p). msal-node 7 dropsacquireTokenByUsernamePassword, so identity's deprecatedUsernamePasswordCredentialwould fail. Nothing here uses it. This is noted in a comment on the override.Release note
N/A. Dependency updates only.
Documentation
N/A.
Security checks
Test environment
trivy fswith--ignore-unfixedand.trivyignore: 0 vulnerabilities in both lockfiles.rush build -t wso2-platform -t mcp-server-inspectorpasses.@azure/identityloads with msal-node 7.1.1, and every msal export it uses is present.vsce publish --azure-credentialwas not exercised.🤖 Generated with Claude Code
Summary by CodeRabbit