From 226c1be96415677e2efe5d9824498be1be03b6ce Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 14 Mar 2026 00:36:24 +0000 Subject: [PATCH] fix: tighten bonjour whitespace error coverage --- src/infra/bonjour-errors.test.ts | 8 ++++++++ src/infra/bonjour-errors.ts | 3 ++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/src/infra/bonjour-errors.test.ts b/src/infra/bonjour-errors.test.ts index d102e96284c..2d25ddefae9 100644 --- a/src/infra/bonjour-errors.test.ts +++ b/src/infra/bonjour-errors.test.ts @@ -14,6 +14,14 @@ describe("formatBonjourError", () => { expect(formatBonjourError(err)).toBe("AbortError"); }); + it("treats whitespace-only messages as blank", () => { + const named = new Error(" "); + named.name = "AbortError"; + expect(formatBonjourError(named)).toBe("AbortError"); + + expect(formatBonjourError(new Error(" "))).toBe("Error"); + }); + it("falls back to plain error strings and non-error values", () => { expect(formatBonjourError(new Error(""))).toBe("Error"); expect(formatBonjourError("boom")).toBe("boom"); diff --git a/src/infra/bonjour-errors.ts b/src/infra/bonjour-errors.ts index 17ac468ec6e..1703f2696bf 100644 --- a/src/infra/bonjour-errors.ts +++ b/src/infra/bonjour-errors.ts @@ -1,6 +1,7 @@ export function formatBonjourError(err: unknown): string { if (err instanceof Error) { - const msg = err.message || String(err); + const trimmedMessage = err.message.trim(); + const msg = trimmedMessage || err.name || String(err).trim(); if (err.name && err.name !== "Error") { return msg === err.name ? err.name : `${err.name}: ${msg}`; }