@nanopm/cli 0.1.20 → 0.1.22

package/dist/index.js

@@ -5426,6 +5426,9 @@ function answerForAgent(answer) {
   return isDecideForMe(answer) ? DECIDE_FOR_ME : answer;
 }
 var BUILD_HISTORY_KINDS = ["worktree", "plan", "step", "question", "answer", "paused", "pr", "handed", "throw", "thrown", "closed"];
+function checkNotesOf(json2) {
+  return (Array.isArray(json2) ? json2 : []).flatMap((n) => typeof n?.ask === "string" && typeof n?.answer === "string" ? [{ ask: n.ask, answer: n.answer }] : []);
+}
 function originOf(job) {
   const named = (job.params ?? {}).origin;
   return typeof named === "string" && /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/.test(named) ? named : job.id;
@@ -5476,6 +5479,9 @@ function buildHistory(events, at) {
   const answered2 = last?.kind === "question" && answers.has(last.seq) ? answers.get(last.seq) : null;
   if (answered2 !== null && last.payload.about === "usage") return found({ why: "limit" });
   if (answered2 !== null && (last.payload.about === "conflict" || last.payload.about === "refused")) return found({ why: last.payload.about, answer: answered2 });
+  if (answered2 !== null && last.payload.about === "check" && typeof last.payload.question === "string" && !isDecideForMe(answered2)) {
+    return found({ why: "answer", answer: answered2, keep: last.payload.question });
+  }
   if (answered2 !== null) return found({ why: "answer", answer: answered2 });
   if (last?.kind === "paused") return found({ why: "limit" });
   return found({ why: "lost" });
@@ -11288,6 +11294,7 @@ async function prepareBuild(client, job, clone, adapter) {
     made = true;
     await appendJobEvent(client, job.id, "worktree", { dir, branch, base, fork }, "nanopm");
     await handOverTheSpec(worktree, specForAgent(solution.agent_spec, solution.spec));
+    await handOverNotes(client, job.project_id, worktree);
     await startSolution(client, job.project_id, solution.ref, job.id);
     return { ok: true, build: { solution, clone, dir, worktree, branch, base, fork, network: await network(dir), history } };
   } catch (err) {
@@ -11311,7 +11318,8 @@ async function runBuildJob(input) {
   };
   const park = async (why) => {
     if ("ask" in why) {
-      await events.write("question", { ...why.ask, ...why.ask.about ? { report: said } : plan ? {} : { about: "plan" } });
+      const atClose = why.ask.about === "conflict" || why.ask.about === "refused";
+      await events.write("question", { ...why.ask, ...atClose ? { report: said } : why.ask.about || plan ? {} : { about: "plan" } });
       await log("build.ask", `${solution.ref} waits on the founder`);
     } else if (why.resetsAt) {
       await events.write("paused", { why: "limit", resets_at: new Date(why.resetsAt * 1e3).toISOString() });
@@ -11401,6 +11409,10 @@ The project's own check is \`${verifyCommand2}\`. Run it before your last build_
   } else {
     await events.write("text", { text: AGAIN[resume2.why] });
     if (resume2.why === "answer" && spec2.answerFile) await writeFile5(spec2.answerFile, answerForAgent(resume2.answer), { mode: 384 });
+    if (resume2.why === "answer" && resume2.keep) {
+      const { error } = await client.rpc("add_check_note", { p_project_id: job.project_id, p_ask: resume2.keep, p_answer: resume2.answer });
+      if (error) await log("build.note", `the answer was not kept: ${error.message}`, "error");
+    }
   }
   if (resume2?.why === "refused") return openPr();
   let turn = null;
@@ -11462,9 +11474,10 @@ var AGAIN = {
   refused: "Asking GitHub for the pull request again."
 };
 function askOf(input) {
-  const { question, options, why } = input ?? {};
+  const { question, options, why, about } = input ?? {};
   return {
     question: String(question ?? ""),
+    ...about === "check" ? { about: "check" } : {},
     ...Array.isArray(options) ? { options: options.map(String) } : {},
     ...typeof why === "string" ? { why } : {}
   };
@@ -11497,6 +11510,16 @@ async function handOverTheSpec(worktree, agentSpec) {
 `);
   }
 }
+async function handOverNotes(client, projectId, worktree) {
+  const { data } = await client.from("project_preferences").select("check_notes").eq("project_id", projectId).maybeSingle();
+  const lines = checkNotesOf(data?.check_notes).map((n) => `- ${n.ask.replace(/\s*\n\s*/g, " ")}
+  ${n.answer.replace(/\n/g, "\n  ")}`);
+  if (!lines.length) return;
+  await writeFile5(join13(worktree.dir, ".nanopm", "CHECK.md"), `# What earlier builds asked the founder, to check the app
+
+${lines.join("\n")}
+`);
+}
 function stepsOf(input) {
   const steps = input?.steps;
   return Array.isArray(steps) && steps.length && steps.every((s) => typeof s === "string") ? steps : null;
@@ -12640,7 +12663,9 @@ ${renderMap()}`,
         // important question of the run must not be refused for being long.
         question: z21.string().min(1).max(8e3),
         options: z21.array(z21.string().min(1)).max(8).optional(),
-        why: z21.string().max(400).optional()
+        why: z21.string().max(400).optional(),
+        // A build asking what it needs to check the app that the code cannot tell (spec 77).
+        about: z21.enum(["check"]).optional().describe("In a build, `check` when you ask what you need to check the app and cannot find in the code (a test user, where an email lands): the answer is kept for the next builds.")
       }
     },
     async ({ question, options, why }, extra) => {
@@ -14727,7 +14752,7 @@ import { promisify as promisify9 } from "util";
 // package.json
 var package_default = {
   name: "@nanopm/cli",
-  version: "0.1.20",
+  version: "0.1.22",
   description: "An autonomous product manager for builders who run several small products. Runs on your machine, drives your own coding harness.",
   type: "module",
   license: "MIT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanopm/cli",
-  "version": "0.1.20",
+  "version": "0.1.22",
   "description": "An autonomous product manager for builders who run several small products. Runs on your machine, drives your own coding harness.",
   "type": "module",
   "license": "MIT",

package/skills/build/SKILL.md

@@ -36,6 +36,9 @@ NanoPM commits each step for you and opens the pull request when you are done.
 5. **Check each acceptance criterion**, inside your last step: go through each one of
    `.nanopm/SPEC.md` and check it as close to what the user sees as you can here — run the code
    path it names (call the handler, run the script, render the component), and note what you saw.
+   `.nanopm/CHECK.md`, when there is one, holds what earlier builds asked the founder to check the
+   app; read it first. What you need and cannot find in the code (a test user, where an email
+   lands), ask with `ask_user` and `about: "check"`: the answer is kept for every build after.
    Your sandbox cannot open a local port, so a running server, a browser or a device is out of
    reach: say so for what needs one. Never touch a database, a service or a file outside your copy
    to check something. Keep package caches in `$TMPDIR` (`npm install --cache "$TMPDIR/npm"`, a
@@ -50,7 +53,9 @@ NanoPM commits each step for you and opens the pull request when you are done.
    - **How I checked it**: first one line per acceptance criterion, exactly `- AC1 — saw: what you
      saw` or `- AC1 — could not: what, because why`. *Saw* is for what you saw happen when you ran
      the code the criterion is about — say how (a handler called with what, a script's output);
-     what you could not reach is *could not*, with why. Never say you saw what you did not run. Then the commands you ran, and their result.
+     what you could not reach is *could not*, with why, and ends with `Try it:` and how a person
+     checks it in one or two steps, in the founder's words (« sign up with a new account; after the
+     notifications step, the warm-up appears »). Never say you saw what you did not run. Then the commands you ran, and their result.
 7. `log` one line: what you built.
 
 ## When to ask the founder