Skip to content

Commit f978967

Browse files
committed
fix(google_search): clamp num to 1-10 and normalize start
Address Cursor Bugbot: num was coerced with Number() but not bounded, so values like 11 or fractionals reached the API and failed. The tool now truncates and clamps num to the documented 1-10 range and only sends a positive integer start, ignoring non-numeric/out-of-range input.
1 parent 5bb3903 commit f978967

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

apps/sim/tools/google/search.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -110,11 +110,13 @@ export const searchTool: ToolConfig<GoogleSearchParams, GoogleSearchResponse> =
110110
searchParams.append('cx', params.searchEngineId)
111111

112112
// Add optional parameters
113-
if (params.num) {
114-
searchParams.append('num', params.num.toString())
113+
const num = Math.trunc(Number(params.num))
114+
if (Number.isFinite(num) && num > 0) {
115+
searchParams.append('num', Math.min(num, 10).toString())
115116
}
116-
if (params.start) {
117-
searchParams.append('start', params.start.toString())
117+
const start = Math.trunc(Number(params.start))
118+
if (Number.isFinite(start) && start > 0) {
119+
searchParams.append('start', start.toString())
118120
}
119121
if (params.dateRestrict) {
120122
searchParams.append('dateRestrict', params.dateRestrict)

0 commit comments

Comments
 (0)