Skip to content

docs(readme): expand Commands into a full command-surface table - #270

Merged
eddietejeda merged 1 commit into
mainfrom
docs/full-command-table
Aug 21, 2026
Merged

docs(readme): expand Commands into a full command-surface table#270
eddietejeda merged 1 commit into
mainfrom
docs/full-command-table

Conversation

@eddietejeda

Copy link
Copy Markdown
Contributor

Replaces the eight-row group summary in the README's ## Commands section with a full surface-area table — every command and subcommand with a one-line description, generated from hotdata --help (v0.26.0). Mirrors the full command reference now on the website's CLI reference page.

Replace the eight-row group summary with every command and subcommand and a
one-line description each, generated from `hotdata --help`.
@eddietejeda
eddietejeda requested a review from a team as a code owner August 21, 2026 00:57
@eddietejeda
eddietejeda requested review from anoop-narang and removed request for a team August 21, 2026 00:57
Comment thread README.md
| `databases context push` | Upload `./<NAME>.md` as named context |
| `databases query` | Execute a SQL query against a database |
| `databases query status` | Check a running query and retrieve results |
| `databases queries list` | List query runs |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

super nit: (not blocking) The table lists positional shortcuts for other groups (query "<sql>", jobs <id>, search "<text>" --index <name>), but a few databases positionals are missing, so "the full command surface" is slightly overstated:

  • databases queries <id> — show one query run (DatabasesCommands::Queries { id: Option<String>, .. }, src/commands/databases.rs:230); this is a distinct capability not covered by queries list.
  • databases results <result-id> (src/commands/databases.rs:247) and databases <name-or-id> (src/cli.rs:31), both shorthands for the show/get rows already listed.

Adding the queries <id> row is the one that adds real information.

Comment thread README.md
## Commands

Run `hotdata <command> --help` for full flags on any command.
The full command surface. The top level has eight groups — `auth`, `workspaces`, `databases`, `query`, `jobs`, `ingest`, `search`, and `manage`. Run `hotdata <command> --help` for full flags on any command.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

super nit: (not blocking) Prose elsewhere in the README hard-wraps at ~80 columns (see lines 15-17, 110-112, 84-87); this paragraph is one ~200-char line. Wrapping it keeps the diff style consistent.

Suggested change
The full command surface. The top level has eight groups — `auth`, `workspaces`, `databases`, `query`, `jobs`, `ingest`, `search`, and `manage`. Run `hotdata <command> --help` for full flags on any command.
The full command surface. The top level has eight groups — `auth`,
`workspaces`, `databases`, `query`, `jobs`, `ingest`, `search`, and `manage`.
Run `hotdata <command> --help` for full flags on any command.

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Verified every row against the clap command tree in src/cli.rs and src/commands/*: names, renames (databases use/remove, tables remove, results get, ingest pause/logs/remove, sources test/add/update-config/remove, embeddings show/add/remove), and the eight top-level groups all match. Two super nits inline, neither blocking.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@eddietejeda
eddietejeda merged commit d4dc837 into main Aug 21, 2026
14 checks passed
@eddietejeda
eddietejeda deleted the docs/full-command-table branch August 21, 2026 01: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.

1 participant