Skip to content

chore: Patch brace-expansion and @babel/helpers vulnerabilities#22

Merged
tony merged 2 commits intomasterfrom
lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities
Feb 10, 2026
Merged

chore: Patch brace-expansion and @babel/helpers vulnerabilities#22
tony merged 2 commits intomasterfrom
lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities

Conversation

@tony
Copy link
Copy Markdown
Collaborator

@tony tony commented Feb 9, 2026

Resolves LOB-1518.

Summary

  • Patch 2 of 3 npm audit vulnerabilities via overrides in package.json
  • brace-expansion 1.1.11 → 1.1.12: fixes ReDoS vulnerability (GHSA-v6h2-p8h4-qcjw)
  • @babel/helpers 7.22.15 → 7.28.6: fixes inefficient RegExp in transpiled named capturing groups (GHSA-968p-4wvh-cqc8)

Dismissed: aws-sdk v2 (GHSA-j965-2qgj-vjmq)

The remaining low-severity advisory affects all aws-sdk v2 versions (>=2.0.0 <=3.0.0) and recommends migrating to v3 or adding region parameter validation. This is a deprecation-level advisory, not a traditional vulnerability. Migration to v3 would be a significant refactor tracked separately.

Security Alerts

Test plan

  • npm install succeeds
  • npm run lint passes
  • npm test passes (25/25 tests, 98.63% coverage)
  • npm run package rebuilds dist/ successfully
  • npm audit confirms 2 vulnerabilities resolved (3 → 1 remaining)
  • Verify GitHub Action works in a test workflow

Note

Low Risk
Lockfile and dependency override updates only; primary risk is unintended build/test differences from updated transitive Babel tooling.

Overview
Updates dependency pinning to address known npm audit vulnerabilities by forcing patched transitive versions.

package.json adds overrides/resolutions for brace-expansion@^1.1.12 and @babel/helpers@^7.26.10, and the lockfiles (package-lock.json, yarn.lock) are refreshed accordingly, bumping related Babel packages (e.g., @babel/helpers, @babel/parser, @babel/types, @babel/code-frame) and removing now-unneeded transitive entries.

Written by Cursor Bugbot for commit 96826ee. Configure here.

@tony tony requested a review from a team as a code owner February 9, 2026 21:31
@linear
Copy link
Copy Markdown

linear bot commented Feb 9, 2026

Comment thread yarn.lock
@tony tony changed the title js(deps): Patch brace-expansion and @babel/helpers vulnerabilities (#LOB-1518) js(deps) Patch brace-expansion and @babel/helpers vulnerabilities Feb 9, 2026
@tony tony force-pushed the lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities branch from 43d51be to 0e55ec2 Compare February 9, 2026 22:21
@tony tony changed the title js(deps) Patch brace-expansion and @babel/helpers vulnerabilities js(deps) Patch brace-expansion and @babel/helpers vulnerabilities Feb 9, 2026
@tony tony force-pushed the lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities branch from 0e55ec2 to 1b17fea Compare February 9, 2026 22:34
@tony tony changed the title js(deps) Patch brace-expansion and @babel/helpers vulnerabilities chore: Patch brace-expansion and @babel/helpers vulnerabilities Feb 9, 2026
@tony tony force-pushed the lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities branch from 1b17fea to 8e246b6 Compare February 9, 2026 22:39
tony added 2 commits February 9, 2026 19:44
See also:
- https://www.npmjs.com/package/brace-expansion
- https://github.com/juliangruber/brace-expansion

Security Alerts:
- GHSA-v6h2-p8h4-qcjw
  - brace-expansion Regular Expression Denial of Service
  - Affects: >=1.0.0 <=1.1.11

Verification (`npm ls brace-expansion`):

Before:
└─┬ eslint@8.57.0
  └─┬ minimatch@3.1.2
    └── brace-expansion@1.1.11

After:
└─┬ eslint@8.57.0
  └─┬ minimatch@3.1.2
    └── brace-expansion@1.1.12

Verification (`yarn why brace-expansion`):

Before:
=> Found "brace-expansion@1.1.11"
info Reasons this module exists
   - "eslint#minimatch" depends on it
   - Hoisted from "eslint#minimatch#brace-expansion"

After:
=> Found "brace-expansion@1.1.12"
info Reasons this module exists
   - "eslint#minimatch" depends on it
   - Hoisted from "eslint#minimatch#brace-expansion"
See also:
- https://www.npmjs.com/package/@babel/helpers
- https://github.com/babel/babel
- https://github.com/babel/babel/blob/main/CHANGELOG.md

Security Alerts:
- GHSA-968p-4wvh-cqc8
  - Babel has inefficient RegExp complexity in generated code
    with .replace when transpiling named capturing groups
  - Affects: <7.26.10

Verification (`npm ls @babel/helpers`):

Before:
└─┬ jest@29.7.0
  └─┬ @jest/core@29.7.0
    └─┬ @jest/transform@29.7.0
      └─┬ @babel/core@7.22.17
        └── @babel/helpers@7.22.15

After:
└─┬ jest@29.7.0
  └─┬ @jest/core@29.7.0
    └─┬ @jest/transform@29.7.0
      └─┬ @babel/core@7.22.17
        └── @babel/helpers@7.28.6

Verification (`yarn why @babel/helpers`):

Before:
=> Found "@babel/helpers@7.22.15"
info Reasons this module exists
   - "jest#@jest#core#@jest#transform#@babel#core" depends on it
   - Hoisted from "jest#@jest#core#@jest#transform#@babel#core#@babel#helpers"
info Disk size with transitive dependencies: "8.56MB"

After:
=> Found "@babel/helpers@7.28.6"
info Reasons this module exists
   - "jest#@jest#core#@jest#transform#@babel#core" depends on it
   - Hoisted from "jest#@jest#core#@jest#transform#@babel#core#@babel#helpers"
info Disk size with transitive dependencies: "7.05MB"
@tony tony force-pushed the lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities branch from 8e246b6 to 96826ee Compare February 10, 2026 01:46
@tony
Copy link
Copy Markdown
Collaborator Author

tony commented Feb 10, 2026

@cursor review

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

@tony tony merged commit 8b6ded8 into master Feb 10, 2026
7 checks passed
@tony tony deleted the lob-1518-amazon-ecs-deploy-task-definition-patch-vulnerabilities branch February 10, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants