Skip to content

fix(deps): resolve 22 npm security vulnerabilities in website and telemetry/ui#773

Open
skrawcz wants to merge 3 commits into
mainfrom
fix/npm-security-updates
Open

fix(deps): resolve 22 npm security vulnerabilities in website and telemetry/ui#773
skrawcz wants to merge 3 commits into
mainfrom
fix/npm-security-updates

Conversation

@skrawcz
Copy link
Copy Markdown
Contributor

@skrawcz skrawcz commented May 11, 2026

Runs npm audit fix in both website/ and telemetry/ui/ to resolve Dependabot security alerts.

website/ (16 alerts fixed, 2 remaining)

Package From To Alerts fixed Severity
next 15.5.12 15.5.18 #160 high
eslint-config-next 15.5.12 15.5.18 aligned with next -
hono 4.12.8 4.12.18 #176 #175 #172 #168 #167 #162 #157 #156 #155 #154 #153 1 low 10 medium
@hono/node-server 1.19.11 1.19.14 #152 medium
fast-uri 3.1.0 3.1.2 #173 #169 2 high
path-to-regexp 8.0.x 8.4.2 #148 #149 1 high 1 medium
ip-address 10.1.0 removed #166 medium

Remaining: postcss (#165) bundled inside next, cannot fix without breaking downgrade to next 9.x.

telemetry/ui/ (4 alerts fixed, 4 remaining)

Package From To Alerts fixed Severity
fast-uri 3.1.0 3.1.2 #174 #170 2 high
@babel/plugin-transform-modules-systemjs 7.29.3 7.29.4 #171 high
next 14.2.14 15.5.15 #159 high (already merged via #739)

Remaining: lodash (#151 #150), postcss (#164), vite (#158) all locked behind react-scripts 5.x (Create React App). Fixing these requires migrating to a modern bundler.

Other

burr/core/graph.py: black formatting fix. The format to fmt rename from #759 left a line that black wants collapsed. Included here since it causes pre-commit failures on every new PR branched off main.

Summary

…try/ui

Run npm audit fix in both website/ and telemetry/ui/ to address
Dependabot security alerts.

website/ (16 alerts → 2 remaining):
- hono 4.12.8 → 4.12.18 (11 alerts)
- @hono/node-server → 1.19.14
- fast-uri → 3.1.2 (2 HIGH)
- path-to-regexp → 8.4.2 (2 alerts)
- ip-address removed
- postcss inside next cannot be fixed without breaking next downgrade

telemetry/ui/ (8 alerts → 4 remaining):
- fast-uri → 3.1.2 (2 HIGH)
- @babel/plugin-transform-modules-systemjs → 7.29.4 (1 HIGH)
- lodash, postcss, vite remain unfixed (locked by react-scripts/CRA)

Resolves 22 of 26 open Dependabot alerts. The remaining 4 require
migrating telemetry/ui from Create React App to a modern bundler.
@skrawcz skrawcz added the dependencies Pull requests that update a dependency file label May 11, 2026
@github-actions github-actions Bot added area/ui Burr UI (telemetry frontend) area/website burr.apache.org website labels May 11, 2026
@github-actions github-actions Bot added the area/core Application, State, Graph, Actions label May 11, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the website’s npm lockfile as the result of running npm audit fix, aiming to reduce known security vulnerabilities by upgrading affected transitive dependencies.

Changes:

  • Upgraded several website/ dependencies in package-lock.json (notably next, hono, @hono/node-server, fast-uri, path-to-regexp, postcss, etc.).
  • Refreshed resolved tarballs / integrity hashes to match the upgraded versions.
Files not reviewed (1)
  • website/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/package-lock.json
Comment on lines 7542 to +7548
"node_modules/next": {
"version": "15.5.12",
"resolved": "https://registry.npmjs.org/next/-/next-15.5.12.tgz",
"integrity": "sha512-Fi/wQ4Etlrn60rz78bebG1i1SR20QxvV8tVp6iJspjLUSHcZoeUXCt+vmWoEcza85ElZzExK/jJ/F6SvtGktjA==",
"version": "15.5.18",
"resolved": "https://registry.npmjs.org/next/-/next-15.5.18.tgz",
"integrity": "sha512-eKL8zUJkX9Y5lE+RX/2YJoItVdGlIscyVyboeD9wSpp0PaGqjoA4tTpT2qPqz9ax+5IzGESyLSeZ/RCwbSZ2uQ==",
"license": "MIT",
"dependencies": {
"@next/env": "15.5.12",
"@next/env": "15.5.18",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

Comment thread website/package-lock.json
Comment on lines 7542 to 7546
"node_modules/next": {
"version": "15.5.12",
"resolved": "https://registry.npmjs.org/next/-/next-15.5.12.tgz",
"integrity": "sha512-Fi/wQ4Etlrn60rz78bebG1i1SR20QxvV8tVp6iJspjLUSHcZoeUXCt+vmWoEcza85ElZzExK/jJ/F6SvtGktjA==",
"version": "15.5.18",
"resolved": "https://registry.npmjs.org/next/-/next-15.5.18.tgz",
"integrity": "sha512-eKL8zUJkX9Y5lE+RX/2YJoItVdGlIscyVyboeD9wSpp0PaGqjoA4tTpT2qPqz9ax+5IzGESyLSeZ/RCwbSZ2uQ==",
"license": "MIT",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

Copy link
Copy Markdown
Collaborator

@andreahlert andreahlert left a comment

Choose a reason for hiding this comment

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

diff in burr/core/graph.py:107-111 looks like it slipped in via pre-commit, outside the deps scope. mind reverting or calling it out in the body?

next major bump (14 → 15) on the website is lockfile-only, but worth confirming npm run build and a local smoke of the site. CI has no build job for website/.

other than that, 22/26 alerts cleared with an honest table on the 4 stuck behind CRA. lgtm after those two.

@skrawcz
Copy link
Copy Markdown
Contributor Author

skrawcz commented May 11, 2026

Thanks for the review!

graph.py change: Yes, that's a pre-commit black formatting fix that slipped in. The formatfmt rename (from #759) made the line short enough for black to collapse it into one line. Main already has this issue (causing pre-commit failures on every new PR), so I included it here rather than a separate PR. Called out in this comment for clarity.

Website build: Confirmed npm run build passes cleanly with the updated deps — static pages generate fine, no issues with the next lockfile bump.

@skrawcz skrawcz enabled auto-merge (squash) May 11, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/core Application, State, Graph, Actions area/ui Burr UI (telemetry frontend) area/website burr.apache.org website dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants