makersclaw 0.4.0 → 0.5.1

package/templates/next/files/lib/hooks.js

@@ -415,16 +415,19 @@ export function useJob() {
   const movedRef = useRef(null); // the last status|progress|updated_at seen
   const aliveRef = useRef(true);
 
-  useEffect(() => {
-    aliveRef.current = true;
-    return () => {
-      aliveRef.current = false;
-      abortRef.current?.abort();
-      followRef.current?.();
-      followRef.current = null;
-    };
+  // No job at all: the panel closes.
+  const idle = useCallback(() => {
+    pathRef.current = null;
+    sessionRef.current = null;
+    requestRef.current = null;
+    movedRef.current = null;
+    setState(IDLE);
   }, []);
 
+  /** An ask or a stream `controller` cut off by an unmount the hook came
+   *  back from, with nothing started since: see the effect below. */
+  const cutOff = (controller) => Boolean(controller.unmounted) && abortRef.current === controller && aliveRef.current;
+
   const apply = useCallback((request) => {
     if (!aliveRef.current) return;
     requestRef.current = request;
@@ -490,6 +493,33 @@ export function useJob() {
     [apply],
   );
 
+  // Unmounting aborts what is in flight and stops following. React runs
+  // that cleanup and mounts the hook again in development (its double
+  // mount; a fast refresh does the same), and up to 0.4.0 the hook came
+  // back to a panel that lied: 0.3.x called the aborted job done — work
+  // that never ran — and 0.4.0 left it running with nothing behind it
+  // (makersclaw-store BLOCKS.md; mind-maps and landing-page-reviewer
+  // started jobs from a timer instead). Now, back on its feet, the hook
+  // follows a live request again, and an ask the cleanup cut off before
+  // it was answered closes the panel: neither done nor failed (start and
+  // legacyStart, via cutOff).
+  useEffect(() => {
+    aliveRef.current = true;
+    const request = requestRef.current;
+    if (!followRef.current && pathRef.current === "request" && request?.id && LIVE.has(request.status)) {
+      const controller = new AbortController();
+      abortRef.current = controller;
+      follow(request.id, controller.signal);
+    }
+    return () => {
+      aliveRef.current = false;
+      if (abortRef.current) abortRef.current.unmounted = true;
+      abortRef.current?.abort();
+      followRef.current?.();
+      followRef.current = null;
+    };
+  }, [follow]);
+
   // The 0.3.x door, as it was: _claw/chat streamed into the app. Taken
   // only on a claw that has no _claw/ask.
   const legacyStart = useCallback(async (text, { sessionId, signal }) => {
@@ -531,8 +561,13 @@ export function useJob() {
     };
 
     const result = await startJob(text, { sessionId: thread ?? undefined, onEvent, signal });
-    // Aborted means a new start, a cancel or a reset already set the state.
-    if (!aliveRef.current || signal.aborted) return result;
+    // Aborted means a new start, a cancel or a reset already set the
+    // state — or an unmount cut the stream, and the panel closes.
+    if (!aliveRef.current || signal.aborted) {
+      const current = abortRef.current;
+      if (current?.signal === signal && cutOff(current)) idle();
+      return result;
+    }
     // A stream that ends without `done` (a dropped connection, a 4xx
     // before the first frame) must still leave the panel closable.
     if (!result.ok && !result.aborted)
@@ -565,7 +600,11 @@ export function useJob() {
         setState((s) => ({ ...s, sessionId: null }));
         result = await asking(null);
       }
-      if (!aliveRef.current || controller.signal.aborted) return result;
+      if (!aliveRef.current || controller.signal.aborted) {
+        // Cut off unanswered by an unmount: nothing to follow, no panel.
+        if (cutOff(controller)) idle();
+        return result;
+      }
 
       // A 404 that names no refusal: a claw from before _claw/ask. Its
       // _claw/chat still runs the job, so a 0.4.0 app works there too.
@@ -624,12 +663,8 @@ export function useJob() {
     abortRef.current?.abort();
     followRef.current?.();
     followRef.current = null;
-    pathRef.current = null;
-    sessionRef.current = null;
-    requestRef.current = null;
-    movedRef.current = null;
-    setState(IDLE);
-  }, []);
+    idle();
+  }, [idle]);
 
   // Take the founder to the conversation: the question when it needs
   // them, the account of the work otherwise. False on the 0.3.x path.

package/templates/next/template.json

@@ -96,8 +96,8 @@
     {
       "name": "calendar",
       "type": "component",
-      "title": "Calendar, parseDate, isDateString, toDateString, todayString, addDays, addMonths, startOfMonth, endOfMonth, dayOfWeek, formatDate, firstDayOfWeek",
-      "description": "A month grid, the WAI-ARIA date grid (arrows a day or a week, Page Up/Down a month, Home/End the week's edges, Enter picks): mode single · multiple · range, dates as \"YYYY-MM-DD\" strings in and out (no timezone drift); numberOfMonths side by side (stacked on a phone); captionLayout label · dropdown (month and year Selects) · none (the parent shows and moves a controlled month); min, max, disabled(date); today ringed, outside days muted, range start/middle/end; marked days (a dot in a tone, a label for readers; emphasizeMarked draws them bold and the rest muted; onlyMarked makes the rest unpickable, unstruck); week numbers; week start and names from Intl, weekStartsOn overrides; size md (44px days) · sm. The string-date helpers do the same arithmetic in a surface.",
+      "title": "Calendar, parseDate, isDateString, toDateString, todayString, addDays, addMonths, startOfMonth, endOfMonth, dayOfWeek, formatDate, toLocalDate, firstDayOfWeek",
+      "description": "A month grid, the WAI-ARIA date grid (arrows a day or a week, Page Up/Down a month, Home/End the week's edges, Enter picks): mode single · multiple · range, dates as \"YYYY-MM-DD\" strings in and out (no timezone drift); numberOfMonths side by side (stacked on a phone); captionLayout label · dropdown (month and year Selects) · none (the parent shows and moves a controlled month); min, max, disabled(date); today ringed, outside days muted, range start/middle/end; marked days (a dot in a tone, a label for readers; emphasizeMarked draws them bold and the rest muted; onlyMarked makes the rest unpickable, unstruck); week numbers; week start and names from Intl, weekStartsOn overrides; size md (44px days) · sm. The string-date helpers do the same arithmetic in a surface; toLocalDate turns a day (a `date` column's \"YYYY-MM-DD\") into its LOCAL midnight, for code that shows days beside times.",
       "tags": ["calendar", "date", "month", "range", "schedule", "day picker", "week"],
       "files": ["components/ui/calendar.jsx"]
     },