From 22683f691060387f52c78986cdc9ec9713d83f46 Mon Sep 17 00:00:00 2001 From: Rod Boev Date: Tue, 10 Feb 2026 17:46:28 -0500 Subject: [PATCH] fix: clarify TypeError order dependency in error classifier Address Greptile review: add comment noting that TypeError('fetch failed') is already handled by transport patterns before the instanceof check. --- src/cli/hook-command.ts | 1 - 1 file changed, 1 deletion(-) diff --git a/src/cli/hook-command.ts b/src/cli/hook-command.ts index 7245e0e1..c39cc573 100644 --- a/src/cli/hook-command.ts +++ b/src/cli/hook-command.ts @@ -54,7 +54,6 @@ export function isWorkerUnavailableError(error: unknown): boolean { // HTTP 4xx client errors — our bug, NOT worker unavailability if (/failed:\s*4\d{2}/.test(message) || /status[:\s]+4\d{2}/.test(message)) return false; - // Programming errors — code bugs, not worker unavailability // Programming errors — code bugs, not worker unavailability // Note: TypeError('fetch failed') already handled by transport patterns above if (error instanceof TypeError || error instanceof ReferenceError || error instanceof SyntaxError) {