From 5c90965ca5b5ec1bcf3c76d7dceb5f691548dba4 Mon Sep 17 00:00:00 2001
From: Matt Hodgson <8408+mhodgson@users.noreply.github.com>
Date: Thu, 17 Sep 2026 04:43:58 +0000
Subject: [PATCH] fix(sdk): skip discarded catalog read on background tool-sync
 failure

Co-authored-by: polylane[bot] <277585245+polylane[bot]@users.noreply.github.com>
---
 packages/core/sdk/src/executor.ts | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/packages/core/sdk/src/executor.ts b/packages/core/sdk/src/executor.ts
index 6916233cb5..11cc350121 100644
--- a/packages/core/sdk/src/executor.ts
+++ b/packages/core/sdk/src/executor.ts
@@ -3618,6 +3618,13 @@ export const createExecutor = <const TPlugins extends readonly AnyPlugin[] = rea
             integration: String(ref.integration),
             connection: String(ref.name),
           });
+          // A background stale-catalog rebuild discards the returned catalog
+          // (the read answers from the persisted rows), so materializing every
+          // preserved tool row — the schema columns included — only to drop it
+          // is a whole-catalog D1 read per reattempt. The rows stay in place;
+          // skip the copy. An explicit read that joined this in-flight run
+          // flips `mode()` to "explicit" and still gets the rows.
+          if (mode() === "background") return [];
           const keptRows = yield* core.findMany("tool", { where });
           return keptRows.map((row) => rowToTool(row as ConnectionToolRow));
         }