Skip to content

build: fix yarn warnings + align typescript version with core/snaps#536

Merged
ccharly merged 7 commits intomainfrom
cc/fix/yarn-warnings
May 4, 2026
Merged

build: fix yarn warnings + align typescript version with core/snaps#536
ccharly merged 7 commits intomainfrom
cc/fix/yarn-warnings

Conversation

@ccharly
Copy link
Copy Markdown
Contributor

@ccharly ccharly commented Apr 28, 2026

Just fixing yarn warnings by updating some deps

Important

We previously upgraded to ts 5.6.3 as this was a suggestion that would get in pair with the new usage of
ts-bridge, but none of our core monorepos are using this version, so I prefer to align it to follow the same pattern everywhere!


Note

Medium Risk
Moderate risk because it changes the TypeScript compiler version across workspaces, which can affect typechecking output and build behavior, though no runtime code paths are modified.

Overview
Aligns the monorepo’s toolchain to reduce yarn warnings by downgrading typescript from ~5.6.3 to ~5.3.3 across the root and multiple packages (including adding typescript where it was missing).

Bumps the formatting tool oxfmt to ^0.45.0 and updates yarn.lock accordingly, plus a small refactor in hd-keyring.test.ts and money-keyring.test.ts to avoid direct indexed access in assertions.

Reviewed by Cursor Bugbot for commit 8d5fde7. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 28, 2026

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedoxfmt@​0.44.0 ⏵ 0.45.09010089 +196 +1100
Updatedtypescript@​5.6.3 ⏵ 5.3.3100 +110090 +19890

View full report

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 28, 2026

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@ccharly ccharly changed the title chore: fix yarn warnings chore: fix yarn warnings + align typescript version with core/snaps Apr 30, 2026
@ccharly ccharly changed the title chore: fix yarn warnings + align typescript version with core/snaps build: fix yarn warnings + align typescript version with core/snaps Apr 30, 2026
@ccharly ccharly marked this pull request as ready for review April 30, 2026 11:49
@ccharly ccharly requested a review from a team as a code owner April 30, 2026 11:49
@ccharly ccharly added this pull request to the merge queue May 4, 2026
Merged via the queue into main with commit 6b33ed0 May 4, 2026
40 checks passed
@ccharly ccharly deleted the cc/fix/yarn-warnings branch May 4, 2026 13:48
@dawnseeker8 dawnseeker8 mentioned this pull request May 6, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants