Implement planned topic: 0034-nexus#222
Open
skill-temporal-developer-updater[bot] wants to merge 8 commits into
Open
Implement planned topic: 0034-nexus#222skill-temporal-developer-updater[bot] wants to merge 8 commits into
skill-temporal-developer-updater[bot] wants to merge 8 commits into
Conversation
Cross-language Nexus concept reference grounded in docs/encyclopedia/nexus/* and docs/cloud/limits.mdx. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Python SDK Nexus reference grounded in
docs/develop/python/nexus/{feature-guide,quickstart,index}.mdx.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
TypeScript SDK Nexus reference (Public Preview) grounded in
docs/develop/typescript/nexus/{feature-guide,quickstart,index}.mdx.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Go SDK Nexus reference grounded in
docs/develop/go/nexus/{feature-guide,quickstart,index}.mdx.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Java SDK Nexus reference grounded in
docs/develop/java/nexus/{feature-guide,quickstart,index}.mdx.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
.NET SDK Nexus reference (Public Preview) grounded in
docs/develop/dotnet/nexus/{feature-guide,quickstart,index}.mdx.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
One-line pointer per file to the new references/core/nexus.md and
references/{lang}/nexus.md files.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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 —
nexusBranch:
draft/0034-nexus(validated againstmain)Scope: 6 authored reference files + entry-point pointers in
SKILL.mdandreferences/{lang}/{lang}.md.Go/no-go
Overall verdict: GO. All four gating checks pass their thresholds. One minor Check 4 finding (single sample) and two minor Check 1 citation overreaches in
references/java/nexus.mdare documented below as optional spot fixes — they do not require re-authoring.Check 1 findings — citation audit
references/core/nexus.mdNone. 126 citations resolved cleanly.
references/python/nexus.mdNone. 58 citations resolved cleanly.
references/typescript/nexus.mdNone. 38 citations resolved cleanly.
references/go/nexus.mdNone. 45 citations resolved cleanly.
references/java/nexus.mdTwo minor citation overreaches:
io.temporal:temporal-sdkGradle/Maven coordinate as the rest of the Java SDK" is cited todocs/develop/java/nexus/feature-guide.mdx:203anddocs/develop/java/nexus/quickstart.mdx:58-59. Those cited ranges support theio.nexusrpc.*andio.temporal.nexus.*package names but do not mention the Gradle/Maven coordinate. The coordinate is documented indocs/develop/java/set-up.mdx. Fix: swap citation todevelop/java/set-up.mdxfor the coordinate clause.\"SampleNexusService\")" is cited tofeature-guide.mdx:537-539. The cited lines show the literal"SampleNexusService"key value but do not explicitly state the rule "must be the Service class simple name." This is an authorial generalization from one example. Fix: soften wording to "matches the Service class name as a string" or annotate with<!-- VERIFY: ... -->.references/dotnet/nexus.mdNone. 54 citations resolved cleanly.
Aggregate Check 1: 357 / 359 = 99.4 %, above the 98 % threshold.
Check 2 findings — reverse-grep audit
All factual tokens (SDK identifiers, decorator/annotation/attribute names, package/module paths, method names, enum values, exception classes, error categories, event names, metric names, Pending-Operation/Callback fields, timeout codes, CLI commands, version numbers, numeric facts) resolve in the cited docs subtrees:
docs/encyclopedia/nexus/*.mdxdocs/develop/{python,typescript,go,java,dotnet}/nexus/*.mdxdocs/evaluate/temporal-cloud/limits.mdxItems not present in the Nexus docs are correctly wrapped with
<!-- VERIFY: ... -->markers in the authored files:BAD_REQUEST,UNAUTHENTICATED,UNAUTHORIZED,NOT_FOUND,NOT_IMPLEMENTED,RESOURCE_EXHAUSTED,UNAVAILABLE) and the exhaustiveendpointsubcommand set (core line 166).schedule_to_start_timeout/start_to_close_timeoutnot shown in Python Nexus docs.scheduleToCloseTimeoutdocumented for Nexus; noCancellationTypeenum surfaced.setScheduleToStartTimeout/setStartToCloseTimeoutnot surfaced in cited files.io.temporal:temporal-sdkcoordinate is not indevelop/java/nexus/— already flagged under Check 1.NewAsyncOperationis correctly absent from Go docs and correctly framed as a Common-Mistake antipattern inreferences/go/nexus.mdline 290.Aggregate Check 2: 0 unexplained misses.
Check 3 findings — regression on known bugs
Universal regression sweep (
--profile,TEMPORAL_TLS_CLIENT_*env vars,tcld service-account,--output text|jsonl,saas-api.tmprl.cloud:7233): no hits across the 6 reference files or the entry-point edits.Topic-specific regression sweep: invented Go
NewAsyncOperationdoes not appear except as a Common-Mistake antipattern; JavaOperationHandler.Synccapitalization is correctly lowercasesync(matching docs); .NETOperationHandler.Syncis correctly capital-S. No hits.Aggregate Check 3: 0 hits.
Check 4 findings — independent re-verification (60 sampled claims)
references/core/nexus.md(10 sampled, 9 match)Divergence at line 51 — "Terminal events on the caller side are one of:
NexusOperationStarted,NexusOperationCompleted,NexusOperationFailed,NexusOperationCanceled, orNexusOperationTimedOut." cited todocs/encyclopedia/nexus/nexus-operations.mdx:169.NexusOperationStartedis intermediate (it marks the start of an asynchronous Operation; in the same file's async lifecycle at lines 119-128, Started precedes Completed/Failed). The resolution/terminal events are Completed, Failed, Canceled, and TimedOut.NexusOperationStartedas a terminal event might short-circuit waits or misclassify the Operation as resolved when in fact the handler Workflow is still running.references/python/nexus.md— 10 / 10 match.references/typescript/nexus.md— 10 / 10 match.references/go/nexus.md— 10 / 10 match.references/java/nexus.md— 10 / 10 match.references/dotnet/nexus.md— 10 / 10 match.Aggregate Check 4: 59 / 60 = 98.3 %, above the 95 % threshold.
Check 5 findings — integration-layout audit
Skipped. Nexus is a Temporal-native feature (encyclopedia topic), not a third-party integration.
references/integrations.mdwas not modified on this branch. The diff places per-language files at the canonicalreferences/{language}/nexus.mdpaths and adds short pointer bullets to each language entry-point — this is the standard layout for core topics likedeterminism.md,patterns.md,versioning.md.Check 6 findings — tone and scope audit
Workaround disclosures (pattern 1, must be zero)
None. Grep across all six files for the workaround-disclosure phrases (
if you really need,as a last resort,if you must,undocumented way,technically you can,circumvent,bypass,hack,workaround) returned zero hits.Negative bullets (e.g. "do not use a
Clientfrom inside an async handler — useawait ctx.start_workflow(...)") consistently follow the "Don't do X, do Y instead" form and name the supported alternative. The handful of "Don't do X" statements (e.g. "Only asynchronous Operations can be canceled") are hard-constraint statements, not workarounds.Public Preview admonition (pattern 4)
The docs mark Nexus support as Public Preview for TypeScript and .NET; both authored files carry the standard
[!NOTE]admonition (verbatim wording) —references/typescript/nexus.md:5-8andreferences/dotnet/nexus.md:5-8. Python, Go, Java, and the cross-language core file omit the admonition — correct, since the docs mark these as Generally Available (develop/python/nexus/feature-guide.mdx:20-24,develop/go/nexus/feature-guide.mdx:21-25,develop/java/nexus/feature-guide.mdx:20-25,encyclopedia/nexus/nexus.mdx:21).In-the-weeds rationale (pattern 2) and multi-sentence bullets (pattern 3)
Spot-check found no notable instances. Bullets are tight; rationale is limited to constraint explanations (e.g. why
ctx.requestIdis preferred over a fresh UUID — "stable across retries"), which is operational information an agent needs.Aggregate Check 6: 0 findings.
Statistics
core/nexus.md,python/nexus.md,typescript/nexus.md,go/nexus.md,java/nexus.md,dotnet/nexus.md)SKILL.md+ 5 language entry-pointsjava/nexus.md)Recommended follow-up (non-blocking spot fixes)
These do not block merge; they are optional cleanups for a follow-up commit.
references/core/nexus.md:51— rewrite the "Terminal events" sentence to distinguish the intermediateNexusOperationStartedevent from the resolution events (Completed / Failed / Canceled / TimedOut), or remove the word "Terminal" and re-citenexus-operations.mdx:169as "events recorded in caller Workflow History."references/java/nexus.md:11— split or re-cite the Gradle/Maven coordinate clause so theio.temporal:temporal-sdkclaim points atdevelop/java/set-up.mdxrather than the Nexus feature-guide.references/java/nexus.md:239— soften the "Service class simple name" wording to match what the docs actually show (a literal interface-name string in the example), or wrap with<!-- VERIFY: ... -->.