@shepai/cli 1.128.0 → 1.129.0-pr410.71f3ab6

package/dist/packages/core/src/application/use-cases/features/delete-feature.use-case.d.ts

@@ -21,6 +21,7 @@ import type { IGitPrService } from '../../ports/output/services/git-pr-service.i
 export interface DeleteFeatureOptions {
     cleanup?: boolean;
     cascadeDelete?: boolean;
+    closePr?: boolean;
 }
 export declare class DeleteFeatureUseCase {
     private readonly featureRepo;

package/dist/packages/core/src/application/use-cases/features/delete-feature.use-case.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-feature.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/delete-feature.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qCAAqC,CAAC;AAEnE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2DAA2D,CAAC;AAClG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,yDAAyD,CAAC;AAE7F,MAAM,WAAW,oBAAoB;IACnC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED,qBACa,oBAAoB;IAEC,OAAO,CAAC,QAAQ,CAAC,WAAW;IAC9B,OAAO,CAAC,QAAQ,CAAC,eAAe;IAE5D,OAAO,CAAC,QAAQ,CAAC,cAAc;IACA,OAAO,CAAC,QAAQ,CAAC,OAAO;IAC9B,OAAO,CAAC,QAAQ,CAAC,YAAY;gBALP,WAAW,EAAE,kBAAkB,EACjC,eAAe,EAAE,gBAAgB,EAE7D,cAAc,EAAE,2BAA2B,EACZ,OAAO,EAAE,mBAAmB,EAClC,YAAY,EAAE,aAAa;IAGjE,OAAO,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,oBAAoB,GAAG,OAAO,CAAC,OAAO,CAAC;IA8BlF,oGAAoG;YACtF,gBAAgB;IAW9B,0DAA0D;YAC5C,iBAAiB;IAQ/B,6DAA6D;YAC/C,yBAAyB;IAWvC,sDAAsD;YACxC,sBAAsB;IAYpC,6EAA6E;YAC/D,oBAAoB;CAsDnC"}
+{"version":3,"file":"delete-feature.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/features/delete-feature.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qCAAqC,CAAC;AAEnE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,iEAAiE,CAAC;AAC1G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,2DAA2D,CAAC;AAClG,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,8DAA8D,CAAC;AAChH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,6DAA6D,CAAC;AACvG,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,yDAAyD,CAAC;AAE7F,MAAM,WAAW,oBAAoB;IACnC,OAAO,CAAC,EAAE,OAAO,CAAC;IAClB,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED,qBACa,oBAAoB;IAEC,OAAO,CAAC,QAAQ,CAAC,WAAW;IAC9B,OAAO,CAAC,QAAQ,CAAC,eAAe;IAE5D,OAAO,CAAC,QAAQ,CAAC,cAAc;IACA,OAAO,CAAC,QAAQ,CAAC,OAAO;IAC9B,OAAO,CAAC,QAAQ,CAAC,YAAY;gBALP,WAAW,EAAE,kBAAkB,EACjC,eAAe,EAAE,gBAAgB,EAE7D,cAAc,EAAE,2BAA2B,EACZ,OAAO,EAAE,mBAAmB,EAClC,YAAY,EAAE,aAAa;IAGjE,OAAO,CAAC,SAAS,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,oBAAoB,GAAG,OAAO,CAAC,OAAO,CAAC;IA8BlF,oGAAoG;YACtF,gBAAgB;IAW9B,0DAA0D;YAC5C,iBAAiB;IAQ/B,6DAA6D;YAC/C,yBAAyB;IAevC,sDAAsD;YACxC,sBAAsB;IAYpC,6EAA6E;YAC/D,oBAAoB;CA+EnC"}

package/dist/packages/core/src/application/use-cases/features/delete-feature.use-case.js

@@ -25,7 +25,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
     return function (target, key) { decorator(target, key, paramIndex); }
 };
 import { injectable, inject } from 'tsyringe';
-import { AgentRunStatus, SdlcLifecycle } from '../../../domain/generated/output.js';
+import { AgentRunStatus, PrStatus, SdlcLifecycle } from '../../../domain/generated/output.js';
 let DeleteFeatureUseCase = class DeleteFeatureUseCase {
     featureRepo;
     worktreeService;
@@ -93,6 +93,10 @@ let DeleteFeatureUseCase = class DeleteFeatureUseCase {
         });
         // Soft-delete: sets deletedAt, hiding from normal queries
         await this.featureRepo.softDelete(feature.id);
+        // Sync the in-memory feature object so subsequent update() calls
+        // (e.g. PR status update in cleanup) don't overwrite deleted_at back to null.
+        feature.lifecycle = SdlcLifecycle.Deleting;
+        feature.deletedAt = new Date();
     }
     /** Recursively cleanup all children (depth-first). */
     async cascadeCleanupChildren(parentId, options) {
@@ -133,22 +137,49 @@ let DeleteFeatureUseCase = class DeleteFeatureUseCase {
         }
         const cleanup = options?.cleanup !== false;
         if (cleanup) {
-            // Delete local feature branch
-            try {
-                await this.gitPrService.deleteBranch(feature.repositoryPath, feature.branch);
-            }
-            catch {
-                // Branch might not exist
-            }
-            // Delete remote feature branch if it exists
-            try {
-                const remoteExists = await this.worktreeService.remoteBranchExists(feature.repositoryPath, feature.branch);
-                if (remoteExists) {
-                    await this.gitPrService.deleteBranch(feature.repositoryPath, feature.branch, true);
+            const shouldDeleteRemote = options?.closePr !== false;
+            // Delete local branch, and also remote if closePr is enabled.
+            // deleteBranch(cwd, branch, true) deletes local then remote in one call,
+            // so we must NOT delete local separately when also deleting remote.
+            if (shouldDeleteRemote) {
+                let remoteDeleted = false;
+                try {
+                    const remoteExists = await this.worktreeService.remoteBranchExists(feature.repositoryPath, feature.branch);
+                    if (remoteExists) {
+                        // Delete local + remote in one call (deleteBranch does local first, then remote)
+                        await this.gitPrService.deleteBranch(feature.repositoryPath, feature.branch, true);
+                        remoteDeleted = true;
+                    }
+                    else {
+                        // No remote branch — just delete local
+                        await this.gitPrService.deleteBranch(feature.repositoryPath, feature.branch);
+                    }
+                }
+                catch {
+                    // Branch cleanup is best-effort
+                }
+                // Update pr.status to Closed after remote branch deletion (GitHub auto-closes the PR)
+                if (remoteDeleted && feature.pr?.status === PrStatus.Open) {
+                    try {
+                        await this.featureRepo.update({
+                            ...feature,
+                            pr: { ...feature.pr, status: PrStatus.Closed },
+                            updatedAt: new Date(),
+                        });
+                    }
+                    catch {
+                        // PR status update is best-effort
+                    }
                 }
             }
-            catch {
-                // Remote branch cleanup is best-effort
+            else {
+                // closePr is false — only delete local branch, preserve remote (and the PR)
+                try {
+                    await this.gitPrService.deleteBranch(feature.repositoryPath, feature.branch);
+                }
+                catch {
+                    // Branch might not exist
+                }
             }
         }
     }

package/dist/packages/core/src/application/use-cases/repositories/add-repository.use-case.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"add-repository.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/repositories/add-repository.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qCAAqC,CAAC;AACtE,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,oEAAoE,CAAC;AAEhH,MAAM,WAAW,kBAAkB;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAcD,qBACa,oBAAoB;IAG7B,OAAO,CAAC,QAAQ,CAAC,cAAc;gBAAd,cAAc,EAAE,qBAAqB;IAGlD,OAAO,CAAC,KAAK,EAAE,kBAAkB,GAAG,OAAO,CAAC,UAAU,CAAC;CA8B9D"}
+{"version":3,"file":"add-repository.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/repositories/add-repository.use-case.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAIH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qCAAqC,CAAC;AACtE,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,oEAAoE,CAAC;AAEhH,MAAM,WAAW,kBAAkB;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAcD,qBACa,oBAAoB;IAG7B,OAAO,CAAC,QAAQ,CAAC,cAAc;gBAAd,cAAc,EAAE,qBAAqB;IAGlD,OAAO,CAAC,KAAK,EAAE,kBAAkB,GAAG,OAAO,CAAC,UAAU,CAAC;CAiC9D"}

package/dist/packages/core/src/application/use-cases/repositories/add-repository.use-case.js

@@ -41,11 +41,14 @@ let AddRepositoryUseCase = class AddRepositoryUseCase {
         if (existing) {
             return existing;
         }
-        // Check for soft-deleted repository — restore it instead of creating a duplicate
+        // Check for soft-deleted repository — restore it instead of creating a duplicate.
+        // Reset createdAt so the restored repo sorts as "newest" on the canvas
+        // (otherwise the old createdAt would place it above repos added later).
         const deleted = await this.repositoryRepo.findByPathIncludingDeleted(normalizedPath);
         if (deleted) {
+            const now = new Date();
             await this.repositoryRepo.restore(deleted.id);
-            return { ...deleted, deletedAt: undefined, updatedAt: new Date() };
+            return { ...deleted, deletedAt: undefined, createdAt: now, updatedAt: now };
         }
         const now = new Date();
         const name = input.name ?? normalizedPath.split('/').pop() ?? normalizedPath;

package/dist/packages/core/src/infrastructure/repositories/sqlite-repository.repository.js

@@ -94,8 +94,8 @@ let SQLiteRepositoryRepository = SQLiteRepositoryRepository_1 = class SQLiteRepo
     }
     async restore(id) {
         const now = Date.now();
-        const stmt = this.db.prepare('UPDATE repositories SET deleted_at = NULL, updated_at = ? WHERE id = ?');
-        stmt.run(now, id);
+        const stmt = this.db.prepare('UPDATE repositories SET deleted_at = NULL, created_at = ?, updated_at = ? WHERE id = ?');
+        stmt.run(now, now, id);
     }
     static normalizeRemoteUrl(url) {
         return url.replace(/\.git$/, '').toLowerCase();

package/dist/src/presentation/cli/commands/feat/del.command.d.ts

@@ -4,12 +4,13 @@
  * Deletes a feature, its worktree, and cancels any running agent.
  * Optionally cleans up worktree and branches (local + remote).
  *
- * Usage: shep feat del <id> [--force] [--no-cleanup]
+ * Usage: shep feat del <id> [--force] [--no-cleanup] [--no-close-pr]
  *
  * @example
  * $ shep feat del feat-123
  * $ shep feat del feat-123 --force
  * $ shep feat del feat-123 --force --no-cleanup
+ * $ shep feat del feat-123 --force --no-close-pr
  */
 import { Command } from 'commander';
 /**

package/dist/src/presentation/cli/commands/feat/del.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"del.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/del.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAYpC;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,OAAO,CAmD1C"}
+{"version":3,"file":"del.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/del.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAapC;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,OAAO,CA2E1C"}

package/dist/src/presentation/cli/commands/feat/del.command.js

@@ -4,12 +4,13 @@
  * Deletes a feature, its worktree, and cancels any running agent.
  * Optionally cleans up worktree and branches (local + remote).
  *
- * Usage: shep feat del <id> [--force] [--no-cleanup]
+ * Usage: shep feat del <id> [--force] [--no-cleanup] [--no-close-pr]
  *
  * @example
  * $ shep feat del feat-123
  * $ shep feat del feat-123 --force
  * $ shep feat del feat-123 --force --no-cleanup
+ * $ shep feat del feat-123 --force --no-close-pr
  */
 import { Command } from 'commander';
 import { container } from '../../../../../packages/core/src/infrastructure/di/container.js';
@@ -26,6 +27,7 @@ export function createDelCommand() {
         .argument('<id>', 'Feature ID or prefix')
         .option('-f, --force', 'Skip confirmation prompt')
         .option('--no-cleanup', 'Skip worktree and branch cleanup')
+        .option('--no-close-pr', 'Keep the pull request open')
         .action(async (featureId, options) => {
         try {
             // First show what we're about to delete
@@ -52,8 +54,29 @@ export function createDelCommand() {
                     default: true,
                 });
             }
+            // Determine close-PR preference
+            const hasOpenPr = feature.pr?.status === 'Open';
+            let closePr;
+            if (options.closePr === false) {
+                closePr = false;
+            }
+            else if (hasOpenPr && cleanup) {
+                if (options.force) {
+                    closePr = true;
+                }
+                else {
+                    closePr = await confirm({
+                        message: `Also close the pull request? (PR #${feature.pr.number})`,
+                        default: true,
+                    });
+                }
+            }
             const deleteUseCase = container.resolve(DeleteFeatureUseCase);
-            await deleteUseCase.execute(feature.id, { cleanup });
+            const executeOptions = { cleanup };
+            if (closePr !== undefined) {
+                executeOptions.closePr = closePr;
+            }
+            await deleteUseCase.execute(feature.id, executeOptions);
             messages.newline();
             messages.success('Feature deleted');
             console.log(`  ${colors.muted('Name:')}   ${feature.name}`);
@@ -61,6 +84,9 @@ export function createDelCommand() {
             if (cleanup) {
                 console.log(`  ${colors.muted('Cleanup:')} worktree, local branch, remote branch`);
             }
+            if (closePr && hasOpenPr) {
+                console.log(`  ${colors.muted('PR:')}     #${feature.pr.number} closed`);
+            }
             messages.newline();
         }
         catch (error) {

package/dist/src/presentation/web/app/(dashboard)/get-graph-data.d.ts

@@ -6,6 +6,14 @@ export interface RepoGitInfo {
     committer: string;
     behindCount: number | null;
 }
+export type GitInfoResult = {
+    status: 'loading';
+} | {
+    status: 'ready';
+    data: RepoGitInfo;
+} | {
+    status: 'not-a-repo';
+};
 export declare function getGraphData(): Promise<{
     nodes: CanvasNodeType[];
     edges: Edge[];

package/dist/src/presentation/web/app/(dashboard)/get-graph-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-graph-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/(dashboard)/get-graph-data.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAE5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAI1C,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAqDD,wBAAsB,YAAY,IAAI,OAAO,CAAC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAAC,CAyDxF"}
+{"version":3,"file":"get-graph-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/(dashboard)/get-graph-data.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAE5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAI1C,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,CAAC;IACtB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAwFD,MAAM,MAAM,aAAa,GACrB;IAAE,MAAM,EAAE,SAAS,CAAA;CAAE,GACrB;IAAE,MAAM,EAAE,OAAO,CAAC;IAAC,IAAI,EAAE,WAAW,CAAA;CAAE,GACtC;IAAE,MAAM,EAAE,YAAY,CAAA;CAAE,CAAC;AA6B7B,wBAAsB,YAAY,IAAI,OAAO,CAAC;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAAC,CA6DxF"}

package/dist/src/presentation/web/app/(dashboard)/get-graph-data.js

@@ -6,6 +6,11 @@ import { getSettings } from '../../../../../packages/core/src/infrastructure/ser
 import { layoutWithDagre, CANVAS_LAYOUT_DEFAULTS } from '../../lib/layout-with-dagre.js';
 import { buildGraphNodes } from '../build-graph-nodes.js';
 const execFileAsync = promisify(execFileCb);
+// Cache git info per repo — refreshed every GIT_INFO_TTL_MS via background-style check.
+// getGraphData() always reads from cache (zero git calls), and only spawns git
+// processes when the cache entry is older than the TTL.
+const GIT_INFO_TTL_MS = 30_000;
+const gitInfoCache = new Map();
 async function gitCommand(cwd, args) {
     try {
         const opts = IS_WINDOWS ? { cwd, windowsHide: true } : { cwd };
@@ -16,22 +21,38 @@ async function gitCommand(cwd, args) {
         return null;
     }
 }
-async function fetchRepoGitInfo(repo) {
+const FIELD_SEP = '\x1f'; // ASCII Unit Separator — safe in git output and child_process
+/** Refresh git info for a single repo (spawns git processes). */
+async function refreshRepoGitInfo(repo) {
     try {
-        const [currentBranch, commitMessage, committer] = await Promise.all([
+        const headSha = await gitCommand(repo.path, ['rev-parse', 'HEAD']);
+        if (!headSha) {
+            // Not a git repo (or bare/broken) — cache this so the UI can show it
+            gitInfoCache.set(repo.path, { kind: 'not-a-repo', checkedAt: Date.now() });
+            return;
+        }
+        const cached = gitInfoCache.get(repo.path);
+        if (cached?.kind === 'repo' && cached.sha === headSha) {
+            // HEAD unchanged — just bump checkedAt
+            cached.checkedAt = Date.now();
+            return;
+        }
+        // HEAD changed — fetch branch + subject & author in parallel
+        const [currentBranch, logLine] = await Promise.all([
             gitCommand(repo.path, ['symbolic-ref', '--short', 'HEAD']),
-            gitCommand(repo.path, ['log', '-1', '--format=%s']),
-            gitCommand(repo.path, ['log', '-1', '--format=%an']),
+            gitCommand(repo.path, ['log', '-1', `--format=%s${FIELD_SEP}%an`]),
         ]);
         if (!currentBranch)
-            return null;
-        // Try to determine the default branch to compute behind count
+            return;
+        const [commitMessage, committer] = (logLine ?? '').split(FIELD_SEP);
+        // Behind count
         let behindCount = null;
-        const defaultBranch = await gitCommand(repo.path, [
+        const defaultBranchRef = await gitCommand(repo.path, [
             'symbolic-ref',
             '--short',
             'refs/remotes/origin/HEAD',
-        ]).then((ref) => ref?.replace('origin/', '') ?? null);
+        ]);
+        const defaultBranch = defaultBranchRef?.replace('origin/', '') ?? null;
         if (defaultBranch && currentBranch !== defaultBranch) {
             const behind = await gitCommand(repo.path, [
                 'rev-list',
@@ -42,32 +63,63 @@ async function fetchRepoGitInfo(repo) {
             if (isNaN(behindCount))
                 behindCount = null;
         }
-        else if (!defaultBranch || currentBranch === defaultBranch) {
+        else {
             behindCount = 0;
         }
-        return {
-            branch: currentBranch,
-            commitMessage: commitMessage ?? 'unknown',
-            committer: committer ?? 'unknown',
-            behindCount,
-        };
+        gitInfoCache.set(repo.path, {
+            kind: 'repo',
+            sha: headSha,
+            checkedAt: Date.now(),
+            data: {
+                branch: currentBranch,
+                commitMessage: commitMessage ?? 'unknown',
+                committer: committer ?? 'unknown',
+                behindCount,
+            },
+        });
     }
     catch {
-        return null;
+        // ignore — stale cache is better than no data
     }
 }
+/**
+ * Return cached git info for a repo. If the cache is stale (> TTL),
+ * kick off a fire-and-forget refresh so the NEXT call gets fresh data.
+ * Never blocks getGraphData() with git subprocess calls.
+ */
+function getRepoGitInfo(repo) {
+    const cached = gitInfoCache.get(repo.path);
+    const now = Date.now();
+    if (!cached) {
+        // First time — trigger async refresh, return loading for this call
+        void refreshRepoGitInfo(repo);
+        return { status: 'loading' };
+    }
+    if (now - cached.checkedAt >= GIT_INFO_TTL_MS) {
+        // Stale — trigger async refresh, return current data immediately
+        void refreshRepoGitInfo(repo);
+    }
+    if (cached.kind === 'not-a-repo') {
+        return { status: 'not-a-repo' };
+    }
+    return { status: 'ready', data: cached.data };
+}
 export async function getGraphData() {
     const listFeatures = resolve('ListFeaturesUseCase');
     const listRepos = resolve('ListRepositoriesUseCase');
     const agentRunRepo = resolve('IAgentRunRepository');
     const [features, repositories] = await Promise.all([listFeatures.execute(), listRepos.execute()]);
-    // Fetch git info (branch, commit, behind count) for each repository
+    // Read git info from cache (zero git calls). Stale entries trigger
+    // a fire-and-forget background refresh for the next poll cycle.
     const repoGitInfoMap = new Map();
-    await Promise.all(repositories.map(async (repo) => {
-        const info = await fetchRepoGitInfo(repo);
-        if (info)
-            repoGitInfoMap.set(repo.path, info);
-    }));
+    const repoGitStatusMap = new Map();
+    for (const repo of repositories) {
+        const result = getRepoGitInfo(repo);
+        repoGitStatusMap.set(repo.path, result.status);
+        if (result.status === 'ready') {
+            repoGitInfoMap.set(repo.path, result.data);
+        }
+    }
     // PR/CI status is kept fresh by PrSyncWatcher (30s background poll).
     // No live GitHub calls here — use cached DB values for fast response.
     const featuresWithRuns = await Promise.all(features.map(async (feature) => {
@@ -80,6 +132,7 @@ export async function getGraphData() {
         commitEvidence: workflow.commitEvidence,
         ciWatchEnabled: workflow.ciWatchEnabled,
         repoGitInfo: repoGitInfoMap,
+        repoGitStatus: repoGitStatusMap,
     });
     // Enrich feature nodes with deployment status
     let deploymentService = null;

package/dist/src/presentation/web/app/actions/delete-feature.d.ts

@@ -1,5 +1,5 @@
 import type { Feature } from '../../../../../packages/core/src/domain/generated/output.js';
-export declare function deleteFeature(featureId: string, cleanup?: boolean, cascadeDelete?: boolean): Promise<{
+export declare function deleteFeature(featureId: string, cleanup?: boolean, cascadeDelete?: boolean, closePr?: boolean): Promise<{
     feature?: Feature;
     error?: string;
 }>;

package/dist/src/presentation/web/app/actions/delete-feature.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/delete-feature.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,sCAAsC,CAAC;AAEpE,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,EACjB,OAAO,CAAC,EAAE,OAAO,EACjB,aAAa,CAAC,EAAE,OAAO,GACtB,OAAO,CAAC;IAAE,OAAO,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAmBhD"}
+{"version":3,"file":"delete-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/delete-feature.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,sCAAsC,CAAC;AAEpE,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,EACjB,OAAO,CAAC,EAAE,OAAO,EACjB,aAAa,CAAC,EAAE,OAAO,EACvB,OAAO,CAAC,EAAE,OAAO,GAChB,OAAO,CAAC;IAAE,OAAO,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAoBhD"}

package/dist/src/presentation/web/app/actions/delete-feature.js

@@ -1,6 +1,6 @@
 'use server';
 import { resolve } from '../../lib/server-container.js';
-export async function deleteFeature(featureId, cleanup, cascadeDelete) {
+export async function deleteFeature(featureId, cleanup, cascadeDelete, closePr) {
     if (!featureId?.trim()) {
         return { error: 'id is required' };
     }
@@ -11,6 +11,8 @@ export async function deleteFeature(featureId, cleanup, cascadeDelete) {
             options.cleanup = cleanup;
         if (cascadeDelete !== undefined)
             options.cascadeDelete = cascadeDelete;
+        if (closePr !== undefined)
+            options.closePr = closePr;
         const feature = Object.keys(options).length > 0
             ? await deleteFeatureUseCase.execute(featureId, options)
             : await deleteFeatureUseCase.execute(featureId);

package/dist/src/presentation/web/app/build-graph-nodes.d.ts

@@ -27,6 +27,8 @@ export interface BuildGraphNodesOptions {
         committer: string;
         behindCount: number | null;
     }>;
+    /** Git info resolution status keyed by repository path */
+    repoGitStatus?: Map<string, 'loading' | 'ready' | 'not-a-repo'>;
 }
 export declare function buildGraphNodes(repositories: Repository[], featuresWithRuns: FeatureWithRun[], options?: BuildGraphNodesOptions): {
     nodes: CanvasNodeType[];

package/dist/src/presentation/web/app/build-graph-nodes.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"build-graph-nodes.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-graph-nodes.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAS1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAU1C,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,GAAG,EAAE,QAAQ,GAAG,IAAI,CAAC;CACtB;AAED;;;;;;;GAOG;AACH,MAAM,WAAW,sBAAsB;IACrC,uEAAuE;IACvE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,6EAA6E;IAC7E,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,qEAAqE;IACrE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,0FAA0F;IAC1F,WAAW,CAAC,EAAE,GAAG,CACf,MAAM,EACN;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,aAAa,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CACzF,CAAC;CACH;AAED,wBAAgB,eAAe,CAC7B,YAAY,EAAE,UAAU,EAAE,EAC1B,gBAAgB,EAAE,cAAc,EAAE,EAClC,OAAO,CAAC,EAAE,sBAAsB,GAC/B;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAkG5C"}
+{"version":3,"file":"build-graph-nodes.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/app/build-graph-nodes.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,sCAAsC,CAAC;AAS1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,eAAe,CAAC;AAU1C,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,OAAO,CAAC;IACjB,GAAG,EAAE,QAAQ,GAAG,IAAI,CAAC;CACtB;AAED;;;;;;;GAOG;AACH,MAAM,WAAW,sBAAsB;IACrC,uEAAuE;IACvE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,6EAA6E;IAC7E,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,qEAAqE;IACrE,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,0FAA0F;IAC1F,WAAW,CAAC,EAAE,GAAG,CACf,MAAM,EACN;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,aAAa,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CACzF,CAAC;IACF,0DAA0D;IAC1D,aAAa,CAAC,EAAE,GAAG,CAAC,MAAM,EAAE,SAAS,GAAG,OAAO,GAAG,YAAY,CAAC,CAAC;CACjE;AAED,wBAAgB,eAAe,CAC7B,YAAY,EAAE,UAAU,EAAE,EAC1B,gBAAgB,EAAE,cAAc,EAAE,EAClC,OAAO,CAAC,EAAE,sBAAsB,GAC/B;IAAE,KAAK,EAAE,cAAc,EAAE,CAAC;IAAC,KAAK,EAAE,IAAI,EAAE,CAAA;CAAE,CAoG5C"}

package/dist/src/presentation/web/app/build-graph-nodes.js

@@ -32,6 +32,7 @@ export function buildGraphNodes(repositories, featuresWithRuns, options) {
         coveredPaths.add(normalizedRepoPath);
         const repoNodeId = `repo-${repo.id}`;
         const gitInfo = options?.repoGitInfo?.get(repo.path);
+        const gitInfoStatus = options?.repoGitStatus?.get(repo.path) ?? 'loading';
         nodes.push({
             id: repoNodeId,
             type: 'repositoryNode',
@@ -41,6 +42,7 @@ export function buildGraphNodes(repositories, featuresWithRuns, options) {
                 repositoryPath: normalizedRepoPath,
                 id: repo.id,
                 createdAt: repo.createdAt instanceof Date ? repo.createdAt.getTime() : Number(repo.createdAt),
+                gitInfoStatus,
                 ...(gitInfo && {
                     branch: gitInfo.branch,
                     commitMessage: gitInfo.commitMessage,

package/dist/src/presentation/web/components/common/control-center-drawer/feature-drawer-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AAuCA,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAI/D,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;IACjB,8FAA8F;IAC9F,MAAM,CAAC,EAAE,aAAa,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE,wBAAwB,2CAqkB1F"}
+{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AAuCA,OAAO,KAAK,EAAE,UAAU,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAI/D,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;IACjB,8FAA8F;IAC9F,MAAM,CAAC,EAAE,aAAa,CAAC;CACxB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE,wBAAwB,2CAskB1F"}

package/dist/src/presentation/web/components/common/control-center-drawer/feature-drawer-client.js

@@ -256,7 +256,7 @@ export function FeatureDrawerClient({ view: initialView, urlTab }) {
     }, [view, featureNode, prdData, prdSelections, onClose]);
     const handleTechApprove = useCallback(() => handleSimpleApprove('Plan'), [handleSimpleApprove]);
     const handleMergeApprove = useCallback(() => handleSimpleApprove('Merge'), [handleSimpleApprove]);
-    const handleDelete = useCallback(async (featureId, cleanup, cascadeDelete) => {
+    const handleDelete = useCallback(async (featureId, cleanup, cascadeDelete, closePr) => {
         setIsDeleting(true);
         // Close the delete dialog and drawer before the server action so the
         // user sees immediate feedback. We dispatch a DOM event so the canvas
@@ -264,7 +264,7 @@ export function FeatureDrawerClient({ view: initialView, urlTab }) {
         // mutation guard, node removal) in parallel.
         setDeleteDialogOpen(false);
         window.dispatchEvent(new CustomEvent('shep:feature-delete-requested', {
-            detail: { featureId, cleanup, cascadeDelete },
+            detail: { featureId, cleanup, cascadeDelete, closePr },
         }));
         router.push('/');
     }, [router]);
@@ -336,7 +336,7 @@ export function FeatureDrawerClient({ view: initialView, urlTab }) {
         const repoName = featureNode.repositoryName ??
             featureNode.repositoryPath.split('/').filter(Boolean).at(-1) ??
             '';
-        header = (_jsxs(_Fragment, { children: [_jsxs("div", { "data-testid": "feature-drawer-header", children: [_jsx(DrawerTitle, { children: featureNode.name }), repoName ? (_jsxs("div", { className: "flex items-center gap-1.5 pt-0.5", children: [_jsx(Code2, { className: "text-muted-foreground size-3.5 shrink-0" }), featureNode.remoteUrl ? (_jsxs("a", { href: featureNode.remoteUrl, target: "_blank", rel: "noopener noreferrer", className: "text-muted-foreground hover:text-foreground inline-flex items-center gap-1 text-xs transition-colors", "data-testid": "feature-drawer-repo-link", children: [repoName, _jsx(ExternalLink, { className: "size-3" })] })) : (_jsx("span", { className: "text-muted-foreground text-xs", children: repoName }))] })) : null, _jsx(DrawerDescription, { className: "sr-only", children: featureNode.name })] }), featureActionsInput ? (_jsxs("div", { className: "flex items-center gap-2 pt-2", "data-testid": "feature-drawer-actions", children: [_jsx(OpenActionMenu, { actions: featureActions, repositoryPath: featureActionsInput.repositoryPath, showSpecs: !!featureActionsInput.specPath }), featureFlags.envDeploy && featureDeployTarget ? (_jsxs(_Fragment, { children: [_jsx(TooltipProvider, { children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("span", { children: _jsx(ActionButton, { label: isFeatureDeployActive ? 'Stop Dev Server' : 'Start Dev Server', onClick: isFeatureDeployActive ? deployAction.stop : deployAction.deploy, loading: deployAction.deployLoading || deployAction.stopLoading, error: !!deployAction.deployError, icon: isFeatureDeployActive ? Square : Play, iconOnly: true, variant: "outline", size: "icon-sm" }) }) }), _jsx(TooltipContent, { children: isFeatureDeployActive ? 'Stop Dev Server' : 'Start Dev Server' })] }) }), isFeatureDeployActive ? (_jsx(DeploymentStatusBadge, { status: deployAction.status, url: deployAction.url, targetId: featureDeployTarget?.targetId })) : null] })) : null, _jsxs("div", { className: "ml-auto flex items-center gap-1.5", children: [_jsx("code", { className: "bg-muted text-muted-foreground rounded px-1.5 py-0.5 font-mono text-xs", children: shortId }), _jsx("button", { type: "button", onClick: handleCopyId, className: "text-muted-foreground hover:text-foreground inline-flex items-center rounded p-0.5 transition-colors", "aria-label": "Copy feature ID", "data-testid": "feature-drawer-copy-id", children: idCopied ? (_jsx(Check, { className: "size-3.5 text-green-600" })) : (_jsx(Copy, { className: "size-3.5" })) })] }), featureNode.featureId ? (_jsxs(_Fragment, { children: [_jsx(Button, { variant: "ghost", size: "icon-sm", "aria-label": "Delete feature", disabled: isDeleting, className: "text-muted-foreground hover:text-destructive", "data-testid": "feature-drawer-delete", onClick: () => setDeleteDialogOpen(true), children: isDeleting ? (_jsx(Loader2, { className: "size-4 animate-spin" })) : (_jsx(Trash2, { className: "size-4" })) }), _jsx(DeleteFeatureDialog, { open: deleteDialogOpen, onOpenChange: setDeleteDialogOpen, onConfirm: (cleanup, cascadeDelete) => handleDelete(featureNode.featureId, cleanup, cascadeDelete), isDeleting: isDeleting, featureName: featureNode.name, featureId: featureNode.featureId, hasChildren: featureNode.hasChildren })] })) : null] })) : null] }));
+        header = (_jsxs(_Fragment, { children: [_jsxs("div", { "data-testid": "feature-drawer-header", children: [_jsx(DrawerTitle, { children: featureNode.name }), repoName ? (_jsxs("div", { className: "flex items-center gap-1.5 pt-0.5", children: [_jsx(Code2, { className: "text-muted-foreground size-3.5 shrink-0" }), featureNode.remoteUrl ? (_jsxs("a", { href: featureNode.remoteUrl, target: "_blank", rel: "noopener noreferrer", className: "text-muted-foreground hover:text-foreground inline-flex items-center gap-1 text-xs transition-colors", "data-testid": "feature-drawer-repo-link", children: [repoName, _jsx(ExternalLink, { className: "size-3" })] })) : (_jsx("span", { className: "text-muted-foreground text-xs", children: repoName }))] })) : null, _jsx(DrawerDescription, { className: "sr-only", children: featureNode.name })] }), featureActionsInput ? (_jsxs("div", { className: "flex items-center gap-2 pt-2", "data-testid": "feature-drawer-actions", children: [_jsx(OpenActionMenu, { actions: featureActions, repositoryPath: featureActionsInput.repositoryPath, showSpecs: !!featureActionsInput.specPath }), featureFlags.envDeploy && featureDeployTarget ? (_jsxs(_Fragment, { children: [_jsx(TooltipProvider, { children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("span", { children: _jsx(ActionButton, { label: isFeatureDeployActive ? 'Stop Dev Server' : 'Start Dev Server', onClick: isFeatureDeployActive ? deployAction.stop : deployAction.deploy, loading: deployAction.deployLoading || deployAction.stopLoading, error: !!deployAction.deployError, icon: isFeatureDeployActive ? Square : Play, iconOnly: true, variant: "outline", size: "icon-sm" }) }) }), _jsx(TooltipContent, { children: isFeatureDeployActive ? 'Stop Dev Server' : 'Start Dev Server' })] }) }), isFeatureDeployActive ? (_jsx(DeploymentStatusBadge, { status: deployAction.status, url: deployAction.url, targetId: featureDeployTarget?.targetId })) : null] })) : null, _jsxs("div", { className: "ml-auto flex items-center gap-1.5", children: [_jsx("code", { className: "bg-muted text-muted-foreground rounded px-1.5 py-0.5 font-mono text-xs", children: shortId }), _jsx("button", { type: "button", onClick: handleCopyId, className: "text-muted-foreground hover:text-foreground inline-flex items-center rounded p-0.5 transition-colors", "aria-label": "Copy feature ID", "data-testid": "feature-drawer-copy-id", children: idCopied ? (_jsx(Check, { className: "size-3.5 text-green-600" })) : (_jsx(Copy, { className: "size-3.5" })) })] }), featureNode.featureId ? (_jsxs(_Fragment, { children: [_jsx(Button, { variant: "ghost", size: "icon-sm", "aria-label": "Delete feature", disabled: isDeleting, className: "text-muted-foreground hover:text-destructive", "data-testid": "feature-drawer-delete", onClick: () => setDeleteDialogOpen(true), children: isDeleting ? (_jsx(Loader2, { className: "size-4 animate-spin" })) : (_jsx(Trash2, { className: "size-4" })) }), _jsx(DeleteFeatureDialog, { open: deleteDialogOpen, onOpenChange: setDeleteDialogOpen, onConfirm: (cleanup, cascadeDelete, closePr) => handleDelete(featureNode.featureId, cleanup, cascadeDelete, closePr), isDeleting: isDeleting, featureName: featureNode.name, featureId: featureNode.featureId, hasChildren: featureNode.hasChildren, hasOpenPr: !!featureNode.pr && featureNode.pr.status === 'Open' })] })) : null] })) : null] }));
     }
     // ── Body ──────────────────────────────────────────────────────────────
     let body = null;

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog-config.d.ts

@@ -1,10 +1,11 @@
 export interface DeleteFeatureDialogProps {
     open: boolean;
     onOpenChange: (open: boolean) => void;
-    onConfirm: (cleanup: boolean, cascadeDelete: boolean) => void;
+    onConfirm: (cleanup: boolean, cascadeDelete: boolean, closePr: boolean) => void;
     isDeleting: boolean;
     featureName: string;
     featureId: string;
     hasChildren?: boolean;
+    hasOpenPr?: boolean;
 }
 //# sourceMappingURL=delete-feature-dialog-config.d.ts.map

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-feature-dialog-config.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog-config.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,OAAO,CAAC;IACd,YAAY,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAC;IACtC,SAAS,EAAE,CAAC,OAAO,EAAE,OAAO,EAAE,aAAa,EAAE,OAAO,KAAK,IAAI,CAAC;IAC9D,UAAU,EAAE,OAAO,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB"}
+{"version":3,"file":"delete-feature-dialog-config.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog-config.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,OAAO,CAAC;IACd,YAAY,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAC;IACtC,SAAS,EAAE,CAAC,OAAO,EAAE,OAAO,EAAE,aAAa,EAAE,OAAO,EAAE,OAAO,EAAE,OAAO,KAAK,IAAI,CAAC;IAChF,UAAU,EAAE,OAAO,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB"}

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.d.ts

@@ -1,3 +1,3 @@
 import type { DeleteFeatureDialogProps } from './delete-feature-dialog-config.js';
-export declare function DeleteFeatureDialog({ open, onOpenChange, onConfirm, isDeleting, featureName, featureId, hasChildren, }: DeleteFeatureDialogProps): import("react/jsx-runtime").JSX.Element;
+export declare function DeleteFeatureDialog({ open, onOpenChange, onConfirm, isDeleting, featureName, featureId, hasChildren, hasOpenPr, }: DeleteFeatureDialogProps): import("react/jsx-runtime").JSX.Element;
 //# sourceMappingURL=delete-feature-dialog.d.ts.map

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-feature-dialog.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.tsx"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,gCAAgC,CAAC;AAE/E,wBAAgB,mBAAmB,CAAC,EAClC,IAAI,EACJ,YAAY,EACZ,SAAS,EACT,UAAU,EACV,WAAW,EACX,SAAS,EACT,WAAW,GACZ,EAAE,wBAAwB,2CAuE1B"}
+{"version":3,"file":"delete-feature-dialog.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.tsx"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,gCAAgC,CAAC;AAE/E,wBAAgB,mBAAmB,CAAC,EAClC,IAAI,EACJ,YAAY,EACZ,SAAS,EACT,UAAU,EACV,WAAW,EACX,SAAS,EACT,WAAW,EACX,SAAS,GACV,EAAE,wBAAwB,2CA+F1B"}

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.js

@@ -5,14 +5,24 @@ import { Loader2 } from 'lucide-react';
 import { AlertDialog, AlertDialogAction, AlertDialogCancel, AlertDialogContent, AlertDialogDescription, AlertDialogFooter, AlertDialogHeader, AlertDialogTitle, } from '../../ui/alert-dialog.js';
 import { Checkbox } from '../../ui/checkbox.js';
 import { Label } from '../../ui/label.js';
-export function DeleteFeatureDialog({ open, onOpenChange, onConfirm, isDeleting, featureName, featureId, hasChildren, }) {
+export function DeleteFeatureDialog({ open, onOpenChange, onConfirm, isDeleting, featureName, featureId, hasChildren, hasOpenPr, }) {
     const [cleanup, setCleanup] = useState(true);
     const [cascadeDelete, setCascadeDelete] = useState(false);
+    const [closePr, setClosePr] = useState(true);
     useEffect(() => {
         if (open) {
             setCleanup(true);
             setCascadeDelete(false);
+            setClosePr(true);
         }
     }, [open]);
-    return (_jsx(AlertDialog, { open: open, onOpenChange: onOpenChange, children: _jsxs(AlertDialogContent, { onCloseAutoFocus: (e) => e.preventDefault(), children: [_jsxs(AlertDialogHeader, { children: [_jsx(AlertDialogTitle, { children: "Delete feature?" }), _jsxs(AlertDialogDescription, { children: ["This will permanently delete ", _jsx("strong", { children: featureName }), " (", featureId, "). This action cannot be undone."] })] }), _jsxs("div", { className: "flex flex-col gap-2", children: [_jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Checkbox, { id: "cleanup-checkbox", checked: cleanup, onCheckedChange: (checked) => setCleanup(checked === true), disabled: isDeleting, "aria-label": "Clean up worktree and branches" }), _jsx(Label, { htmlFor: "cleanup-checkbox", className: "cursor-pointer text-sm", children: "Clean up worktree and branches" })] }), hasChildren ? (_jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Checkbox, { id: "cascade-delete-checkbox", checked: cascadeDelete, onCheckedChange: (checked) => setCascadeDelete(checked === true), disabled: isDeleting, "aria-label": "Delete sub-features" }), _jsx(Label, { htmlFor: "cascade-delete-checkbox", className: "cursor-pointer text-sm", children: "Delete sub-features" })] })) : null] }), _jsxs(AlertDialogFooter, { children: [_jsx(AlertDialogCancel, { disabled: isDeleting, onClick: () => onOpenChange(false), children: "Cancel" }), _jsx(AlertDialogAction, { variant: "destructive", disabled: isDeleting, onClick: () => onConfirm(cleanup, cascadeDelete), children: isDeleting ? (_jsxs(_Fragment, { children: [_jsx(Loader2, { className: "mr-2 h-4 w-4 animate-spin" }), "Deleting\u2026"] })) : ('Delete') })] })] }) }));
+    useEffect(() => {
+        if (cleanup) {
+            setClosePr(true);
+        }
+        else {
+            setClosePr(false);
+        }
+    }, [cleanup]);
+    return (_jsx(AlertDialog, { open: open, onOpenChange: onOpenChange, children: _jsxs(AlertDialogContent, { onCloseAutoFocus: (e) => e.preventDefault(), children: [_jsxs(AlertDialogHeader, { children: [_jsx(AlertDialogTitle, { children: "Delete feature?" }), _jsxs(AlertDialogDescription, { children: ["This will permanently delete ", _jsx("strong", { children: featureName }), " (", featureId, "). This action cannot be undone."] })] }), _jsxs("div", { className: "flex flex-col gap-2", children: [_jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Checkbox, { id: "cleanup-checkbox", checked: cleanup, onCheckedChange: (checked) => setCleanup(checked === true), disabled: isDeleting, "aria-label": "Clean up worktree and branches" }), _jsx(Label, { htmlFor: "cleanup-checkbox", className: "cursor-pointer text-sm", children: "Clean up worktree and branches" })] }), hasChildren ? (_jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Checkbox, { id: "cascade-delete-checkbox", checked: cascadeDelete, onCheckedChange: (checked) => setCascadeDelete(checked === true), disabled: isDeleting, "aria-label": "Delete sub-features" }), _jsx(Label, { htmlFor: "cascade-delete-checkbox", className: "cursor-pointer text-sm", children: "Delete sub-features" })] })) : null, hasOpenPr ? (_jsxs("div", { className: "flex items-center gap-2", children: [_jsx(Checkbox, { id: "close-pr-checkbox", checked: closePr, onCheckedChange: (checked) => setClosePr(checked === true), disabled: isDeleting || !cleanup, "aria-label": "Close pull request" }), _jsx(Label, { htmlFor: "close-pr-checkbox", className: "cursor-pointer text-sm", children: "Close pull request" })] })) : null] }), _jsxs(AlertDialogFooter, { children: [_jsx(AlertDialogCancel, { disabled: isDeleting, onClick: () => onOpenChange(false), children: "Cancel" }), _jsx(AlertDialogAction, { variant: "destructive", disabled: isDeleting, onClick: () => onConfirm(cleanup, cascadeDelete, hasOpenPr ? closePr : false), children: isDeleting ? (_jsxs(_Fragment, { children: [_jsx(Loader2, { className: "mr-2 h-4 w-4 animate-spin" }), "Deleting\u2026"] })) : ('Delete') })] })] }) }));
 }

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.stories.d.ts

@@ -7,4 +7,7 @@ export declare const Default: Story;
 export declare const WithChildren: Story;
 export declare const Deleting: Story;
 export declare const DeletingWithChildren: Story;
+export declare const WithOpenPr: Story;
+export declare const WithOpenPrAndChildren: Story;
+export declare const DeletingWithOpenPr: Story;
 //# sourceMappingURL=delete-feature-dialog.stories.d.ts.map

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"delete-feature-dialog.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEvD,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AAE9D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,mBAAmB,CAe1C,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,mBAAmB,CAAC,CAAC;AAElD,eAAO,MAAM,OAAO,EAAE,KAAU,CAAC;AAEjC,eAAO,MAAM,YAAY,EAAE,KAI1B,CAAC;AAEF,eAAO,MAAM,QAAQ,EAAE,KAItB,CAAC;AAEF,eAAO,MAAM,oBAAoB,EAAE,KAKlC,CAAC"}
+{"version":3,"file":"delete-feature-dialog.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEvD,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AAE9D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,mBAAmB,CAe1C,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,mBAAmB,CAAC,CAAC;AAElD,eAAO,MAAM,OAAO,EAAE,KAAU,CAAC;AAEjC,eAAO,MAAM,YAAY,EAAE,KAI1B,CAAC;AAEF,eAAO,MAAM,QAAQ,EAAE,KAItB,CAAC;AAEF,eAAO,MAAM,oBAAoB,EAAE,KAKlC,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,KAIxB,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,KAKnC,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,KAKhC,CAAC"}

package/dist/src/presentation/web/components/common/delete-feature-dialog/delete-feature-dialog.stories.js

@@ -34,3 +34,20 @@ export const DeletingWithChildren = {
         hasChildren: true,
     },
 };
+export const WithOpenPr = {
+    args: {
+        hasOpenPr: true,
+    },
+};
+export const WithOpenPrAndChildren = {
+    args: {
+        hasOpenPr: true,
+        hasChildren: true,
+    },
+};
+export const DeletingWithOpenPr = {
+    args: {
+        isDeleting: true,
+        hasOpenPr: true,
+    },
+};

package/dist/src/presentation/web/components/common/feature-node/feature-node-state-config.d.ts

@@ -89,7 +89,7 @@ export interface FeatureNodeData {
     onAction?: () => void;
     onSettings?: () => void;
     hasChildren?: boolean;
-    onDelete?: (featureId: string, cleanup?: boolean, cascadeDelete?: boolean) => void;
+    onDelete?: (featureId: string, cleanup?: boolean, cascadeDelete?: boolean, closePr?: boolean) => void;
     onRetry?: (featureId: string) => void;
     onStart?: (featureId: string) => void;
     onStop?: (featureId: string) => void;