fix: standardize cli io

[giuseppegrieco]

Summary by cubic

Standardizes CLI input/output and UX across commands, focusing on simple text output and consistent interactive flows. Adds a top-level describe command and streamlines ports translation. Addresses Linear 34.

• New Features

  • Added top-level "describe" to show product info, then ports and connections.
  • Ports/connections describe and remove now use consistent interactive selectors and themed output.
  • Ports translate supports multiple ports and --all; prompts for ports/format/output when missing; prints written file paths.

• Migration

  • Removed -o/--output (json|yaml) from ports|connections list/describe; output is text only.
  • Ports translate flags:
    • --name accepts comma-separated names; --all added.
    • --output is the output directory (replaces --output-dir); per-port single-file --output removed.
    • -f shorthand for --format removed.
  • Ports list shows connections as name:location and drops the schema type column; ports describe shows a connections table.

^{Written for commit d02c9e5. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 17 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="internal/prompts/ports_describe.go">

<violation number="1" location="internal/prompts/ports_describe.go:28">
P2: Truncating the description by byte index can split multibyte UTF-8 characters, producing invalid or corrupted labels. Use rune-based slicing instead.</violation>
</file>

<file name="internal/commands/connections_describe.go">

<violation number="1" location="internal/commands/connections_describe.go:96">
P2: String truncation using byte indexing can corrupt multi-byte UTF-8 characters. Use `utf8.RuneCountInString()` for length check and `[]rune` conversion for safe slicing.</violation>
</file>

<file name="internal/prompts/ports_translate.go">

<violation number="1" location="internal/prompts/ports_translate.go:30">
P2: Missing empty check for `ports` map. The code validates that `formats` is not empty before creating the select field (line 55-57), but doesn't apply the same validation for `ports` before creating the multi-select. If `ports` is empty, users will see an empty multi-select with no options.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[gummiorri]

LGTM