-
Notifications
You must be signed in to change notification settings - Fork 3
docs: add --output json flag documentation for CLI commands #161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Document JSON output support for CLI commands: - Add JSON Output section to main CLI reference page - Document --output json flag for deploy, invoke, app commands - Document --output json flag for browsers, browser-pools, profiles - Document --output json flag for extensions, proxies - Add browser-pools and profiles CLI sections Related to kernel/cli#67
reference/cli/apps.mdx
Outdated
| | `--force` | Overwrite an existing version with the same label. | | ||
| | `--env <KEY=VALUE>`, `-e` | Set environment variables (repeatable). | | ||
| | `--env-file <file>` | Load environment variables from a file (repeatable). | | ||
| | `--output json`, `-o` | Output JSONL (one JSON object per line for each deployment event). | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor: in these flag tables, -o likely still needs the json argument (consistent with the examples in reference/cli.mdx). Might be clearer to document as -o json (same for the other --output json rows in this PR).
| | `--output json`, `-o` | Output JSONL (one JSON object per line for each deployment event). | | |
| | `--output json`, `-o json` | Output JSONL (one JSON object per line for each deployment event). | |
| |------|-------------| | ||
| | `--yes`, `-y` | Skip confirmation. | | ||
|
|
||
| ## Browser pools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could cross link to /browsers/pools page here
| ### `kernel browser-pools flush <id-or-name>` | ||
| Destroy all idle browsers in the pool. | ||
|
|
||
| ## Profiles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could cross link to /browsers/profiles page here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeet
I do think tembo is right though ^^
- Add cross-links to /browsers/pools and /browsers/profiles pages - Fix flag documentation: use `-o json` instead of just `-o` for consistency
Summary
-o jsonusage--output jsonflag for all supported commands in apps.mdx and browsers.mdxThis documents the JSON output feature being added in kernel/cli#67.
Documentation added for:
Apps (reference/cli/apps.mdx):
kernel deploy- JSONL streaming outputkernel deploy history- JSON arraykernel invoke- JSONL streaming outputkernel app list- JSON arraykernel app history- JSON arrayBrowsers (reference/cli/browsers.mdx):
kernel browsers list/create/view/get- JSON outputkernel browsers replays list/start- JSON outputkernel browsers process exec/spawn- JSON outputkernel browsers fs file-info/list-files- JSON outputkernel extensions list/upload- JSON outputkernel proxies list/get/create- JSON outputkernel browser-pools list/create/get/update/acquire- JSON outputkernel profiles list/get/create- JSON outputTest plan
Note
Introduces machine-readable output docs and expands CLI coverage.
reference/cli.mdxwith examples for-o jsonand JSONL streaming--output jsonfor key commands in Apps (deploy/invokeJSONL,deploy history,app list,app historyJSON arrays)--output jsonforbrowsers list/create/view/get, replays (list/start), process (exec/spawn), and filesystem (file-info/list-files); also for extensions and proxiesWritten by Cursor Bugbot for commit dd12f8f. This will update automatically on new commits. Configure here.