argsbarg 3.3.13 → 3.3.14

package/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [3.3.14] - 2026-06-21
+
+### Changed
+
+- **Generated notes** — deduplicated agent, docs, MCP, and completion help; each topic owns its guidance in one place.
+
 ## [3.3.13] - 2026-06-21
 
 ### Changed
@@ -329,7 +335,8 @@ const cli = { ... } satisfies CliProgram;  // or : CliProgram
 - Migrate schemas: rename every `children` property to **`commands`**; move positional definitions to **`CliPositional`** objects on `positionals` and strip `positional` / `argMin` / `argMax` from flag definitions under `options` (flags only carry `name`, `description`, `kind`, and optional `shortName`).
 - Imports: use `CliPositional` where needed; replace `CliOptionDef` with `CliOption` or `CliPositional` as appropriate.
 
-[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v3.3.13...HEAD
+[Unreleased]: https://github.com/bdombro/bun-argsbarg/compare/v3.3.14...HEAD
+[3.3.14]: https://github.com/bdombro/bun-argsbarg/releases/tag/v3.3.14
 [3.3.13]: https://github.com/bdombro/bun-argsbarg/releases/tag/v3.3.13
 [3.3.12]: https://github.com/bdombro/bun-argsbarg/releases/tag/v3.3.12
 [3.3.11]: https://github.com/bdombro/bun-argsbarg/releases/tag/v3.3.11

package/docs/bundled-docs.md

@@ -35,7 +35,7 @@ myapp docs readme --save   # write ./docs/readme.md
 myapp docs schema --save   # write ./docs/schema.json
 ```
 
-When `docs` is enabled, top-level `myapp --help` includes a Notes line: `Agents: run \`myapp docs skill\` to learn how to use this app`.
+When `docs` is enabled, top-level `myapp --help` points agents at `myapp docs skill`. The `docs skill` subcommand description recommends `install --skill` for a persisted bundle.
 
 ## Configuration
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "argsbarg",
-  "version": "3.3.13",
+  "version": "3.3.14",
   "type": "module",
   "engines": {
     "bun": ">=1.3"

package/src/builtins/builtins.test.ts

@@ -57,9 +57,14 @@ describe("builtins help copy", () => {
   });
 
   test("mcp builtin description is user-facing", () => {
-    const mcp = cliBuiltinMcpCommand();
+    const withDocs: CliProgram = {
+      ...fixture,
+      docs: { enabled: true, topics: { readme: { text: "# r\n" } } },
+    };
+    const mcp = cliBuiltinMcpCommand(withDocs);
     expect(mcp.description).toContain("MCP server");
-    expect(mcp.notes).toContain('["mcp"]');
+    expect(mcp.notes).toContain("install --mcp --yes");
+    expect(mcp.notes).toContain("docs mcp");
   });
 });
 
@@ -87,7 +92,8 @@ describe("presentation root", () => {
       docs: { enabled: true, topics: { readme: { text: "# r\n" } } },
     };
     const root = cliPresentationRoot(withDocs);
-    expect(root.notes).toContain("Agents: run `myapp docs skill` to learn how to use this app");
+    expect(root.notes).toContain("For AI agents: `myapp docs skill`.");
+    expect(root.notes).not.toContain("install --skill");
   });
 });
 

package/src/builtins/completion-group.ts

@@ -1,10 +1,13 @@
-import { type CliLeaf, type CliNode, type CliRouter } from "../types.ts";
+import { resolveCapabilities } from "../capabilities.ts";
+import { type CliLeaf, type CliProgram, type CliRouter } from "../types.ts";
 
 /**
  * Builds the static `completion` / `bash` / `zsh` / `fish` command subtree (merged into the program root at runtime).
  */
-export function cliBuiltinCompletionGroup(appName: string): CliRouter {
-  return {
+export function cliBuiltinCompletionGroup(program: CliProgram): CliRouter {
+  const appName = program.key;
+  const caps = resolveCapabilities(program);
+  const router: CliRouter = {
     key: "completion",
     description: "Generate the autocompletion script for shells.",
     commands: [
@@ -12,15 +15,10 @@ export function cliBuiltinCompletionGroup(appName: string): CliRouter {
         key: "bash",
         description: "Print a bash tab-completion script.",
         notes:
-          "Output is the whole script.\n" +
-          "Pipe it to a file, or feed it straight into your shell.\n\n" +
-          "To keep it across restarts, save it and source that file from ~/.bashrc.\n\n" +
-          "For example:\n\n" +
-          `echo 'eval \"$(${appName} completion bash)\"' >> ~/.bashrc\n` +
-          `\nor\n` +
+          "Manual install:\n\n" +
           `  ${appName} completion bash > ~/.bash_completion.d/${appName}\n` +
           `  echo 'source ~/.bash_completion.d/${appName}' >> ~/.bashrc\n\n` +
-          "To try it only in this session (nothing written to disk):\n" +
+          "Try this session only:\n\n" +
           `  source <(${appName} completion bash)`,
         handler: () => {},
       },
@@ -28,23 +26,26 @@ export function cliBuiltinCompletionGroup(appName: string): CliRouter {
         key: "zsh",
         description: "Print a zsh tab-completion script.",
         notes:
-          "Output is the whole script.\n\n" +
-          `fpath setup: ${appName} completion zsh > ~/.zsh/completions/_${appName}\n\n` +
-          `source setup: echo 'eval \"$(${appName} completion zsh)\"' >> ~/.zshrc\n\n` +
-          "To try it only in this session (nothing written to disk):\n" +
-          `  eval \"$(${appName} completion zsh)\"`,
+          "Manual install:\n\n" +
+          `  ${appName} completion zsh > ~/.zsh/completions/_${appName}\n\n` +
+          "Ensure ~/.zsh/completions is on your fpath, then restart zsh.\n\n" +
+          "Try this session only:\n\n" +
+          `  eval "$(${appName} completion zsh)"`,
         handler: () => {},
       },
       {
         key: "fish",
         description: "Print a fish tab-completion script.",
         notes:
-          "Output is the whole script.\n\n" +
-          "Install:\n" +
+          "Manual install:\n\n" +
           `  ${appName} completion fish > ~/.config/fish/completions/${appName}.fish\n\n` +
           "Fish loads completions from that directory automatically.",
         handler: () => {},
       },
     ],
   };
+  if (caps.install) {
+    router.notes = `Install for all shells:\n\n  ${appName} install --completions --yes`;
+  }
+  return router;
 }

package/src/builtins/dispatch.ts

@@ -110,7 +110,7 @@ export function builtinInterceptRoot(
       parseRoot: {
         key: program.key,
         description: program.description,
-        commands: [completionGroup(program.key)],
+        commands: [completionGroup(program)],
       },
       isLeafCompletionIntercept: true,
     };
@@ -132,7 +132,7 @@ export function builtinInterceptRoot(
       parseRoot: {
         key: program.key,
         description: program.description,
-        commands: [cliBuiltinMcpCommand()],
+        commands: [cliBuiltinMcpCommand(program)],
       },
       isLeafCompletionIntercept: false,
     };

package/src/builtins/export.ts

@@ -45,7 +45,7 @@ function exportBuiltinNode(cmd: {
 export function exportPresentationBuiltins(program: CliProgram, caps?: CliCapabilities): CliSchemaExport[] {
   const resolved = caps ?? resolveCapabilities(program);
   const builtins: CliSchemaExport[] = [
-    exportBuiltinNode(cliBuiltinCompletionGroup(program.key)),
+    exportBuiltinNode(cliBuiltinCompletionGroup(program)),
     exportBuiltinNode(cliBuiltinVersionCommand()),
   ];
   if (resolved.install) {
@@ -56,7 +56,7 @@ export function exportPresentationBuiltins(program: CliProgram, caps?: CliCapabi
     builtins.push(exportBuiltinNode(docsGroup));
   }
   if (resolved.mcp) {
-    builtins.push(exportBuiltinNode(cliBuiltinMcpCommand()));
+    builtins.push(exportBuiltinNode(cliBuiltinMcpCommand(program)));
   }
   return builtins;
 }

package/src/builtins/install.ts

@@ -117,7 +117,8 @@ export function cliBuiltinInstallCommand(root: CliProgram): CliLeaf {
     "Remove everything installed with --all:",
     `  ${app} install --uninstall --all --yes`,
     "",
-    "Use --dry to preview changes, --json for machine-readable output.",
+    "Use --dry to preview changes without writing files.",
+    "Use --json for machine-readable output.",
   );
   return {
     key: "install",

package/src/builtins/mcp.ts

@@ -1,13 +1,27 @@
-import { type CliLeaf } from "../types.ts";
+import { resolveCapabilities } from "../capabilities.ts";
+import { docsEnabled } from "../docs/resolve.ts";
+import { type CliLeaf, type CliProgram } from "../types.ts";
 
 /** Presence options for the top-level `mcp` built-in (leaf). */
-export function cliBuiltinMcpCommand(): CliLeaf {
+export function cliBuiltinMcpCommand(program: CliProgram): CliLeaf {
+  const caps = resolveCapabilities(program);
+  const lines = [
+    "Stdio MCP server. Add to Cursor or Claude:",
+    "",
+    "  command: {argsbarg:program}",
+    "  args: mcp",
+    "",
+  ];
+  if (caps.install) {
+    lines.push("Or:", "", "  {argsbarg:program} install --mcp --yes", "");
+  }
+  if (docsEnabled(program)) {
+    lines.push("Full setup guide: {argsbarg:program} docs mcp");
+  }
   return {
     key: "mcp",
     description: "Run as an MCP server over stdio for AI agents.",
-    notes:
-      "Configure MCP clients with `command` set to this program name and `args` set to `[\"mcp\"]`.\n\n" +
-      "See docs/mcp.md for setup details.",
+    notes: lines.join("\n"),
     handler: () => {},
   };
 }

package/src/builtins/presentation.ts

@@ -11,7 +11,7 @@ import { cliBuiltinDocsGroupIfEnabled } from "../docs/builtin.ts";
 /** Built-in command nodes injected for help, schema, and completions. */
 export function presentationBuiltins(program: CliProgram, caps: CliCapabilities): CliNode[] {
   const builtins: CliNode[] = [
-    cliBuiltinCompletionGroup(program.key),
+    cliBuiltinCompletionGroup(program),
     cliBuiltinVersionCommand(),
   ];
   if (caps.install) {
@@ -22,7 +22,7 @@ export function presentationBuiltins(program: CliProgram, caps: CliCapabilities)
     builtins.push(docsGroup);
   }
   if (caps.mcp) {
-    builtins.push(cliBuiltinMcpCommand());
+    builtins.push(cliBuiltinMcpCommand(program));
   }
   return builtins;
 }
@@ -64,8 +64,7 @@ export function presentationRootNotes(program: CliProgram, caps: CliCapabilities
     parts.push(program.notes!.trim());
   }
   if (caps.docs) {
-    const cmd = `${program.key} docs skill`;
-    parts.push(`Agents: run \`${cmd}\` to learn how to use this app`);
+    parts.push(`For AI agents: \`${program.key} docs skill\`.`);
   }
   if (parts.length === 0) {
     return undefined;

package/src/docs/builtin.ts

@@ -36,6 +36,11 @@ function docsLeaf(program: CliProgram, key: string, description: string): CliLea
   };
 }
 
+/** Help notes for the `docs` router. */
+function docsRouterNotes(): string {
+  return "Topics print to stdout. Add --save to write files under ./docs/.";
+}
+
 /** Built-in `docs` router with bundled topic subcommands. */
 export function cliBuiltinDocsGroup(program: CliProgram): CliRouter {
   const docs = program.docs!;
@@ -54,26 +59,14 @@ export function cliBuiltinDocsGroup(program: CliProgram): CliRouter {
 
   leaves.push(
     docsLeaf(program, "schema", "Print the full command tree as JSON."),
-    docsLeaf(program, "api", "Print the command tree as markdown."),
-    {
-      key: "skill",
-      description: "Print generated SKILL.md (compact command index).",
-      notes: [
-        "Prefer `{argsbarg:program} install --skill --yes` for agents: it persists an optimized skill bundle",
-        "(`SKILL.md` index + `reference.md` full API) to your skill directory.",
-        "`docs skill` prints the index only; use `docs api` or installed `reference.md` for full detail.",
-      ].join(" "),
-      options: [DOCS_SAVE_OPTION],
-      mcpTool: { enabled: false },
-      handler: (ctx) => {
-        runDocsTopic(program, "skill", ctx);
-      },
-    },
+    docsLeaf(program, "api", "Print the full command reference as markdown."),
+    docsLeaf(program, "skill", "Print a reference agent SKILL, use `install --skill` for optimized."),
   );
 
   return {
     key: "docs",
     description: docs.description ?? DOCS_ROUTER_DESCRIPTION,
+    notes: docsRouterNotes(),
     options: [DOCS_SAVE_OPTION],
     fallbackCommand: docsEffectiveDefaultTopic(docs),
     fallbackMode: CliFallbackMode.MissingOnly,

package/src/docs/docs.test.ts

@@ -153,7 +153,7 @@ test("docs skill prints Cursor SKILL.md", async () => {
   expect(result.stdout).toContain("---");
   expect(result.stdout).toContain("name: myapp");
   expect(result.stdout).toContain("## Commands");
-  expect(result.stdout).toContain("read `reference.md`");
+  expect(result.stdout).toContain("For full detail, open `reference.md`");
   expect(result.stdout).not.toContain("#### Options");
   expect(result.stdout).not.toContain("mcp.json");
 });
@@ -164,9 +164,11 @@ test("docs skill help recommends install --skill", async () => {
   expect(docsNode && "commands" in docsNode).toBe(true);
   if (docsNode && "commands" in docsNode) {
     const skill = docsNode.commands.find((c) => c.key === "skill");
-    expect(skill?.description).toContain("compact command index");
-    expect(skill?.notes).toContain("install --skill --yes");
-    expect(skill?.notes).toContain("reference.md");
+    expect(skill?.description).toContain("reference agent SKILL");
+    expect(skill?.description).toContain("install --skill");
+    expect(skill?.notes).toBeUndefined();
+    expect(docsNode.notes).toContain("--save");
+    expect(docsNode.notes).not.toContain("install --skill");
   }
 });
 

package/src/index.test.ts

@@ -496,7 +496,7 @@ test("leaf completion help prints correctly", async () => {
   const out = stdout.toString();
   expect(exitCode).toBe(0);
   expect(out).toContain("Show help for this command.");
-  expect(out).toContain("Output is the whole script.");
+  expect(out).toContain("Manual install:");
   expect(stderr.toString()).toBe("");
 });
 
@@ -658,7 +658,7 @@ test("docs help lists schema, api, and skill subcommands", () => {
   expect(help).toContain("api");
   expect(help).toContain("markdown");
   expect(help).toContain("skill");
-  expect(help).toContain("SKILL.md");
+  expect(help).toContain("reference agent SKILL");
 });
 
 test("root help omits legacy --schema flag", () => {
@@ -690,7 +690,8 @@ test("root help shows agent docs hint when docs enabled", () => {
     commands: [{ key: "run", description: "Run.", handler: () => {} }],
   });
   const help = cliHelpRender(cliPresentationRoot(root), [], false);
-  expect(help).toContain("Agents: run `myapp docs skill` to learn how to use this app");
+  expect(help).toContain("For AI agents: `myapp docs skill`.");
+  expect(help).not.toContain("install --skill");
 });
 
 test("root help omits agent hint when docs disabled", () => {
@@ -1815,7 +1816,7 @@ test("generateSkillBundle includes frontmatter and compact command index", () =>
   expect(bundle.skillMd).toContain("## Commands");
   expect(bundle.skillMd).toContain("`nested.ts stat owner lookup <path>`");
   expect(bundle.skillMd).toContain("Invoke via shell:");
-  expect(bundle.skillMd).toContain("read `reference.md`");
+  expect(bundle.skillMd).toContain("For full detail, open `reference.md`");
   expect(bundle.skillMd).not.toContain("#### Options");
   expect(bundle.skillMd).not.toContain("CLI API reference");
   expect(bundle.skillMd).not.toContain("mcp.json");

package/src/skill/generate.ts

@@ -107,14 +107,12 @@ function buildSkillMd(root: CliProgram, target: SkillTarget, dirName: string): s
   lines.push(
     "## Pitfalls",
     "",
-    "- Use `--` before tokens that look like flags when they are positional arguments.",
-    "- Required environment variables are listed per command above (`requires env`) and in `reference.md`.",
+    "- Pass `--` before arguments that look like flags.",
+    "- Commands marked `[requires env: ...]` need those variables set in the shell.",
     "",
     "## Reference",
     "",
-    "This file is a command index. For full option tables, positionals, notes, and built-ins,",
-    `read \`reference.md\` in this skill directory (same content as \`${root.key} docs api\`).`,
-    "Search for the command path you need before invoking.",
+    `For full detail, open \`reference.md\` in this skill directory (same as \`${root.key} docs api\`).`,
     "",
   );