Pine/deepseek-harness
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases 1 Wiki Activity

fix(tools): avoid retaining result executions

Browse Source
This commit is contained in:
Yichen Jiang
2026-07-19 20:14:32 +08:00
parent 55a66024fa
commit d49808c6dd
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/core/tools/src/index.ts
+2 -1
View File
@@ -958,8 +958,9 @@ export class ToolRegistry extends Service {
// Freeze the remaining mutable signal slot before observers receive the // Freeze the remaining mutable signal slot before observers receive the
// shared WeakMap-keyable execution object. // shared WeakMap-keyable execution object.
Object.freeze(exec) Object.freeze(exec)
const { name: toolName, callId } = exec
const reportFailure = (error: unknown): void => { const reportFailure = (error: unknown): void => {
this.ctx.logger.warn(`tool "${exec.name}" (${exec.callId}): tools/result observer failed: ${errorMessage(error)}`) this.ctx.logger.warn(`tool "${toolName}" (${callId}): tools/result observer failed: ${errorMessage(error)}`)
} }
const callbacks = this.ctx.events.dispatch('emit', [ const callbacks = this.ctx.events.dispatch('emit', [
scopeTarget(this, exec.agent), 'tools/result', exec, result, scopeTarget(this, exec.agent), 'tools/result', exec, result,