fix(hooks): improve error handling and logging in branch name validation

Author: TerrorSquad

booster/.husky/commit-msg.ts

@@ -122,18 +122,15 @@ async function validateBranchName(branchName: string): Promise<boolean> {
 
   return await runTool('Branch Name', 'Validating branch name...', async () => {
     try {
-      // Try quiet validation first
       await runWithRunner([binPath, '-t', branchName], {
         quiet: true,
       })
-    } catch {
-      // If failed, run again without quiet to show error
+    } catch (error: unknown) {
       log.error('Branch name validation failed')
-      try {
-        await runWithRunner([binPath, '-t', branchName])
-      } catch {
-        // Error output will be shown by runWithRunner
-      }
+      if (typeof error === 'object' && error !== null && 'stdout' in error)
+        console.log(error.stdout)
+      if (typeof error === 'object' && error !== null && 'stderr' in error)
+        console.error(error.stderr)
       log.info('See rules in validate-branch-name.config.cjs')
       throw new Error('Branch name validation failed')
     }

booster/.husky/shared/core.ts

@@ -91,16 +91,8 @@ export async function runWithRunner(
     log.info(`Executing: ${commandStr}`)
   }
 
-  // Set clean environment to avoid locale warnings
-  const cleanEnv = {
-    ...process.env,
-    LC_ALL: 'C',
-    LANG: 'C',
-  }
-
   return await $({
     stdio: quiet ? 'pipe' : 'inherit',
-    env: cleanEnv,
   })`${command}`
 }
 

booster/.husky/shared/workflow.ts

@@ -71,18 +71,12 @@ export async function runQualityTools(files: string[], tools: ToolConfig[]): Pro
     if (filesToRun.length === 0) continue
 
     // Check binary existence
-    const binPath = tool.command
-    let exists = false
-
-    try {
-      await which(binPath)
-      exists = true
-    } catch {
-      exists = false
-    }
+    const exists = await which(tool.command)
+      .then(() => true)
+      .catch(() => false)
 
     if (!exists) {
-      log.skip(`${tool.name} not found at ${binPath}. Skipping...`)
+      log.skip(`${tool.name} not found at ${tool.command}. Skipping...`)
       continue
     }
 
@@ -102,15 +96,15 @@ export async function runQualityTools(files: string[], tools: ToolConfig[]): Pro
 
         for (const chunk of chunks) {
           await Promise.all(
-            chunk.map((file) => runWithRunner([binPath, ...args, file], { quiet: true })),
+            chunk.map((file) => runWithRunner([tool.command, ...args, file], { quiet: true })),
           )
         }
       } else {
         // Run command once with all files
         if (tool.passFiles !== false) {
           args.push(...filesToRun)
         }
-        await runWithRunner([binPath, ...args])
+        await runWithRunner([tool.command, ...args])
       }
     })