Skip to content

fix(cli): point help to agent mode and stop printing env secrets in --help - #669

Merged
ajianaz merged 1 commit into
developfrom
fix/help-agent-mode-and-hide-env-secrets
Oct 16, 2026
Merged

ajianaz merged 1 commit into
developfrom
fix/help-agent-mode-and-hide-env-secrets

Conversation

@ajianaz

@ajianaz ajianaz commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

What

  • cora --help, cora review/scan --help, cora mcp --help and cora install --help now say how to review without an LLM key: --no-llm, the agent-as-reviewer MCP tools (cora.review_prepare/review_finalize, cora.scan_prepare/scan_finalize), the /cora-review skill, and claude mcp add cora -- cora mcp for Claude Code. docs/cli-reference.md gets the same rows.
  • --api-key (CORA_API_KEY) and --token (GITHUB_TOKEN) hide their environment value in help. clap prints [env: NAME=value] by default, so cora --help with a key in the environment printed the key (found while checking the help text).

Why

Agent mode shipped in v0.19.0 with docs but no pointer from the CLI itself, and a user who runs --help in CI or a shared terminal must not see their key.

Testing

  • cargo fmt --check, cargo clippy --all-targets --features tree-sitter -- -D warnings, cargo test --features tree-sitter (1258 unit + 20 CLI tests). New CLI tests: mcp --help explains agent-as-reviewer, install --help mentions the skill, review/scan --help point to --no-llm and cora mcp --help, and --help / review --help / upload-sarif --help never print a CORA_API_KEY or GITHUB_TOKEN set in the environment.
  • Real agent run (claude -p, cora MCP with no key) on the final build: prepare, judge, finalize works end to end.

…-help

Signed-off-by: ajianaz <ajianaz@users.noreply.github.com>
@ajianaz
ajianaz merged commit f98a1e0 into develop Oct 16, 2026
14 checks passed
@ajianaz
ajianaz deleted the fix/help-agent-mode-and-hide-env-secrets branch October 16, 2026 01:52
ajianaz added a commit that referenced this pull request Oct 16, 2026
## What
Move the two `[Unreleased]` entries (help points to agent mode; `--help`
no longer prints env secrets, both from #669) into the `[0.19.0]`
section. `[Unreleased]` is empty again. No code, version or link
changes.

## Why
v0.19.0 is not tagged yet, so everything merged since the bump belongs
in it.

## Testing
- Docs-only. `CHANGELOG.md` diff checked: only the two entries moved;
the `[0.19.0]` and `[Unreleased]` headings and the compare links are
unchanged.

Signed-off-by: ajianaz <ajianaz@users.noreply.github.com>
Co-authored-by: ajianaz <ajianaz@users.noreply.github.com>
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