refactor: Remove UUID package in favour of node:crypto

Author: Codex-

dist/index.mjs

@@ -30,16 +30,14 @@
   "license": "MIT",
   "dependencies": {
     "@actions/core": "^1.11.1",
-    "@actions/github": "^6.0.0",
-    "uuid": "^10.0.0"
+    "@actions/github": "^6.0.0"
   },
   "devDependencies": {
     "@eslint/js": "^9.17.0",
     "@opentf/std": "^0.13.0",
     "@total-typescript/ts-reset": "^0.6.1",
     "@types/eslint__js": "^8.42.3",
     "@types/node": "~20.17.10",
-    "@types/uuid": "^10.0.0",
     "@typescript-eslint/eslint-plugin": "^8.19.0",
     "@typescript-eslint/parser": "^8.19.0",
     "@vitest/coverage-v8": "^2.1.8",

package.json

@@ -1,13 +1,14 @@
+import { randomUUID } from "node:crypto";
+
 import * as core from "@actions/core";
-import { v4 } from "uuid";
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 
 import { type ActionConfig, getConfig } from "./action.ts";
 
-vi.mock("@actions/core");
-vi.mock("uuid", () => ({
-  v4: vi.fn(),
+vi.mock("node:crypto", () => ({
+  randomUUID: vi.fn(),
 }));
+vi.mock("@actions/core");
 
 describe("Action", () => {
   const workflowInputs = {
@@ -126,12 +127,12 @@ describe("Action", () => {
     });
 
     it("should handle no distinct_id being provided", () => {
-      const v4Mock = vi.mocked(v4);
-      v4Mock.mockImplementationOnce(() => "test-uuid-is-used");
+      const v4Mock = vi.mocked(randomUUID);
+      v4Mock.mockImplementationOnce(() => "test-mocked-uuid-is-used");
       mockEnvConfig.distinct_id = "";
       const config: ActionConfig = getConfig();
 
-      expect(config.distinctId).toStrictEqual("test-uuid-is-used");
+      expect(config.distinctId).toStrictEqual("test-mocked-uuid-is-used");
       expect(v4Mock).toHaveBeenCalledOnce();
     });
   });

pnpm-lock.yaml

@@ -1,5 +1,6 @@
+import { randomUUID } from "node:crypto";
+
 import * as core from "@actions/core";
-import { v4 as uuid } from "uuid";
 
 const WORKFLOW_TIMEOUT_SECONDS = 5 * 60;
 
@@ -69,7 +70,7 @@ export function getConfig(): ActionConfig {
       getNumberFromValue(core.getInput("workflow_timeout_seconds")) ??
       WORKFLOW_TIMEOUT_SECONDS,
     distinctId:
-      getOptionalWorkflowValue(core.getInput("distinct_id")) ?? uuid(),
+      getOptionalWorkflowValue(core.getInput("distinct_id")) ?? randomUUID(),
   };
 }
 

src/action.spec.ts

@@ -1,5 +1,6 @@
+import { randomUUID } from "node:crypto";
+
 import * as core from "@actions/core";
-import { v4 as uuid } from "uuid";
 import {
   afterAll,
   afterEach,
@@ -175,7 +176,7 @@ describe("return-dispatch", () => {
   });
 
   describe("attemptToFindRunId", () => {
-    const testId = uuid();
+    const testId = randomUUID();
 
     let getWorkflowRunJobStepMock: MockInstance<
       typeof api.fetchWorkflowRunJobSteps