Add enforceModelTimeout setting

[Tyriar]

Fixes #286598

[Copilot]

Pull request overview

This PR adds a new experimental setting chat.tools.terminal.enforceTimeoutFromModel to enable timeout enforcement for terminal commands executed via the run-in-terminal tool. When enabled, the model can specify a timeout duration for commands, and the tool will stop tracking after that duration and return partial output.

Key changes:

• Added EnforceTimeoutFromModel configuration setting as an experimental feature
• Extended the tool schema to include a required timeout parameter
• Implemented timeout handling that cancels command execution and returns partial output when timeout is reached

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
src/vs/workbench/contrib/terminalContrib/chatAgentTools/common/terminalChatAgentToolsConfiguration.ts	Adds new experimental setting EnforceTimeoutFromModel with boolean type, defaulting to false
src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts	Adds timeout parameter to tool schema, implements timeout enforcement logic with cancellation and partial output collection

Comments suppressed due to low confidence (1)

src/vs/workbench/contrib/terminalContrib/chatAgentTools/browser/tools/runInTerminalTool.ts:232

• The timeout parameter is marked as 'required' in the tool schema, but it's defined as optional in the IRunInTerminalInputParams interface with 'timeout?:'. This inconsistency means the model must always provide a timeout value, but the code treats it as optional. Either remove 'timeout' from the required array, or change the interface to make it non-optional (timeout:).

			required: [
				'command',
				'explanation',
				'isBackground',
				'timeout',

[meganrogge]

I'm slightly worried about giving this control over to the model, but 🤷🏼‍♀️ we shall see!