release: prepare v5.1.0 (#249)

* release: prepare v5.1.0 — ESM migration with esbuild/vitest

* sync package-lock.json version to 5.1.0

* add version sync check to pre-commit hook

* regenerate package-lock.json for npm ci compatibility

.github/workflows/codeql.yml

@@ -59,7 +59,7 @@ jobs:
 
          # Initializes the CodeQL tools for scanning.
          - name: Initialize CodeQL
-           uses: github/codeql-action/init@b1bff81932f5cdfc8695c7752dcee935dcd061c8 # v3.29.5
+           uses: github/codeql-action/init@c10b8064de6f491fea524254123dbe5e09572f13 # v4.35.1
            with:
               languages: ${{ matrix.language }}
               build-mode: ${{ matrix.build-mode }}
@@ -86,6 +86,6 @@ jobs:
               echo '  make release'
               exit 1
          - name: Perform CodeQL Analysis
-           uses: github/codeql-action/analyze@b1bff81932f5cdfc8695c7752dcee935dcd061c8 # v3.29.5
+           uses: github/codeql-action/analyze@c10b8064de6f491fea524254123dbe5e09572f13 # v4.35.1
            with:
               category: '/language:${{matrix.language}}'

.github/workflows/unit-tests.yml

@@ -18,4 +18,5 @@ jobs:
          - name: Build and run L0 tests.
            run: |
               npm install
+              npm run build
               npm test

.gitignore

@@ -330,3 +330,4 @@ ASALocalRun/
 node_modules
 
 # Transpiled JS
+lib/

.husky/pre-commit

@@ -1,9 +1,32 @@
 set +e
 npm test
-# Run format check
 npm run format-check || {
   echo ""
   echo "❌ Formatting check failed."
   echo "💡 Run 'npm run format' or 'prettier --write .' to fix formatting issues."
   exit 1
-}
+}
+
+STAGED=$(git diff --cached --name-only)
+if echo "$STAGED" | grep -qE '^(package\.json|package-lock\.json|CHANGELOG\.md)$'; then
+  PKG_VERSION=$(node -p "require('./package.json').version")
+  LOCK_VERSION=$(node -p "require('./package-lock.json').version")
+  CHANGELOG_VERSION=$(sed -n 's/^## \[\([^]]*\)\].*/\1/p' CHANGELOG.md | head -1)
+
+  MISMATCH=""
+  if [ "$PKG_VERSION" != "$LOCK_VERSION" ]; then
+    MISMATCH="${MISMATCH}\n  package.json:      ${PKG_VERSION}\n  package-lock.json: ${LOCK_VERSION}"
+  fi
+  if [ "$PKG_VERSION" != "$CHANGELOG_VERSION" ]; then
+    MISMATCH="${MISMATCH}\n  package.json: ${PKG_VERSION}\n  CHANGELOG.md: ${CHANGELOG_VERSION}"
+  fi
+
+  if [ -n "$MISMATCH" ]; then
+    echo ""
+    echo "❌ Version mismatch detected:"
+    echo -e "$MISMATCH"
+    echo ""
+    echo "💡 Ensure package.json, package-lock.json, and CHANGELOG.md all have the same version."
+    exit 1
+  fi
+fi

CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [5.1.0] - 2026-04-11
+
+### Changed
+
+- #243 [Migrate to ESM with esbuild and vitest](https://github.com/Azure/setup-kubectl/pull/243)
+   - Replaced `@vercel/ncc` with `esbuild` for ESM bundling
+   - Replaced `jest`/`ts-jest` with `vitest` for testing
+   - Upgraded `@actions/core` to `^3.0.0`, `@actions/exec` to `^3.0.0`, `@actions/tool-cache` to `^4.0.0`
+   - Updated `tsconfig.json` to `NodeNext` module resolution
+- Add `npm run build` step to CI unit-tests workflow
+
+### Security
+
+- #242 [Bump picomatch](https://github.com/Azure/setup-kubectl/pull/242)
+- #244 [Bump handlebars from 4.7.8 to 4.7.9](https://github.com/Azure/setup-kubectl/pull/244)
+- #247 [Bump vite from 8.0.3 to 8.0.5](https://github.com/Azure/setup-kubectl/pull/247)
+- #245 [Bump github/codeql-action in CI workflows](https://github.com/Azure/setup-kubectl/pull/245)
+
 ## [5.0.0] - 2026-03-25
 
 ### Changed

jest.config.js

@@ -1,18 +0,0 @@
-module.exports = {
-   clearMocks: true,
-   moduleFileExtensions: ['js', 'ts'],
-   testEnvironment: 'node',
-   testMatch: ['**/*.test.ts'],
-   transform: {
-      '^.+\\.ts$': 'ts-jest'
-   },
-   verbose: true,
-   coverageThreshold: {
-      global: {
-         branches: 0,
-         functions: 14,
-         lines: 27,
-         statements: 27
-      }
-   }
-}

package.json

@@ -1,12 +1,14 @@
 {
    "name": "setup-kubectl-action",
-   "version": "5.0.0",
+   "version": "5.1.0",
    "private": true,
    "main": "lib/index.js",
+   "type": "module",
    "scripts": {
-      "build": "npm i ncc && npx ncc build src/index.ts -o lib",
-      "test": "jest",
-      "test-coverage": "jest --coverage",
+      "build": "tsc --noEmit && esbuild src/index.ts --bundle --platform=node --target=node20 --format=esm --outfile=lib/index.js --banner:js=\"import { createRequire } from 'module';const require = createRequire(import.meta.url);\"",
+      "typecheck": "tsc --noEmit",
+      "test": "vitest run",
+      "test-coverage": "vitest run --coverage",
       "format": "prettier --write .",
       "format-check": "prettier --check .",
       "prepare": "husky"
@@ -19,19 +21,16 @@
    "author": "GitHub",
    "license": "MIT",
    "dependencies": {
-      "@actions/core": "^2.0.2",
-      "@actions/exec": "^2.0.0",
-      "@actions/tool-cache": "^3.0.0",
-      "ncc": "^0.3.6"
+      "@actions/core": "^3.0.0",
+      "@actions/exec": "^3.0.0",
+      "@actions/tool-cache": "^4.0.0"
    },
    "devDependencies": {
-      "@types/jest": "^30.0.0",
       "@types/node": "^25.0.9",
-      "@vercel/ncc": "^0.38.4",
+      "esbuild": "^0.27.4",
       "husky": "^9.1.7",
-      "jest": "^30.2.0",
       "prettier": "3.8.0",
-      "ts-jest": "^29.4.6",
-      "typescript": "5.9.3"
+      "typescript": "5.9.3",
+      "vitest": "^4.0.18"
    }
 }

src/helpers.ts

@@ -42,7 +42,7 @@ export async function getLatestPatchVersion(
       }
       return latestPatch
    } catch (error) {
-      core.debug(error)
+      core.debug(String(error))
       core.warning('GetLatestPatchVersionFailed')
       throw new Error(`Failed to get latest patch version for ${version}`)
    }

src/index.ts

@@ -1,4 +1,4 @@
-import {run} from './run'
+import {run} from './run.js'
 import * as core from '@actions/core'
 
 run().catch(core.setFailed)

src/run.test.ts

@@ -1,28 +1,43 @@
-import * as run from './run'
-import {
+import {vi, describe, test, expect, beforeEach} from 'vitest'
+import * as path from 'path'
+import * as util from 'util'
+
+vi.mock('os')
+vi.mock('fs')
+vi.mock('@actions/tool-cache', async (importOriginal) => {
+   const actual = await importOriginal<typeof import('@actions/tool-cache')>()
+   return {
+      ...actual,
+      downloadTool: vi.fn(),
+      find: vi.fn(),
+      cacheFile: vi.fn()
+   }
+})
+vi.mock('@actions/core')
+
+const os = await import('os')
+const fs = await import('fs')
+const toolCache = await import('@actions/tool-cache')
+const core = await import('@actions/core')
+const run = await import('./run.js')
+const {
    getkubectlDownloadURL,
    getKubectlArch,
    getExecutableExtension,
    getLatestPatchVersion
-} from './helpers'
-import * as os from 'os'
-import * as toolCache from '@actions/tool-cache'
-import * as fs from 'fs'
-import * as path from 'path'
-import * as core from '@actions/core'
-import * as util from 'util'
+} = await import('./helpers.js')
 
 describe('Testing all functions in run file.', () => {
    beforeEach(() => {
-      jest.clearAllMocks()
+      vi.clearAllMocks()
    })
    test('getExecutableExtension() - return .exe when os is Windows', () => {
-      jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
+      vi.mocked(os.type).mockReturnValue('Windows_NT')
       expect(getExecutableExtension()).toBe('.exe')
       expect(os.type).toHaveBeenCalled()
    })
    test('getExecutableExtension() - return empty string for non-windows OS', () => {
-      jest.spyOn(os, 'type').mockReturnValue('Darwin')
+      vi.mocked(os.type).mockReturnValue('Darwin')
       expect(getExecutableExtension()).toBe('')
       expect(os.type).toHaveBeenCalled()
    })
@@ -33,7 +48,7 @@ describe('Testing all functions in run file.', () => {
    ])(
       'getKubectlArch() - return on %s os arch %s kubectl arch',
       (osArch, kubectlArch) => {
-         jest.spyOn(os, 'arch').mockReturnValue(osArch as NodeJS.Architecture)
+         vi.mocked(os.arch).mockReturnValue(osArch as NodeJS.Architecture)
          expect(getKubectlArch()).toBe(kubectlArch)
          expect(os.arch).toHaveBeenCalled()
       }
@@ -41,7 +56,7 @@ describe('Testing all functions in run file.', () => {
    test.each([['arm'], ['arm64'], ['amd64']])(
       'getkubectlDownloadURL() - return the URL to download %s kubectl for Linux',
       (arch) => {
-         jest.spyOn(os, 'type').mockReturnValue('Linux')
+         vi.mocked(os.type).mockReturnValue('Linux')
          const kubectlLinuxUrl = util.format(
             'https://dl.k8s.io/release/v1.15.0/bin/linux/%s/kubectl',
             arch
@@ -53,7 +68,7 @@ describe('Testing all functions in run file.', () => {
    test.each([['arm'], ['arm64'], ['amd64']])(
       'getkubectlDownloadURL() - return the URL to download %s kubectl for Darwin',
       (arch) => {
-         jest.spyOn(os, 'type').mockReturnValue('Darwin')
+         vi.mocked(os.type).mockReturnValue('Darwin')
          const kubectlDarwinUrl = util.format(
             'https://dl.k8s.io/release/v1.15.0/bin/darwin/%s/kubectl',
             arch
@@ -65,7 +80,7 @@ describe('Testing all functions in run file.', () => {
    test.each([['arm'], ['arm64'], ['amd64']])(
       'getkubectlDownloadURL() - return the URL to download %s kubectl for Windows',
       (arch) => {
-         jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
+         vi.mocked(os.type).mockReturnValue('Windows_NT')
          const kubectlWindowsUrl = util.format(
             'https://dl.k8s.io/release/v1.15.0/bin/windows/%s/kubectl.exe',
             arch
@@ -75,40 +90,30 @@ describe('Testing all functions in run file.', () => {
       }
    )
    test('getStableKubectlVersion() - download stable version file, read version and return it', async () => {
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockReturnValue(Promise.resolve('pathToTool'))
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('v1.20.4')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('v1.20.4')
       expect(await run.getStableKubectlVersion()).toBe('v1.20.4')
       expect(toolCache.downloadTool).toHaveBeenCalled()
       expect(fs.readFileSync).toHaveBeenCalledWith('pathToTool', 'utf8')
    })
    test('getStableKubectlVersion() - return default v1.15.0 if version read is empty', async () => {
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockReturnValue(Promise.resolve('pathToTool'))
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('')
       expect(await run.getStableKubectlVersion()).toBe('v1.15.0')
       expect(toolCache.downloadTool).toHaveBeenCalled()
       expect(fs.readFileSync).toHaveBeenCalledWith('pathToTool', 'utf8')
    })
    test('getStableKubectlVersion() - return default v1.15.0 if unable to download file', async () => {
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockRejectedValue('Unable to download.')
+      vi.mocked(toolCache.downloadTool).mockRejectedValue('Unable to download.')
       expect(await run.getStableKubectlVersion()).toBe('v1.15.0')
       expect(toolCache.downloadTool).toHaveBeenCalled()
    })
    test('downloadKubectl() - download kubectl, add it to toolCache and return path to it', async () => {
-      jest.spyOn(toolCache, 'find').mockReturnValue('')
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockReturnValue(Promise.resolve('pathToTool'))
-      jest
-         .spyOn(toolCache, 'cacheFile')
-         .mockReturnValue(Promise.resolve('pathToCachedTool'))
-      jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
-      jest.spyOn(fs, 'chmodSync').mockImplementation(() => {})
+      vi.mocked(toolCache.find).mockReturnValue('')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(toolCache.cacheFile).mockResolvedValue('pathToCachedTool')
+      vi.mocked(os.type).mockReturnValue('Windows_NT')
+      vi.mocked(fs.chmodSync).mockImplementation(() => {})
       expect(await run.downloadKubectl('v1.15.0')).toBe(
          path.join('pathToCachedTool', 'kubectl.exe')
       )
@@ -122,10 +127,10 @@ describe('Testing all functions in run file.', () => {
       )
    })
    test('downloadKubectl() - throw DownloadKubectlFailed error when unable to download kubectl', async () => {
-      jest.spyOn(toolCache, 'find').mockReturnValue('')
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockRejectedValue('Unable to download kubectl.')
+      vi.mocked(toolCache.find).mockReturnValue('')
+      vi.mocked(toolCache.downloadTool).mockRejectedValue(
+         'Unable to download kubectl.'
+      )
       await expect(run.downloadKubectl('v1.15.0')).rejects.toThrow(
          'DownloadKubectlFailed'
       )
@@ -135,9 +140,9 @@ describe('Testing all functions in run file.', () => {
    test('downloadKubectl() - throw kubectl not found error when receive 404 response', async () => {
       const kubectlVersion = 'v1.15.0'
       const arch = 'arm128'
-      jest.spyOn(os, 'arch').mockReturnValue(arch as any)
-      jest.spyOn(toolCache, 'find').mockReturnValue('')
-      jest.spyOn(toolCache, 'downloadTool').mockImplementation((_) => {
+      vi.mocked(os.arch).mockReturnValue(arch as NodeJS.Architecture)
+      vi.mocked(toolCache.find).mockReturnValue('')
+      vi.mocked(toolCache.downloadTool).mockImplementation((_) => {
          throw new toolCache.HTTPError(404)
       })
       await expect(run.downloadKubectl(kubectlVersion)).rejects.toThrow(
@@ -152,11 +157,10 @@ describe('Testing all functions in run file.', () => {
       expect(toolCache.downloadTool).toHaveBeenCalled()
    })
    test('downloadKubectl() - return path to existing cache of kubectl', async () => {
-      jest.spyOn(core, 'getInput').mockImplementation(() => 'v1.15.5')
-      jest.spyOn(toolCache, 'find').mockReturnValue('pathToCachedTool')
-      jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
-      jest.spyOn(fs, 'chmodSync').mockImplementation(() => {})
-      jest.spyOn(toolCache, 'downloadTool')
+      vi.mocked(core.getInput).mockImplementation(() => 'v1.15.5')
+      vi.mocked(toolCache.find).mockReturnValue('pathToCachedTool')
+      vi.mocked(os.type).mockReturnValue('Windows_NT')
+      vi.mocked(fs.chmodSync).mockImplementation(() => {})
       expect(await run.downloadKubectl('v1.15.0')).toBe(
          path.join('pathToCachedTool', 'kubectl.exe')
       )
@@ -169,8 +173,8 @@ describe('Testing all functions in run file.', () => {
       expect(toolCache.downloadTool).not.toHaveBeenCalled()
    })
    test('getLatestPatchVersion() - download and return latest patch version', async () => {
-      jest.spyOn(toolCache, 'downloadTool').mockResolvedValue('pathToTool')
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('v1.27.15')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('v1.27.15')
 
       const result = await getLatestPatchVersion('1', '27')
 
@@ -181,8 +185,8 @@ describe('Testing all functions in run file.', () => {
    })
 
    test('getLatestPatchVersion() - throw error when patch version is empty', async () => {
-      jest.spyOn(toolCache, 'downloadTool').mockResolvedValue('pathToTool')
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('')
 
       await expect(getLatestPatchVersion('1', '27')).rejects.toThrow(
          'Failed to get latest patch version for 1.27'
@@ -190,17 +194,17 @@ describe('Testing all functions in run file.', () => {
    })
 
    test('getLatestPatchVersion() - throw error when download fails', async () => {
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockRejectedValue(new Error('Network error'))
+      vi.mocked(toolCache.downloadTool).mockRejectedValue(
+         new Error('Network error')
+      )
 
       await expect(getLatestPatchVersion('1', '27')).rejects.toThrow(
          'Failed to get latest patch version for 1.27'
       )
    })
    test('resolveKubectlVersion() - expands major.minor to latest patch', async () => {
-      jest.spyOn(toolCache, 'downloadTool').mockResolvedValue('pathToTool')
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('v1.27.15')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('v1.27.15')
 
       const result = await run.resolveKubectlVersion('1.27')
       expect(result).toBe('v1.27.15')
@@ -215,20 +219,20 @@ describe('Testing all functions in run file.', () => {
       expect(result).toBe('v1.27.15')
    })
    test('resolveKubectlVersion() - expands v-prefixed major.minor to latest patch', async () => {
-      jest.spyOn(toolCache, 'downloadTool').mockResolvedValue('pathToTool')
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('v1.27.15')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('v1.27.15')
 
       const result = await run.resolveKubectlVersion('v1.27')
       expect(result).toBe('v1.27.15')
    })
    test('run() - download specified version and set output', async () => {
-      jest.spyOn(core, 'getInput').mockReturnValue('v1.15.5')
-      jest.spyOn(toolCache, 'find').mockReturnValue('pathToCachedTool')
-      jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
-      jest.spyOn(fs, 'chmodSync').mockImplementation()
-      jest.spyOn(core, 'addPath').mockImplementation()
-      jest.spyOn(console, 'log').mockImplementation()
-      jest.spyOn(core, 'setOutput').mockImplementation()
+      vi.mocked(core.getInput).mockReturnValue('v1.15.5')
+      vi.mocked(toolCache.find).mockReturnValue('pathToCachedTool')
+      vi.mocked(os.type).mockReturnValue('Windows_NT')
+      vi.mocked(fs.chmodSync).mockImplementation()
+      vi.mocked(core.addPath).mockImplementation()
+      vi.spyOn(console, 'log').mockImplementation()
+      vi.mocked(core.setOutput).mockImplementation()
       expect(await run.run()).toBeUndefined()
       expect(core.getInput).toHaveBeenCalledWith('version', {required: true})
       expect(core.addPath).toHaveBeenCalledWith('pathToCachedTool')
@@ -238,17 +242,15 @@ describe('Testing all functions in run file.', () => {
       )
    })
    test('run() - get latest version, download it and set output', async () => {
-      jest.spyOn(core, 'getInput').mockReturnValue('latest')
-      jest
-         .spyOn(toolCache, 'downloadTool')
-         .mockReturnValue(Promise.resolve('pathToTool'))
-      jest.spyOn(fs, 'readFileSync').mockReturnValue('v1.20.4')
-      jest.spyOn(toolCache, 'find').mockReturnValue('pathToCachedTool')
-      jest.spyOn(os, 'type').mockReturnValue('Windows_NT')
-      jest.spyOn(fs, 'chmodSync').mockImplementation()
-      jest.spyOn(core, 'addPath').mockImplementation()
-      jest.spyOn(console, 'log').mockImplementation()
-      jest.spyOn(core, 'setOutput').mockImplementation()
+      vi.mocked(core.getInput).mockReturnValue('latest')
+      vi.mocked(toolCache.downloadTool).mockResolvedValue('pathToTool')
+      vi.mocked(fs.readFileSync).mockReturnValue('v1.20.4')
+      vi.mocked(toolCache.find).mockReturnValue('pathToCachedTool')
+      vi.mocked(os.type).mockReturnValue('Windows_NT')
+      vi.mocked(fs.chmodSync).mockImplementation()
+      vi.mocked(core.addPath).mockImplementation()
+      vi.spyOn(console, 'log').mockImplementation()
+      vi.mocked(core.setOutput).mockImplementation()
       expect(await run.run()).toBeUndefined()
       expect(toolCache.downloadTool).toHaveBeenCalledWith(
          'https://dl.k8s.io/release/stable.txt'

src/run.ts

@@ -8,7 +8,7 @@ import {
    getKubectlArch,
    getExecutableExtension,
    getLatestPatchVersion
-} from './helpers'
+} from './helpers.js'
 
 const kubectlToolName = 'kubectl'
 const stableKubectlVersion = 'v1.15.0'

tsconfig.json

@@ -1,8 +1,12 @@
 {
    "compilerOptions": {
       "target": "ES2020",
-      "module": "commonjs",
-      "lib": ["ES2020", "DOM"]
+      "module": "NodeNext",
+      "moduleResolution": "NodeNext",
+      "rootDir": "./src",
+      "outDir": "./lib",
+      "skipLibCheck": true,
+      "noEmit": true
    },
-   "exclude": ["node_modules", "test"]
+   "exclude": ["node_modules", "**/*.test.ts", "vitest.config.ts"]
 }

vitest.config.ts

@@ -0,0 +1,10 @@
+import {defineConfig} from 'vitest/config'
+
+export default defineConfig({
+   test: {
+      globals: true,
+      environment: 'node',
+      include: ['**/*.test.ts'],
+      clearMocks: true
+   }
+})