Skip to content

chore(gui): consolidate Dependabot updates and lockfile - #203

Open
OnlineChef (ChefGroep) wants to merge 3 commits into
mainfrom
chore/gui-dependabot-lockfile-convergence
Open

chore(gui): consolidate Dependabot updates and lockfile#203
OnlineChef (ChefGroep) wants to merge 3 commits into
mainfrom
chore/gui-dependabot-lockfile-convergence

Conversation

@ChefGroep

@ChefGroep OnlineChef (ChefGroep) commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Validation

  • bun install --frozen-lockfile
  • bun run typecheck
  • bun run privacy:scan
  • cd gui && bun run lint
  • cd gui && bun run build
  • cd gui && bun test tests --isolate — 456 passed, 0 failed

Supersedes #198, #199, #200, #201 and #202.

Summary by CodeRabbit

  • Chores
    • Updated several development and UI package dependencies to newer compatible versions.
    • No user-facing functionality or application behavior was changed.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 4ae8df0d-d148-4f69-800c-8904dc6f628f

📥 Commits

Reviewing files that changed from the base of the PR and between 1e07891 and d63aa41.

⛔ Files ignored due to path filters (1)
  • gui/bun.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • gui/package.json

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The GUI package manifest updates six dependency versions. No scripts, package structure, or other dependency entries changed.

Changes

GUI dependency updates

Layer / File(s) Summary
Update GUI dependency versions
gui/package.json
Updates @designcodeio/threeui, @types/node, @types/react-dom, eslint-plugin-react-refresh, globals, and happy-dom to newer versions. Other listed dependencies remain unchanged.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to d63aa

This updates GUI runtime and development dependencies with a regenerated lockfile; the reported installation and GUI validation pass, with no remaining merge-blocking risk.

Suggested reviewers: wibias, lidge-jun

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and concisely describes the primary changes: consolidating GUI Dependabot dependency updates and regenerating the lockfile.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/gui-dependabot-lockfile-convergence

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@socket-security

socket-security Bot commented Sep 7, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedhappy-dom@​20.12.0 ⏵ 20.14.0661008896 +1100
Updated@​types/​react-dom@​19.2.5 ⏵ 19.2.7100 +110075 +193100
Updated@​types/​node@​26.4.0 ⏵ 26.5.010010081 +196100
Updatedglobals@​17.11.0 ⏵ 17.12.0100 +110086 +195 -1100
Updated@​designcodeio/​threeui@​1.1.0 ⏵ 1.2.086 +810097 +291 +190
Updatedeslint-plugin-react-refresh@​0.5.5 ⏵ 0.5.6100 +110099 +194 +1100

View full report

@socket-security

socket-security Bot commented Sep 7, 2026

Copy link
Copy Markdown

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
License policy violation: npm @designcodeio/threeui under OFL-1.1

License: OFL-1.1 - The applicable license policy does not permit this license (5) (package/THIRD_PARTY_NOTICES.md)

License: OFL-1.1 - The applicable license policy does not permit this license (5) (package/ASSET-LICENSES.md)

License: OFL-1.1 - The applicable license policy does not permit this license (5) (package/FONT-LICENSES.md)

From: gui/package.jsonnpm/@designcodeio/threeui@1.2.0

ℹ Read more on: This package | This alert | What is a license policy violation?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Find a package that does not violate your license policy or adjust your policy to allow this package's license.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@designcodeio/threeui@1.2.0. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Dependabot lockfile regeneration typo left a wrong sha512 for
@fontsource/instrument-serif, breaking frozen bun install in CI.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant