Fix RegExp serialization and add ts-node error handling

justin808 · justin808 · commit c522d268f488 · 2025-10-08T21:25:08.000-10:00
- Use serializeString() for RegExp to properly escape quotes/backslashes
- Add try-catch around ts-node require with helpful error message
- Verified doctor spec still passes with new output text
diff --git a/package/configExporter/cli.ts b/package/configExporter/cli.ts
@@ -319,8 +319,15 @@ async function loadConfigsForEnv(
   // Load the config
   // Register ts-node for TypeScript config files
   if (configFile.endsWith(".ts")) {
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    require("ts-node/register/transpile-only")
+    try {
+      // eslint-disable-next-line @typescript-eslint/no-var-requires
+      require("ts-node/register/transpile-only")
+    } catch (error) {
+      throw new Error(
+        "TypeScript config detected but ts-node is not available. " +
+          "Install ts-node as a dev dependency: npm install --save-dev ts-node"
+      )
+    }
   }
 
   // Clear require cache for config file and all related modules
diff --git a/package/configExporter/yamlSerializer.ts b/package/configExporter/yamlSerializer.ts
@@ -68,7 +68,7 @@ export class YamlSerializer {
     }
 
     if (value instanceof RegExp) {
-      return JSON.stringify(value.toString())
+      return this.serializeString(value.toString())
     }
 
     if (Array.isArray(value)) {

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ export class YamlSerializer {`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`if (value instanceof RegExp) {`
`71`		`- return JSON.stringify(value.toString())`
	`71`	`+ return this.serializeString(value.toString())`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`if (Array.isArray(value)) {`