Skip to content

fix(security): close open npm Dependabot vulnerability alerts - #868

Open
santoshkumarradha wants to merge 3 commits into
mainfrom
santosh/fix-npm-security-alerts-fc5c
Open

fix(security): close open npm Dependabot vulnerability alerts#868
santoshkumarradha wants to merge 3 commits into
mainfrom
santosh/fix-npm-security-alerts-fc5c

Conversation

@santoshkumarradha

@santoshkumarradha santoshkumarradha commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

Single PR that upgrades all currently selected open Dependabot-alerted npm packages so the security alerts can auto-close once merged.

High

Alerts Package Fix
#355, #357, #363, #365 brace-expansion (web client locks) 1.1.16/2.1.21.1.18/2.1.4
#371, #375 fast-uri (desktop, mastra-bench) 3.1.43.1.5

Also patches desktop brace-expansion lines 1/2/51.1.18/2.1.4/5.0.9 (same advisories).

Moderate

Alerts Package Fix
#366, #370, #376 postcss (web client, desktop, sdk/typescript) 8.5.188.5.25
#372, #373 undici (desktop) 6.27.06.28.0

CI reliability

UI Docker builder stages (Dockerfile.control-plane, -cloud, functional Dockerfile.test-runner) switched from node:*-alpine to bookworm/glibc so Vite's lightningcss optional native binding stops flaking under npm ci on musl.

Merged main after #866 landed; kept postcss@8.5.25 over #866’s 8.5.24 and retained brace-expansion overrides.

Supersedes #855. Partially overlaps merged #866 (extends it with remaining alerts).

Type of change

  • Bug fix
  • CI / tooling

Test plan

  • Confirmed lockfiles resolve patched versions (brace-expansion 1.1.18/2.1.4/5.0.9, postcss 8.5.25, fast-uri 3.1.5, undici 6.28.0)
  • npm audit / pnpm audit no longer report the targeted advisories
  • Local: web client tests 683 passed; TypeScript SDK 807 passed; desktop typecheck + 386 tests passed; web client vite build succeeded
  • CI green after merge-conflict resolution

Related issues / PRs

Open in Web Open in Cursor 

Bump transitive overrides across web client, desktop, TypeScript SDK,
and mastra-bench lockfiles:

- brace-expansion → 1.1.18 / 2.1.4 / 5.0.9 (CVE-2026-14257, CVE-2026-69152)
- postcss → 8.5.25 (GHSA-fxqj-rqcc-2cmp incomplete sourceMappingURL fix)
- fast-uri → 3.1.5 (backslash authority host confusion)
- undici → 6.28.0 (cookie injection, retry desync, CRLF blob type)

Closes Dependabot alerts #355, #357, #363, #365, #366, #370, #371, #372,
#373, #375, #376. Supersedes #855 and #866.

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

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Performance

SDK Memory Δ Latency Δ Tests Status
TS 474 B +35% 1.56 µs -22%

Regression detected:

  • TypeScript memory: 350 B → 474 B (+35%)

@github-actions

github-actions Bot commented Aug 3, 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.60% 92.00% ↑ +0.60 pp 🟢
sdk-python 93.82% 93.73% ↑ +0.09 pp 🟢
sdk-typescript 91.05% 90.42% ↑ +0.63 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 3, 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.

npm ci on alpine intermittently omits lightningcss-linux-*-musl when the
lockfile lacks libc metadata, breaking vite build in functional-test
image builds. Use glibc Node images for the UI builder stages instead.

Co-authored-by: Santosh kumar <santoshkumarradha@users.noreply.github.com>
@cursor
cursor Bot marked this pull request as ready for review August 3, 2026 23:28
@cursor
cursor Bot requested review from a team and AbirAbbas as code owners August 3, 2026 23:28
Keep postcss ^8.5.25 / brace-expansion 1.1.18+2.1.4 after #866
landed overlapping web-client dependency bumps on main.

Co-authored-by: Santosh kumar <santoshkumarradha@users.noreply.github.com>
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