@mgiles/perk 2.3.0 → 3.0.0

package/shared/schemas/outputs/objective-stack-sync.schema.json

@@ -0,0 +1,184 @@
+{
+  "$defs": {
+    "ObjectiveOut": {
+      "additionalProperties": false,
+      "description": "The resolved (ACTIVE) objective identity; ``redirected_from`` names the originally\nrequested objective when a ``superseded_by`` chain redirected forward.",
+      "properties": {
+        "id": {
+          "title": "Id",
+          "type": "string"
+        },
+        "url": {
+          "title": "Url",
+          "type": "string"
+        },
+        "redirected_from": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "type": "null"
+            }
+          ],
+          "title": "Redirected From"
+        }
+      },
+      "required": [
+        "id",
+        "url",
+        "redirected_from"
+      ],
+      "title": "ObjectiveOut",
+      "type": "object"
+    },
+    "SyncedLayerOut": {
+      "additionalProperties": false,
+      "properties": {
+        "node_id": {
+          "title": "Node Id",
+          "type": "string"
+        },
+        "plan_id": {
+          "title": "Plan Id",
+          "type": "string"
+        },
+        "branch": {
+          "title": "Branch",
+          "type": "string"
+        },
+        "pr_number": {
+          "title": "Pr Number",
+          "type": "integer"
+        },
+        "before_sha": {
+          "title": "Before Sha",
+          "type": "string"
+        },
+        "after_sha": {
+          "title": "After Sha",
+          "type": "string"
+        }
+      },
+      "required": [
+        "node_id",
+        "plan_id",
+        "branch",
+        "pr_number",
+        "before_sha",
+        "after_sha"
+      ],
+      "title": "SyncedLayerOut",
+      "type": "object"
+    }
+  },
+  "additionalProperties": false,
+  "description": "The ``perk objective stack sync --json`` envelope (the \u00a78.49 result-arm table).\n``dry_run`` / ``adopted_node`` / ``continued`` / ``aborted`` are the additive\ncontrol-surface arms (declared last, order load-bearing).",
+  "properties": {
+    "success": {
+      "title": "Success",
+      "type": "boolean"
+    },
+    "objective": {
+      "$ref": "#/$defs/ObjectiveOut"
+    },
+    "operation_id": {
+      "anyOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "null"
+        }
+      ],
+      "title": "Operation Id"
+    },
+    "abandoned_operation_id": {
+      "anyOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "null"
+        }
+      ],
+      "title": "Abandoned Operation Id"
+    },
+    "no_op": {
+      "title": "No Op",
+      "type": "boolean"
+    },
+    "declined": {
+      "title": "Declined",
+      "type": "boolean"
+    },
+    "resumed": {
+      "title": "Resumed",
+      "type": "boolean"
+    },
+    "base_cascaded": {
+      "title": "Base Cascaded",
+      "type": "boolean"
+    },
+    "base_advanced": {
+      "title": "Base Advanced",
+      "type": "boolean"
+    },
+    "affected": {
+      "items": {
+        "$ref": "#/$defs/SyncedLayerOut"
+      },
+      "title": "Affected",
+      "type": "array"
+    },
+    "notes": {
+      "items": {
+        "type": "string"
+      },
+      "title": "Notes",
+      "type": "array"
+    },
+    "dry_run": {
+      "title": "Dry Run",
+      "type": "boolean"
+    },
+    "adopted_node": {
+      "anyOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "null"
+        }
+      ],
+      "title": "Adopted Node"
+    },
+    "continued": {
+      "title": "Continued",
+      "type": "boolean"
+    },
+    "aborted": {
+      "title": "Aborted",
+      "type": "boolean"
+    }
+  },
+  "required": [
+    "success",
+    "objective",
+    "operation_id",
+    "abandoned_operation_id",
+    "no_op",
+    "declined",
+    "resumed",
+    "base_cascaded",
+    "base_advanced",
+    "affected",
+    "notes",
+    "dry_run",
+    "adopted_node",
+    "continued",
+    "aborted"
+  ],
+  "title": "ObjectiveStackSyncOut",
+  "type": "object"
+}

package/shared/schemas/outputs/plan-save.schema.json

@@ -123,6 +123,18 @@
           ],
           "default": null,
           "title": "Base"
+        },
+        "delivery_lineage": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "type": "null"
+            }
+          ],
+          "default": null,
+          "title": "Delivery Lineage"
         }
       },
       "required": [

package/shared/schemas/outputs/pr-submit.schema.json

@@ -1,5 +1,70 @@
 {
   "$defs": {
+    "DeliveryOperationOut": {
+      "additionalProperties": false,
+      "description": "The typed delivery operation nested in a stacked cascade result.",
+      "properties": {
+        "kind": {
+          "title": "Kind",
+          "type": "string"
+        },
+        "operation_id": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "type": "null"
+            }
+          ],
+          "title": "Operation Id"
+        },
+        "abandoned_operation_id": {
+          "anyOf": [
+            {
+              "type": "string"
+            },
+            {
+              "type": "null"
+            }
+          ],
+          "title": "Abandoned Operation Id"
+        },
+        "resumed": {
+          "title": "Resumed",
+          "type": "boolean"
+        },
+        "no_op": {
+          "title": "No Op",
+          "type": "boolean"
+        },
+        "affected": {
+          "items": {
+            "$ref": "#/$defs/SyncedLayerOut"
+          },
+          "title": "Affected",
+          "type": "array"
+        },
+        "notes": {
+          "items": {
+            "type": "string"
+          },
+          "title": "Notes",
+          "type": "array"
+        }
+      },
+      "required": [
+        "kind",
+        "operation_id",
+        "abandoned_operation_id",
+        "resumed",
+        "no_op",
+        "affected",
+        "notes"
+      ],
+      "title": "DeliveryOperationOut",
+      "type": "object"
+    },
     "PlanHeaderUpdateOut": {
       "additionalProperties": false,
       "description": "The serialization boundary of the picked :class:`issue_backend.PlanHeaderUpdate` subset\n(field order load-bearing).",
@@ -18,6 +83,31 @@
       "title": "PlanHeaderUpdateOut",
       "type": "object"
     },
+    "StackRefOut": {
+      "additionalProperties": false,
+      "description": "The native-stack facts of a stacked submit (field order load-bearing): the stack\nnumber, its total size, and this PR's 1-based bottom\u2192top position.",
+      "properties": {
+        "number": {
+          "title": "Number",
+          "type": "integer"
+        },
+        "size": {
+          "title": "Size",
+          "type": "integer"
+        },
+        "position": {
+          "title": "Position",
+          "type": "integer"
+        }
+      },
+      "required": [
+        "number",
+        "size",
+        "position"
+      ],
+      "title": "StackRefOut",
+      "type": "object"
+    },
     "SubmitPrOut": {
       "additionalProperties": false,
       "description": "The serialization boundary of the picked :class:`github.PullRequest` subset\n(field order load-bearing).",
@@ -47,6 +137,46 @@
       ],
       "title": "SubmitPrOut",
       "type": "object"
+    },
+    "SyncedLayerOut": {
+      "additionalProperties": false,
+      "description": "One affected suffix layer in the delivery-operation block.",
+      "properties": {
+        "node_id": {
+          "title": "Node Id",
+          "type": "string"
+        },
+        "plan_id": {
+          "title": "Plan Id",
+          "type": "string"
+        },
+        "branch": {
+          "title": "Branch",
+          "type": "string"
+        },
+        "pr_number": {
+          "title": "Pr Number",
+          "type": "integer"
+        },
+        "before_sha": {
+          "title": "Before Sha",
+          "type": "string"
+        },
+        "after_sha": {
+          "title": "After Sha",
+          "type": "string"
+        }
+      },
+      "required": [
+        "node_id",
+        "plan_id",
+        "branch",
+        "pr_number",
+        "before_sha",
+        "after_sha"
+      ],
+      "title": "SyncedLayerOut",
+      "type": "object"
     }
   },
   "additionalProperties": false,
@@ -125,6 +255,48 @@
       },
       "title": "Conflicts",
       "type": "array"
+    },
+    "delivery": {
+      "anyOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "null"
+        }
+      ],
+      "title": "Delivery"
+    },
+    "stack": {
+      "anyOf": [
+        {
+          "$ref": "#/$defs/StackRefOut"
+        },
+        {
+          "type": "null"
+        }
+      ]
+    },
+    "operation_id": {
+      "anyOf": [
+        {
+          "type": "string"
+        },
+        {
+          "type": "null"
+        }
+      ],
+      "title": "Operation Id"
+    },
+    "operation": {
+      "anyOf": [
+        {
+          "$ref": "#/$defs/DeliveryOperationOut"
+        },
+        {
+          "type": "null"
+        }
+      ]
     }
   },
   "required": [
@@ -140,7 +312,11 @@
     "dry_run",
     "base",
     "mergeable",
-    "conflicts"
+    "conflicts",
+    "delivery",
+    "stack",
+    "operation_id",
+    "operation"
   ],
   "title": "PrSubmitOut",
   "type": "object"

package/extension/adapters/todoAdapterJuicesharp.ts

@@ -1,102 +0,0 @@
-// The FIRST 3rd-party todo adapter. A perk-owned, injection-only bridge that re-enables
-// `@juicesharp/rpiv-todo` as a REAL, selectable todo provider: it carries perk's implement-progress
-// discipline onto that package's checklist-overlay surface (the todo-seam mirror of
-// planAdapterTombell, which bridges the plan seam).
-//
-// INERT BY DEFAULT. This shim is ALWAYS registered in index.ts but does nothing unless the resolved
-// `[providers] todo` selection is `juicesharp-todo` (read fresh per-event via the same
-// `resolvedTodoProviderId` the reference checkpoints provider uses) AND the session is an active
-// workflow (`active_plan_ref != null`, the same gate the reference checkpoints provider seeds on).
-// On any non-juicesharp selection it injects nothing and only strips its own stale marker — zero
-// behavior change on the default path.
-//
-// WHAT IT DOES (and does NOT do):
-//   - It injects a hidden (`display:false`, once-only: branch-scan dedup'd on the marker)
-//     `perk:todo-adapter-juicesharp` context that tells the
-//     model the foreign `@juicesharp/rpiv-todo` checklist overlay is the sole progress surface here
-//     (perk's own checkpoints stepped aside) and directs it to carry perk's
-//     implement-progress DISCIPLINE onto that overlay: seed it from the plan body's `## Steps` and
-//     mark each item complete in order — the same gather-then-advance flow perk's checkpoints embody.
-//   - It does NOT own, replace, or duplicate the read-only gate (Invariant 1) and NEVER calls
-//     `setActiveTools` / registers a `tool_call` handler (the todo seam composes no shared primitive).
-//   - It does NOT write perk's `perk:checkpoint` entry and does NOT revive perk's deferred marker
-//     scanner (Correction 2): that entry is a transient TS-only progress overlay nothing downstream
-//     consumes, and perk's render + scanner are already deferred. Re-populating it would
-//     be dead duplication — the foreign overlay is the sole, uncontested progress surface.
-//   - It adds NO registration-time vacating (Correction 1): unlike the plan seam (where perk and the
-//     foreign package both register `/plan`, forcing Pi to suffix the duplicate), the todo seam has
-//     no command-name collision — perk registers `/checkpoints`; the foreign overlay registers its
-//     own differently-named command(s) — so the runtime deferral is already sufficient.
-
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-import { resolvedTodoProviderId } from "../checkpoints/checkpoints.ts";
-import { render } from "../substrate/prompts.ts";
-import { JUICESHARP_TODO_PROVIDER_ID } from "../substrate/providers.ts";
-import { branchCarries, branchOf, rebuildWorkflowState } from "../substrate/workflowState.ts";
-
-/** The juicesharp todo-adapter bridge customType (distinct from checkpoints' `perk:checkpoint`). */
-export const TODO_ADAPTER_JUICESHARP_CONTEXT_TYPE = "perk:todo-adapter-juicesharp";
-const TODO_ADAPTER_JUICESHARP_MARKER = "[TODO ADAPTER: JUICESHARP]";
-
-/**
- * The bridge prompt: carries perk's implement-progress discipline onto the foreign checklist
- * overlay. Prompting, NOT enforcement (perk's checkpoint scanner is deferred under this selection).
- * Durable anchors only, no line numbers.
- */
-export const TODO_ADAPTER_JUICESHARP_CONTEXT = render("contexts/adapters/juicesharp-todo.md", {
-  marker: TODO_ADAPTER_JUICESHARP_MARKER,
-});
-
-/** Whether the foreign `juicesharp-todo` provider is the selected todo provider for `cwd`. */
-export function isJuicesharpTodoSelected(cwd: string): boolean {
-  return resolvedTodoProviderId(cwd) === JUICESHARP_TODO_PROVIDER_ID;
-}
-
-/**
- * Register the juicesharp todo adapter: an injection-only bridge, inert unless `[providers] todo =
- * "juicesharp-todo"` AND the session is an active workflow. It NEVER touches tool gating /
- * setActiveTools (Invariant 1) and never throws.
- */
-export function registerTodoAdapterJuicesharp(pi: ExtensionAPI): void {
-  // Inject the bridge context while the foreign juicesharp-todo provider is selected AND a workflow
-  // is active (active_plan_ref != null) — the same active-workflow scope the reference checkpoints
-  // provider seeds on, so the bridge never reaches planning/objective sessions (display:false).
-  pi.on("before_agent_start", async (_event, ctx) => {
-    if (!isJuicesharpTodoSelected(ctx.cwd)) return;
-    const branch = branchOf(ctx);
-    if (rebuildWorkflowState(branch).active_plan_ref == null) return;
-    // Once-only: injected customs persist to the branch, so a live copy suppresses re-injection;
-    // compaction dropping it makes the scan come up clean and the next turn re-injects.
-    if (branchCarries(branch, TODO_ADAPTER_JUICESHARP_MARKER)) return;
-    return {
-      message: {
-        customType: TODO_ADAPTER_JUICESHARP_CONTEXT_TYPE,
-        content: TODO_ADAPTER_JUICESHARP_CONTEXT,
-        display: false,
-      },
-    };
-  });
-
-  // Strip the stale bridge marker from context when juicesharp-todo is no longer selected (same
-  // hygiene the sibling adapters apply), so it never lingers across a deselect.
-  pi.on("context", async (event, ctx) => {
-    if (isJuicesharpTodoSelected(ctx.cwd)) return;
-    return {
-      messages: event.messages.filter((m) => {
-        const msg = m as { customType?: string; role?: string; content?: unknown };
-        if (msg.customType === TODO_ADAPTER_JUICESHARP_CONTEXT_TYPE) return false;
-        if (msg.role !== "user") return true;
-        const content = msg.content;
-        if (typeof content === "string") return !content.includes(TODO_ADAPTER_JUICESHARP_MARKER);
-        if (Array.isArray(content)) {
-          return !content.some(
-            (c) =>
-              (c as { type?: string; text?: string }).type === "text" &&
-              ((c as { text?: string }).text ?? "").includes(TODO_ADAPTER_JUICESHARP_MARKER),
-          );
-        }
-        return true;
-      }),
-    };
-  });
-}