📦 Dependencies/fix critical and high#1796
Merged
adriangohjw merged 12 commits intofeat/use-bunfrom Mar 10, 2026
Merged
Conversation
…ility and performance
…i-regex versions in bun.lock and package.json
2 tasks
… multiple package.json files
seaerchin
approved these changes
Feb 10, 2026
…to dependencies/fix-critical-and-high
Contributor
There was a problem hiding this comment.
Automated risk assessment for this update:
- Risk level:
Medium-High - Why: The diff is dependency-manifest/lockfile only, but it updates a large set of runtime and build dependencies across
apps/studio,packages/components, root workspace config, and multiple tooling packages. This creates broad behavioral surface area and non-trivial regression risk despite no direct source-code edits. - Code review required:
Yes - Approval action:
Not approved by automation(per policy: Medium-High risk requires human review; also avoid self-approval when CODEOWNERS review applies). - Reviewer action: Requested
@dcshzjand@gautammmanekbased on recent/historical edits on the touched package-manifest paths.
Existing approval state was detected on the PR; no new automation approval was added.
Contributor
There was a problem hiding this comment.
Security Review Result
PASS - No high-confidence vulnerabilities found in this PR diff.
I reviewed the changes with focus on injection, auth/authz boundaries, secret handling, SSRF/XSS/CSRF, unsafe deserialization, and supply-chain risk. The diff is limited to dependency manifests and bun.lock; no application/runtime logic changes were introduced.
Confirmed Findings
- None.
Lower-Confidence Supply-Chain Concern (Validation Needed)
- Some dependencies changed from exact pins to caret ranges (for example
@aws-sdk/client-s3,@aws-sdk/s3-request-presigner,@tiptap/extension-linkinapps/studio/package.json).
If any environment performs non-frozen installs, future upstream releases could be pulled unexpectedly.
Recommended validation/remediation:
- Ensure CI/CD and production builds use frozen lockfile installs (for Bun, enforce
bun install --frozen-lockfile). - Consider exact pinning for security-sensitive runtime dependencies if strict reproducibility is required.
Overall, this PR appears to reduce known dependency exposure and does not introduce a concrete exploitable issue based on the current diff.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Problem
Fix most high and all critical dependencies
before
After (WIP)
Note
Medium Risk
Large set of dependency upgrades (including
next,@trpc/*, TipTap, and observability libs) may introduce subtle runtime/build behavior changes despite no application code changes.Overview
Updates
apps/studio/package.jsonto pull in newer dependency versions to address reported critical/high vulnerabilities.This is primarily a broad dependency refresh across core runtime libraries (e.g.
next,@trpc/*, TipTap,dd-trace,isomorphic-dompurify) and tooling/test stack (e.g.eslint,prettier,playwright,msw,vitest).Written by Cursor Bugbot for commit 7419048. This will update automatically on new commits. Configure here.