@cosmicdrift/kumiko-renderer 0.164.0 → 0.165.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.164.0",
+  "version": "0.165.1",
   "description": "Platform-agnostic React renderer for Kumiko screens. Contains the shared logic — primitives-contract, hooks, KumikoScreen, navigation & SSE abstractions — that any platform-specific renderer (web, native) composes. No DOM, no EventSource, no react-dom.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -15,8 +15,8 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.164.0",
-    "@cosmicdrift/kumiko-headless": "0.164.0",
+    "@cosmicdrift/kumiko-framework": "0.165.1",
+    "@cosmicdrift/kumiko-headless": "0.165.1",
     "react": "^19.2.6",
     "temporal-polyfill": "^0.3.2"
   },

package/src/__tests__/sort-by-accessor.test.ts

@@ -3,11 +3,13 @@ import { sortByAccessor } from "../sort-by-accessor";
 
 type Row = { readonly name: string; readonly count: number };
 
-const rows: readonly Row[] = [
-  { name: "b", count: 2 },
-  { name: "a", count: 3 },
-  { name: "c", count: 1 },
-];
+function makeRows(): readonly Row[] {
+  return [
+    { name: "b", count: 2 },
+    { name: "a", count: 3 },
+    { name: "c", count: 1 },
+  ];
+}
 
 const accessors = {
   name: (r: Row) => r.name,
@@ -16,24 +18,29 @@ const accessors = {
 
 describe("sortByAccessor", () => {
   test("sort === null returns rows unchanged (same reference)", () => {
+    const rows = makeRows();
     expect(sortByAccessor(rows, null, accessors)).toBe(rows);
   });
 
   test("an unknown field returns rows unchanged (same reference)", () => {
+    const rows = makeRows();
     expect(sortByAccessor(rows, { field: "nope", dir: "asc" }, accessors)).toBe(rows);
   });
 
   test("sorts ascending by the given accessor", () => {
+    const rows = makeRows();
     const sorted = sortByAccessor(rows, { field: "name", dir: "asc" }, accessors);
     expect(sorted.map((r) => r.name)).toEqual(["a", "b", "c"]);
   });
 
   test("sorts descending by the given accessor", () => {
+    const rows = makeRows();
     const sorted = sortByAccessor(rows, { field: "count", dir: "desc" }, accessors);
     expect(sorted.map((r) => r.count)).toEqual([3, 2, 1]);
   });
 
   test("does not mutate the input array", () => {
+    const rows = makeRows();
     const original = [...rows];
     sortByAccessor(rows, { field: "name", dir: "asc" }, accessors);
     expect(rows).toEqual(original);

package/src/format-when.ts

@@ -1,21 +1,11 @@
-import { Temporal } from "temporal-polyfill";
+import { toInstant } from "@cosmicdrift/kumiko-headless";
 
 // Shared timestamp formatter for operator screens (audit log, job runs) —
 // falls back to the raw ISO string on an unparseable value instead of "Invalid Date".
 export function formatWhen(value: string): string {
   try {
-    return Temporal.Instant.from(value).toLocaleString();
+    return toInstant(value).toLocaleString();
   } catch {
-    // Temporal.Instant.from requires a UTC designator/offset — offset-less
-    // timestamps (still valid `new Date` input) throw here. Retry as a
-    // local wall-clock time before giving up and returning the raw string.
-    try {
-      return Temporal.PlainDateTime.from(value)
-        .toZonedDateTime(Temporal.Now.timeZoneId())
-        .toInstant()
-        .toLocaleString();
-    } catch {
-      return value;
-    }
+    return value;
   }
 }

package/src/hooks/use-stream-handler.ts

@@ -38,7 +38,6 @@ export function useStreamHandler<TChunk = unknown>(
   const [error, setError] = useState<DispatcherError | null>(null);
 
   const activeCtrl = useRef<AbortController | null>(null);
-  const payloadKey = JSON.stringify(payload);
   const payloadRef = useRef(payload);
   payloadRef.current = payload;
 
@@ -56,7 +55,12 @@ export function useStreamHandler<TChunk = unknown>(
     setError(null);
   }, [abort]);
 
-  // biome-ignore lint/correctness/useExhaustiveDependencies: payload via payloadKey / payloadRef
+  // Shared by every abort-detection branch below: only touch status if this
+  // run is still the active one (or activeCtrl was already cleared by abort()).
+  const settleAborted = useCallback((ctrl: AbortController): void => {
+    if (activeCtrl.current === null || activeCtrl.current === ctrl) setStatus("idle");
+  }, []);
+
   const start = useCallback(
     async (overridePayload?: unknown): Promise<void> => {
       activeCtrl.current?.abort();
@@ -69,48 +73,43 @@ export function useStreamHandler<TChunk = unknown>(
 
       const body = overridePayload !== undefined ? overridePayload : payloadRef.current;
       try {
-        const accumulated: TChunk[] = [];
         for await (const chunk of dispatcher.stream<TChunk>(type, body, { signal: ctrl.signal })) {
           // skip: a newer start() already superseded this run
           if (ctrl.signal.aborted) {
-            if (activeCtrl.current === null || activeCtrl.current === ctrl) setStatus("idle");
+            settleAborted(ctrl);
             return;
           }
-          accumulated.push(chunk);
-          setChunks([...accumulated]);
+          setChunks((prev) => [...prev, chunk]);
         }
         // skip: aborted after last chunk, don't mark done
         if (ctrl.signal.aborted) {
-          if (activeCtrl.current === null || activeCtrl.current === ctrl) setStatus("idle");
+          settleAborted(ctrl);
           return;
         }
         setStatus("done");
       } catch (e) {
         // skip: abort is not an error toast
         if (ctrl.signal.aborted) {
-          if (activeCtrl.current === null || activeCtrl.current === ctrl) setStatus("idle");
+          settleAborted(ctrl);
           return;
         }
         const mapped = asDispatcherError(e);
         // skip: dispatcher-mapped abort (fetch cancelled)
         if (mapped.code === "aborted") {
-          if (activeCtrl.current === null || activeCtrl.current === ctrl) setStatus("idle");
+          settleAborted(ctrl);
           return;
         }
         setError(mapped);
         setStatus("error");
       }
     },
-    [dispatcher, type, payloadKey],
+    [dispatcher, type, settleAborted],
   );
 
   useEffect(() => {
     // skip: autoStart off — streams are usually user-triggered
     if (!autoStart) return;
     void start();
-    return () => {
-      activeCtrl.current?.abort();
-    };
   }, [autoStart, start]);
 
   useEffect(() => {