From 265a4fe86948bd8b1bbf6f7c93fc3e1c469f9ac8 Mon Sep 17 00:00:00 2001
From: Yuvraj Chandra <singhyuvraj179@gmail.com>
Date: Wed, 5 Aug 2026 01:21:23 +0530
Subject: [PATCH] fix(cli): help-text and MCP-note cleanup from bug bash
 follow-up
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 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>
---
 src/cli/asset-command.ts    | 8 ++++----
 src/cli/commands/config.ts  | 3 ++-
 src/installers/mcp/index.ts | 1 -
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/cli/asset-command.ts b/src/cli/asset-command.ts
index 8438cca..2c49946 100644
--- a/src/cli/asset-command.ts
+++ b/src/cli/asset-command.ts
@@ -87,8 +87,8 @@ export function makeAssetCommand(type: AssetType, summary: string): Command {
 function extraUsage(type: AssetType): string {
   if (type === "skill") return "|validate|generate";
   if (type === "template") return "|create";
-  if (type === "recipe") return "|run|explain";
-  if (type === "workflow") return "|run|explain";
+  if (type === "recipe") return "|run";
+  if (type === "workflow") return "|run";
   if (type === "eval") return "|run|report";
   return "";
 }
@@ -97,9 +97,9 @@ function helpFor(type: AssetType): string {
   const lines = [`Manage ${type}s from the installable asset registry.`, ""];
   lines.push("Subcommands:");
   lines.push(`  list                 list all ${type}s in the registry (status-aware)`);
-  lines.push(`  install <name>       copy a ${type} into .zenrows/`);
+  lines.push(`  install <name>       copy ${type === "eval" ? "an" : "a"} ${type} into .zenrows/`);
   if (type === "skill") lines.push("  install --all        install every available skill");
-  lines.push(`  explain <name>       print metadata + docs for a ${type}`);
+  lines.push(`  explain <name>       print metadata + docs for ${type === "eval" ? "an" : "a"} ${type}`);
   lines.push(`  update [name]        reinstall (refresh) installed ${type}s`);
   lines.push(`  remove <name>        remove an installed ${type}`);
   if (type === "template") lines.push("  create <name> --output <dir>   instantiate a template into <dir>");
diff --git a/src/cli/commands/config.ts b/src/cli/commands/config.ts
index e9db16d..009d3ea 100644
--- a/src/cli/commands/config.ts
+++ b/src/cli/commands/config.ts
@@ -3,6 +3,7 @@
  *   show
  *   get <key>
  *   set <key> <value>
+ *   reset
  */
 import { defaultConfig, loadConfig, saveConfig } from "../../core/config.ts";
 import { log } from "../../core/logger.ts";
@@ -26,7 +27,7 @@ const SETTABLE: Record<string, (c: ToolkitConfig, v: string) => void> = {
 export const config: Command = {
   name: "config",
   summary: "View or update toolkit configuration (non-secret).",
-  usage: "zenrows config <show|get <key>|set <key> <value>>",
+  usage: "zenrows config <show|get <key>|set <key> <value>|reset>",
   run(argv: string[], ctx: RunContext): number {
     const [sub, key, value] = argv;
     const cfg = loadConfig();
diff --git a/src/installers/mcp/index.ts b/src/installers/mcp/index.ts
index 60461cf..8759053 100644
--- a/src/installers/mcp/index.ts
+++ b/src/installers/mcp/index.ts
@@ -36,7 +36,6 @@ export const MCP_CLIENTS: Record<string, McpClientSpec> = {
     configFile: ".mcp.json (project) or run `claude mcp add`",
     format: "cli",
     autoConfigurable: true,
-    notes: "Confirmed: `claude mcp add zenrows -e ZENROWS_API_KEY=… -- npx -y @zenrows/mcp`.",
   },
   cursor: {
     id: "cursor",