Skip to content

Commit 8717fe9

Browse files
committed
fix: add missing status and data fields to ToolStartedOrCompletedActivityPayload
The schema for tool.started and tool.completed activities was missing the optional 'status' and 'data' fields that ToolUpdatedActivityPayload has. When activities are decoded through Schema.decodeUnknownSync, these fields were silently stripped, causing work log entries to lose command and changed-file information from completion events.
1 parent 0458f4e commit 8717fe9

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

packages/contracts/src/orchestration.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,9 @@ const ContextCompactionActivityPayload = Schema.Struct({
362362

363363
const ToolStartedOrCompletedActivityPayload = Schema.Struct({
364364
itemType: TrimmedNonEmptyString,
365+
status: Schema.optional(TrimmedNonEmptyString),
365366
detail: Schema.optional(TrimmedNonEmptyString),
367+
data: Schema.optional(CanonicalToolLifecycleData),
366368
});
367369

368370
const ToolUpdatedActivityPayload = Schema.Struct({

0 commit comments

Comments
 (0)