style: rename catch (err|error) identifiers to catch (e)

Mechanical rename across 44 files in src/, scripts/, and test/ to
match the CLAUDE.md catch-binding convention. 79 catch clauses
converted; body references renamed including inside template literal
interpolations. Property keys and member-access expressions
(e.g. logger.error, console.error) are left untouched.

tsgo --noEmit -p .config/tsconfig.check.json clean post-sweep.

Author: jdalton

.config/esbuild.config.mts

@@ -185,10 +185,10 @@ function createPathShorteningPlugin() {
               // Write the modified content
               // eslint-disable-next-line no-await-in-loop
               await fs.writeFile(outputPath, magicString.toString(), 'utf8')
-            } catch (error) {
+            } catch (e) {
               console.error(
                 `Failed to shorten paths in ${outputPath}:`,
-                error.message,
+                e.message,
               )
               // Continue without failing the build
             }

scripts/build-externals/bundler.mts

@@ -110,14 +110,14 @@ ${contentWithoutStrict}`
       logger.log(`    ✓ Bundled ${packageName} (${sizeKB}KB)`)
     }
     return sizeKB
-  } catch (error) {
+  } catch (e) {
     if (!quiet) {
-      logger.log(`    ✗ Failed to bundle ${packageName}: ${error.message}`)
+      logger.log(`    ✗ Failed to bundle ${packageName}: ${e.message}`)
     }
     // Propagate the failure. The orchestrator wraps optional packages in
     // try/catch and logs a "Skipping optional package" message; required
     // packages bubble up so the build exits non-zero instead of silently
     // shipping throw-stubs that only fail later, at consumer runtime.
-    throw error
+    throw e
   }
 }

scripts/check.mts

@@ -155,8 +155,8 @@ async function main(): Promise<void> {
 
     logger.success('All checks passed')
     printFooter()
-  } catch (error) {
-    logger.error(`Check failed: ${error.message}`)
+  } catch (e) {
+    logger.error(`Check failed: ${e.message}`)
     process.exitCode = 1
   }
 }

scripts/claude.mts

@@ -1140,7 +1140,7 @@ async function runClaude(claudeCmd, prompt, options = {}) {
     modelStrategy.recordAttempt(task, true)
 
     return result
-  } catch (error) {
+  } catch (e) {
     // Record failure for potential escalation
     modelStrategy.recordAttempt(task, false)
 
@@ -1152,7 +1152,7 @@ async function runClaude(claudeCmd, prompt, options = {}) {
       return runClaude(claudeCmd, prompt, opts)
     }
 
-    throw error
+    throw e
   }
 }
 
@@ -4791,8 +4791,8 @@ Fix all issues by making necessary file changes. Be direct, don't ask questions.
           if (exitCode !== 0) {
             log.warn(`Claude fix exited with code ${exitCode}`)
           }
-        } catch (error) {
-          log.warn(`Claude fix error: ${error.message}`)
+        } catch (e) {
+          log.warn(`Claude fix error: ${e.message}`)
         } finally {
           clearInterval(progressInterval)
           log.done('Claude fix attempt completed')
@@ -5111,8 +5111,8 @@ Fix the issue by making necessary file changes. Be direct, don't ask questions.`
                 if (exitCode !== 0) {
                   log.warn(`Claude fix exited with code ${exitCode}`)
                 }
-              } catch (error) {
-                log.warn(`Claude fix error: ${error.message}`)
+              } catch (e) {
+                log.warn(`Claude fix error: ${e.message}`)
               } finally {
                 clearInterval(progressInterval)
                 log.done(`Fix attempt for ${job.name} completed`)
@@ -5313,8 +5313,8 @@ async function runWatchMode(claudeCmd, options = {}) {
           } else {
             log.done('No issues found')
           }
-        } catch (error) {
-          log.failed(`Error scanning ${project.name}: ${error.message}`)
+        } catch (e) {
+          log.failed(`Error scanning ${project.name}: ${e.message}`)
         }
       },
     )
@@ -5346,8 +5346,8 @@ async function runWatchMode(claudeCmd, options = {}) {
               },
             )
           }
-        } catch (error) {
-          log.failed(`Full scan error in ${project.name}: ${error.message}`)
+        } catch (e) {
+          log.failed(`Full scan error in ${project.name}: ${e.message}`)
         }
       }
       // 30 minutes
@@ -5750,8 +5750,8 @@ async function main(): Promise<void> {
     }
 
     process.exitCode = success ? 0 : 1
-  } catch (error) {
-    log.error(`Operation failed: ${error.message}`)
+  } catch (e) {
+    log.error(`Operation failed: ${e.message}`)
     process.exitCode = 1
   }
 }

scripts/fix/commonjs-exports.mts

@@ -209,10 +209,10 @@ async function processDirectory(
         }
       }
     }
-  } catch (error) {
+  } catch (e) {
     // Skip directories that don't exist
-    if (error.code !== 'ENOENT') {
-      throw error
+    if (e.code !== 'ENOENT') {
+      throw e
     }
   }
 
@@ -233,8 +233,8 @@ async function fixConstantExports() {
           : 'CommonJS Exports (no changes)'
       logger.success(title)
     }
-  } catch (error) {
-    logger.error(`Failed to fix CommonJS exports: ${error.message}`)
+  } catch (e) {
+    logger.error(`Failed to fix CommonJS exports: ${e.message}`)
     process.exitCode = 1
   }
 }

scripts/fix/external-imports.mts

@@ -123,10 +123,10 @@ async function processDirectory(dir, verbose = false) {
         }
       }
     }
-  } catch (error) {
+  } catch (e) {
     // Skip directories that don't exist.
-    if (error.code !== 'ENOENT') {
-      throw error
+    if (e.code !== 'ENOENT') {
+      throw e
     }
   }
 
@@ -147,8 +147,8 @@ async function fixExternalImports() {
           : 'External Imports (no changes)'
       logger.success(title)
     }
-  } catch (error) {
-    logger.error(`Failed to fix external imports: ${error.message}`)
+  } catch (e) {
+    logger.error(`Failed to fix external imports: ${e.message}`)
     process.exitCode = 1
   }
 }

scripts/lint.mts

@@ -457,8 +457,8 @@ async function main(): Promise<void> {
         logger.success('All lint checks passed!')
       }
     }
-  } catch (error) {
-    logger.error(`Lint runner failed: ${error.message}`)
+  } catch (e) {
+    logger.error(`Lint runner failed: ${e.message}`)
     process.exitCode = 1
   }
 }

scripts/test/main.mts

@@ -506,12 +506,12 @@ async function main(): Promise<void> {
     } else {
       logger.success('All tests passed!')
     }
-  } catch (error) {
+  } catch (e) {
     // Ensure spinner is stopped
     try {
       spinner.stop()
     } catch {}
-    logger.error(`Test runner failed: ${error.message}`)
+    logger.error(`Test runner failed: ${e.message}`)
     process.exitCode = 1
   } finally {
     // Ensure spinner is stopped

scripts/update.mts

@@ -91,12 +91,12 @@ async function main(): Promise<void> {
         logger.log('')
       }
     }
-  } catch (error) {
+  } catch (e) {
     if (!quiet) {
-      logger.fail(`Update failed: ${error.message}`)
+      logger.fail(`Update failed: ${e.message}`)
     }
     if (verbose) {
-      logger.error(error)
+      logger.error(e)
     }
     process.exitCode = 1
   }

scripts/utils/parse-args.mts

@@ -42,15 +42,15 @@ export function parseArgs(
       values: result.values,
       positionals: result.positionals || [],
     }
-  } catch (error) {
+  } catch (e) {
     // If parsing fails in non-strict mode, return empty values
     if (!strict) {
       return {
         values: {},
         positionals: args.filter(arg => !arg.startsWith('-')),
       }
     }
-    throw error
+    throw e
   }
 }