Skip to content

feat: resolve definition names in --definition flag#53

Merged
omattsson merged 1 commit intomainfrom
feat/48-definition-flag-resolve-names
Apr 24, 2026
Merged

feat: resolve definition names in --definition flag#53
omattsson merged 1 commit intomainfrom
feat/48-definition-flag-resolve-names

Conversation

@omattsson
Copy link
Copy Markdown
Owner

Summary

  • --definition flag on stack create and stack list now accepts a definition name (e.g. klaravik-dev) in addition to a UUID
  • Adds resolveDefinitionID resolver following the same pattern as resolveStackID
  • Updates help text and examples for both commands

Closes #48

Depends on: omattsson/k8s-stack-manager PR (adds ?name= query param to ListDefinitions API)

Test plan

  • UUID passthrough — no API call made
  • Numeric ID passthrough
  • Single name match — resolves to correct ID
  • No match — descriptive error
  • Multiple matches — lists all with IDs and owners
  • Name mismatch (server returns wrong name) — error
  • API error — wrapped error message
  • Empty / whitespace-only input — rejected
  • Whitespace-padded name — trimmed before API call
  • All existing tests pass (go test ./...)

🤖 Generated with Claude Code

The --definition flag on `stack create` and `stack list` now accepts
a definition name (e.g. klaravik-dev) in addition to a UUID.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@omattsson omattsson merged commit 51f877a into main Apr 24, 2026
6 checks passed
@omattsson omattsson deleted the feat/48-definition-flag-resolve-names branch April 24, 2026 12:53
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.

definition flag should resolve names, not just IDs

1 participant