singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: limit native web_search to max_uses:5 per response (#817) (#869)

Browse source
This commit is contained in:
Tom Boucher 2026-03-17 10:23:05 -04:00 • committed by GitHub
parent 4c7c64f7f5
commit ae8b8eeca3
2 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/resources/extensions/search-the-web/native-search.ts
View file

@ -157,6 +157,10 @@ export function registerNativeSearchHooks(pi: NativeSearchPI): { getIsAnthropic:
tools.push({
type: "web_search_20250305",
name: "web_search",
// Cap server-side searches per response to prevent the model from
// looping on web_search without synthesizing results (#817).
// 5 searches is generous — most queries need 1-2.
max_uses: 5,
});
return payload;

6
src/tests/native-search.test.ts
View file

@ -92,11 +92,12 @@ test("before_provider_request injects web_search for claude models", async () =>
});
const tools = (result as any)?.tools ?? payload.tools;
const hasNative = (tools as any[]).some(
const nativeTool = (tools as any[]).find(
(t: any) => t.type === "web_search_20250305"
);
assert.ok(hasNative, "Should inject web_search_20250305 tool");
assert.ok(nativeTool, "Should inject web_search_20250305 tool");
assert.equal((tools as any[]).length, 2, "Should have original + injected tool");
assert.equal(nativeTool.max_uses, 5, "Should set max_uses to 5 to prevent search loops (#817)");
});
test("before_provider_request injects web_search for claude models even without model_select", async () => {
@ -278,6 +279,7 @@ test("before_provider_request creates tools array if missing", async () => {
assert.ok(Array.isArray(tools), "Should create tools array");
assert.equal((tools as any[]).length, 1, "Should have exactly 1 tool");
assert.equal((tools as any[])[0].type, "web_search_20250305");
assert.equal((tools as any[])[0].max_uses, 5, "Should include max_uses limit");
});
test("before_provider_request skips when payload is falsy", async () => {