Skip to content

Conversation

@JoshuaBatty
Copy link
Collaborator

@JoshuaBatty JoshuaBatty commented Dec 17, 2025

Documents the --json-output flag for saving structured CLI output to disk.

Changes

  • Documents default output location: build/json-outputs/<command>.json
  • Documents custom path option: --json-output=path/to/file.json
  • Supported commands: deploy, upgrade, run, execute, test, query, synthesize

Related implementation: ProvableHQ/leo#29051

@JoshuaBatty JoshuaBatty marked this pull request as draft December 17, 2025 02:14
Document the new --json flag that outputs results in machine-readable JSON format:
- Add --json to Universal Flags in CLI overview
- Add JSON Output sections to deploy and run command docs with examples
mohammadfawaz
mohammadfawaz previously approved these changes Dec 22, 2025
Copy link
Collaborator

@mohammadfawaz mohammadfawaz left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks.

Copy link
Collaborator

@AleoAlexander AleoAlexander left a comment

Choose a reason for hiding this comment

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

One minor change, but otherwise LGTM!

Co-authored-by: Alexander Kim <alexander@aleo.org>
Signed-off-by: Joshua Batty <joshpbatty@gmail.com>
@JoshuaBatty
Copy link
Collaborator Author

I'm making some changes to the linked PR that change the definition. Will put this in draft until we have finalised the CLI implementation.

- Rename --json to --json-output
- Document default output location (build/json-outputs/<command>.json)
- Document custom path option (--json-output=path)
- Remove stdout piping examples (no longer needed)
@JoshuaBatty JoshuaBatty changed the title Add --json flag documentation for CLI commands Add --json-output flag documentation for CLI commands Jan 8, 2026
@JoshuaBatty JoshuaBatty marked this pull request as ready for review January 8, 2026 22:36
@JoshuaBatty
Copy link
Collaborator Author

@AleoAlexander This is ready for review again now. The CLI changes have been merged in the linked PR. Thanks.

Copy link
Collaborator

@mohammadfawaz mohammadfawaz left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks.

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.

4 participants