@henryqw/pi-subagent 16.0.1 → 16.0.2

package/README.md

@@ -21,6 +21,7 @@ Run `/task-models` and configure the `fast` profile before delegating. Open `/ta
 | --- | --- | --- |
 | [`@henryqw/pi-orchestrator`](https://pi.henry.wang/extensions/pi-orchestrator) | Consumer | Owns durable checked local implementation graphs. |
 | [`@henryqw/pi-task-models`](https://pi.henry.wang/extensions/pi-task-models) | Required | Supplies `fast`, `balanced`, `frontier`, and `fav` model routes. |
+| [`@henryqw/pi-process`](https://pi.henry.wang/packages/pi-process) | Required | Runs bounded captured Git commands. |
 
 Routes come from `~/.pi/agent/config/pi-task-models/config.json`. It stores explicit task overrides. Missing shared model config warns once because delegation needs a route.
 

package/dist/git-process.d.ts

@@ -0,0 +1,8 @@
+export type GitResult = {
+    code: number;
+    stdout: string;
+    stderr: string;
+};
+export type GitRunner = (args: string[], cwd: string, signal?: AbortSignal) => Promise<GitResult>;
+/** Run one ordinary Git command with bounded output; transport failures never throw. */
+export declare const runGit: GitRunner;

package/dist/git-process.js

@@ -0,0 +1,32 @@
+import { spawnBounded } from "@henryqw/pi-process";
+const GIT_TIMEOUT_MS = 30_000;
+const GIT_OUTPUT_LIMIT_BYTES = 1 * 1024 * 1024;
+const GIT_DIAGNOSTIC_LIMIT = 200;
+/** Run one ordinary Git command with bounded output; transport failures never throw. */
+export const runGit = async (args, cwd, signal) => {
+    try {
+        const result = await spawnBounded("git", ["--no-pager", ...args], {
+            cwd,
+            signal,
+            timeoutMs: GIT_TIMEOUT_MS,
+            stdoutLimitBytes: GIT_OUTPUT_LIMIT_BYTES,
+            stderrLimitBytes: GIT_OUTPUT_LIMIT_BYTES,
+        });
+        if (result.killed) {
+            return {
+                code: -1,
+                stdout: "",
+                stderr: result.stderr.trim().slice(0, GIT_DIAGNOSTIC_LIMIT) || "git was killed",
+            };
+        }
+        return { code: result.code, stdout: result.stdout, stderr: result.stderr };
+    }
+    catch (error) {
+        const reason = error instanceof Error ? error.message : String(error);
+        return {
+            code: -1,
+            stdout: "",
+            stderr: reason.slice(0, GIT_DIAGNOSTIC_LIMIT) || "git execution failed",
+        };
+    }
+};

package/dist/review-evidence.js

@@ -1,28 +1,20 @@
-import { execFile, spawn } from "node:child_process";
+import { spawn } from "node:child_process";
 import fs from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { isAbsolute, join } from "node:path";
 import { inspectIndexFlags } from "./worktree.js";
+import { runGit as runCapturedGit } from "./git-process.js";
 const GIT_TIMEOUT_MS = 30_000;
 export const REVIEW_MAX_PATHS = 1_000;
 export const REVIEW_MAX_PATCH_BYTES = 512 * 1024;
 const STDERR_LIMIT = 200;
-const git = (args, cwd, signal) => new Promise((resolve) => {
-    execFile("git", ["--no-pager", ...args], { cwd, signal, timeout: GIT_TIMEOUT_MS }, (error, stdout, stderr) => {
-        resolve({
-            code: error ? (typeof error.code === "number" ? error.code : -1) : 0,
-            stdout: String(stdout),
-            stderr: String(stderr).slice(0, STDERR_LIMIT),
-        });
-    });
-});
 function failure(args, result) {
-    const detail = result.stderr.trim();
+    const detail = result.stderr.slice(0, STDERR_LIMIT).trim();
     return new Error(`git ${args.join(" ")} failed with exit ${result.code}${detail ? `: ${detail}` : ""}`);
 }
 async function runGit(args, cwd, signal) {
     signal?.throwIfAborted();
-    const result = await git(args, cwd, signal);
+    const result = await runCapturedGit(args, cwd, signal);
     signal?.throwIfAborted();
     if (result.code !== 0)
         throw failure(args, result);
@@ -126,7 +118,7 @@ async function assertCleanRegisteredWorktree(worktree, tip, signal) {
     if (await runGit(["status", "--porcelain=v1", "--untracked-files=all", "--ignore-submodules=none"], worktree, signal)) {
         throw new Error("Review evidence worktree is not clean.");
     }
-    const flags = await inspectIndexFlags(worktree, git, signal);
+    const flags = await inspectIndexFlags(worktree, runCapturedGit, signal);
     if (flags.failure)
         throw new Error(`Review evidence index inspection failed: ${flags.failure}`);
     if (flags.hidden)

package/dist/worktree.d.ts

@@ -1,3 +1,4 @@
+import { type GitRunner } from "./git-process.ts";
 export interface WorktreeInfo {
     path: string;
     cwd: string;
@@ -23,11 +24,6 @@ export type WorktreePayload = {
     commits?: number;
     dirty?: boolean;
 };
-export type GitRunner = (args: string[], cwd: string, signal?: AbortSignal) => Promise<{
-    code: number;
-    stdout: string;
-    stderr: string;
-}>;
 export declare class WorktreeSetupError extends Error {
     name: string;
     readonly worktree: WorktreeInfo;

package/dist/worktree.js

@@ -1,9 +1,8 @@
-import { execFile } from "node:child_process";
 import { createHash } from "node:crypto";
 import { existsSync, lstatSync } from "node:fs";
 import { appendFile, mkdir, readFile } from "node:fs/promises";
 import { dirname, isAbsolute, join, resolve } from "node:path";
-const GIT_TIMEOUT_MS = 30_000;
+import { runGit } from "./git-process.js";
 const WORKTREES_DIRNAME = ".worktrees";
 const BRANCH_NAMESPACE = "pi-subagent";
 export class WorktreeSetupError extends Error {
@@ -14,16 +13,6 @@ export class WorktreeSetupError extends Error {
         this.worktree = worktree;
     }
 }
-/** Runs git, capturing output; never throws on non-zero exit or spawn failure. */
-const runGit = (args, cwd, signal) => new Promise((resolve) => {
-    execFile("git", args, { cwd, timeout: GIT_TIMEOUT_MS, signal }, (error, stdout, stderr) => {
-        resolve({
-            code: error ? (typeof error.code === "number" ? error.code : -1) : 0,
-            stdout: String(stdout),
-            stderr: String(stderr),
-        });
-    });
-});
 const sanitizeShortId = (childId) => createHash("sha256").update(childId).digest("hex").slice(0, 24);
 const stripGitLineEnd = (value) => value.replace(/\r?\n$/, "");
 function hasRepositoryMarker(cwd) {
@@ -71,10 +60,15 @@ async function ensureLocalExclude(gitDir) {
  */
 export async function createChildWorktree(cwd, childId, run = runGit, signal, onPrepared) {
     const root = await run(["rev-parse", "--show-toplevel"], cwd, signal);
+    signal?.throwIfAborted();
+    if (root.code === -1)
+        throw new Error(`git rev-parse failed (${root.stderr.trim().slice(0, 200)})`);
     if (root.code !== 0) {
         signal?.throwIfAborted();
         const repository = await run(["-c", "safe.directory=*", "rev-parse", "--show-toplevel"], cwd, signal);
         signal?.throwIfAborted();
+        if (repository.code === -1)
+            throw new Error(`git rev-parse failed (${repository.stderr.trim().slice(0, 200)})`);
         if (repository.code !== 0 && !hasRepositoryMarker(cwd) && !process.env.GIT_DIR && !process.env.GIT_WORK_TREE)
             return undefined;
         throw new Error(`git rev-parse failed (${root.stderr.trim().slice(0, 200)})`); // dubious ownership, timeout, …
@@ -87,6 +81,9 @@ export async function createChildWorktree(cwd, childId, run = runGit, signal, on
         throw new Error(`git rev-parse --show-prefix failed (${prefix.stderr.trim().slice(0, 200)})`);
     const relativeCwd = stripGitLineEnd(prefix.stdout);
     const base = await run(["rev-parse", "HEAD"], repoRoot, signal);
+    signal?.throwIfAborted();
+    if (base.code === -1)
+        throw new Error(`git rev-parse HEAD failed (${base.stderr.trim().slice(0, 200)})`);
     if (base.code !== 0) {
         signal?.throwIfAborted();
         const head = await run(["symbolic-ref", "--quiet", "HEAD"], repoRoot, signal);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-subagent",
-  "version": "16.0.1",
+  "version": "16.0.2",
   "description": "Delegate bounded single, parallel, or chained tasks to isolated Pi roles.",
   "keywords": [
     "pi-package",
@@ -63,6 +63,7 @@
   "dependencies": {
     "@henryqw/pi-config-store": "^1.0.0",
     "@henryqw/pi-multi-codex": "^1.0.0",
+    "@henryqw/pi-process": "^0.1.0",
     "@henryqw/pi-task-models": "^5.0.0"
   }
 }