Implement planned topic: 0001-standalone-activities#224
Merged
Conversation
Rename the worker section to "Worker setup & activity registration" in all four SDK files and lead with a single sentence noting the Activity is defined and registered exactly as normal. Drop the .NET "Define the Activity" section so no file repeats how to define an activity, matching the Python structure. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Collaborator
|
Note, this does NOT include Go's standalone activities. Will be done in a follow-up PR. This is just due to some complications with skill-sync-pipeline. |
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 — standalone-activities
Branch under validation:
draft/0001-standalone-activitiesFiles validated (added on this branch, vs.
main):references/python/standalone-activities.mdreferences/typescript/standalone-activities.mdreferences/java/standalone-activities.mdreferences/dotnet/standalone-activities.mdSource of truth:
../documentation/docs/, specifically:docs/encyclopedia/activities/standalone-activity.mdxdocs/develop/python/activities/standalone-activities.mdxdocs/develop/typescript/activities/standalone-activities.mdxdocs/develop/java/activities/standalone-activities.mdxdocs/develop/dotnet/activities/standalone-activities.mdxTopic is not a third-party integration —
references/integrations.mdexists but this is a per-language Temporal feature. Check 5 skipped.Go/no-go
<!-- undocumented: ... -->marker. ≤ 5 → fits MINOR-FIXES criterion.TEMPORAL_TLS_CLIENT_CERT_PATH/TEMPORAL_TLS_CLIENT_KEY_PATHatreferences/java/standalone-activities.md:209andreferences/dotnet/standalone-activities.md:211.## Activity context inside a Standalone Activitysection whose entire body is an HTML<!-- VERIFY: ... -->comment — leftover authoring scaffolding that renders an empty heading.Overall verdict: RE-RUN AUTHORING for the Java and .NET files due to Check 3 hits. Python is essentially clean; TypeScript has MINOR fixes only.
Per the §5 rubric: "RE-RUN AUTHORING — Check 3 has any hit." This applies to Java and .NET; the affected
## Temporal Cloudsection in each should be re-authored.Check 1 findings
100% of cited lines resolve in the docs clone for all four files. The one minor citation-quality note:
references/typescript/standalone-activities.md:10citesdevelop/typescript/activities/standalone-activities.mdx:33-35for "the same function can run as either with no code changes." The cited range introduces the topic but the verbatim no-code-change claim is at L127-128 of the same file. Range is in the same doc, conceptually correct, off-by-section. Not a finding for verdict purposes.Check 2 findings
Tokens used in authored files that are absent from the cited docs subtree:
references/python/standalone-activities.mdworkflow.execute_activity(line 12) — referenced as the Workflow-side alternative inside the hard-guardrail bullet. Not present in either cited file. Real Python SDK API used elsewhere in the docs but lacks a citation or<!-- undocumented -->marker here. Minor.references/typescript/standalone-activities.mdproxyActivities(line 14) — referenced as the Workflow-side alternative inside the hard-guardrail bullet. Not present in either cited file. Minor.@temporalio/*packages must be pinned to the same version (line 19) — uncited heads-up. Not present in either cited file. Real TypeScript-SDK practice but should either be cited from a different docs page (e.g., the TS install/quickstart) or removed. Minor.--inputvalue is JSON-encoded" (line 188) — interpretive gloss; the cited line (L274) shows--input '"World"'but does not describe the encoding. Soft. Trivial.references/java/standalone-activities.mdWorkflow.newActivityStub(line 11) — referenced as the Workflow-side alternative inside the hard-guardrail bullet. Not present in either cited file. Real Java SDK API; same pattern as Python'sworkflow.execute_activity. Minor.references/dotnet/standalone-activities.mdPattern: in three of the four files the "use the Workflow-side API instead" parenthetical inside the hard-guardrail bullet names an API symbol that the cited docs don't mention. This is consistent authoring choice (helpful cross-reference) but lacks the
<!-- undocumented: source = … -->marker the plan requires. Spot-fix: add a citation to the language's basic-activities or workflow-activity doc.Check 3 findings — regression hits (failure-gating)
references/java/standalone-activities.mdTEMPORAL_TLS_CLIENT_CERT_PATHTEMPORAL_TLS_CERTreferences/java/standalone-activities.mdTEMPORAL_TLS_CLIENT_KEY_PATHTEMPORAL_TLS_KEYreferences/dotnet/standalone-activities.mdTEMPORAL_TLS_CLIENT_CERT_PATHTEMPORAL_TLS_CERTreferences/dotnet/standalone-activities.mdTEMPORAL_TLS_CLIENT_KEY_PATHTEMPORAL_TLS_KEYContext: the source docs themselves currently contain these
_PATHvariants (e.g., Javastandalone-activities.mdx:428-429, .NETstandalone-activities.mdx:443-444), so the authoring faithfully reproduced what the docs said. The universal regression rule exists precisely because these doc env-var names are wrong —envconfigreadsTEMPORAL_TLS_CERT/TEMPORAL_TLS_KEY. The skill must override the docs here.Python and TypeScript avoided the hit because their Cloud sections describe the variables abstractly without inlining a code-block list; Java and .NET inline the verbatim variable names. Re-authoring should either (a) drop the inline env-var names and point at a Temporal Cloud connection reference, or (b) substitute the correct env-var names.
Check 4 findings — sampled-claim divergences
40 samples total (10 per file). 38 match cleanly.
The 2 divergences are both in
references/typescript/standalone-activities.md:proxyActivitiesas the recommended Workflow-side alternative; the citation (develop/typescript/activities/standalone-activities.mdx:228-229) only contains "Call this from your application code, not from inside a Workflow Definition." The cited line does not nameproxyActivities. Already captured under Check 2.@temporalio/*packages must be pinned to the same version" is uncited and not in either cited docs file. Already captured under Check 2.Python, Java, and .NET sampled 10/10 each.
Match rate: 38/40 = 95.0%, exactly at the pass threshold. Combined with the Check 3 hits, fix-then-revalidate is preferable to relying on the marginal pass.
Check 6 findings — tone & scope
Pattern-2 finding (in-the-weeds / dead scaffolding) — all four files
Each authored file ends with:
The heading renders to the reader, but the body is empty (HTML comment only). This is leftover authoring scaffolding flagging an open verification question. It either belongs as a real section with content (after the open question is answered against the source docs or SDK source) or should be removed before merge. As written, it's pattern-2 noise (in-the-weeds rationale that doesn't change generated code).
references/python/standalone-activities.md:202-204references/typescript/standalone-activities.md:199-201references/java/standalone-activities.md:216-218references/dotnet/standalone-activities.md:218-220No findings for patterns 1, 3, 4
.NET:88("This is a hard constraint — there is no workaround; you must supply all three"), is the acceptable form — it names the constraint and reinforces it, does not disclose an escape hatch.> [!NOTE]block with the required phrasing at the top.Statistics
Recommended actions
TEMPORAL_TLS_CERT/TEMPORAL_TLS_KEY, or remove the inline variable names and link to the canonical Cloud connection reference. The skill's regression table treats these strings as bugs even when the cited docs reproduce them.proxyActivitiesparenthetical (line 14) and the@temporalio/*pinning heads-up (line 19).workflow.execute_activityparenthetical (line 12).## Activity context inside a Standalone Activitysections. If the answer to the verify question is "docs don't say," that itself should be the section body, not a hidden HTML comment.Items 2–4 alone would fit a single MINOR FIXES commit. Item 1 should go through another authoring pass for the two affected files.