halfcycle 0.4.10 → 0.4.12

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "halfcycle",
-  "version": "0.4.10",
+  "version": "0.4.12",
   "description": "Halfcycle Method bundle — resolution-stub slash commands, remote method-delivery registration, and governance-hook wiring for a Halfcycle engagement repo. It ships NO worker role files: a project's roles are authored from that project's own recorded decisions at orchestration kickoff (FX-2, W3-F-27).",
   "commands": [
     {

package/bin/bin.bundle.mjs

@@ -17024,7 +17024,7 @@ function approvalSentence(phase, offerTyping) {
     return TYPED_APPROVAL_OFF_SENTENCE;
   if (offerTyping === "update")
     return CLIENT_UPDATE_SENTENCE;
-  return `In their own Claude Code session, they ask to see the plan, check each part, and then type: approve ${phase}`;
+  return `In their own Claude Code session, they ask to see the plan, check each part, and then type: approve ${phase}.`;
 }
 function phaseWords(phase) {
   return `phase "${phase}"`;
@@ -20814,6 +20814,9 @@ function reviewAlreadyDecidedLine(name) {
 function reviewAwaitingAnswerLine(name) {
   return line(`${NOTHING_RECORDED}: "${name}" is waiting for the agent's answer to your question. Once it is in, say "review what was built" to list it again and decide the part.`);
 }
+function reviewAnswerNotListedLine(name) {
+  return line(`${NOTHING_RECORDED}: the review listed here does not show the agent's answer to your question on "${name}". Say "review what was built" to list it again: the answer shows there once it is in, and then you can decide the part.`);
+}
 function reviewSentBackLine(name) {
   return line(`${NOTHING_RECORDED}: "${name}" was sent back to building in this round, so it waits for the next round.`);
 }
@@ -21022,7 +21025,7 @@ async function checkAndSendReview(match, engagementId, credential, env, opts) {
         return reviewWordsLookSecretLine();
     }
     if (part.awaitingAnswer)
-      return reviewAwaitingAnswerLine(part.name);
+      return reviewAnswerNotListedLine(part.name);
     if (part.sentBack)
       return reviewSentBackLine(part.name);
     if (!part.decidable)

package/dist/build-record/render-v2.d.ts

@@ -71,6 +71,11 @@ export declare function printItemStartText(text: string): string;
  */
 export declare function latestCloseHeadline(record: BuildRecord): string;
 /**
+ * A v3 record whose review ran more than one round says so in its opening line and
+ * points down to "Earlier rounds of the review", and the close's walk line names the
+ * round that closed the phase (review R-FW10: a reader who stopped at the opening never
+ * learned an earlier round said something else).
+ *
  * Render a parsed Build Record as markdown: v2's observed and attested halves, which
  * a v3 record carries unchanged, and for a v3 record the owner's review after the
  * latest close (RM1–RM3, feature `close-review-and-carry-drop`).

package/dist/build-record/render-v2.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"render-v2.d.ts","sourceRoot":"","sources":["../../src/build-record/render-v2.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AAEH,OAAO,EAA4B,KAAK,WAAW,EAA0C,MAAM,mBAAmB,CAAC;AA8JvH;;;;;;;;GAQG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAepD;AAKD;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CASpD;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAOvD;AAgMD;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CAO/D;AAmOD;;;;;;;GAOG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CA4DvE"}
+{"version":3,"file":"render-v2.d.ts","sourceRoot":"","sources":["../../src/build-record/render-v2.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AAEH,OAAO,EAGL,KAAK,WAAW,EAKjB,MAAM,mBAAmB,CAAC;AA8J3B;;;;;;;;GAQG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAepD;AAKD;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CASpD;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAOvD;AAgMD;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CAO/D;AAmRD;;;;;;;;;;;;GAYG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CAyEvE"}

package/dist/cli-contract.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli-contract.d.ts","sourceRoot":"","sources":["../src/cli-contract.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA8DG;AAIH,iFAAiF;AACjF,eAAO,MAAM,SAAS,mIAAoI,CAAC;AAE3J,MAAM,MAAM,OAAO,GAAG,CAAC,OAAO,SAAS,CAAC,CAAC,MAAM,CAAC,CAAC;AAEjD,sDAAsD;AACtD,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB;;;;OAIG;IACH,QAAQ,CAAC,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,2DAA2D;IAC3D,QAAQ,CAAC,UAAU,EAAE,OAAO,CAAC;IAC7B,+EAA+E;IAC/E,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;CACjC;AAED,iDAAiD;AACjD,MAAM,WAAW,eAAe;IAC9B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,UAAU,EAAE,OAAO,CAAC;IAC7B,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;CACjC;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,SAAS,YAAY,EAAE,CAAC;IAC3C,QAAQ,CAAC,WAAW,EAAE,SAAS,eAAe,EAAE,CAAC;CAClD;AAiFD,6EAA6E;AAC7E,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,GAAG,YAAY,GAAG,SAAS,CAElE;AAED,iDAAiD;AACjD,MAAM,WAAW,SAAS;IACxB,kEAAkE;IAClE,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,oCAAoC;IACpC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;CAC1B;AAmBD;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,SAAS,EAAE,CAyBxF;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAMtE;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,GAAG,MAAM,CAMvE;AASD;;;;GAIG;AACH,eAAO,MAAM,kBAAkB;;CAErB,CAAC;AAmBX;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,GAAG,IAAI,CAa1F"}
+{"version":3,"file":"cli-contract.d.ts","sourceRoot":"","sources":["../src/cli-contract.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA8DG;AAIH,iFAAiF;AACjF,eAAO,MAAM,SAAS,mIAAoI,CAAC;AAE3J,MAAM,MAAM,OAAO,GAAG,CAAC,OAAO,SAAS,CAAC,CAAC,MAAM,CAAC,CAAC;AAEjD,sDAAsD;AACtD,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB;;;;OAIG;IACH,QAAQ,CAAC,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,2DAA2D;IAC3D,QAAQ,CAAC,UAAU,EAAE,OAAO,CAAC;IAC7B,+EAA+E;IAC/E,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;CACjC;AAED,iDAAiD;AACjD,MAAM,WAAW,eAAe;IAC9B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,UAAU,EAAE,OAAO,CAAC;IAC7B,QAAQ,CAAC,cAAc,EAAE,MAAM,CAAC;CACjC;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,SAAS,YAAY,EAAE,CAAC;IAC3C,QAAQ,CAAC,WAAW,EAAE,SAAS,eAAe,EAAE,CAAC;CAClD;AAiFD,6EAA6E;AAC7E,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,GAAG,YAAY,GAAG,SAAS,CAElE;AAED,iDAAiD;AACjD,MAAM,WAAW,SAAS;IACxB,kEAAkE;IAClE,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,oCAAoC;IACpC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAC;CAC1B;AAmBD;;;;;;;;;;;GAWG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,SAAS,EAAE,CAyBxF;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,GAAG,MAAM,CAMtE;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,GAAG,MAAM,CAMvE;AASD;;;;GAIG;AACH,eAAO,MAAM,kBAAkB;;CAErB,CAAC;AAyBX;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,OAAO,EAAE,MAAM,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,GAAG,IAAI,CAa1F"}

package/dist/cli.js

@@ -2867,11 +2867,16 @@ var phaseReviewViewSchema = z14.discriminatedUnion("state", [
 
 // ../events/dist/build-record-v3.js
 var BUILD_RECORD_V3_FORMAT = "halfcycle-build-record/v3";
+var recordRoundStatedSchema = z15.object({
+  actor: attestedCloseSchema.shape.actor,
+  statements: attestedStatementsSchema
+}).strict();
 var recordReviewRoundSchema = z15.object({
   round: z15.number().int().min(1),
   openedAt: utcIso8601,
   verdict: walkVerdictSchema,
-  findings: z15.number().int().min(0)
+  findings: z15.number().int().min(0),
+  stated: recordRoundStatedSchema.optional()
 }).strict();
 var recordReviewApprovalSchema = z15.discriminatedUnion("kind", [
   z15.object({
@@ -2933,6 +2938,9 @@ var recordReviewSchema = z15.object({
     if (r.round !== i + 1) {
       ctx.addIssue({ code: "custom", path: ["rounds", i, "round"], message: "rounds must be numbered 1, 2, 3 \u2026 in order with no gap" });
     }
+    if (i === v.rounds.length - 1 && r.stated !== void 0) {
+      ctx.addIssue({ code: "custom", path: ["rounds", i, "stated"], message: "the last round's words are the close's own, so the round does not repeat them" });
+    }
   });
   const seen = /* @__PURE__ */ new Set();
   v.parts.forEach((p, i) => {
@@ -7138,12 +7146,13 @@ function machines(coverage, checks) {
   const line = `${label}${String(coverage.enrolled)}. Of these, ${String(coverage.conformant)} had reported Halfcycle's checks as fully set up.`;
   return checks === "not-observed" && coverage.conformant > 0 ? `${line} Set up is not the same as running: Halfcycle has no record of any check running during this phase.` : line;
 }
-function walkLines(walk, p) {
+function walkLines(walk, p, closingRound = false) {
   const label = "> - Acceptance walk (the finished work tried as a user would, by someone other than its builder): ";
   if (walk === null)
     return [`${label}the person closing the phase reported none.`];
+  const it = closingRound ? "the round that closed the phase" : "it";
   const lines = [
-    `${label}${walk.verdict === "clean" ? "the person closing the phase said it found no problems." : "the person closing the phase said it found problems."}`
+    `${label}${walk.verdict === "clean" ? `the person closing the phase said ${it} found no problems.` : `the person closing the phase said ${it} found problems.`}`
   ];
   walk.findings.forEach((finding, i) => {
     lines.push(`>   - Finding ${String(i + 1)}: ${p.text(finding)}`);
@@ -7162,12 +7171,12 @@ function decisionLine(attested) {
   }
   return attested.walk !== null && attested.walk.verdict === "defects" ? "> - Decision: close the phase as it stood, with the problems its acceptance walk found." : "> - Decision: the phase's agreed finish conditions were met.";
 }
-function renderStated(attested, p) {
+function renderStated(attested, p, closingRound = false) {
   const lines = [
     decisionLine(attested),
     `> - Name given: ${attested.actor === null ? "removed with the account that made this close" : p.text(attested.actor)}`,
     `> - Reason given: ${attested.reason === null ? "none." : p.text(attested.reason)}`,
-    ...walkLines(attested.walk, p),
+    ...walkLines(attested.walk, p, closingRound),
     `> - Code closed at (commit): ${statedOrNot(attested.closeCommit, p)}`,
     `> - What the phase set out to prove: ${statedOrNot(attested.statements.thesis, p)}`,
     `> - What came out of it: ${statedOrNot(attested.statements.outcome, p)}`
@@ -7183,7 +7192,7 @@ function renderStated(attested, p) {
   }
   return lines.join("\n");
 }
-function renderClose(heading, observed, attested, checks, p) {
+function renderClose(heading, observed, attested, checks, p, closingRound = false) {
   return [
     heading,
     headline(observed, attested, checks),
@@ -7195,7 +7204,7 @@ function renderClose(heading, observed, attested, checks, p) {
       machines(observed.seatCoverage, checks)
     ].join("\n"),
     STATED_LABEL,
-    renderStated(attested, p)
+    renderStated(attested, p, closingRound)
   ];
 }
 var REVIEW_LABEL = "**Recorded by Halfcycle from the owner's review.** Each part the review held, and how the project's owner decided it. A line marked as the agent's is the agent's own answer, not Halfcycle's.";
@@ -7244,6 +7253,37 @@ function renderReview(record2, p) {
     blocks.push(removals);
   return blocks;
 }
+var EARLIER_ROUNDS_LABEL = `${STATED_LABEL} What was said each time the owner's review was asked for, before the round that closed the phase. What was said for that last round is under the close above. If the account that asked for a round is later deleted, the name given for that round is removed.`;
+function renderRoundStated(round, stated, p) {
+  const label = "> - Acceptance walk (the finished work tried as a user would, by someone other than its builder): ";
+  const walk = round.verdict === "clean" ? "the person asking for this round said it found no problems." : round.findings === 0 ? "the person asking for this round said it found problems." : `the person asking for this round said it found ${counted(round.findings, "problem", "problems")}.`;
+  const lines = [
+    `> - Name given: ${stated.actor === null ? "removed with the account that asked for this round" : p.text(stated.actor)}`,
+    `${label}${walk}`,
+    `> - What the phase set out to prove: ${statedOrNot(stated.statements.thesis, p)}`,
+    `> - What came out of it: ${statedOrNot(stated.statements.outcome, p)}`
+  ];
+  const deviations = stated.statements.deviations;
+  if (deviations.length === 0) {
+    lines.push("> - Changes from the plan: None stated by the client.");
+  } else {
+    lines.push("> - Changes from the plan:");
+    deviations.forEach((deviation, i) => {
+      lines.push(`>   - Change ${String(i + 1)}: ${p.text(deviation)}`);
+    });
+  }
+  return lines.join("\n");
+}
+function renderEarlierRounds(record2, p) {
+  const rounds = record2.review.rounds;
+  const blocks = [];
+  for (const round of rounds) {
+    if (round.stated === void 0)
+      continue;
+    blocks.push(`### Round ${String(round.round)} of ${String(rounds.length)}`, renderRoundStated(round, round.stated, p));
+  }
+  return blocks.length === 0 ? [] : ["## Earlier rounds of the review", EARLIER_ROUNDS_LABEL, ...blocks];
+}
 function renderBuildRecordV2Markdown(record2) {
   const p = makePrinter();
   const attestedBySequence = /* @__PURE__ */ new Map();
@@ -7257,6 +7297,8 @@ function renderBuildRecordV2Markdown(record2) {
   const closes = [...record2.observed.closes].sort((a, b) => a.sequence - b.sequence);
   const total = closes.length;
   const latest = closes[total - 1];
+  const rounds = record2.format === "halfcycle-build-record/v3" ? record2.review.rounds : [];
+  const roundsAfter = rounds.length <= 1 ? null : `after ${String(rounds.length)} rounds of the owner's review.${rounds.some((r) => r.stated !== void 0) ? ' What was said for each earlier round is under "Earlier rounds of the review".' : ""}`;
   const body = [
     `Phase: ${phase} (the name the client gave it)`,
     OBSERVED_LABEL,
@@ -7265,12 +7307,12 @@ function renderBuildRecordV2Markdown(record2) {
       `- Project identifier: ${p.text(record2.observed.engagementId)}. Halfcycle keeps this project's records under this identifier.`,
       `- Record format: ${p.text(record2.format)}. This names the layout this file and the .json file beside it follow, for software that reads them.`
     ].join("\n"),
-    total === 1 ? "This phase was closed once." : `This phase was closed ${String(total)} times. The latest close is described first; each earlier close follows, as it was recorded when it was made.`,
+    total === 1 ? roundsAfter === null ? "This phase was closed once." : `This phase was closed once, ${roundsAfter}` : `This phase was closed ${String(total)} times. The latest close is described first; each earlier close follows, as it was recorded when it was made.${roundsAfter === null ? "" : ` The latest close came ${roundsAfter}`}`,
     ...renderChecks(record2.observed.checks, p),
-    ...renderClose(total === 1 ? "## The close" : `## The latest close (close ${String(latest.sequence)} of ${String(total)})`, latest, attestedBySequence.get(latest.sequence), record2.observed.checks.state, p)
+    ...renderClose(total === 1 ? "## The close" : `## The latest close (close ${String(latest.sequence)} of ${String(total)})`, latest, attestedBySequence.get(latest.sequence), record2.observed.checks.state, p, rounds.length > 1)
   ];
   if (record2.format === "halfcycle-build-record/v3")
-    body.push(...renderReview(record2, p));
+    body.push(...renderReview(record2, p), ...renderEarlierRounds(record2, p));
   if (total > 1) {
     body.push("## Earlier closes");
     for (const close of closes.slice(0, -1)) {
@@ -8891,7 +8933,7 @@ ${redraw}`;
       exitCode: 1
     };
   }
-  const stdout = listed + `${approvalLine2(shownPhase, outcome.phaseNumber, outcome.typedApprovalOffered, deps.pinnedClientVersion(repoRoot), written.entries, stage.credential.engagementId)}
+  const stdout = outcome.state === "approved" ? listed : listed + `${approvalLine2(shownPhase, outcome.phaseNumber, outcome.typedApprovalOffered, deps.pinnedClientVersion(repoRoot), written.entries, stage.credential.engagementId)}
 `;
   return { stdout, stderr: warnings, exitCode: 0 };
 }
@@ -9636,6 +9678,7 @@ var REASON_PLACEHOLDER = {
   "open-phase": "<why you are opening it anyway>"
 };
 var OWN_WORDS = {
+  "--actor": "name, the one you go by as the agent",
   "--reason": "reason",
   "--evidence": "evidence",
   "--finding": "finding"
@@ -10834,7 +10877,7 @@ ${PROJECT_RESET_LINE}
       }
       process.stdout.write(`[halfcycle] Found: ${foundLine}
 `);
-      process.stdout.write(`[halfcycle] Set up: ${written} file${written === 1 ? "" : "s"} written${merged > 0 ? ` (${merged} merged)` : ""}, ${credential ? "guards armed on every edit" : "not checked \u2014 this copy is not this account\u2019s project"}
+      process.stdout.write(`[halfcycle] Set up: ${written} file${written === 1 ? "" : "s"} written${merged > 0 ? ` (${merged} merged)` : ""}, ${credential ? "checked on every edit" : "not checked \u2014 this copy is not this account\u2019s project"}
 `);
       if (result.notShared)
         process.stdout.write(`${renderNotShared(result.notShared)}