Skip to content

Add pretty JSON output flag#11

Open
MyTH-zyxeon wants to merge 1 commit into
kcolbchain:mainfrom
bonginkan:codex/pretty-json-output-3
Open

Add pretty JSON output flag#11
MyTH-zyxeon wants to merge 1 commit into
kcolbchain:mainfrom
bonginkan:codex/pretty-json-output-3

Conversation

@MyTH-zyxeon
Copy link
Copy Markdown

@MyTH-zyxeon MyTH-zyxeon commented May 20, 2026

Summary

  • adds a JsonPretty output mode using serde_json pretty serialization
  • adds a global --pretty flag that conflicts with --human
  • documents compact JSON, pretty JSON, and human output examples

Validation

  • cargo fmt -- --check
  • cargo test: 11 passed
  • cargo run --quiet -- info --pretty: produced indented JSON
  • cargo run --quiet -- info --human --pretty: exits with clap conflict status 2

Closes #3

@abhicris
Copy link
Copy Markdown
Contributor

Welcome to kcolbchain, @MyTH-zyxeon — glad you're here. 🌱

Here's what happens from this PR:

  1. Our automated review looks for obvious issues (tests, secrets, size) within a couple of hours.
  2. If it's clean and CI passes, we merge without back-and-forth.
  3. If we need changes, we'll leave a specific comment — not a generic nit. Push another commit and we re-review.

While you wait:

  • Run the repo's tests locally (see the repo README.md).
  • Keep the PR scoped to one concern — bigger PRs land slower.
  • Don't commit tokens or .env contents.

What happens after your first merge

Thanks for writing the code. We're building this to last.

@abhicris
Copy link
Copy Markdown
Contributor

🤖 Audit verdict: safe

Straightforward feature addition using standard library functions with no malicious code, credential leakage, or supply-chain risk.

Audited by the kcolbchain PR pipeline. See pipeline docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[good first issue] Add --pretty flag for indented JSON output (distinct from --human)

2 participants