Skip to content

fix: remediate template dependency vulnerabilities - #112

Merged
Kylejeong2 merged 2 commits into
devfrom
kylejeong/gro-2507-fix-security-vulns-in-templates
Oct 22, 2026
Merged

Kylejeong2 merged 2 commits into
devfrom
kylejeong/gro-2507-fix-security-vulns-in-templates

Conversation

@Kylejeong2

@Kylejeong2 Kylejeong2 commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Summary

Remediate dependency vulnerabilities reported by socket fix --all for GRO-2507. Update the root lockfile and Next.js, Puppeteer, adm-zip, and the pinned MCP SDK; add lockfiles for the three affected templates so Socket can apply and preserve transitive fixes.

Socket still reports GHSA-vfj7-8cjw-p6xm in braces, introduced by eslint-config-next, with no patched version available.

Validation

  • Passed frozen-lockfile installs (lifecycle scripts disabled), TypeScript checks for the human-in-the-loop and Puppeteer templates, and an adm-zip round-trip smoke check.
  • Passed template transpilation, README index validation, changed-file formatting, and git diff --check.
  • Root lint fails because ESLint ignores all matching files. Live browser/API flows were not tested; the Git-pinned codemode package also lacks its built CLI artifact with lifecycle scripts disabled.

Refs https://linear.app/browserbase/issue/GRO-2507/fix-security-vulns-in-templates


Note

Low Risk
Changes are lockfile and semver dependency bumps for security remediation; no runtime or auth logic is modified.

Overview
Addresses Socket-reported vulnerabilities by refreshing dependency versions and lockfiles rather than changing application code.

The agent-with-human-in-loop template bumps Next.js and eslint-config-next from 16.2.1 to 16.4.1, raises the pnpm override for @modelcontextprotocol/sdk to 1.32.0, and adds a dedicated pnpm-lock.yaml so installs and transitive fixes are pinned for that package. extend-browserbase upgrades adm-zip from ^0.5.16 to ^0.6.1. The root pnpm-lock.yaml is updated with newer transitive packages in the ESLint/tooling graph (e.g. ajv, minimatch, js-yaml, @humanfs/*).

One known braces advisory via eslint-config-next remains without a patched upstream release, as noted in the PR description.

Reviewed by Cursor Bugbot for commit 746f37d. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security

socket-security Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​eslint@​9.39.59710010050100
Updatednpm/​next@​16.2.1 ⏵ 16.4.161 -2100 +7590 +19970
Updatednpm/​eslint-config-next@​16.2.1 ⏵ 16.4.19910068 +198100
Updatednpm/​puppeteer-core@​24.43.1 ⏵ 25.0.395 +910092 +197 +1100

View full report

@Kylejeong2
Kylejeong2 marked this pull request as ready for review October 21, 2026 23:15
@Kylejeong2
Kylejeong2 merged commit 30e2295 into dev Oct 22, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants