@iseoane/agent-depot 0.6.4 → 0.6.6

package/dist/src/tui/sources-recipes.js

@@ -10,10 +10,11 @@ import { hasUnmanagedCopy, isInstalled, loadInstalledSkills } from "./catalog-in
 import { appName, appRecipePreviewLines, appStepLines } from "./app-actions.js";
 import { errorText } from "./batch.js";
 import { useMounted } from "./view-state.js";
-import { BorderPanel, DecisionPrompt, PreviewConfirm } from "./panel-parts.js";
+import { BorderPanel, DecisionPrompt, PreviewConfirm, Working } from "./panel-parts.js";
 import { answerYesNo, movePreviewCursor } from "./mode-keys.js";
 import { previewViewport } from "./preview-window.js";
 import { theme } from "./theme.js";
+import { busy } from "./working-model.js";
 function recipeApprovalChrome(entry) {
     return {
         title: "Recipe approval · does not install",
@@ -43,6 +44,9 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
     const [error, setError] = useState();
     const [notice, setNotice] = useState();
     const [loadError, setLoadError] = useState();
+    // When the oldest running reload began; reloads can overlap, so a count decides when loading ends.
+    const [loadingSince, setLoadingSince] = useState(() => Date.now());
+    const reloading = useRef(0);
     const [mode, setModeState] = useState({ kind: "browse" });
     const modeRef = useRef(mode);
     const setMode = (next) => {
@@ -50,6 +54,8 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
         onCapturingChange?.(next.kind !== "browse");
         setModeState(next);
     };
+    /** Shows the busy panel; an operation already busy keeps its start time. */
+    const setBusy = (label) => setMode(busy(label, modeRef.current));
     const finish = async () => {
         await reload();
         if (mounted.current)
@@ -57,7 +63,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
     };
     const preview = async (entry, status, removeAfterApproval = false) => {
         setError(undefined);
-        setMode({ kind: "busy" });
+        setBusy("Preparing App recipe preview…");
         try {
             const steps = await apps.preview(entry);
             if (mounted.current) {
@@ -95,7 +101,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
         },
     };
     const startAI = async () => {
-        setMode({ kind: "busy" });
+        setBusy("Checking the authoring Skill…");
         try {
             if (!operations.discoverSkills)
                 throw new Error("Discovery is not supported by the configured operations");
@@ -124,7 +130,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
     };
     const startRemoval = async (entry) => {
         setError(undefined);
-        setMode({ kind: "busy" });
+        setBusy("Checking App recipe…");
         try {
             const approval = await apps.approvalStatus(entry);
             if (approval.status === "needs approval") {
@@ -143,7 +149,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
         }
     };
     const uninstall = async (current) => {
-        setMode({ kind: "busy" });
+        setBusy(current.kind === "uninstall-manual" ? "Checking App uninstall…" : "Uninstalling App…");
         try {
             const result = current.kind === "uninstall-manual"
                 ? await apps.completeManual(current.plan, true)
@@ -200,7 +206,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
             else if (input === "n" || key.escape)
                 cancel();
             else if (input === "y" && viewport.reachedEnd) {
-                setMode({ kind: "busy" });
+                setBusy("Approving App recipe…");
                 void apps.approve(current.entry).then(async () => {
                     await reload();
                     if (!mounted.current)
@@ -220,7 +226,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
         }
         if (current.kind === "uninstall-question") {
             answerYesNo(input, key, () => {
-                setMode({ kind: "busy" });
+                setBusy("Preparing App uninstall…");
                 void apps.planLifecycle(current.removal.entry, "uninstall").then(plan => {
                     if (mounted.current)
                         setMode({ kind: !plan.argv && plan.manual ? "uninstall-manual" : "uninstall-confirm", removal: current.removal, plan, cursor: 0 });
@@ -245,7 +251,7 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
         }
         if (current.kind === "remove") {
             answerYesNo(input, key, () => {
-                setMode({ kind: "busy" });
+                setBusy("Removing App recipe…");
                 void removeAppRecipe(apps, current.removal, true).then(async () => {
                     if (mounted.current)
                         setNotice(`Removed App recipe: ${current.removal.file}`);
@@ -276,6 +282,8 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
     };
     /** Reloads parsing and approval receipts, never version/latest commands. */
     const reload = useCallback(async () => {
+        if (reloading.current++ === 0 && mounted.current)
+            setLoadingSince(Date.now());
         try {
             const entries = await apps.load();
             const rows = await Promise.all(entries.map(async (entry) => {
@@ -294,9 +302,13 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
             if (mounted.current)
                 setLoadError(errorText(error));
         }
+        finally {
+            if (--reloading.current === 0 && mounted.current)
+                setLoadingSince(undefined);
+        }
     }, [apps, mounted]);
     useEffect(() => { void reload(); }, [reload]);
-    const panel = (_jsxs(Box, { flexDirection: "column", children: [loadError ? _jsxs(Text, { color: theme.error, children: ["Error loading App recipes: ", loadError] }) : null, error ? _jsx(Text, { color: theme.error, children: error }) : null, notice ? _jsx(Text, { color: theme.success, children: notice }) : null, mode.kind === "guide" ? (_jsxs(BorderPanel, { title: "Add an App recipe", children: [_jsx(Text, { children: "1 With the AI skill \u00B7 install if missing, then copy the prompt" }), _jsx(Text, { children: "2 Manual \u00B7 write a recipe using the schema" }), _jsx(Text, { children: "n or Esc closes" })] })) : null, mode.kind === "manual" ? (_jsxs(BorderPanel, { title: "Write an App recipe", children: [_jsxs(Text, { children: ["Write a JSON recipe in ", apps.directory] }), _jsx(Text, { children: "Schema: agent-depot app schema \u00B7 n or Esc closes" })] })) : null, mode.kind === "prompt" ? (_jsxs(BorderPanel, { title: "App recipe authoring prompt", children: [_jsx(Text, { children: "Use the agent-depot-apprecipe skill to create the recipe for <app>" }), _jsxs(Text, { children: ["Recipes: ", apps.directory, " \u00B7 n or Esc closes"] })] })) : null, mode.kind === "install" ? _jsx(ActionPanel, { mode: mode.action }) : null, mode.kind === "busy" ? _jsx(BorderPanel, { title: "App recipe workflow", children: _jsx(Text, { children: "Working..." }) }) : null, mode.kind === "approve" ? (() => {
+    const panel = (_jsxs(Box, { flexDirection: "column", children: [loadError ? _jsxs(Text, { color: theme.error, children: ["Error loading App recipes: ", loadError] }) : null, error ? _jsx(Text, { color: theme.error, children: error }) : null, notice ? _jsx(Text, { color: theme.success, children: notice }) : null, mode.kind === "guide" ? (_jsxs(BorderPanel, { title: "Add an App recipe", children: [_jsx(Text, { children: "1 With the AI skill \u00B7 install if missing, then copy the prompt" }), _jsx(Text, { children: "2 Manual \u00B7 write a recipe using the schema" }), _jsx(Text, { children: "n or Esc closes" })] })) : null, mode.kind === "manual" ? (_jsxs(BorderPanel, { title: "Write an App recipe", children: [_jsxs(Text, { children: ["Write a JSON recipe in ", apps.directory] }), _jsx(Text, { children: "Schema: agent-depot app schema \u00B7 n or Esc closes" })] })) : null, mode.kind === "prompt" ? (_jsxs(BorderPanel, { title: "App recipe authoring prompt", children: [_jsx(Text, { children: "Use the agent-depot-apprecipe skill to create the recipe for <app>" }), _jsxs(Text, { children: ["Recipes: ", apps.directory, " \u00B7 n or Esc closes"] })] })) : null, mode.kind === "install" ? _jsx(ActionPanel, { mode: mode.action }) : null, mode.kind === "busy" ? _jsx(BorderPanel, { title: "App recipe workflow", children: _jsx(Working, { label: mode.label, since: mode.startedAt }) }) : null, mode.kind === "approve" ? (() => {
                 const chrome = recipeApprovalChrome(mode.entry);
                 const viewport = previewViewport(mode.lines, mode.cursor, terminalRows, terminalColumns, chrome);
                 return _jsx(PreviewConfirm, { title: chrome.title, lines: mode.lines, cursor: mode.cursor, question: viewport.reachedEnd ? chrome.questions[0] : chrome.questions[1], chrome: chrome });
@@ -306,5 +318,5 @@ export function useSourcesRecipes({ operations, environment, onCapturingChange,
                 const viewport = previewViewport(lines, mode.cursor, terminalRows, terminalColumns, chrome);
                 return _jsx(PreviewConfirm, { title: chrome.title, lines: lines, cursor: mode.cursor, question: viewport.reachedEnd ? chrome.questions[0] : chrome.questions[1], chrome: chrome });
             })() : null, mode.kind === "remove" ? (_jsx(PreviewConfirm, { title: "Recipe removal", lines: [`Delete recipe: ${mode.removal.file}`, "Only this recipe file or symlink will be removed; App data and Host configuration are not deleted."], question: `Remove recipe for ${appName(mode.removal.entry)}? y/n` })) : null] }));
-    return { rows, handleKey, startAdd, reload, clearMessage: () => { setError(undefined); setNotice(undefined); }, panel, directory: apps.directory, capturing: mode.kind !== "browse" };
+    return { rows, loadingSince, handleKey, startAdd, reload, clearMessage: () => { setError(undefined); setNotice(undefined); }, panel, directory: apps.directory, capturing: mode.kind !== "browse" };
 }

package/dist/src/tui/sources-view.js

@@ -6,7 +6,7 @@ import { rowStyle, theme } from "./theme.js";
 import { computeWindow, pageStep, useListHeight } from "./window.js";
 import { useKeys } from "./keys.js";
 import { useSourcesRecipes } from "./sources-recipes.js";
-import { BorderPanel, DecisionPrompt, FieldLine, PreviewLines } from "./panel-parts.js";
+import { BorderPanel, DecisionPrompt, FieldLine, PreviewLines, Working } from "./panel-parts.js";
 import { previewViewport } from "./preview-window.js";
 import { movePreviewCursor } from "./mode-keys.js";
 /** Footer keys for the selected row; App seeds the footer before the view reports a selection. */
@@ -53,7 +53,7 @@ function refreshViewport(mode, rows, columns) {
 export function SourcesView({ operations, onCapturingChange, onOpenCatalog, listHeight, environment, onHintsChange }) {
     const { columns, rows: terminalRows } = useWindowSize();
     const height = useListHeight(listHeight, 9);
-    const [state, setState] = useState({ status: "loading" });
+    const [state, setState] = useState(() => ({ status: "loading", startedAt: Date.now() }));
     // The highlight follows the source id; the index is only the fallback once that source is gone.
     // Selection and marks are mirrored in refs updated first, so keys delivered in one burst act on the latest state.
     const [selection, setSelectionState] = useState({ index: 0 });
@@ -147,8 +147,8 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
         setSelection({ id: listed[bounded]?.id, index: bounded });
     };
     /** Runs a mutating action in the busy mode, reports its outcome and reloads the list. */
-    const run = async (action) => {
-        dispatch({ type: "busy" });
+    const run = async (label, action) => {
+        dispatch({ type: "busy", label, startedAt: Date.now() });
         setMessage(undefined);
         let outcome;
         try {
@@ -180,7 +180,7 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
             stop("Removing sources is not supported by the configured operations");
             return;
         }
-        dispatch({ type: "busy" });
+        dispatch({ type: "busy", label: "Checking dependent Skills…", startedAt: Date.now() });
         setMessage(undefined);
         try {
             const installations = await operations.listUserGlobalInstallations();
@@ -226,7 +226,7 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
                 const url = typed.current.trim();
                 if (url === "")
                     return;
-                void run(async () => `Added Git Source: ${(await operations.addGitSource(url)).id}`);
+                void run("Adding Git Source…", async () => `Added Git Source: ${(await operations.addGitSource(url)).id}`);
             }
             else if (key.backspace || key.delete) {
                 typed.current = typed.current.slice(0, -1);
@@ -252,7 +252,7 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
             if (input === "y") {
                 if (mode.action === "refresh") {
                     const { sources: targets } = mode;
-                    void run(async () => {
+                    void run(targets.length === 1 ? "Refreshing Git Source…" : "Refreshing Git Sources…", async () => {
                         const result = await refreshEach(operations, targets);
                         setMarked(new Set());
                         return result;
@@ -260,7 +260,7 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
                 }
                 else {
                     const { source } = mode;
-                    void run(async () => {
+                    void run("Removing Git Source…", async () => {
                         await operations.removeGitSource?.(source.id);
                         return `Removed Git Source: ${source.id}`;
                     });
@@ -324,7 +324,7 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
     // Sources removed since they were marked no longer count.
     const selectedCount = sources.filter((source) => marked.has(source.id)).length;
     if (state.status === "loading")
-        return _jsx(BorderPanel, { title: "Sources", children: _jsx(FieldLine, { label: "Status", value: "Loading sources..." }) });
+        return _jsx(BorderPanel, { title: "Sources", children: _jsx(Working, { label: "Loading sources\u2026", since: state.startedAt }) });
     if (state.status === "error")
         return _jsx(BorderPanel, { title: "Source error", children: _jsx(Text, { color: theme.error, children: state.message }) });
     const window = computeWindow(rows.length, selected, height);
@@ -332,13 +332,14 @@ export function SourcesView({ operations, onCapturingChange, onOpenCatalog, list
         return _jsx(Prompt, { mode: mode, terminalRows: terminalRows || 24, terminalColumns: columns || 80 });
     if (recipes.capturing)
         return _jsx(Box, { flexDirection: "column", children: recipes.panel });
-    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Text, { color: theme.muted, children: ["      ", "ID  KIND  LOCATION / STATUS"] }), builtin ? _jsxs(Text, { children: ["    ", _jsx(Text, { color: theme.value, children: builtin.id }), _jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: theme.field, children: "skills" }), _jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: theme.warning, children: "included" })] }) : null, rows.length === 0 ? _jsx(Text, { children: "No sources or App recipes \u00B7 n to add" }) : null, rows.slice(window.start, window.end).map((row, offset) => {
+    return (_jsxs(Box, { flexDirection: "column", children: [_jsxs(Text, { color: theme.muted, children: ["      ", "ID  KIND  LOCATION / STATUS"] }), builtin ? _jsxs(Text, { children: ["    ", _jsx(Text, { color: theme.value, children: builtin.id }), _jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: theme.field, children: "skills" }), _jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: theme.warning, children: "included" })] }) : null, rows.length === 0 && recipes.loadingSince === undefined ? _jsx(Text, { children: "No sources or App recipes \u00B7 n to add" }) : null, rows.slice(window.start, window.end).map((row, offset) => {
                 const index = window.start + offset;
                 const id = row.kind === "skills" ? row.source.id : row.recipe.entry.recipe?.name ?? row.recipe.entry.file.split(/[\\/]/u).pop();
                 return (_jsxs(Text, { ...rowStyle(index === selected), children: [index === selected ? "> " : "  ", row.kind === "skills" ? (marked.has(row.id) ? "[x] " : "[ ] ") : "    ", _jsx(Text, { color: theme.value, children: id }), _jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: theme.field, children: row.kind }), _jsx(Text, { color: theme.legend, children: "  " }), row.kind === "skills"
                             ? _jsx(Text, { color: theme.value, children: row.source.url })
                             : _jsxs(_Fragment, { children: [_jsx(Text, { color: theme.value, children: row.recipe.entry.file }), _jsx(Text, { color: theme.legend, children: " \u00B7 " }), _jsx(Text, { color: row.recipe.status === "approved" ? theme.success : theme.warning, children: row.recipe.status })] })] }, row.id));
-            }), window.indicator ? _jsx(Text, { color: theme.muted, children: window.indicator }) : null, selectedCount > 0 ? _jsxs(Text, { children: [selectedCount, " selected"] }) : null, recipes.panel, _jsx(Prompt, { mode: mode }), message ? (_jsx(BorderPanel, { title: message.kind === "error" ? "Source action · failed" : "Source action · complete", children: _jsx(Text, { color: message.kind === "error" ? theme.error : theme.success, children: message.text }) })) : null] }));
+            }), window.indicator ? _jsx(Text, { color: theme.muted, children: window.indicator }) : null, selectedCount > 0 ? _jsxs(Text, { children: [selectedCount, " selected"] }) : null, recipes.loadingSince !== undefined && mode.kind !== "busy" && !recipes.capturing
+                ? _jsx(Working, { label: "Loading App recipes\u2026", since: recipes.loadingSince }) : null, recipes.panel, _jsx(Prompt, { mode: mode }), message ? (_jsx(BorderPanel, { title: message.kind === "error" ? "Source action · failed" : "Source action · complete", children: _jsx(Text, { color: message.kind === "error" ? theme.error : theme.success, children: message.text }) })) : null] }));
 }
 function Prompt({ mode, terminalRows = 24, terminalColumns = 80 }) {
     if (mode.kind === "add-kind")
@@ -346,7 +347,7 @@ function Prompt({ mode, terminalRows = 24, terminalColumns = 80 }) {
     if (mode.kind === "input")
         return _jsxs(Text, { children: ["URL: ", mode.value] });
     if (mode.kind === "busy")
-        return _jsx(BorderPanel, { title: "Sources", children: _jsx(FieldLine, { label: "Status", value: "Working..." }) });
+        return _jsx(BorderPanel, { title: "Sources", children: _jsx(Working, { label: mode.label, since: mode.startedAt }) });
     if (mode.kind === "confirm") {
         return mode.action === "refresh" ? (() => {
             const viewport = refreshViewport(mode, terminalRows, terminalColumns);

package/dist/src/tui/text-wrap.js

@@ -36,6 +36,19 @@ export function fitLines(lines, maxRows, width) {
     visible[maxRows - 1] = `${last}…`;
     return visible;
 }
+/**
+ * `lines` capped to `maxRows` by dropping the middle: the first and last rows stay (the start and end of
+ * a path) with a row counting what was left out. With two rows or fewer it falls back to `fitLines`.
+ */
+export function elideMiddle(lines, maxRows, width) {
+    if (lines.length <= maxRows)
+        return lines;
+    if (maxRows <= 2)
+        return fitLines(lines, maxRows, width);
+    const tail = lines.slice(lines.length - (maxRows - 2));
+    const hidden = lines.length - 1 - tail.length;
+    return [lines[0], `… ${hidden} more ${hidden === 1 ? "row" : "rows"}`, ...tail];
+}
 /** One line shortened to `width` columns plus an ellipsis, for single-row labels. */
 export function clipText(value, width) {
     if (stringWidth(value) <= width)

package/dist/src/tui/unmanaged-actions.js

@@ -4,6 +4,7 @@ import { describeUnmanagedRemoval, describeUnmanagedSymlinkRemoval, removeInspec
 import { inspectUserGlobalSkillRemoval, inspectUserGlobalSymlinkRemoval, } from "../user-global-skill-inventory.js";
 import { answerYesNo, moveCursor } from "./mode-keys.js";
 import { homeOf } from "./installations.js";
+import { busy } from "./working-model.js";
 export const UNMANAGED_KINDS = ["unmanaged-scope", "unmanaged-pick", "confirm-unmanaged"];
 const errorText = (error) => (error instanceof Error ? error.message : String(error));
 /** Links first, so a Claude link never outlives the canonical directory it points to. */
@@ -99,7 +100,7 @@ export async function runUnmanagedRemoval(prepared, operations, environment) {
     return { kind: failed ? "error" : "ok", text: lines.join("\n") };
 }
 async function startPreview(context, group, locations) {
-    context.setMode({ kind: "busy", label: `Preparing removal of ${group.name}...` });
+    context.setMode(busy(`Preparing removal of ${group.name}…`));
     try {
         const prepared = await prepareUnmanagedRemoval(context.operations, context.env, group, locations);
         if (!context.isMounted())
@@ -125,7 +126,7 @@ function cancel(context) {
     context.browse();
 }
 async function execute(context, group, prepared) {
-    context.setMode({ kind: "busy", label: `Removing ${group.name}...` });
+    context.setMode(busy(`Removing ${group.name}…`));
     await context.finish(await runUnmanagedRemoval(prepared, context.operations, context.env));
 }
 /** Handles a key in an unmanaged removal mode; returns false for any other mode. */

package/dist/src/tui/updates-flow.js

@@ -2,10 +2,11 @@ import { useCallback, useEffect, useRef, useState } from "react";
 import { errorText } from "./batch.js";
 import { BROWSE } from "./updates-mode.js";
 import { CHECKING, emptyData, PHASE_LABEL, REFRESHING } from "./updates-model.js";
+import { busy } from "./working-model.js";
 import { loadUpdateRows, prepareUpdates, runUpdates } from "./updates.js";
 export function useUpdatesFlow(inputs) {
-    const { operations, env, clock, mounted, checkedRef, latest, setChecked, setCursor, setMode, setMessage } = inputs;
-    const [state, setState] = useState({ status: "loading", label: REFRESHING });
+    const { operations, env, clock, mounted, checkedRef, latest, setChecked, setCursor, setMode, modeRef, setMessage } = inputs;
+    const [state, setState] = useState(() => ({ status: "loading", label: REFRESHING, startedAt: clock() }));
     const loadToken = useRef(0);
     const pendingAnswer = useRef(undefined);
     useEffect(() => () => {
@@ -13,14 +14,16 @@ export function useUpdatesFlow(inputs) {
     }, []);
     const load = useCallback(async (refresh) => {
         const token = ++loadToken.current;
+        // The check after an applied update continues that update's clock; a check from browsing starts one.
+        const running = busy(refresh ? REFRESHING : CHECKING, modeRef.current, clock);
         const phase = (next) => {
             if (!mounted.current || token !== loadToken.current)
                 return;
-            setState({ status: "loading", label: PHASE_LABEL[next] });
+            setState({ status: "loading", label: PHASE_LABEL[next], startedAt: running.startedAt });
         };
         setChecked([]);
         setCursor(0);
-        setMode({ kind: "busy", label: refresh ? REFRESHING : CHECKING });
+        setMode(running);
         let next;
         try {
             next = { status: "ready", data: await loadUpdateRows(operations, env, { refresh, onPhase: phase, now: clock }) };
@@ -34,7 +37,7 @@ export function useUpdatesFlow(inputs) {
             return;
         setState(next);
         setMode(BROWSE);
-    }, [operations, env, mounted, setMode, setChecked, setCursor, clock]);
+    }, [operations, env, mounted, setMode, modeRef, setChecked, setCursor, clock]);
     const startPreview = async () => {
         const { data: current, updatable: listed } = latest();
         if (!current)
@@ -45,7 +48,7 @@ export function useUpdatesFlow(inputs) {
             return;
         }
         setMessage(undefined);
-        setMode({ kind: "busy", label: "Preparing preview..." });
+        setMode(busy("Preparing preview…", undefined, clock));
         try {
             const prepared = await prepareUpdates(current, selected);
             if (mounted.current)
@@ -59,7 +62,7 @@ export function useUpdatesFlow(inputs) {
         }
     };
     const apply = async (prepared, confirmedPaths) => {
-        setMode({ kind: "busy", label: "Applying updates..." });
+        setMode(busy("Applying updates…", undefined, clock));
         let result;
         try {
             const outcome = await runUpdates(prepared, confirmedPaths, (plan, reason) => new Promise(resolve => {

package/dist/src/tui/updates-keys.js

@@ -1,4 +1,5 @@
 import { answerYesNo, movePreviewCursor } from "./mode-keys.js";
+import { busy } from "./working-model.js";
 import { pageStep } from "./window.js";
 const SCOPE_KEYS = new Map([["t", "all"], ["p", "project"], ["g", "user-global"]]);
 /** Moves the cursor with the arrows, j/k and paging; returns whether the key was one of them. */
@@ -100,10 +101,10 @@ export function handleUpdatesKey(context, mode, input, key) {
             answerYesNo(input, key, () => {
                 if (!context.previewAtEnd(mode))
                     return;
-                context.setMode({ kind: "busy", label: "Checking App version..." });
+                context.setMode(busy("Checking App version…", undefined, context.now));
                 mode.answer(true);
             }, () => {
-                context.setMode({ kind: "busy", label: "Continuing updates..." });
+                context.setMode(busy("Continuing updates…", undefined, context.now));
                 mode.answer(false);
             });
             return;

package/dist/src/tui/updates-model.js

@@ -1,5 +1,5 @@
-export const REFRESHING = "Refreshing sources...";
-export const CHECKING = "Checking for updates...";
+export const REFRESHING = "Refreshing sources…";
+export const CHECKING = "Checking for updates…";
 export const PHASE_LABEL = { refreshing: REFRESHING, checking: CHECKING };
 /** Names the reasons worth knowing before expanding the collapsed "cannot be checked" row. */
 export function describeUnknownReasons(rows) {

package/dist/src/tui/updates-panel.js

@@ -3,7 +3,7 @@ import { Box, Text, useWindowSize } from "ink";
 import { rowStyle, theme } from "./theme.js";
 import { BorderPanel, FieldLine, PreviewConfirm } from "./panel-parts.js";
 import { previewViewport } from "./preview-window.js";
-import { CHECKING, describeRefresh, describeUnknownReasons, REFRESHING } from "./updates-model.js";
+import { describeRefresh, describeUnknownReasons } from "./updates-model.js";
 import { formatAgo, previewLineText } from "./updates.js";
 import { updateModePreviewChrome, updateModePreviewLines, updateModeQuestion } from "./updates-preview.js";
 function updateResultField(line, defaultOk) {
@@ -61,7 +61,8 @@ export function ModePanel({ mode }) {
         case "browse":
             return null;
         case "busy":
-            return mode.label === REFRESHING || mode.label === CHECKING ? null : (_jsx(BorderPanel, { title: "Updates", children: _jsx(FieldLine, { label: "Status", value: mode.label }) }));
+            // The view renders the one indicator of a busy mode, so a check never shows two.
+            return null;
         case "preview":
         case "confirm-paths":
             return _jsx(CapturedUpdatePreview, { mode: mode });

package/dist/src/tui/updates-view.js

@@ -13,7 +13,7 @@ import { CheckStatus, EntryLine, ModePanel, UpdatesFooter } from "./updates-pane
 import { previewLineText } from "./updates.js";
 import { updateModePreviewChrome, updateModePreviewLines } from "./updates-preview.js";
 import { useKeys } from "./keys.js";
-import { BorderPanel, FieldLine } from "./panel-parts.js";
+import { BorderPanel, Working } from "./panel-parts.js";
 const NO_ENVIRONMENT = {};
 /** How often the age of the check is redrawn. */
 const CLOCK_TICK_MS = 10_000;
@@ -69,7 +69,7 @@ export function UpdatesView({ operations, environment, onCapturingChange, listHe
         setMode(BROWSE);
     };
     const { state, load, startPreview, apply } = useUpdatesFlow({
-        operations, env, clock, mounted, checkedRef, latest, setChecked, setCursor, setMode, setMessage,
+        operations, env, clock, mounted, checkedRef, latest, setChecked, setCursor, setMode, modeRef, setMessage,
     });
     // Entering the view refreshes the Git sources first, like `r`.
     useEffect(() => {
@@ -85,6 +85,7 @@ export function UpdatesView({ operations, environment, onCapturingChange, listHe
     };
     const keyContext = {
         height,
+        now: clock,
         previewMetrics: current => {
             const geometry = previewGeometry(current);
             return { length: geometry.total, pageSize: geometry.height };
@@ -112,5 +113,7 @@ export function UpdatesView({ operations, environment, onCapturingChange, listHe
     const view = { shown, updatable, unknown, upToDate, entries };
     const hasErrors = data?.appError !== undefined || (data?.results ?? []).some((result) => result.error !== undefined);
     const showingBrowse = mode.kind === "browse";
-    return (_jsxs(Box, { flexDirection: "column", children: [showingBrowse ? _jsxs(Text, { color: theme.title, bold: true, children: ["Updates (scope: ", scope, ")  t all \u00B7 p project \u00B7 g user-global \u00B7 u update"] }) : null, state.status === "loading" ? _jsx(BorderPanel, { title: "Update check", children: _jsx(FieldLine, { label: "Status", value: state.label }) }) : null, showingBrowse && data ? _jsx(CheckStatus, { data: data, now: clock() }) : null, showingBrowse && data && !hasErrors && shown.length === 0 ? _jsx(Text, { children: "No installations" }) : null, showingBrowse ? entries.slice(window.start, window.end).map((entry, offset) => (_jsx(EntryLine, { entry: entry, selected: window.start + offset === index, checked: entry.kind === "update" && checked.includes(entry.row.key), view: view, showUnknown: showUnknown }, entry.kind === "summary" ? "summary" : entry.row.key))) : null, showingBrowse ? _jsx(UpdatesFooter, { indicator: window.indicator, upToDate: upToDate, updatableCount: updatable.length, checkedCount: checked.length, message: message }) : null, _jsx(ModePanel, { mode: mode })] }));
+    // One indicator for the whole operation: a check's own phase label, else the busy mode's.
+    const working = state.status === "loading" ? state : mode.kind === "busy" ? mode : undefined;
+    return (_jsxs(Box, { flexDirection: "column", children: [showingBrowse ? _jsxs(Text, { color: theme.title, bold: true, children: ["Updates (scope: ", scope, ")  t all \u00B7 p project \u00B7 g user-global \u00B7 u update"] }) : null, working ? _jsx(BorderPanel, { title: state.status === "loading" ? "Update check" : "Updates", children: _jsx(Working, { label: working.label, since: working.startedAt, now: clock }) }) : null, showingBrowse && data ? _jsx(CheckStatus, { data: data, now: clock() }) : null, showingBrowse && data && !hasErrors && shown.length === 0 ? _jsx(Text, { children: "No installations" }) : null, showingBrowse ? entries.slice(window.start, window.end).map((entry, offset) => (_jsx(EntryLine, { entry: entry, selected: window.start + offset === index, checked: entry.kind === "update" && checked.includes(entry.row.key), view: view, showUnknown: showUnknown }, entry.kind === "summary" ? "summary" : entry.row.key))) : null, showingBrowse ? _jsx(UpdatesFooter, { indicator: window.indicator, upToDate: upToDate, updatableCount: updatable.length, checkedCount: checked.length, message: message }) : null, _jsx(ModePanel, { mode: mode })] }));
 }

package/dist/src/tui/view-state.js

@@ -10,6 +10,18 @@ export function useMounted() {
     }, []);
     return mounted;
 }
+/**
+ * When the current stretch of loading began, so a loading line shows how long it has waited even
+ * though the view re-renders. `seed` continues an operation that was already running (its busy start).
+ */
+export function useLoadingSince(loading, seed) {
+    const [since, setSince] = useState(() => (loading ? seed ?? Date.now() : undefined));
+    useEffect(() => {
+        setSince((previous) => (loading ? previous ?? seed ?? Date.now() : undefined));
+    }, [loading, seed]);
+    // Until the effect records it, the first loading render derives the same start.
+    return since ?? seed ?? Date.now();
+}
 /** Marks by node id. The ref is updated first so keys in one burst see every mark. */
 export function useMarks() {
     const [marked, setMarkedState] = useState(new Set());

package/dist/src/tui/working-model.js

@@ -0,0 +1,33 @@
+/** Braille spinner frames; one is shown per tick while an operation runs. */
+export const SPINNER_FRAMES = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
+/** Fixed glyph shown instead of the spinner when animation is off. */
+export const STATIC_GLYPH = "·";
+/** How often the spinner advances and the elapsed time is redrawn. */
+export const SPINNER_TICK_MS = 80;
+/** Elapsed time stays hidden until the operation has run this long. */
+const ELAPSED_THRESHOLD_MS = 1000;
+/** `12s` or `1m 05s`; empty under one second. */
+export function formatElapsed(ms) {
+    if (!(ms >= ELAPSED_THRESHOLD_MS))
+        return "";
+    const seconds = Math.floor(ms / 1000);
+    if (seconds < 60)
+        return `${seconds}s`;
+    return `${Math.floor(seconds / 60)}m ${String(seconds % 60).padStart(2, "0")}s`;
+}
+/** Animation is on unless `AGENT_DEPOT_NO_ANIMATION=1`, which tests set so frames stay static. */
+export function animationEnabled(env = process.env) {
+    return env.AGENT_DEPOT_NO_ANIMATION !== "1";
+}
+/** Whether a rendered frame starts with a Working line, so tests can wait for the real content. */
+export function isWorkingFrame(text) {
+    return /^[⠋⠙⠹⠸⠼⠴⠦⠧⠇⠏·] \S[^\n]*…/u.test(text.trimStart());
+}
+/** When an operation began: the start time of the busy mode it continues, otherwise now. Only busy modes carry one. */
+export function operationStart(previous, now = Date.now) {
+    return previous?.startedAt ?? now();
+}
+/** The busy mode for `label`; it keeps the start time of an operation already busy, so the clock survives label changes. */
+export function busy(label, previous, now = Date.now) {
+    return { kind: "busy", label, startedAt: operationStart(previous, now) };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iseoane/agent-depot",
-  "version": "0.6.4",
+  "version": "0.6.6",
   "description": "A cross-platform catalog for reusable coding-agent skills.",
   "license": "MIT",
   "author": "iseoane",
@@ -74,7 +74,7 @@
     "typecheck": "tsc --noEmit",
     "lint": "eslint \"src/**/*.{ts,tsx}\" \"resources/pi/**/*.{ts,tsx}\" \"tests/**/*.{ts,tsx}\"",
     "test": "pnpm build && node --test --import ./dist/tests/isolated-env.js \"dist/tests/*.test.js\"",
-    "test:single": "pnpm build && node -e \"const { spawnSync } = require('node:child_process'); const input = process.argv[1] || 'tests/version.test.ts'; const file = input.replace(/^tests[\\\\/]/, 'dist/tests/').replace(/\\\\.ts$/i, '.js'); process.exit(spawnSync(process.execPath, ['--import', './dist/tests/isolated-env.js', '--test', file], { stdio: 'inherit' }).status ?? 1);\"",
+    "test:single": "pnpm build && node -e \"const { spawnSync } = require('node:child_process'); const input = process.argv[1] || 'tests/version.test.ts'; const file = input.replace(/^tests[\\\\/]/, 'dist/tests/').replace(/\\\\.tsx?$/i, '.js'); process.exit(spawnSync(process.execPath, ['--import', './dist/tests/isolated-env.js', '--test', file], { stdio: 'inherit' }).status ?? 1);\"",
     "test:coverage": "node -e \"require('node:fs').rmSync('coverage',{recursive:true,force:true})\" && tsc -p tsconfig.coverage.json && node -e \"const { spawnSync } = require('node:child_process'); process.exit(spawnSync(process.execPath, ['--enable-source-maps','--import','./dist/tests/isolated-env.js','--test','--test-reporter=dot','dist/tests/*.test.js'], { stdio: 'inherit', env: { ...process.env, NODE_V8_COVERAGE: 'coverage/v8' } }).status ?? 1);\"",
     "audit:dead-code": "npx --yes fallow@3.30.0 dead-code --fail-on-issues",
     "audit:dupes": "npx --yes fallow@3.30.0 dupes",