Replace node --loader with --import

jdalton · jdalton · commit 8ac2df652cbd · 2025-10-22T11:45:31.000-04:00
diff --git a/scripts/build.mjs b/scripts/build.mjs
@@ -141,4 +141,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  console.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/check.mjs b/scripts/check.mjs
@@ -204,4 +204,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/claude.mjs b/scripts/claude.mjs
@@ -5717,4 +5717,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  console.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/clean.mjs b/scripts/clean.mjs
@@ -207,4 +207,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/cover.mjs b/scripts/cover.mjs
@@ -294,4 +294,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/fix.mjs b/scripts/fix.mjs
@@ -81,4 +81,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/lint.mjs b/scripts/lint.mjs
@@ -464,4 +464,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/load.js b/scripts/load.js
@@ -28,8 +28,8 @@ if (require.main === module) {
   const scriptPath = path.join(__dirname, scriptFile)
 
   // Run the script with the alias loader.
-  const loaderPath = path.join(__dirname, 'load.mjs')
-  const args = [`--loader=${loaderPath}`, scriptPath, ...flags]
+  const registerPath = path.join(__dirname, 'register.mjs')
+  const args = [`--import=${registerPath}`, scriptPath, ...flags]
 
   const child = spawn(process.execPath, args, {
     stdio: 'inherit',
diff --git a/scripts/load.mjs b/scripts/load.mjs
@@ -1,11 +1,11 @@
 /**
- * @fileoverview ESM loader that provides alias resolution.
+ * @fileoverview Deprecated - use register.mjs instead.
  *
- * This is the actual loader used by Node.js with --loader flag.
- * For convenience, use scripts/load.js which wraps this automatically.
+ * This file is kept for backward compatibility but should not be used.
+ * Use register.mjs with --import flag instead.
  *
- * Direct usage:
- *   node --loader=./scripts/load.mjs script.mjs
+ * New usage:
+ *   node --import=./scripts/register.mjs script.mjs
  *
  * Wrapper usage (recommended):
  *   node scripts/load script-name
diff --git a/scripts/test.mjs b/scripts/test.mjs
@@ -446,7 +446,7 @@ async function main() {
   }
 }
 
-main().catch(error => {
-  console.error(error)
+main().catch(e => {
+  logger.error(e)
   process.exit(1)
 })
diff --git a/scripts/update.mjs b/scripts/update.mjs
@@ -107,4 +107,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(e => {
+  logger.error(e)
+  process.exitCode = 1
+})
diff --git a/scripts/utils/alias-loader.mjs b/scripts/utils/alias-loader.mjs
@@ -5,7 +5,7 @@
  * This file should be copied or imported from socket-registry to other repos.
  *
  * Usage:
- *   node --loader=./scripts/utils/alias-loader.mjs script.mjs
+ *   node --import=./scripts/register.mjs script.mjs
  */
 
 import { existsSync } from 'node:fs'

Original file line number	Diff line number	Diff line change
`@@ -141,4 +141,7 @@ async function main() {`
`141`	`141`	`}`
`142`	`142`	`}`
`143`	`143`
`144`		`-main().catch(console.error)`
	`144`	`+main().catch(e => {`
	`145`	`+ console.error(e)`
	`146`	`+ process.exitCode = 1`
	`147`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -204,4 +204,7 @@ async function main() {`
`204`	`204`	`}`
`205`	`205`	`}`
`206`	`206`
`207`		`-main().catch(console.error)`
	`207`	`+main().catch(e => {`
	`208`	`+ logger.error(e)`
	`209`	`+ process.exitCode = 1`
	`210`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -5717,4 +5717,7 @@ async function main() {`
`5717`	`5717`	`}`
`5718`	`5718`	`}`
`5719`	`5719`
`5720`		`-main().catch(console.error)`
	`5720`	`+main().catch(e => {`
	`5721`	`+ console.error(e)`
	`5722`	`+ process.exitCode = 1`
	`5723`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -207,4 +207,7 @@ async function main() {`
`207`	`207`	`}`
`208`	`208`	`}`
`209`	`209`
`210`		`-main().catch(console.error)`
	`210`	`+main().catch(e => {`
	`211`	`+ logger.error(e)`
	`212`	`+ process.exitCode = 1`
	`213`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -294,4 +294,7 @@ async function main() {`
`294`	`294`	`}`
`295`	`295`	`}`
`296`	`296`
`297`		`-main().catch(console.error)`
	`297`	`+main().catch(e => {`
	`298`	`+ logger.error(e)`
	`299`	`+ process.exitCode = 1`
	`300`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -81,4 +81,7 @@ async function main() {`
`81`	`81`	`}`
`82`	`82`	`}`
`83`	`83`
`84`		`-main().catch(console.error)`
	`84`	`+main().catch(e => {`
	`85`	`+ logger.error(e)`
	`86`	`+ process.exitCode = 1`
	`87`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -464,4 +464,7 @@ async function main() {`
`464`	`464`	`}`
`465`	`465`	`}`
`466`	`466`
`467`		`-main().catch(console.error)`
	`467`	`+main().catch(e => {`
	`468`	`+ logger.error(e)`
	`469`	`+ process.exitCode = 1`
	`470`	`+})`
Original file line number	Diff line number	Diff line change
`@@ -1,11 +1,11 @@`
`1`	`1`	`/**`
`2`		`- * @fileoverview ESM loader that provides alias resolution.`
	`2`	`+ * @fileoverview Deprecated - use register.mjs instead.`
`3`	`3`	`*`
`4`		`- * This is the actual loader used by Node.js with --loader flag.`
`5`		`- * For convenience, use scripts/load.js which wraps this automatically.`
	`4`	`+ * This file is kept for backward compatibility but should not be used.`
	`5`	`+ * Use register.mjs with --import flag instead.`
`6`	`6`	`*`
`7`		`- * Direct usage:`
`8`		`- * node --loader=./scripts/load.mjs script.mjs`
	`7`	`+ * New usage:`
	`8`	`+ * node --import=./scripts/register.mjs script.mjs`
`9`	`9`	`*`
`10`	`10`	`* Wrapper usage (recommended):`
`11`	`11`	`* node scripts/load script-name`
Original file line number	Diff line number	Diff line change
`@@ -446,7 +446,7 @@ async function main() {`
`446`	`446`	`}`
`447`	`447`	`}`
`448`	`448`
`449`		`-main().catch(error => {`
`450`		`- console.error(error)`
	`449`	`+main().catch(e => {`
	`450`	`+ logger.error(e)`
`451`	`451`	`process.exit(1)`
`452`	`452`	`})`