Skip to content

Absorb CLI skill: workflow commands, dev server, CLI gotchas#231

Merged
donald-pinckney merged 15 commits into
mainfrom
cli-skill-consolidation
May 29, 2026
Merged

Absorb CLI skill: workflow commands, dev server, CLI gotchas#231
donald-pinckney merged 15 commits into
mainfrom
cli-skill-consolidation

Conversation

@starfleeth

Copy link
Copy Markdown
Contributor

Summary

Migrates developer-facing content from skill-temporal-cli into the developer skill as part of retiring the CLI skill.

New file (1):

  • references/core/cli-workflow-commands.md (266 lines) — CLI reference for workflow start, execute, signal, query, update, signal-with-start, result, metadata

Enriched files (4):

  • dev-management.md (26→114 lines) — full temporal server start-dev flag table, persistence guidance, dev-to-prod recipe
  • gotchas.md — 5 CLI gotchas (in-memory default, not-for-prod, update is a group, wait-for-stage, reapply-type)
  • install_cli.md (25→59 lines) — CDN tarball, Docker, tcld installation
  • SKILL.md — absorbed CLI trigger phrases, new reference entry, ops cross-reference

Verification:

  • Grounding: all flags/enums verified against docs/cli/workflow.mdx and docs/cli/server.mdx
  • No duplication with existing patterns.md, interactive-workflows.md, or troubleshooting.md

Test plan

  • Read cli-workflow-commands.md for completeness against temporal workflow --help
  • Verify dev-management.md flags match temporal server start-dev --help
  • Confirm new gotchas don't duplicate existing entries in gotchas.md
  • Spot-check 5 inline citations against ../documentation/ source files

🤖 Generated with Claude Code

starfleeth and others added 3 commits May 20, 2026 17:35
Migrate developer-facing content from skill-temporal-cli as part of the
CLI skill consolidation. Adds cli-workflow-commands.md (start, execute,
signal, query, update reference), expands dev-management.md with full
dev server flags and dev-to-prod recipe, appends CLI gotchas, and
enriches install_cli.md. Updates SKILL.md triggers and routing.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ons, dedup gotchas

Standardize cli-workflow-commands.md flag tables to use Required column
throughout. Annotate --log-level and --log-format as global flags in
dev-management.md. Collapse redundant dev server gotchas into a
cross-reference to dev-management.md.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Workers can target a local dev server, self-hosted cluster, or
Temporal Cloud. The previous wording implied a local server was
mandatory.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

@tlalfano tlalfano left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! small comments

Comment thread references/core/install_cli.md
Comment thread SKILL.md Outdated
Comment thread references/core/cli-workflow-commands.md
Comment thread references/core/dev-management.md Outdated
Comment thread references/core/dev-management.md Outdated
Comment thread references/core/cli-workflow-commands.md
Comment thread references/core/cli-workflow-commands.md Outdated
Comment thread references/core/dev-management.md Outdated
@donald-pinckney donald-pinckney marked this pull request as ready for review May 29, 2026 18:25
@donald-pinckney donald-pinckney requested a review from a team as a code owner May 29, 2026 18:25
Comment thread references/core/dev-management.md Outdated
### 4. Smoke-test prod

```bash
temporal workflow list --env prod --limit 1 --output json

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And this would be:

Suggested change
temporal workflow list --env prod --limit 1 --output json
temporal workflow list --profile prod --limit 1 --output json


### `temporal server start-dev` flags

| Flag | Default | Purpose |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we should have all these flags in here for start-dev. But OK to merge as-is and back-off later.

Comment thread references/core/gotchas.md Outdated
Comment thread SKILL.md
- **`references/core/error-reference.md`** - Common error types, workflow status reference
- **`references/core/interactive-workflows.md`** - Testing signals, updates, queries
- **`references/core/dev-management.md`** - Dev cycle & management of server and workers
- **`references/core/cli-workflow-commands.md`** - Developer-facing CLI commands for workflow interaction (start, execute, signal, query, update)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- **`references/core/cli-workflow-commands.md`** - Developer-facing CLI commands for workflow interaction (start, execute, signal, query, update)
- **`references/core/cli-workflow-commands.md`** - Key `temporal` CLI commands to use during development for workflow interaction (start, execute, signal, query, update)

@donald-pinckney donald-pinckney merged commit d52c54a into main May 29, 2026
3 checks passed
@donald-pinckney donald-pinckney deleted the cli-skill-consolidation branch May 29, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants