fix: handle chunked command output

__tests__/utils.test.ts

@@ -1,9 +1,49 @@
 import {getVersionControlledFiles} from '../src/utils'
+import child_process from 'child_process'
+import {PassThrough} from 'stream'
 
-test('getVersionControlledFiles should return an array of strings', async () => {
-  let versionControlledFiles = await getVersionControlledFiles()
-  expect(Array.isArray(versionControlledFiles)).toBe(true)
-  expect(versionControlledFiles.every(entry => typeof entry === 'string')).toBe(
-    true
-  )
+// Store real spawn
+const realSpawn = child_process.spawn
+
+describe('getVersionControlledFiles()', () => {
+  beforeEach(() => {
+    // Restore original spawn before each test
+    child_process.spawn = realSpawn
+    jest.resetModules()
+  })
+
+  test('should return an array of strings', async () => {
+    let versionControlledFiles = await getVersionControlledFiles()
+    expect(Array.isArray(versionControlledFiles)).toBe(true)
+    expect(
+      versionControlledFiles.every(entry => typeof entry === 'string')
+    ).toBe(true)
+  })
+
+  test('should handle chunked output', async () => {
+    const mockStdout = new PassThrough()
+    const mockProcess = {
+      stdout: mockStdout,
+      stderr: new PassThrough(),
+      on: jest.fn()
+    }
+
+    // Replace spawn with mock
+    child_process.spawn = jest.fn().mockReturnValue(mockProcess)
+
+    // Simulate process.on('close')
+    mockProcess.on.mockImplementation((event, cb) => {
+      if (event === 'close') setImmediate(() => cb(0))
+    })
+
+    const promise = getVersionControlledFiles()
+
+    // Simulate chunked output
+    mockStdout.write('README.md\nsrc/uti')
+    mockStdout.write('ls.js\nlib/core.js\n')
+    mockStdout.end()
+
+    const result = await promise
+    expect(result).toEqual(['README.md', 'src/utils.js', 'lib/core.js'])
+  })
 })

src/utils.ts

@@ -10,14 +10,27 @@ export async function getVersionControlledFiles(): Promise<string[]> {
       '--name-only'
     ])
 
+    let stdout = ''
+    let stderr = ''
+
     listVersionControlledFilesCommand.stdout.on('data', (data: Buffer) => {
-      const stdout = data.toString()
-      resolve(stdout.split(/\r?\n/).filter(Boolean))
+      stdout += data.toString()
     })
 
     listVersionControlledFilesCommand.stderr.on('data', (data: Buffer) => {
-      if (data != null) reject(data)
-      reject(new Error('Unknown error.'))
+      stderr += data.toString()
+    })
+
+    listVersionControlledFilesCommand.on('error', (error: Error) => {
+      reject(error)
+    })
+
+    listVersionControlledFilesCommand.on('close', (code: number) => {
+      if (code !== 0) {
+        reject(new Error(`Command failed with exit code ${code}: ${stderr}`))
+      } else {
+        resolve(stdout.split(/\r?\n/).filter(Boolean))
+      }
     })
   })
 }