@bastani/atomic 0.5.4 → 0.5.5

package/src/lib/spawn.ts

@@ -5,7 +5,8 @@
  * eliminating duplication across postinstall-playwright, postinstall-liteparse, etc.
  */
 
-import { join } from "path";
+import { join } from "node:path";
+import { homedir } from "node:os";
 
 export interface SpawnResult {
   success: boolean;
@@ -73,10 +74,12 @@ export function prependPath(directory: string): void {
 }
 
 /**
- * Get the user's home directory from environment variables.
+ * Get the user's home directory.
+ * Uses Node.js os.homedir() which handles cross-platform resolution
+ * (HOME on Unix, USERPROFILE on Windows, and fallback to passwd on Linux).
  */
-export function getHomeDir(): string | undefined {
-  return process.env.HOME ?? process.env.USERPROFILE;
+export function getHomeDir(): string {
+  return homedir();
 }
 
 /**
@@ -163,13 +166,16 @@ async function installNodeViaFnm(quiet: boolean): Promise<boolean> {
 
   // Activate the installed version by adding its bin dir to PATH.
   const envShell = process.platform === "win32" ? "cmd" : "bash";
-  const envResult = Bun.spawnSync({
+  const envProc = Bun.spawn({
     cmd: [fnmPath, "env", "--shell", envShell],
     stdout: "pipe",
     stderr: "pipe",
   });
-  if (envResult.success) {
-    const envOutput = envResult.stdout.toString();
+  const [envOutput, envExitCode] = await Promise.all([
+    new Response(envProc.stdout).text(),
+    envProc.exited,
+  ]);
+  if (envExitCode === 0) {
     if (process.platform === "win32") {
       // cmd output: SET "PATH=C:\...\fnm_multishells\...;..."
       const pathMatch = envOutput.match(/SET "PATH=([^"]+?)"/i);

package/src/scripts/bump-version.ts

@@ -18,7 +18,7 @@
  */
 
 import { $ } from "bun";
-import { resolve } from "path";
+import { resolve } from "node:path";
 import { VERSION_FILES } from "./constants-base.ts";
 
 const ROOT = resolve(import.meta.dir, "../..");

package/src/scripts/constants.ts

@@ -5,8 +5,8 @@
  * change propagates everywhere.
  */
 
-import { AGENTS } from "@/sdk/workflows/index.ts";
-import type { AgentType } from "@/sdk/workflows/index.ts";
+import { AGENTS } from "../sdk/workflows/index.ts";
+import type { AgentType } from "../sdk/workflows/index.ts";
 
 export {
   SDK_PACKAGE_NAME,

package/src/sdk/components/header.tsx

@@ -1,14 +1,28 @@
 /** @jsxImportSource @opentui/react */
 
+import { useMemo } from "react";
 import type { SessionStatus } from "./orchestrator-panel-types.ts";
-import { useStore, useGraphTheme } from "./orchestrator-panel-contexts.ts";
+import { useStore, useGraphTheme, useStoreVersion } from "./orchestrator-panel-contexts.ts";
+
+function CountBadge({ color, icon, count }: { color: string; icon: string; count: number }) {
+  if (count <= 0) return null;
+  return (
+    <text>
+      <span fg={color}>{icon} {count}</span>
+    </text>
+  );
+}
 
 export function Header() {
   const store = useStore();
   const theme = useGraphTheme();
+  const storeVersion = useStoreVersion(store);
 
-  const counts: Record<SessionStatus, number> = { complete: 0, running: 0, pending: 0, error: 0 };
-  for (const s of store.sessions) counts[s.status]++;
+  const counts = useMemo(() => {
+    const c: Record<SessionStatus, number> = { complete: 0, running: 0, pending: 0, error: 0 };
+    for (const s of store.sessions) c[s.status]++;
+    return c;
+  }, [storeVersion]);
 
   const isFailed = store.fatalError !== null;
   const isDone = store.completionInfo !== null;
@@ -34,26 +48,10 @@ export function Header() {
       </text>
 
       <box flexGrow={1} justifyContent="flex-end" flexDirection="row" gap={2}>
-        {counts.complete > 0 ? (
-          <text>
-            <span fg={theme.success}>{"\u2713"} {counts.complete}</span>
-          </text>
-        ) : null}
-        {counts.running > 0 ? (
-          <text>
-            <span fg={theme.warning}>{"\u25CF"} {counts.running}</span>
-          </text>
-        ) : null}
-        {counts.pending > 0 ? (
-          <text>
-            <span fg={theme.textDim}>{"\u25CB"} {counts.pending}</span>
-          </text>
-        ) : null}
-        {counts.error > 0 ? (
-          <text>
-            <span fg={theme.error}>{"\u2717"} {counts.error}</span>
-          </text>
-        ) : null}
+        <CountBadge color={theme.success} icon={"\u2713"} count={counts.complete} />
+        <CountBadge color={theme.warning} icon={"\u25CF"} count={counts.running} />
+        <CountBadge color={theme.textDim} icon={"\u25CB"} count={counts.pending} />
+        <CountBadge color={theme.error} icon={"\u2717"} count={counts.error} />
       </box>
     </box>
   );

package/src/sdk/components/hooks.ts

@@ -0,0 +1,21 @@
+/**
+ * Shared React hooks for OpenTUI components.
+ */
+
+import { useRef } from "react";
+
+/**
+ * Return a ref whose `.current` always holds the latest value.
+ *
+ * Useful for reading state inside event callbacks (e.g. `useKeyboard`)
+ * that capture the initial closure and would otherwise go stale.
+ *
+ * This is safe because OpenTUI's React reconciler is synchronous —
+ * the ref is assigned during render, which is guaranteed to complete
+ * before any event handler fires.
+ */
+export function useLatest<T>(value: T): React.RefObject<T> {
+  const ref = useRef(value);
+  ref.current = value;
+  return ref;
+}

package/src/sdk/components/node-card.tsx

@@ -1,11 +1,12 @@
 /** @jsxImportSource @opentui/react */
 
+import React from "react";
 import { lerpColor } from "./color-utils.ts";
 import { useGraphTheme } from "./orchestrator-panel-contexts.ts";
 import { statusColor, fmtDuration } from "./status-helpers.ts";
 import { NODE_W, type LayoutNode } from "./layout.ts";
 
-export function NodeCard({
+export const NodeCard = React.memo(function NodeCard({
   node,
   focused,
   pulsePhase,
@@ -65,4 +66,4 @@ export function NodeCard({
       </box>
     </box>
   );
-}
+});

package/src/sdk/components/session-graph-panel.tsx

@@ -25,9 +25,7 @@ import {
   useStoreVersion,
   TmuxSessionContext,
 } from "./orchestrator-panel-contexts.ts";
-import { computeLayout } from "./layout.ts";
-import { NODE_W, NODE_H } from "./layout.ts";
-import type { LayoutNode } from "./layout.ts";
+import { computeLayout, NODE_W, NODE_H, type LayoutNode } from "./layout.ts";
 import { buildConnector, buildMergeConnector } from "./connectors.ts";
 import type { ConnectorResult } from "./connectors.ts";
 import { NodeCard } from "./node-card.tsx";
@@ -85,23 +83,21 @@ export function SessionGraphPanel() {
   }, [storeVersion]);
 
   // Pulse animation for running nodes — paused when nothing is running
-  const hasRunning = store.sessions.some((s) => s.status === "running");
+  const hasRunning = useMemo(
+    () => store.sessions.some((s) => s.status === "running"),
+    [storeVersion],
+  );
   const [pulsePhase, setPulsePhase] = useState(0);
+  // Pulse animation doubles as a live timer refresh — 60ms updates keep
+  // both the pulse animation and duration displays current, so a separate
+  // 1s tick interval is unnecessary.
   useEffect(() => {
     if (!hasRunning) return;
-    const id = setInterval(
+    const pulseId = setInterval(
       () => setPulsePhase((p: number) => (p + 1) % PULSE_FRAME_COUNT),
       PULSE_INTERVAL_MS,
     );
-    return () => clearInterval(id);
-  }, [hasRunning]);
-
-  // Live timer refresh — re-render every second while any session is running
-  const [, setTick] = useState(0);
-  useEffect(() => {
-    if (!hasRunning) return;
-    const id = setInterval(() => setTick((t) => t + 1), 1000);
-    return () => clearInterval(id);
+    return () => clearInterval(pulseId);
   }, [hasRunning]);
 
   // Attach flash message
@@ -129,13 +125,13 @@ export function SessionGraphPanel() {
 
       tmuxRun(["switch-client", "-t", `${tmuxSession}:${n.name}`]);
     },
-    [layout.map, tmuxSession, store.sessions],
+    [layout.map, tmuxSession],
   );
 
   // Spatial navigation
   const navigate = useCallback(
     (dir: "left" | "right" | "up" | "down") => {
-      const cur = layout.map[focusedId];
+      const cur = layout.map[focusedIdRef.current];
       if (!cur) return;
       const cx = cur.x + NODE_W / 2;
       const cy = cur.y + NODE_H / 2;
@@ -143,7 +139,7 @@ export function SessionGraphPanel() {
       let bestDist = Infinity;
 
       for (const n of nodeList) {
-        if (n.name === focusedId) continue;
+        if (n.name === focusedIdRef.current) continue;
         const nx = n.x + NODE_W / 2;
         const ny = n.y + NODE_H / 2;
         const dx = nx - cx;
@@ -169,7 +165,7 @@ export function SessionGraphPanel() {
 
       if (best) setFocusedId(best.name);
     },
-    [focusedId, layout.map, nodeList],
+    [layout.map, nodeList],
   );
 
   // gg double-tap tracking