chore: remove uuid from the backend (#10807)

  Backend only of: #10806

  ## About the changes
  This PR drops the uuid package from node modules and replaces it with
  standard randomUUID usage that is available from 14.17 onwards, and we
  have a minimum requirement of node 20 at Unleash.

  [Node.js
  crypto](https://nodejs.org/api/crypto.html#cryptorandomuuidoptions)
  [Web
  crypto](https://developer.mozilla.org/en-US/docs/Web/API/Crypto/randomUUID)

  Co-authored-by: Anastasiia Hladina <[email protected]>

Author: gastonfournier

package.json

@@ -129,8 +129,7 @@
     "ts-toolbelt": "^9.6.0",
     "type-is": "^2.0.0",
     "ulidx": "^2.4.1",
-    "unleash-client": "^6.8.0-beta.0",
-    "uuid": "^11.0.0"
+    "unleash-client": "^6.8.0-beta.0"
   },
   "devDependencies": {
     "@apidevtools/swagger-parser": "10.1.1",
@@ -161,7 +160,6 @@
     "@types/stoppable": "1.1.3",
     "@types/supertest": "6.0.3",
     "@types/type-is": "1.6.7",
-    "@types/uuid": "9.0.8",
     "@vitest/coverage-v8": "^3.1.3",
     "@vitest/ui": "^3.1.3",
     "concurrently": "^9.0.0",

src/lib/db/feature-environment-store.ts

@@ -8,9 +8,9 @@ import metricsHelper from '../util/metrics-helper.js';
 import { DB_TIME } from '../metric-events.js';
 import type { IFeatureEnvironment, IVariant } from '../types/model.js';
 import NotFoundError from '../error/notfound-error.js';
-import { v4 as uuidv4 } from 'uuid';
 import type { Db } from './db.js';
 import type { IUnleashConfig } from '../types/index.js';
+import { randomId } from '../util/index.js';
 
 const T = {
     featureEnvs: 'feature_environments',
@@ -495,7 +495,7 @@ export class FeatureEnvironmentStore implements IFeatureEnvironmentStore {
             const clonedStrategyRows = sourceFeatureStrategies.map(
                 (featureStrategy) => ({
                     ...featureStrategy,
-                    id: uuidv4(),
+                    id: randomId(),
                     environment: destinationEnvironment,
                     parameters: JSON.stringify(featureStrategy.parameters),
                     constraints: JSON.stringify(featureStrategy.constraints),

src/lib/error/unleash-error.ts

@@ -1,5 +1,5 @@
-import { v4 as uuidV4 } from 'uuid';
 import type { FromSchema } from 'json-schema-to-ts';
+import { randomId } from '../util/random-id.js';
 
 export const UnleashApiErrorTypes = [
     'ContentTypeError',
@@ -48,7 +48,7 @@ export abstract class UnleashError extends Error {
 
     constructor(message: string, name?: string) {
         super();
-        this.id = uuidV4();
+        this.id = randomId();
         this.name = name || this.constructor.name;
         super.message = message;
     }

src/lib/features/dependent-features/dependent.features.e2e.test.ts

@@ -1,4 +1,3 @@
-import { v4 as uuidv4 } from 'uuid';
 import dbInit, {
     type ITestDb,
 } from '../../../test/e2e/helpers/database-init.js';
@@ -13,7 +12,7 @@ import {
     FEATURE_DEPENDENCY_ADDED,
     FEATURE_DEPENDENCY_REMOVED,
 } from '../../events/index.js';
-import { DEFAULT_ENV } from '../../util/index.js';
+import { DEFAULT_ENV, randomId } from '../../util/index.js';
 import type { IEventStore } from '../../server-impl.js';
 
 let app: IUnleashTest;
@@ -169,9 +168,9 @@ const checkDependenciesExist = async (expectedCode = 200) => {
 };
 
 test('should add and delete feature dependencies', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
-    const child2 = uuidv4();
+    const parent = randomId();
+    const child = randomId();
+    const child2 = randomId();
     await app.createFeature(parent);
     await app.createFeature(child);
     await app.createFeature(child2);
@@ -209,10 +208,10 @@ test('should add and delete feature dependencies', async () => {
 });
 
 test('should sort potential parent features alphabetically', async () => {
-    const parent1 = `a${uuidv4()}`;
-    const parent2 = `c${uuidv4()}`;
-    const parent3 = `b${uuidv4()}`;
-    const child = uuidv4();
+    const parent1 = `a${randomId()}`;
+    const parent2 = `c${randomId()}`;
+    const parent3 = `b${randomId()}`;
+    const child = randomId();
     await app.createFeature(parent1);
     await app.createFeature(parent2);
     await app.createFeature(parent3);
@@ -223,7 +222,7 @@ test('should sort potential parent features alphabetically', async () => {
 });
 
 test('should sort potential parent variants', async () => {
-    const parent = uuidv4();
+    const parent = randomId();
     await app.createFeature(parent);
     await addFeatureEnvironmentVariant(parent, 'e');
     await addStrategyVariants(parent, ['c', 'a', 'd']);
@@ -235,9 +234,9 @@ test('should sort potential parent variants', async () => {
 });
 
 test('should not allow to add grandparent', async () => {
-    const grandparent = uuidv4();
-    const parent = uuidv4();
-    const child = uuidv4();
+    const grandparent = randomId();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(grandparent);
     await app.createFeature(parent);
     await app.createFeature(child);
@@ -255,9 +254,9 @@ test('should not allow to add grandparent', async () => {
 });
 
 test('should not allow to add grandchild', async () => {
-    const grandparent = uuidv4();
-    const parent = uuidv4();
-    const child = uuidv4();
+    const grandparent = randomId();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(grandparent);
     await app.createFeature(parent);
     await app.createFeature(child);
@@ -276,8 +275,8 @@ test('should not allow to add grandchild', async () => {
 });
 
 test('should not allow to add non-existent parent dependency', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(child);
 
     await addFeatureDependency(
@@ -290,8 +289,8 @@ test('should not allow to add non-existent parent dependency', async () => {
 });
 
 test('should not allow to add archived parent dependency', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(child);
     await app.createFeature(parent);
     await app.archiveFeature(parent);
@@ -306,8 +305,8 @@ test('should not allow to add archived parent dependency', async () => {
 });
 
 test('should check if any dependencies exist', async () => {
-    const parent = uuidv4();
-    const child = uuidv4();
+    const parent = randomId();
+    const child = randomId();
     await app.createFeature(child);
     await app.createFeature(parent);
 
@@ -323,7 +322,7 @@ test('should check if any dependencies exist', async () => {
 });
 
 test('should not allow to add dependency to self', async () => {
-    const parent = uuidv4();
+    const parent = randomId();
     await app.createFeature(parent);
 
     await addFeatureDependency(
@@ -336,8 +335,8 @@ test('should not allow to add dependency to self', async () => {
 });
 
 test('should not allow to add dependency to feature from another project', async () => {
-    const child = uuidv4();
-    const parent = uuidv4();
+    const child = randomId();
+    const parent = randomId();
     await app.createFeature(parent);
     await createProject('another-project');
     await app.createFeature(child, 'another-project');
@@ -351,8 +350,8 @@ test('should not allow to add dependency to feature from another project', async
     );
 });
 test('should create feature-dependency-removed when archiving and has dependency', async () => {
-    const child = uuidv4();
-    const parent = uuidv4();
+    const child = randomId();
+    const parent = randomId();
     await app.createFeature(parent);
     await app.createFeature(child);
 
@@ -369,8 +368,8 @@ test('should create feature-dependency-removed when archiving and has dependency
 });
 
 test('should not create feature-dependency-removed when archiving and no dependency', async () => {
-    const child = uuidv4();
-    const parent = uuidv4();
+    const child = randomId();
+    const parent = randomId();
     await app.createFeature(parent);
     await app.createFeature(child);
 

src/lib/features/feature-toggle/feature-toggle-strategies-store.ts

@@ -1,6 +1,5 @@
 import { Knex } from 'knex';
 import type EventEmitter from 'events';
-import { v4 as uuidv4 } from 'uuid';
 import metricsHelper from '../../util/metrics-helper.js';
 import { DB_TIME } from '../../metric-events.js';
 import type { Logger, LogProvider } from '../../logger.js';
@@ -24,6 +23,7 @@ import {
     ensureStringValue,
     generateImageUrl,
     mapValues,
+    randomId,
 } from '../../util/index.js';
 import type { IFeatureProjectUserParams } from './feature-toggle-controller.js';
 import type { Db } from '../../db/db.js';
@@ -238,7 +238,7 @@ class FeatureStrategiesStore implements IFeatureStrategiesStore {
                 strategyConfig.environment,
             ));
         const strategyRow = mapInput({
-            id: uuidv4(),
+            id: randomId(),
             ...strategyConfig,
             sortOrder,
         });