chore: refactors and cleanups

Signed-off-by: Sam Gammon <sam@elide.ventures>

Author: sgammon

__tests__/command.test.ts

@@ -22,8 +22,9 @@ mock.module('@actions/core', () => ({
   addPath: jest.fn()
 }))
 
-const { prewarm, info, obtainVersion, ElideCommand, ElideArgument } =
-  await import('../src/command')
+const { elideInfo, obtainVersion, ElideCommand, ElideArgument } = await import(
+  '../src/command'
+)
 
 describe('command', () => {
   beforeEach(() => {
@@ -39,20 +40,10 @@ describe('command', () => {
     })
   })
 
-  it('prewarm should execute the info command', async () => {
-    await prewarm('/usr/bin/elide')
+  it('elideInfo should execute the info command', async () => {
+    await elideInfo('/usr/bin/elide')
     expect(infoMock).toHaveBeenCalledWith(
-      'Prewarming Elide at bin: /usr/bin/elide'
-    )
-    expect(execMock).toHaveBeenCalledWith('"/usr/bin/elide"', [
-      ElideCommand.INFO
-    ])
-  })
-
-  it('info should execute the info command', async () => {
-    await info('/usr/bin/elide')
-    expect(debugMock).toHaveBeenCalledWith(
-      'Printing runtime info at bin: /usr/bin/elide'
+      'Running Elide info at bin: /usr/bin/elide'
     )
     expect(execMock).toHaveBeenCalledWith('"/usr/bin/elide"', [
       ElideCommand.INFO
@@ -82,8 +73,8 @@ describe('command', () => {
     expect(version).toBe('1.0.0')
   })
 
-  it('prewarm should propagate exec errors', async () => {
+  it('elideInfo should propagate exec errors', async () => {
     execMock.mockRejectedValue(new Error('exec failed'))
-    await expect(prewarm('/bad/path')).rejects.toThrow('exec failed')
+    await expect(elideInfo('/bad/path')).rejects.toThrow('exec failed')
   })
 })

__tests__/install-apt.test.ts

@@ -31,8 +31,7 @@ mock.module('@actions/core', () => ({
 }))
 mock.module('../src/command', () => ({
   obtainVersion: obtainVersionMock,
-  prewarm: jest.fn(),
-  info: jest.fn()
+  elideInfo: jest.fn()
 }))
 
 const { installViaApt } = await import('../src/install-apt')

__tests__/install-script.test.ts

@@ -40,8 +40,7 @@ mock.module('@actions/tool-cache', () => ({
 }))
 mock.module('../src/command', () => ({
   obtainVersion: obtainVersionMock,
-  prewarm: jest.fn(),
-  info: jest.fn()
+  elideInfo: jest.fn()
 }))
 
 const { installViaScript } = await import('../src/install-script')

__tests__/main.test.ts

@@ -16,8 +16,7 @@ const errorMock = jest.fn()
 const addPathMock = jest.fn()
 const isDebianLikeMock = jest.fn().mockResolvedValue(false)
 const downloadToolMock = jest.fn().mockResolvedValue('/tmp/install.sh')
-const prewarmMock = jest.fn().mockResolvedValue(undefined)
-const cmdInfoMock = jest.fn().mockResolvedValue(undefined)
+const elideInfoMock = jest.fn().mockResolvedValue(undefined)
 const obtainVersionMock = jest.fn().mockResolvedValue('1.0.0')
 
 // Mock modules before any project imports
@@ -38,6 +37,7 @@ mock.module('@actions/core', () => ({
   error: errorMock,
   warning: warningMock,
   getInput: getInputMock,
+  getBooleanInput: jest.fn().mockReturnValue(true),
   setFailed: setFailedMock,
   setOutput: setOutputMock,
   addPath: addPathMock
@@ -50,8 +50,7 @@ mock.module('@actions/tool-cache', () => ({
   find: jest.fn()
 }))
 mock.module('../src/command', () => ({
-  prewarm: prewarmMock,
-  info: cmdInfoMock,
+  elideInfo: elideInfoMock,
   obtainVersion: obtainVersionMock,
   ElideCommand: { RUN: 'run', INFO: 'info' },
   ElideArgument: { VERSION: '--version' }
@@ -63,7 +62,7 @@ mock.module('../src/platform', () => ({
 const main = await import('../src/main')
 const { default: buildOptions, OptionName } = await import('../src/options')
 const { ElideArch, ElideOS } = await import('../src/releases')
-const { ActionOutputName } = await import('../src/outputs')
+const { ActionOutputName } = await import('../src/main')
 
 const setupMocks = () => {
   debugMock.mockImplementation((...args: unknown[]) =>
@@ -97,8 +96,7 @@ describe('action', () => {
     addPathMock.mockClear()
     isDebianLikeMock.mockClear()
     downloadToolMock.mockClear()
-    prewarmMock.mockClear()
-    cmdInfoMock.mockClear()
+    elideInfoMock.mockClear()
     obtainVersionMock.mockClear()
     // Default: getInput returns empty
     getInputMock.mockReturnValue('')
@@ -115,8 +113,7 @@ describe('action', () => {
       stderr: '',
       exitCode: 0
     })
-    prewarmMock.mockResolvedValue(undefined)
-    cmdInfoMock.mockResolvedValue(undefined)
+    elideInfoMock.mockResolvedValue(undefined)
     obtainVersionMock.mockResolvedValue('1.0.0')
   })
 
@@ -129,7 +126,6 @@ describe('action', () => {
     expect(getInputMock).toHaveBeenCalledWith(OptionName.ARCH)
     expect(getInputMock).toHaveBeenCalledWith(OptionName.TOKEN)
     expect(getInputMock).toHaveBeenCalledWith(OptionName.CUSTOM_URL)
-    expect(getInputMock).toHaveBeenCalledWith(OptionName.EXPORT_PATH)
   })
 
   it('sets the `path` and `version` outputs', async () => {
@@ -196,23 +192,13 @@ describe('action', () => {
     )
   })
 
-  it('should gracefully handle prewarm failure', async () => {
+  it('should gracefully handle post-install info failure', async () => {
     setupMocks()
-    prewarmMock.mockRejectedValueOnce(new Error('prewarm boom'))
+    elideInfoMock.mockRejectedValueOnce(new Error('info boom'))
     await main.run({ force: true })
     expect(setFailedMock).not.toHaveBeenCalled()
     expect(debugMock).toHaveBeenCalledWith(
-      expect.stringContaining('Prewarm failed; proceeding anyway')
-    )
-  })
-
-  it('should gracefully handle info command failure', async () => {
-    setupMocks()
-    cmdInfoMock.mockRejectedValue(new Error('info boom'))
-    await main.run({ force: true })
-    expect(setFailedMock).not.toHaveBeenCalled()
-    expect(debugMock).toHaveBeenCalledWith(
-      expect.stringContaining('Info command failed; proceeding anyway')
+      expect.stringContaining('Post-install info failed; proceeding anyway')
     )
   })
 

__tests__/releases-download.test.ts

@@ -53,8 +53,7 @@ mock.module('octokit', () => ({
 }))
 mock.module('../src/command', () => ({
   obtainVersion: obtainVersionMock,
-  prewarm: jest.fn(),
-  info: jest.fn()
+  elideInfo: jest.fn()
 }))
 
 const { downloadRelease, resolveLatestVersion } = await import(
@@ -415,15 +414,4 @@ describe('downloadRelease', () => {
       expect(findMock).toHaveBeenCalledWith('elide', '1.0.0', 'aarch64')
     })
   })
-
-  it('should not wrap with directory root for alpha7/alpha8', async () => {
-    const options = buildOptions({
-      os: 'linux',
-      arch: 'amd64',
-      version: '1.0.0-alpha7'
-    })
-    const result = await downloadRelease(options)
-    // alpha7 has the early-return path in unpackRelease
-    expect(result).toBeDefined()
-  })
 })

__tests__/releases.test.ts

@@ -57,10 +57,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0',
       channel: 'release'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0/elide.linux-amd64.tgz'
     )
@@ -73,10 +70,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0',
       channel: 'release'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0/elide.macos-arm64.tgz'
     )
@@ -89,10 +83,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0',
       channel: 'release'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0/elide.macos-amd64.tgz'
     )
@@ -105,10 +96,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0',
       channel: 'release'
     })
-    const { url, archiveType } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url, archiveType } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0/elide.windows-amd64.zip'
     )
@@ -124,10 +112,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0-beta10',
       channel: 'release'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0-beta10',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0-beta10/elide.linux-amd64.tgz'
     )
@@ -140,10 +125,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0-alpha7',
       channel: 'release'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0-alpha7',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0-alpha7/elide.macos-arm64.tgz'
     )
@@ -157,10 +139,7 @@ describe('buildDownloadUrl', () => {
       arch: 'amd64',
       version: '1.0.0'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/nightly/1.0.0/elide.linux-amd64.tgz'
     )
@@ -172,10 +151,7 @@ describe('buildDownloadUrl', () => {
       arch: 'amd64',
       version: 'latest'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: 'ignored',
-      userProvided: false
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/nightly/latest/elide.linux-amd64.tgz'
     )
@@ -190,10 +166,7 @@ describe('buildDownloadUrl', () => {
       version: 'latest',
       channel: 'preview'
     })
-    const { url } = await buildDownloadUrl(options, {
-      tag_name: 'ignored',
-      userProvided: false
-    })
+    const { url } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/preview/latest/elide.macos-arm64.tgz'
     )
@@ -207,10 +180,7 @@ describe('buildDownloadUrl', () => {
       arch: 'amd64',
       version: 'latest'
     })
-    const { url, archiveType } = await buildDownloadUrl(options, {
-      tag_name: 'ignored',
-      userProvided: false
-    })
+    const { url, archiveType } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/nightly/latest/elide.windows-amd64.zip'
     )
@@ -225,10 +195,7 @@ describe('buildDownloadUrl', () => {
       version: '1.0.0',
       channel: 'release'
     })
-    const { url, archiveType } = await buildDownloadUrl(options, {
-      tag_name: '1.0.0',
-      userProvided: true
-    })
+    const { url, archiveType } = await buildDownloadUrl(options)
     expect(url.toString()).toBe(
       'https://elide.zip/artifacts/release/1.0.0/elide.linux-amd64.txz'
     )

action.yml

@@ -24,7 +24,7 @@ inputs:
     description: 'Architecture'
     required: false
 
-  target:
+  install_path:
     description: 'Custom install path'
     required: false
 
@@ -35,12 +35,21 @@ inputs:
   export_path:
     description: 'Export to Path'
     required: false
-    default: true
+    default: 'true'
+
+  no_cache:
+    description: 'Disable tool caching'
+    required: false
+    default: 'false'
 
   custom_url:
     description: 'Custom Download URL'
     required: false
 
+  version_tag:
+    description: 'Version tag for custom download URL'
+    required: false
+
   token:
     description: 'GitHub Token'
     required: false
@@ -49,6 +58,9 @@ outputs:
   path:
     description: 'Path to Elide'
 
+  version:
+    description: 'Installed Elide version'
+
 runs:
   using: node24
   main: dist/index.js