@difflab/pi 0.7.1 → 0.7.2

package/dist/tools/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,iCAAiC,CAAC;AACpF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAY/C,OAAO,EAAE,WAAW,EAAE,MAAM,MAAM,CAAC;AACnC,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AACtC,OAAO,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AACxC,OAAO,EAAE,sBAAsB,EAAE,MAAM,UAAU,CAAC;AAClD,OAAO,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAC1C,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAC7C,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAC9D,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AAIjD,wBAAgB,aAAa,CAC3B,EAAE,EAAE,IAAI,CAAC,YAAY,EAAE,QAAQ,GAAG,iBAAiB,CAAC,EACpD,KAAK,EAAE,cAAc,GACpB,SAAS,cAAc,EAAE,CAY3B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/tools/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,cAAc,EAAE,MAAM,iCAAiC,CAAC;AAWpF,OAAO,EAAE,WAAW,EAAE,MAAM,MAAM,CAAC;AACnC,OAAO,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AACtC,OAAO,EAAE,cAAc,EAAE,MAAM,QAAQ,CAAC;AACxC,OAAO,EAAE,sBAAsB,EAAE,MAAM,UAAU,CAAC;AAClD,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAC;AAC7C,OAAO,EAAE,eAAe,EAAE,kBAAkB,EAAE,MAAM,SAAS,CAAC;AAC9D,OAAO,EAAE,kBAAkB,EAAE,MAAM,aAAa,CAAC;AAIjD,wBAAgB,aAAa,CAAC,EAAE,EAAE,IAAI,CAAC,YAAY,EAAE,QAAQ,GAAG,iBAAiB,CAAC,GAAG,SAAS,cAAc,EAAE,CAW7G"}

package/dist/tools/index.js

@@ -1083,127 +1083,12 @@ function createDiffpiReloadTool(pi) {
   });
 }
 
-// src/tools/modes.ts
-import { defineTool as defineTool5 } from "@earendil-works/pi-coding-agent";
-import { z as z6 } from "zod";
-var emptyParametersSchema = z6.object({});
-var emptyParameters = z6.toJSONSchema(emptyParametersSchema, { io: "input" });
-var listParametersSchema = z6.object({
-  includeSkills: z6.boolean().optional().describe("Include skill-owned agents using skill:agent ids.")
-});
-var listParameters = z6.toJSONSchema(listParametersSchema, { io: "input" });
-var setParametersSchema = z6.object({
-  agent: zx.text.describe("Inline agent id from diffpi_modes_list.")
-});
-var setParameters = z6.toJSONSchema(setParametersSchema, { io: "input" });
-function createModeTools(controller) {
-  return [
-    defineTool5({
-      name: "diffpi_modes_status",
-      label: "diffpi modes status",
-      description: "Report the actual runtime model, thinking level, and active tools for capability inspection.",
-      promptSnippet: "Inspect actual post-compaction inline mode capabilities before claiming them.",
-      parameters: emptyParameters,
-      executionMode: "parallel",
-      async execute(_toolCallId, input, _signal, _onUpdate, ctx) {
-        emptyParametersSchema.parse(input);
-        const status = controller.getStatus(ctx);
-        return {
-          content: [{ type: "text", text: JSON.stringify(status) }],
-          details: status
-        };
-      }
-    }),
-    defineTool5({
-      name: "diffpi_modes_list",
-      label: "diffpi modes list",
-      description: "List inline agents shared with the subagent plugin, optionally including skill-owned agents.",
-      promptSnippet: "List inline agents before selecting one when the requested agent is unclear",
-      promptGuidelines: [
-        "Call diffpi_modes_list when the user asks which inline agents are available.",
-        "Set includeSkills to true only when the user asks for skill agents or runs /skill:mode --include-skills.",
-        "Inline mode applies the profile prompt, first available preferred model, thinking level, and available tools."
-      ],
-      parameters: listParameters,
-      executionMode: "parallel",
-      async execute(_toolCallId, input, _signal, _onUpdate, ctx) {
-        const params = listParametersSchema.parse(input);
-        const catalog = await controller.list(ctx, { includeSkills: params.includeSkills });
-        return {
-          content: [{ type: "text", text: formatCatalog(catalog, controller.getActive()?.id) }],
-          details: { active: controller.getActive()?.id, catalog }
-        };
-      }
-    }),
-    defineTool5({
-      name: "diffpi_modes_set",
-      label: "diffpi modes set",
-      description: "Set a validated available agent as the inline behavioral agent for subsequent chat turns.",
-      promptSnippet: "Set the inline behavioral agent only after the user chooses one",
-      promptGuidelines: [
-        "Call diffpi_modes_set only after the user explicitly selects an agent.",
-        "Use the exact skill:agent id for a skill-owned agent.",
-        "The selected prompt takes effect on the next model turn."
-      ],
-      parameters: setParameters,
-      executionMode: "sequential",
-      async execute(_toolCallId, input, _signal, _onUpdate, ctx) {
-        const params = setParametersSchema.parse(input);
-        const result = await controller.set(params.agent, ctx);
-        if (!result.ok)
-          throw new Error(result.message);
-        return {
-          content: [{ type: "text", text: `${result.message} The prompt takes effect on the next turn.` }],
-          details: { active: result.active }
-        };
-      }
-    }),
-    defineTool5({
-      name: "diffpi_modes_unset",
-      label: "diffpi modes unset",
-      description: "Clear the inline behavioral agent and restore default Pi prompting for subsequent turns.",
-      promptSnippet: "Clear the inline agent when the user asks for default behavior",
-      promptGuidelines: [
-        "Call diffpi_modes_unset only when the user explicitly asks to clear the active inline agent."
-      ],
-      parameters: emptyParameters,
-      executionMode: "sequential",
-      async execute(_toolCallId, input, _signal, _onUpdate, ctx) {
-        emptyParametersSchema.parse(input);
-        const result = await controller.unset(ctx);
-        return {
-          content: [{ type: "text", text: result.message }],
-          details: { active: controller.getActive()?.id }
-        };
-      }
-    })
-  ];
-}
-function formatCatalog(catalog, active) {
-  const lines = [`Active inline agent: ${active ?? "default"}.`, "", "Available inline agents:"];
-  for (const mode of catalog.modes) {
-    const runtime = [mode.modelPreferences[0], mode.thinkingLevel].filter(Boolean).join(", ");
-    lines.push(`- ${mode.id} [${mode.promptStrategy}${runtime ? `; ${runtime}` : ""}] — ${sanitize(mode.description)} (${mode.source})`);
-  }
-  if (catalog.diagnostics.length > 0) {
-    lines.push("", "Skipped agent files:");
-    for (const diagnostic of catalog.diagnostics)
-      lines.push(`- ${sanitize(diagnostic)}`);
-  }
-  lines.push("", "Inline mode applies the profile prompt, preferred available model, thinking level, and tool set.");
-  return lines.join(`
-`);
-}
-function sanitize(value) {
-  return value.replace(/[\r\n\t]+/g, " ").replace(/\s+/g, " ").trim();
-}
-
 // src/tools/review.ts
 import { existsSync as existsSync2 } from "node:fs";
 import { mkdir as mkdir4, readdir as readdir2, readFile as readFile10, rename as rename2, stat as stat2, unlink as unlink2, writeFile as writeFile5 } from "node:fs/promises";
 import { join as join12 } from "node:path";
-import { defineTool as defineTool6 } from "@earendil-works/pi-coding-agent";
-import { z as z10 } from "zod";
+import { defineTool as defineTool5 } from "@earendil-works/pi-coding-agent";
+import { z as z9 } from "zod";
 
 // src/extensions/misex.ts
 import { mkdir as mkdir3, readFile as readFile5, writeFile as writeFile2 } from "node:fs/promises";
@@ -1436,16 +1321,16 @@ async function discoverMiseTasks(cwd) {
 }
 
 // src/review/types.ts
-import { z as z7 } from "zod";
-var severitySchema = z7.enum(["BLOCKING", "CONSIDER", "NOTE"]);
-var findingSchema = z7.object({
-  file: z7.string().min(1),
-  line: z7.number().int().nonnegative(),
+import { z as z6 } from "zod";
+var severitySchema = z6.enum(["BLOCKING", "CONSIDER", "NOTE"]);
+var findingSchema = z6.object({
+  file: z6.string().min(1),
+  line: z6.number().int().nonnegative(),
   severity: severitySchema,
-  body: z7.string().min(1),
-  reference: z7.string().optional().default("")
+  body: z6.string().min(1),
+  reference: z6.string().optional().default("")
 });
-var findingsSchema = z7.array(findingSchema);
+var findingsSchema = z6.array(findingSchema);
 function reviewSlug(input) {
   return input.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 50);
 }
@@ -1518,17 +1403,17 @@ function renderCommentBody(finding) {
   return `${prefix}${finding.body}${reference}`;
 }
 // src/review/review-markdown.ts
-import { z as z8 } from "zod";
-var reviewThreadRecordSchema = z8.object({
-  id: z8.string().min(1),
-  file: z8.string().nullable().optional().transform((value) => value ?? undefined),
-  line: z8.number().int().positive().nullable().optional().transform((value) => value ?? undefined),
-  body: z8.string(),
-  author: z8.string().optional(),
-  resolved: z8.boolean(),
-  addressed: z8.boolean().optional(),
-  question: z8.boolean(),
-  replies: z8.array(z8.string()).optional()
+import { z as z7 } from "zod";
+var reviewThreadRecordSchema = z7.object({
+  id: z7.string().min(1),
+  file: z7.string().nullable().optional().transform((value) => value ?? undefined),
+  line: z7.number().int().positive().nullable().optional().transform((value) => value ?? undefined),
+  body: z7.string(),
+  author: z7.string().optional(),
+  resolved: z7.boolean(),
+  addressed: z7.boolean().optional(),
+  question: z7.boolean(),
+  replies: z7.array(z7.string()).optional()
 });
 function renderReviewDoc(input) {
   const anchored = dedupeFindings(input.findings).filter((finding) => finding.line > 0);
@@ -1615,7 +1500,7 @@ function parseThreadArtifact(content) {
   let threads;
   try {
     const decoded = JSON.parse(Buffer.from(payload, "base64url").toString("utf8"));
-    threads = z8.array(reviewThreadRecordSchema).parse(decoded);
+    threads = z7.array(reviewThreadRecordSchema).parse(decoded);
   } catch {
     throw new Error("Cannot parse the Diffpi thread artifact payload.");
   }
@@ -2286,14 +2171,14 @@ function createLocalReviewBackend(options) {
 import { createHash as createHash2 } from "node:crypto";
 import { readFile as readFile8, rename, writeFile as writeFile4 } from "node:fs/promises";
 import { join as join10 } from "node:path";
-import { z as z9 } from "zod";
-var reviewPublicationStateSchema = z9.object({
-  target: z9.string().optional(),
-  bodies: z9.array(z9.string()).default([]),
-  comments: z9.array(z9.string()).default([]),
-  stagedComments: z9.array(z9.string()).default([]),
-  replies: z9.array(z9.string()).default([]),
-  overlayPath: z9.string().optional()
+import { z as z8 } from "zod";
+var reviewPublicationStateSchema = z8.object({
+  target: z8.string().optional(),
+  bodies: z8.array(z8.string()).default([]),
+  comments: z8.array(z8.string()).default([]),
+  stagedComments: z8.array(z8.string()).default([]),
+  replies: z8.array(z8.string()).default([]),
+  overlayPath: z8.string().optional()
 });
 function reviewBodyFingerprint(body) {
   return digest(body);
@@ -2317,7 +2202,7 @@ async function loadReviewPublicationState(cwd, vcs, number, homeDir) {
     if (error.code === "ENOENT") {
       return { path, state: { target, bodies: [], comments: [], stagedComments: [], replies: [] } };
     }
-    if (error instanceof SyntaxError || error instanceof z9.ZodError) {
+    if (error instanceof SyntaxError || error instanceof z8.ZodError) {
       throw new Error(`Cannot parse review publication state: ${path}`);
     }
     throw error;
@@ -2369,41 +2254,41 @@ async function readOptionalFile(path) {
 }
 
 // src/tools/review.ts
-var contextSchema = z10.object({
-  cwd: z10.string().optional(),
-  target: z10.string().optional(),
-  workingTree: z10.boolean().optional()
+var contextSchema = z9.object({
+  cwd: z9.string().optional(),
+  target: z9.string().optional(),
+  workingTree: z9.boolean().optional()
 });
-var localSchema = contextSchema.extend({ local: z10.boolean().optional() });
+var localSchema = contextSchema.extend({ local: z9.boolean().optional() });
 var openSchema = localSchema.extend({
-  title: z10.string().optional(),
-  intent: z10.string().optional(),
-  issueUrl: z10.string().url().optional(),
-  base: z10.string().optional()
+  title: z9.string().optional(),
+  intent: z9.string().optional(),
+  issueUrl: z9.string().url().optional(),
+  base: z9.string().optional()
 });
 var submitSchema = localSchema.extend({
   findings: findingsSchema,
-  overallIssues: z10.array(z10.string()).optional(),
-  notVerified: z10.array(z10.string()).optional(),
-  title: z10.string().optional()
+  overallIssues: z9.array(z9.string()).optional(),
+  notVerified: z9.array(z9.string()).optional(),
+  title: z9.string().optional()
 });
 var addCommentSchema = localSchema.extend({
-  body: z10.string().min(1),
-  file: z10.string().min(1),
-  line: z10.number().int().positive(),
-  side: z10.enum(["LEFT", "RIGHT"]).optional()
+  body: z9.string().min(1),
+  file: z9.string().min(1),
+  line: z9.number().int().positive(),
+  side: z9.enum(["LEFT", "RIGHT"]).optional()
 });
 var respondSchema = localSchema.extend({
-  threadId: z10.string().min(1),
-  body: z10.string().min(1),
-  question: z10.boolean().optional(),
-  resolve: z10.boolean().optional()
+  threadId: z9.string().min(1),
+  body: z9.string().min(1),
+  question: z9.boolean().optional(),
+  resolve: z9.boolean().optional()
 });
 var publishSchema = localSchema.extend({
-  status: z10.enum(["COMMENT", "APPROVE", "REQUEST_CHANGES", "CLOSE"]).optional()
+  status: z9.enum(["COMMENT", "APPROVE", "REQUEST_CHANGES", "CLOSE"]).optional()
 });
 var completeSchema = localSchema.extend({
-  action: z10.enum(["approve", "reject", "abandon"]).optional()
+  action: z9.enum(["approve", "reject", "abandon"]).optional()
 });
 function hasReviewDraft(comments, body) {
   return comments.length > 0 || body.trim().length > 0;
@@ -2477,7 +2362,7 @@ async function workingTreeDiff(cwd) {
 }
 function createReviewTools() {
   return [
-    defineTool6({
+    defineTool5({
       name: "review_context",
       label: "review context",
       description: "Orient to the target, backend, forge, environment, shared store, PR/MR, and tuicr session.",
@@ -2516,7 +2401,7 @@ function createReviewTools() {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_status",
       label: "review status",
       description: "Report local and remote review lifecycle status without requiring a forge or tuicr.",
@@ -2531,7 +2416,7 @@ function createReviewTools() {
         return result2(status.text, status.details);
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_new",
       label: "review new",
       description: "Create a new local tuicr review or a remote draft PR/MR and open it in tuicr.",
@@ -2574,7 +2459,7 @@ ${openMessage}`, {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_open",
       label: "review open",
       description: "Open an existing remote PR/MR in the system browser; never creates one.",
@@ -2599,7 +2484,7 @@ ${openMessage}`, {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_edit",
       label: "review edit",
       description: "Open an existing local tuicr session or remote PR/MR in tuicr without generating findings.",
@@ -2623,7 +2508,7 @@ ${openMessage}`, {
         return launchLocalReview(review, false);
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_diff",
       label: "review diff",
       description: "Fetch the target PR/MR diff or auto-detected local working-tree diff.",
@@ -2646,7 +2531,7 @@ ${openMessage}`, {
         return result2(diff || "Empty diff.", { diff, pr: review.pr });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_gates",
       label: "review gates",
       description: "Run format, lint, test, conventional-subject, and available CI checks.",
@@ -2676,7 +2561,7 @@ ${openMessage}`, {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_submit",
       label: "review submit",
       description: "Write the review artifact and stage comments in the selected local or remote backend.",
@@ -2739,7 +2624,7 @@ ${openMessage}`, {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_add_comment",
       label: "review add comment",
       description: "Add one provenance-marked comment through tuicr or the remote pending-review backend.",
@@ -2778,7 +2663,7 @@ ${openMessage}`, {
         return result2(`Draft comment added to #${review.pr.number}.`, { pr: review.pr, comment });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_comments",
       label: "review comments",
       description: "Pull review threads or local tuicr comments and write a target-named artifact.",
@@ -2809,7 +2694,7 @@ ${openMessage}`, {
 `) || "No comments.", { artifact, threads });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_respond",
       label: "review respond",
       description: "Post local question replies to tuicr and record their publish overlay, or reply to remote threads.",
@@ -2870,7 +2755,7 @@ ${params.body}`, {
         });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_publish",
       label: "review publish",
       description: "Publish pending remote draft PR/MR review comments and status, or promote a local tuicr draft before publishing.",
@@ -2886,7 +2771,7 @@ ${params.body}`, {
         return publishResolvedReview(review, params, modelRoute(ctx));
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_complete",
       label: "review complete",
       description: "Approve, reject, or abandon a remote review, or archive a local review artifact.",
@@ -2920,7 +2805,7 @@ ${params.body}`, {
         return publishResolvedReview(review, { ...params, status }, modelRoute(ctx));
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_merge",
       label: "review merge",
       description: "Squash-merge an open, ready GitHub PR through gh after checking its conventional subject.",
@@ -2929,10 +2814,10 @@ ${params.body}`, {
         "Do not query GitHub through MCP or duplicate readiness checks before calling review_merge.",
         "This is intentionally GitHub-only until GitLab merge support is added."
       ],
-      parameters: parameters(contextSchema.extend({ subject: z10.string().optional() })),
+      parameters: parameters(contextSchema.extend({ subject: z9.string().optional() })),
       executionMode: "sequential",
       async execute(_id, input) {
-        const params = contextSchema.extend({ subject: z10.string().optional() }).parse(input);
+        const params = contextSchema.extend({ subject: z9.string().optional() }).parse(input);
         const review = await resolveReviewContext(resolveWorkingDirectory(params), params.target);
         if (review.vcs.provider !== "github" || !review.forge)
           return result2("review_merge currently supports GitHub only.");
@@ -2946,7 +2831,7 @@ ${params.body}`, {
         return result2(`Merged #${review.pr.number} with subject: ${subject}.`, { pr: review.pr, guard });
       }
     }),
-    defineTool6({
+    defineTool5({
       name: "review_launch_ui",
       label: "review launch UI",
       description: "Open a tuicr UI in a mux tab, configure Zed, or return the command to run.",
@@ -2990,7 +2875,7 @@ async function syncLocalThreadArtifact(path, title, sessionSlug, threads) {
   return merged;
 }
 function parameters(schema) {
-  return z10.toJSONSchema(schema, { io: "input" });
+  return z9.toJSONSchema(schema, { io: "input" });
 }
 function resolveWorkingDirectory(params) {
   return params.cwd ?? process.cwd();
@@ -3348,8 +3233,8 @@ function artifactNameMatches(name, suffix) {
 }
 
 // src/tools/setup.ts
-import { defineTool as defineTool7 } from "@earendil-works/pi-coding-agent";
-import { z as z12 } from "zod";
+import { defineTool as defineTool6 } from "@earendil-works/pi-coding-agent";
+import { z as z11 } from "zod";
 
 // src/setup.ts
 import { parseFrontmatter as parseFrontmatter2 } from "@earendil-works/pi-coding-agent";
@@ -3359,7 +3244,7 @@ import { basename as basename5, join as join16 } from "node:path";
 
 // src/config.ts
 import { parseFrontmatter } from "@earendil-works/pi-coding-agent";
-import { z as z11 } from "zod";
+import { z as z10 } from "zod";
 import { homedir as homedir5 } from "node:os";
 import { join as join13 } from "node:path";
 
@@ -3380,11 +3265,11 @@ function isMissingPath(error) {
 
 // src/config.ts
 var modelReferenceSchema = zx.text;
-var agentConfigSchema = z11.object({
-  models: z11.array(modelReferenceSchema).optional()
+var agentConfigSchema = z10.object({
+  models: z10.array(modelReferenceSchema).optional()
 }).strict();
-var diffpiConfigSchema = z11.object({
-  agents: z11.record(z11.string(), agentConfigSchema).optional()
+var diffpiConfigSchema = z10.object({
+  agents: z10.record(z10.string(), agentConfigSchema).optional()
 }).strict();
 function diffpiConfigPaths(homeDir = homedir5()) {
   const directory = join13(homeDir, ".difflab", "diffpi");
@@ -3923,14 +3808,14 @@ function getRecord(value) {
 }
 
 // src/tools/setup.ts
-var setupParametersSchema = z12.object({
-  issueTracker: z12.enum(["none", "linear", "jira"]).default("none").describe("Issue tracker MCP server to configure. Use none unless the user explicitly selects Linear or Jira."),
-  forges: z12.array(z12.enum(["github", "gitlab"])).default([]).describe("Hosted VCS MCP servers to configure. Installs matching CLIs and registers each MCP server."),
-  forge: z12.enum(["none", "github", "gitlab"]).optional().describe("Deprecated single-host alias. Prefer forges for one or more VCS MCP servers."),
-  bindZedKey: z12.boolean().default(false).describe("Opt in to a Zed keybinding for the tuicr review task.")
+var setupParametersSchema = z11.object({
+  issueTracker: z11.enum(["none", "linear", "jira"]).default("none").describe("Issue tracker MCP server to configure. Use none unless the user explicitly selects Linear or Jira."),
+  forges: z11.array(z11.enum(["github", "gitlab"])).default([]).describe("Hosted VCS MCP servers to configure. Installs matching CLIs and registers each MCP server."),
+  forge: z11.enum(["none", "github", "gitlab"]).optional().describe("Deprecated single-host alias. Prefer forges for one or more VCS MCP servers."),
+  bindZedKey: z11.boolean().default(false).describe("Opt in to a Zed keybinding for the tuicr review task.")
 });
-var setupParameters = z12.toJSONSchema(setupParametersSchema, { io: "input" });
-var diffpiSetupTool = defineTool7({
+var setupParameters = z11.toJSONSchema(setupParametersSchema, { io: "input" });
+var diffpiSetupTool = defineTool6({
   name: "diffpi_setup",
   label: "diffpi setup",
   description: "Install or repair the @difflab/pi environment. This mutates user-level tool installations and configuration files.",
@@ -3959,7 +3844,7 @@ var diffpiSetupTool = defineTool7({
     return formatResult(result, "Setup complete.");
   }
 });
-var diffpiValidateTool = defineTool7({
+var diffpiValidateTool = defineTool6({
   name: "diffpi_validate",
   label: "diffpi validate",
   description: "Inspect the @difflab/pi environment without installing software or changing configuration files.",
@@ -4001,20 +3886,20 @@ ${lines.join(`
 }
 
 // src/tools/templates.ts
-import { defineTool as defineTool8 } from "@earendil-works/pi-coding-agent";
-import { z as z13 } from "zod";
-var templateSchema = z13.object({
-  name: z13.string().min(1),
-  variables: z13.record(z13.string(), z13.string()).optional(),
-  homeDir: z13.string().optional()
+import { defineTool as defineTool7 } from "@earendil-works/pi-coding-agent";
+import { z as z12 } from "zod";
+var templateSchema = z12.object({
+  name: z12.string().min(1),
+  variables: z12.record(z12.string(), z12.string()).optional(),
+  homeDir: z12.string().optional()
 });
-var diffpiTemplateTool = defineTool8({
+var diffpiTemplateTool = defineTool7({
   name: "diffpi_template",
   label: "diffpi template",
   description: "Load a bundled Diffpi template or a user override and render named variables.",
   promptSnippet: "Use diffpi_template for package workflow templates",
   promptGuidelines: ["User overrides live under ~/.difflab/diffpi/templates."],
-  parameters: z13.toJSONSchema(templateSchema, { io: "input" }),
+  parameters: z12.toJSONSchema(templateSchema, { io: "input" }),
   executionMode: "parallel",
   async execute(_id, input) {
     const params = templateSchema.parse(input);
@@ -4028,7 +3913,7 @@ var diffpiTemplateTool = defineTool8({
 });
 
 // src/tools/index.ts
-function createPiTools(pi, modes) {
+function createPiTools(pi) {
   return [
     diffpiSetupTool,
     diffpiValidateTool,
@@ -4037,13 +3922,11 @@ function createPiTools(pi, modes) {
     diffpiTemplateTool,
     planVerifyTool,
     watchCiTool,
-    ...createModeTools(modes),
     ...createReviewTools()
   ];
 }
 export {
   createDiffpiReloadTool,
-  createModeTools,
   createPiTools,
   createReviewTools,
   diffpiLogTool,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@difflab/pi",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "description": "Tools and skills for the pi coding agent",
   "repository": {
     "type": "git",
@@ -59,7 +59,7 @@
     "@juicesharp/rpiv-ask-user-question": "^2.10.1",
     "commander": "^14.0.0",
     "mdast-builder": "1.1.1",
-    "pi-mcp-adapter": "^2.33.0",
+    "pi-mcp-adapter": "^2.38.0",
     "remark-stringify": "11.0.0",
     "unified": "11.0.5",
     "zod": "^4.6.5"

package/skills/diffpi-setup/SKILL.md

@@ -47,4 +47,4 @@ Map `None`, `Linear`, and `Jira` to `none`, `linear`, and `jira`. Infer the revi
 
 When the user is using Zed, ask separately whether to bind `cmd-alt-r` to the `diffpi: tuicr review` task. Only pass `bindZedKey: true` when the user opts in; otherwise pass `false`.
 
-Setup installs Diffpi's bundled agent Markdown into Pi's standard global agent directory. Explain that each default can run through the subagent plugin or as the current inline prompt. `/skill:mode` opens the structured picker. It also accepts `--include-skills`, one agent id, a qualified `skill:agent` id, or `clear`.
+Setup installs Diffpi's bundled agent Markdown into Pi's standard global agent directory. The agents run through the subagent plugin; no inline profile selection is required for plan or review workflows.

package/skills/plan/SKILL.md

@@ -1,31 +1,21 @@
 ---
 name: plan
 description: Create, revise, review, finalize, and execute live Diffpi plans with direct files and one verified plan reviewer.
-allowed-tools: read grep find write edit ask_user_question Agent get_subagent_result steer_subagent diffpi_modes_set diffpi_modes_status plan_verify bash ctx_execute ctx_execute_file watch_ci
 ---
 
 # Plan
 
-`/plan` forwards raw arguments to this skill. Parse the first explicit verb (`init`, `new`, `update`, `annotate`, `finalize`, `go`, `help`); for unknown or empty input, infer a verb from a clear natural-language request. Use `help` only when neither verb nor intent is identifiable; when plan/intent is ambiguous, ask one structured material-decision question. Preserve the exact incoming user request for the Planner. Read the matching reference first. Keep the alias handler thin. Git and project tasks run through `bash` (or the upstream git skill); do not invent tool names for tasks, format, lint, test, CI, or push.
+Route the verb and remaining request to its workflow. Fuzzy-match recognizable natural language; treat argument shapes as hints, not rigid gates.
 
-## Roles and preflight
+| Input                                        | Workflow                                     |
+| -------------------------------------------- | -------------------------------------------- |
+| `init <slug> [--branch <branch>]`            | [init](references/workflows/init.md)         |
+| `new <slug> <request>`                       | [new](references/workflows/new.md)           |
+| `update <slug> <request>`                    | [update](references/workflows/update.md)     |
+| `annotate <plan-path>`                       | [annotate](references/workflows/annotate.md) |
+| `validate <slug>`                            | [validate](references/workflows/validate.md) |
+| `finalize <slug>`                            | [finalize](references/workflows/finalize.md) |
+| `go <slug> [--mode no-commit\|commit\|push]` | [go](references/workflows/go.md)             |
+| `help`, missing or unrecognizable request    | [help](references/workflows/help.md)         |
 
-- **Planner** authors and repairs live `PLAN.md` and numbered briefs with frontier/high thinking.
-- **diffpi-plan-reviewer** is exactly one independent frontier/high, read/search-only child. It first calls `diffpi_modes_status` and returns actual model, thinking, and tools evidence, then reviews structure, parity, action labels, quality, risk, and Worker executability. Reject missing or unverifiable evidence; do not use a fallback reviewer.
-- **Orchestrator** owns go coordination, live-file checkboxes/status, project gates, Git, push, and CI. **Worker** edits only its declared source/test scope and never changes plan files or commits.
-
-For foreground authoring, call `diffpi_modes_set({agent: "planner"})` using the shared inline id (never `plan:planner`); on the next model turn call `diffpi_modes_status` and stop if the switch or evidence is unavailable. For foreground `go`, call `diffpi_modes_set({agent: "orchestrator"})` and verify its status on the next turn. `--bg` launches exactly one named same-session child with `Agent` (Planner for `init/new/update/finalize`, Orchestrator for `go`), passing the exact verb, target, cwd, mode, and `background: true`; remove `--bg`. The child never redispatches, asks questions, or falls back inline.
-
-## Live-file contract
-
-Plans live under the initiating nested Git root in one flat `.diffpi/plan/<YYMMDD[-ticket]-short-slug>/` directory. The date and slug are parts of ONE directory name, never nested `date/slug` directories. Resolve collisions before writing. Use ordinary `read`/`write`/`edit` calls, read every file back, and preserve intent, IDs, completed evidence, order, dependencies, scopes, and constraints. Document phase constraints in each phase's `PLAN.md` entry (use `None` if empty). Each brief puts its sole action-labeled, fenced `text` file tree under `## Files Affected` immediately after `## Objective` (never a bullet list); `## Implementation Constraints` is free-form with optional subheadings. Edit only live plan files; never managed records, plugin source, tests, or profiles. `INCOMPLETE` is a visible marker inside an open `DRAFT`, not a status enum.
-
-`new` and `update` run the stateless read-only `plan_verify` on the current files, then review the complete current draft with one reviewer profile. Repair actionable findings, rerun mechanical verification after each repair, and invoke the same named reviewer again in a fresh `Agent` call (never resume a finished child) within bounded attempts. They remain `DRAFT` after a successful review. Require the child to complete with an attested explicit PASS; partial, steered, stopped, missing, and BLOCKING reports never count. `finalize` and draft `go` write `READY` only after both a passing `plan_verify` on current files and that completed reviewer PASS; do not dispatch Workers or write ready after repairing a failed review without a new completed review. An already-ready plan is reread and mechanically verified too. `plan_verify` does not judge quality, change status, freeze files, or replace the reviewer. No retired managed-plan tools.
-
-## Execution
-
-`go --mode no-commit|commit|push` defaults to `no-commit`. The Orchestrator is the only writer of plan status/checkboxes and coordinates Workers. A draft gets one mechanical `plan_verify` check and one reviewer per pass; both must pass on current files before ready. Schedule a phase only after all prerequisites complete; derive each bounded Worker's task scope from its steps and the phase's sole Files Affected tree. Serialize tasks with overlapping or uncertain derived scopes; independent tasks may run concurrently only when scopes are unambiguously disjoint and prerequisites are satisfied. Run project format, lint, test, and CI checks via `bash`/`ctx_execute` as appropriate. On any failure, preserve the exact command/output, attempts, and affected role and block.
-
-With `commit`, after gates pass make one phase commit. With `push`, push it and call `watch_ci` for that exact SHA; do not push the next phase until settled. `--bg` has one child only and that child does not redispatch. No stateful plan tools.
-
-Read: [init](references/workflows/init.md), [new](references/workflows/new.md), [update](references/workflows/update.md), [annotate](references/workflows/annotate.md), [finalize](references/workflows/finalize.md), [go](references/workflows/go.md), [help](references/workflows/help.md).
+Follow the selected workflow. For `init`, `new`, `update`, DRAFT `validate`, `finalize`, or `go` when no native subagents are available, follow [the explicit inline exception](references/workflows/inline-fallback.md) only after the caller obtains confirmation; never silently fall back.