@duyquangnvx/create-agent-cli 0.3.0 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@duyquangnvx/create-agent-cli",
-  "version": "0.3.0",
+  "version": "0.5.0",
   "description": "Creates a new Agent CLI repository wired to @duyquangnvx/agent-cli.",
   "type": "module",
   "engines": {
@@ -19,7 +19,7 @@
     "access": "public"
   },
   "devDependencies": {
-    "@duyquangnvx/agent-cli": "0.3.0",
+    "@duyquangnvx/agent-cli": "0.5.0",
     "tsdown": "0.23.0",
     "zod": "4.6.7"
   },

package/template/skills/{{binName}}/SKILL.md

@@ -41,11 +41,13 @@ The line above should show 0.0.0, the version this skill describes. If it says {
 2. If it states a cost, ask the user before going on.
 3. Run it again without `--dry-run`, adding `--yes` once the user approved the cost.
 
+To put a message under the greeting, pass `-` after the name and pipe the message in with a heredoc, so its quotes and newlines reach the card intact.
+
 Pass `--json` whenever you parse the output: stdout then holds only the data. A Command that changes something takes `--dry-run`, which prints its Plan, the files it writes and its cost, and changes nothing: run it first when unsure. A destructive Command also needs `--confirm=<target>`, and its error's hint gives the exact command.
 
 ## Recovering from errors
 
-Errors go to stderr and end with a `hint:` line naming the next step. Under `--json` they are `{ errors: [{ code, message, hint }] }`. Match the first row that fits:
+Errors go to stderr and end with a `hint:` line naming the next step. Under `--json`, every stderr line is one JSON object: `{"progress":"..."}` while it runs, then `{ errors: [{ code, message, hint }] }` as the last line. Match the first row that fits:
 
 | Code or exit | What to do |
 |---|---|

package/template/src/cli.ts

@@ -1,4 +1,4 @@
-import { agentCli, cliError } from "@duyquangnvx/agent-cli";
+import { agentCli, cliError, orStdin } from "@duyquangnvx/agent-cli";
 import * as z from "zod";
 import manifest from "../package.json" with { type: "json" };
 import { build } from "./build.js";
@@ -28,12 +28,16 @@ const greet = defineCommand({
 const card = defineCommand({
   name: "card",
   description: "Write a greeting card to <name>.txt",
+  help:
+    `A card in English is free; one in another language costs ${PRICE_PER_CARD} and needs --yes. ` +
+    "Give - as the message to read it from stdin: `{{binName}} card Ana - < note.txt`.",
   examples: [
     { command: "{{binName}} card Ana", note: "write Ana.txt with an English greeting" },
     {
       command: "{{binName}} card Ana --language es --dry-run",
       note: "show the file it would write and what it costs",
     },
+    { command: "{{binName}} card Ana --from Ben --from Cy", note: "sign the card from Ben and Cy" },
   ],
   effect: "write",
   args: z.object({
@@ -41,14 +45,24 @@ const card = defineCommand({
       .string()
       .regex(/^[\w-]+$/, "letters, digits, - and _ only")
       .describe("who the card is for; also the file's name"),
+    message: orStdin(z.string().min(1))
+      .optional()
+      .describe("a message to write under the greeting; - reads it from stdin"),
   }),
   options: z.object({
     language: z
       .string()
       .default(FREE_LANGUAGE)
       .describe("the greeting's language code, such as en, es or fr"),
+    from: z.array(z.string().min(1)).default([]).describe("who signs the card; repeat for each"),
+  }),
+  plan: ({ name, message, language, from }) => ({
+    name,
+    message,
+    language,
+    from,
+    file: `${name}.txt`,
   }),
-  plan: ({ name, language }) => ({ name, language, file: `${name}.txt` }),
   outputs: (plan) => [plan.file],
   cost: (plan) =>
     plan.language === FREE_LANGUAGE
@@ -62,7 +76,12 @@ const card = defineCommand({
         hint: `run \`{{binName}} card ${plan.name} --language ${FREE_LANGUAGE}\` for an English card`,
       });
     }
-    await ctx.files.write(plan.file, `${greeting}\n`);
+    const parts = [
+      greeting,
+      ...(plan.message === undefined ? [] : [plan.message.trimEnd()]),
+      ...(plan.from.length === 0 ? [] : [`From ${plan.from.join(", ")}`]),
+    ];
+    await ctx.files.write(plan.file, `${parts.join("\n\n")}\n`);
     return { file: plan.file, greeting };
   },
   render: ({ file }) => `Wrote ${file}`,

package/template/test/cli.test.ts

@@ -14,6 +14,22 @@ function fakeFiles(existing: readonly string[] = []) {
     write: async (path, data) => {
       written.set(path, String(data));
     },
+    read: async (path) => {
+      const text = written.get(path);
+      if (text === undefined) {
+        throw Object.assign(new Error(`ENOENT: no such file, open '${path}'`), { code: "ENOENT" });
+      }
+      return new TextEncoder().encode(text);
+    },
+    list: async () => [],
+    remove: async (path) => {
+      written.delete(path);
+    },
+    mkdir: async () => {},
+    stat: async (path) => {
+      const text = written.get(path);
+      return text === undefined ? undefined : { size: text.length, id: path };
+    },
   };
   return { files, written };
 }
@@ -30,8 +46,13 @@ function fakeGreetings() {
   return { greetings, calls };
 }
 
-function run(argv: string[], files = fakeFiles().files, greetings = fakeGreetings().greetings) {
-  return runCli(cli, argv, { env: {}, cwd: CWD, files, ports: { greetings } });
+function run(
+  argv: string[],
+  files = fakeFiles().files,
+  greetings = fakeGreetings().greetings,
+  stdin?: string,
+) {
+  return runCli(cli, argv, { env: {}, cwd: CWD, files, ports: { greetings }, stdin });
 }
 
 describe("greet", () => {
@@ -66,6 +87,35 @@ describe("card", () => {
     expect(written).toEqual(new Map([[resolve(CWD, "Ana.txt"), "Hi Ana\n"]]));
   });
 
+  test("- reads a message for under the greeting from stdin", async () => {
+    const { files, written } = fakeFiles();
+
+    const result = await run(
+      ["card", "Ana", "-"],
+      files,
+      undefined,
+      "Happy birthday!\nSee you soon.\n",
+    );
+
+    expect(result.exitCode).toBe(0);
+    expect(written.get(resolve(CWD, "Ana.txt"))).toBe("Hi Ana\n\nHappy birthday!\nSee you soon.\n");
+  });
+
+  test("--from, repeated, signs the card from each name", async () => {
+    const { files, written } = fakeFiles();
+
+    const result = await run(["card", "Ana", "--from", "Ben", "--from", "Cy"], files);
+
+    expect(result.exitCode).toBe(0);
+    expect(written.get(resolve(CWD, "Ana.txt"))).toBe("Hi Ana\n\nFrom Ben, Cy\n");
+  });
+
+  test("--help ends its options with a paragraph on cost and stdin", async () => {
+    const result = await run(["card", "--help"]);
+
+    expect(result.stdout).toMatch(/Options:[\s\S]*\n\nA card in English is free; /);
+  });
+
   test("refuses to overwrite an existing card without --force", async () => {
     const { files, written } = fakeFiles(["Ana.txt"]);