Skip to content

Commit 1168863

Browse files
tconley1428claude
andauthored
Standardize resource_id annotations to use bare values (#737)
## Summary - Remove prefix responsibility (`workflow:`, `worker:`) from proto annotations for `resource_id` fields - All `resource_id` annotations now use bare `{resource_id}`, making the SDK responsible for providing the fully-prefixed value - This standardizes the contract: previously activity request annotations used bare `{resource_id}` while workflow/worker request annotations used `workflow:{resource_id}` or `worker:{resource_id}` ## Affected RPCs - `ExecuteMultiOperation`: `workflow:{resource_id}` → `{resource_id}` - `RespondWorkflowTaskCompleted`: `workflow:{resource_id}` → `{resource_id}` - `RespondWorkflowTaskFailed`: `workflow:{resource_id}` → `{resource_id}` - `RecordWorkerHeartbeat`: `worker:{resource_id}` → `{resource_id}` - `FetchWorkerConfig`: `worker:{resource_id}` → `{resource_id}` - `UpdateWorkerConfig`: `worker:{resource_id}` → `{resource_id}` 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
1 parent ee1ff0d commit 1168863

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

temporal/api/workflowservice/v1/service.proto

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ service WorkflowService {
121121
rpc ExecuteMultiOperation (ExecuteMultiOperationRequest) returns (ExecuteMultiOperationResponse) {
122122
option (temporal.api.protometa.v1.request_header) = {
123123
header: "temporal-resource-id"
124-
value: "workflow:{resource_id}"
124+
value: "{resource_id}"
125125
};
126126
}
127127

@@ -180,7 +180,7 @@ service WorkflowService {
180180
rpc RespondWorkflowTaskCompleted (RespondWorkflowTaskCompletedRequest) returns (RespondWorkflowTaskCompletedResponse) {
181181
option (temporal.api.protometa.v1.request_header) = {
182182
header: "temporal-resource-id"
183-
value: "workflow:{resource_id}"
183+
value: "{resource_id}"
184184
};
185185
}
186186

@@ -199,7 +199,7 @@ service WorkflowService {
199199
rpc RespondWorkflowTaskFailed (RespondWorkflowTaskFailedRequest) returns (RespondWorkflowTaskFailedResponse) {
200200
option (temporal.api.protometa.v1.request_header) = {
201201
header: "temporal-resource-id"
202-
value: "workflow:{resource_id}"
202+
value: "{resource_id}"
203203
};
204204
}
205205

@@ -1460,7 +1460,7 @@ service WorkflowService {
14601460
};
14611461
option (temporal.api.protometa.v1.request_header) = {
14621462
header: "temporal-resource-id"
1463-
value: "worker:{resource_id}"
1463+
value: "{resource_id}"
14641464
};
14651465
};
14661466

@@ -1505,7 +1505,7 @@ service WorkflowService {
15051505
};
15061506
option (temporal.api.protometa.v1.request_header) = {
15071507
header: "temporal-resource-id"
1508-
value: "worker:{resource_id}"
1508+
value: "{resource_id}"
15091509
};
15101510
}
15111511

@@ -1523,7 +1523,7 @@ service WorkflowService {
15231523
};
15241524
option (temporal.api.protometa.v1.request_header) = {
15251525
header: "temporal-resource-id"
1526-
value: "worker:{resource_id}"
1526+
value: "{resource_id}"
15271527
};
15281528
}
15291529

0 commit comments

Comments
 (0)