Skip to content

fix(computer-use): drop activity from core definitions to match TOOLS.json #29378

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
siddseethepalli merged 1 commit into from
May 3, 2026
Merged

fix(computer-use): drop activity from core definitions to match TOOLS.json #29378

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 2 additions & 20 deletions assistant/src/tools/computer-use/definitions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,6 @@ function proxyExecute(): Promise<ToolExecutionResult> {
);
}

const activityProperty = {
type: "string" as const,
description:
"Brief non-technical explanation of why this tool is being called",
};

// ---------------------------------------------------------------------------
// click (unified - click_type selects single / double / right)
// ---------------------------------------------------------------------------
Expand Down Expand Up @@ -69,7 +63,6 @@ export const computerUseClickTool: Tool = {
description:
"Explanation of what you see and why you are clicking here",
},
activity: activityProperty,
},
required: ["reasoning"],
},
Expand Down Expand Up @@ -106,7 +99,6 @@ export const computerUseTypeTextTool: Tool = {
type: "string",
description: "Explanation of what you are typing and why",
},
activity: activityProperty,
},
required: ["text", "reasoning"],
},
Expand Down Expand Up @@ -144,7 +136,6 @@ export const computerUseKeyTool: Tool = {
type: "string",
description: "Explanation of why you are pressing this key",
},
activity: activityProperty,
},
required: ["key", "reasoning"],
},
Expand Down Expand Up @@ -199,7 +190,6 @@ export const computerUseScrollTool: Tool = {
type: "string",
description: "Explanation of why you are scrolling",
},
activity: activityProperty,
},
required: ["direction", "amount", "reasoning"],
},
Expand Down Expand Up @@ -260,7 +250,6 @@ export const computerUseDragTool: Tool = {
type: "string",
description: "Explanation of what you are dragging and why",
},
activity: activityProperty,
},
required: ["reasoning"],
},
Expand Down Expand Up @@ -296,7 +285,6 @@ export const computerUseWaitTool: Tool = {
type: "string",
description: "Explanation of what you are waiting for",
},
activity: activityProperty,
},
required: ["duration_ms", "reasoning"],
},
Expand Down Expand Up @@ -335,7 +323,6 @@ export const computerUseOpenAppTool: Tool = {
description:
"Explanation of why you need to open or switch to this app",
},
activity: activityProperty,
},
required: ["app_name", "reasoning"],
},
Expand Down Expand Up @@ -373,7 +360,6 @@ export const computerUseRunAppleScriptTool: Tool = {
description:
"Explanation of what this script does and why AppleScript is better than UI interaction for this step",
},
activity: activityProperty,
},
required: ["script", "reasoning"],
},
Expand Down Expand Up @@ -406,7 +392,6 @@ export const computerUseDoneTool: Tool = {
type: "string",
description: "Human-readable summary of what was accomplished",
},
activity: activityProperty,
},
required: ["summary"],
},
Expand Down Expand Up @@ -443,7 +428,6 @@ export const computerUseRespondTool: Tool = {
type: "string",
description: "Explanation of how you determined the answer",
},
activity: activityProperty,
},
required: ["answer", "reasoning"],
},
Expand Down Expand Up @@ -471,10 +455,8 @@ const computerUseObserveTool: Tool = {
description: this.description,
input_schema: {
type: "object",
properties: {
activity: activityProperty,
},
required: ["activity"],
properties: {},
required: [],
},
};
},
Expand Down
Loading