@popoverai/dotrequirements 0.37.1 → 0.37.2

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.1",
+  "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": {