@iseoane/agent-depot 0.6.8 → 0.6.10

package/dist/src/tui/batch.js

@@ -3,9 +3,10 @@ export const errorText = (error) => (error instanceof Error ? error.message : St
  * Applies the items one after another. A failure is reported for that item and
  * the rest still run, so the outcomes always cover every item in order.
  */
-export async function runBatch(items, run, describeFailure) {
+export async function runBatch(items, run, describeFailure, onStart) {
     const outcomes = [];
     for (const item of items) {
+        onStart?.(item, outcomes.length + 1, items.length, outcomes);
         try {
             outcomes.push({ item, ok: true, line: await run(item) });
         }

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

@@ -78,8 +78,16 @@ function resultOf(outcomes, skipped) {
         applied: outcomes.filter((outcome) => outcome.ok).map((outcome) => outcome.item.id),
     };
 }
+/** `Removing name (2/5)…` for each item as it starts. */
+function startOf(verb, progress) {
+    if (!progress)
+        return undefined;
+    return (item, position, total, finished) => {
+        progress(`${verb} ${item.label} (${position}/${total})…`, finished.map(outcome => outcome.line));
+    };
+}
 /** Execute step: each item is rechecked and applied on its own against the records the previous items were expected to leave. */
-export async function runBulkUninstall(plan, operations, env) {
+export async function runBulkUninstall(plan, operations, env, progress) {
     let expected = plan.installations;
     const outcomes = await runBatch(plan.items, async (item) => {
         await assertNoDrift(operations, expected);
@@ -95,7 +103,7 @@ export async function runBulkUninstall(plan, operations, env) {
         return item.prepared.plan.inspections.every((inspection) => inspection.recordOnly)
             ? `Forgot missing installation record for ${item.label}; no files were removed`
             : `Removed ${item.label}`;
-    }, (item, error) => (item.kind === "unmanaged" ? errorText(error) : `Failed ${item.label}: ${errorText(error)}`));
+    }, (item, error) => (item.kind === "unmanaged" ? errorText(error) : `Failed ${item.label}: ${errorText(error)}`), startOf("Removing", progress));
     return resultOf(outcomes, plan.skipped);
 }
 /** Managed user-global leaves that can gain hosts, and the leaves that cannot with their reasons. */
@@ -146,7 +154,7 @@ export async function prepareBulkHostAddition(context, targets, alreadySkipped,
     return { items, skipped, installations };
 }
 /** Execute step: the single exposure confirmation covers every item; each one is rechecked on its own. */
-export async function runBulkHostAddition(plan, operations, env) {
+export async function runBulkHostAddition(plan, operations, env, progress) {
     let expected = plan.installations;
     const outcomes = await runBatch(plan.items, async (item) => {
         await assertNoDrift(operations, expected);
@@ -155,7 +163,7 @@ export async function runBulkHostAddition(plan, operations, env) {
         const { selection, updated } = item.prepared.plan;
         expected = expected.map((record) => (sameRecord(record, selection) ? updated : record));
         return line;
-    }, (item, error) => `Failed ${item.label}: ${errorText(error)}`);
+    }, (item, error) => `Failed ${item.label}: ${errorText(error)}`, startOf("Changing hosts of", progress));
     return resultOf(outcomes, plan.skipped);
 }
 function cancel(context, text) {
@@ -202,10 +210,15 @@ function prepareHosts(context, mode) {
     return preparePlan(context, `Preparing host change of ${count(mode.targets.length, "item")}…`, () => prepareBulkHostAddition(context, mode.targets, mode.skipped, mode.selected), (plan) => ({ kind: "confirm-bulk-hosts", plan }));
 }
 async function apply(context, label, action) {
-    context.setMode(busy(label));
+    const started = busy(label);
+    context.setMode(started);
     let result;
     try {
-        result = await action();
+        // Each item names itself, but the clock of the whole batch keeps running.
+        result = await action((itemLabel, done) => {
+            if (context.isMounted())
+                context.setMode({ ...busy(itemLabel, started), done });
+        });
     }
     catch (error) {
         result = { message: { kind: "error", text: errorText(error) }, applied: [] };
@@ -229,7 +242,7 @@ export function handleBulkKey(context, mode, input, key) {
         }
         case "confirm-bulk-uninstall":
             answerYesNo(input, key, () => {
-                void apply(context, `Removing ${count(current.plan.items.length, "item")}…`, () => runBulkUninstall(current.plan, operations, env));
+                void apply(context, `Removing ${count(current.plan.items.length, "item")}…`, (progress) => runBulkUninstall(current.plan, operations, env, progress));
             }, () => cancel(context, "Uninstall cancelled"));
             return true;
         case "confirm-bulk-hosts":
@@ -237,7 +250,7 @@ export function handleBulkKey(context, mode, input, key) {
             return true;
         case "confirm-bulk-exposure":
             answerYesNo(input, key, () => {
-                void apply(context, `Changing hosts of ${count(current.plan.items.length, "item")}…`, () => runBulkHostAddition(current.plan, operations, env));
+                void apply(context, `Changing hosts of ${count(current.plan.items.length, "item")}…`, (progress) => runBulkHostAddition(current.plan, operations, env, progress));
             }, () => cancel(context, "Add hosts cancelled"));
             return true;
         default:

package/dist/src/tui/catalog-installs.js

@@ -83,17 +83,18 @@ export async function prepareInstall(operations, environment, skill, choice) {
             root: path.resolve(environment.projectRoot ?? process.cwd()),
             existing: (await manifestStore.load()).skills,
             recordedIn: manifestStore.path,
+            manifestPath: manifestStore.path,
         });
     }
     const lines = [];
     outputSingleInstallPreview(plan, (line) => lines.push(line));
     // The TUI confirms interactively, so the CLI's --yes wording does not apply.
-    const preview = lines.filter((line) => !line.startsWith("  confirmation:"));
+    const preview = lines.filter((line) => !plan.projectTarget.includes(line) && !line.startsWith("  confirmation:"));
     const additionalHostExposure = needsAdditionalHostExposure(plan.inspection, choice.hosts);
     if (additionalHostExposure) {
         preview.push("  ADDITIONAL HOST EXPOSURE: an identical Skill already exists; installing adds the missing Host location (equivalent to --confirm-additional-host) and needs a separate confirmation");
     }
-    return { plan, manifestStore, preview, runsExternalCommand: plan.resolved.method !== undefined, additionalHostExposure };
+    return { plan, manifestStore, projectTarget: plan.projectTarget, preview, runsExternalCommand: plan.resolved.method !== undefined, additionalHostExposure };
 }
 /** Execute step: installs and persists the record; returns the result lines. */
 export async function runInstall(prepared, operations, confirmAdditionalHostExposure = false) {

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

@@ -19,7 +19,7 @@ export function CatalogRowLine({ row, selected, marked, installed }) {
     const isGroup = data.kind === "source" || data.kind === "group";
     const ownedBy = data.kind === "skill" ? data.ownedBy : undefined;
     return (_jsxs(Text, { ...rowStyle(selected), color: isGroup ? theme.group : undefined, wrap: "truncate-end", children: [selected ? "> " : "  ", "  ".repeat(row.depth), data.kind === "skill" ? marked ? "[x] " : "[ ] " : "", data.kind === "skill" && ownedBy !== undefined
-                ? _jsxs(_Fragment, { children: [data.skill.name, _jsxs(Text, { color: theme.marker, children: ["  ", "[", packageOwnerBadge(ownedBy), "]"] }), "  ", truncateDescription(data.skill.description)] })
+                ? _jsxs(_Fragment, { children: [data.skill.name, _jsxs(Text, { color: theme.marker, children: ["  ", "[", packageOwnerBadge(ownedBy), "]"] }), "  ", truncateDescription(data.skill.description), data.variant === true ? `  (${data.skill.path})` : ""] })
                 : data.kind === "app" ? `App ${data.row.entry.recipe?.name ?? data.row.entry.file}` : describeRow(row), data.kind === "app" ? (_jsxs(_Fragment, { children: [_jsx(Text, { color: theme.legend, children: "  " }), _jsx(Text, { color: data.row.inspection.status === "not installed" ? theme.warning : theme.value, children: data.row.inspection.status })] })) : null, data.kind === "skill" && hasUnmanagedCopy(data.skill, installed)
                 ? _jsxs(Text, { color: theme.marker, children: ["  ", UNMANAGED_MARKER] })
                 : null] }));
@@ -47,7 +47,8 @@ export function ActionPanel({ mode }) {
         case "confirm-install": {
             const ready = mode.items.filter((item) => item.prepared !== undefined);
             const single = mode.items.length === 1;
-            return (_jsxs(BorderPanel, { title: `Skill install preview · ${ready.length} ready`, children: [mode.items.map(({ skill, prepared, error }) => prepared === undefined ? (_jsxs(Text, { color: theme.error, children: ["Cannot install ", skill.name, ": ", error] }, skillKey(skill))) : (_jsxs(Box, { flexDirection: "column", children: [single ? null : _jsx(Text, { color: theme.group, children: skill.name }), _jsx(PreviewLines, { lines: prepared.preview })] }, skillKey(skill)))), ready.some((item) => item.prepared?.runsExternalCommand) ? _jsx(Text, { color: theme.warning, children: "This install runs an external command (see above)." }) : null, _jsxs(DecisionPrompt, { children: ["Install ", describeSkills(ready.map((item) => item.skill)), "?"] })] }));
+            const projectTarget = ready[0]?.prepared?.projectTarget ?? [];
+            return (_jsxs(BorderPanel, { title: `Skill install preview · ${ready.length} ready`, children: [projectTarget.length === 0 ? null : _jsx(PreviewLines, { lines: projectTarget }), mode.items.map(({ skill, prepared, error }) => prepared === undefined ? (_jsxs(Text, { color: theme.error, children: ["Cannot install ", skill.name, ": ", error] }, skillKey(skill))) : (_jsxs(Box, { flexDirection: "column", children: [single ? null : _jsx(Text, { color: theme.group, children: skill.name }), _jsx(PreviewLines, { lines: prepared.preview })] }, skillKey(skill)))), ready.some((item) => item.prepared?.runsExternalCommand) ? _jsx(Text, { color: theme.warning, children: "This install runs an external command (see above)." }) : null, _jsxs(DecisionPrompt, { children: ["Install ", describeSkills(ready.map((item) => item.skill)), "?"] })] }));
         }
         case "confirm-exposure": {
             const names = mode.items.filter((item) => item.prepared?.additionalHostExposure).map((item) => item.skill.name);

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

@@ -6,15 +6,22 @@ import { loadInstalledSkills, NO_INSTALLED } from "./catalog-installs.js";
 import { sourceLabel } from "./catalog-tree.js";
 import { packageOperationsOf } from "./environment.js";
 /** Discovers the Skills of one Source, or of all of them, and again whenever those inputs or `reloadKey` change. */
-export function useCatalogSkills(operations, sourceId, all, env, reloadKey = 0) {
-    const [state, setState] = useState({ status: "loading" });
+export function useCatalogSkills(operations, sourceId, all, env, reloadKey = 0, retention) {
+    const key = `${sourceId ?? ""}|${all}`;
+    /** The last result of this scope, shown while it is discovered again. */
+    const retained = () => {
+        const kept = retention?.catalogKey === key ? retention.catalogSkills : undefined;
+        return kept ? { ...kept, stale: true } : { status: "loading" };
+    };
+    const [state, setState] = useState(retained);
     useEffect(() => {
         let cancelled = false;
+        const generation = retention?.generation;
         if (!operations.discoverSkills) {
             setState({ status: "error", message: "Discovery is not supported by the configured operations" });
             return;
         }
-        setState({ status: "loading" });
+        setState(retained());
         void (async () => {
             try {
                 const sources = await operations.listSources();
@@ -50,6 +57,9 @@ export function useCatalogSkills(operations, sourceId, all, env, reloadKey = 0)
                     try {
                         const projectSource = projectSourceOf(source);
                         for (const descriptor of await packages.discover([id])) {
+                            // A Package Agent Depot cannot install is not a choice; its warning stays in `package inspect`.
+                            if (!descriptor.installable)
+                                continue;
                             bundles.push({ sourceId: id, selection: packageSelectionFor(descriptor, projectSource), descriptor });
                             discovered.push({ source: projectSource, descriptor });
                         }
@@ -73,8 +83,14 @@ export function useCatalogSkills(operations, sourceId, all, env, reloadKey = 0)
                     const detail = error instanceof Error ? error.message : String(error);
                     warnings.push(`Package ownership unavailable: ${detail}`);
                 }
+                const ready = { status: "ready", skills, bundles, activeBundles, warnings, sources };
+                // A result that began before an invalidation may predate the change, so it is not kept.
+                if (!cancelled && retention && retention.generation === generation) {
+                    retention.catalogKey = key;
+                    retention.catalogSkills = ready;
+                }
                 if (!cancelled)
-                    setState({ status: "ready", skills, bundles, activeBundles, warnings, sources });
+                    setState(ready);
             }
             catch (error) {
                 if (!cancelled)
@@ -87,49 +103,59 @@ export function useCatalogSkills(operations, sourceId, all, env, reloadKey = 0)
     }, [operations, sourceId, all, env, reloadKey]);
     return state;
 }
-export function useCatalogApps(apps) {
-    const [state, setState] = useState({ status: "loading" });
+export function useCatalogApps(apps, retention) {
+    const [state, setState] = useState(() => retention?.catalogApps ? { ...retention.catalogApps, stale: true } : { status: "loading" });
     const generation = useRef(0);
-    const reload = useCallback(async () => {
+    /** `keepShown` reloads behind the rows already on screen instead of replacing them with a loading state. */
+    const load = useCallback(async (keepShown) => {
         const current = ++generation.current;
-        setState({ status: "loading" });
+        const retained = retention?.generation;
+        if (!keepShown)
+            setState({ status: "loading" });
         try {
             const rows = await Promise.all((await apps.load()).filter(entry => entry.applicable).map(async (entry) => ({
                 entry,
                 inspection: await apps.inspect(entry),
             })));
-            if (generation.current === current)
-                setState({
-                    status: "ready",
-                    rows: rows.filter(row => row.inspection.status === "not installed"),
-                });
+            if (generation.current === current) {
+                const ready = { status: "ready", rows: rows.filter(row => row.inspection.status === "not installed") };
+                if (retention && retention.generation === retained)
+                    retention.catalogApps = ready;
+                setState(ready);
+            }
         }
         catch (error) {
             if (generation.current === current)
                 setState({ status: "ready", rows: [], error: error instanceof Error ? error.message : String(error) });
         }
-    }, [apps]);
+    }, [apps, retention]);
+    const reload = useCallback(() => load(false), [load]);
     useEffect(() => {
-        void reload();
+        void load(retention?.catalogApps !== undefined);
         return () => { generation.current += 1; };
-    }, [reload]);
+    }, [load, retention]);
     return { state, reload };
 }
 /** The Skills already installed, loaded on mount and on demand. */
-export function useInstalledSkills(operations, env, mounted) {
-    const [installed, setInstalled] = useState(NO_INSTALLED);
-    const [installedReady, setInstalledReady] = useState(false);
+export function useInstalledSkills(operations, env, mounted, retention) {
+    const [installed, setInstalled] = useState(retention?.installed ?? NO_INSTALLED);
+    const [installedReady, setInstalledReady] = useState(retention?.installed !== undefined);
+    const [installedStale, setInstalledStale] = useState(retention?.installed !== undefined);
     const reloadInstalled = useCallback(async () => {
+        const generation = retention?.generation;
         const loaded = await loadInstalledSkills(operations, env);
+        if (retention && retention.generation === generation)
+            retention.installed = loaded;
         if (!mounted.current)
             return;
         setInstalled(loaded);
         setInstalledReady(true);
-    }, [operations, env, mounted]);
+        setInstalledStale(false);
+    }, [operations, env, mounted, retention]);
     useEffect(() => {
         void reloadInstalled();
     }, [reloadInstalled]);
-    return { installed, installedReady, reloadInstalled };
+    return { installed, installedReady, installedStale, reloadInstalled };
 }
 /**
  * The filter box. Its state is mirrored in refs, updated first, so the shell

package/dist/src/tui/catalog-tree.js

@@ -1,12 +1,13 @@
 import { githubTreeLocation } from "../git-source.js";
-import { bundleRoot } from "../package-model.js";
+import { selectionKey } from "../package-model.js";
 import { packageOwnedSkillMatcher } from "../package-owned-skills.js";
 import { packageVersionLabel } from "./package-rows.js";
 import { defaultExpanded } from "./tree.js";
 const DESCRIPTION_LIMIT = 60;
 export const UNMANAGED_MARKER = "on disk, unmanaged · adopt it from Installations";
 export const skillKey = (skill) => `skill:${skill.sourceId}:${skill.path}`;
-export const bundleKey = (entry) => `bundle:${entry.sourceId}:${entry.descriptor.host}:${bundleRoot(entry.descriptor.coordinates)}`;
+/** Keyed on the full selection, so two plugins of one marketplace root stay distinct nodes. */
+export const bundleKey = (entry) => `bundle:${entry.sourceId}:${selectionKey(entry.selection)}`;
 export function truncateDescription(text) {
     return text.length > DESCRIPTION_LIMIT ? `${text.slice(0, DESCRIPTION_LIMIT - 3)}...` : text;
 }
@@ -30,11 +31,20 @@ export function buildTree(skills, sources = [], bundles = [], activeDescriptors
     for (const skill of skills)
         bySource.set(skill.sourceId, [...(bySource.get(skill.sourceId) ?? []), skill]);
     const bundlesBySource = new Map();
-    for (const bundle of bundles)
+    const seenBundles = new Set();
+    for (const bundle of bundles) {
+        // Descriptors with identical coordinates are one Package; a duplicate node id would corrupt the rows.
+        if (seenBundles.has(bundleKey(bundle)))
+            continue;
+        seenBundles.add(bundleKey(bundle));
         bundlesBySource.set(bundle.sourceId, [...(bundlesBySource.get(bundle.sourceId) ?? []), bundle]);
+    }
     const sourcesTree = [...new Set([...bySource.keys(), ...bundlesBySource.keys()])].map((sourceId) => {
         const members = bySource.get(sourceId) ?? [];
         const sourceBundles = bundlesBySource.get(sourceId) ?? [];
+        const variants = new Map();
+        for (const skill of members)
+            variants.set(skill.name, [...(variants.get(skill.name) ?? []), skill]);
         const owned = packageOwnedSkillMatcher(sourceBundles.map((entry) => entry.descriptor).filter((descriptor) => active.has(descriptor)));
         return {
             id: `source:${sourceId}`,
@@ -45,10 +55,21 @@ export function buildTree(skills, sources = [], bundles = [], activeDescriptors
                         data: { kind: "group", label: `Packages (${sourceBundles.length})` },
                         children: sourceBundles.map((entry) => ({ id: bundleKey(entry), data: { kind: "bundle", entry } })),
                     }]),
-                ...members.map((skill) => {
-                    const ownedBy = owned(skill.path);
-                    return { id: skillKey(skill), data: { kind: "skill", skill, ...(ownedBy === undefined ? {} : { ownedBy }) } };
-                }),
+                ...(members.length === 0 ? [] : [{
+                        id: `skills:${sourceId}`,
+                        data: { kind: "group", label: `Skills (${members.length})` },
+                        children: [...variants.values()].map((group) => {
+                            const variant = group.length > 1;
+                            const nodes = group.map((skill) => {
+                                const ownedBy = owned(skill.path);
+                                return { id: skillKey(skill), data: { kind: "skill", skill, ...(ownedBy === undefined ? {} : { ownedBy }), ...(variant ? { variant } : {}) } };
+                            });
+                            // Same-named Skills of one Source differ in content, so each stays its own installable row under one group.
+                            return variant
+                                ? { id: `variants:${sourceId}:${group[0].name}`, data: { kind: "group", label: `${group[0].name} (${group.length} variants)` }, children: nodes }
+                                : nodes[0];
+                        }),
+                    }]),
             ],
         };
     });
@@ -86,13 +107,29 @@ export function describeRow(row) {
     }
     if (data.kind === "app")
         return `App ${data.row.entry.recipe?.name ?? data.row.entry.file}  ${data.row.inspection.status}`;
-    return `${data.skill.name}  ${truncateDescription(data.skill.description)}`;
+    return `${data.skill.name}  ${truncateDescription(data.skill.description)}${data.variant === true ? `  (${data.skill.path})` : ""}`;
+}
+/** Skill nodes inside a Source's Skills group, directly or within one of its variant groups. */
+function skillNodesOf(roots) {
+    const found = [];
+    const visit = (node) => {
+        if (node.data.kind === "skill")
+            found.push(node);
+        else if (node.id.startsWith("variants:"))
+            node.children?.forEach(visit);
+    };
+    for (const root of roots) {
+        for (const group of root.children ?? [])
+            if (group.id.startsWith("skills:"))
+                group.children?.forEach(visit);
+    }
+    return found;
 }
 /** The Skills under the tree's Sources, in display order. */
 export function leavesOf(roots) {
-    return roots.flatMap((root) => (root.children ?? []).flatMap((child) => child.data.kind === "skill" ? [child.data.skill] : []));
+    return skillNodesOf(roots).map((node) => node.data.skill);
 }
 /** Skills that can still be installed as loose Skills, excluding active Package-owned rows. */
 export function looseLeavesOf(roots) {
-    return roots.flatMap((root) => (root.children ?? []).flatMap((child) => child.data.kind === "skill" && child.data.ownedBy === undefined ? [child.data.skill] : []));
+    return skillNodesOf(roots).filter((node) => node.data.ownedBy === undefined).map((node) => node.data.skill);
 }

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

@@ -17,6 +17,7 @@ import { useCatalogApps, useCatalogFilter, useCatalogSkills, useInstalledSkills
 import { buildTree, leavesOf, openByDefault, skillKey } from "./catalog-tree.js";
 import { theme } from "./theme.js";
 import { useTreeNavigation } from "./tree-navigation.js";
+import { invalidateRetention } from "./view-retention.js";
 import { useLoadingSince, useMarks, useMounted } from "./view-state.js";
 import { computeWindow, useListHeight } from "./window.js";
 import { useKeys } from "./keys.js";
@@ -39,7 +40,7 @@ function wrappedLineCount(text, columns) {
     }
     return lines;
 }
-export function CatalogView({ operations, sourceId, onCapturingChange, environment, listHeight }) {
+export function CatalogView({ operations, sourceId, onCapturingChange, environment, listHeight, retention }) {
     const { columns, rows: terminalRows } = useWindowSize();
     const env = environment ?? NO_ENVIRONMENT;
     const [all, setAll] = useState(sourceId === undefined);
@@ -52,12 +53,12 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
     // Mirrors the action mode synchronously, like the filter, so later keystrokes and the shell never see stale state.
     const actionRef = useRef(BROWSE);
     const filter = useCatalogFilter(actionRef, onCapturingChange);
-    const state = useCatalogSkills(operations, sourceId, all, env, reloadKey);
+    const state = useCatalogSkills(operations, sourceId, all, env, reloadKey, retention);
     const packages = useMemo(() => packageOperationsOf(env), [env]);
     const apps = useMemo(() => createAppOperations({ homeDirectory: env.homeDirectory, ...env.appEnvironment }), [env]);
-    const appCatalogState = useCatalogApps(apps);
+    const appCatalogState = useCatalogApps(apps, retention);
     const appCatalog = appCatalogState.state;
-    const { installed, installedReady, reloadInstalled } = useInstalledSkills(operations, env, mounted);
+    const { installed, installedReady, installedStale, reloadInstalled } = useInstalledSkills(operations, env, mounted, retention);
     const [appMode, setAppMode] = useState();
     const appModeRef = useRef(undefined);
     const setAppAction = (mode) => {
@@ -104,6 +105,8 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
         }
     }, [loading]);
     const loadingSince = useLoadingSince(loading, runningSince ?? resumedSince);
+    const refreshing = !loading && ((state.status === "ready" && state.stale === true) || (appCatalog.status === "ready" && appCatalog.stale === true) || installedStale);
+    const refreshingSince = useLoadingSince(refreshing);
     const flow = {
         operations,
         env,
@@ -112,6 +115,7 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
         isMounted: () => mounted.current,
         keepMarked: marks.setMarked,
         finish: async (result) => {
+            invalidateRetention(retention);
             await reloadInstalled();
             if (!mounted.current)
                 return;
@@ -126,6 +130,7 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
         isMounted: () => mounted.current,
         unmark: ids => marks.setMarked(new Set([...marks.markedRef.current].filter(id => !ids.includes(id)))),
         finish: async (result) => {
+            invalidateRetention(retention);
             await reloadInstalled();
             if (!mounted.current)
                 return;
@@ -142,6 +147,7 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
         setMessage,
         isMounted: () => mounted.current,
         finish: async (result) => {
+            invalidateRetention(retention);
             await reloadInstalled();
             if (!mounted.current)
                 return;
@@ -194,5 +200,5 @@ export function CatalogView({ operations, sourceId, onCapturingChange, environme
     // A paged declaration owns the view: rows of the tree above it would not be counted in its height.
     if (isPackageMode(action))
         return _jsx(PackagePanel, { mode: action });
-    return (_jsxs(Box, { flexDirection: "column", children: [state.warnings.map((warning) => _jsx(Text, { color: theme.warning, children: warning }, warning)), appCatalog.status === "ready" && appCatalog.error ? _jsxs(Text, { color: theme.error, children: ["Apps unavailable: ", appCatalog.error] }) : null, _jsx(CatalogHeader, { scope: scope, listed: visible.length, installable: installable.length, total: skills.length, appCount: visibleApps.length, filter: filter.filter }), rows.slice(window.start, window.end).map((row, offset) => (_jsx(CatalogRowLine, { row: row, selected: window.start + offset === index, marked: marks.marked.has(row.node.id), installed: installed }, row.node.id))), _jsx(CatalogSelectionHelp, { row: rows[index], height: helpLines }), _jsx(ListFooter, { indicator: window.indicator, selectedCount: selectedCount, message: message }), appMode ? _jsx(AppPanel, { mode: appMode }) : _jsx(ActionPanel, { mode: action })] }));
+    return (_jsxs(Box, { flexDirection: "column", children: [refreshing ? _jsx(Working, { label: "Refreshing catalog (last data shown)\u2026", since: refreshingSince }) : null, state.warnings.map((warning) => _jsx(Text, { color: theme.warning, children: warning }, warning)), appCatalog.status === "ready" && appCatalog.error ? _jsxs(Text, { color: theme.error, children: ["Apps unavailable: ", appCatalog.error] }) : null, _jsx(CatalogHeader, { scope: scope, listed: visible.length, installable: installable.length, total: skills.length, appCount: visibleApps.length, filter: filter.filter }), rows.slice(window.start, window.end).map((row, offset) => (_jsx(CatalogRowLine, { row: row, selected: window.start + offset === index, marked: marks.marked.has(row.node.id), installed: installed }, row.node.id))), _jsx(CatalogSelectionHelp, { row: rows[index], height: helpLines }), _jsx(ListFooter, { indicator: window.indicator, selectedCount: selectedCount, message: message }), appMode ? _jsx(AppPanel, { mode: appMode }) : _jsx(ActionPanel, { mode: action })] }));
 }

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

@@ -22,15 +22,18 @@ import { busy } from "./working-model.js";
 import { useLoadingSince, useMarks, useMounted } from "./view-state.js";
 import { computeWindow, useListHeight } from "./window.js";
 import { useKeys } from "./keys.js";
+import { invalidateRetention } from "./view-retention.js";
 const NO_ENVIRONMENT = {};
 const LEGEND = "adopted: tracked in place · modified: changed on disk";
-export function InstallationsView({ operations, environment, onCapturingChange, listHeight }) {
+export function InstallationsView({ operations, environment, onCapturingChange, listHeight, retention }) {
     const { rows: terminalRows, columns: terminalColumns } = useWindowSize();
     const height = useListHeight(listHeight, 7);
     const env = environment ?? NO_ENVIRONMENT;
     const apps = useMemo(() => createAppOperations({ homeDirectory: env.homeDirectory, ...env.appEnvironment }), [env]);
     const packages = useMemo(() => packageOperationsOf(env), [env]);
-    const [state, setState] = useState({ status: "loading" });
+    const [state, setState] = useState(() => retention?.installations ? { status: "ready", data: retention.installations } : { status: "loading" });
+    // The data shown comes from an earlier visit until the first reload of this one ends.
+    const [stale, setStale] = useState(retention?.installations !== undefined);
     const [mode, setModeState] = useState(BROWSE);
     // Mirrors the mode synchronously so later keystrokes and the shell never see stale state.
     const modeRef = useRef(BROWSE);
@@ -50,22 +53,30 @@ export function InstallationsView({ operations, environment, onCapturingChange,
     const loadGeneration = useRef(0);
     const reload = useCallback(async () => {
         const generation = ++loadGeneration.current;
+        const retained = retention?.generation;
         if (reloads.current++ === 0 && mounted.current)
             setRefreshingSince(Date.now());
         try {
             const data = await loadInstallations(operations, env, apps, packages);
-            if (mounted.current && generation === loadGeneration.current)
+            // A result that began before an invalidation may predate the change, so it is not kept.
+            if (retention && retention.generation === retained)
+                retention.installations = data;
+            if (mounted.current && generation === loadGeneration.current) {
                 setState({ status: "ready", data });
+                setStale(false);
+            }
         }
         catch (error) {
-            if (mounted.current && generation === loadGeneration.current)
+            if (mounted.current && generation === loadGeneration.current) {
                 setState({ status: "error", message: errorText(error) });
+                setStale(false);
+            }
         }
         finally {
             if (--reloads.current === 0 && mounted.current)
                 setRefreshingSince(undefined);
         }
-    }, [operations, env, apps, packages, mounted]);
+    }, [operations, env, apps, packages, mounted, retention]);
     useEffect(() => {
         void reload();
     }, [reload]);
@@ -129,6 +140,8 @@ export function InstallationsView({ operations, environment, onCapturingChange,
         const current = modeRef.current;
         if ("startedAt" in current && mounted.current)
             setMode(busy("Refreshing installations…", current));
+        // The action changed what is installed, so what was kept no longer describes it.
+        invalidateRetention(retention);
         await reload();
     };
     const manage = {
@@ -197,6 +210,6 @@ export function InstallationsView({ operations, environment, onCapturingChange,
     const markedApps = markedLeaves.filter(node => node.data.kind === "app").length;
     // The legend explains the markers, so it shows only while a rendered row carries one.
     const legendNeeded = listed.some((row) => row.node.data.kind === "installation" && markersOf(row.node.data.row) !== "");
-    return (_jsxs(Box, { flexDirection: "column", children: [refreshingSince !== undefined && mode.kind === "browse" ? _jsx(Working, { label: "Refreshing installations\u2026", since: refreshingSince }) : null, ready.global === undefined ? _jsx(Text, { children: "User-global installations are not supported by the configured operations" }) : null, ready.projectError === undefined ? null : _jsxs(Text, { color: theme.error, children: ["Error: ", ready.projectError] }), listed.map((row, offset) => (_jsx(TreeRowLine, { row: row, selected: window.start + offset === index, marked: marks.marked.has(row.node.id) }, row.node.id))), _jsx(ListFooter, { indicator: window.indicator, selectedCount: selectedCount, message: message }), markedApps > 0 ? _jsxs(Text, { color: theme.muted, children: [markedApps, " Apps marked"] }) : null, legendNeeded ? _jsx(Text, { color: theme.muted, children: LEGEND }) : null, checkingSince !== undefined && refreshingSince === undefined && mode.kind === "browse"
+    return (_jsxs(Box, { flexDirection: "column", children: [refreshingSince !== undefined && mode.kind === "browse" ? _jsx(Working, { label: stale ? "Refreshing installations (last data shown)…" : "Refreshing installations…", since: refreshingSince }) : null, ready.global === undefined ? _jsx(Text, { children: "User-global installations are not supported by the configured operations" }) : null, ready.projectError === undefined ? null : _jsxs(Text, { color: theme.error, children: ["Error: ", ready.projectError] }), listed.map((row, offset) => (_jsx(TreeRowLine, { row: row, selected: window.start + offset === index, marked: marks.marked.has(row.node.id) }, row.node.id))), _jsx(ListFooter, { indicator: window.indicator, selectedCount: selectedCount, message: message }), markedApps > 0 ? _jsxs(Text, { color: theme.muted, children: [markedApps, " Apps marked"] }) : null, legendNeeded ? _jsx(Text, { color: theme.muted, children: LEGEND }) : null, checkingSince !== undefined && refreshingSince === undefined && mode.kind === "browse"
                 ? _jsx(Working, { label: "Checking App status\u2026", since: checkingSince }) : null, _jsx(ModePanel, { mode: mode })] }));
 }

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

@@ -66,6 +66,6 @@ export function ManagePanel({ mode }) {
         case "confirm-bulk-exposure":
             return (_jsxs(Text, { color: theme.warning, children: ["Adding hosts exposes the installed ", mode.plan.items.map((item) => item.label).join(", "), " to more hosts and needs separate confirmation (CLI: --confirm-additional-host). ", _jsx(DecisionPrompt, { children: "Confirm?" })] }));
         case "busy":
-            return _jsx(Working, { label: mode.label, since: mode.startedAt });
+            return (_jsxs(Box, { flexDirection: "column", children: [_jsx(Working, { label: mode.label, since: mode.startedAt }), (mode.done ?? []).length > 0 ? _jsxs(Text, { color: theme.muted, children: [mode.done.length, " finished"] }) : null] }));
     }
 }