@popoverai/dotrequirements 0.37.0 → 0.37.2

package/dist/commands/solution/index.js

@@ -97,8 +97,11 @@ const PR_STATUS_HELP = `
 Reads the pull request of the outcome you are at, or of the outcome it awaits that
 is kept on one, as GitHub reports it. The merge state is GitHub's word for whether
 it can merge: clean means it can, with every required check passing and no
-conflict. Failed checks come with a link to them; checks still running are named.
-Reading it changes nothing.
+conflict. Failed checks come with a link to them and what they reported, as GitHub
+gives it: their output and annotations, or the end of their job's log. Checks still
+running are named. Each review follows, with its verdict and its comments, from the
+repository's owners, members and collaborators and the apps installed on it; those
+by anyone else are left out and counted. Reading it changes nothing.
 `;
 /** What `pr open` does, on its `--help`, where `author --help` sends the agent (PULL-2). */
 const PULL_REQUEST_HELP = `

package/dist/commands/solution/pull-request.d.ts

@@ -18,12 +18,15 @@ export type PullRequestState = {
         name: string;
         result: string;
         link: string | null;
+        /** What the check reported, as GitHub gave it, or why it couldn't (PULL-4.7). */
+        reported?: unknown;
     }>;
     running: Array<{
         name: string;
         link: string | null;
     }>;
     head: string;
+    reviews?: unknown;
 };
 /** Where the pull request stands, as a reviewer reads it (PULL-4). */
 export declare function describeState(state: PullRequestState): string;

package/dist/commands/solution/pull-request.js

@@ -27,7 +27,14 @@ async function stateFor(where) {
     }
     if (!said.state)
         throw new Error("The app gave no pull request.");
-    return said.state;
+    return said.reviews === undefined
+        ? said.state
+        : { ...said.state, reviews: said.reviews };
+}
+/** What GitHub reported, as it gave it, indented under the line it belongs to. */
+function asGiven(reported, indent) {
+    const text = typeof reported === "string" ? reported : JSON.stringify(reported, null, 2);
+    return text.split("\n").map((line) => `${indent}${line}`);
 }
 /** Where the pull request stands, as a reviewer reads it (PULL-4). */
 export function describeState(state) {
@@ -40,6 +47,10 @@ export function describeState(state) {
     ];
     for (const check of state.failed) {
         lines.push(`  failed: ${check.name} (${check.result})${check.link ? ` ${check.link}` : ""}`);
+        // PULL-4.7: what it reported, as GitHub gave it.
+        if (check.reported !== undefined) {
+            lines.push("    It reported:", ...asGiven(check.reported, "      "));
+        }
     }
     for (const check of state.running) {
         lines.push(`  still running: ${check.name}${check.link ? ` ${check.link}` : ""}`);
@@ -48,6 +59,10 @@ export function describeState(state) {
         // PULL-4.3.
         lines.push("GitHub hasn't yet worked out whether it can merge: read it again shortly.");
     }
+    // PULL-4.8: each review, with its verdict and its comments.
+    if (state.reviews !== undefined) {
+        lines.push("Reviews and their comments, as GitHub gave them:", ...asGiven(state.reviews, "  "));
+    }
     return lines.join("\n");
 }
 /**

package/dist/rehearsal/derive.d.ts

@@ -7,7 +7,9 @@ import type { Plan, PlanPart } from "./types.js";
  * So a scenario naming Scores alone, of the default plan, runs the scores
  * with the pitch an input. An outcome on one left out is kept on that
  * one's surface instead. The checkpoints on what is kept stay, presenting
- * what is still there; loops go, since part of a plan starts or ends
+ * what is still there, and one named into a phase the part doesn't have
+ * names none; a pull-request surface's ready for review or draft on an
+ * outcome left out goes; and loops go, since part of a plan starts or ends
  * nothing.
  *
  * The one exception is the outcome that makes the solution exist: the

package/dist/rehearsal/derive.js

@@ -35,7 +35,9 @@ const worksFrom = (plan, outcome) => [
  * So a scenario naming Scores alone, of the default plan, runs the scores
  * with the pitch an input. An outcome on one left out is kept on that
  * one's surface instead. The checkpoints on what is kept stay, presenting
- * what is still there; loops go, since part of a plan starts or ends
+ * what is still there, and one named into a phase the part doesn't have
+ * names none; a pull-request surface's ready for review or draft on an
+ * outcome left out goes; and loops go, since part of a plan starts or ends
  * nothing.
  *
  * The one exception is the outcome that makes the solution exist: the
@@ -85,18 +87,38 @@ export function planForOutcomes(plan, names) {
     const known = new Set([...kept, ...inputs]);
     const keptOn = (on) => kept.has(on) ||
         [...kept].some((name) => surfaceOf(plan, name) === on && name !== on);
+    // A checkpoint named into a phase the part no longer has sits after what
+    // it presents, as it would naming none.
+    const placed = (checkpoint) => {
+        if (checkpoint.phase === undefined ||
+            phases.some((phase) => phase.name === checkpoint.phase)) {
+            return checkpoint;
+        }
+        const { phase: _phase, ...rest } = checkpoint;
+        return rest;
+    };
     const checkpoints = plan.checkpoints.flatMap((checkpoint) => {
         if (!keptOn(checkpoint.on))
             return [];
         if (!checkpoint.presents?.length)
-            return [checkpoint];
+            return [placed(checkpoint)];
         const presents = checkpoint.presents.filter((name) => known.has(name));
-        return presents.length > 0 ? [{ ...checkpoint, presents }] : [];
+        return presents.length > 0 ? [placed({ ...checkpoint, presents })] : [];
     });
     return {
         format: "outcomes",
         inputs,
-        surfaces: plan.surfaces,
+        // A pull request's moment on an outcome left out never comes (PULL-7.3).
+        surfaces: plan.surfaces.map((surface) => {
+            const own = { ...surface };
+            if (surface.readyForReview && !kept.has(surface.readyForReview.outcome)) {
+                delete own.readyForReview;
+            }
+            if (surface.draft && !kept.has(surface.draft.outcome)) {
+                delete own.draft;
+            }
+            return own;
+        }),
         phases,
         checkpoints,
         agents: plan.agents,

package/dist/rehearsal/types.d.ts

@@ -93,6 +93,16 @@ export type Plan = {
     surfaces: Array<{
         name: string;
         kind: string;
+        /** When a pull-request surface's pull request asks for review, or goes back to draft (PULL-7). */
+        readyForReview?: {
+            outcome: string;
+            when: string;
+        };
+        draft?: {
+            outcome: string;
+            when: string;
+        };
+        watchedBy?: string;
     }>;
     phases: Array<{
         name: string;
@@ -103,6 +113,7 @@ export type Plan = {
         name: string;
         on: string;
         presents?: string[];
+        phase?: string;
     }>;
     agents: Array<{
         name: string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@popoverai/dotrequirements",
-  "version": "0.37.0",
+  "version": "0.37.2",
   "description": "Requirements as testable, human-readable data — CLI and test harness for spec-driven development with AI coding agents",
   "type": "module",
   "bin": {