gsd-pi 2.46.1-dev.79664f2 → 2.46.1-dev.eee1457

package/src/resources/extensions/gsd/state.ts

@@ -49,6 +49,7 @@ import {
   getReplanHistory,
   getSlice,
   insertMilestone,
+  updateTaskStatus,
   type MilestoneRow,
   type SliceRow,
   type TaskRow,
@@ -629,7 +630,38 @@ export async function deriveStateFromDb(basePath: string): Promise<GSDState> {
   }
 
   // ── Get tasks from DB ────────────────────────────────────────────────
-  const tasks = getSliceTasks(activeMilestone.id, activeSlice.id);
+  let tasks = getSliceTasks(activeMilestone.id, activeSlice.id);
+
+  // ── Reconcile stale task status (#2514) ──────────────────────────────
+  // When a session disconnects after the agent writes SUMMARY + VERIFY
+  // artifacts but before postUnitPostVerification updates the DB, tasks
+  // remain "pending" in the DB despite being complete on disk. Without
+  // reconciliation, deriveState keeps returning the stale task as active,
+  // causing the dispatcher to re-dispatch the same completed task forever.
+  let reconciled = false;
+  for (const t of tasks) {
+    if (isStatusDone(t.status)) continue;
+    const summaryPath = resolveTaskFile(basePath, activeMilestone.id, activeSlice.id, t.id, "SUMMARY");
+    if (summaryPath && existsSync(summaryPath)) {
+      try {
+        updateTaskStatus(activeMilestone.id, activeSlice.id, t.id, "complete");
+        process.stderr.write(
+          `gsd-reconcile: task ${activeMilestone.id}/${activeSlice.id}/${t.id} had SUMMARY on disk but DB status was "${t.status}" — updated to "complete" (#2514)\n`,
+        );
+        reconciled = true;
+      } catch (e) {
+        // DB write failed — continue with stale status rather than crash
+        process.stderr.write(
+          `gsd-reconcile: failed to update task ${t.id}: ${(e as Error).message}\n`,
+        );
+      }
+    }
+  }
+  // Re-fetch tasks if any were reconciled so downstream logic sees fresh status
+  if (reconciled) {
+    tasks = getSliceTasks(activeMilestone.id, activeSlice.id);
+  }
+
   const taskProgress = {
     done: tasks.filter(t => isStatusDone(t.status)).length,
     total: tasks.length,

package/src/resources/extensions/gsd/tests/inherited-repo-home-dir.test.ts

@@ -119,3 +119,73 @@ describe("isInheritedRepo when git root is HOME (#2393)", () => {
     );
   });
 });
+
+describe("isInheritedRepo with stale .gsd at parent git root", () => {
+  let parentRepo: string;
+
+  beforeEach(() => {
+    parentRepo = realpathSync(mkdtempSync(join(tmpdir(), "gsd-stale-parent-")));
+    run("git", ["init", "-b", "main"], parentRepo);
+    run("git", ["config", "user.name", "Test"], parentRepo);
+    run("git", ["config", "user.email", "test@example.com"], parentRepo);
+    writeFileSync(join(parentRepo, "README.md"), "# Parent\n", "utf-8");
+    run("git", ["add", "README.md"], parentRepo);
+    run("git", ["commit", "-m", "init"], parentRepo);
+  });
+
+  afterEach(() => {
+    rmSync(parentRepo, { recursive: true, force: true });
+  });
+
+  test("stale .gsd dir at parent git root does not suppress inherited detection", () => {
+    // Simulate a stale .gsd directory at the parent git root (e.g. from a
+    // prior doctor run or accidental init). This is a real directory, NOT
+    // a symlink, and NOT the global GSD home.
+    mkdirSync(join(parentRepo, ".gsd"), { recursive: true });
+
+    const projectDir = join(parentRepo, "my-project");
+    mkdirSync(projectDir, { recursive: true });
+
+    // Without fix: isProjectGsd(join(root, ".gsd")) returns true because
+    // the stale .gsd is a real directory that isn't the global GSD home,
+    // causing isInheritedRepo to return false (false negative).
+    //
+    // The stale .gsd at parent is still treated as a "project .gsd" by
+    // isProjectGsd(), so the git root check at line 128 returns false.
+    // This is the expected behavior for that check — the defense-in-depth
+    // fix in auto-start.ts handles this case by checking for local .git.
+    //
+    // Verify the function behavior is consistent:
+    assert.strictEqual(
+      isInheritedRepo(projectDir),
+      false,
+      "stale .gsd dir at git root still causes isInheritedRepo to return false " +
+      "(defense-in-depth in auto-start.ts handles this case)",
+    );
+  });
+
+  test("basePath's own .gsd symlink does not suppress inherited detection", () => {
+    // Create a project subdir with its own .gsd symlink (set up during
+    // the discuss phase, before auto-mode bootstrap runs).
+    const projectDir = join(parentRepo, "my-project");
+    mkdirSync(projectDir, { recursive: true });
+
+    const externalState = mkdtempSync(join(tmpdir(), "gsd-ext-state-"));
+    symlinkSync(externalState, join(projectDir, ".gsd"));
+
+    // Before fix: the walk-up loop started at normalizedBase (projectDir),
+    // found .gsd at projectDir, and returned false — even though projectDir
+    // has no .git of its own. The .gsd at basePath is irrelevant to whether
+    // the git repo is inherited from a parent.
+    //
+    // After fix: the walk-up starts at dirname(normalizedBase), skipping
+    // basePath's own .gsd.
+    assert.strictEqual(
+      isInheritedRepo(projectDir),
+      true,
+      "project's own .gsd symlink must not suppress inherited repo detection",
+    );
+
+    rmSync(externalState, { recursive: true, force: true });
+  });
+});