Better IPC error logging (#9727)

Author: cte

packages/ipc/src/ipc-client.ts

@@ -58,14 +58,16 @@ export class IpcClient extends EventEmitter<IpcClientEvents> {
 
 	private onMessage(data: unknown) {
 		if (typeof data !== "object") {
-			this._log("[client#onMessage] invalid data", data)
+			this.log(`[client#onMessage] invalid data -> ${JSON.stringify(data)}`)
 			return
 		}
 
 		const result = ipcMessageSchema.safeParse(data)
 
 		if (!result.success) {
-			this.log("[client#onMessage] invalid payload", result.error, data)
+			this.log(
+				`[client#onMessage] invalid payload -> ${JSON.stringify(result.error.issues)} -> ${JSON.stringify(data)}`,
+			)
 			return
 		}
 
@@ -113,9 +115,10 @@ export class IpcClient extends EventEmitter<IpcClientEvents> {
 	public disconnect() {
 		try {
 			ipc.disconnect(this._id)
-			// @TODO: Should we set _disconnect here?
 		} catch (error) {
-			this.log("[client#disconnect] error disconnecting", error)
+			this.log(
+				`[client#disconnect] error disconnecting -> ${error instanceof Error ? error.message : String(error)}`,
+			)
 		}
 	}
 

packages/ipc/src/ipc-server.ts

@@ -76,18 +76,17 @@ export class IpcServer extends EventEmitter<IpcServerEvents> implements RooCodeI
 
 	private onMessage(data: unknown) {
 		if (typeof data !== "object") {
-			this.log("[server#onMessage] invalid data", data)
+			this.log(`[server#onMessage] invalid data -> ${JSON.stringify(data)}`)
 			return
 		}
 
 		const result = ipcMessageSchema.safeParse(data)
 
 		if (!result.success) {
 			this.log(
-				"[server#onMessage] invalid paylooooad",
-				JSON.stringify(result.error.format()),
-				JSON.stringify(data),
+				`[server#onMessage] invalid payload -> ${JSON.stringify(result.error.issues)} -> ${JSON.stringify(data)}`,
 			)
+
 			return
 		}