COM-293: Fix skill issues found during manual review - #6
Merged
Conversation
Provider scope. GCP Cloud Run was described as Pre-release and gated, which reads as "attempt it and see." Replace the scattered prose with a Supported compute providers table (provider, compute service, support, reference directory) and a rule that only a Supported provider is covered. Workflow steps and the routing table now use references/<provider>/... so the file stays provider-neutral. Control-plane CLI. Guidance mixed tcld and the unified CLI's `temporal cloud ...` subcommands, presenting them as interchangeable in a comparison table. They are not. `temporal cloud ...` is still pre-release, so tcld is the supported path for control-plane work today. The two also diverge in ways that fail quietly: the confirmation flags differ (--auto-confirm vs --auto_confirm), and TEMPORAL_CLOUD_API_KEY and TEMPORAL_API_KEY authenticate different planes. Standardize on tcld and reserve `temporal worker deployment ...` for the Namespace frontend, which has no tcld equivalent. Revisit and move the skill to `temporal cloud ...` once it is generally available. Namespace/provider pairing. Generalize from "a GCP Namespace can't run Lambda" to the rule itself: a Serverless Worker runs only on the cloud provider hosting its Namespace, and regions need not match. Step 1's Namespace pick now settles the provider answer, catching a mismatch there rather than at connection time several steps later. Browser fallback. Where the Temporal CLIs cannot authenticate, offer fixing the CLI and working in the Cloud UI as equals rather than as a preference and a fallback; every step exists in the UI. Clarify that "None" in the access matrix means no authenticated CLI, not no access. Restore the unauthenticated-vs-unauthorized classification so a stale credential is not mistaken for a permissions wall. Environment-specific content. Drop a real test-account Namespace used as an example, and the IdP-initiated corporate SSO diagnosis, which generalized one org's root cause into a universal claim. The rule it justified — stop, do not retry, do not install another CLI — survives on its own. Also in this pass: reprint the run checklist at every step boundary rather than narrating progress; announce what a command will do above the command block, since the user sees a collapsed "Ran N shell commands"; deliver the inventory before offering teardown, and do not write a teardown script unasked. Version 0.5.0 -> 0.6.0. Refs: https://temporalio.atlassian.net/browse/COM-293 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
starfleeth
reviewed
Aug 14, 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.
Changes
Provider scope
SKILL.mdgains a Supported compute providers table — provider, compute service, support status, reference directory — replacing release-status prose spread across four files. Provider support is now answerable without reading paragraphsreferences/<provider>/...with the directory layout stated once, so the file stays provider-neutral as providers are addedgcp-region prefix used to classify ineligible Namespaces in step 1descriptionno longer names Cloud RunControl-plane CLI
temporal cloud ...is still pre-release, sotcldis the supported path for control-plane work today — accounts, Namespaces, API keys, users, service accounts. The comparison table presenting the two as alternatives is gonetcldequivalent, and keep usingtemporal worker deployment ...tcld, including teardown. Confirmation flag corrected to--auto_confirm(underscore) — thetemporal cloudspelling exits clean having deleted nothingtcld login --disable-pop-upprints the URL instead of relying on auto-open, and mutating operations need--auto_confirmorAUTO_CONFIRM=truetemporal cloud ...reaches GANamespace and provider pairing
Browser fallback
setup.mdnow names which steps belong to which planeEnvironment-specific content removed
<name>-<suffix>.<account>Run transparency
Skill version
0.5.0→0.6.0.Testing
Manually verified — driven end-to-end against Temporal Cloud on with a Go Worker on Lambda arm64 and a Namespace on AWS us-west-2, after these changes:
🤖 Generated with Claude Code