Skip to content

Commit b3b3c74

Browse files
IlyaasKclaude
andcommitted
Address PR 112 review nits
- Remove unused textResponse import in apps.ts (dead import) - Add .min(0) to browser pool fill_rate_per_minute so the schema rejects negatives; left unbounded above and non-integer since the SDK/API type it as a plain number (fractional/>100% rates are valid) Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent f438766 commit b3b3c74

2 files changed

Lines changed: 1 addition & 1 deletion

File tree

src/lib/mcp/tools/apps.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ import {
66
errorResponse,
77
jsonResponse,
88
paginatedJsonResponse,
9-
textResponse,
109
toolErrorResponse,
1110
} from "@/lib/mcp/responses";
1211
import { paginationParams } from "@/lib/mcp/schemas";

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,7 @@ export function registerBrowserPoolCapabilities(server: McpServer) {
257257
.optional(),
258258
fill_rate_per_minute: z
259259
.number()
260+
.min(0)
260261
.describe(
261262
"(create, update) Pool fill rate percentage per minute. Default 10%.",
262263
)

0 commit comments

Comments
 (0)