@shepai/cli 1.72.8-pr216.5a2ec32 → 1.72.8-pr216.e1e2876

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

@@ -1 +1 @@
-{"version":3,"file":"create-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/create-drawer-client.tsx"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAG5E,MAAM,WAAW,uBAAuB;IACtC,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AAED,wBAAgB,kBAAkB,CAAC,EACjC,cAAc,EACd,eAAe,EACf,QAAQ,EACR,gBAAgB,GACjB,EAAE,uBAAuB,2CA+EzB"}
+{"version":3,"file":"create-drawer-client.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/control-center-drawer/create-drawer-client.tsx"],"names":[],"mappings":"AAQA,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,iEAAiE,CAAC;AAC3G,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,qCAAqC,CAAC;AAG5E,MAAM,WAAW,uBAAuB;IACtC,cAAc,EAAE,MAAM,CAAC;IACvB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,mBAAmB,EAAE,CAAC;IAChC,gBAAgB,CAAC,EAAE,gBAAgB,CAAC;CACrC;AAED,wBAAgB,kBAAkB,CAAC,EACjC,cAAc,EACd,eAAe,EACf,QAAQ,EACR,gBAAgB,GACjB,EAAE,uBAAuB,2CAgFzB"}

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

@@ -17,14 +17,16 @@ export function CreateDrawerClient({ repositoryPath, initialParentId, features,
     // When submitting, force the drawer closed immediately — router.push('/')
     // is async and the pathname may not update before the next render.
     const pathname = usePathname();
-    const isOpen = !isSubmitting && pathname.startsWith('/create');
-    // Safety net: if the drawer is closed (e.g. via form submit + isSubmitting)
-    // but router.push('/') hasn't taken effect yet, force-clear the route.
+    const isOnCreateRoute = pathname.startsWith('/create');
+    const isOpen = !isSubmitting && isOnCreateRoute;
+    // Reset isSubmitting once the route has actually changed away from /create,
+    // so the drawer can reopen on future visits. Without this, isSubmitting
+    // would stay true forever since parallel routes preserve component state.
     useEffect(() => {
-        if (!isOpen && pathname.startsWith('/create')) {
-            router.replace('/');
+        if (!isOnCreateRoute && isSubmitting) {
+            setIsSubmitting(false);
         }
-    }, [isOpen, pathname, router]);
+    }, [isOnCreateRoute, isSubmitting]);
     const onClose = useCallback(() => {
         router.push('/');
     }, [router]);
@@ -55,8 +57,7 @@ export function CreateDrawerClient({ repositoryPath, initialParentId, features,
             .catch(() => {
             toast.error('Failed to create feature');
             window.dispatchEvent(new CustomEvent('shep:feature-create-failed'));
-        })
-            .finally(() => {
+            // Reset on error so the user can retry without navigating away
             setIsSubmitting(false);
         });
     }, [router, createSound]);

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":"AAMA,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,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAYjC,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;CACb;AAID,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CA+jBpB"}
+{"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":"AAMA,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,cAAc,EAAE,MAAM,uCAAuC,CAAC;AAC5E,OAAO,EAGL,KAAK,eAAe,EACrB,MAAM,yBAAyB,CAAC;AAYjC,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;CACb;AAID,wBAAgB,qBAAqB,CACnC,YAAY,EAAE,cAAc,EAAE,EAC9B,YAAY,EAAE,IAAI,EAAE,GACnB,kBAAkB,CAwlBpB"}

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

@@ -206,13 +206,36 @@ export function useControlCenterState(initialNodes, initialEdges) {
                             },
                         };
                     });
-                    // Add any new server nodes that don't exist on client
+                    // Find new server nodes not yet on the client
                     const clientIds = new Set(currentNodes.map((n) => n.id));
-                    const newNodes = serverNodes.filter((n) => !clientIds.has(n.id));
-                    if (newNodes.length > 0) {
-                        changed = true;
+                    const newServerNodes = serverNodes.filter((n) => !clientIds.has(n.id));
+                    // Replace optimistic "creating" temp nodes with their real server
+                    // counterparts. Temp nodes have IDs like "feature-<ts>" while real
+                    // nodes use "feat-<uuid>". Match by repositoryPath + name.
+                    if (newServerNodes.length > 0) {
+                        const creatingIndices = [];
+                        merged.forEach((n, i) => {
+                            if (n.type === 'featureNode' && n.data.state === 'creating') {
+                                creatingIndices.push(i);
+                            }
+                        });
+                        for (const serverNode of newServerNodes) {
+                            if (serverNode.type !== 'featureNode' || creatingIndices.length === 0)
+                                continue;
+                            const idx = creatingIndices.shift();
+                            // Replace the temp node in-place, keeping its position
+                            merged[idx] = { ...serverNode, position: merged[idx].position };
+                            changed = true;
+                        }
+                        // Any remaining new server nodes that didn't replace a temp node
+                        const replacedIds = new Set(merged.map((n) => n.id));
+                        const trueNewNodes = newServerNodes.filter((n) => !replacedIds.has(n.id));
+                        if (trueNewNodes.length > 0) {
+                            changed = true;
+                            return [...merged, ...trueNewNodes];
+                        }
                     }
-                    return changed || newNodes.length > 0 ? [...merged, ...newNodes] : currentNodes;
+                    return changed ? merged : currentNodes;
                 });
             }
             catch {