gsd-pi 2.49.0 → 2.50.0-dev.d210a87

package/dist/resources/extensions/gsd/bootstrap/register-hooks.js

@@ -3,7 +3,7 @@ import { isToolCallEventType } from "@gsd/pi-coding-agent";
 import { buildMilestoneFileName, resolveMilestonePath, resolveSliceFile, resolveSlicePath } from "../paths.js";
 import { buildBeforeAgentStartResult } from "./system-context.js";
 import { handleAgentEnd } from "./agent-end-recovery.js";
-import { isDepthVerified, isQueuePhaseActive, markDepthVerified, resetWriteGateState, shouldBlockContextWrite } from "./write-gate.js";
+import { clearDiscussionFlowState, isDepthVerified, isQueuePhaseActive, markDepthVerified, resetWriteGateState, shouldBlockContextWrite } from "./write-gate.js";
 import { isBlockedStateFile, isBashWriteToStateFile, BLOCKED_WRITE_ERROR } from "../write-intercept.js";
 import { getDiscussionMilestoneId } from "../guided-flow.js";
 import { loadToolApiKeys } from "../commands-config.js";
@@ -13,6 +13,8 @@ import { getAutoDashboardData, isAutoActive, isAutoPaused, markToolEnd, markTool
 import { isParallelActive, shutdownParallel } from "../parallel-orchestrator.js";
 import { checkToolCallLoop, resetToolCallLoopGuard } from "./tool-call-loop-guard.js";
 import { saveActivityLog } from "../activity-log.js";
+import { startRtkStatusUpdates, stopRtkStatusUpdates } from "../rtk-status.js";
+import { rewriteCommandWithRtk } from "../../shared/rtk.js";
 // Skip the welcome screen on the very first session_start — cli.ts already
 // printed it before the TUI launched. Only re-print on /clear (subsequent sessions).
 let isFirstSession = true;
@@ -21,10 +23,19 @@ async function syncServiceTierStatus(ctx) {
     ctx.ui.setStatus("gsd-fast", formatServiceTierFooterStatus(getEffectiveServiceTier(), ctx.model?.id));
 }
 export function registerHooks(pi) {
+    // Route all agent bash tool commands through RTK rewrite when opted in.
+    // This is a no-op when RTK is disabled or not installed.
+    pi.on("bash_transform", async (event) => {
+        const rewritten = rewriteCommandWithRtk(event.command);
+        if (rewritten === event.command)
+            return undefined;
+        return { command: rewritten };
+    });
     pi.on("session_start", async (_event, ctx) => {
         resetWriteGateState();
         resetToolCallLoopGuard();
         await syncServiceTierStatus(ctx);
+        startRtkStatusUpdates(ctx);
         // Apply show_token_cost preference (#1515)
         try {
             const { loadEffectiveGSDPreferences } = await import("../preferences.js");
@@ -63,6 +74,17 @@ export function registerHooks(pi) {
             // ignore
         }
     });
+    pi.on("session_switch", async (_event, ctx) => {
+        resetWriteGateState();
+        resetToolCallLoopGuard();
+        clearDiscussionFlowState();
+        await syncServiceTierStatus(ctx);
+        loadToolApiKeys();
+        startRtkStatusUpdates(ctx);
+    });
+    pi.on("session_fork", async (_event, ctx) => {
+        startRtkStatusUpdates(ctx);
+    });
     pi.on("before_agent_start", async (event, ctx) => {
         return buildBeforeAgentStartResult(event, ctx);
     });
@@ -108,6 +130,7 @@ export function registerHooks(pi) {
         }));
     });
     pi.on("session_shutdown", async (_event, ctx) => {
+        stopRtkStatusUpdates(ctx);
         if (isParallelActive()) {
             try {
                 await shutdownParallel(process.cwd());

package/dist/resources/extensions/gsd/branch-patterns.js

@@ -0,0 +1,13 @@
+/**
+ * GSD branch naming patterns — single source of truth.
+ *
+ * gsd/<worktree>/<milestone>/<slice>  → SLICE_BRANCH_RE
+ * gsd/quick/<id>-<slug>               → QUICK_BRANCH_RE
+ * gsd/<workflow>/<...>                 → WORKFLOW_BRANCH_RE (non-milestone gsd/ branches)
+ */
+/** Matches gsd/ slice branches: gsd/[worktree/]M001[-hash]/S01 */
+export const SLICE_BRANCH_RE = /^gsd\/(?:([a-zA-Z0-9_-]+)\/)?(M\d+(?:-[a-z0-9]{6})?)\/(S\d+)$/;
+/** Matches gsd/quick/ task branches */
+export const QUICK_BRANCH_RE = /^gsd\/quick\//;
+/** Matches gsd/ workflow branches (non-milestone, e.g. gsd/workflow-name/...) */
+export const WORKFLOW_BRANCH_RE = /^gsd\/(?!M\d)[\w-]+\//;

package/dist/resources/extensions/gsd/crash-recovery.js

@@ -63,14 +63,18 @@ export function readCrashLock(basePath) {
 /**
  * Check whether the process that wrote the lock is still running.
  * Uses `process.kill(pid, 0)` which sends no signal but checks liveness.
- * Returns false if the PID matches our own (recycled PID from a prior run).
+ * Returns true if the PID matches our own — we are the lock holder (#2470).
  */
 export function isLockProcessAlive(lock) {
     const pid = lock.pid;
     if (!Number.isInteger(pid) || pid <= 0)
         return false;
+    // Our own PID means WE hold this lock — we are alive. (#2470)
+    // Callers that need to distinguish "our lock" from "someone else's lock"
+    // (e.g. startAuto checking for a prior crashed session with a recycled PID)
+    // already guard with `crashLock.pid !== process.pid` before calling us.
     if (pid === process.pid)
-        return false;
+        return true;
     try {
         process.kill(pid, 0);
         return true;

package/dist/resources/extensions/gsd/custom-execution-policy.js

@@ -11,6 +11,7 @@
  * - recover() returns retry — simple default recovery strategy.
  */
 import { runCustomVerification } from "./custom-verification.js";
+import { parseUnitId } from "./unit-id.js";
 export class CustomExecutionPolicy {
     runDir;
     constructor(runDir) {
@@ -32,8 +33,8 @@ export class CustomExecutionPolicy {
      * to determine which policy to apply.
      */
     async verify(_unitType, unitId, _context) {
-        const parts = unitId.split("/");
-        const stepId = parts[parts.length - 1];
+        const { milestone, slice, task } = parseUnitId(unitId);
+        const stepId = task ?? slice ?? milestone;
         return runCustomVerification(this.runDir, stepId);
     }
     /** Default recovery: retry the step. */

package/dist/resources/extensions/gsd/custom-verification.js

@@ -20,6 +20,7 @@ import { readFileSync, existsSync, statSync } from "node:fs";
 import { resolve, sep } from "node:path";
 import { spawnSync } from "node:child_process";
 import { readFrozenDefinition } from "./custom-workflow-engine.js";
+import { rewriteCommandWithRtk } from "../shared/rtk.js";
 /**
  * Run custom verification for a specific step in a workflow run.
  *
@@ -131,7 +132,8 @@ function handleShellCommand(runDir, verify) {
         console.warn(`custom-verification: shell-command contains suspicious pattern, skipping: ${verify.command}`);
         return "pause";
     }
-    const result = spawnSync("sh", ["-c", verify.command], {
+    const rewrittenCommand = rewriteCommandWithRtk(verify.command);
+    const result = spawnSync("sh", ["-c", rewrittenCommand], {
         cwd: runDir,
         timeout: 30_000,
         encoding: "utf-8",

package/dist/resources/extensions/gsd/custom-workflow-engine.js

@@ -16,6 +16,7 @@ import { join } from "node:path";
 import { parse } from "yaml";
 import { readGraph, writeGraph, getNextPendingStep, markStepComplete, expandIteration, } from "./graph.js";
 import { injectContext } from "./context-injector.js";
+import { parseUnitId } from "./unit-id.js";
 /** Read and parse the frozen DEFINITION.yaml from a run directory. */
 export function readFrozenDefinition(runDir) {
     const defPath = join(runDir, "DEFINITION.yaml");
@@ -136,8 +137,8 @@ export class CustomWorkflowEngine {
     async reconcile(state, completedStep) {
         const graph = state.raw;
         // Extract stepId from "<workflowName>/<stepId>"
-        const parts = completedStep.unitId.split("/");
-        const stepId = parts[parts.length - 1];
+        const { milestone, slice, task } = parseUnitId(completedStep.unitId);
+        const stepId = task ?? slice ?? milestone;
         const updatedGraph = markStepComplete(graph, stepId);
         writeGraph(this.runDir, updatedGraph);
         const allDone = updatedGraph.steps.every((s) => s.status === "complete" || s.status === "expanded");

package/dist/resources/extensions/gsd/dashboard-overlay.js

@@ -482,6 +482,10 @@ export class GSDDashboardOverlay {
             if (cacheRate > 0) {
                 lines.push(row(`${th.fg("dim", "cache hit rate:")} ${th.fg("text", `${cacheRate}%`)}`));
             }
+            if (this.dashData.rtkEnabled && this.dashData.rtkSavings && this.dashData.rtkSavings.commands > 0) {
+                const rtk = this.dashData.rtkSavings;
+                lines.push(row(`${th.fg("dim", "rtk saved:")} ${th.fg("text", formatTokenCount(rtk.savedTokens))} ${th.fg("dim", `(${Math.round(rtk.savingsPct)}% · ${rtk.commands} cmd${rtk.commands === 1 ? "" : "s"})`)}`));
+            }
         }
         // Environment health section (#1221) — only show issues
         const envResults = runEnvironmentChecks(this.dashData.basePath || process.cwd());