@oh-my-pi/pi-utils 15.3.2 → 15.4.2

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-utils",
-	"version": "15.3.2",
+	"version": "15.4.2",
 	"description": "Shared utilities for pi packages",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -31,7 +31,7 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "15.3.2",
+		"@oh-my-pi/pi-natives": "15.4.2",
 		"beautiful-mermaid": "^1.1.3",
 		"handlebars": "^4.7.9",
 		"winston": "^3.19.0",

package/src/fetch-retry.ts

@@ -270,7 +270,7 @@ export function isUnexpectedSocketCloseMessage(message: string): boolean {
 }
 
 const TRANSIENT_MESSAGE_PATTERN =
-	/overloaded|rate.?limit|too many requests|service.?unavailable|server error|internal error|connection.?error|unable to connect|fetch failed|network error|stream stall|other side closed/i;
+	/overloaded|rate.?limit|too many requests|service.?unavailable|server error|internal error|connection.?error|unable to connect|fetch failed|network error|stream stall|other side closed|HTTP2(?:StreamReset|RefusedStream|EnhanceYourCalm)/i;
 
 const VALIDATION_MESSAGE_PATTERN =
 	/invalid|validation|bad request|unsupported|schema|missing required|not found|unauthorized|forbidden/i;

package/src/logger.ts

@@ -23,6 +23,28 @@ function ensureDir(dir: string): string {
 	return dir;
 }
 
+/**
+ * JSON.stringify replacer that unwraps {@link Error} instances. Error's own
+ * properties are non-enumerable, so a plain `JSON.stringify(err)` produces
+ * `"{}"`. Without this, a context like `{ err }` lost every useful field and
+ * forensic logs showed only an opaque empty object.
+ */
+function jsonReplacer(_key: string, value: unknown): unknown {
+	if (value instanceof Error) {
+		const out: Record<string, unknown> = {
+			name: value.name,
+			message: value.message,
+			stack: value.stack,
+		};
+		// Preserve `.cause` and any custom enumerable fields the caller attached.
+		const errAsRecord = value as unknown as Record<string, unknown>;
+		for (const k in errAsRecord) out[k] = errAsRecord[k];
+		if (value.cause !== undefined) out.cause = value.cause;
+		return out;
+	}
+	return value;
+}
+
 /** Custom format that includes pid and flattens metadata */
 const logFormat = winston.format.combine(
 	winston.format.timestamp({ format: "YYYY-MM-DDTHH:mm:ss.SSSZ" }),
@@ -39,7 +61,7 @@ const logFormat = winston.format.combine(
 				entry[key] = value;
 			}
 		}
-		return JSON.stringify(entry);
+		return JSON.stringify(entry, jsonReplacer);
 	}),
 );
 

package/src/postmortem.ts

@@ -90,14 +90,14 @@ if (isMainThread) {
 		})
 		.on("uncaughtException", async err => {
 			process.stderr.write(formatFatalError("Uncaught Exception", err));
-			logger.error("Uncaught exception", { err, stack: err.stack });
+			logger.error("Uncaught exception", { err });
 			await runCleanup(Reason.UNCAUGHT_EXCEPTION);
 			process.exit(1);
 		})
 		.on("unhandledRejection", async reason => {
 			const err = reason instanceof Error ? reason : new Error(String(reason));
 			process.stderr.write(formatFatalError("Unhandled Rejection", err));
-			logger.error("Unhandled rejection", { err, stack: err.stack });
+			logger.error("Unhandled rejection", { err });
 			await runCleanup(Reason.UNHANDLED_REJECTION);
 			process.exit(1);
 		})