koneck 2.69.0 → 2.71.0

package/dist/ink-chat.js

@@ -1004,7 +1004,15 @@ export function defaultAltScreen(stdout = process.stdout, env = process.env) {
         return false;
     return true;
 }
-function App({ config: initialConfig, clearFrame }) {
+/**
+ * Exported for one reason: so a test can mount it.
+ *
+ * The CLI shipped unable to start — an expression in a useState initialiser reached for a helper
+ * declared further down the component, and every release after that crashed on the first render.
+ * Typechecking passes on that code and no test could reach it, because nothing in the suite mounted
+ * an Ink app. Something does now.
+ */
+export function App({ config: initialConfig, clearFrame }) {
     const { exit } = useApp();
     const [cfg, setCfg] = useState(initialConfig);
     // Tool approval callbacks belong to a session created on an earlier render. Keep the control
@@ -1330,9 +1338,22 @@ function App({ config: initialConfig, clearFrame }) {
             realTokRef.current = Math.max(0, s.completionTokens - tokBaseRef.current);
         },
     };
-    const activeSession = useState(() => ({
-        p: createAgentSession({ ...initialConfig, pace: paceFrom(initialConfig.pace ?? savedEffort(undefined), 'medium'), ...failoverHooks(), silent: true, ...sessionOpts }),
-    }))[0];
+    /**
+     * The live session, built on first use rather than while the component renders.
+     *
+     * It used to be created in this useState's initialiser, which runs during the render — and the
+     * hooks it is built with close over things declared further down the component, so the very first
+     * render threw "Cannot access 'failoverHooks' before initialization" and the CLI would not start
+     * at all. The initialiser had been fine until those hooks existed; adding them moved a working
+     * expression above its own dependencies, and nothing caught it because the test suite has no way
+     * to mount an Ink app. There is one now.
+     *
+     * Building it on first use fixes the ordering by construction: every caller runs from an effect or
+     * a callback, which is after the whole body has been evaluated. The mount effect that sets the
+     * pace asks for it immediately, so it is still created when the app starts, not lazily on the
+     * first message.
+     */
+    const activeSession = useState(() => ({ p: null }))[0];
     const [rows, setRows] = useState([{ role: 'header' }]);
     const [draft, setDraft] = useState('');
     // Recall is deliberately session-local. It is for quick retries and refinements, not another
@@ -1601,12 +1622,46 @@ function App({ config: initialConfig, clearFrame }) {
         // What this turn leaves behind, said where the rest of the session's own account of itself is.
         onHandoff: (said) => addSystem(said),
     });
+    /**
+     * The session, creating it if this is the first ask.
+     *
+     * Deliberately next to failoverHooks, because the two are why it cannot be built during the
+     * render: these hooks reach for state declared below them, so the session can only be assembled
+     * once the whole component body has run.
+     */
+    const sessionNow = () => {
+        if (!activeSession.p) {
+            activeSession.p = hold(createAgentSession({
+                ...initialConfig,
+                pace: paceFrom(initialConfig.pace ?? savedEffort(undefined), 'medium'),
+                ...failoverHooks(), silent: true, ...sessionOpts,
+            }));
+        }
+        return activeSession.p;
+    };
+    /**
+     * Parks a session promise so a failure cannot take the process down before anyone asks.
+     *
+     * A session is created and stored, then read later by whoever needs it. If creating it fails — an
+     * endpoint that is not listening is the ordinary case — the stored promise is rejected with nobody
+     * yet attached, and on Node 24 an unhandled rejection ends the process. So a no-op handler is
+     * attached the moment it is stored. The promise handed out is the original, so every caller still
+     * sees the failure and reports it in its own way; this only stops the copy sitting in a variable
+     * from being counted as ignored.
+     *
+     * Found by the test that mounts the app: it points at a stub that answers nothing useful, and six
+     * unhandled rejections came out of a suite that had been green.
+     */
+    function hold(p) {
+        p.catch(() => { });
+        return p;
+    }
     const effortRef = useRef('medium');
     useEffect(() => {
         effortRef.current = effort;
         void (async () => {
             try {
-                const live = await activeSession.p;
+                const live = await sessionNow();
                 live.setPace?.(paceFrom(effort, 'medium'));
             }
             catch { /* no session yet, and the next one is built with this pace */ }
@@ -1962,13 +2017,13 @@ function App({ config: initialConfig, clearFrame }) {
         submitDraft(task);
     }
     async function getSession() {
-        return activeSession.p;
+        return sessionNow();
     }
     async function resetSession(newCfg) {
         const c = newCfg ?? cfg;
         cfgRef.current = c;
         setDetectedContext(null);
-        activeSession.p = createAgentSession({ ...c, pace: paceFrom(effortRef.current, 'medium'), ...failoverHooks(), silent: true, ...sessionOpts });
+        activeSession.p = hold(createAgentSession({ ...c, pace: paceFrom(effortRef.current, 'medium'), ...failoverHooks(), silent: true, ...sessionOpts }));
         if (newCfg)
             setCfg(newCfg);
     }
@@ -2001,7 +2056,7 @@ function App({ config: initialConfig, clearFrame }) {
         setRepaint(n => n + 1);
     }
     async function doSave() {
-        const s = await activeSession.p.catch(() => null);
+        const s = await sessionNow().catch(() => null);
         const firstUser = s?.messages.find((m) => m.role === 'user');
         // Masked before it is written. A key pasted into the composer became the first user message,
         // which is where this title comes from — so it was persisted to disk and then printed in the
@@ -2124,7 +2179,7 @@ function App({ config: initialConfig, clearFrame }) {
                         ...(st.note ? { note: st.note } : {}),
                     })), updatedAt: Date.now() }
                 : undefined;
-            activeSession.p = createAgentSession({ ...restoredCfg, silent: true, ...sessionOpts }, messages, { turns: meta.turns, totalTokens: meta.totalTokens }, restoredPlan);
+            activeSession.p = hold(createAgentSession({ ...restoredCfg, silent: true, ...sessionOpts }, messages, { turns: meta.turns, totalTokens: meta.totalTokens }, restoredPlan));
             if (restoredPlan)
                 setPlan(restoredPlan);
             setSessionGoal(meta.goal ?? null);
@@ -2399,7 +2454,7 @@ function App({ config: initialConfig, clearFrame }) {
             return;
         }
         if (signalsRef.current === null) {
-            const live = await activeSession.p.catch(() => null);
+            const live = await sessionNow().catch(() => null);
             signalsRef.current = await gatherSignals(cfg.cwd, live?.trajectory ?? [])
                 .catch(() => ({}));
         }
@@ -2721,7 +2776,7 @@ function App({ config: initialConfig, clearFrame }) {
             }
             case '/mcp': {
                 const configured = await loadMCPConfig(cfg.cwd);
-                const session = await activeSession.p.catch(() => null);
+                const session = await sessionNow().catch(() => null);
                 addSystem(mcpStatusText(Object.keys(configured?.servers ?? {}), session?.mcpServers ?? []));
                 return;
             }
@@ -3130,7 +3185,7 @@ function App({ config: initialConfig, clearFrame }) {
             case '/trajectory': {
                 // The timings the transcript cannot answer for: where the time went, which tool was slow,
                 // how much of a long turn was spent waiting for a first token rather than generating.
-                const s = await activeSession.p.catch(() => null);
+                const s = await sessionNow().catch(() => null);
                 addSystem(renderTrajectory(s?.trajectory ?? []));
                 return;
             }
@@ -3398,7 +3453,7 @@ function App({ config: initialConfig, clearFrame }) {
                 return;
             }
             case '/rewind': {
-                const s = await activeSession.p.catch(() => null);
+                const s = await sessionNow().catch(() => null);
                 if (!s) {
                     addSystem('Nothing to rewind.');
                     return;