chore(deps): update dependencies and clear transitive advisories - #72
Merged
Conversation
Bump dev tooling (biome 2.5.5, changesets 2.31.1, fs-extra 11.4.0, knip 6.29.0, tsdown 0.22.14, react/react-dom 19.2.8) and the @wallet-standard patch line. Five of seven Dependabot advisories clear from the lockfile refresh alone (brace-expansion, js-yaml x2, fast-uri). The remaining two are pinned by dependents that have not moved yet, so they need overrides: postcss by vite (dev only) and valibot by bitcoinjs-lib. tsdown 0.22.14 deprecates deps.skipNodeModulesBundle in favour of deps.neverBundle; the build emitted a warning for all three packages. Verified the emitted output still leaves dependencies external. Refresh minimumReleaseAgeExclude to the versions the lockfile now pins — every entry referred to the superseded tsdown 0.22.7 tree, and tsdown 0.22.14 is ~23h old, right at the gate that broke the release pipeline in #63. @noble/hashes 2.x is deliberately excluded, see PR description.
🦋 Changeset detectedLatest commit: 115ee72 The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Dependency maintenance. No behaviour change in any published package.
Direct bumps
@biomejs/biome@changesets/clifs-extrakniptsdownreact/react-dom@bigmi/react, dev@wallet-standard/app/base@bigmi/client, runtimeThe
@wallet-standardchange is a range bump only —^1.1.0already resolved to 1.1.1, so nothing shifts for consumers.Security advisories
All seven open Dependabot alerts are now clear. Five resolve from the lockfile refresh alone:
brace-expansion→ 5.0.8 (2 alerts, high)js-yaml→ 3.15.0 / 4.3.0 (2 alerts, high)fast-uri→ 3.1.4 (high)The other two are pinned by dependents that have not moved yet, so they need
overridesinpnpm-workspace.yaml:postcss8.5.15 → ≥8.5.18 (high) — pinned byvite@8.1.0, dev-onlyvalibot1.4.1 → ≥1.4.2 (medium) — pinned bybitcoinjs-lib@7.0.1Each override carries a note to drop it once the parent ships a release that resolves past the patched version. Worth knowing: the
valibotoverride fixes our lockfile, not downstream consumers' — they resolvebitcoinjs-lib's own range. The real fix there is upstream.tsdown deprecation
0.22.14 deprecates
deps.skipNodeModulesBundlein favour ofdeps.neverBundle, and the build started warning twice per package. Updated all threetsdown.config.tsfiles. Verified the emitted output is equivalent —packages/core/dist/esm/utils/getAddressInfo.jsstill hasimport { sha256 } from "@noble/hashes/sha256"rather than an inlined copy, so dependencies stay external.Release-age gate
minimumReleaseAgeExcludestill listed the supersededtsdown@0.22.7tree — every entry was stale (yuku-*@0.6.1,rolldown-plugin-dts@0.27.8). Refreshed to what the lockfile now pins (0.8.0/0.27.14/0.22.14).This is not cosmetic:
tsdown@0.22.14was published ~23h before this PR, right at the gate that broke the release pipeline in #63. Verified withpnpm install --frozen-lockfile, which re-validated all 564 entries in 10.4s from cold rather than reusing the cached result.Deliberately excluded:
@noble/hashes2.xpnpm outdatedreports 1.8.0 → 2.2.0, but it should not go in a routine bump:bitcoinjs-lib@7.0.1andbs58check@4.0.0both declare^1.2.0, which cannot resolve to 2.x. Today all three dependents dedupe to a single 1.8.0; bumping only@bigmi/coreputs two copies of the hashing library in every consumer's bundle../sha256subpath — its exports map has only./sha2.js.getAddressInfo.ts:1doesimport { sha256 } from '@noble/hashes/sha256', which would fail to resolve outright.So it needs its own PR with a
@bigmi/corechangeset, and is best done oncebitcoinjs-libsupports v2 — otherwise consumers pay the duplication.One preparatory step is available now if wanted:
@noble/hashes/sha2.jsexportssha256on both 1.8.0 and 2.2.0 (verified against the installed 1.8.0), so switching the import path is forward-compatible and would decouple that change from the version bump.Validation
pnpm check,check:types,check:circular-deps,knip:check,test(28 client + 32 core / 1 skipped), andbuildall pass. Empty changeset included — release-less by design.