Migrate to ESM, esbuild, vitest, and update actions/* to latest

- Update @actions/core to 3.x, @actions/exec to 3.x, @actions/io to 3.x
- Replace @vercel/ncc + babel with esbuild (build.mjs) targeting Node 20 ESM
- Replace jest/babel-jest/ts-jest with vitest and @vitest/coverage-v8
- Update tsconfig to NodeNext module resolution with strict mode
- Add explicit .js extensions to all relative imports (NodeNext requirement)
- Fix implicit any index signatures in parseCluster and parseMethod
- Migrate all test files from jest to vi.mock/vi.mocked APIs
- Fix ESM module spying limitations using vi.mock() at module level
- Fix env var test pollution in default.test.ts with afterEach cleanup

src/action.test.ts

@@ -1,7 +1,9 @@
-import {getRequiredInputError} from '../tests/util'
-import {run} from './action'
-import fs from 'fs'
-import * as utils from './utils'
+import {vi, describe, it, expect} from 'vitest'
+import {getRequiredInputError} from '../tests/util.js'
+import {run} from './action.js'
+
+vi.mock('fs')
+vi.mock('./utils.js')
 
 describe('Run', () => {
    it('throws error without cluster type', async () => {
@@ -9,22 +11,23 @@ describe('Run', () => {
    })
 
    it('writes kubeconfig and sets context', async () => {
+      const {getKubeconfig, setContext} = await import('./utils.js')
+      const fs = await import('fs')
       const kubeconfig = 'kubeconfig'
 
       process.env['INPUT_CLUSTER-TYPE'] = 'default'
       process.env['RUNNER_TEMP'] = '/sample/path'
 
-      jest
-         .spyOn(utils, 'getKubeconfig')
-         .mockImplementation(async () => kubeconfig)
-      jest.spyOn(fs, 'writeFileSync').mockImplementation(() => {})
-      jest.spyOn(fs, 'chmodSync').mockImplementation(() => {})
-      jest.spyOn(utils, 'setContext').mockImplementation(() => kubeconfig)
+      vi.mocked(getKubeconfig).mockResolvedValue(kubeconfig)
+      vi.mocked(setContext).mockReturnValue(kubeconfig)
+      vi.mocked(fs.writeFileSync).mockImplementation(() => {})
+      vi.mocked(fs.chmodSync).mockImplementation(() => {})
 
-      expect(await run())
-      expect(utils.getKubeconfig).toHaveBeenCalled()
+      await run()
+
+      expect(getKubeconfig).toHaveBeenCalled()
       expect(fs.writeFileSync).toHaveBeenCalled()
       expect(fs.chmodSync).toHaveBeenCalled()
-      expect(utils.setContext).toHaveBeenCalled()
+      expect(setContext).toHaveBeenCalled()
    })
 })

src/action.ts

@@ -1,8 +1,8 @@
 import * as core from '@actions/core'
 import * as path from 'path'
 import * as fs from 'fs'
-import {Cluster, parseCluster} from './types/cluster'
-import {setContext, getKubeconfig} from './utils'
+import {Cluster, parseCluster} from './types/cluster.js'
+import {setContext, getKubeconfig} from './utils.js'
 
 /**
  * Sets the Kubernetes context based on supplied action inputs
@@ -14,7 +14,7 @@ export async function run() {
          required: true
       })
    )
-   const runnerTempDirectory: string = process.env['RUNNER_TEMP']
+   const runnerTempDirectory: string = process.env['RUNNER_TEMP'] ?? ''
    const kubeconfigPath: string = path.join(
       runnerTempDirectory,
       `kubeconfig_${Date.now()}`

src/kubeconfigs/arc.test.ts

@@ -1,8 +1,11 @@
-import * as actions from '@actions/exec'
+import {vi, describe, test, it, expect, beforeEach} from 'vitest'
 import * as io from '@actions/io'
-import {getRequiredInputError} from '../../tests/util'
-import {getArcKubeconfig, KUBECONFIG_LOCATION} from './arc'
-import * as az from './azCommands'
+import {getRequiredInputError} from '../../tests/util.js'
+import {getArcKubeconfig, KUBECONFIG_LOCATION} from './arc.js'
+import * as az from './azCommands.js'
+
+vi.mock('@actions/io')
+vi.mock('./azCommands.js')
 
 describe('Arc kubeconfig', () => {
    test('it throws error without resource group', async () => {
@@ -28,11 +31,11 @@ describe('Arc kubeconfig', () => {
          process.env['INPUT_RESOURCE-GROUP'] = group
          process.env['INPUT_CLUSTER-NAME'] = name
 
-         jest.spyOn(io, 'which').mockImplementation(async () => path)
-         jest.spyOn(az, 'runAzCliCommand').mockImplementation(async () => {})
-         jest
-            .spyOn(az, 'runAzKubeconfigCommandBlocking')
-            .mockImplementation(async () => kubeconfig)
+         vi.mocked(io.which).mockResolvedValue(path)
+         vi.mocked(az.runAzCliCommand).mockResolvedValue(undefined)
+         vi.mocked(az.runAzKubeconfigCommandBlocking).mockResolvedValue(
+            kubeconfig
+         )
       })
 
       it('throws an error without method', async () => {

src/kubeconfigs/arc.ts

@@ -1,8 +1,8 @@
 import * as core from '@actions/core'
 import * as io from '@actions/io'
-import {Method, parseMethod} from '../types/method'
+import {Method, parseMethod} from '../types/method.js'
 import * as path from 'path'
-import {runAzCliCommand, runAzKubeconfigCommandBlocking} from './azCommands'
+import {runAzCliCommand, runAzKubeconfigCommandBlocking} from './azCommands.js'
 
 const RUNNER_TEMP: string = process.env['RUNNER_TEMP'] || ''
 export const KUBECONFIG_LOCATION: string = path.join(

src/kubeconfigs/azCommands.test.ts

@@ -1,14 +1,17 @@
+import {vi, describe, test, expect, beforeEach} from 'vitest'
 import * as actions from '@actions/exec'
-import {runAzCliCommand} from './azCommands'
+import {runAzCliCommand} from './azCommands.js'
+
+vi.mock('@actions/exec')
 
 describe('Az commands', () => {
    test('it runs an az cli command', async () => {
       const path = 'path'
       const args = ['args']
 
-      jest.spyOn(actions, 'exec').mockImplementation(async () => 0)
+      vi.mocked(actions.exec).mockResolvedValue(0)
 
-      expect(await runAzCliCommand(path, args))
+      await runAzCliCommand(path, args)
       expect(actions.exec).toHaveBeenCalledWith(path, args, {})
    })
 })

src/kubeconfigs/azCommands.ts

@@ -1,6 +1,5 @@
 import * as fs from 'fs'
-import {ExecOptions} from '@actions/exec/lib/interfaces'
-import {exec} from '@actions/exec'
+import {ExecOptions, exec} from '@actions/exec'
 import {spawn} from 'child_process'
 
 const AZ_TIMEOUT_SECONDS: number = 120

src/kubeconfigs/default.test.ts

@@ -1,7 +1,7 @@
+import {vi, describe, test, expect, beforeEach, afterEach} from 'vitest'
 import * as fs from 'fs'
-import * as core from '@actions/core'
-import {getRequiredInputError} from '../../tests/util'
-import {createKubeconfig, getDefaultKubeconfig} from './default'
+import {getRequiredInputError} from '../../tests/util.js'
+import {createKubeconfig, getDefaultKubeconfig} from './default.js'
 
 describe('Default kubeconfig', () => {
    test('it creates a kubeconfig with proper format', () => {
@@ -51,6 +51,11 @@ describe('Default kubeconfig', () => {
          process.env['INPUT_METHOD'] = 'default'
       })
 
+      afterEach(() => {
+         delete process.env['INPUT_KUBECONFIG']
+         delete process.env['INPUT_KUBECONFIG-ENCODING']
+      })
+
       test('it throws error without kubeconfig', () => {
          expect(() => getDefaultKubeconfig()).toThrow(
             getRequiredInputError('kubeconfig')
@@ -66,39 +71,25 @@ describe('Default kubeconfig', () => {
 
       test('returns kubeconfig as plaintext when encoding is plaintext', () => {
          const kc = 'example kc'
-         jest.spyOn(core, 'getInput').mockImplementation((name: string) => {
-            if (name === 'method') return 'default'
-            if (name === 'kubeconfig-encoding') return 'plaintext'
-            if (name === 'kubeconfig') return kc
-            return ''
-         })
+         process.env['INPUT_KUBECONFIG'] = kc
+         process.env['INPUT_KUBECONFIG-ENCODING'] = 'plaintext'
+
          expect(getDefaultKubeconfig()).toBe(kc)
       })
 
       test('it gets default config through base64 kubeconfig input', () => {
          const kc = 'example kc'
          const base64Kc = Buffer.from(kc, 'utf-8').toString('base64')
-
-         jest.spyOn(core, 'getInput').mockImplementation((name: string) => {
-            if (name === 'method') return 'default'
-            if (name === 'kubeconfig-encoding') return 'base64'
-            if (name === 'kubeconfig') return base64Kc
-            return ''
-         })
+         process.env['INPUT_KUBECONFIG'] = base64Kc
+         process.env['INPUT_KUBECONFIG-ENCODING'] = 'base64'
 
          expect(getDefaultKubeconfig()).toBe(kc)
       })
 
       test('it throws error for unknown kubeconfig-encoding', () => {
          const kc = 'example kc'
-         const unknownEncoding = 'foobar'
-
-         jest.spyOn(core, 'getInput').mockImplementation((name: string) => {
-            if (name === 'method') return 'default'
-            if (name === 'kubeconfig-encoding') return unknownEncoding
-            if (name === 'kubeconfig') return kc
-            return ''
-         })
+         process.env['INPUT_KUBECONFIG'] = kc
+         process.env['INPUT_KUBECONFIG-ENCODING'] = 'foobar'
 
          expect(() => getDefaultKubeconfig()).toThrow(
             "Invalid kubeconfig-encoding: 'foobar'. Must be 'plaintext' or 'base64'."

src/kubeconfigs/default.ts

@@ -1,8 +1,8 @@
 import * as core from '@actions/core'
 import * as jsyaml from 'js-yaml'
 import {KubeConfig} from '@kubernetes/client-node'
-import {K8sSecret, parseK8sSecret} from '../types/k8sSecret'
-import {Method, parseMethod} from '../types/method'
+import {K8sSecret, parseK8sSecret} from '../types/k8sSecret.js'
+import {Method, parseMethod} from '../types/method.js'
 
 /**
  * Gets the kubeconfig based on provided method for a default Kubernetes cluster

src/run.ts

@@ -1,4 +1,4 @@
-import {run} from './action'
+import {run} from './action.js'
 import * as core from '@actions/core'
 
 // Run the application

src/types/cluster.test.ts

@@ -1,8 +1,9 @@
-import {Cluster, parseCluster} from './cluster'
+import {describe, test, expect} from 'vitest'
+import {Cluster, parseCluster} from './cluster.js'
 
 describe('Cluster type', () => {
    test('it has required values', () => {
-      const vals = <any>Object.values(Cluster)
+      const vals = Object.values(Cluster) as string[]
       expect(vals.includes('arc')).toBe(true)
       expect(vals.includes('generic')).toBe(true)
    })

src/types/cluster.ts

@@ -8,9 +8,10 @@ export enum Cluster {
  * @param str The cluster type (case insensitive)
  * @returns The Cluster enum or undefined if it can't be parsed
  */
-export const parseCluster = (str: string): Cluster | undefined =>
-   Cluster[
-      Object.keys(Cluster).filter(
-         (k) => Cluster[k].toString().toLowerCase() === str.toLowerCase()
-      )[0] as keyof typeof Cluster
-   ]
+export const parseCluster = (str: string): Cluster | undefined => {
+   const key = Object.keys(Cluster).find(
+      (k) =>
+         Cluster[k as keyof typeof Cluster].toLowerCase() === str.toLowerCase()
+   ) as keyof typeof Cluster | undefined
+   return key !== undefined ? Cluster[key] : undefined
+}

src/types/k8sSecret.test.ts

@@ -1,4 +1,5 @@
-import {parseK8sSecret, K8sSecret} from './k8sSecret'
+import {describe, test, expect} from 'vitest'
+import {parseK8sSecret} from './k8sSecret.js'
 
 describe('K8sSecret type', () => {
    describe('Parsing from any', () => {

src/types/method.test.ts

@@ -1,8 +1,9 @@
-import {Method, parseMethod} from './method'
+import {describe, test, expect} from 'vitest'
+import {Method, parseMethod} from './method.js'
 
 describe('Method type', () => {
    test('it has required values', () => {
-      const vals = <any>Object.values(Method)
+      const vals = Object.values(Method) as string[]
       expect(vals.includes('kubeconfig')).toBe(true)
       expect(vals.includes('service-account')).toBe(true)
       expect(vals.includes('service-principal')).toBe(true)

src/types/method.ts

@@ -9,9 +9,10 @@ export enum Method {
  * @param str The method (case insensitive)
  * @returns The Method enum or undefined if it can't be parsed
  */
-export const parseMethod = (str: string): Method | undefined =>
-   Method[
-      Object.keys(Method).filter(
-         (k) => Method[k].toString().toLowerCase() === str.toLowerCase()
-      )[0] as keyof typeof Method
-   ]
+export const parseMethod = (str: string): Method | undefined => {
+   const key = Object.keys(Method).find(
+      (k) =>
+         Method[k as keyof typeof Method].toLowerCase() === str.toLowerCase()
+   ) as keyof typeof Method | undefined
+   return key !== undefined ? Method[key] : undefined
+}

src/utils.test.ts

@@ -1,25 +1,26 @@
+import {vi, describe, test, expect} from 'vitest'
 import fs from 'fs'
-import * as arc from './kubeconfigs/arc'
-import * as def from './kubeconfigs/default'
-import {Cluster} from './types/cluster'
-import {getKubeconfig, setContext} from './utils'
+import * as arc from './kubeconfigs/arc.js'
+import * as def from './kubeconfigs/default.js'
+import {Cluster} from './types/cluster.js'
+import {getKubeconfig, setContext} from './utils.js'
 
 describe('Utils', () => {
    describe('get kubeconfig', () => {
       test('it gets arc kubeconfig when type is arc', async () => {
          const arcKubeconfig = 'arckubeconfig'
-         jest
-            .spyOn(arc, 'getArcKubeconfig')
-            .mockImplementation(async () => arcKubeconfig)
+         vi.spyOn(arc, 'getArcKubeconfig').mockImplementation(
+            async () => arcKubeconfig
+         )
 
          expect(await getKubeconfig(Cluster.ARC)).toBe(arcKubeconfig)
       })
 
       test('it defaults to default kubeconfig', async () => {
          const defaultKubeconfig = 'arckubeconfig'
-         jest
-            .spyOn(def, 'getDefaultKubeconfig')
-            .mockImplementation(() => defaultKubeconfig)
+         vi.spyOn(def, 'getDefaultKubeconfig').mockImplementation(
+            () => defaultKubeconfig
+         )
 
          expect(await getKubeconfig(undefined)).toBe(defaultKubeconfig)
          expect(await getKubeconfig(Cluster.GENERIC)).toBe(defaultKubeconfig)

src/utils.ts

@@ -1,9 +1,9 @@
 import * as core from '@actions/core'
 import * as fs from 'fs'
 import {KubeConfig} from '@kubernetes/client-node'
-import {getDefaultKubeconfig} from './kubeconfigs/default'
-import {getArcKubeconfig} from './kubeconfigs/arc'
-import {Cluster} from './types/cluster'
+import {getDefaultKubeconfig} from './kubeconfigs/default.js'
+import {getArcKubeconfig} from './kubeconfigs/arc.js'
+import {Cluster} from './types/cluster.js'
 
 /**
  * Gets the kubeconfig based on Kubernetes cluster type