From 22db77d2b6aebd13a2a33e86e946a564c5eaa389 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sun, 5 Apr 2026 09:33:50 +0100 Subject: [PATCH] fix: avoid eager web provider config reads --- src/agents/tools/web-tools.enabled-defaults.test.ts | 7 +++++++ src/plugins/web-fetch-providers.runtime.ts | 1 - src/plugins/web-search-providers.runtime.ts | 1 - 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/src/agents/tools/web-tools.enabled-defaults.test.ts b/src/agents/tools/web-tools.enabled-defaults.test.ts index 95187a97b8a..5ee63c5adb9 100644 --- a/src/agents/tools/web-tools.enabled-defaults.test.ts +++ b/src/agents/tools/web-tools.enabled-defaults.test.ts @@ -2,6 +2,7 @@ import { EnvHttpProxyAgent } from "undici"; import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import { createEmptyPluginRegistry } from "../../plugins/registry.js"; import { setActivePluginRegistry } from "../../plugins/runtime.js"; +import { clearSecretsRuntimeSnapshot } from "../../secrets/runtime.js"; import { withFetchPreconnect } from "../../test-utils/fetch-mock.js"; import { __testing as webSearchTesting } from "./web-search.js"; import { createWebFetchTool, createWebSearchTool } from "./web-tools.js"; @@ -152,8 +153,14 @@ function createProviderSuccessPayload( }; } +beforeEach(() => { + setActivePluginRegistry(createEmptyPluginRegistry()); + clearSecretsRuntimeSnapshot(); +}); + afterEach(() => { setActivePluginRegistry(createEmptyPluginRegistry()); + clearSecretsRuntimeSnapshot(); }); describe("web tools defaults", () => { diff --git a/src/plugins/web-fetch-providers.runtime.ts b/src/plugins/web-fetch-providers.runtime.ts index da5691ed85f..0fb978c037d 100644 --- a/src/plugins/web-fetch-providers.runtime.ts +++ b/src/plugins/web-fetch-providers.runtime.ts @@ -62,7 +62,6 @@ function buildWebFetchSnapshotCacheKey(params: { onlyPluginIds: [...new Set(params.onlyPluginIds ?? [])].toSorted((left, right) => left.localeCompare(right), ), - config: params.config ?? null, env: buildPluginSnapshotCacheEnvKey(params.env), }); } diff --git a/src/plugins/web-search-providers.runtime.ts b/src/plugins/web-search-providers.runtime.ts index a0c8d7fdc90..55f28576bc7 100644 --- a/src/plugins/web-search-providers.runtime.ts +++ b/src/plugins/web-search-providers.runtime.ts @@ -61,7 +61,6 @@ function buildWebSearchSnapshotCacheKey(params: { onlyPluginIds: [...new Set(params.onlyPluginIds ?? [])].toSorted((left, right) => left.localeCompare(right), ), - config: params.config ?? null, env: buildPluginSnapshotCacheEnvKey(params.env), }); }