Skip to content

Commit b58e464

Browse files
committed
core: gate browser by supportsImages; remove supportsComputerUse usage in prompt generation
1 parent b5122b2 commit b58e464

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/core/task/Task.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2417,7 +2417,7 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
24172417
return SYSTEM_PROMPT(
24182418
provider.context,
24192419
this.cwd,
2420-
(this.api.getModel().info.supportsComputerUse ?? false) && (browserToolEnabled ?? true),
2420+
(this.api.getModel().info.supportsImages ?? false) && (browserToolEnabled ?? true),
24212421
mcpHub,
24222422
this.diffStrategy,
24232423
browserViewportSize,

src/core/webview/generateSystemPrompt.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,13 @@ export const generateSystemPrompt = async (provider: ClineProvider, message: Web
4545
const rooIgnoreInstructions = provider.getCurrentTask()?.rooIgnoreController?.getInstructions()
4646

4747
// Determine if browser tools can be used based on model support, mode, and user settings
48-
let modelSupportsComputerUse = false
48+
let modelSupportsImages = false
4949

50-
// Create a temporary API handler to check if the model supports computer use
50+
// Create a temporary API handler to check if the model supports images
5151
// This avoids relying on an active Cline instance which might not exist during preview
5252
try {
5353
const tempApiHandler = buildApiHandler(apiConfiguration)
54-
modelSupportsComputerUse = tempApiHandler.getModel().info.supportsComputerUse ?? false
54+
modelSupportsImages = tempApiHandler.getModel().info.supportsImages ?? false
5555
} catch (error) {
5656
console.error("Error checking if model supports computer use:", error)
5757
}
@@ -62,7 +62,7 @@ export const generateSystemPrompt = async (provider: ClineProvider, message: Web
6262

6363
// Only enable browser tools if the model supports it, the mode includes browser tools,
6464
// and browser tools are enabled in settings
65-
const canUseBrowserTool = modelSupportsComputerUse && modeSupportsBrowser && (browserToolEnabled ?? true)
65+
const canUseBrowserTool = modelSupportsImages && modeSupportsBrowser && (browserToolEnabled ?? true)
6666

6767
const systemPrompt = await SYSTEM_PROMPT(
6868
provider.context,

0 commit comments

Comments
 (0)