@shepai/cli 1.72.9-pr220.f17b60a → 1.73.0-pr222.ef53005

package/dist/packages/core/src/application/ports/output/agents/agent-executor-provider.interface.d.ts

@@ -1,5 +1,5 @@
 import type { IAgentExecutor } from './agent-executor.interface.js';
 export interface IAgentExecutorProvider {
-    getExecutor(): IAgentExecutor;
+    getExecutor(): Promise<IAgentExecutor>;
 }
 //# sourceMappingURL=agent-executor-provider.interface.d.ts.map

package/dist/packages/core/src/application/ports/output/agents/agent-executor-provider.interface.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-executor-provider.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/agents/agent-executor-provider.interface.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAEpE,MAAM,WAAW,sBAAsB;IACrC,WAAW,IAAI,cAAc,CAAC;CAC/B"}
+{"version":3,"file":"agent-executor-provider.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/agents/agent-executor-provider.interface.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAEpE,MAAM,WAAW,sBAAsB;IACrC,WAAW,IAAI,OAAO,CAAC,cAAc,CAAC,CAAC;CACxC"}

package/dist/packages/core/src/infrastructure/di/container.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/di/container.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,kBAAkB,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AAiHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CAuQrE;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,IAAI,OAAO,CAEhD;AAED;;;GAGG;AACH,OAAO,EAAE,SAAS,EAAE,CAAC"}
+{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/di/container.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,kBAAkB,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AAiHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CAwQrE;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,IAAI,OAAO,CAEhD;AAED;;;GAGG;AACH,OAAO,EAAE,SAAS,EAAE,CAAC"}

package/dist/packages/core/src/infrastructure/di/container.js

@@ -171,7 +171,8 @@ export async function initializeContainer() {
     container.register('IAgentExecutorProvider', {
         useFactory: (c) => {
             const factory = c.resolve('IAgentExecutorFactory');
-            return new AgentExecutorProvider(factory);
+            const settingsRepo = c.resolve('ISettingsRepository');
+            return new AgentExecutorProvider(factory, settingsRepo);
         },
     });
     container.register('IStructuredAgentCaller', {

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-provider.service.d.ts

@@ -1,9 +1,11 @@
 import type { IAgentExecutorProvider } from '../../../../application/ports/output/agents/agent-executor-provider.interface.js';
 import type { IAgentExecutorFactory } from '../../../../application/ports/output/agents/agent-executor-factory.interface.js';
 import type { IAgentExecutor } from '../../../../application/ports/output/agents/agent-executor.interface.js';
+import type { ISettingsRepository } from '../../../../application/ports/output/repositories/settings.repository.interface.js';
 export declare class AgentExecutorProvider implements IAgentExecutorProvider {
     private readonly factory;
-    constructor(factory: IAgentExecutorFactory);
-    getExecutor(): IAgentExecutor;
+    private readonly settingsRepository;
+    constructor(factory: IAgentExecutorFactory, settingsRepository: ISettingsRepository);
+    getExecutor(): Promise<IAgentExecutor>;
 }
 //# sourceMappingURL=agent-executor-provider.service.d.ts.map

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-provider.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-executor-provider.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-executor-provider.service.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,kFAAkF,CAAC;AAC/H,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,iFAAiF,CAAC;AAC7H,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,yEAAyE,CAAC;AAG9G,qBAAa,qBAAsB,YAAW,sBAAsB;IACtD,OAAO,CAAC,QAAQ,CAAC,OAAO;gBAAP,OAAO,EAAE,qBAAqB;IAE3D,WAAW,IAAI,cAAc;CAI9B"}
+{"version":3,"file":"agent-executor-provider.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-executor-provider.service.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,kFAAkF,CAAC;AAC/H,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,iFAAiF,CAAC;AAC7H,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,yEAAyE,CAAC;AAC9G,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oFAAoF,CAAC;AAE9H,qBAAa,qBAAsB,YAAW,sBAAsB;IAEhE,OAAO,CAAC,QAAQ,CAAC,OAAO;IACxB,OAAO,CAAC,QAAQ,CAAC,kBAAkB;gBADlB,OAAO,EAAE,qBAAqB,EAC9B,kBAAkB,EAAE,mBAAmB;IAGpD,WAAW,IAAI,OAAO,CAAC,cAAc,CAAC;CAO7C"}

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-provider.service.js

@@ -1,11 +1,15 @@
-import { getSettings } from '../../settings.service.js';
 export class AgentExecutorProvider {
     factory;
-    constructor(factory) {
+    settingsRepository;
+    constructor(factory, settingsRepository) {
         this.factory = factory;
+        this.settingsRepository = settingsRepository;
     }
-    getExecutor() {
-        const settings = getSettings();
+    async getExecutor() {
+        const settings = await this.settingsRepository.load();
+        if (!settings) {
+            throw new Error('Settings not found. Please run initialization first.');
+        }
         return this.factory.createExecutor(settings.agent.type, settings.agent);
     }
 }

package/dist/packages/core/src/infrastructure/services/agents/common/agent-runner.service.js

@@ -76,7 +76,7 @@ export class AgentRunnerService {
         }
         const settings = getSettings();
         const agentType = settings.agent.type;
-        const executor = this.executorProvider.getExecutor();
+        const executor = await this.executorProvider.getExecutor();
         const runId = crypto.randomUUID();
         const threadId = crypto.randomUUID();
         const now = new Date().toISOString();

package/dist/packages/core/src/infrastructure/services/agents/common/structured-agent-caller.service.js

@@ -25,7 +25,7 @@ let StructuredAgentCallerService = class StructuredAgentCallerService {
         this.executorProvider = executorProvider;
     }
     async call(prompt, schema, options) {
-        const executor = this.executorProvider.getExecutor();
+        const executor = await this.executorProvider.getExecutor();
         if (executor.supportsFeature(AgentFeature.structuredOutput)) {
             return this.callWithNativeSchema(executor, prompt, schema, options);
         }

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/feature-agent-worker.js

@@ -133,7 +133,7 @@ export async function runWorker(args) {
     }
     else {
         log('Creating executor from configured agent settings...');
-        executor = executorProvider.getExecutor();
+        executor = await executorProvider.getExecutor();
     }
     // Resolve merge node dependencies
     const gitPrService = container.resolve('IGitPrService');

package/dist/src/presentation/web/app/layout.js

@@ -36,5 +36,5 @@ export default function RootLayout({ children, }) {
                 } catch (e) {}
               })();
             `,
-                    } }) }), _jsxs("body", { className: "min-h-screen antialiased", children: [_jsx(AppShell, { children: children }), _jsx(Toaster, {})] })] }));
+                    } }) }), _jsxs("body", { className: "min-h-screen antialiased", children: [_jsx(AppShell, { children: children }), _jsx(Toaster, { position: "bottom-center" })] })] }));
 }

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":"AAyDA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAEhD,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;CAClB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,wBAAwB,2CA6mBlF"}
+{"version":3,"file":"feature-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/feature-drawer-client.tsx"],"names":[],"mappings":"AAyDA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAGhD,MAAM,WAAW,wBAAwB;IACvC,IAAI,EAAE,UAAU,CAAC;CAClB;AAED,wBAAgB,mBAAmB,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,EAAE,wBAAwB,2CAykBlF"}

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

@@ -36,6 +36,7 @@ import { featureNodeStateConfig, lifecycleDisplayLabels } from '../../common/fea
 import { useFeatureActions } from '../../common/feature-drawer/use-feature-actions.js';
 import { resolveSseEventUpdates } from '../../common/feature-node/derive-feature-state.js';
 import { deriveFeatureViewType } from './drawer-view.js';
+import { useArtifactFetch } from './use-artifact-fetch.js';
 export function FeatureDrawerClient({ view: initialView }) {
     const router = useRouter();
     const deleteSound = useSoundAction('delete');
@@ -56,6 +57,10 @@ export function FeatureDrawerClient({ view: initialView }) {
     const { events } = useAgentEventsContext();
     const processedCountRef = useRef(0);
     useEffect(() => {
+        // Clamp cursor if events were pruned
+        if (processedCountRef.current > events.length) {
+            processedCountRef.current = 0;
+        }
         if (!featureNode || events.length <= processedCountRef.current)
             return;
         const newEvents = events.slice(processedCountRef.current);
@@ -64,9 +69,9 @@ export function FeatureDrawerClient({ view: initialView }) {
             if (update.featureId !== featureNode.featureId)
                 continue;
             // Trigger a server refresh to get the latest drawer view,
-            // but skip when the user has unsaved changes or a rejection is in-flight
-            // to avoid the refresh interfering with the reject handler's navigation.
-            if (!isDirtyRef.current && !isRejectingRef.current)
+            // but skip when the drawer is closed, the user has unsaved changes,
+            // or a rejection is in-flight to avoid unnecessary refreshes.
+            if (isOpenRef.current && !isDirtyRef.current && !isRejectingRef.current)
                 router.refresh();
             if (update.state !== undefined || update.lifecycle !== undefined) {
                 // Optimistically update the node data AND re-derive the view type so the
@@ -94,6 +99,17 @@ export function FeatureDrawerClient({ view: initialView }) {
     // the close animation when the path no longer matches a feature route.
     const pathname = usePathname();
     const isOpen = pathname.startsWith('/feature/');
+    const isOpenRef = useRef(isOpen);
+    const wasOpenRef = useRef(isOpen);
+    // When the drawer re-opens, force a server refresh so the view is always fresh.
+    // SSE skips router.refresh() while the drawer is closed, so data can go stale.
+    useEffect(() => {
+        if (isOpen && !wasOpenRef.current) {
+            router.refresh();
+        }
+        wasOpenRef.current = isOpen;
+    }, [isOpen, router]);
+    isOpenRef.current = isOpen;
     const onClose = useCallback(() => {
         router.push('/');
     }, [router]);
@@ -103,16 +119,12 @@ export function FeatureDrawerClient({ view: initialView }) {
     const [prdData, setPrdData] = useState(null);
     const [prdSelections, setPrdSelections] = useState({});
     const [prdDefaultSelections, setPrdDefaultSelections] = useState({});
-    const [isLoadingPrd, setIsLoadingPrd] = useState(false);
     // ── Tech state ─────────────────────────────────────────────────────────
     const [techData, setTechData] = useState(null);
-    const [isLoadingTech, setIsLoadingTech] = useState(false);
     // ── Product decisions state (for tech review Product tab) ─────────────
     const [techProductData, setTechProductData] = useState(undefined);
-    const [isLoadingTechProduct, setIsLoadingTechProduct] = useState(false);
     // ── Merge state ────────────────────────────────────────────────────────
     const [mergeData, setMergeData] = useState(null);
-    const [isLoadingMerge, setIsLoadingMerge] = useState(false);
     // ── Delete state ───────────────────────────────────────────────────────
     const [isDeleting, setIsDeleting] = useState(false);
     // ── Shared reject state ────────────────────────────────────────────────
@@ -125,130 +137,48 @@ export function FeatureDrawerClient({ view: initialView }) {
     }, [viewType]);
     // ── Data fetching ─────────────────────────────────────────────────────
     const prdFeatureId = view.type === 'prd-review' ? view.node.featureId : null;
-    useEffect(() => {
+    const isLoadingPrd = useArtifactFetch(prdFeatureId, getFeatureArtifact, (result) => {
+        if (result.error) {
+            toast.error(result.error);
+            return;
+        }
+        if (result.questionnaire) {
+            setPrdData(result.questionnaire);
+            const defaults = {};
+            for (const q of result.questionnaire.questions) {
+                const recommended = q.options.find((o) => o.recommended);
+                if (recommended)
+                    defaults[q.id] = recommended.id;
+            }
+            setPrdSelections(defaults);
+            setPrdDefaultSelections(defaults);
+        }
+    }, () => {
         setPrdSelections({});
         setPrdDefaultSelections({});
         setPrdData(null);
-        if (!prdFeatureId)
-            return;
-        let cancelled = false;
-        setIsLoadingPrd(true);
-        getFeatureArtifact(prdFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.error) {
-                toast.error(result.error);
-                return;
-            }
-            if (result.questionnaire) {
-                setPrdData(result.questionnaire);
-                const defaults = {};
-                for (const q of result.questionnaire.questions) {
-                    const recommended = q.options.find((o) => o.recommended);
-                    if (recommended)
-                        defaults[q.id] = recommended.id;
-                }
-                setPrdSelections(defaults);
-                setPrdDefaultSelections(defaults);
-            }
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load questionnaire');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingPrd(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [prdFeatureId]);
+    }, 'Failed to load questionnaire');
     const techFeatureId = view.type === 'tech-review' ? view.node.featureId : null;
-    useEffect(() => {
-        setTechData(null);
-        if (!techFeatureId)
+    const isLoadingTech = useArtifactFetch(techFeatureId, getResearchArtifact, (result) => {
+        if (result.error) {
+            toast.error(result.error);
             return;
-        let cancelled = false;
-        setIsLoadingTech(true);
-        getResearchArtifact(techFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.error) {
-                toast.error(result.error);
-                return;
-            }
-            if (result.techDecisions)
-                setTechData(result.techDecisions);
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load tech decisions');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingTech(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [techFeatureId]);
-    useEffect(() => {
-        setTechProductData(undefined);
-        if (!techFeatureId)
-            return;
-        let cancelled = false;
-        setIsLoadingTechProduct(true);
-        getFeatureArtifact(techFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if (result.productDecisions) {
-                setTechProductData(result.productDecisions);
-            }
-        })
-            .catch(() => {
-            // Silent failure — the product tab is supplementary
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingTechProduct(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [techFeatureId]);
+        }
+        if (result.techDecisions)
+            setTechData(result.techDecisions);
+    }, () => setTechData(null), 'Failed to load tech decisions');
+    const isLoadingTechProduct = useArtifactFetch(techFeatureId, getFeatureArtifact, (result) => {
+        if (result.productDecisions)
+            setTechProductData(result.productDecisions);
+    }, () => setTechProductData(undefined));
     const mergeFeatureId = view.type === 'merge-review' ? view.node.featureId : null;
-    useEffect(() => {
-        setMergeData(null);
-        if (!mergeFeatureId)
+    const isLoadingMerge = useArtifactFetch(mergeFeatureId, getMergeReviewData, (result) => {
+        if ('error' in result) {
+            toast.error(result.error);
             return;
-        let cancelled = false;
-        setIsLoadingMerge(true);
-        getMergeReviewData(mergeFeatureId)
-            .then((result) => {
-            if (cancelled)
-                return;
-            if ('error' in result) {
-                toast.error(result.error);
-                return;
-            }
-            setMergeData(result);
-        })
-            .catch(() => {
-            if (!cancelled)
-                toast.error('Failed to load merge review data');
-        })
-            .finally(() => {
-            if (!cancelled)
-                setIsLoadingMerge(false);
-        });
-        return () => {
-            cancelled = true;
-        };
-    }, [mergeFeatureId]);
+        }
+        setMergeData(result);
+    }, () => setMergeData(null), 'Failed to load merge review data');
     // ── Close guard ──────────────────────────────────────────────────────
     const isChatDirty = chatInput.trim().length > 0;
     const isPrdDirty = view.type === 'prd-review' &&

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Generic hook for fetching drawer artifact data when a feature ID changes.
+ * Handles loading state, cancellation on unmount/re-fetch, and error toasting.
+ *
+ * @param featureId - The feature ID to fetch for, or null to reset.
+ * @param fetcher - Async function that fetches the artifact data.
+ * @param onSuccess - Called with the fetch result when successful and not cancelled.
+ * @param onReset - Called immediately when featureId changes (before fetching).
+ * @param errorMessage - Optional toast message shown on fetch failure.
+ * @returns Whether the fetch is currently in progress.
+ */
+export declare function useArtifactFetch<TResult>(featureId: string | null, fetcher: (id: string) => Promise<TResult>, onSuccess: (result: TResult) => void, onReset: () => void, errorMessage?: string): boolean;
+//# sourceMappingURL=use-artifact-fetch.d.ts.map

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"use-artifact-fetch.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.ts"],"names":[],"mappings":"AAKA;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EACtC,SAAS,EAAE,MAAM,GAAG,IAAI,EACxB,OAAO,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,CAAC,OAAO,CAAC,EACzC,SAAS,EAAE,CAAC,MAAM,EAAE,OAAO,KAAK,IAAI,EACpC,OAAO,EAAE,MAAM,IAAI,EACnB,YAAY,CAAC,EAAE,MAAM,GACpB,OAAO,CAiCT"}

package/dist/src/presentation/web/components/common/control-center-drawer/use-artifact-fetch.js

@@ -0,0 +1,46 @@
+'use client';
+import { useState, useEffect, useRef } from 'react';
+import { toast } from 'sonner';
+/**
+ * Generic hook for fetching drawer artifact data when a feature ID changes.
+ * Handles loading state, cancellation on unmount/re-fetch, and error toasting.
+ *
+ * @param featureId - The feature ID to fetch for, or null to reset.
+ * @param fetcher - Async function that fetches the artifact data.
+ * @param onSuccess - Called with the fetch result when successful and not cancelled.
+ * @param onReset - Called immediately when featureId changes (before fetching).
+ * @param errorMessage - Optional toast message shown on fetch failure.
+ * @returns Whether the fetch is currently in progress.
+ */
+export function useArtifactFetch(featureId, fetcher, onSuccess, onReset, errorMessage) {
+    const [isLoading, setIsLoading] = useState(false);
+    // Use refs to avoid stale callbacks in the effect without triggering re-runs.
+    const onSuccessRef = useRef(onSuccess);
+    const onResetRef = useRef(onReset);
+    onSuccessRef.current = onSuccess;
+    onResetRef.current = onReset;
+    useEffect(() => {
+        onResetRef.current();
+        if (!featureId)
+            return;
+        let cancelled = false;
+        setIsLoading(true);
+        fetcher(featureId)
+            .then((result) => {
+            if (!cancelled)
+                onSuccessRef.current(result);
+        })
+            .catch(() => {
+            if (!cancelled && errorMessage)
+                toast.error(errorMessage);
+        })
+            .finally(() => {
+            if (!cancelled)
+                setIsLoading(false);
+        });
+        return () => {
+            cancelled = true;
+        };
+    }, [featureId, fetcher, errorMessage]);
+    return isLoading;
+}

package/dist/src/presentation/web/components/common/feature-drawer/use-feature-actions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-feature-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer/use-feature-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,mBAAmB;IAClC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,mBAAmB;IAClC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,eAAe,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAID,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,mBAAmB,GAAG,IAAI,GAAG,mBAAmB,CA+GxF"}
+{"version":3,"file":"use-feature-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-drawer/use-feature-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,mBAAmB;IAClC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,mBAAmB;IAClC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,eAAe,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACrC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,YAAY,EAAE,OAAO,CAAC;IACtB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAID,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,mBAAmB,GAAG,IAAI,GAAG,mBAAmB,CA8GxF"}

package/dist/src/presentation/web/components/common/feature-drawer/use-feature-actions.js

@@ -14,18 +14,15 @@ export function useFeatureActions(input) {
     const ideTimerRef = useRef(null);
     const shellTimerRef = useRef(null);
     const specsTimerRef = useRef(null);
-    // Clear timers on unmount
+    // Clear timers on unmount — read .current inside cleanup so we get the
+    // actual timer value at teardown time, not the always-null value at mount.
     useEffect(() => {
-        const ideTimer = ideTimerRef.current;
-        const shellTimer = shellTimerRef.current;
-        const specsTimer = specsTimerRef.current;
+        const refs = [ideTimerRef, shellTimerRef, specsTimerRef];
         return () => {
-            if (ideTimer)
-                clearTimeout(ideTimer);
-            if (shellTimer)
-                clearTimeout(shellTimer);
-            if (specsTimer)
-                clearTimeout(specsTimer);
+            for (const ref of refs) {
+                if (ref.current)
+                    clearTimeout(ref.current);
+            }
         };
     }, []);
     const performAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {

package/dist/src/presentation/web/components/common/repository-node/use-repository-actions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-repository-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/repository-node/use-repository-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,UAAU,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,aAAa,EAAE,OAAO,CAAC;IACvB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAID,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,sBAAsB,GAAG,IAAI,GAAG,sBAAsB,CAkIjG"}
+{"version":3,"file":"use-repository-actions.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/repository-node/use-repository-actions.ts"],"names":[],"mappings":"AAOA,MAAM,WAAW,sBAAsB;IACrC,cAAc,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,WAAW,sBAAsB;IACrC,SAAS,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC/B,WAAW,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,UAAU,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,CAAC;IACpB,YAAY,EAAE,OAAO,CAAC;IACtB,aAAa,EAAE,OAAO,CAAC;IACvB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAID,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,sBAAsB,GAAG,IAAI,GAAG,sBAAsB,CAyGjG"}

package/dist/src/presentation/web/components/common/repository-node/use-repository-actions.js

@@ -16,28 +16,27 @@ export function useRepositoryActions(input) {
     const folderTimerRef = useRef(null);
     // Clear timers on unmount
     useEffect(() => {
-        const ideTimer = ideTimerRef.current;
-        const shellTimer = shellTimerRef.current;
-        const folderTimer = folderTimerRef.current;
+        const ideRef = ideTimerRef;
+        const shellRef = shellTimerRef;
+        const folderRef = folderTimerRef;
         return () => {
-            if (ideTimer)
-                clearTimeout(ideTimer);
-            if (shellTimer)
-                clearTimeout(shellTimer);
-            if (folderTimer)
-                clearTimeout(folderTimer);
+            if (ideRef.current)
+                clearTimeout(ideRef.current);
+            if (shellRef.current)
+                clearTimeout(shellRef.current);
+            if (folderRef.current)
+                clearTimeout(folderRef.current);
         };
     }, []);
-    const performToolbarAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
+    const performAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
         if (!input || isLoading)
             return;
-        // Clear any existing timer
         if (timerRef.current)
             clearTimeout(timerRef.current);
         setLoading(true);
         setError(null);
         try {
-            const result = await action({ repositoryPath: input.repositoryPath });
+            const result = await action();
             if (!result.success) {
                 const errorMessage = result.error ?? 'An unexpected error occurred';
                 setError(errorMessage);
@@ -53,33 +52,9 @@ export function useRepositoryActions(input) {
             setLoading(false);
         }
     }, [input]);
-    const performFolderAction = useCallback(async (action, setLoading, setError, timerRef, isLoading) => {
-        if (!input || isLoading)
-            return;
-        if (timerRef.current)
-            clearTimeout(timerRef.current);
-        setLoading(true);
-        setError(null);
-        try {
-            const result = await action(input.repositoryPath);
-            if (!result.success) {
-                const errorMessage = result.error ?? 'An unexpected error occurred';
-                setError(errorMessage);
-                timerRef.current = setTimeout(() => setError(null), ERROR_CLEAR_DELAY);
-            }
-        }
-        catch (err) {
-            const errorMessage = err instanceof Error ? err.message : 'An unexpected error occurred';
-            setError(errorMessage);
-            timerRef.current = setTimeout(() => setError(null), ERROR_CLEAR_DELAY);
-        }
-        finally {
-            setLoading(false);
-        }
-    }, [input]);
-    const handleOpenIde = useCallback(() => performToolbarAction(openIde, setIdeLoading, setIdeError, ideTimerRef, ideLoading), [performToolbarAction, ideLoading]);
-    const handleOpenShell = useCallback(() => performToolbarAction(openShell, setShellLoading, setShellError, shellTimerRef, shellLoading), [performToolbarAction, shellLoading]);
-    const handleOpenFolder = useCallback(() => performFolderAction(openFolder, setFolderLoading, setFolderError, folderTimerRef, folderLoading), [performFolderAction, folderLoading]);
+    const handleOpenIde = useCallback(() => performAction(() => openIde({ repositoryPath: input.repositoryPath }), setIdeLoading, setIdeError, ideTimerRef, ideLoading), [performAction, ideLoading, input]);
+    const handleOpenShell = useCallback(() => performAction(() => openShell({ repositoryPath: input.repositoryPath }), setShellLoading, setShellError, shellTimerRef, shellLoading), [performAction, shellLoading, input]);
+    const handleOpenFolder = useCallback(() => performAction(() => openFolder(input.repositoryPath), setFolderLoading, setFolderError, folderTimerRef, folderLoading), [performAction, folderLoading, input]);
     return {
         openInIde: handleOpenIde,
         openInShell: handleOpenShell,

package/dist/src/presentation/web/components/features/control-center/control-center-inner.js

@@ -18,19 +18,18 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
     const { nodes, edges, onNodesChange, handleConnect, handleAddRepository, handleDeleteFeature, handleDeleteRepository, createFeatureNode, setCallbacks, } = useControlCenterState(initialNodes, initialEdges);
     // Publish sidebar features to context whenever feature node data changes
     const { setFeatures: setSidebarFeatures } = useSidebarFeaturesContext();
+    const featureNodes = useMemo(() => nodes.filter((n) => n.type === 'featureNode'), [nodes]);
     const sidebarKey = useMemo(() => {
-        return nodes
-            .filter((n) => n.type === 'featureNode')
+        return featureNodes
             .map((n) => {
             const d = n.data;
-            return `${d.featureId}:${d.state}:${d.runtime ?? ''}:${d.startedAt ?? ''}`;
+            return `${d.featureId}:${d.state}:${d.name}`;
         })
             .sort()
             .join(',');
-    }, [nodes]);
+    }, [featureNodes]);
     useEffect(() => {
-        const sidebarItems = nodes
-            .filter((n) => n.type === 'featureNode')
+        const sidebarItems = featureNodes
             .map((n) => {
             const d = n.data;
             const status = mapNodeStateToSidebarStatus(d.state);
@@ -46,7 +45,7 @@ export function ControlCenterInner({ initialNodes, initialEdges }) {
         })
             .filter(Boolean);
         setSidebarFeatures(sidebarItems);
-    }, [sidebarKey, nodes, setSidebarFeatures]);
+    }, [sidebarKey, featureNodes, setSidebarFeatures]);
     // ── URL-based navigation handlers ────────────────────────────────────
     const handleNodeClick = useCallback((_event, node) => {
         if (node.type === 'featureNode') {

package/dist/src/presentation/web/components/features/control-center/use-control-center-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-control-center-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/use-control-center-state.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAClE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAWjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAG7E,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,aAAa,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAC/D,aAAa,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAChD,mBAAmB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,sBAAsB,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,iBAAiB,EAAE,CACjB,YAAY,EAAE,MAAM,GAAG,IAAI,EAC3B,YAAY,CAAC,EAAE,OAAO,CAAC,eAAe,CAAC,EACvC,QAAQ,CAAC,EAAE,MAAM,KACd,MAAM,CAAC;IACZ,wDAAwD;IACxD,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,gDAAgD;IAChD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,0EAA0E;IAC1E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;CACnD;AAOD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CA+UpB"}
+{"version":3,"file":"use-control-center-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/control-center/use-control-center-state.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,UAAU,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAClE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AACxE,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AAC9E,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAcjC,OAAO,EAAiB,KAAK,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAM7E,MAAM,WAAW,kBAAkB;IACjC,KAAK,EAAE,cAAc,EAAE,CAAC;IACxB,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,aAAa,EAAE,CAAC,OAAO,EAAE,UAAU,CAAC,cAAc,CAAC,EAAE,KAAK,IAAI,CAAC;IAC/D,aAAa,EAAE,CAAC,UAAU,EAAE,UAAU,KAAK,IAAI,CAAC;IAChD,mBAAmB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAC;IAC5C,YAAY,EAAE,CAAC,SAAS,EAAE,eAAe,KAAK,IAAI,CAAC;IACnD,mBAAmB,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;IACjD,sBAAsB,EAAE,CAAC,YAAY,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,iBAAiB,EAAE,CACjB,YAAY,EAAE,MAAM,GAAG,IAAI,EAC3B,YAAY,CAAC,EAAE,OAAO,CAAC,eAAe,CAAC,EACvC,QAAQ,CAAC,EAAE,MAAM,KACd,MAAM,CAAC;IACZ,wDAAwD;IACxD,wBAAwB,EAAE,CAAC,aAAa,EAAE,MAAM,KAAK,MAAM,GAAG,SAAS,CAAC;IACxE,gDAAgD;IAChD,iBAAiB,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,kBAAkB,GAAG,SAAS,CAAC;IACtE,0EAA0E;IAC1E,YAAY,EAAE,CAAC,SAAS,EAAE,cAAc,KAAK,IAAI,CAAC;CACnD;AAOD,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CAmXpB"}