Skip to content

fix(core): process tool calls when checkpoint status is completed#423

Merged
FL4TLiN3 merged 2 commits intomainfrom
fix/get-activities-completed-status-tool-calls
Jan 18, 2026
Merged

fix(core): process tool calls when checkpoint status is completed#423
FL4TLiN3 merged 2 commits intomainfrom
fix/get-activities-completed-status-tool-calls

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • Fixed getActivities() to process tool calls when checkpoint status is "completed"
  • Previously, tool calls were ignored and only CompleteActivity was returned
  • Now returns both tool activities and CompleteActivity

Closes #420

Test plan

  • Unit tests added for completed status with tool calls
  • Unit tests added for completed status with parallel tool calls
  • All existing tests pass (56 tests)
  • TypeScript type check passes
  • Lint passes

🤖 Generated with Claude Code

FL4TLiN3 and others added 2 commits January 18, 2026 15:52
Previously, getActivities() would immediately return only CompleteActivity
when status === "completed", ignoring any tool calls in the step.

This fix ensures tool calls are processed for completed status, then
appends CompleteActivity at the end.

Closes #420

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
packages/core/src/utils/activity.ts 75.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 merged commit 808df71 into main Jan 18, 2026
9 checks passed
@FL4TLiN3 FL4TLiN3 deleted the fix/get-activities-completed-status-tool-calls branch January 18, 2026 15:56
@FL4TLiN3 FL4TLiN3 mentioned this pull request Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

getActivities() ignores tool calls when checkpoint status is "completed"

2 participants