Switch to ESM for source and test, use CommonJS for dist (#806)

__tests__/download/download-version.test.ts

@@ -1,9 +1,10 @@
 import { beforeEach, describe, expect, it, jest } from "@jest/globals";
+import * as semver from "semver";
 
 const mockInfo = jest.fn();
 const mockWarning = jest.fn();
 
-jest.mock("@actions/core", () => ({
+jest.unstable_mockModule("@actions/core", () => ({
   debug: jest.fn(),
   info: mockInfo,
   warning: mockWarning,
@@ -18,20 +19,17 @@ const mockExtractZip = jest.fn<any>();
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockCacheDir = jest.fn<any>();
 
-jest.mock("@actions/tool-cache", () => {
-  const actual = jest.requireActual("@actions/tool-cache") as Record<
-    string,
-    unknown
-  >;
-
-  return {
-    ...actual,
-    cacheDir: mockCacheDir,
-    downloadTool: mockDownloadTool,
-    extractTar: mockExtractTar,
-    extractZip: mockExtractZip,
-  };
-});
+jest.unstable_mockModule("@actions/tool-cache", () => ({
+  cacheDir: mockCacheDir,
+  downloadTool: mockDownloadTool,
+  evaluateVersions: (versions: string[], range: string) =>
+    semver.maxSatisfying(versions, range) ?? "",
+  extractTar: mockExtractTar,
+  extractZip: mockExtractZip,
+  find: () => "",
+  findAllVersions: () => [],
+  isExplicitVersion: (version: string) => semver.valid(version) !== null,
+}));
 
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockGetLatestVersionFromNdjson = jest.fn<any>();
@@ -40,7 +38,7 @@ const mockGetAllVersionsFromNdjson = jest.fn<any>();
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockGetArtifactFromNdjson = jest.fn<any>();
 
-jest.mock("../../src/download/versions-client", () => ({
+jest.unstable_mockModule("../../src/download/versions-client", () => ({
   getAllVersions: mockGetAllVersionsFromNdjson,
   getArtifact: mockGetArtifactFromNdjson,
   getLatestVersion: mockGetLatestVersionFromNdjson,
@@ -53,7 +51,7 @@ const mockGetLatestVersionInManifest = jest.fn<any>();
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockGetManifestArtifact = jest.fn<any>();
 
-jest.mock("../../src/download/version-manifest", () => ({
+jest.unstable_mockModule("../../src/download/version-manifest", () => ({
   getAllVersions: mockGetAllManifestVersions,
   getLatestKnownVersion: mockGetLatestVersionInManifest,
   getManifestArtifact: mockGetManifestArtifact,
@@ -62,15 +60,15 @@ jest.mock("../../src/download/version-manifest", () => ({
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockValidateChecksum = jest.fn<any>();
 
-jest.mock("../../src/download/checksum/checksum", () => ({
+jest.unstable_mockModule("../../src/download/checksum/checksum", () => ({
   validateChecksum: mockValidateChecksum,
 }));
 
-import {
+const {
   downloadVersionFromManifest,
   downloadVersionFromNdjson,
   resolveVersion,
-} from "../../src/download/download-version";
+} = await import("../../src/download/download-version");
 
 describe("download-version", () => {
   beforeEach(() => {

__tests__/download/version-manifest.test.ts

@@ -2,7 +2,7 @@ import { beforeEach, describe, expect, it, jest } from "@jest/globals";
 
 const mockWarning = jest.fn();
 
-jest.mock("@actions/core", () => ({
+jest.unstable_mockModule("@actions/core", () => ({
   debug: jest.fn(),
   info: jest.fn(),
   warning: mockWarning,
@@ -10,16 +10,16 @@ jest.mock("@actions/core", () => ({
 
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockFetch = jest.fn<any>();
-jest.mock("../../src/utils/fetch", () => ({
+jest.unstable_mockModule("../../src/utils/fetch", () => ({
   fetch: mockFetch,
 }));
 
-import {
+const {
   clearManifestCache,
   getAllVersions,
   getLatestKnownVersion,
   getManifestArtifact,
-} from "../../src/download/version-manifest";
+} = await import("../../src/download/version-manifest");
 
 const legacyManifestResponse = JSON.stringify([
   {

__tests__/download/versions-client.test.ts

@@ -2,18 +2,19 @@ import { beforeEach, describe, expect, it, jest } from "@jest/globals";
 
 // biome-ignore lint/suspicious/noExplicitAny: Mock requires flexible typing in tests.
 const mockFetch = jest.fn<any>();
-jest.mock("../../src/utils/fetch", () => ({
+
+jest.unstable_mockModule("../../src/utils/fetch", () => ({
   fetch: mockFetch,
 }));
 
-import {
+const {
   clearCache,
   fetchVersionData,
   getAllVersions,
   getArtifact,
   getLatestVersion,
   parseVersionData,
-} from "../../src/download/versions-client";
+} = await import("../../src/download/versions-client");
 
 const sampleNdjsonResponse = `{"version":"0.9.26","artifacts":[{"platform":"aarch64-apple-darwin","variant":"default","url":"https://github.com/astral-sh/uv/releases/download/0.9.26/uv-aarch64-apple-darwin.tar.gz","archive_format":"tar.gz","sha256":"fcf0a9ea6599c6ae28a4c854ac6da76f2c889354d7c36ce136ef071f7ab9721f"},{"platform":"x86_64-pc-windows-msvc","variant":"default","url":"https://github.com/astral-sh/uv/releases/download/0.9.26/uv-x86_64-pc-windows-msvc.zip","archive_format":"zip","sha256":"eb02fd95d8e0eed462b4a67ecdd320d865b38c560bffcda9a0b87ec944bdf036"}]}
 {"version":"0.9.25","artifacts":[{"platform":"aarch64-apple-darwin","variant":"default","url":"https://github.com/astral-sh/uv/releases/download/0.9.25/uv-aarch64-apple-darwin.tar.gz","archive_format":"tar.gz","sha256":"606b3c6949d971709f2526fa0d9f0fd23ccf60e09f117999b406b424af18a6a6"}]}`;

__tests__/utils/inputs.test.ts

@@ -1,14 +1,3 @@
-jest.mock("@actions/core", () => {
-  return {
-    debug: jest.fn(),
-    getBooleanInput: jest.fn(
-      (name: string) => (mockInputs[name] ?? "") === "true",
-    ),
-    getInput: jest.fn((name: string) => mockInputs[name] ?? ""),
-    warning: jest.fn(),
-  };
-});
-
 import {
   afterEach,
   beforeEach,
@@ -22,6 +11,26 @@ import {
 let mockInputs: Record<string, string> = {};
 const ORIGINAL_HOME = process.env.HOME;
 
+const mockDebug = jest.fn();
+const mockGetBooleanInput = jest.fn(
+  (name: string) => (mockInputs[name] ?? "") === "true",
+);
+const mockGetInput = jest.fn((name: string) => mockInputs[name] ?? "");
+const mockInfo = jest.fn();
+const mockWarning = jest.fn();
+
+jest.unstable_mockModule("@actions/core", () => ({
+  debug: mockDebug,
+  getBooleanInput: mockGetBooleanInput,
+  getInput: mockGetInput,
+  info: mockInfo,
+  warning: mockWarning,
+}));
+
+async function importInputsModule() {
+  return await import("../../src/utils/inputs");
+}
+
 describe("cacheDependencyGlob", () => {
   beforeEach(() => {
     jest.resetModules();
@@ -36,21 +45,21 @@ describe("cacheDependencyGlob", () => {
 
   it("returns empty string when input not provided", async () => {
     mockInputs["working-directory"] = "/workspace";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe("");
   });
 
   it("resolves a single relative path", async () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-dependency-glob"] = "requirements.txt";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe("/workspace/requirements.txt");
   });
 
   it("strips leading ./ from relative path", async () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-dependency-glob"] = "./uv.lock";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe("/workspace/uv.lock");
   });
 
@@ -58,7 +67,7 @@ describe("cacheDependencyGlob", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-dependency-glob"] =
       "  ~/.cache/file1\n ./rel/file2  \nfile3.txt";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe(
       [
         "/home/testuser/.cache/file1", // expanded tilde, absolute path unchanged
@@ -71,7 +80,7 @@ describe("cacheDependencyGlob", () => {
   it("keeps absolute path unchanged in multiline input", async () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-dependency-glob"] = "/abs/path.lock\nrelative.lock";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe(
       ["/abs/path.lock", "/workspace/relative.lock"].join("\n"),
     );
@@ -80,7 +89,7 @@ describe("cacheDependencyGlob", () => {
   it("handles exclusions in relative paths correct", async () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-dependency-glob"] = "!/abs/path.lock\n!relative.lock";
-    const { cacheDependencyGlob } = await import("../../src/utils/inputs");
+    const { cacheDependencyGlob } = await importInputsModule();
     expect(cacheDependencyGlob).toBe(
       ["!/abs/path.lock", "!/workspace/relative.lock"].join("\n"),
     );
@@ -104,7 +113,7 @@ describe("tool directories", () => {
     mockInputs["tool-bin-dir"] = "~/tool-bin-dir";
     mockInputs["tool-dir"] = "~/tool-dir";
 
-    const { toolBinDir, toolDir } = await import("../../src/utils/inputs");
+    const { toolBinDir, toolDir } = await importInputsModule();
 
     expect(toolBinDir).toBe("/home/testuser/tool-bin-dir");
     expect(toolDir).toBe("/home/testuser/tool-dir");
@@ -127,9 +136,7 @@ describe("cacheLocalPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["cache-local-path"] = "~/uv-cache/cache-local-path";
 
-    const { CacheLocalSource, cacheLocalPath } = await import(
-      "../../src/utils/inputs"
-    );
+    const { CacheLocalSource, cacheLocalPath } = await importInputsModule();
 
     expect(cacheLocalPath).toEqual({
       path: "/home/testuser/uv-cache/cache-local-path",
@@ -152,7 +159,7 @@ describe("venvPath", () => {
 
   it("defaults to .venv in the working directory", async () => {
     mockInputs["working-directory"] = "/workspace";
-    const { venvPath } = await import("../../src/utils/inputs");
+    const { venvPath } = await importInputsModule();
     expect(venvPath).toBe("/workspace/.venv");
   });
 
@@ -160,7 +167,7 @@ describe("venvPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["activate-environment"] = "true";
     mockInputs["venv-path"] = "custom-venv";
-    const { venvPath } = await import("../../src/utils/inputs");
+    const { venvPath } = await importInputsModule();
     expect(venvPath).toBe("/workspace/custom-venv");
   });
 
@@ -168,7 +175,7 @@ describe("venvPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["activate-environment"] = "true";
     mockInputs["venv-path"] = "custom-venv/";
-    const { venvPath } = await import("../../src/utils/inputs");
+    const { venvPath } = await importInputsModule();
     expect(venvPath).toBe("/workspace/custom-venv");
   });
 
@@ -176,7 +183,7 @@ describe("venvPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["activate-environment"] = "true";
     mockInputs["venv-path"] = "/tmp/custom-venv";
-    const { venvPath } = await import("../../src/utils/inputs");
+    const { venvPath } = await importInputsModule();
     expect(venvPath).toBe("/tmp/custom-venv");
   });
 
@@ -184,7 +191,7 @@ describe("venvPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["activate-environment"] = "true";
     mockInputs["venv-path"] = "~/.venv";
-    const { venvPath } = await import("../../src/utils/inputs");
+    const { venvPath } = await importInputsModule();
     expect(venvPath).toBe("/home/testuser/.venv");
   });
 
@@ -192,18 +199,11 @@ describe("venvPath", () => {
     mockInputs["working-directory"] = "/workspace";
     mockInputs["venv-path"] = "custom-venv";
 
-    const { activateEnvironment, venvPath } = await import(
-      "../../src/utils/inputs"
-    );
+    const { activateEnvironment, venvPath } = await importInputsModule();
 
     expect(activateEnvironment).toBe(false);
     expect(venvPath).toBe("/workspace/custom-venv");
-
-    const mockedCore = jest.requireMock("@actions/core") as {
-      warning: jest.Mock;
-    };
-
-    expect(mockedCore.warning).toHaveBeenCalledWith(
+    expect(mockWarning).toHaveBeenCalledWith(
       "venv-path is only used when activate-environment is true",
     );
   });

__tests__/version/tool-versions-file.test.ts

@@ -1,113 +1,121 @@
-jest.mock("node:fs");
-jest.mock("@actions/core", () => ({
-  warning: jest.fn(),
+import { beforeEach, describe, expect, it, jest } from "@jest/globals";
+
+const mockReadFileSync = jest.fn();
+const mockWarning = jest.fn();
+
+jest.unstable_mockModule("node:fs", () => ({
+  default: {
+    readFileSync: mockReadFileSync,
+  },
 }));
 
-import fs from "node:fs";
-import * as core from "@actions/core";
-import { beforeEach, describe, expect, it, jest } from "@jest/globals";
-import { getUvVersionFromToolVersions } from "../../src/version/tool-versions-file";
+jest.unstable_mockModule("@actions/core", () => ({
+  warning: mockWarning,
+}));
 
-const mockedFs = fs as jest.Mocked<typeof fs>;
-const mockedCore = core as jest.Mocked<typeof core>;
+async function getVersionFromToolVersions(filePath: string) {
+  const { getUvVersionFromToolVersions } = await import(
+    "../../src/version/tool-versions-file"
+  );
+
+  return getUvVersionFromToolVersions(filePath);
+}
 
 describe("getUvVersionFromToolVersions", () => {
   beforeEach(() => {
+    jest.resetModules();
     jest.clearAllMocks();
   });
 
-  it("should return undefined for non-.tool-versions files", () => {
-    const result = getUvVersionFromToolVersions("package.json");
+  it("should return undefined for non-.tool-versions files", async () => {
+    const result = await getVersionFromToolVersions("package.json");
     expect(result).toBeUndefined();
-    expect(mockedFs.readFileSync).not.toHaveBeenCalled();
+    expect(mockReadFileSync).not.toHaveBeenCalled();
   });
 
-  it("should return version for valid uv entry", () => {
+  it("should return version for valid uv entry", async () => {
     const fileContent = "python 3.11.0\nuv 0.1.0\nnodejs 18.0.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBe("0.1.0");
-    expect(mockedFs.readFileSync).toHaveBeenCalledWith(
-      ".tool-versions",
-      "utf8",
-    );
+    expect(mockReadFileSync).toHaveBeenCalledWith(".tool-versions", "utf8");
   });
 
-  it("should return version for uv entry with v prefix", () => {
+  it("should return version for uv entry with v prefix", async () => {
     const fileContent = "uv v0.2.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBe("0.2.0");
   });
 
-  it("should handle whitespace around uv entry", () => {
+  it("should handle whitespace around uv entry", async () => {
     const fileContent = "  uv   0.3.0  ";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBe("0.3.0");
   });
 
-  it("should skip commented lines", () => {
+  it("should skip commented lines", async () => {
     const fileContent = "# uv 0.1.0\npython 3.11.0\nuv 0.2.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBe("0.2.0");
   });
 
-  it("should return first matching uv version", () => {
+  it("should return first matching uv version", async () => {
     const fileContent = "uv 0.1.0\npython 3.11.0\nuv 0.2.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBe("0.1.0");
   });
 
-  it("should return undefined when no uv entry found", () => {
+  it("should return undefined when no uv entry found", async () => {
     const fileContent = "python 3.11.0\nnodejs 18.0.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBeUndefined();
   });
 
-  it("should return undefined for empty file", () => {
-    mockedFs.readFileSync.mockReturnValue("");
+  it("should return undefined for empty file", async () => {
+    mockReadFileSync.mockReturnValue("");
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBeUndefined();
   });
 
-  it("should warn and return undefined for ref syntax", () => {
+  it("should warn and return undefined for ref syntax", async () => {
     const fileContent = "uv ref:main";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions(".tool-versions");
+    const result = await getVersionFromToolVersions(".tool-versions");
 
     expect(result).toBeUndefined();
-    expect(mockedCore.warning).toHaveBeenCalledWith(
+    expect(mockWarning).toHaveBeenCalledWith(
       "The ref syntax of .tool-versions is not supported. Please use a released version instead.",
     );
   });
 
-  it("should handle file path with .tool-versions extension", () => {
+  it("should handle file path with .tool-versions extension", async () => {
     const fileContent = "uv 0.1.0";
-    mockedFs.readFileSync.mockReturnValue(fileContent);
+    mockReadFileSync.mockReturnValue(fileContent);
 
-    const result = getUvVersionFromToolVersions("path/to/.tool-versions");
+    const result = await getVersionFromToolVersions("path/to/.tool-versions");
 
     expect(result).toBe("0.1.0");
-    expect(mockedFs.readFileSync).toHaveBeenCalledWith(
+    expect(mockReadFileSync).toHaveBeenCalledWith(
       "path/to/.tool-versions",
       "utf8",
     );