From 798b93a42f84e4a3d21a82125a2d55c7d497ad79 Mon Sep 17 00:00:00 2001 From: Zeeshan Ahmad Date: Thu, 9 Jul 2026 22:41:26 +0500 Subject: [PATCH] fix: noop provider check handles ResilientProvider wrapper (#1020) ResilientProvider renames 'noop' to 'resilient(noop)' in its constructor, so provider.name === 'noop' in mem::summarize never matched. Zero-LLM installs got misleading parse failures instead of a clean no-op skip. Now checks both 'noop' and 'resilient(noop)'. --- src/functions/summarize.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/functions/summarize.ts b/src/functions/summarize.ts index 4c501ca8c..fca1b30b4 100644 --- a/src/functions/summarize.ts +++ b/src/functions/summarize.ts @@ -260,7 +260,7 @@ export function registerSummarizeFunction( return { success: false, error: "no_observations" }; } - if (provider.name === "noop") { + if (provider.name === "noop" || provider.name === "resilient(noop)") { logger.info("Summarize skipped — no LLM provider configured", { sessionId, });