Fix Javy and function-runner download on Windows

.changeset/smart-mugs-love.md

@@ -0,0 +1,5 @@
+---
+'@shopify/app': patch
+---
+
+Fix Javy and function-runner downloads on Windows

packages/app/src/cli/services/function/binaries.test.ts

@@ -1,4 +1,5 @@
-import {javyBinary, functionRunnerBinary} from './binaries.js'
+import {javyBinary, functionRunnerBinary, installBinary} from './binaries.js'
+import {exec} from '@shopify/cli-kit/node/system'
 import {describe, expect, test} from 'vitest'
 
 const javy = javyBinary()
@@ -8,7 +9,11 @@ describe('javy', () => {
   test('properties are set correctly', () => {
     expect(javy.name).toBe('javy')
     expect(javy.version).match(/^v\d\.\d\.\d$/)
-    expect(javy.path).toMatch(/(\/|\\)javy$/)
+    if (process.platform === 'win32') {
+      expect(javy.path).toMatch(/(\/|\\)javy.exe$/)
+    } else {
+      expect(javy.path).toMatch(/(\/|\\)javy$/)
+    }
   })
 
   describe('downloadUrl returns the correct URL', () => {
@@ -101,13 +106,22 @@ describe('javy', () => {
       )
     })
   })
+
+  test('Javy installs and runs', async () => {
+    await installBinary(javy)
+    await exec(javy.path, ['--help'])
+  })
 })
 
 describe('functionRunner', () => {
   test('properties are set correctly', () => {
     expect(functionRunner.name).toBe('function-runner')
     expect(functionRunner.version).match(/^v\d\.\d\.\d$/)
-    expect(functionRunner.path).toMatch(/(\/|\\)function-runner$/)
+    if (process.platform === 'win32') {
+      expect(functionRunner.path).toMatch(/(\/|\\)function-runner.exe$/)
+    } else {
+      expect(functionRunner.path).toMatch(/(\/|\\)function-runner$/)
+    }
   })
 
   describe('downloadUrl returns the correct URL', () => {
@@ -200,4 +214,9 @@ describe('functionRunner', () => {
       )
     })
   })
+
+  test('function-runner installs and runs', async () => {
+    await installBinary(functionRunner)
+    await exec(functionRunner.path, ['--help'])
+  })
 })

packages/app/src/cli/services/function/binaries.ts

@@ -22,7 +22,8 @@ class DownloadableBinary {
   constructor(name: string, version: string, gitHubRepo: string) {
     this.name = name
     this.version = version
-    this.path = joinPath(dirname(fileURLToPath(import.meta.url)), '..', 'bin', `${this.name}`)
+    const filename = process.platform === 'win32' ? `${name}.exe` : name
+    this.path = joinPath(dirname(fileURLToPath(import.meta.url)), '..', 'bin', filename)
     this.gitHubRepo = gitHubRepo
   }