Skip to content

plan schema: honor -o flag, default to pretty JSON#329

Merged
daniel-de-vera merged 1 commit into
mainfrom
plan-schema-output-format
May 6, 2026
Merged

plan schema: honor -o flag, default to pretty JSON#329
daniel-de-vera merged 1 commit into
mainfrom
plan-schema-output-format

Conversation

@daniel-de-vera
Copy link
Copy Markdown
Contributor

Summary

  • plan schema previously ignored the global -o/--output flag and always emitted compact JSON.
  • Switch to the standard OutputFormatDefault/JSON/YAML dispatch used elsewhere in the CLI (e.g. secret get).
  • Default output is now pretty JSON (via print.RawJSON), making the schema human-readable out of the box. -o yaml is also supported.

Test plan

  • signadot plan schema → pretty JSON
  • signadot plan schema -o json → pretty JSON
  • signadot plan schema -o yaml → YAML
  • signadot plan schema -o xml → rejected by the flag validator

🤖 Generated with Claude Code

Previously the hidden `plan schema` command always emitted compact JSON,
ignoring the global `-o` flag. Switch to the standard
default/json/yaml dispatch used elsewhere in the CLI, with pretty JSON
as the default so the schema is human-readable out of the box.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Member

@scott-cotton scott-cotton left a comment

Choose a reason for hiding this comment

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

LGTM

@daniel-de-vera daniel-de-vera merged commit 6841b07 into main May 6, 2026
@daniel-de-vera daniel-de-vera deleted the plan-schema-output-format branch May 6, 2026 15:00
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.

2 participants