Skip to content

Commit b37fa51

Browse files
committed
style: rename catch (err|error) identifiers to catch (e)
Mechanical rename across 11 files to match the CLAUDE.md catch-binding convention. 12 catch clauses converted; body references renamed including inside template literal interpolations. Property keys and member-access expressions (logger.error, console.error) are left untouched. tsgo --noEmit clean post-sweep.
1 parent fc5591f commit b37fa51

11 files changed

Lines changed: 31 additions & 31 deletions

File tree

packages/cli/scripts/build-js.mts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ async function main() {
4545
}
4646

4747
logger.success('Build completed successfully')
48-
} catch (error) {
49-
logger.error(`Build failed: ${error.message}`)
48+
} catch (e) {
49+
logger.error(`Build failed: ${e.message}`)
5050
process.exitCode = 1
5151
}
5252
}

packages/cli/scripts/build.mts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -282,12 +282,12 @@ async function main() {
282282
printSuccess('Build completed')
283283
printFooter()
284284
}
285-
} catch (error) {
285+
} catch (e) {
286286
if (!quiet) {
287-
printError(`Build failed: ${error.message}`)
287+
printError(`Build failed: ${e.message}`)
288288
}
289289
if (verbose) {
290-
logger.error(error)
290+
logger.error(e)
291291
}
292292
process.exitCode = 1
293293
}

packages/cli/scripts/cover.mts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -304,12 +304,12 @@ async function main() {
304304
}
305305
}
306306
}
307-
} catch (error) {
307+
} catch (e) {
308308
if (!quiet) {
309-
printError(`Coverage failed: ${error.message}`)
309+
printError(`Coverage failed: ${e.message}`)
310310
}
311311
if (verbose) {
312-
logger.error(error)
312+
logger.error(e)
313313
}
314314
process.exitCode = 1
315315
}

packages/cli/scripts/download-assets.mts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,11 @@ async function downloadAsset(config) {
125125
logger.groupEnd()
126126
logger.success(`${name} extraction complete`)
127127
return { name, ok: true }
128-
} catch (error) {
128+
} catch (e) {
129129
logger.groupEnd()
130-
logger.error(`Failed to extract ${name}: ${error.message}`)
131-
await logTransientErrorHelp(error)
132-
return { error, name, ok: false }
130+
logger.error(`Failed to extract ${name}: ${e.message}`)
131+
await logTransientErrorHelp(e)
132+
return { e, name, ok: false }
133133
}
134134
}
135135

packages/cli/scripts/restore-cache.mts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -261,9 +261,9 @@ async function restoreCache(repo, cacheKey) {
261261

262262
logger.success('Cache restored successfully!')
263263
return true
264-
} catch (error) {
264+
} catch (e) {
265265
if (isVerbose()) {
266-
logger.error(`Cache restoration failed: ${error.message}`)
266+
logger.error(`Cache restoration failed: ${e.message}`)
267267
}
268268
return false
269269
} finally {

packages/cli/scripts/sync-checksums.mts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,8 @@ async function fetchGitHubReleaseChecksums(
141141
` Parsed ${Object.keys(checksums).length} checksums from checksums.txt`,
142142
)
143143
return checksums
144-
} catch (error) {
145-
console.log(` Failed to download checksums.txt: ${error.message}`)
144+
} catch (e) {
145+
console.log(` Failed to download checksums.txt: ${e.message}`)
146146
await fs.rm(tempDir, { recursive: true }).catch(() => {})
147147
// Fall through to download assets.
148148
}
@@ -280,8 +280,8 @@ async function main() {
280280
const newCount = Object.keys(newChecksums).length
281281
console.log(` Updated: ${oldCount} -> ${newCount} checksums\n`)
282282
updated++
283-
} catch (error) {
284-
console.log(` Error: ${error.message}\n`)
283+
} catch (e) {
284+
console.log(` Error: ${e.message}\n`)
285285
failed++
286286
}
287287
}

packages/cli/scripts/test-asset-manager.mts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,16 +136,16 @@ async function main() {
136136
console.log('')
137137
console.log('Phase 1 (Foundation) complete. Ready for Phase 2 (Migration).')
138138
console.log('')
139-
} catch (error) {
139+
} catch (e) {
140140
console.error('\n' + '='.repeat(60))
141141
console.error('❌ TEST FAILED')
142142
console.error('='.repeat(60))
143143
console.error('')
144-
console.error('Error:', error.message)
144+
console.error('Error:', e.message)
145145
console.error('')
146-
if (error.stack) {
146+
if (e.stack) {
147147
console.error('Stack trace:')
148-
console.error(error.stack)
148+
console.error(e.stack)
149149
}
150150
process.exitCode = 1
151151
}

packages/cli/scripts/utils/patches.mts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -229,12 +229,12 @@ export async function createPatch(patchDef) {
229229

230230
// Commit the patch.
231231
await commitPatch(patchPath, packageName)
232-
} catch (error) {
233-
logger.error(`Error creating patch for ${packageName}:`, error.message)
232+
} catch (e) {
233+
logger.error(`Error creating patch for ${packageName}:`, e.message)
234234
// Cleanup temp directory on error.
235235
if (patchPath && existsSync(patchPath)) {
236236
rmSync(patchPath, { force: true, recursive: true })
237237
}
238-
throw error
238+
throw e
239239
}
240240
}

packages/cli/scripts/validate-bundle.mts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,8 +81,8 @@ async function main() {
8181
logger.log('')
8282

8383
process.exitCode = 1
84-
} catch (error) {
85-
logger.fail(`Validation failed: ${error.message}`)
84+
} catch (e) {
85+
logger.fail(`Validation failed: ${e.message}`)
8686
process.exitCode = 1
8787
}
8888
}

packages/cli/src/commands/scan/output-diff-scan.mts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,9 @@ async function handleJson(
9191
try {
9292
await fs.writeFile(file, json, 'utf8')
9393
logger.success(`Data successfully written to \`${fileLink(file)}\``)
94-
} catch (err) {
94+
} catch (e) {
9595
logger.fail(`Writing to \`${file}\` failed...`)
96-
logger.error(err)
96+
logger.error(e)
9797
process.exitCode = 1
9898
}
9999
logger.info(dashboardMessage)

0 commit comments

Comments
 (0)