Skip to content

chore(deps): bump pbkdf2, sha.js and cipher-base in yarn.lock - #258

Merged
CassioMG merged 1 commit into
mainfrom
chore/bump-crypto-polyfill-deps
Oct 14, 2026
Merged

CassioMG merged 1 commit into
mainfrom
chore/bump-crypto-polyfill-deps

Conversation

@CassioMG

Copy link
Copy Markdown
Contributor

TL;DR

Lockfile-only refresh that moves three browserify crypto polyfill packages to newer patch versions: pbkdf2, sha.js and cipher-base. No package.json changes and no behavior changes. The goal is to clear the open Dependabot alerts.

Implementation details (for agents)

What changed: only yarn.lock. The three entries were dropped and re-resolved by yarn install within their existing semver ranges. The new lock entries are the helper packages the newer versions depend on (to-buffer, hash-base@3.1.2, ripemd160@2.0.3, typed-array-buffer, call-bound, …), added alongside the existing ones. No other package changed version.

Package From To Pulled in by
pbkdf2 3.1.2 3.1.7 crypto-browserify
sha.js 2.4.11 2.4.12 create-hash, @ledgerhq/hw-app-str
cipher-base 1.0.4 1.0.7 create-hash, browserify-aes

Downstream impact: consumers who install from npm resolve their own versions of these packages, so the lockfile has no effect on them. Only the contents of our prebuilt bundles change, at the next release.

Verification:

  • yarn why reports a single resolved version for each package: pbkdf2@3.1.7, sha.js@2.4.12, cipher-base@1.0.7.
  • yarn build succeeds for all three packages.
  • npx jest --ci: 17 suites passed (1 skipped), 343 tests passed, 5 skipped.
  • sha.js@2.4.12 sha256 on a random Buffer matches Node's crypto.createHash('sha256').

Follow-ups / out of scope: core's crypto-browserify polyfill exists only because jws → jwa imports crypto, and core only uses jws.decode. A small base64url JWT decoder would let us drop jws and the polyfill (about 600 KB of browserify code) from the core browser bundle.

🤖 Generated with Claude Code

Lockfile-only refresh of transitive dependencies:
- pbkdf2 3.1.2 -> 3.1.7 (via crypto-browserify)
- sha.js 2.4.11 -> 2.4.12 (via create-hash, @ledgerhq/hw-app-str)
- cipher-base 1.0.4 -> 1.0.7 (via create-hash / browserify-aes)

No package.json changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 14, 2026 18:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@CassioMG CassioMG self-assigned this Oct 14, 2026
@CassioMG
CassioMG requested a review from a team October 14, 2026 18:47
@CassioMG
CassioMG merged commit d512fba into main Oct 14, 2026
8 checks passed
@CassioMG
CassioMG deleted the chore/bump-crypto-polyfill-deps branch October 14, 2026 19:10
@CassioMG CassioMG mentioned this pull request Oct 15, 2026
CassioMG added a commit that referenced this pull request Oct 19, 2026
* Release/5.1.0

Bumps all three packages to `5.1.0` and updates their changelogs.

* fix(recovery): validate SEP-30 signer set before locking the master key (#255)
* fix(km): make changePassword work with BrowserStorageKeyStore (#257)
* chore(deps): bump pbkdf2, sha.js and cipher-base in yarn.lock (#258)
* fix(km): make Albedo and Freighter signing follow the configured network (#259)

A minor release rather than a patch: #255 adds four exported error
classes, and #255 and #259 make calls that used to succeed throw.

`@stellar/typescript-wallet-sdk-soroban` has no shipped changes since 5.0.0
and is bumped to stay in lockstep, matching #250.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* test: raise testnet test timeouts to 30s

Unit tests that fund accounts through friendbot or authenticate against
testanchor kept timing out in CI: friendbot fundings were taking 4-30s,
against hook and test limits of 5-20s and Jest's 5s default. The
timeouts fail `yarn test:ci` in the npm publish workflows before the
publish step runs.

- Set `testTimeout: 30000` at the top level of jest.config.js. It's a
  global option, so it can't go in the per-project config.
- Raise every explicit test and hook timeout under 30s to 30s. Those
  already at 30s or more are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants