Skip to content

Commit 894ab56

Browse files
committed
chore(scripts): scope the payload-param allowlist to internal operations
Every listed tool is an internal operation, so the probed-request fallback no longer builds or consults it.
1 parent b5206f6 commit 894ab56

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

‎apps/sim/scripts/saved-credential-contract.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,9 @@ const DESTINATION_PARAM =
3131
/(host|hostname|domain|endpoint|url|uri|origin|server|region|instance|shop|account|site|org|tenant|workspace|team|subdomain|space|cluster|environment|location|project)(name|slug|id)?$/i
3232

3333
/**
34-
* Destination-named params of operation-backed tools that are request data, not the host
34+
* Destination-named params of internal-operation tools that are request data, not the host
3535
* the credential goes to: each operation sends its token only to the vendor's fixed API.
36+
* Only the operation branch reads it; a request tool's URL is probed instead.
3637
*/
3738
const OPERATION_PAYLOAD_PARAMS: Readonly<Record<string, readonly string[]>> = {
3839
// The page the remote browser opens; the token goes to api.browser-use.com.
@@ -166,9 +167,8 @@ export function hostSteeringParams(tool: ExecutableToolConfig): HostSteeringResu
166167
}
167168
}
168169
if (!probed) {
169-
const payload = new Set(OPERATION_PAYLOAD_PARAMS[tool.id] ?? [])
170170
return {
171-
params: stringParams.filter((name) => DESTINATION_PARAM.test(name) && !payload.has(name)),
171+
params: stringParams.filter((name) => DESTINATION_PARAM.test(name)),
172172
probed: false,
173173
}
174174
}

0 commit comments

Comments
 (0)