From 6b518dc9e4c601c0108768932dc1450c036075fd Mon Sep 17 00:00:00 2001 From: Taylor Mullen Date: Fri, 9 May 2025 23:29:02 -0700 Subject: Enable tools to cancel active execution. - Plumbed abort signals through to tools - Updated the shell tool to properly cancel active requests by killing the entire child process tree of the underlying shell process and then report that the shell itself was canceled. Fixes https://b.corp.google.com/issues/416829935 --- packages/server/src/tools/read-many-files.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'packages/server/src/tools/read-many-files.ts') diff --git a/packages/server/src/tools/read-many-files.ts b/packages/server/src/tools/read-many-files.ts index 0b4b090d..44882e44 100644 --- a/packages/server/src/tools/read-many-files.ts +++ b/packages/server/src/tools/read-many-files.ts @@ -237,7 +237,10 @@ Default excludes apply to common non-text files and large dependency directories return `Will attempt to read and concatenate files ${pathDesc}. ${excludeDesc}. File encoding: ${DEFAULT_ENCODING}. Separator: "${DEFAULT_OUTPUT_SEPARATOR_FORMAT.replace('{filePath}', 'path/to/file.ext')}".`; } - async execute(params: ReadManyFilesParams): Promise { + async execute( + params: ReadManyFilesParams, + _signal: AbortSignal, + ): Promise { const validationError = this.validateParams(params); if (validationError) { return { -- cgit v1.2.3