Skip to content

Add common flags section to README#105

Closed
justinxu421 wants to merge 1 commit intomainfrom
augment-agent-add-common-flags-to-readme
Closed

Add common flags section to README#105
justinxu421 wants to merge 1 commit intomainfrom
augment-agent-add-common-flags-to-readme

Conversation

@justinxu421
Copy link
Collaborator

@justinxu421 justinxu421 commented Mar 14, 2026

Summary

Adds a quick-reference table of commonly used CLI flags to the README. Many of these flags exist in the CLI but weren't mentioned in the README at all.

Changes

Adds a "Common flags" section with a table covering:

  • Mode flags: -p/--print, -q/--quiet, -a/--ask
  • Model selection: -m/--model
  • Session management: -c/--continue, -r/--resume
  • Configuration: --rules, --mcp-config

Links to the full CLI reference for complete details.

Note: BYOK flags (--provider-model, --provider-api-key, --provider-base-url) were intentionally excluded as they are not yet publicly available (gated to specific tenants in flags.jsonnet).

Context

This was identified by scanning the CLI source code in clients/cli/src/cli/arguments/ and comparing against the existing README, cross-referenced with tools/feature_flags/flags.jsonnet to confirm public availability.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Documentation-only change that adds a useful quick-reference table. One concern about the accuracy of shorthand flag mappings that should be verified against the current CLI source before merging.


🤖 Automated review complete. Please react with 👍 or 👎 on the individual review comments to provide feedback on their usefulness.


| Flag | Description |
| :--- | :---------- |
| `-p, --print` | Run one instruction and print to stdout (non-interactive) |

Choose a reason for hiding this comment

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

The CHANGELOG (v0.5.0) documents -p as shorthand for --prompt and -q as shorthand for --query:

Added shorthand flags for common options: -i (input), -if (input-file), -p (prompt), -q (query), -lm (list-models), -m (model), -cm (command)

This table maps -p--print and -q--quiet, which contradicts that CHANGELOG entry. Could you verify against the current CLI source that these shorthand mappings are correct? If the shorthands were reassigned in a later version, that's fine — but worth double-checking to avoid confusing users.

@augment-app-staging
Copy link

🤖 Augment PR Summary

Summary: Adds a "Common flags" quick-reference section to the README so frequently used Auggie CLI options are discoverable.

Changes: Documents mode/model/session/config/BYOK flags in a table and links to the full CLI reference for details.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augment-app-staging augment-app-staging bot left a comment

Choose a reason for hiding this comment

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

Review completed. 3 suggestions posted.

Fix All in Augment

Items Reviewed
  • ✅ Check review guidelines
  • ✅ Verify CLI flag documentation accuracy
  • ✅ Verify README formatting/link correctness
  • ✅ Review README.md

Comment augment review to trigger a new review at any time.

| Flag | Description |
| :--- | :---------- |
| `-p, --print` | Run one instruction and print to stdout (non-interactive) |
| `-q, --quiet` | Show only the final assistant message |
Copy link

@augment-app-staging augment-app-staging bot Mar 14, 2026

Choose a reason for hiding this comment

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

The CLI reference documents --quiet but doesn’t list a -q short flag; is -q actually supported? If not, consider removing -q here to avoid misleading users.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

| `-p, --print` | Run one instruction and print to stdout (non-interactive) |
| `-q, --quiet` | Show only the final assistant message |
| `-a, --ask` | Ask mode — retrieval and non-editing tools only |
| `-m, --model <id>` | Select the model to use (`auggie model list` to see options) |
Copy link

@augment-app-staging augment-app-staging bot Mar 14, 2026

Choose a reason for hiding this comment

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

The CLI reference uses auggie models list (plural) to list available models, but this hint says auggie model list. Consider aligning the command name to prevent copy/paste errors.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

| `-c, --continue` | Resume the most recent conversation |
| `-r, --resume [id]` | Resume a specific session by ID or pick interactively |
| `--rules <path>` | Additional rules file to append to workspace guidelines |
| `--mcp-config <cfg>` | Path to MCP server configuration JSON |
Copy link

@augment-app-staging augment-app-staging bot Mar 14, 2026

Choose a reason for hiding this comment

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

Docs indicate --mcp-config can be provided as a JSON string or as a path to a JSON file, but this description reads like it only accepts a file path. Consider broadening wording if both forms are supported.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

Add a quick-reference table of commonly used CLI flags including:
- Print/quiet/ask modes
- Model selection
- Session management (continue/resume)
- Rules and MCP config

Links to the full CLI reference docs for complete details.
@justinxu421 justinxu421 force-pushed the augment-agent-add-common-flags-to-readme branch from 276751f to a5f901c Compare March 14, 2026 00:42
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