Skip to content

Refactor: Move worker commands dispatcher to common/workercommands - #10849

Merged
rkannan82 merged 5 commits into
mainfrom
kannan/chasm-activity-task-token
Jun 26, 2026
Merged

Refactor: Move worker commands dispatcher to common/workercommands#10849
rkannan82 merged 5 commits into
mainfrom
kannan/chasm-activity-task-token

Conversation

@rkannan82

@rkannan82 rkannan82 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What

Moves worker_commands_task_dispatcher.go from service/history to common/workercommands/dispatcher.go and exports its types. The call site in outbound_queue_active_task_executor.go is updated to use the new import path.

Why

To reuse this in PR that sends cancel commands for standalone activities.

How did you test it?

  • Existing unit tests

🤖 Generated with Claude Code

rkannan82 and others added 2 commits June 25, 2026 17:48
Move the entire workerCommandsTaskDispatcher from service/history to
common/workercommands so it can be reused by CHASM standalone activities
(which cannot import service/history due to circular dependency).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkannan82
rkannan82 force-pushed the kannan/chasm-activity-task-token branch from 1cd6858 to ce0370d Compare June 26, 2026 00:51
@rkannan82 rkannan82 changed the title Extract worker commands dispatch into common/workercommands Move worker commands dispatcher to common/workercommands Jun 26, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rkannan82
rkannan82 marked this pull request as ready for review June 26, 2026 01:21
@rkannan82
rkannan82 requested review from a team as code owners June 26, 2026 01:21
@rkannan82
rkannan82 requested a review from yycptt June 26, 2026 01:23
@rkannan82 rkannan82 changed the title Move worker commands dispatcher to common/workercommands Refactor: Move worker commands dispatcher to common/workercommands Jun 26, 2026
@rkannan82
rkannan82 enabled auto-merge (squash) June 26, 2026 18:33
@rkannan82
rkannan82 merged commit 5d609a2 into main Jun 26, 2026
49 checks passed
@rkannan82
rkannan82 deleted the kannan/chasm-activity-task-token branch June 26, 2026 22:12
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.

2 participants