refactor: enhance runWithRunner to accept command type and update related calls

Author: TerrorSquad

booster/.husky/pre-push.ts

@@ -35,7 +35,7 @@ async function runTests(): Promise<boolean> {
   if (await fs.pathExists('vendor/bin/pest')) {
     log.tool('PHPUnit', 'Running tests...')
     try {
-      await runWithRunner(['composer', 'test:pest'])
+      await runWithRunner(['composer', 'test:pest'], { type: 'php' })
       log.success('Tests passed')
     } catch {
       log.error('Tests failed')

booster/.husky/shared/core.ts

@@ -65,6 +65,7 @@ export async function initEnvironment(): Promise<void> {
  */
 export interface RunOptions {
   quiet?: boolean
+  type?: 'php' | 'node' | 'system'
 }
 
 /**
@@ -76,17 +77,19 @@ export async function runWithRunner(
   command: string[],
   options: RunOptions = {},
 ): Promise<ProcessOutput> {
-  const { quiet = false } = options
+  const { quiet = false, type = 'system' } = options
+
+  const finalCommand = await getExecCommand(command, type)
 
   // Log command execution if not quiet
   if (!quiet) {
-    const commandStr = command.join(' ')
+    const commandStr = finalCommand.join(' ')
     log.info(`Executing: ${commandStr}`)
   }
 
   return await $({
     stdio: quiet ? 'pipe' : 'inherit',
-  })`${command}`
+  })`${finalCommand}`
 }
 
 /**

booster/.husky/shared/extras.ts

@@ -12,11 +12,10 @@ export async function generateDeptracImage(): Promise<void> {
 
   try {
     // Use graphviz-image formatter to generate PNG directly
-    await runWithRunner([
-      './vendor/bin/deptrac',
-      '--formatter=graphviz-image',
-      '--output=deptrac.png',
-    ])
+    await runWithRunner(
+      ['./vendor/bin/deptrac', '--formatter=graphviz-image', '--output=deptrac.png'],
+      { type: 'php' },
+    )
     if (await fs.pathExists('./deptrac.png')) {
       await runWithRunner(['git', 'add', 'deptrac.png'], { quiet: true })
 
@@ -45,7 +44,7 @@ export async function generateApiDocs(): Promise<void> {
     // Check if swagger-php is installed by looking for the binary
     // This avoids reading/parsing composer.lock
     if (await fs.pathExists('./vendor/bin/openapi')) {
-      await runWithRunner(['composer', 'generate-api-spec'])
+      await runWithRunner(['composer', 'generate-api-spec'], { type: 'php' })
 
       const diffResult = await runWithRunner(['git', 'diff', '--name-only'], { quiet: true })
       const modifiedFiles = diffResult.toString().trim().split('\n')

booster/.husky/shared/workflow.ts

@@ -2,7 +2,6 @@ import { $, which } from 'zx'
 import {
   ensureMutagenSync,
   formatDuration,
-  getExecCommand,
   initEnvironment,
   isDdevProject,
   isSkipped,
@@ -111,8 +110,10 @@ export async function runQualityTools(files: string[], tools: ToolConfig[]): Pro
         for (const chunk of chunks) {
           await Promise.all(
             chunk.map(async (file) => {
-              const cmd = await getExecCommand([tool.command, ...args, file], tool.type)
-              return runWithRunner(cmd, { quiet: true })
+              return runWithRunner([tool.command, ...args, file], {
+                quiet: true,
+                type: tool.type,
+              })
             }),
           )
         }
@@ -122,8 +123,7 @@ export async function runQualityTools(files: string[], tools: ToolConfig[]): Pro
         if (tool.passFiles !== false) {
           cmdArgs.push(...filesToRun)
         }
-        const cmd = await getExecCommand([tool.command, ...cmdArgs], tool.type)
-        await runWithRunner(cmd)
+        await runWithRunner([tool.command, ...cmdArgs], { type: tool.type })
       }
     })