Skip to content

Conversation

@cto-new
Copy link
Contributor

@cto-new cto-new bot commented Nov 12, 2025

Summary

Replace all usage of uuidv4 from the uuid package with Node.js crypto.randomUUID() across the js-recall monorepo to remove external dependency and leverage native UUID generation.

Details

  • Replace imports like import { v4 as uuidv4 } from "uuid" with native crypto.randomUUID() or import { randomUUID } from "crypto" as appropriate to existing patterns
  • Update all uuidv4() calls to crypto.randomUUID()
  • Remove uuid from dependencies in root and relevant packages (uuid in api, test-utils, comps)
  • Ensure Node.js version compatibility (Node 14.17+ supports crypto.randomUUID; project uses Node 22)
  • CI steps remain unchanged; validated via local checks to ensure builds and tests pass

Warning: Task VM test is not passing, cto.new will perform much better if you fix the setup

Linear issue

@vercel
Copy link
Contributor

vercel bot commented Nov 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
comps Error Error Nov 12, 2025 0:39am
comps-staging2 Error Error Nov 12, 2025 0:39am

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.

1 participant