Skip to content

Commit 6f3e4a7

Browse files
IlyaasKclaude
andcommitted
Omit empty pool proxy_id
buildPoolConfigParams included proxy_id whenever it was defined, so an empty string was forwarded to the API on pool create/update. Match the truthiness check manage_browsers create uses (and the sibling name field) so empty proxy_id is dropped; pools have no clear-proxy semantics for an empty string. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 87a580c commit 6f3e4a7

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/lib/mcp/tools/browser-pools.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ function buildPoolConfigParams(
5959
...(params.timeout_seconds !== undefined && {
6060
timeout_seconds: params.timeout_seconds,
6161
}),
62-
...(params.proxy_id !== undefined && { proxy_id: params.proxy_id }),
62+
...(params.proxy_id && { proxy_id: params.proxy_id }),
6363
...(params.fill_rate_per_minute !== undefined && {
6464
fill_rate_per_minute: params.fill_rate_per_minute,
6565
}),

0 commit comments

Comments
 (0)