refactor: add TypeScript types to script files (catch blocks + main functions)

Author: jdalton

scripts/build.mts

@@ -22,7 +22,7 @@ const registryPath = path.join(rootPath, 'registry')
 const args = process.argv.slice(2)
 const isQuiet = args.includes('--quiet') || args.includes('--silent')
 
-async function main() {
+async function main(): Promise<void> {
   // Build the @socketsecurity/registry package.
   // This is required before running tests that import from it.
   // Pass all arguments through to the registry build script.

scripts/check.mts

@@ -28,7 +28,7 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const rootPath = path.resolve(__dirname, '..')
 const registryDistPath = path.join(rootPath, 'registry', 'dist', 'index.js')
 
-async function main() {
+async function main(): Promise<void> {
   try {
     const all = process.argv.includes('--all')
     const staged = process.argv.includes('--staged')
@@ -148,13 +148,13 @@ async function main() {
       logger.success('All checks passed')
       printFooter()
     }
-  } catch (error) {
+  } catch (error: unknown) {
     logger.error(`Check failed: ${error.message}`)
     process.exitCode = 1
   }
 }
 
-main().catch(e => {
+main().catch((e: unknown) => {
   logger.error(e)
   process.exitCode = 1
 })

scripts/ci/generate-actions-allow-list.mts

@@ -65,7 +65,7 @@ async function getAllYamlFiles(dir) {
 /**
  * Generate and display GitHub Actions allow list.
  */
-async function main() {
+async function main(): Promise<void> {
   const allDependencies = new Map()
 
   // Process workflow files.
@@ -181,4 +181,4 @@ async function main() {
   }
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))

scripts/ci/inline-action-versions-as-shas.mts

@@ -90,7 +90,7 @@ async function processFile(filePath, token, dryRun) {
     try {
       const sha = await resolveRefToSha(owner, repo, ref, { token })
       depMap.set(`${owner}/${repoPath}@${ref}`, { owner, ref, repoPath, sha })
-    } catch (e) {
+    } catch (e: unknown) {
       logger.error(
         `Failed to resolve ${owner}/${repoPath}@${ref}: ${e.message}`,
       )
@@ -168,7 +168,7 @@ async function getAllYamlFiles(dir) {
 /**
  * Main function to process all workflow and action files.
  */
-async function main() {
+async function main(): Promise<void> {
   const args = process.argv.slice(2)
   const dryRun = args.includes('--dry-run')
   const token = process.env.GITHUB_TOKEN || ''
@@ -285,4 +285,4 @@ async function main() {
   }
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))

scripts/ci/show-actions-tree.mts

@@ -91,7 +91,7 @@ async function getAllYamlFiles(dir) {
 /**
  * Generate and display dependency tree for all GitHub Actions.
  */
-async function main() {
+async function main(): Promise<void> {
   const allDependencies = new Set()
   // Map of file -> structured dependencies.
   const dependencyTree = new Map()
@@ -184,4 +184,4 @@ async function main() {
   logger.info(`Total: ${allDependencies.size} unique actions/workflows`)
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))

scripts/clean.mts

@@ -54,7 +54,7 @@ async function cleanDirectories(tasks, options = {}) {
           logger.done(`Cleaned ${name} (already clean)`)
         }
       }
-    } catch (error) {
+    } catch (error: unknown) {
       if (!quiet) {
         logger.error(`Failed to clean ${name}`)
         logger.error(error.message)
@@ -66,7 +66,7 @@ async function cleanDirectories(tasks, options = {}) {
   return 0
 }
 
-async function main() {
+async function main(): Promise<void> {
   try {
     // Parse arguments
     const { values } = parseArgs({
@@ -202,10 +202,10 @@ async function main() {
         logger.success('Clean completed successfully!')
       }
     }
-  } catch (error) {
+  } catch (error: unknown) {
     logger.error(`Clean runner failed: ${error.message}`)
     process.exitCode = 1
   }
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))

scripts/cover.mts

@@ -305,7 +305,7 @@ try {
   }
 
   process.exitCode = exitCode
-} catch (error) {
+} catch (error: unknown) {
   logger.error(`Coverage script failed: ${error.message}`)
   process.exitCode = 1
 }

scripts/fix.mts

@@ -32,7 +32,7 @@ async function run(cmd, args, { label, required = true } = {}) {
       logger.warn(`${label || cmd}: exited ${result.code} (non-blocking)`)
     }
     return 0
-  } catch (e) {
+  } catch (e: unknown) {
     if (!required) {
       logger.warn(`${label || cmd}: ${e.message} (non-blocking)`)
       return 0
@@ -41,7 +41,7 @@ async function run(cmd, args, { label, required = true } = {}) {
   }
 }
 
-async function main() {
+async function main(): Promise<void> {
   // Step 1: Lint fix — delegates to per-package lint scripts.
   const lintExit = await run(
     'pnpm',
@@ -71,7 +71,7 @@ async function main() {
   }
 }
 
-main().catch(e => {
+main().catch((e: unknown) => {
   logger.error(e)
   process.exitCode = 1
 })

scripts/lint.mts

@@ -321,7 +321,7 @@ async function getFilesToLint(options) {
   return { files: lintableFiles, reason: undefined, mode }
 }
 
-async function main() {
+async function main(): Promise<void> {
   try {
     // Parse arguments
     const { positionals, values } = parseArgs({
@@ -441,10 +441,10 @@ async function main() {
         logger.success('All lint checks passed!')
       }
     }
-  } catch (error) {
+  } catch (error: unknown) {
     logger.error(`Lint runner failed: ${error.message}`)
     process.exitCode = 1
   }
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))

scripts/maintenance/taze.mts

@@ -15,7 +15,7 @@ function includesProvenanceDowngradeWarning(output) {
   )
 }
 
-async function main() {
+async function main(): Promise<void> {
   // Run with command line arguments.
   const args = process.argv.slice(2)
 
@@ -62,4 +62,4 @@ async function main() {
   await tazePromise
 }
 
-main().catch(e => logger.error(e))
+main().catch((e: unknown) => logger.error(e))