diff --git a/packages/cli/scripts/build-js.mts b/packages/cli/scripts/build-js.mts index ceec839fe..238c16e33 100644 --- a/packages/cli/scripts/build-js.mts +++ b/packages/cli/scripts/build-js.mts @@ -45,8 +45,8 @@ async function main() { } logger.success('Build completed successfully') - } catch (error) { - logger.error(`Build failed: ${error.message}`) + } catch (e) { + logger.error(`Build failed: ${e.message}`) process.exitCode = 1 } } diff --git a/packages/cli/scripts/build.mts b/packages/cli/scripts/build.mts index 9386a5d6c..d49040278 100644 --- a/packages/cli/scripts/build.mts +++ b/packages/cli/scripts/build.mts @@ -282,12 +282,12 @@ async function main() { printSuccess('Build completed') printFooter() } - } catch (error) { + } catch (e) { if (!quiet) { - printError(`Build failed: ${error.message}`) + printError(`Build failed: ${e.message}`) } if (verbose) { - logger.error(error) + logger.error(e) } process.exitCode = 1 } diff --git a/packages/cli/scripts/cover.mts b/packages/cli/scripts/cover.mts index 23461d250..7b7b11507 100644 --- a/packages/cli/scripts/cover.mts +++ b/packages/cli/scripts/cover.mts @@ -304,12 +304,12 @@ async function main() { } } } - } catch (error) { + } catch (e) { if (!quiet) { - printError(`Coverage failed: ${error.message}`) + printError(`Coverage failed: ${e.message}`) } if (verbose) { - logger.error(error) + logger.error(e) } process.exitCode = 1 } diff --git a/packages/cli/scripts/download-assets.mts b/packages/cli/scripts/download-assets.mts index 4e642896a..ea6c1196f 100644 --- a/packages/cli/scripts/download-assets.mts +++ b/packages/cli/scripts/download-assets.mts @@ -125,11 +125,11 @@ async function downloadAsset(config) { logger.groupEnd() logger.success(`${name} extraction complete`) return { name, ok: true } - } catch (error) { + } catch (e) { logger.groupEnd() - logger.error(`Failed to extract ${name}: ${error.message}`) - await logTransientErrorHelp(error) - return { error, name, ok: false } + logger.error(`Failed to extract ${name}: ${e.message}`) + await logTransientErrorHelp(e) + return { error: e, name, ok: false } } } diff --git a/packages/cli/scripts/restore-cache.mts b/packages/cli/scripts/restore-cache.mts index eefb1086c..c3c9ccc82 100644 --- a/packages/cli/scripts/restore-cache.mts +++ b/packages/cli/scripts/restore-cache.mts @@ -261,9 +261,9 @@ async function restoreCache(repo, cacheKey) { logger.success('Cache restored successfully!') return true - } catch (error) { + } catch (e) { if (isVerbose()) { - logger.error(`Cache restoration failed: ${error.message}`) + logger.error(`Cache restoration failed: ${e.message}`) } return false } finally { diff --git a/packages/cli/scripts/sync-checksums.mts b/packages/cli/scripts/sync-checksums.mts index 42aa27c0d..61788a12d 100644 --- a/packages/cli/scripts/sync-checksums.mts +++ b/packages/cli/scripts/sync-checksums.mts @@ -143,8 +143,8 @@ async function fetchGitHubReleaseChecksums( ` Parsed ${Object.keys(checksums).length} checksums from checksums.txt`, ) return checksums - } catch (error) { - console.log(` Failed to download checksums.txt: ${error.message}`) + } catch (e) { + console.log(` Failed to download checksums.txt: ${e.message}`) await safeDelete(tempDir).catch(() => {}) // Fall through to download assets. } @@ -282,8 +282,8 @@ async function main() { const newCount = Object.keys(newChecksums).length console.log(` Updated: ${oldCount} -> ${newCount} checksums\n`) updated++ - } catch (error) { - console.log(` Error: ${error.message}\n`) + } catch (e) { + console.log(` Error: ${e.message}\n`) failed++ } } diff --git a/packages/cli/scripts/test-asset-manager.mts b/packages/cli/scripts/test-asset-manager.mts index 31fb40679..185656ae2 100644 --- a/packages/cli/scripts/test-asset-manager.mts +++ b/packages/cli/scripts/test-asset-manager.mts @@ -136,16 +136,16 @@ async function main() { console.log('') console.log('Phase 1 (Foundation) complete. Ready for Phase 2 (Migration).') console.log('') - } catch (error) { + } catch (e) { console.error('\n' + '='.repeat(60)) console.error('❌ TEST FAILED') console.error('='.repeat(60)) console.error('') - console.error('Error:', error.message) + console.error('Error:', e.message) console.error('') - if (error.stack) { + if (e.stack) { console.error('Stack trace:') - console.error(error.stack) + console.error(e.stack) } process.exitCode = 1 } diff --git a/packages/cli/scripts/utils/patches.mts b/packages/cli/scripts/utils/patches.mts index c73a3bc73..c2214b84a 100644 --- a/packages/cli/scripts/utils/patches.mts +++ b/packages/cli/scripts/utils/patches.mts @@ -229,12 +229,12 @@ export async function createPatch(patchDef) { // Commit the patch. await commitPatch(patchPath, packageName) - } catch (error) { - logger.error(`Error creating patch for ${packageName}:`, error.message) + } catch (e) { + logger.error(`Error creating patch for ${packageName}:`, e.message) // Cleanup temp directory on error. if (patchPath && existsSync(patchPath)) { rmSync(patchPath, { force: true, recursive: true }) } - throw error + throw e } } diff --git a/packages/cli/scripts/validate-bundle.mts b/packages/cli/scripts/validate-bundle.mts index 6f3189143..7bb6175bc 100644 --- a/packages/cli/scripts/validate-bundle.mts +++ b/packages/cli/scripts/validate-bundle.mts @@ -81,8 +81,8 @@ async function main() { logger.log('') process.exitCode = 1 - } catch (error) { - logger.fail(`Validation failed: ${error.message}`) + } catch (e) { + logger.fail(`Validation failed: ${e.message}`) process.exitCode = 1 } } diff --git a/packages/cli/src/commands/scan/output-diff-scan.mts b/packages/cli/src/commands/scan/output-diff-scan.mts index 3625daf08..fe4c12fb9 100644 --- a/packages/cli/src/commands/scan/output-diff-scan.mts +++ b/packages/cli/src/commands/scan/output-diff-scan.mts @@ -91,9 +91,9 @@ async function handleJson( try { await fs.writeFile(file, json, 'utf8') logger.success(`Data successfully written to \`${fileLink(file)}\``) - } catch (err) { + } catch (e) { logger.fail(`Writing to \`${file}\` failed...`) - logger.error(err) + logger.error(e) process.exitCode = 1 } logger.info(dashboardMessage) diff --git a/packages/cli/src/utils/telemetry/integration.mts b/packages/cli/src/utils/telemetry/integration.mts index c6710a341..6dd37149c 100644 --- a/packages/cli/src/utils/telemetry/integration.mts +++ b/packages/cli/src/utils/telemetry/integration.mts @@ -369,9 +369,9 @@ export async function trackEvent( await telemetry.flush() } } - } catch (err) { + } catch (e) { // Telemetry errors should never block CLI execution. - debug(`Failed to track event ${eventType}: ${err}`) + debug(`Failed to track event ${eventType}: ${e}`) } }