gsd-pi 2.28.0-dev.4009980 → 2.28.0-dev.e19bf89

package/dist/resources/extensions/gsd/auto-post-unit.ts

@@ -35,7 +35,6 @@ import {
 import { writeUnitRuntimeRecord, clearUnitRuntimeRecord } from "./unit-runtime.js";
 import { resolveAutoSupervisorConfig, loadEffectiveGSDPreferences } from "./preferences.js";
 import { runGSDDoctor, rebuildState, summarizeDoctorIssues } from "./doctor.js";
-import { COMPLETION_TRANSITION_CODES } from "./doctor-types.js";
 import { recordHealthSnapshot, checkHealEscalation } from "./doctor-proactive.js";
 import { syncStateToProjectRoot } from "./auto-worktree-sync.js";
 import { resetRewriteCircuitBreaker } from "./auto-dispatch.js";
@@ -155,17 +154,13 @@ export async function postUnitPreVerification(pctx: PostUnitContext): Promise<"d
         ctx.ui.notify(`Post-hook: applied ${report.fixesApplied.length} fix(es).`, "info");
       }
 
-      // Proactive health tracking — exclude completion-transition codes at task level
-      // since they are expected after the last task and resolved by complete-slice
-      const issuesForHealth = effectiveFixLevel === "task"
-        ? report.issues.filter(i => !COMPLETION_TRANSITION_CODES.has(i.code))
-        : report.issues;
-      const summary = summarizeDoctorIssues(issuesForHealth);
+      // Proactive health tracking
+      const summary = summarizeDoctorIssues(report.issues);
       recordHealthSnapshot(summary.errors, summary.warnings, report.fixesApplied.length);
 
       // Check if we should escalate to LLM-assisted heal
       if (summary.errors > 0) {
-        const unresolvedErrors = issuesForHealth
+        const unresolvedErrors = report.issues
           .filter(i => i.severity === "error" && !i.fixable)
           .map(i => ({ code: i.code, message: i.message, unitId: i.unitId }));
         const escalation = checkHealEscalation(summary.errors, unresolvedErrors);

package/dist/resources/extensions/gsd/auto-start.ts

@@ -26,13 +26,6 @@ import {
 import { invalidateAllCaches } from "./cache.js";
 import { synthesizeCrashRecovery } from "./session-forensics.js";
 import { writeLock, clearLock, readCrashLock, formatCrashInfo, isLockProcessAlive } from "./crash-recovery.js";
-import {
-  acquireSessionLock,
-  updateSessionLock,
-  releaseSessionLock,
-  readSessionLockData,
-  isSessionLockProcessAlive,
-} from "./session-lock.js";
 import { selfHealRuntimeRecords } from "./auto-recovery.js";
 import { ensureGitignore, untrackRuntimeFiles } from "./gitignore.js";
 import { nativeIsRepo, nativeInit, nativeAddAll, nativeCommit } from "./native-git-bridge.js";
@@ -88,18 +81,6 @@ export async function bootstrapAutoSession(
 ): Promise<boolean> {
   const { shouldUseWorktreeIsolation, registerSigtermHandler, lockBase } = deps;
 
-  // ── Session lock: acquire FIRST, before any state mutation ──────────────
-  // This is the primary guard against concurrent sessions on the same project.
-  // Uses OS-level file locking (proper-lockfile) to prevent TOCTOU races.
-  const lockResult = acquireSessionLock(base);
-  if (!lockResult.acquired) {
-    ctx.ui.notify(
-      `${lockResult.reason}\nStop it with \`kill ${lockResult.existingPid ?? "the other process"}\` before starting a new session.`,
-      "error",
-    );
-    return false;
-  }
-
   // Ensure git repo exists
   if (!nativeIsRepo(base)) {
     const mainBranch = loadEffectiveGSDPreferences()?.preferences?.git?.main_branch || "main";
@@ -128,11 +109,16 @@ export async function bootstrapAutoSession(
   // Initialize GitServiceImpl
   s.gitService = new GitServiceImpl(s.basePath, loadEffectiveGSDPreferences()?.preferences?.git ?? {});
 
-  // Check for crash from previous session (use both old and new lock data)
+  // Check for crash from previous session
   const crashLock = readCrashLock(base);
   if (crashLock) {
-    // We already hold the session lock, so no concurrent session is running.
-    // The crash lock is from a dead process — recover context from it.
+    if (isLockProcessAlive(crashLock)) {
+      ctx.ui.notify(
+        `Another auto-mode session (PID ${crashLock.pid}) appears to be running.\nStop it with \`kill ${crashLock.pid}\` before starting a new session.`,
+        "error",
+      );
+      return false;
+    }
     const recoveredMid = crashLock.unitId.split("/")[0];
     const milestoneAlreadyComplete = recoveredMid
       ? !!resolveMilestoneFile(base, recoveredMid, "SUMMARY")
@@ -421,8 +407,7 @@ export async function bootstrapAutoSession(
     : "Will loop until milestone complete.";
   ctx.ui.notify(`${modeLabel} started. ${scopeMsg}`, "info");
 
-  // Update lock file with milestone info (OS lock already acquired at bootstrap start)
-  updateSessionLock(lockBase(), "starting", s.currentMilestoneId ?? "unknown", 0);
+  // Write initial lock file
   writeLock(lockBase(), "starting", s.currentMilestoneId ?? "unknown", 0);
 
   // Secrets collection gate — pause instead of blocking (#1146)

package/dist/resources/extensions/gsd/auto.ts

@@ -33,12 +33,6 @@ import { invalidateAllCaches } from "./cache.js";
 import { saveActivityLog, clearActivityLogState } from "./activity-log.js";
 import { synthesizeCrashRecovery, getDeepDiagnostic } from "./session-forensics.js";
 import { writeLock, clearLock, readCrashLock, formatCrashInfo, isLockProcessAlive } from "./crash-recovery.js";
-import {
-  acquireSessionLock,
-  validateSessionLock,
-  releaseSessionLock,
-  updateSessionLock,
-} from "./session-lock.js";
 import {
   clearUnitRuntimeRecord,
   inspectExecuteTaskDurability,
@@ -457,10 +451,7 @@ export async function stopAuto(ctx?: ExtensionContext, pi?: ExtensionAPI, reason
   if (!s.active && !s.paused) return;
   const reasonSuffix = reason ? ` — ${reason}` : "";
   clearUnitTimeout();
-  if (lockBase()) {
-    releaseSessionLock(lockBase());
-    clearLock(lockBase());
-  }
+  if (lockBase()) clearLock(lockBase());
   clearSkillSnapshot();
   resetSkillTelemetry();
   s.dispatching = false;
@@ -574,10 +565,7 @@ export async function pauseAuto(ctx?: ExtensionContext, _pi?: ExtensionAPI): Pro
 
   s.pausedSessionFile = ctx?.sessionManager?.getSessionFile() ?? null;
 
-  if (lockBase()) {
-    releaseSessionLock(lockBase());
-    clearLock(lockBase());
-  }
+  if (lockBase()) clearLock(lockBase());
 
   deregisterSigtermHandler();
 
@@ -610,16 +598,6 @@ export async function startAuto(
 
   // If resuming from paused state, just re-activate and dispatch next unit.
   if (s.paused) {
-    // Re-acquire session lock before resuming
-    const resumeLock = acquireSessionLock(base);
-    if (!resumeLock.acquired) {
-      ctx.ui.notify(
-        `Cannot resume: ${resumeLock.reason}`,
-        "error",
-      );
-      return;
-    }
-
     s.paused = false;
     s.active = true;
     s.verbose = verboseMode;
@@ -721,7 +699,6 @@ export async function startAuto(
       s.pausedForSecrets = false;
     }
 
-    updateSessionLock(lockBase(), "resuming", s.currentMilestoneId ?? "unknown", s.completedUnits.length);
     writeLock(lockBase(), "resuming", s.currentMilestoneId ?? "unknown", s.completedUnits.length);
 
     await dispatchNextUnit(ctx, pi);
@@ -973,24 +950,6 @@ async function dispatchNextUnit(
     return;
   }
 
-  // ── Session lock validation: detect if another process has taken over ──
-  if (lockBase() && !validateSessionLock(lockBase())) {
-    debugLog("dispatchNextUnit session-lock-lost — another process may have taken over");
-    ctx.ui.notify(
-      "Session lock lost — another GSD process appears to have taken over. Stopping gracefully.",
-      "error",
-    );
-    // Don't call stopAuto here to avoid releasing the lock we don't own
-    s.active = false;
-    s.paused = false;
-    clearUnitTimeout();
-    deregisterSigtermHandler();
-    ctx.ui.setStatus("gsd-auto", undefined);
-    ctx.ui.setWidget("gsd-progress", undefined);
-    ctx.ui.setFooter(undefined);
-    return;
-  }
-
   // Reentrancy guard
   if (s.dispatching && s.skipDepth === 0) {
     debugLog("dispatchNextUnit reentrancy guard — another dispatch in progress, bailing");
@@ -1624,7 +1583,6 @@ async function dispatchNextUnit(
   }
 
   const sessionFile = ctx.sessionManager.getSessionFile();
-  updateSessionLock(lockBase(), unitType, unitId, s.completedUnits.length, sessionFile);
   writeLock(lockBase(), unitType, unitId, s.completedUnits.length, sessionFile);
 
   // Prompt injection
@@ -1851,7 +1809,6 @@ export async function dispatchHookUnit(
   }
 
   const sessionFile = ctx.sessionManager.getSessionFile();
-  updateSessionLock(lockBase(), hookUnitType, triggerUnitId, s.completedUnits.length, sessionFile);
   writeLock(lockBase(), hookUnitType, triggerUnitId, s.completedUnits.length, sessionFile);
 
   clearUnitTimeout();

package/dist/resources/extensions/gsd/commands.ts

@@ -43,7 +43,6 @@ import { handleConfig } from "./commands-config.js";
 import { handleInspect } from "./commands-inspect.js";
 import { handleCleanupBranches, handleCleanupSnapshots, handleSkip, handleDryRun } from "./commands-maintenance.js";
 import { handleDoctor, handleSteer, handleCapture, handleTriage, handleKnowledge, handleRunHook, handleUpdate, handleSkillHealth } from "./commands-handlers.js";
-import { handleLogs } from "./commands-logs.js";
 
 // ─── Re-exports (preserve public API surface) ───────────────────────────────
 export { handlePrefs, handlePrefsMode, handlePrefsWizard, ensurePreferencesFile, handleImportClaude, buildCategorySummaries, serializePreferencesToFrontmatter, yamlSafeString, configureMode } from "./commands-prefs-wizard.js";
@@ -108,7 +107,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
         { cmd: "run-hook", desc: "Manually trigger a specific hook" },
         { cmd: "skill-health", desc: "Skill lifecycle dashboard" },
         { cmd: "doctor", desc: "Runtime health checks with auto-fix" },
-        { cmd: "logs", desc: "Browse activity logs, debug logs, and metrics" },
         { cmd: "forensics", desc: "Examine execution logs" },
         { cmd: "init", desc: "Project init wizard — detect, configure, bootstrap .gsd/" },
         { cmd: "setup", desc: "Global setup status and configuration" },
@@ -186,18 +184,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
           .map((s) => ({ value: `setup ${s.cmd}`, label: s.cmd, description: s.desc }));
       }
 
-      if (parts[0] === "logs" && parts.length <= 2) {
-        const subPrefix = parts[1] ?? "";
-        const subs = [
-          { cmd: "debug", desc: "List or view debug log files" },
-          { cmd: "tail", desc: "Show last N activity log summaries" },
-          { cmd: "clear", desc: "Remove old activity and debug logs" },
-        ];
-        return subs
-          .filter((s) => s.cmd.startsWith(subPrefix))
-          .map((s) => ({ value: `logs ${s.cmd}`, label: s.cmd, description: s.desc }));
-      }
-
       if (parts[0] === "keys" && parts.length <= 2) {
         const subPrefix = parts[1] ?? "";
         const subs = [
@@ -406,11 +392,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
         return;
       }
 
-      if (trimmed === "logs" || trimmed.startsWith("logs ")) {
-        await handleLogs(trimmed.replace(/^logs\s*/, "").trim(), ctx);
-        return;
-      }
-
       if (trimmed === "forensics" || trimmed.startsWith("forensics ")) {
         const { handleForensics } = await import("./forensics.js");
         await handleForensics(trimmed.replace(/^forensics\s*/, "").trim(), ctx, pi);

package/dist/resources/extensions/gsd/doctor-types.ts

@@ -32,19 +32,6 @@ export type DoctorIssueCode =
   | "gitignore_missing_patterns"
   | "unresolvable_dependency";
 
-/**
- * Issue codes that represent expected completion-transition states.
- * These are detected by the doctor but should NOT be auto-fixed at task level —
- * they are resolved by the complete-slice/complete-milestone dispatch units.
- * Consumers (e.g. auto-post-unit health tracking) should exclude these from
- * error counts when running at task fixLevel to avoid false escalation.
- */
-export const COMPLETION_TRANSITION_CODES = new Set<DoctorIssueCode>([
-  "all_tasks_done_missing_slice_summary",
-  "all_tasks_done_missing_slice_uat",
-  "all_tasks_done_roadmap_not_checked",
-]);
-
 export interface DoctorIssue {
   severity: DoctorSeverity;
   code: DoctorIssueCode;

package/dist/resources/extensions/gsd/doctor.ts

@@ -8,7 +8,6 @@ import { invalidateAllCaches } from "./cache.js";
 import { loadEffectiveGSDPreferences, type GSDPreferences } from "./preferences.js";
 
 import type { DoctorIssue, DoctorIssueCode } from "./doctor-types.js";
-import { COMPLETION_TRANSITION_CODES } from "./doctor-types.js";
 import { checkGitHealth, checkRuntimeHealth } from "./doctor-checks.js";
 
 // ── Re-exports ─────────────────────────────────────────────────────────────
@@ -357,11 +356,16 @@ export async function runGSDDoctor(basePath: string, options?: { fix?: boolean;
   // dispatch lifecycle (complete-slice, complete-milestone units), not to
   // mechanical post-hook bookkeeping. When fixLevel is "task", these are
   // detected and reported but never auto-fixed.
+  const completionTransitionCodes = new Set<DoctorIssueCode>([
+    "all_tasks_done_missing_slice_summary",
+    "all_tasks_done_missing_slice_uat",
+    "all_tasks_done_roadmap_not_checked",
+  ]);
 
   /** Whether a given issue code should be auto-fixed at the current fixLevel. */
   const shouldFix = (code: DoctorIssueCode): boolean => {
     if (!fix) return false;
-    if (fixLevel === "task" && COMPLETION_TRANSITION_CODES.has(code)) return false;
+    if (fixLevel === "task" && completionTransitionCodes.has(code)) return false;
     return true;
   };
 

package/dist/resources/extensions/gsd/gsd-db.ts

@@ -348,8 +348,6 @@ function migrateSchema(db: DbAdapter): void {
 
 let currentDb: DbAdapter | null = null;
 let currentPath: string | null = null;
-/** PID that opened the current connection — used for diagnostic logging. */
-let currentPid: number = 0;
 
 // ─── Public API ────────────────────────────────────────────────────────────
 
@@ -397,7 +395,6 @@ export function openDatabase(path: string): boolean {
 
   currentDb = adapter;
   currentPath = path;
-  currentPid = process.pid;
   return true;
 }
 
@@ -413,7 +410,6 @@ export function closeDatabase(): void {
     }
     currentDb = null;
     currentPath = null;
-    currentPid = 0;
   }
 }
 
@@ -728,21 +724,6 @@ export function reconcileWorktreeDb(
   }
 }
 
-/**
- * Returns the PID of the process that opened the current DB connection.
- * Returns 0 if no connection is open.
- */
-export function getDbOwnerPid(): number {
-  return currentPid;
-}
-
-/**
- * Returns the path of the currently open database, or null if none.
- */
-export function getDbPath(): string | null {
-  return currentPath;
-}
-
 // ─── Internal Access (for testing) ─────────────────────────────────────────
 
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gsd-pi",
-  "version": "2.28.0-dev.4009980",
+  "version": "2.28.0-dev.e19bf89",
   "description": "GSD — Get Shit Done coding agent",
   "license": "MIT",
   "repository": {

package/src/resources/extensions/gsd/auto-post-unit.ts

@@ -35,7 +35,6 @@ import {
 import { writeUnitRuntimeRecord, clearUnitRuntimeRecord } from "./unit-runtime.js";
 import { resolveAutoSupervisorConfig, loadEffectiveGSDPreferences } from "./preferences.js";
 import { runGSDDoctor, rebuildState, summarizeDoctorIssues } from "./doctor.js";
-import { COMPLETION_TRANSITION_CODES } from "./doctor-types.js";
 import { recordHealthSnapshot, checkHealEscalation } from "./doctor-proactive.js";
 import { syncStateToProjectRoot } from "./auto-worktree-sync.js";
 import { resetRewriteCircuitBreaker } from "./auto-dispatch.js";
@@ -155,17 +154,13 @@ export async function postUnitPreVerification(pctx: PostUnitContext): Promise<"d
         ctx.ui.notify(`Post-hook: applied ${report.fixesApplied.length} fix(es).`, "info");
       }
 
-      // Proactive health tracking — exclude completion-transition codes at task level
-      // since they are expected after the last task and resolved by complete-slice
-      const issuesForHealth = effectiveFixLevel === "task"
-        ? report.issues.filter(i => !COMPLETION_TRANSITION_CODES.has(i.code))
-        : report.issues;
-      const summary = summarizeDoctorIssues(issuesForHealth);
+      // Proactive health tracking
+      const summary = summarizeDoctorIssues(report.issues);
       recordHealthSnapshot(summary.errors, summary.warnings, report.fixesApplied.length);
 
       // Check if we should escalate to LLM-assisted heal
       if (summary.errors > 0) {
-        const unresolvedErrors = issuesForHealth
+        const unresolvedErrors = report.issues
           .filter(i => i.severity === "error" && !i.fixable)
           .map(i => ({ code: i.code, message: i.message, unitId: i.unitId }));
         const escalation = checkHealEscalation(summary.errors, unresolvedErrors);

package/src/resources/extensions/gsd/auto-start.ts

@@ -26,13 +26,6 @@ import {
 import { invalidateAllCaches } from "./cache.js";
 import { synthesizeCrashRecovery } from "./session-forensics.js";
 import { writeLock, clearLock, readCrashLock, formatCrashInfo, isLockProcessAlive } from "./crash-recovery.js";
-import {
-  acquireSessionLock,
-  updateSessionLock,
-  releaseSessionLock,
-  readSessionLockData,
-  isSessionLockProcessAlive,
-} from "./session-lock.js";
 import { selfHealRuntimeRecords } from "./auto-recovery.js";
 import { ensureGitignore, untrackRuntimeFiles } from "./gitignore.js";
 import { nativeIsRepo, nativeInit, nativeAddAll, nativeCommit } from "./native-git-bridge.js";
@@ -88,18 +81,6 @@ export async function bootstrapAutoSession(
 ): Promise<boolean> {
   const { shouldUseWorktreeIsolation, registerSigtermHandler, lockBase } = deps;
 
-  // ── Session lock: acquire FIRST, before any state mutation ──────────────
-  // This is the primary guard against concurrent sessions on the same project.
-  // Uses OS-level file locking (proper-lockfile) to prevent TOCTOU races.
-  const lockResult = acquireSessionLock(base);
-  if (!lockResult.acquired) {
-    ctx.ui.notify(
-      `${lockResult.reason}\nStop it with \`kill ${lockResult.existingPid ?? "the other process"}\` before starting a new session.`,
-      "error",
-    );
-    return false;
-  }
-
   // Ensure git repo exists
   if (!nativeIsRepo(base)) {
     const mainBranch = loadEffectiveGSDPreferences()?.preferences?.git?.main_branch || "main";
@@ -128,11 +109,16 @@ export async function bootstrapAutoSession(
   // Initialize GitServiceImpl
   s.gitService = new GitServiceImpl(s.basePath, loadEffectiveGSDPreferences()?.preferences?.git ?? {});
 
-  // Check for crash from previous session (use both old and new lock data)
+  // Check for crash from previous session
   const crashLock = readCrashLock(base);
   if (crashLock) {
-    // We already hold the session lock, so no concurrent session is running.
-    // The crash lock is from a dead process — recover context from it.
+    if (isLockProcessAlive(crashLock)) {
+      ctx.ui.notify(
+        `Another auto-mode session (PID ${crashLock.pid}) appears to be running.\nStop it with \`kill ${crashLock.pid}\` before starting a new session.`,
+        "error",
+      );
+      return false;
+    }
     const recoveredMid = crashLock.unitId.split("/")[0];
     const milestoneAlreadyComplete = recoveredMid
       ? !!resolveMilestoneFile(base, recoveredMid, "SUMMARY")
@@ -421,8 +407,7 @@ export async function bootstrapAutoSession(
     : "Will loop until milestone complete.";
   ctx.ui.notify(`${modeLabel} started. ${scopeMsg}`, "info");
 
-  // Update lock file with milestone info (OS lock already acquired at bootstrap start)
-  updateSessionLock(lockBase(), "starting", s.currentMilestoneId ?? "unknown", 0);
+  // Write initial lock file
   writeLock(lockBase(), "starting", s.currentMilestoneId ?? "unknown", 0);
 
   // Secrets collection gate — pause instead of blocking (#1146)

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

@@ -33,12 +33,6 @@ import { invalidateAllCaches } from "./cache.js";
 import { saveActivityLog, clearActivityLogState } from "./activity-log.js";
 import { synthesizeCrashRecovery, getDeepDiagnostic } from "./session-forensics.js";
 import { writeLock, clearLock, readCrashLock, formatCrashInfo, isLockProcessAlive } from "./crash-recovery.js";
-import {
-  acquireSessionLock,
-  validateSessionLock,
-  releaseSessionLock,
-  updateSessionLock,
-} from "./session-lock.js";
 import {
   clearUnitRuntimeRecord,
   inspectExecuteTaskDurability,
@@ -457,10 +451,7 @@ export async function stopAuto(ctx?: ExtensionContext, pi?: ExtensionAPI, reason
   if (!s.active && !s.paused) return;
   const reasonSuffix = reason ? ` — ${reason}` : "";
   clearUnitTimeout();
-  if (lockBase()) {
-    releaseSessionLock(lockBase());
-    clearLock(lockBase());
-  }
+  if (lockBase()) clearLock(lockBase());
   clearSkillSnapshot();
   resetSkillTelemetry();
   s.dispatching = false;
@@ -574,10 +565,7 @@ export async function pauseAuto(ctx?: ExtensionContext, _pi?: ExtensionAPI): Pro
 
   s.pausedSessionFile = ctx?.sessionManager?.getSessionFile() ?? null;
 
-  if (lockBase()) {
-    releaseSessionLock(lockBase());
-    clearLock(lockBase());
-  }
+  if (lockBase()) clearLock(lockBase());
 
   deregisterSigtermHandler();
 
@@ -610,16 +598,6 @@ export async function startAuto(
 
   // If resuming from paused state, just re-activate and dispatch next unit.
   if (s.paused) {
-    // Re-acquire session lock before resuming
-    const resumeLock = acquireSessionLock(base);
-    if (!resumeLock.acquired) {
-      ctx.ui.notify(
-        `Cannot resume: ${resumeLock.reason}`,
-        "error",
-      );
-      return;
-    }
-
     s.paused = false;
     s.active = true;
     s.verbose = verboseMode;
@@ -721,7 +699,6 @@ export async function startAuto(
       s.pausedForSecrets = false;
     }
 
-    updateSessionLock(lockBase(), "resuming", s.currentMilestoneId ?? "unknown", s.completedUnits.length);
     writeLock(lockBase(), "resuming", s.currentMilestoneId ?? "unknown", s.completedUnits.length);
 
     await dispatchNextUnit(ctx, pi);
@@ -973,24 +950,6 @@ async function dispatchNextUnit(
     return;
   }
 
-  // ── Session lock validation: detect if another process has taken over ──
-  if (lockBase() && !validateSessionLock(lockBase())) {
-    debugLog("dispatchNextUnit session-lock-lost — another process may have taken over");
-    ctx.ui.notify(
-      "Session lock lost — another GSD process appears to have taken over. Stopping gracefully.",
-      "error",
-    );
-    // Don't call stopAuto here to avoid releasing the lock we don't own
-    s.active = false;
-    s.paused = false;
-    clearUnitTimeout();
-    deregisterSigtermHandler();
-    ctx.ui.setStatus("gsd-auto", undefined);
-    ctx.ui.setWidget("gsd-progress", undefined);
-    ctx.ui.setFooter(undefined);
-    return;
-  }
-
   // Reentrancy guard
   if (s.dispatching && s.skipDepth === 0) {
     debugLog("dispatchNextUnit reentrancy guard — another dispatch in progress, bailing");
@@ -1624,7 +1583,6 @@ async function dispatchNextUnit(
   }
 
   const sessionFile = ctx.sessionManager.getSessionFile();
-  updateSessionLock(lockBase(), unitType, unitId, s.completedUnits.length, sessionFile);
   writeLock(lockBase(), unitType, unitId, s.completedUnits.length, sessionFile);
 
   // Prompt injection
@@ -1851,7 +1809,6 @@ export async function dispatchHookUnit(
   }
 
   const sessionFile = ctx.sessionManager.getSessionFile();
-  updateSessionLock(lockBase(), hookUnitType, triggerUnitId, s.completedUnits.length, sessionFile);
   writeLock(lockBase(), hookUnitType, triggerUnitId, s.completedUnits.length, sessionFile);
 
   clearUnitTimeout();

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

@@ -43,7 +43,6 @@ import { handleConfig } from "./commands-config.js";
 import { handleInspect } from "./commands-inspect.js";
 import { handleCleanupBranches, handleCleanupSnapshots, handleSkip, handleDryRun } from "./commands-maintenance.js";
 import { handleDoctor, handleSteer, handleCapture, handleTriage, handleKnowledge, handleRunHook, handleUpdate, handleSkillHealth } from "./commands-handlers.js";
-import { handleLogs } from "./commands-logs.js";
 
 // ─── Re-exports (preserve public API surface) ───────────────────────────────
 export { handlePrefs, handlePrefsMode, handlePrefsWizard, ensurePreferencesFile, handleImportClaude, buildCategorySummaries, serializePreferencesToFrontmatter, yamlSafeString, configureMode } from "./commands-prefs-wizard.js";
@@ -108,7 +107,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
         { cmd: "run-hook", desc: "Manually trigger a specific hook" },
         { cmd: "skill-health", desc: "Skill lifecycle dashboard" },
         { cmd: "doctor", desc: "Runtime health checks with auto-fix" },
-        { cmd: "logs", desc: "Browse activity logs, debug logs, and metrics" },
         { cmd: "forensics", desc: "Examine execution logs" },
         { cmd: "init", desc: "Project init wizard — detect, configure, bootstrap .gsd/" },
         { cmd: "setup", desc: "Global setup status and configuration" },
@@ -186,18 +184,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
           .map((s) => ({ value: `setup ${s.cmd}`, label: s.cmd, description: s.desc }));
       }
 
-      if (parts[0] === "logs" && parts.length <= 2) {
-        const subPrefix = parts[1] ?? "";
-        const subs = [
-          { cmd: "debug", desc: "List or view debug log files" },
-          { cmd: "tail", desc: "Show last N activity log summaries" },
-          { cmd: "clear", desc: "Remove old activity and debug logs" },
-        ];
-        return subs
-          .filter((s) => s.cmd.startsWith(subPrefix))
-          .map((s) => ({ value: `logs ${s.cmd}`, label: s.cmd, description: s.desc }));
-      }
-
       if (parts[0] === "keys" && parts.length <= 2) {
         const subPrefix = parts[1] ?? "";
         const subs = [
@@ -406,11 +392,6 @@ export function registerGSDCommand(pi: ExtensionAPI): void {
         return;
       }
 
-      if (trimmed === "logs" || trimmed.startsWith("logs ")) {
-        await handleLogs(trimmed.replace(/^logs\s*/, "").trim(), ctx);
-        return;
-      }
-
       if (trimmed === "forensics" || trimmed.startsWith("forensics ")) {
         const { handleForensics } = await import("./forensics.js");
         await handleForensics(trimmed.replace(/^forensics\s*/, "").trim(), ctx, pi);

package/src/resources/extensions/gsd/doctor-types.ts

@@ -32,19 +32,6 @@ export type DoctorIssueCode =
   | "gitignore_missing_patterns"
   | "unresolvable_dependency";
 
-/**
- * Issue codes that represent expected completion-transition states.
- * These are detected by the doctor but should NOT be auto-fixed at task level —
- * they are resolved by the complete-slice/complete-milestone dispatch units.
- * Consumers (e.g. auto-post-unit health tracking) should exclude these from
- * error counts when running at task fixLevel to avoid false escalation.
- */
-export const COMPLETION_TRANSITION_CODES = new Set<DoctorIssueCode>([
-  "all_tasks_done_missing_slice_summary",
-  "all_tasks_done_missing_slice_uat",
-  "all_tasks_done_roadmap_not_checked",
-]);
-
 export interface DoctorIssue {
   severity: DoctorSeverity;
   code: DoctorIssueCode;

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

@@ -8,7 +8,6 @@ import { invalidateAllCaches } from "./cache.js";
 import { loadEffectiveGSDPreferences, type GSDPreferences } from "./preferences.js";
 
 import type { DoctorIssue, DoctorIssueCode } from "./doctor-types.js";
-import { COMPLETION_TRANSITION_CODES } from "./doctor-types.js";
 import { checkGitHealth, checkRuntimeHealth } from "./doctor-checks.js";
 
 // ── Re-exports ─────────────────────────────────────────────────────────────
@@ -357,11 +356,16 @@ export async function runGSDDoctor(basePath: string, options?: { fix?: boolean;
   // dispatch lifecycle (complete-slice, complete-milestone units), not to
   // mechanical post-hook bookkeeping. When fixLevel is "task", these are
   // detected and reported but never auto-fixed.
+  const completionTransitionCodes = new Set<DoctorIssueCode>([
+    "all_tasks_done_missing_slice_summary",
+    "all_tasks_done_missing_slice_uat",
+    "all_tasks_done_roadmap_not_checked",
+  ]);
 
   /** Whether a given issue code should be auto-fixed at the current fixLevel. */
   const shouldFix = (code: DoctorIssueCode): boolean => {
     if (!fix) return false;
-    if (fixLevel === "task" && COMPLETION_TRANSITION_CODES.has(code)) return false;
+    if (fixLevel === "task" && completionTransitionCodes.has(code)) return false;
     return true;
   };