Skip to content

fix(deps): resolve npm audit vulnerabilities - #3809

Closed
0xMudit wants to merge 1 commit into
apache:mainfrom
0xMudit:fix/3724-npm-audit-vulnerabilities
Closed

fix(deps): resolve npm audit vulnerabilities#3809
0xMudit wants to merge 1 commit into
apache:mainfrom
0xMudit:fix/3724-npm-audit-vulnerabilities

Conversation

@0xMudit

@0xMudit 0xMudit commented Aug 25, 2026

Copy link
Copy Markdown

Summary

Fixes #3724

Resolves three npm audit vulnerabilities by running
pm audit fix:

  • @babel/core <=7.29.0: Arbitrary File Read via sourceMappingURL Comment (GHSA-4x5r-pxfx-6jf8)
  • brace-expansion 4.0.0-5.0.8: DoS via unbounded expansion length (GHSA-mh99-v99m-4gvg, GHSA-rgw5-rvv9-x895)
  • undici 7.0.0-7.28.0: Multiple vulnerabilities in @electron/get\ transitive dependency (5 advisories)

All fixes are minor/patch version bumps within existing semver ranges. Only \package-lock.json\ is modified.

Verification


  • pm audit: 0 vulnerabilities (was 3)

  • pm run lint: passed

  • pm run format:check: passed

  • pm run build: passes (pre-existing @maka/ui\ type errors are unrelated to this change)

  • pm --workspace @maka/core test: 656/656 pass

  • pm run typecheck: passes (pre-existing @maka/ui\ errors are on clean \main\ too)

AI use

  • Generative tooling made a substantive contribution

Tool(s) and scope: opencode/big-pickle — identified vulnerabilities, ran audit fix, verified results

Checklist

  • Tests cover the change and fail without it
  • Lint, format, typecheck and the affected suites pass locally

Does this PR entail a change in behavior?

  • No

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Update on ebe6d30: CODE GO — lockfile only (+59/-56), deps patch (Babel 7.29.7, brace-expansion, undici), 0 audit vuln, lint green. No P0-P3.

简体中文仅锁文件更新,无阻断。

@Phoenix500526

Copy link
Copy Markdown
Contributor

Update on ebe6d30: CODE GO — lockfile only (+59/-56), deps patch (Babel 7.29.7, brace-expansion, undici), 0 audit vuln, lint green. No P0-P3.

简体中文
仅锁文件更新,无阻断。

This PR is duplicated to PR #3725 @Astro-Han

@M4n5ter
M4n5ter force-pushed the fix/3724-npm-audit-vulnerabilities branch from ebe6d30 to fc1e96c Compare August 26, 2026 09:00
- @babel/core: arbitrary file read via sourceMappingURL (GHSA-4x5r-pxfx-6jf8)
- brace-expansion: DoS via unbounded expansion (GHSA-mh99-v99m-4gvg, GHSA-rgw5-rvv9-x895)
- undici: multiple vulnerabilities in @electron/get transitive dependency

Generated-by: opencode/big-pickle
@M4n5ter
M4n5ter force-pushed the fix/3724-npm-audit-vulnerabilities branch from fc1e96c to a78c545 Compare August 26, 2026 09:56
@M4n5ter M4n5ter mentioned this pull request Aug 27, 2026
6 tasks
@github-actions github-actions Bot added the effort/XS Under 10 readable lines label Aug 27, 2026
@M4n5ter

M4n5ter commented Aug 27, 2026

Copy link
Copy Markdown
Member

Thank you for fixing these dependency vulnerabilities and verifying the updated lockfile. PR #3725 was the earlier canonical fix for the same issue and has now merged as 9d62ee9a8e16f6ad75e9e31beb7d4644d49ad8d9; its lockfile resolves the same three audit findings and #3724 is closed. I am therefore closing this duplicate as superseded so the repository keeps one dependency-update owner.


Posted by an automated review agent operated by @M4n5ter. This is not an independent human review and does not satisfy the committer review required by CONTRIBUTING.md. A human is accountable for this comment — please push back if anything here is wrong.

简体中文

本条评论由 @M4n5ter 运行的自动化审查程序发出。它不构成 CONTRIBUTING.md 所要求的独立人类审查,也不能替代人类审查。有人类对本条评论负责,如有错误请直接指出。

@M4n5ter M4n5ter closed this Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/XS Under 10 readable lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dep: npm audit report three vulnerabilities

4 participants