Skip to content

fix(security): bump brace-expansion for CVE-2026-14257 DoS - #855

Open
santoshkumarradha wants to merge 1 commit into
mainfrom
santosh/fix-brace-expansion-dos-d09d
Open

fix(security): bump brace-expansion for CVE-2026-14257 DoS#855
santoshkumarradha wants to merge 1 commit into
mainfrom
santosh/fix-brace-expansion-dos-d09d

Conversation

@santoshkumarradha

Copy link
Copy Markdown
Member

Summary

Closes Dependabot alerts #355 (package-lock.json) and #357 (pnpm-lock.yaml) for CVE-2026-14257 / GHSA-mh99-v99m-4gvg — brace-expansion DoS via unbounded expansion length causing an OOM process crash.

Bumps npm/pnpm overrides in control-plane/web/client from vulnerable 1.1.16 / 2.1.2 to patched maintenance releases 1.1.18 / 2.1.4, and regenerates both lockfiles.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Docs only
  • Tests only
  • CI / tooling
  • Breaking change

Test plan

  • Confirmed lockfiles resolve brace-expansion@1.1.18 and brace-expansion@2.1.4
  • npm audit no longer reports brace-expansion / GHSA-mh99-v99m-4gvg
  • pnpm audit no longer reports brace-expansion
  • CI green for web client workflows

Test coverage

Dependency override / lockfile-only change — no application code paths changed.

  • I ran tests for the surface(s) I changed locally. (audit verification)
  • New code paths are covered by tests in this PR (no bare additions). (N/A — lockfile only)
  • If I removed code, I updated coverage-baseline.json(N/A)
  • The coverage gate check is green in CI before requesting review.

Checklist

Related issues / PRs

Open in Web Open in Cursor 

Override brace-expansion@1 to 1.1.18 and brace-expansion@2 to 2.1.4
in the web client to close Dependabot alerts for the unbounded
expansion length DoS (OOM crash) in both package-lock.json and
pnpm-lock.yaml.

Co-authored-by: Santosh kumar <santoshkumarradha@users.noreply.github.com>
@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

📊 Coverage gate

Thresholds from .coverage-gate.toml: per-surface ≥ 84%, aggregate ≥ 85%, max per-surface regression ≤ 1.0 pp, max aggregate regression ≤ 0.50 pp.

Surface Current Baseline Δ
control-plane 87.00% 87.40% ↓ -0.40 pp 🟡
sdk-go 92.50% 92.00% ↑ +0.50 pp 🟢
sdk-python 93.82% 93.73% ↑ +0.09 pp 🟢
sdk-typescript 91.08% 90.42% ↑ +0.66 pp 🟢
web-ui 84.75% 84.79% ↓ -0.04 pp 🟡
aggregate 85.58% 85.75% ↓ -0.17 pp 🟡

✅ Gate passed

No surface regressed past the allowed threshold and the aggregate stayed above the floor.

@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

📐 Patch coverage gate

Threshold: 80% on lines this PR touches vs origin/main (from .coverage-gate.toml:thresholds.min_patch).

Surface Touched lines Patch coverage Status
control-plane 0 ➖ no changes
sdk-go 0 ➖ no changes
sdk-python 0 ➖ no changes
sdk-typescript 0 ➖ no changes
web-ui 0 ➖ no changes

✅ Patch gate passed

Every surface whose lines were touched by this PR has patch coverage at or above the threshold.

@cursor

cursor Bot commented Aug 3, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

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