Implement planned topic: 0037-standalone-activities-go#241
Merged
Conversation
Edits throughout Go standalone activities
donald-pinckney
approved these changes
Jun 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Validation Report — Go Standalone Activities (
0037-standalone-activities-go)Branch:
draft/0037-standalone-activities-go. The change is on the working tree (uncommitted), expected for this CI run.Files validated (the diff against
main):references/go/standalone-activities.md— NEW (181 lines).references/core/standalone-activities.md— MODIFIED (adds"Go"to two language listings).references/go/go.md— MODIFIED (one new bullet under "Reference Files").Validation independence: this session read only the authored files, the sibling per-language pages (
references/python/standalone-activities.md,references/typescript/standalone-activities.md, plus heading lists forjava/anddotnet/),SKILL.md, and the docs clone at../documentation/docs/. No authoring plan, log, or prior-session content was read.Go/no-go
Overall verdict: GO.
Check 1 findings — citation audit
None. All 26 inline
<!-- docs/… -->citations inreferences/go/standalone-activities.mdresolve cleanly and the cited text substantively supports the claim.Citations spot-confirmed:
docs/develop/go/activities/standalone-activities.mdx:29-31, 33-34, 59, 61, 91-96, 148-150, 157-184, 196-200, 226-237, 273-280, 282-283, 309-322, 332-341, 353-374, 388-401.docs/encyclopedia/activities/standalone-activity.mdx:115, 139, 143.docs/develop/go/client/temporal-client.mdx:176-198.External hyperlink check:
https://docs.temporal.io/list-filter(L145) is a fullhttps://URL — compliant. Internal repo links (references/core/install_cli.md,references/core/standalone-activities.md) point to existing files.The modified core file (
references/core/standalone-activities.md) introduces no new citations; the diff is two adjacent language-list edits ("…Python, TypeScript, Java, .NET, Go") — verified in-place.Check 2 findings — reverse-grep audit
None.
Tokens extracted and confirmed in
docs/develop/go/activities/standalone-activities.mdx(or the cited companion docs):client.ExecuteActivity,client.GetActivityHandle,client.ListActivities,client.CountActivities,client.StartActivityOptions,client.GetActivityHandleOptions,client.ListActivitiesOptions,client.CountActivitiesOptions,ActivityHandle,ActivityExecutionInfo,handle.Get,handle.GetID,handle.GetRunID,envconfig.MustLoadDefaultClientOptions,worker.New,worker.InterruptCh,worker.Options,RegisterActivity,Run. All present in cited docs.go.temporal.io/sdk/client,go.temporal.io/sdk/worker,go.temporal.io/sdk/contrib/envconfig. All present.ID,TaskQueue,ScheduleToCloseTimeout,StartToCloseTimeout,ActivityID,RunID,Query,Results,Count,ActivityType,Status. All present.v1.41.0,1.22+,v1.7.0,v1.31.0. All match cited docs.Check 3 findings — regression patterns
None.
--profile,TEMPORAL_TLS_CLIENT_CERT_PATH,TEMPORAL_TLS_CLIENT_KEY_PATH,TEMPORAL_TLS_SERVER_CA_CERT_PATH,tcld service-account,--output text/jsonl, orsaas-api.tmprl.cloud:7233anywhere in the diff.workflow.ExecuteActivityappears only in negative-guidance form ("useworkflow.ExecuteActivity(ctx, …)instead ofclient.ExecuteActivityinside a workflow") — correct usage.client.StartActivityfunction — the page explicitly says "There is no separateStartfunction in the Go SDK" (L79).GetActivityHandlecorrectly requires bothActivityIDandRunID(L117).envconfig.MustLoadDefaultClientOptions(), matching docs L91-96 and L169.Check 4 findings — independent re-verification
None. 10/10 sampled claims match docs after independent re-read. Sample (deterministic stride across the 26 citations, includes one version-prerequisite claim):
client.ExecuteActivity()from a Client vsworkflow.ExecuteActivity()from inside a Workflowclient.ExecuteActivityfrom inside a WorkflowExecuteActivityreturnsActivityHandleimmediately; no separateStartfunctionGetActivityHandlecode form withGetActivityHandleOptionsclient.ListActivitieswith List Filter;Resultsis a range-over-func iterator yielding(ActivityExecutionInfo, error)pairsclient.CountActivitiestakes same arguments asListActivities;Countis total executions, not queued tasksCountActivitiessample withCountActivitiesOptions{Query: …}No documented-default omissions: every required-field claim (
StartActivityOptionsrequiresID/TaskQueue/one-of-two timeouts;GetActivityHandleOptionsrequiresActivityIDandRunID) matches docs exactly. No documented defaults exist on these fields that the authored text suppresses.Check 6 findings — tone and scope audit
None.
Startfunction in the Go SDK" L78-79) are hard-constraint statements paired with the supported alternative.[!NOTE]admonition is at L1-2 and matches Python/TypeScript verbatim.context.*mention is the standard-librarycontextpackage; unambiguous.Execute + handle.Getto the cross-SDK "Execute and wait" pattern is a single sentence and reinforces the cross-SDK concept page — useful, not noise.Check 7 findings — cross-language structure audit
None gating; one acceptable structural delta noted.
Sub-check 1 — page split. Core page at
references/core/standalone-activities.mdexists; new per-language page atreferences/go/standalone-activities.mdexists. ✓Sub-check 2 — core page language-agnostic. Diff to the core file is two adjacent language-list edits adding "Go". No Go-specific code or API leaked into the core page. ✓
Sub-check 3 — identical per-language outline. Heading lists across sibling SDKs and Go:
Python/TS/Java/.NET share this exact outline:
Go's outline diverges in two related places:
### Execute (wait for result)→### Execute (start and get a handle).### Start (do not wait for result)is absent.Both reflect a genuine SDK-specific API delta: the Go SDK exposes a single
client.ExecuteActivityentry point that returns a handle immediately (it does not wait), with no separateStartfunction. The "(wait for result)" naming would be misleading for Go'sExecuteActivity. The authored page documents this delta explicitly at L78-79. Per Check 7 sub-check 3, this is acceptable — not a finding.Sub-check 4 — Overview back-link. The first content line of the Go Overview (L6) is
Standalone Activities are conceptually the same across all SDKs. Read the [cross-SDK concept file](references/core/standalone-activities.md)…. ✓Sub-check 5 — link graph (4/4 edges).
SKILL.mdL74-75: core bullet +references/{your_language}/standalone-activities.mdsub-bullet. ✓references/go/go.mdL255: bullet forreferences/go/standalone-activities.md, mentioning the core page. ✓references/core/standalone-activities.mdL6: "Go" included in the language list pointing per-language. ✓references/go/standalone-activities.mdL6: links back toreferences/core/standalone-activities.md. ✓No
SKILL.mdmodification in the diff (correct — sub-bullet was already present).Sub-check 6 — bonus contextual cross-links. Advisory; consistent with sibling SDKs (no per-language
patterns.mdorai-patterns.mdcross-link to standalone activities in Python/TS either). Not a finding.Statistics
Notes for future authors / reviewers
The page's documented deviation from the canonical heading outline — collapsing
### Execute (wait for result)+### Start (do not wait for result)into a single### Execute (start and get a handle)— is the right call given Go's API and is well-flagged at L78-79. If a future Go SDK release adds a separateStartentry point, the outline should be re-aligned to match the sibling SDKs.End of report.