Skip to content

Help-text and MCP-note cleanup (bug bash follow-up) - #13

Merged
AnderRV merged 1 commit into
ZenRows:fix/audit-findingsfrom
Yuvrajchandra:fix/help-text-and-mcp-note
Aug 4, 2026
Merged

Help-text and MCP-note cleanup (bug bash follow-up)#13
AnderRV merged 1 commit into
ZenRows:fix/audit-findingsfrom
Yuvrajchandra:fix/help-text-and-mcp-note

Conversation

@Yuvrajchandra

Copy link
Copy Markdown
Contributor

Stacked on #12 — small, self-contained help-text/DX fixes from the ongoing bug bash that weren't part of the audit-findings pass.

What

  • workflow/recipe --help showed explain twice in the usage string (<list|install|explain|remove|update|run|explain>) — extraUsage() was appending |run|explain on top of the shared base usage, which already includes explain. Now appends just |run.
  • eval --help said "copy a eval" / "print metadata + docs for a eval" — fixed to "an eval" in both spots.
  • config --help never listed the working reset subcommand, even though zenrows config reset has always worked. Usage string now reads <show|get <key>|set <key> <value>|reset>, matching how policy --help documents its own reset.
  • zenrows mcp config --client claude-code printed a leftover internal note (Confirmed: \claude mcp add zenrows -e ZENROWS_API_KEY=… -- npx -y @zenrows/mcp`.`) that just repeated the command already shown two lines below it, unlike the genuinely useful vscode/codex notes. Removed.

Testing

  • npm run build && npm run typecheck && npm test — clean, 179/179 passing (no test asserted the old strings).
  • Manually verified each fix's --help/output before and after.

🤖 Generated with Claude Code

- Drop the duplicate "explain" in `workflow`/`recipe --help` usage strings
  (extraUsage() appended "|run|explain" even though "explain" is already
  part of the shared base usage string).
- Fix "a eval" -> "an eval" in both the `install` and `explain` help lines
  for the eval asset type.
- Add the working but undocumented `reset` subcommand to `config --help`'s
  usage string, matching how `policy --help` already documents its own reset.
- Remove the leftover internal dev note from `mcp config --client claude-code`
  ("Confirmed: `claude mcp add ...`") — it repeated the command already
  printed below it and read as an internal QA annotation, not user-facing
  guidance, unlike the genuinely useful notes on vscode/codex.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@Yuvrajchandra
Yuvrajchandra requested a review from AnderRV August 4, 2026 19:52
@AnderRV
AnderRV merged commit ce13881 into ZenRows:fix/audit-findings Aug 4, 2026
2 checks passed
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.

2 participants