Skip to content

chore: replace uuid dependency with node:crypto randomUUID - #245

Merged
matt-evervault merged 1 commit into
masterfrom
chore/replace-uuid-node-crypto
Jul 28, 2026
Merged

chore: replace uuid dependency with node:crypto randomUUID#245
matt-evervault merged 1 commit into
masterfrom
chore/replace-uuid-node-crypto

Conversation

@matt-evervault

Copy link
Copy Markdown
Contributor

What changed

Removed the direct uuid npm dependency and replaced its only usage with randomUUID from node:crypto (built-in, no dependency needed).

  • e2e/outboundRelay.test.js: const { v4 } = require('uuid')const { randomUUID } = require('node:crypto'); v4()randomUUID() (CJS, matching the file's existing module system).
  • package.json: removed "uuid": "^8.1.0" from dependencies. (@types/uuid was not present.)
  • package-lock.json: regenerated with npm install --package-lock-only --ignore-scripts. The uuid entry now remains only as a transitive dev dependency (via istanbul-lib-processinfo, used by the nyc coverage tooling) — it is no longer a direct/production dependency.

Unmigrated

None. The single usage was v4(), which maps directly to randomUUID(). No v1/v3/v5/NIL/parse/stringify/validate/version usages were found.

Lockfile status

Updated successfully. --ignore-scripts was required because the prepare (husky install) script is unavailable in this environment; the dependency-tree changes themselves resolved cleanly.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@ev-vaultkeeper

Copy link
Copy Markdown

Vaultkeeper Commands

Mention @ev-vaultkeeper <command> in a PR review thread:

  • review — Review this PR and leave a review.
  • address-comments — Push commits that address the review feedback on this PR.
  • fix-ci — Investigate the failing CI on this PR and push a fix.

You can also request evervault-dependencies as a reviewer to trigger a review.

@changeset-bot

changeset-bot Bot commented Jul 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c13a6d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@matt-evervault matt-evervault self-assigned this Jul 23, 2026
@matt-evervault
matt-evervault marked this pull request as ready for review July 23, 2026 11:31
@matt-evervault
matt-evervault requested a review from a team as a code owner July 23, 2026 11:31
@matt-evervault
matt-evervault merged commit ed24f57 into master Jul 28, 2026
12 of 14 checks passed
@matt-evervault
matt-evervault deleted the chore/replace-uuid-node-crypto branch July 28, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants