[DO NOT MERGE] Implement operator commands for Standalone Activities by GregoryTravis · Pull Request #543 · temporalio/sdk-ruby

GregoryTravis · 2026-08-18T01:22:07Z

Implement operator commands for standalone activities

Adds pause, unpause, reset, and update-options to standalone activities, plus
the describe surface needed to observe their effects.

Standalone activities already supported start, result, describe, cancel, and
terminate. This adds the four operator commands the server exposes for them, so
an operator can hold, resume, restart, and retune a running activity without
going through a workflow.

Describe: payload fields are opt-in

DescribeActivityExecutionRequest gates four payload-bearing fields behind
per-call flags (api#792). All four are now plumbed through describe and
default to false.

…e + bridge updates) # Conflicts: # temporalio/ext/sdk-core # temporalio/lib/temporalio/api/enums/v1/failed_cause.rb # temporalio/lib/temporalio/api/namespace/v1/message.rb # temporalio/lib/temporalio/api/workflowservice/v1/request_response.rb # temporalio/sig/temporalio/api/enums/v1/failed_cause.rbs

…in + api overlay + api-adapter fixups), Cargo.lock regenerated, pin Ruby 3.3.11

…well

…exact values

CLAassistant · 2026-08-18T01:22:14Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ GregoryTravis
❌ SAA-OC-Tracking

SAA-OC-Tracking seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

CLAassistant · 2026-08-18T01:22:14Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ GregoryTravis
❌ SAA-OC-Tracking

SAA-OC-Tracking seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

chris-olszewski

Largest concern is the update_options API

chris-olszewski · 2026-08-18T17:17:10Z

@@ -0,0 +1 @@
+3.3.11


Intentional?

Removed; this was part of tracking upstream server changes (and working around a CI segfault) and should have been removed before the PR.

chris-olszewski · 2026-08-18T17:19:07Z


 See https://docs.temporal.io/standalone-activity for the cross-SDK feature overview.

+#### Standalone Activity operator commands


This is in the wrong section of the changelog

chris-olszewski · 2026-08-18T17:19:56Z

Was there a reason/need for updating the Rust lockfile? I don't see any bridge changes

Removed; this was part of tracking upstream server changes (and working around a CI segfault) and should have been removed before the PR.

chris-olszewski · 2026-08-18T19:42:17Z

+      def update_options(
+        task_queue: UNSET,
+        schedule_to_close_timeout: UNSET,
+        schedule_to_start_timeout: UNSET,
+        start_to_close_timeout: UNSET,
+        heartbeat_timeout: UNSET,
+        retry_policy: UNSET,
+        priority: UNSET,
+        start_delay: UNSET,
+        restore_original: false,
+        rpc_options: nil
+      )


I'm really not sold on this approach to distinguishing clearing a value compared to not updating it. UNSET will be visible as the default for users, but not the declaration/existence in the YARD documentation. Also, to me UNSET reads as "unsetting" the value. I think either UNDEFINED or NOT_SET would be better along with making the value public.

There's a similar design problem for search attribute updates that we resolved by having an explicit Update class, but that doesn't map super well here since these are real attributes and not just user chosen names. We could get away with **kwargs inspection directly, but that might make for less useful docs/autocomplete. It is a pattern used by Rails for partial updates so not outlandish to use here.

Used kwargs inspection.

chris-olszewski · 2026-08-18T19:45:25Z

I think we need to provide signatures for update_options

Added, and also added a bunch more missing types.

chris-olszewski · 2026-08-18T19:49:10Z

+      # execution_time is populated by the server (api#807 + temporal#11017); without start_delay
+      # it should be at or near schedule_time.
+      refute_nil desc.execution_time
+      assert_in_delta desc.schedule_time.to_f, desc.execution_time.to_f, 1.0


Would this assertion capture an SDK bug, isn't this server behavior?

Yes, removed.

chris-olszewski · 2026-08-18T19:52:42Z

+    # unpause carries the reason, jitter, and an auto-generated dedup request_id (api#844).
+    unpause_req = captured.fetch(:unpause)
+    assert_equal 'go', unpause_req.reason
+    assert_equal 5, unpause_req.jitter.seconds
+    assert_equal 0, unpause_req.jitter.nanos
+    refute_empty unpause_req.request_id
+
+    # reset carries jitter and an auto-generated dedup request_id (api#844).
+    reset_req = captured.fetch(:reset)
+    assert_equal 2, reset_req.jitter.seconds
+    assert_equal 0, reset_req.jitter.nanos
+    refute_empty reset_req.request_id
+
+    # update_options carries an auto-generated dedup request_id (api#844).


All of these references to the specific API PR that introduced these fields feels unnecessary

Removed, the assertions are self-explanatory.

…ested optional describe payloads.

GregoryTravis and others added 30 commits June 18, 2026 15:50

wip

b009d3e

comments, assert_eventually_paused

bcf1e9c

wip

6eddd1f

Unit test for unobservable params. test_update_options_all_fields.

ed16a37

Bump sdk-core to v0.4.0 (d936c6cc) for activity operator command RPCs.

ccee65c

Redundant test

b3ad5f7

Merge branch 'main' into gmt/operator-commands

5d46dea

Task queue update fix

d816ea7

CHANGELOG

abb6b3d

30s timeout

f4bceb0

WIP: pre-main-merge — accumulated tier2 code + optionC-attempt regen

d9185f6

SAA-OC + Option C on sdk-core v0.5: submodule at sdk-rust-scratch (ma…

19c5913

…in + api overlay + api-adapter fixups), Cargo.lock regenerated, pin Ruby 3.3.11

Fix heartbeat tests

85f81aa

Confirm UpdateOptions surface handles start_delay

5ffa7f6

upstream update

fd7693d

update submodule

44ed201

upstream update

3a54e9d

fmt, submodule

81f7ced

Merge branch 'main' into gmt/operator-commands

c1bf444

Merge branch 'main' into gmt/operator-commands

7cff123

cleanup

949022b

test: update options requires at least one option, and enforce it as …

4d1215f

…well

Add PAUSED, test_describe_paused_activity_reports_paused_status

71c5c0d

test_update_options_on_paused_activity; don't compare with delta for …

a2211da

…exact values

Rename UpdatedOptions to ActivityExecutionOptions

7a68ca0

Round out implementation of four payload details fields, default false.

c32fca1

get input/result/failure

c8412b1

result_hint

753af47

Merge branch 'main' into gmt/operator-commands

fa5b19e

GregoryTravis requested a review from a team as a code owner August 18, 2026 01:22

chris-olszewski requested changes Aug 18, 2026

View reviewed changes

GregoryTravis added 2 commits August 19, 2026 16:48

Merge branch 'main' into gmt/operator-commands

c548ff6

Revert .ruby_version and Cargo.lock changes used for upstream tracking.

52a272a

GregoryTravis changed the title ~~Implement operator commands for Standalone Activities~~ [DO NOT MERGE] Implement operator commands for Standalone Activities Aug 20, 2026

GregoryTravis added 5 commits August 20, 2026 14:38

Move activity_options proto conversion into interceptor. Strip unrequ…

7b7d663

…ested optional describe payloads.

Merge branch 'main' into gmt/operator-commands

0443862

move changelog and shorten

c63c477

Updated types for previous change and eveyrthing else

1e2fba2

cleanup

1fb81d3

GregoryTravis requested a review from chris-olszewski August 20, 2026 19:38

Removed an untyped

4696a81


		See https://docs.temporal.io/standalone-activity for the cross-SDK feature overview.

		#### Standalone Activity operator commands

Uh oh!

Conversation

GregoryTravis commented Aug 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Implement operator commands for standalone activities

Describe: payload fields are opt-in

Uh oh!

CLAassistant commented Aug 18, 2026

Uh oh!

CLAassistant commented Aug 18, 2026

Uh oh!

chris-olszewski left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

GregoryTravis commented Aug 18, 2026 •

edited

Loading