chore: remove redundant catch type annotations (useUnknownInCatchVariables is enabled)

Author: jdalton

scripts/build.mts

@@ -136,7 +136,7 @@ async function buildSource(
     const buildTime = Date.now() - startTime
 
     return { exitCode: 0, buildTime, result }
-  } catch (error: unknown) {
+  } catch (error) {
     if (!quiet) {
       logger.error('Source build failed')
       console.error(error)
@@ -245,7 +245,7 @@ async function watchBuild(options: WatchBuildOptions = {}): Promise<number> {
 
     // Wait indefinitely
     await new Promise<never>(() => {})
-  } catch (error: unknown) {
+  } catch (error) {
     if (!quiet) {
       logger.error('Watch mode failed:', error)
     }
@@ -473,7 +473,7 @@ async function main(): Promise<void> {
     if (exitCode !== 0) {
       process.exitCode = exitCode
     }
-  } catch (error: unknown) {
+  } catch (error) {
     logger.error(`Build runner failed: ${getErrorMessage(error)}`)
     process.exitCode = 1
   }

scripts/bump.mts

@@ -923,7 +923,7 @@ async function main(): Promise<void> {
     logger.substep('2. Create GitHub release if needed')
 
     process.exitCode = 0
-  } catch (e: unknown) {
+  } catch (e) {
     const message = e instanceof Error ? e.message : String(e)
     logger.error(`Version bump failed: ${message}`)
     process.exitCode = 1

scripts/check.mts

@@ -402,7 +402,7 @@ async function main(): Promise<void> {
       logger.success('All checks passed')
       printFooter()
     }
-  } catch (error: unknown) {
+  } catch (error) {
     logger.error(
       `Check runner failed: ${error instanceof Error ? error.message : String(error)}`,
     )

scripts/ci-validate.mts

@@ -75,7 +75,7 @@ async function main(): Promise<void> {
     logger.success('Build completed')
 
     logger.success('CI validation completed successfully!')
-  } catch (e: unknown) {
+  } catch (e) {
     const message = e instanceof Error ? e.message : String(e)
     logger.error(`CI validation failed: ${message}`)
     process.exitCode = 1

scripts/claude.mts

@@ -1305,7 +1305,7 @@ async function runClaude(
     modelStrategy.recordAttempt(task, true)
 
     return result
-  } catch (e: unknown) {
+  } catch (e) {
     // Record failure for potential escalation
     modelStrategy.recordAttempt(task, false)
 
@@ -5033,7 +5033,7 @@ 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 (e: unknown) {
+        } catch (e) {
           const message = e instanceof Error ? e.message : String(e)
           log.warn(`Claude fix error: ${message}`)
         } finally {
@@ -5356,7 +5356,7 @@ 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 (e: unknown) {
+              } catch (e) {
                 const message = e instanceof Error ? e.message : String(e)
                 log.warn(`Claude fix error: ${message}`)
               } finally {
@@ -5562,7 +5562,7 @@ async function runWatchMode(
           } else {
             log.done('No issues found')
           }
-        } catch (e: unknown) {
+        } catch (e) {
           const message = e instanceof Error ? e.message : String(e)
           log.failed(`Error scanning ${project.name}: ${message}`)
         }
@@ -5596,7 +5596,7 @@ async function runWatchMode(
               },
             )
           }
-        } catch (e: unknown) {
+        } catch (e) {
           const message = e instanceof Error ? e.message : String(e)
           log.failed(`Full scan error in ${project.name}: ${message}`)
         }
@@ -6005,7 +6005,7 @@ async function main(): Promise<void> {
     }
 
     process.exitCode = success ? 0 : 1
-  } catch (e: unknown) {
+  } catch (e) {
     const message = e instanceof Error ? e.message : String(e)
     log.error(`Operation failed: ${message}`)
     process.exitCode = 1

scripts/clean.mts

@@ -83,7 +83,7 @@ async function cleanDirectories(
           logger.done(`Cleaned ${name} (already clean)`)
         }
       }
-    } catch (e: unknown) {
+    } catch (e) {
       if (!quiet) {
         logger.error(`Failed to clean ${name}`)
         const message = e instanceof Error ? e.message : String(e)
@@ -234,7 +234,7 @@ async function main(): Promise<void> {
         logger.success('Clean completed successfully!')
       }
     }
-  } catch (e: unknown) {
+  } catch (e) {
     const message = e instanceof Error ? e.message : String(e)
     logger.error(`Clean runner failed: ${message}`)
     process.exitCode = 1

scripts/cover.mts

@@ -235,7 +235,7 @@ try {
   }
 
   process.exitCode = exitCode
-} catch (e: unknown) {
+} catch (e) {
   const message = e instanceof Error ? e.message : String(e)
   logger.error(`Coverage script failed: ${message}`)
   process.exitCode = 1

scripts/fix.mts

@@ -47,7 +47,7 @@ async function run(
       logger.warn(`${label || cmd}: exited ${result.code} (non-blocking)`)
     }
     return 0
-  } catch (error: unknown) {
+  } catch (error) {
     if (!required) {
       logger.warn(`${label || cmd}: ${getErrorMessage(error)} (non-blocking)`)
       return 0

scripts/lint.mts

@@ -462,7 +462,7 @@ async function main(): Promise<void> {
         logger.success('All lint checks passed!')
       }
     }
-  } catch (error: unknown) {
+  } catch (error) {
     logger.error(
       `Lint runner failed: ${error instanceof Error ? error.message : String(error)}`,
     )

scripts/publish.mts

@@ -81,7 +81,7 @@ async function runCommand(
       ...options,
     })
     return result.code
-  } catch (e: unknown) {
+  } catch (e) {
     // spawn() throws on non-zero exit
     if (e && typeof e === 'object' && 'code' in e) {
       return (e as SpawnError).code
@@ -108,7 +108,7 @@ async function runCommandWithOutput(
       stderr: result.stderr,
       stdout: result.stdout,
     }
-  } catch (e: unknown) {
+  } catch (e) {
     // spawn() throws on non-zero exit
     if (
       e &&
@@ -442,7 +442,7 @@ async function main(): Promise<void> {
       width: 56,
     })
     process.exitCode = 0
-  } catch (e: unknown) {
+  } catch (e) {
     const message = e instanceof Error ? e.message : String(e)
     logger.error(`Publish runner failed: ${message}`)
     process.exitCode = 1