@difflab/pi 0.2.0-rc.202609191250.90d35bd → 0.2.0-rc.202609191441.52cd715

package/README.md

@@ -38,11 +38,11 @@ Standard agents come from the same global and trusted-project directories used b
 
 The skill delegates mechanics to `review_context`, `review_new`, `review_edit`, `review_diff`, `review_gates`, `review_submit`, `review_add_comment`, `review_comments`, `review_respond`, `review_publish`, `review_complete`, `review_merge`, and `review_launch_ui`. Inline `auto` and `address` activate Reviewer on Sol; lifecycle verbs activate Orchestrator on Luna. Reviewer classifies address threads and delegates bounded edits to lightweight workers. `--bg` leaves the current chat mode unchanged and launches a tracked Orchestrator child. The generic `diffpi_template` tool loads bundled templates or user overrides. GitHub and GitLab support review creation and publication. Merge is intentionally GitHub-only and remains separate from publish and complete.
 
-`--local` selects the `tuicr` working-tree review backend. Without it, targets are a PR/MR number, URL, or branch; no target uses the current branch. Local address flows apply fixes without commits; remote address flows use the bundled `/git commit --no-push` workflow before posting draft responses for changed threads. Local reply overlays preserve remote thread IDs until `publish --local` promotes comments and replies to the forge. Remote comments carry a generated-review notice with the exact provider/model route; local comments use `Agent: <provider/model>` as the author.
+`--local` selects the `tuicr` working-tree review backend. Without it, targets are a PR/MR number, URL, or branch; no target uses the current branch. Local address flows apply fixes without commits; remote address flows use the upstream `/git commit --no-push` workflow before posting draft responses for changed threads. Local reply overlays preserve remote thread IDs until `publish --local` promotes comments and replies to the forge. Remote comments carry a generated-review notice with the exact provider/model route; local comments use `Agent: <provider/model>` as the author.
 
 Local artifacts live in `.diffpi/review/` and use `YYMMDD-<short-head-sha>.md` or `YYMMDD-uncommitted.md` names. `.diffpi` links to `~/.difflab/diffpi/projects/<repository-name>-<identity-hash>/`, so all worktrees for one remote share records while unrelated same-named repositories remain isolated. The launcher opens a repository-scoped mux tab when zellij, tmux, or screen is detected; without a mux, Zed lazily gets a `diffpi: tuicr review` task, and other environments receive the command to run.
 
-Draft PR bodies use the bundled `review/draft-pr.md` template. Override it at `~/.difflab/diffpi/templates/review/draft-pr.md`. The bundled `/git` skill provides conventional commit and safe rebase workflows. During package development, run `mise watch //packages/pi:dev`; the task builds and installs the package when its sources change. Run `/reload` in Pi after each successful install.
+Draft PR bodies use the bundled `review/draft-pr.md` template. Override it at `~/.difflab/diffpi/templates/review/draft-pr.md`. Setup installs the upstream Codevoyant `/git` skill for conventional commit and safe rebase workflows. During package development, run `mise watch //packages/pi:dev`; the task builds and installs the package when its sources change. Run `/reload` in Pi after each successful install.
 
 The package root exports environment and forge lifecycle adapters, review backends, gate checks, review schemas and artifact helpers, template helpers, store helpers, tuicr helpers, setup operations, and inline-mode control. `@difflab/pi/tools` exports `createReviewTools` and the complete tool catalog.
 

package/dist/extensions/index.js

@@ -13699,7 +13699,7 @@ function isModeBaseline(value) {
 }
 
 // src/commands/mode.ts
-var MODE_USAGE = `/mode [agent|clear|reset]
+var MODE_USAGE = `/mode [agent|default|clear|reset]
 
 Use /mode to pick a profile. Available profiles: copilot, orchestrator, reviewer, tutor, worker.`;
 function registerModeCommand(pi, modes) {
@@ -13727,7 +13727,7 @@ async function handleModeCommand(args, ctx, modes) {
     ctx.ui.notify(MODE_USAGE, "error");
     return;
   }
-  const result = requested === "clear" || requested === "reset" ? await modes.unset(ctx) : await modes.set(requested, ctx);
+  const result = requested === "default" || requested === "clear" || requested === "reset" ? await modes.unset(ctx) : await modes.set(requested, ctx);
   ctx.ui.notify(`${result.message} Changes apply on the next turn.`, result.ok ? "info" : "error");
 }
 
@@ -14032,30 +14032,37 @@ import { homedir as homedir4 } from "node:os";
 import { dirname as dirname3, join as join8 } from "node:path";
 var ZED_REVIEW_TASK_NAME = "diffpi: tuicr review";
 var REVIEW_KEYBINDING = "cmd-alt-r";
-var REVIEW_TASK = {
-  label: ZED_REVIEW_TASK_NAME,
-  command: "tuicr",
-  cwd: "$ZED_WORKTREE_ROOT",
-  use_new_terminal: true,
-  reveal: "always",
-  reveal_target: "center"
-};
+function reviewTask(command = ["tuicr"]) {
+  const [executable, ...args] = command;
+  if (!executable)
+    throw new Error("The Zed review task requires a command.");
+  return {
+    label: ZED_REVIEW_TASK_NAME,
+    command: executable,
+    args: args.length > 0 ? args : undefined,
+    cwd: "$ZED_WORKTREE_ROOT",
+    use_new_terminal: true,
+    reveal: "always",
+    reveal_target: "center"
+  };
+}
 function zedTasksPath(homeDir = homedir4()) {
   return join8(homeDir, ".config", "zed", "tasks.json");
 }
 function zedKeymapPath(homeDir = homedir4()) {
   return join8(homeDir, ".config", "zed", "keymap.json");
 }
-async function ensureZedReviewTask(homeDir = homedir4()) {
+async function ensureZedReviewTask(homeDir = homedir4(), command = ["tuicr"]) {
   const path = zedTasksPath(homeDir);
   const currentText = await readOptional(path);
   const tasks = parseJsonArray(currentText, path);
   const index = tasks.findIndex((task) => task.label === ZED_REVIEW_TASK_NAME);
   const next = [...tasks];
+  const task = reviewTask(command);
   if (index >= 0)
-    next[index] = { ...tasks[index], ...REVIEW_TASK };
+    next[index] = { ...tasks[index], ...task };
   else
-    next.push(REVIEW_TASK);
+    next.push(task);
   const changed = JSON.stringify(tasks) !== JSON.stringify(next);
   if (changed)
     await writeJson(path, next);
@@ -14169,7 +14176,7 @@ async function openInNewTab(command, opts) {
   }
   if (detectIde(env) === "zed") {
     try {
-      await ensureZedReviewTask(opts.homeDir);
+      await ensureZedReviewTask(opts.homeDir, command);
       return {
         launched: false,
         configured: true,
@@ -15706,6 +15713,9 @@ function modelRoute(ctx) {
     throw new Error("Cannot record review provenance because Pi did not provide an active model route.");
   return `${ctx.model.provider}/${ctx.model.id}`;
 }
+function hasReviewDraft(comments, body) {
+  return comments.length > 0 || body.trim().length > 0;
+}
 async function workingTreeDiff(cwd) {
   const tracked = await run("git", ["-C", cwd, "diff", "HEAD"], { capture: "unbounded" });
   if (tracked.code !== 0)
@@ -15890,6 +15900,8 @@ ${openMessage}`, {
           notVerified: params.notVerified ?? []
         }), "utf8");
         const comments = toReviewComments(findings, useLocalBackend ? undefined : model);
+        const body = (params.overallIssues ?? []).join(`
+`);
         if (useLocalBackend) {
           const session = await resolveTuicrSession(review, Boolean(params.local || params.workingTree));
           if (!session) {
@@ -15903,16 +15915,16 @@ ${openMessage}`, {
             artifactPath: artifact,
             author: localReviewAuthor(model)
           });
-          await backend.stage({ comments, body: (params.overallIssues ?? []).join(`
-`) });
+          await backend.stage({ comments, body });
           return result(`Local review staged in tuicr: ${artifact}`, { artifact, count: findings.length, session });
         }
         if (!review.pr)
           return result(`Review written: ${artifact}. No PR/MR matches this remote target.`, { artifact });
-        if (comments.length > 0) {
-          await createRemoteReviewBackend(review.vcs, review.pr.number).stage({ comments, body: "" });
+        const hasDraft = hasReviewDraft(comments, body);
+        if (hasDraft) {
+          await createRemoteReviewBackend(review.vcs, review.pr.number).stage({ comments, body });
         }
-        return result(`${comments.length > 0 ? "Pending review staged" : "Clean review recorded"} on #${review.pr.number}. Artifact: ${artifact}`, {
+        return result(`${hasDraft ? "Pending review staged" : "Clean review recorded"} on #${review.pr.number}. Artifact: ${artifact}`, {
           artifact,
           pr: review.pr,
           count: findings.length
@@ -16069,7 +16081,7 @@ ${params.body}`, {
       label: "review complete",
       description: "Approve, reject, or abandon a remote review, or archive a local review artifact.",
       promptSnippet: "Call review_complete to finish a review without merging it.",
-      promptGuidelines: ["A local completion archives the Diffpi artifact but leaves the tuicr session intact."],
+      promptGuidelines: ["A local completion archives the Diffpi artifact and deletes the matching tuicr session."],
       parameters: parameters5(completeSchema),
       executionMode: "sequential",
       async execute(_id, input, _signal, _onUpdate, ctx) {
@@ -16558,7 +16570,8 @@ var PI_PACKAGES = [
 ];
 var PI_SKILL_SOURCES = [
   { repository: "arabold/docs-mcp-server", skills: ["docs-manage", "docs-search", "fetch-url"] },
-  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] }
+  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] },
+  { repository: "cloudvoyant/codevoyant", skills: ["git"] }
 ];
 var MCP_ADAPTER_PACKAGE = "npm:pi-mcp-adapter";
 var BUNDLED_AGENTS_DIR2 = resolveBundledAgentsDir();
@@ -16806,7 +16819,8 @@ ${source}`;
 }
 async function gitlabMcpHost(projectDir) {
   try {
-    return (await detectVcs(projectDir)).host || "gitlab.com";
+    const vcs = await detectVcs(projectDir);
+    return vcs.provider === "gitlab" && vcs.host ? vcs.host : "gitlab.com";
   } catch {
     return "gitlab.com";
   }

package/dist/index.js

@@ -177,30 +177,37 @@ import { homedir as homedir2 } from "node:os";
 import { dirname, join as join3 } from "node:path";
 var ZED_REVIEW_TASK_NAME = "diffpi: tuicr review";
 var REVIEW_KEYBINDING = "cmd-alt-r";
-var REVIEW_TASK = {
-  label: ZED_REVIEW_TASK_NAME,
-  command: "tuicr",
-  cwd: "$ZED_WORKTREE_ROOT",
-  use_new_terminal: true,
-  reveal: "always",
-  reveal_target: "center"
-};
+function reviewTask(command = ["tuicr"]) {
+  const [executable, ...args] = command;
+  if (!executable)
+    throw new Error("The Zed review task requires a command.");
+  return {
+    label: ZED_REVIEW_TASK_NAME,
+    command: executable,
+    args: args.length > 0 ? args : undefined,
+    cwd: "$ZED_WORKTREE_ROOT",
+    use_new_terminal: true,
+    reveal: "always",
+    reveal_target: "center"
+  };
+}
 function zedTasksPath(homeDir = homedir2()) {
   return join3(homeDir, ".config", "zed", "tasks.json");
 }
 function zedKeymapPath(homeDir = homedir2()) {
   return join3(homeDir, ".config", "zed", "keymap.json");
 }
-async function ensureZedReviewTask(homeDir = homedir2()) {
+async function ensureZedReviewTask(homeDir = homedir2(), command = ["tuicr"]) {
   const path = zedTasksPath(homeDir);
   const currentText = await readOptional(path);
   const tasks = parseJsonArray(currentText, path);
   const index = tasks.findIndex((task) => task.label === ZED_REVIEW_TASK_NAME);
   const next = [...tasks];
+  const task = reviewTask(command);
   if (index >= 0)
-    next[index] = { ...tasks[index], ...REVIEW_TASK };
+    next[index] = { ...tasks[index], ...task };
   else
-    next.push(REVIEW_TASK);
+    next.push(task);
   const changed = JSON.stringify(tasks) !== JSON.stringify(next);
   if (changed)
     await writeJson(path, next);
@@ -314,7 +321,7 @@ async function openInNewTab(command, opts) {
   }
   if (detectIde(env) === "zed") {
     try {
-      await ensureZedReviewTask(opts.homeDir);
+      await ensureZedReviewTask(opts.homeDir, command);
       return {
         launched: false,
         configured: true,
@@ -2191,7 +2198,8 @@ var PI_PACKAGES = [
 ];
 var PI_SKILL_SOURCES = [
   { repository: "arabold/docs-mcp-server", skills: ["docs-manage", "docs-search", "fetch-url"] },
-  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] }
+  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] },
+  { repository: "cloudvoyant/codevoyant", skills: ["git"] }
 ];
 var MCP_ADAPTER_PACKAGE = "npm:pi-mcp-adapter";
 var BUNDLED_AGENTS_DIR2 = resolveBundledAgentsDir();
@@ -2439,7 +2447,8 @@ ${source}`;
 }
 async function gitlabMcpHost(projectDir) {
   try {
-    return (await detectVcs(projectDir)).host || "gitlab.com";
+    const vcs = await detectVcs(projectDir);
+    return vcs.provider === "gitlab" && vcs.host ? vcs.host : "gitlab.com";
   } catch {
     return "gitlab.com";
   }

package/dist/setup.d.ts

@@ -43,4 +43,5 @@ export declare function ensureMcpAdapters(miseExecutable: string, options?: Setu
 export declare function setupPi(options?: SetupOptions): Promise<SetupResult>;
 export declare function setupRequiresRestart(actions: readonly SetupAction[]): boolean;
 export declare function ensureZedIntegration(options?: SetupOptions): Promise<SetupAction[]>;
+export declare function gitlabMcpHost(projectDir: string): Promise<string>;
 //# sourceMappingURL=setup.d.ts.map

package/dist/setup.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"setup.d.ts","sourceRoot":"","sources":["../src/setup.ts"],"names":[],"mappings":"AAqDA,MAAM,MAAM,YAAY,GAAG,MAAM,GAAG,QAAQ,GAAG,MAAM,CAAC;AACtD,MAAM,MAAM,KAAK,GAAG,MAAM,GAAG,QAAQ,GAAG,QAAQ,CAAC;AACjD,MAAM,MAAM,WAAW,GAAG,OAAO,GAAG,WAAW,GAAG,SAAS,GAAG,SAAS,GAAG,SAAS,CAAC;AAEpF,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,WAAW,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,YAAY;IAC3B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,0FAA0F;IAC1F,MAAM,CAAC,EAAE,SAAS,OAAO,CAAC,KAAK,EAAE,MAAM,CAAC,EAAE,CAAC;IAC3C,KAAK,CAAC,EAAE,KAAK,CAAC;IACd,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC,QAAQ,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,eAAe,CAAC,EAAE,SAAS;QAAE,QAAQ,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC9D,UAAU,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CACxC;AASD,MAAM,WAAW,WAAW;IAC1B,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,SAAS,EAAE,OAAO,CAAC;CACpB;AAID,wBAAsB,UAAU,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,WAAW,CAAA;CAAE,CAAC,CAgBjH;AAED,wBAAsB,eAAe,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,CAAC,CAU9G;AAED,wBAAsB,cAAc,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAsB/G;AAED,wBAAsB,eAAe,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAqBxF;AAED,wBAAsB,cAAc,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAkBvF;AAED,wBAAsB,cAAc,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAsB/G;AAED,wBAAsB,iBAAiB,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAwClH;AAID,wBAAsB,OAAO,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,CAAC,CAgB9E;AAED,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,SAAS,WAAW,EAAE,GAAG,OAAO,CAW7E;AA6CD,wBAAsB,oBAAoB,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CA0B7F"}
+{"version":3,"file":"setup.d.ts","sourceRoot":"","sources":["../src/setup.ts"],"names":[],"mappings":"AAsDA,MAAM,MAAM,YAAY,GAAG,MAAM,GAAG,QAAQ,GAAG,MAAM,CAAC;AACtD,MAAM,MAAM,KAAK,GAAG,MAAM,GAAG,QAAQ,GAAG,QAAQ,CAAC;AACjD,MAAM,MAAM,WAAW,GAAG,OAAO,GAAG,WAAW,GAAG,SAAS,GAAG,SAAS,GAAG,SAAS,CAAC;AAEpF,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,WAAW,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,YAAY;IAC3B,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,0FAA0F;IAC1F,MAAM,CAAC,EAAE,SAAS,OAAO,CAAC,KAAK,EAAE,MAAM,CAAC,EAAE,CAAC;IAC3C,KAAK,CAAC,EAAE,KAAK,CAAC;IACd,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC,QAAQ,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,eAAe,CAAC,EAAE,SAAS;QAAE,QAAQ,EAAE,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC9D,UAAU,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,IAAI,CAAC;CACxC;AASD,MAAM,WAAW,WAAW;IAC1B,OAAO,EAAE,WAAW,EAAE,CAAC;IACvB,SAAS,EAAE,OAAO,CAAC;CACpB;AAID,wBAAsB,UAAU,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC;IAAE,UAAU,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,WAAW,CAAA;CAAE,CAAC,CAgBjH;AAED,wBAAsB,eAAe,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,CAAC,CAU9G;AAED,wBAAsB,cAAc,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAsB/G;AAED,wBAAsB,eAAe,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAqBxF;AAED,wBAAsB,cAAc,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAkBvF;AAED,wBAAsB,cAAc,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAsB/G;AAED,wBAAsB,iBAAiB,CAAC,cAAc,EAAE,MAAM,EAAE,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CAwClH;AAID,wBAAsB,OAAO,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,CAAC,CAgB9E;AAED,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,SAAS,WAAW,EAAE,GAAG,OAAO,CAW7E;AA6CD,wBAAsB,oBAAoB,CAAC,OAAO,GAAE,YAAiB,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC,CA0B7F;AA2BD,wBAAsB,aAAa,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAOvE"}

package/dist/tools/index.js

@@ -214,30 +214,37 @@ import { homedir } from "node:os";
 import { dirname, join as join2 } from "node:path";
 var ZED_REVIEW_TASK_NAME = "diffpi: tuicr review";
 var REVIEW_KEYBINDING = "cmd-alt-r";
-var REVIEW_TASK = {
-  label: ZED_REVIEW_TASK_NAME,
-  command: "tuicr",
-  cwd: "$ZED_WORKTREE_ROOT",
-  use_new_terminal: true,
-  reveal: "always",
-  reveal_target: "center"
-};
+function reviewTask(command = ["tuicr"]) {
+  const [executable, ...args] = command;
+  if (!executable)
+    throw new Error("The Zed review task requires a command.");
+  return {
+    label: ZED_REVIEW_TASK_NAME,
+    command: executable,
+    args: args.length > 0 ? args : undefined,
+    cwd: "$ZED_WORKTREE_ROOT",
+    use_new_terminal: true,
+    reveal: "always",
+    reveal_target: "center"
+  };
+}
 function zedTasksPath(homeDir = homedir()) {
   return join2(homeDir, ".config", "zed", "tasks.json");
 }
 function zedKeymapPath(homeDir = homedir()) {
   return join2(homeDir, ".config", "zed", "keymap.json");
 }
-async function ensureZedReviewTask(homeDir = homedir()) {
+async function ensureZedReviewTask(homeDir = homedir(), command = ["tuicr"]) {
   const path = zedTasksPath(homeDir);
   const currentText = await readOptional(path);
   const tasks = parseJsonArray(currentText, path);
   const index = tasks.findIndex((task) => task.label === ZED_REVIEW_TASK_NAME);
   const next = [...tasks];
+  const task = reviewTask(command);
   if (index >= 0)
-    next[index] = { ...tasks[index], ...REVIEW_TASK };
+    next[index] = { ...tasks[index], ...task };
   else
-    next.push(REVIEW_TASK);
+    next.push(task);
   const changed = JSON.stringify(tasks) !== JSON.stringify(next);
   if (changed)
     await writeJson(path, next);
@@ -351,7 +358,7 @@ async function openInNewTab(command, opts) {
   }
   if (detectIde(env) === "zed") {
     try {
-      await ensureZedReviewTask(opts.homeDir);
+      await ensureZedReviewTask(opts.homeDir, command);
       return {
         launched: false,
         configured: true,
@@ -1906,6 +1913,9 @@ function modelRoute(ctx) {
     throw new Error("Cannot record review provenance because Pi did not provide an active model route.");
   return `${ctx.model.provider}/${ctx.model.id}`;
 }
+function hasReviewDraft(comments, body) {
+  return comments.length > 0 || body.trim().length > 0;
+}
 async function workingTreeDiff(cwd) {
   const tracked = await run("git", ["-C", cwd, "diff", "HEAD"], { capture: "unbounded" });
   if (tracked.code !== 0)
@@ -2090,6 +2100,8 @@ ${openMessage}`, {
           notVerified: params.notVerified ?? []
         }), "utf8");
         const comments = toReviewComments(findings, useLocalBackend ? undefined : model);
+        const body = (params.overallIssues ?? []).join(`
+`);
         if (useLocalBackend) {
           const session = await resolveTuicrSession(review, Boolean(params.local || params.workingTree));
           if (!session) {
@@ -2103,16 +2115,16 @@ ${openMessage}`, {
             artifactPath: artifact,
             author: localReviewAuthor(model)
           });
-          await backend.stage({ comments, body: (params.overallIssues ?? []).join(`
-`) });
+          await backend.stage({ comments, body });
           return result(`Local review staged in tuicr: ${artifact}`, { artifact, count: findings.length, session });
         }
         if (!review.pr)
           return result(`Review written: ${artifact}. No PR/MR matches this remote target.`, { artifact });
-        if (comments.length > 0) {
-          await createRemoteReviewBackend(review.vcs, review.pr.number).stage({ comments, body: "" });
+        const hasDraft = hasReviewDraft(comments, body);
+        if (hasDraft) {
+          await createRemoteReviewBackend(review.vcs, review.pr.number).stage({ comments, body });
         }
-        return result(`${comments.length > 0 ? "Pending review staged" : "Clean review recorded"} on #${review.pr.number}. Artifact: ${artifact}`, {
+        return result(`${hasDraft ? "Pending review staged" : "Clean review recorded"} on #${review.pr.number}. Artifact: ${artifact}`, {
           artifact,
           pr: review.pr,
           count: findings.length
@@ -2269,7 +2281,7 @@ ${params.body}`, {
       label: "review complete",
       description: "Approve, reject, or abandon a remote review, or archive a local review artifact.",
       promptSnippet: "Call review_complete to finish a review without merging it.",
-      promptGuidelines: ["A local completion archives the Diffpi artifact but leaves the tuicr session intact."],
+      promptGuidelines: ["A local completion archives the Diffpi artifact and deletes the matching tuicr session."],
       parameters: parameters(completeSchema),
       executionMode: "sequential",
       async execute(_id, input, _signal, _onUpdate, ctx) {
@@ -2846,7 +2858,8 @@ var PI_PACKAGES = [
 ];
 var PI_SKILL_SOURCES = [
   { repository: "arabold/docs-mcp-server", skills: ["docs-manage", "docs-search", "fetch-url"] },
-  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] }
+  { repository: "AminBlg/SimpleEnglish", skills: ["simple-english"] },
+  { repository: "cloudvoyant/codevoyant", skills: ["git"] }
 ];
 var MCP_ADAPTER_PACKAGE = "npm:pi-mcp-adapter";
 var BUNDLED_AGENTS_DIR = resolveBundledAgentsDir();
@@ -3094,7 +3107,8 @@ ${source}`;
 }
 async function gitlabMcpHost(projectDir) {
   try {
-    return (await detectVcs(projectDir)).host || "gitlab.com";
+    const vcs = await detectVcs(projectDir);
+    return vcs.provider === "gitlab" && vcs.host ? vcs.host : "gitlab.com";
   } catch {
     return "gitlab.com";
   }

package/dist/tools/review.d.ts

@@ -1,5 +1,6 @@
 import { type ToolDefinition } from '@earendil-works/pi-coding-agent';
-import { type ReviewThreadRecord } from '../review';
+import { type ReviewComment, type ReviewThreadRecord } from '../review';
+export declare function hasReviewDraft(comments: readonly ReviewComment[], body: string): boolean;
 export declare function workingTreeDiff(cwd: string): Promise<string>;
 export declare function createReviewTools(): readonly ToolDefinition[];
 export declare function syncLocalThreadArtifact(path: string, title: string, sessionSlug: string, threads: readonly ReviewThreadRecord[]): Promise<ReviewThreadRecord[]>;

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

@@ -1 +1 @@
-{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../src/tools/review.ts"],"names":[],"mappings":"AAGA,OAAO,EAAqC,KAAK,cAAc,EAAE,MAAM,iCAAiC,CAAC;AAMzG,OAAO,EAuBL,KAAK,kBAAkB,EACxB,MAAM,WAAW,CAAC;AA+EnB,wBAAsB,eAAe,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAalE;AAID,wBAAgB,iBAAiB,IAAI,SAAS,cAAc,EAAE,CA+a7D;AAyKD,wBAAsB,uBAAuB,CAC3C,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,MAAM,EACb,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,SAAS,kBAAkB,EAAE,GACrC,OAAO,CAAC,kBAAkB,EAAE,CAAC,CAwB/B"}
+{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../../src/tools/review.ts"],"names":[],"mappings":"AAGA,OAAO,EAAqC,KAAK,cAAc,EAAE,MAAM,iCAAiC,CAAC;AAMzG,OAAO,EAsBL,KAAK,aAAa,EAClB,KAAK,kBAAkB,EACxB,MAAM,WAAW,CAAC;AA+EnB,wBAAgB,cAAc,CAAC,QAAQ,EAAE,SAAS,aAAa,EAAE,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAExF;AAED,wBAAsB,eAAe,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAalE;AAID,wBAAgB,iBAAiB,IAAI,SAAS,cAAc,EAAE,CAib7D;AAyKD,wBAAsB,uBAAuB,CAC3C,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,MAAM,EACb,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,SAAS,kBAAkB,EAAE,GACrC,OAAO,CAAC,kBAAkB,EAAE,CAAC,CAwB/B"}

package/dist/zed.d.ts

@@ -6,6 +6,6 @@ export interface ZedEnsureResult {
 }
 export declare function zedTasksPath(homeDir?: string): string;
 export declare function zedKeymapPath(homeDir?: string): string;
-export declare function ensureZedReviewTask(homeDir?: string): Promise<ZedEnsureResult>;
+export declare function ensureZedReviewTask(homeDir?: string, command?: readonly string[]): Promise<ZedEnsureResult>;
 export declare function ensureZedReviewKeybinding(homeDir?: string): Promise<ZedEnsureResult>;
 //# sourceMappingURL=zed.d.ts.map

package/dist/zed.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"zed.d.ts","sourceRoot":"","sources":["../src/zed.ts"],"names":[],"mappings":"AAIA,eAAO,MAAM,oBAAoB,yBAAyB,CAAC;AAmB3D,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;CAClB;AAWD,wBAAgB,YAAY,CAAC,OAAO,SAAY,GAAG,MAAM,CAExD;AAED,wBAAgB,aAAa,CAAC,OAAO,SAAY,GAAG,MAAM,CAEzD;AAED,wBAAsB,mBAAmB,CAAC,OAAO,SAAY,GAAG,OAAO,CAAC,eAAe,CAAC,CAWvF;AAED,wBAAsB,yBAAyB,CAAC,OAAO,SAAY,GAAG,OAAO,CAAC,eAAe,CAAC,CAgB7F"}
+{"version":3,"file":"zed.d.ts","sourceRoot":"","sources":["../src/zed.ts"],"names":[],"mappings":"AAIA,eAAO,MAAM,oBAAoB,yBAAyB,CAAC;AAmB3D,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,OAAO,CAAC;IACjB,OAAO,EAAE,OAAO,CAAC;CAClB;AAgBD,wBAAgB,YAAY,CAAC,OAAO,SAAY,GAAG,MAAM,CAExD;AAED,wBAAgB,aAAa,CAAC,OAAO,SAAY,GAAG,MAAM,CAEzD;AAED,wBAAsB,mBAAmB,CACvC,OAAO,SAAY,EACnB,OAAO,GAAE,SAAS,MAAM,EAAc,GACrC,OAAO,CAAC,eAAe,CAAC,CAY1B;AAED,wBAAsB,yBAAyB,CAAC,OAAO,SAAY,GAAG,OAAO,CAAC,eAAe,CAAC,CAgB7F"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@difflab/pi",
-  "version": "0.2.0-rc.202609191250.90d35bd",
+  "version": "0.2.0-rc.202609191441.52cd715",
   "description": "Tools and skills for the pi coding agent",
   "repository": {
     "type": "git",

package/skills/review/references/workflows/address.md

@@ -5,7 +5,7 @@
 3. Call `review_comments`. With `--local`, it synchronizes every comment in the selected tuicr session to `.diffpi/review/<session-slug>.md`. Existing replies and statuses remain; source comments removed by the user become resolved ledger entries.
 4. Classify every thread, group justified code changes into bounded non-overlapping tasks, and delegate those tasks to lightweight worker agents. Keep questions, outcome decisions, integration, and response text in the reviewer coordinator. With `--local`, workers modify the current working tree without committing. Without `--local`, apply a fix only when the comment requests a change.
 5. Respond to every thread with its outcome. Answer questions and leave them open. For substantive requests, state what changed and whether it is resolved, but leave the thread open for reviewer confirmation. For a trivial request, reply `Resolved`; remote threads may close, while the user must delete the local tuicr source comment before the next sync marks it resolved.
-6. When a remote address flow changes code, invoke the bundled `/git commit --no-push` workflow after checks pass and before drafting responses. Use `--atomic` when the fixes form separate logical commits.
+6. When a remote address flow changes code, invoke the upstream `/git commit --no-push` workflow after checks pass and before drafting responses. Use `--atomic` when the fixes form separate logical commits.
 7. Call `review_respond` for each thread. With `--local`, every response is posted to tuicr and recorded in the session ledger. Without `--local`, replies post to the forge immediately.
 8. With `--local`, call `review_launch_ui` with `local: true` after replies are recorded so the user can inspect the working-tree review in tuicr.
 9. Report fixed, committed, answered, addressed, resolved, and unresolved counts. Do not publish, complete, or merge in this workflow.

package/skills/git/LICENSE.md

@@ -1,9 +0,0 @@
-MIT License
-
-Copyright (c) codevoyant contributors
-
-Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

package/skills/git/SKILL.md

@@ -1,47 +0,0 @@
----
-name: git
-description: 'Git version control workflows: create conventional commits or run interactive rebase. Triggers on: "git commit", "git rebase", "git help", "create a commit", "conventional commit", "commit my changes", "interactive rebase", "squash commits".'
-license: MIT
-allowed-tools: ask_user_question
-compatibility: Works on Claude Code, OpenCode, GitHub Copilot (VS Code), and Codex. No platform-specific features used.
----
-
-# git
-
-Git workflow skill dispatcher.
-
-## Critical Rules
-
-- **Never execute workflow logic here** — this file only parses args and dispatches
-- **Step 0 always runs first** — no exceptions
-- **Unknown verb → run `help.md`** — never error silently
-- **Pass all remaining args through** — workflow receives `$REMAINING_ARGS` unchanged
-- **Markdown output: soft-wrap prose, never hard-wrap** — when writing markdown, write each paragraph as one continuous line; do not insert manual newlines to wrap prose at a fixed column width. Newlines still separate paragraphs, list items, headings, and code fences.
-
-## Step 0: Parse Arguments
-
-The raw invocation args (filled by Claude Code / OpenCode slash commands): `$ARGUMENTS`. If this line is not filled in, read the verb and remaining args from the user's current message.
-
-```bash
-VERB="[first non-flag argument, or empty]"
-REMAINING_ARGS="[everything after VERB, preserving order and flags]"
-
-case "$VERB" in
-  "")  VERB="help" ;;
-esac
-```
-
-## Step 1: Dispatch to Workflow
-
-Read and execute `references/workflows/{VERB}.md`, passing `$REMAINING_ARGS` as the argument string.
-
-If `references/workflows/{VERB}.md` does not exist, fall back to `references/workflows/help.md` and note the unknown verb.
-
-## Workflow Index
-
-- **commit** (`references/workflows/commit.md`) — create a conventional commit with optional CI monitoring
-- **exclude** (`references/workflows/exclude.md`) — hide files/dirs from git via `.git/info/exclude` (no `.gitignore` change)
-- **hooks** (`references/workflows/hooks.md`) — install/manage the commit-msg hook that strips agent self-attribution
-- **help** (`references/workflows/help.md`) — print command reference
-- **rebase** (`references/workflows/rebase.md`) — interactive rebase helper
-- **worktree** (`references/workflows/worktree.md`) — create/switch a branch and/or create a worktree (shared routine reused by spec)

package/skills/git/references/hooks/commit-msg

@@ -1,46 +0,0 @@
-#!/usr/bin/env bash
-#
-# codevoyant git skill — commit-msg hook
-# Strips agent self-attribution from EVERY commit message, no matter which
-# tool or agent created the commit. This is the robust backstop for the rule
-# in skills/git/references/workflows/commit.md: commits must never carry
-# "Co-Authored-By: <agent>", "Generated with <agent>", or a 🤖 line.
-#
-# Install with:  /git hooks install   (or copy to .git/hooks/commit-msg, chmod +x)
-#
-set -euo pipefail
-
-MSG_FILE="${1:-}"
-[ -n "$MSG_FILE" ] && [ -f "$MSG_FILE" ] || exit 0
-
-# Identifiers that mark a trailer/line as AGENT self-attribution (case-insensitive).
-# Human co-authors are preserved — only lines naming a coding agent are removed.
-AGENT='claude|anthropic|noreply@anthropic\.com|chatgpt|openai|gpt-[0-9]|copilot|codex|cursor|devin|sourcegraph|amp code|ai assistant|ai agent'
-
-before="$(cat "$MSG_FILE")"
-tmp="$(mktemp)"
-
-# 1. Drop "Co-authored-by:" trailers that name an agent.
-# 2. Drop "Generated with/by <agent>" (and "Generated with Claude Code") lines.
-# 3. Drop any line containing the robot emoji.
-grep -ivaE "^[[:space:]]*Co-authored-by:.*(${AGENT})" "$MSG_FILE" \
-  | grep -ivaE "generated (with|by).*(${AGENT}|claude code)" \
-  | grep -vaF '🤖' \
-  > "$tmp" || true
-
-# Did the filters actually remove an attribution line? (Compare before the
-# cosmetic whitespace trim, so blank-line tidying alone never trips the notice.)
-stripped="$(cat "$tmp")"
-
-# Collapse any trailing blank lines the removals left behind.
-awk '{ lines[NR]=$0 }
-     END { last=NR
-           while (last>0 && lines[last] ~ /^[[:space:]]*$/) last--
-           for (i=1;i<=last;i++) print lines[i] }' "$tmp" > "$MSG_FILE"
-rm -f "$tmp"
-
-if [ "$before" != "$stripped" ]; then
-  echo "git hook: stripped agent self-attribution from commit message" >&2
-fi
-
-exit 0