Skip to content

Commit 1d2676a

Browse files
yuyutaotaoclaude
andauthored
refactor(core): remove unused onExecutionDumpUpdate callback (#2143)
No internal callers exist in this repository. The dumpUpdateListeners mechanism already covers the same use case. https://claude.ai/code/session_01JaRzHvgrSXYDL7oeJRhaJp Co-authored-by: Claude <noreply@anthropic.com>
1 parent d73e9f9 commit 1d2676a

File tree

2 files changed

+0
-27
lines changed

2 files changed

+0
-27
lines changed

packages/core/src/agent/agent.ts

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -335,20 +335,6 @@ export class Agent<
335335
runner,
336336
);
337337

338-
if (this.opts.onExecutionDumpUpdate) {
339-
try {
340-
await this.opts.onExecutionDumpUpdate(
341-
this.dump.executions[executionIndex],
342-
{
343-
executionIndex,
344-
groupedDump: this.dump,
345-
},
346-
);
347-
} catch (error) {
348-
console.error('Error in onExecutionDumpUpdate hook', error);
349-
}
350-
}
351-
352338
// Call all registered dump update listeners
353339
const dumpString = this.dumpDataString();
354340
for (const listener of this.dumpUpdateListeners) {

packages/core/src/types.ts

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1235,19 +1235,6 @@ export interface AgentOpt {
12351235
* ```
12361236
*/
12371237
createOpenAIClient?: CreateOpenAIClientFn;
1238-
1239-
/**
1240-
* Persist the latest snapshot of an execution whenever it updates.
1241-
* This is primarily used by stateless CLI flows that need to stitch
1242-
* together executions across separate processes.
1243-
*/
1244-
onExecutionDumpUpdate?: (
1245-
execution: ExecutionDump,
1246-
metadata: {
1247-
executionIndex: number;
1248-
groupedDump: GroupedActionDump;
1249-
},
1250-
) => Promise<void> | void;
12511238
}
12521239

12531240
export type TestStatus =

0 commit comments

Comments
 (0)