Skip to content

Ingest API PR #709: Add target_worker_deployment_version_changed field#1148

Merged
tconley1428 merged 3 commits intomasterfrom
ingest-api-pr-709-target-version-changed
Mar 11, 2026
Merged

Ingest API PR #709: Add target_worker_deployment_version_changed field#1148
tconley1428 merged 3 commits intomasterfrom
ingest-api-pr-709-target-version-changed

Conversation

@tconley1428
Copy link
Contributor

Summary

Changes Made

  • Added target_worker_deployment_version_changed field to WorkflowMachines struct
  • Extended WorkflowActivation protobuf definition with new boolean field
  • Updated workflow activation generation to include the new field
  • Ensured all builds and tests pass

Context

The original API change removes SUGGEST_CONTINUE_AS_NEW_REASON_TARGET_WORKER_DEPLOYMENT_VERSION_CHANGED from the suggest-continue-as-new reasons and replaces it with a separate target_worker_deployment_version_changed flag. This provides better ergonomics by making target version change detection opt-in rather than forcing users to filter it out from continue-as-new suggestions.

Test plan

  • Code compiles without errors
  • Existing tests pass
  • Field is properly propagated from API layer to bridge layer

🤖 Generated with Claude Code

This change implements the API changes from temporalio/api PR #709: - Adds `target_worker_deployment_version_changed` boolean field to WorkflowTaskStartedEventAttributes - Propagates the field through core to the bridge layers via WorkflowActivation - The field indicates when a workflow's Target Worker Deployment Version differs from its Pinned Version 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
@tconley1428 tconley1428 requested a review from a team as a code owner March 11, 2026 15:48
tconley1428 and others added 2 commits March 11, 2026 08:59
🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants