Repository navigation
Fix code scanning and Dependabot security findings - #814
Merged
Merged
Conversation
Restrict the API Browser proxy and the reference app's outbound loop test to their intended hosts, stop logging API key prefixes, harden regexes and scaffold JSON merging against crafted input, bump zod and http-proxy-middleware, refresh vulnerable transitive dependencies within their declared ranges, and declare read-only permissions on the e2e and Action test workflows (with issues: write where e2e failures open issues).
clavery
requested review from
patricksullivansf,
wei-liu-sf and
yhsieh1
as code owners
October 23, 2026 21:20
1 task
3 tasks
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
Fixes the actionable code scanning and Dependabot findings.
commercecloud.salesforce.comanywhere in the URL, and the extension-host proxy forwarded any URL it was given. That let a request reach other hosts and get around the webview'sconnect-srcpolicy. Both now require HTTPS and a*.commercecloud.salesforce.comhostname.?path=, so an absolute URL could make the app fetch any host. It now returns 400 unless the target stays on the external domain. The app is private and not published.__proto__/constructor/prototypepath segments and skips__proto__keys in merged content.zod4.5.4 → 4.6.7 in b2c-dx-mcp (memory exhaustion validating oversized collections).http-proxy-middleware3.0.5 → 3.0.7 in mrt-utilities. The workspace override hid this in the repo, but npm consumers still resolved 3.0.5.@humanfs/node). Each override stays within the range its parent declares.pnpm-workspace.yaml:diff@7(via mocha 11),uuid@8(via@vscode/vsce) andhttp-cache-semantics(no patched release). All three are dev-only.permissions: contents: readtoe2e-tests.yml,e2e-shell-tests.ymlandtest-actions.yml. Removed the CLI e2e step that opened an issue when a scheduled run failed. It neededissues: writeand had stopped working when the default token became read-only.Testing
pnpm run lint:agent,pnpm run typecheck:agentandpnpm run buildpass.pnpm -r run test:agentpasses in every package except the SDK, whereauth/oauth-implicit/auth/oauth-pkcefail withEADDRINUSEunder--parallel. This is an existing race ingetAvailablePort(): the port changes every run, and those files pass when run on their own.tests/test_notebooks.pywas not run becausenbformatisn't installed locally.