@jini-ai/ui 0.5.4 → 0.5.6

package/dist/features/panel-kit/fetch-query/adapter.tanstack.js

@@ -2,7 +2,7 @@ import { jsx as _jsx } from "react/jsx-runtime";
 /** Optional TanStack translation layer. No library types escape the fetch-query contract. */
 import { createContext, useCallback, useContext, useEffect, useMemo, useSyncExternalStore } from 'react';
 import { isCancelledError, replaceEqualDeep, QueryClient, QueryClientProvider, useMutation, useQuery } from '@tanstack/react-query';
-import { DEFAULT_STALE_TIME, FetchQueryCache, IDLE_RETENTION, toQueryError } from '@jini-ai/core/fetch';
+import { DEFAULT_STALE_TIME, FetchQueryCache, IDLE_RETENTION, toQueryError, withCachedLoaderFetch } from '@jini-ai/core/fetch';
 import { browserFetchQueryEnvironment } from './environment.browser.js';
 import { resolveFetchQueryError, resolveFetchQueryStatus } from './adapter.react.js';
 import { FetchQueryBinding } from './binding.react.js';
@@ -42,12 +42,12 @@ export function FetchQueryProvider({ children, environment = browserFetchQueryEn
     }, [scope, environment]);
     return _jsx(QueryClientProvider, { client: scope.client, children: _jsx(ScopeContext.Provider, { value: scope, children: _jsx(FetchQueryBinding, { adapter: adapter, children: children }) }) });
 }
-/** Fail closed when an adapter hook is used without its provider. */
+/** The provider's scope. This adapter's hooks are not exported: the only route to them is the
+ * `adapter` object this module's own `FetchQueryProvider` binds INSIDE its `ScopeContext`, and the
+ * public hooks already fail closed without a provider (`binding.react.tsx`'s `useAdapter`). So the
+ * old "no scope" throw here was unreachable and is gone; the guarantee is that binding order. */
 function useScope() {
-    const scope = useContext(ScopeContext);
-    if (!scope)
-        throw new Error('Fetch-query hooks require FetchQueryProvider');
-    return scope;
+    return useContext(ScopeContext);
 }
 /** Observe only this provider's connectivity; paused reads do not claim network activity. */
 function useOnline({ environment }) {
@@ -174,40 +174,53 @@ function useFetchMutation({ run }, { invalidates } = {}) {
     }, [mutateAsync]);
     return { mutate, status: mutation.status, error: mutation.status === 'error' ? toQueryError(mutation.error) : null, reset };
 }
+/**
+ * One loader read of `required.key`, unwrapped to its value. Lifted out of `useCachedLoader`'s `load`
+ * verbatim so the missing-fetch guard (`withCachedLoaderFetch`) wrapping it does not push its
+ * callbacks past the function-nesting limit.
+ */
+function loadLoaderQuery({ scope, options, required, staleTime }) {
+    const selected = { ...options, staleTime, queryKey: required.key, queryFn: () => read({ scope, key: required.key, fetch: required.fetch }),
+        structuralSharing: (previous, incoming) => saveResult({ scope, key: required.key, previous, incoming }) };
+    return scope.client.fetchQuery(selected).catch(error => {
+        // A forced refresh can supersede a loader's request without aborting its transport.
+        // Follow the new request instead of exposing TanStack's cancellation vocabulary.
+        if (!isCancelledError(error))
+            throw error;
+        // Invalidation also supersedes loader-only reads. Join (or start) the post-write
+        // request rather than returning an obsolete value while the key is invalidated.
+        const query = scope.client.getQueryCache().find({ queryKey: required.key, exact: true });
+        if (query.state.isInvalidated)
+            return scope.client.fetchQuery(selected);
+        return (query.promise ?? scope.client.getQueryData(required.key));
+    }).then(result => {
+        const query = scope.client.getQueryCache().find({ queryKey: required.key, exact: true });
+        const history = historyFor({ scope, query });
+        // Query.fetch returns the transport envelope, not structuralSharing's accepted value.
+        // Unwrap against the current generation so awaiting loaders agree with observers.
+        if (result.generation !== history.replacement)
+            return scope.client.getQueryData(required.key).value;
+        if ('error' in result)
+            throw result.error;
+        return result.value;
+    });
+}
 /** Loader shares reactive reads, retains fresh idle values, and protects replacement writes. */
 function useCachedLoader({ key, fetch }, { staleTime = DEFAULT_STALE_TIME } = {}) {
     const scope = useScope();
     return useMemo(() => {
-        const options = { queryKey: key, queryFn: () => read({ scope, key, fetch }),
+        // The template every loader query starts from. It carries no `queryFn`: `load` always supplies
+        // its own (for `required.key`), an observer that later mounts on a `replace`-created query
+        // installs the observer's, and an unobserved query is never refetched — so a template `queryFn`
+        // here could never run. Its `structuralSharing` does run: `replace` on a cold key builds the query
+        // from this template and publishes through it.
+        const options = { queryKey: key,
             structuralSharing: (previous, incoming) => saveResult({ scope, key, previous, incoming }), staleTime, gcTime: gcTimeFor({ staleTime }) };
         return {
             peek: () => scope.client.getQueryData(key)?.value,
-            load: (required = { key, fetch }) => {
-                const selected = { ...options, queryKey: required.key, queryFn: () => read({ scope, key: required.key, fetch: required.fetch }),
-                    structuralSharing: (previous, incoming) => saveResult({ scope, key: required.key, previous, incoming }) };
-                return scope.client.fetchQuery(selected).catch(error => {
-                    // A forced refresh can supersede a loader's request without aborting its transport.
-                    // Follow the new request instead of exposing TanStack's cancellation vocabulary.
-                    if (!isCancelledError(error))
-                        throw error;
-                    // Invalidation also supersedes loader-only reads. Join (or start) the post-write
-                    // request rather than returning an obsolete value while the key is invalidated.
-                    const query = scope.client.getQueryCache().find({ queryKey: required.key, exact: true });
-                    if (query.state.isInvalidated)
-                        return scope.client.fetchQuery(selected);
-                    return (query.promise ?? scope.client.getQueryData(required.key));
-                }).then(result => {
-                    const query = scope.client.getQueryCache().find({ queryKey: required.key, exact: true });
-                    const history = historyFor({ scope, query });
-                    // Query.fetch returns the transport envelope, not structuralSharing's accepted value.
-                    // Unwrap against the current generation so awaiting loaders agree with observers.
-                    if (result.generation !== history.replacement)
-                        return scope.client.getQueryData(required.key).value;
-                    if ('error' in result)
-                        throw result.error;
-                    return result.value;
-                });
-            },
+            // `fetch` is the loader's optional default; `withCachedLoaderFetch` rejects an argument-less read
+            // that has none before any query is built (see `CachedLoaderDefaults`).
+            load: (requested = { key, fetch }, { staleTime: readStaleTime = staleTime } = {}) => withCachedLoaderFetch(requested, required => loadLoaderQuery({ scope, options, required, staleTime: readStaleTime })),
             replace: ({ value }) => {
                 const query = scope.client.getQueryCache().build(scope.client, options);
                 const history = historyFor({ scope, query });

package/dist/features/panel-kit/helpers/standing-draft-local-backup.js

@@ -26,8 +26,10 @@ export const LOCAL_BACKUP_PRINCIPAL_ID = "local-tab-backup";
  * @complexity O(draft size) for reads/writes; O(1) for clears.
  */
 export function createStandingDraftLocalBackup({ keyPrefix: KEY_PREFIX, getStorage }, _options = {}) {
+    // Never sees a `null` principal: write/read/clear each return before reaching storage for one, so
+    // the old `principalId ?? ""` fallback here could not run.
     function keyFor(entryId, principalId) {
-        return principalId === undefined ? `${KEY_PREFIX}${entryId}` : `${KEY_PREFIX}${encodeURIComponent(principalId ?? "")}.${encodeURIComponent(entryId)}`;
+        return principalId === undefined ? `${KEY_PREFIX}${entryId}` : `${KEY_PREFIX}${encodeURIComponent(principalId)}.${encodeURIComponent(entryId)}`;
     }
     /** Mirrors a refused draft for this browser profile. Silent no-op on any storage failure.
      *  @complexity Time/space: O(size of the draft) for the one `JSON.stringify`. */

package/dist/features/panel-kit/hooks/use-settings-slice.hooks.js

@@ -261,7 +261,9 @@ export function useSettingsSlice(options, _optional = {}) {
         // left to paint. The write itself still has to happen — through the same
         // lane as the debounce path, so it diffs against the base that is
         // current when it RUNS, not the one that was current when it was queued.
-        void writes.run({ task: () => runSave().then(() => { }, () => { }) });
+        void writes.run({ task: () => runSave().then(() => { }, 
+            // No view remains to show the failure; settings and native errors may contain secrets.
+            () => { console.error('[settings] Final save failed after unmount.'); }) });
     }, [runSave, writes]);
     const onChange = useCallback((next) => {
         setValue(next);

package/dist/features/privacy/react/components/PrivacyTab.js

@@ -6,6 +6,27 @@ import { hasMadeConsentDecision, isSharingEnabled, nextStateForDeclineAll, nextS
 function ToggleRow({ label, hint, checked, onChange, agentHandle }) {
     return (_jsxs("button", { type: "button", className: `jini-toggle-row${checked ? ' on' : ''}`, onClick: () => onChange(!checked), "aria-pressed": checked, ...agentHandleProps({}, { base: agentHandle, ...({ role: 'checkbox', label }) }), children: [_jsxs("div", { className: "jini-toggle-row-text", children: [_jsx("span", { className: "jini-toggle-row-label", children: label }), _jsx("span", { className: "jini-toggle-row-hint", children: hint })] }), _jsx("span", { className: "jini-toggle-row-switch", "aria-hidden": true })] }));
 }
+function resolveConsentLabels({ labels }, { t }) {
+    return {
+        consentKicker: labels.consentKicker ?? t('Help improve this product'),
+        consentLead: labels.consentLead ?? t('Choose what you share. You can change this anytime.'),
+        consentFooter: labels.consentFooter ?? t('Nothing is shared unless you opt in.'),
+        declineLabel: labels.declineLabel ?? t("Don't share"),
+        shareLabel: labels.shareLabel ?? t('Share usage'),
+        metricsLabel: labels.metricsLabel ?? t('Anonymous metrics'),
+        metricsHint: labels.metricsHint ?? t('Feature usage and performance data, with no personal content.'),
+        contentLabel: labels.contentLabel ?? t('Conversation and tool content'),
+        contentHint: labels.contentHint ?? t('Helps improve model quality. Off by default.'),
+    };
+}
+function resolveInstallationLabels({ labels }, { t }) {
+    return {
+        installationIdLabel: labels.installationIdLabel ?? t('Installation ID'),
+        installationIdHint: labels.installationIdHint ?? t('An anonymous id used to group your shared data.'),
+        optedOutLabel: labels.optedOutLabel ?? t('Not sharing'),
+        deleteMyDataLabel: labels.deleteMyDataLabel ?? t('Delete my data'),
+    };
+}
 /**
  * Telemetry consent card (share/decline + two per-category toggles) over a
  * generic `{ metrics, content }` shape, plus an installation-id
@@ -19,19 +40,9 @@ export function PrivacyTab({ state, onChange, labels, now = Date.now, agentHandl
     const t = useT();
     const decided = hasMadeConsentDecision(state);
     const sharing = decided ? isSharingEnabled(state.telemetry) : undefined;
-    const consentKicker = labels?.consentKicker ?? t('Help improve this product');
-    const consentLead = labels?.consentLead ?? t('Choose what you share. You can change this anytime.');
-    const consentFooter = labels?.consentFooter ?? t('Nothing is shared unless you opt in.');
-    const declineLabel = labels?.declineLabel ?? t("Don't share");
-    const shareLabel = labels?.shareLabel ?? t('Share usage');
-    const metricsLabel = labels?.metricsLabel ?? t('Anonymous metrics');
-    const metricsHint = labels?.metricsHint ?? t('Feature usage and performance data, with no personal content.');
-    const contentLabel = labels?.contentLabel ?? t('Conversation and tool content');
-    const contentHint = labels?.contentHint ?? t('Helps improve model quality. Off by default.');
-    const installationIdLabel = labels?.installationIdLabel ?? t('Installation ID');
-    const installationIdHint = labels?.installationIdHint ?? t('An anonymous id used to group your shared data.');
-    const optedOutLabel = labels?.optedOutLabel ?? t('Not sharing');
-    const deleteMyDataLabel = labels?.deleteMyDataLabel ?? t('Delete my data');
+    const labelOverrides = labels ?? {};
+    const { consentKicker, consentLead, consentFooter, declineLabel, shareLabel, metricsLabel, metricsHint, contentLabel, contentHint, } = resolveConsentLabels({ labels: labelOverrides }, { t });
+    const { installationIdLabel, installationIdHint, optedOutLabel, deleteMyDataLabel, } = resolveInstallationLabels({ labels: labelOverrides }, { t });
     return (_jsxs("section", { className: "jini-settings-section jini-settings-privacy", children: [_jsxs("div", { className: "jini-settings-subsection", children: [_jsx("div", { className: "jini-section-head", children: _jsxs("div", { children: [_jsx("h4", { children: consentKicker }), _jsx("p", { className: "jini-hint", children: consentLead })] }) }), _jsxs("dl", { className: "jini-settings-privacy-disclosure", children: [_jsxs("div", { children: [_jsx("dt", { children: metricsLabel }), _jsx("dd", { children: metricsHint })] }), _jsxs("div", { children: [_jsx("dt", { children: contentLabel }), _jsx("dd", { children: contentHint })] })] }), _jsx("p", { className: "jini-hint", children: consentFooter }), _jsxs("div", { className: "jini-privacy-consent-actions", role: "group", "aria-label": consentKicker, children: [_jsx("button", { type: "button", className: `jini-privacy-consent-action${sharing === false ? ' is-active' : ''}`, "aria-pressed": sharing === false, onClick: () => onChange(nextStateForDeclineAll(state, now())), ...agentHandleProps({}, { base: agentHandle, ...({ action: 'decline', role: 'button', label: declineLabel }) }), children: declineLabel }), _jsx("button", { type: "button", className: `jini-privacy-consent-action jini-privacy-consent-action--primary${sharing === true ? ' is-active' : ''}`, "aria-pressed": sharing === true, onClick: () => onChange(nextStateForShareAll(state, now())), ...agentHandleProps({}, { base: agentHandle, ...({ action: 'share', role: 'button', label: shareLabel }) }), children: shareLabel })] })] }), decided ? (_jsxs(_Fragment, { children: [_jsxs("div", { className: "jini-settings-privacy-toggles", children: [_jsx(ToggleRow, { label: metricsLabel, hint: metricsHint, checked: state.telemetry.metrics === true, onChange: (v) => onChange(nextStateForTelemetryPatch(state, { metrics: v }, now())), ...(agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'metrics-toggle' }) } : {}) }), _jsx(ToggleRow, { label: contentLabel, hint: contentHint, checked: state.telemetry.content === true, onChange: (v) => onChange(nextStateForTelemetryPatch(state, { content: v }, now())), ...(agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'content-toggle' }) } : {}) })] }), _jsxs("div", { className: "jini-settings-subsection", children: [_jsx("div", { className: "jini-section-head", children: _jsxs("div", { children: [_jsx("h4", { children: installationIdLabel }), _jsx("p", { className: "jini-hint", children: installationIdHint })] }) }), _jsx("div", { className: "jini-settings-field", children: _jsx("input", { type: "text", readOnly: true, value: state.installationId ?? optedOutLabel, "aria-label": installationIdLabel, ...agentHandleProps({}, { base: agentHandle, ...({ action: 'installation-id', role: 'field', label: installationIdLabel }) }) }) }), _jsxs("button", { type: "button", className: "jini-button jini-button-ghost", onClick: () => onChange(nextStateForDeleteMyData(state, now())), ...agentHandleProps({}, { base: agentHandle, ...({ action: 'delete-my-data', role: 'button', label: deleteMyDataLabel }) }), children: [_jsx(Icon, { name: "trash", size: 13 }), _jsx("span", { children: deleteMyDataLabel })] })] })] })) : null] }));
 }
 //# sourceMappingURL=PrivacyTab.js.map

package/dist/features/progress-card/components/ProgressCard.js

@@ -14,11 +14,15 @@ export function ProgressCard({ data, maxSteps = 6, maxSecondaryItems = 5 }) {
     const title = data.title ?? t(defaultProgressCardTitle(data.status));
     const detail = data.detail ?? t(defaultProgressCardDetail(data.status));
     const secondaryHeading = t(data.secondaryItemsLabel ?? 'Files touched');
-    const widthPercent = progressBarWidthPercent(data.progress);
-    const ariaValueNow = progressBarAriaValueNow(data.progress);
     const visibleSteps = data.steps.slice(0, maxSteps);
     const visibleSecondaryItems = (data.secondaryItems ?? []).slice(0, maxSecondaryItems);
-    return (_jsxs("section", { className: `progress-card is-${data.status}`, "data-progress-card-id": data.id, children: [_jsxs("div", { className: "progress-card-head", children: [_jsx(Icon, { name: progressCardStatusIcon(data.status) }), _jsxs("span", { children: [_jsx("strong", { children: title }), _jsx("small", { children: detail })] })] }), _jsx("div", { className: `progress-card-bar${widthPercent === null ? ' is-indeterminate' : ''}`, role: "progressbar", "aria-label": t('{title} progress', { title }), "aria-valuemin": 0, "aria-valuemax": 100, "aria-valuenow": ariaValueNow, children: _jsx("span", { style: widthPercent === null ? undefined : { width: `${widthPercent}%` } }) }), visibleSteps.length > 0 ? _jsx(ProgressCardItemList, { items: visibleSteps, className: "progress-card-steps" }) : null, visibleSecondaryItems.length > 0 ? (_jsxs("div", { className: "progress-card-secondary-items", children: [_jsx("span", { children: secondaryHeading }), _jsx(ProgressCardItemList, { items: visibleSecondaryItems, className: "progress-card-secondary-item-list", asCode: true })] })) : null] }));
+    return (_jsxs("section", { className: `progress-card is-${data.status}`, "data-progress-card-id": data.id, children: [_jsxs("div", { className: "progress-card-head", children: [_jsx(Icon, { name: progressCardStatusIcon(data.status) }), _jsxs("span", { children: [_jsx("strong", { children: title }), _jsx("small", { children: detail })] })] }), _jsx(ProgressCardBar, { progress: data.progress, label: t('{title} progress', { title }) }), visibleSteps.length > 0 ? _jsx(ProgressCardItemList, { items: visibleSteps, className: "progress-card-steps" }) : null, visibleSecondaryItems.length > 0 ? (_jsxs("div", { className: "progress-card-secondary-items", children: [_jsx("span", { children: secondaryHeading }), _jsx(ProgressCardItemList, { items: visibleSecondaryItems, className: "progress-card-secondary-item-list", asCode: true })] })) : null] }));
+}
+/** Presents the existing progress rules as a determinate or indeterminate accessible bar. */
+function ProgressCardBar({ progress, label }) {
+    const widthPercent = progressBarWidthPercent(progress);
+    const ariaValueNow = progressBarAriaValueNow(progress);
+    return (_jsx("div", { className: `progress-card-bar${widthPercent === null ? ' is-indeterminate' : ''}`, role: "progressbar", "aria-label": label, "aria-valuemin": 0, "aria-valuemax": 100, "aria-valuenow": ariaValueNow, children: _jsx("span", { style: widthPercent === null ? undefined : { width: `${widthPercent}%` } }) }));
 }
 function ProgressCardItemList({ items, className, asCode, }) {
     const Tag = asCode ? 'code' : 'span';

package/dist/features/progress-card/reference-adapters.js

@@ -29,17 +29,13 @@ export function parseTodoWriteInput(input) {
         if (!todo || typeof todo !== 'object')
             return null;
         const todoRecord = todo;
-        const content = typeof todoRecord.content === 'string'
-            ? todoRecord.content
-            : typeof todoRecord.step === 'string'
-                ? todoRecord.step
-                : typeof todoRecord.description === 'string'
-                    ? todoRecord.description
-                    : typeof todoRecord.label === 'string'
-                        ? todoRecord.label
-                        : typeof todoRecord.text === 'string'
-                            ? todoRecord.text
-                            : '';
+        let content = '';
+        for (const key of ['content', 'step', 'description', 'label', 'text']) {
+            if (typeof todoRecord[key] === 'string') {
+                content = todoRecord[key];
+                break;
+            }
+        }
         if (!content)
             return null;
         return { content, status: normalizeTodoStatus(todoRecord.status) };
@@ -118,48 +114,55 @@ export function dedupeToolUsesById(events) {
     let deduped = null;
     for (let index = 0; index < events.length; index += 1) {
         const event = events[index];
-        if (isToolUseEvent(event)) {
-            if (seen.has(event.id)) {
-                if (!deduped)
-                    deduped = events.slice(0, index);
-                continue;
-            }
+        if (!isToolUseEvent(event)) {
+            deduped?.push(event);
+        }
+        else if (seen.has(event.id)) {
+            deduped ??= events.slice(0, index);
+        }
+        else {
             seen.add(event.id);
+            deduped?.push(event);
         }
-        if (deduped)
-            deduped.push(event);
     }
     return deduped ?? events;
 }
+function fileOpFromAgentEvent({ event, resultByToolUseId }, _options = {}) {
+    if (!isToolUseEvent(event))
+        return null;
+    const kind = classifyFileOp(event.name);
+    if (!kind)
+        return null;
+    const fullPath = extractFileOpPath(event.input);
+    if (!fullPath)
+        return null;
+    const result = resultByToolUseId.get(event.id);
+    const status = result === undefined ? 'running' : result.isError ? 'error' : 'done';
+    return { fullPath, kind, status };
+}
 export function deriveFileOpsFromAgentEvents(events) {
     if (!events || events.length === 0)
         return [];
     const dedupedEvents = dedupeToolUsesById(events);
-    const resultByToolUseId = new Map();
-    for (const event of dedupedEvents) {
-        if (isToolResultEvent(event))
-            resultByToolUseId.set(event.toolUseId, event);
-    }
+    const resultByToolUseId = new Map(dedupedEvents.filter(isToolResultEvent).map((event) => [event.toolUseId, event]));
     const byPath = new Map();
     for (const event of dedupedEvents) {
-        if (!isToolUseEvent(event))
-            continue;
-        const kind = classifyFileOp(event.name);
-        if (!kind)
-            continue;
-        const fullPath = extractFileOpPath(event.input);
-        if (!fullPath)
+        const entry = fileOpFromAgentEvent({ event, resultByToolUseId });
+        if (!entry)
             continue;
-        const result = resultByToolUseId.get(event.id);
-        const status = result === undefined ? 'running' : result.isError ? 'error' : 'done';
-        const existing = byPath.get(fullPath);
+        const existing = byPath.get(entry.fullPath);
         if (existing) {
-            if (!existing.ops.includes(kind))
-                existing.ops.push(kind);
-            existing.status = mergeFileOpStatus(existing.status, status);
+            if (!existing.ops.includes(entry.kind))
+                existing.ops.push(entry.kind);
+            existing.status = mergeFileOpStatus(existing.status, entry.status);
             continue;
         }
-        byPath.set(fullPath, { path: basename(fullPath), fullPath, ops: [kind], status });
+        byPath.set(entry.fullPath, {
+            path: basename(entry.fullPath),
+            fullPath: entry.fullPath,
+            ops: [entry.kind],
+            status: entry.status,
+        });
     }
     return Array.from(byPath.values());
 }
@@ -204,15 +207,7 @@ function fallbackChatActivitySteps(status, fileOps) {
         {
             id: 'update-files',
             label: 'Update files',
-            status: hasError
-                ? 'failed'
-                : hasMutation
-                    ? fileOps.some((entry) => entry.status === 'running')
-                        ? 'running'
-                        : 'succeeded'
-                    : status === 'running'
-                        ? 'pending'
-                        : 'succeeded',
+            status: fallbackUpdateStatus({ status, fileOps, hasError, hasMutation }),
         },
         {
             id: 'finalize',
@@ -221,6 +216,22 @@ function fallbackChatActivitySteps(status, fileOps) {
         },
     ];
 }
+function fallbackUpdateStatus({ status, fileOps, hasError, hasMutation }, _options = {}) {
+    if (hasError)
+        return 'failed';
+    if (hasMutation)
+        return fileOps.some((entry) => entry.status === 'running') ? 'running' : 'succeeded';
+    return status === 'running' ? 'pending' : 'succeeded';
+}
+function chatActivitySteps({ todos, status, fileOps }, _options = {}) {
+    if (todos.length === 0)
+        return fallbackChatActivitySteps(status, fileOps);
+    return todos.map((todo, index) => ({
+        id: `${todo.content}-${index}`,
+        label: todo.content,
+        status: todoStatusToProgressStatus(todo.status),
+    }));
+}
 /** Maps a chat message's agent-event stream onto `ProgressCardData`. Returns
  *  `null` when there's nothing worth showing (mirrors the source's own
  *  `hasActivity` early-return — a message with no run, no todos, no file
@@ -234,13 +245,7 @@ export function chatActivityToProgressCard(message, options) {
     const hasActivity = options.active || todos.length > 0 || fileOps.length > 0 || statusDetail !== null || status === 'failed';
     if (!hasActivity)
         return null;
-    const steps = todos.length > 0
-        ? todos.map((todo, index) => ({
-            id: `${todo.content}-${index}`,
-            label: todo.content,
-            status: todoStatusToProgressStatus(todo.status),
-        }))
-        : fallbackChatActivitySteps(status, fileOps);
+    const steps = chatActivitySteps({ todos, status, fileOps });
     const secondaryItems = fileOps.map((entry) => ({
         id: entry.fullPath,
         label: entry.path,

package/dist/features/project-locations/react/components/ProjectLocationsTab.js

@@ -3,6 +3,29 @@ import { useT } from '../../../i18n/index.js';
 import { Icon } from '../../../../react/components/Icon.js';
 import { locationLabel } from '../../rules.js';
 import { useProjectLocationsTab } from '../hooks/useProjectLocationsTab.js';
+function resolveLabels({ t }, labels = {}) {
+    const copy = {
+        title: 'Project locations',
+        description: 'Where new projects are created and existing ones are found.',
+        builtInLabel: 'Default location',
+        defaultBadge: 'Default',
+        makeDefaultLabel: 'Make default',
+        deleteLabel: 'Delete',
+        addFolderLabel: 'Add folder',
+        rowHint: 'Used as a project root.',
+        noFolderSelectedLabel: 'No folder selected.',
+        duplicateLabel: 'That folder is already a project location.',
+        savedLabel: 'Saved.',
+        defaultSavedLabel: 'Default location updated.',
+        saveErrorLabel: 'Could not save project locations.',
+        scanErrorLabel: 'Could not scan the new folder for existing projects.',
+        scanCompleteTemplate: 'Imported {imported}, {existing} already tracked.',
+    };
+    for (const key of Object.keys(copy)) {
+        copy[key] = labels?.[key] ?? t(copy[key]);
+    }
+    return copy;
+}
 /**
  * Configured project roots: the host's built-in default plus operator-added
  * folders, each markable as the new-project default and removable. Origin:
@@ -13,21 +36,7 @@ import { useProjectLocationsTab } from '../hooks/useProjectLocationsTab.js';
 export function ProjectLocationsTab({ port, defaultLocationId, onDefaultLocationIdChange, labels }) {
     const t = useT();
     const { drafts, builtIn, effectiveDefaultLocationId, saving, result, addFolder, removeDraft, setDefaultLocationId } = useProjectLocationsTab({ port, defaultLocationId, onDefaultLocationIdChange });
-    const title = labels?.title ?? t('Project locations');
-    const description = labels?.description ?? t('Where new projects are created and existing ones are found.');
-    const builtInLabel = labels?.builtInLabel ?? t('Default location');
-    const defaultBadge = labels?.defaultBadge ?? t('Default');
-    const makeDefaultLabel = labels?.makeDefaultLabel ?? t('Make default');
-    const deleteLabel = labels?.deleteLabel ?? t('Delete');
-    const addFolderLabel = labels?.addFolderLabel ?? t('Add folder');
-    const rowHint = labels?.rowHint ?? t('Used as a project root.');
-    const noFolderSelectedLabel = labels?.noFolderSelectedLabel ?? t('No folder selected.');
-    const duplicateLabel = labels?.duplicateLabel ?? t('That folder is already a project location.');
-    const savedLabel = labels?.savedLabel ?? t('Saved.');
-    const defaultSavedLabel = labels?.defaultSavedLabel ?? t('Default location updated.');
-    const saveErrorLabel = labels?.saveErrorLabel ?? t('Could not save project locations.');
-    const scanErrorLabel = labels?.scanErrorLabel ?? t('Could not scan the new folder for existing projects.');
-    const scanCompleteTemplate = labels?.scanCompleteTemplate ?? t('Imported {imported}, {existing} already tracked.');
+    const { title, description, builtInLabel, defaultBadge, makeDefaultLabel, deleteLabel, addFolderLabel, rowHint, noFolderSelectedLabel, duplicateLabel, savedLabel, defaultSavedLabel, saveErrorLabel, scanErrorLabel, scanCompleteTemplate, } = resolveLabels({ t }, labels);
     function defaultControlLabel(active) {
         return active ? defaultBadge : makeDefaultLabel;
     }

package/dist/features/project-locations/react/hooks/useProjectLocationsTab.d.ts

@@ -26,6 +26,14 @@ export interface UseProjectLocationsTabResult {
     removeDraft: (index: number) => void;
     setDefaultLocationId: (id: string) => void;
 }
+/** Scan a saved location; disposed callers cannot publish success or failure. */
+export declare function scanAddedLocation({ port, alive, setResult }: {
+    port: ProjectLocationsPort;
+    alive: {
+        readonly current: boolean;
+    };
+    setResult: (result: ProjectLocationsActionResult) => void;
+}, _options?: {}): Promise<void>;
 /**
  * Owns the tab's async edges: the initial locations fetch, the add-folder ->
  * duplicate-check -> save -> (optional) scan sequence, and removal. Origin:

package/dist/features/project-locations/react/hooks/useProjectLocationsTab.js

@@ -1,5 +1,22 @@
 import { useCallback, useEffect, useRef, useState } from 'react';
 import { DEFAULT_LOCATION_ID, externalLocations, isDuplicatePath, resolveDefaultLocationId, saveableDrafts, } from '../../rules.js';
+/** Scan a saved location; disposed callers cannot publish success or failure. */
+export async function scanAddedLocation({ port, alive, setResult }, _options = {}) {
+    const scan = port.scanLocations;
+    if (!scan)
+        return;
+    try {
+        const scanResult = await scan.call(port);
+        if (!alive.current)
+            return;
+        setResult({ status: 'scan-complete', imported: scanResult.imported.length, existing: scanResult.existing.length });
+    }
+    catch (error) {
+        if (!alive.current)
+            return;
+        setResult({ status: 'scan-error', message: error instanceof Error ? error.message : String(error) });
+    }
+}
 /**
  * Owns the tab's async edges: the initial locations fetch, the add-folder ->
  * duplicate-check -> save -> (optional) scan sequence, and removal. Origin:
@@ -56,11 +73,7 @@ export function useProjectLocationsTab({ port, defaultLocationId, onDefaultLocat
         };
     }, [port]);
     const persist = useCallback(async (nextDrafts) => {
-        setSaving(true);
-        try {
-            const saved = await portRef.current.saveLocations(saveableDrafts(nextDrafts));
-            if (!alive.current)
-                return null;
+        const applySaved = (saved) => {
             setLocations(saved);
             setDrafts(externalLocations(saved));
             setResult({ status: 'saved' });
@@ -72,17 +85,25 @@ export function useProjectLocationsTab({ port, defaultLocationId, onDefaultLocat
             if (effective !== configured)
                 onDefaultLocationIdChangeRef.current?.(effective);
             return saved;
+        };
+        setSaving(true);
+        // One return after the try statement (null unless a still-mounted save succeeded), so no
+        // unreachable continuation follows a try and a catch that both return.
+        let persisted = null;
+        try {
+            const saved = await portRef.current.saveLocations(saveableDrafts(nextDrafts));
+            if (alive.current)
+                persisted = applySaved(saved);
         }
         catch (error) {
-            if (!alive.current)
-                return null;
-            setResult({ status: 'save-error', message: error instanceof Error ? error.message : String(error) });
-            return null;
+            if (alive.current)
+                setResult({ status: 'save-error', message: error instanceof Error ? error.message : String(error) });
         }
         finally {
             if (alive.current)
                 setSaving(false);
         }
+        return persisted;
     }, []);
     const addFolder = useCallback(() => {
         void (async () => {
@@ -111,20 +132,7 @@ export function useProjectLocationsTab({ port, defaultLocationId, onDefaultLocat
                     setDrafts(previous);
                 return;
             }
-            const scan = portRef.current.scanLocations;
-            if (!scan)
-                return;
-            try {
-                const scanResult = await scan.call(portRef.current);
-                if (!alive.current)
-                    return;
-                setResult({ status: 'scan-complete', imported: scanResult.imported.length, existing: scanResult.existing.length });
-            }
-            catch (error) {
-                if (!alive.current)
-                    return;
-                setResult({ status: 'scan-error', message: error instanceof Error ? error.message : String(error) });
-            }
+            await scanAddedLocation({ port: portRef.current, alive, setResult });
         })();
     }, [persist]);
     const removeDraft = useCallback((index) => {

package/dist/features/resource-dashboard/react/components/ResourceBoard.js

@@ -2,7 +2,8 @@ import { jsx as _jsx } from "react/jsx-runtime";
 import { useT } from '../../../i18n/index.js';
 import { useWiredResourceBoard } from '../hooks/useResourceBoard.js';
 import { ResourceBoardView } from './ResourceBoardView.js';
-import { UNMATCHED_STATUS_BUCKET } from '../../constants.js';
+import { boardCreateProps, buildKanbanColumns, createItemActionHandler, resourceBoardHookParams, statusLabelFor } from './resourceBoardProps.js';
+import { translatedErrorLabelProps } from './translatedErrorLabelProps.js';
 /**
  * The DesignsTab-shaped orchestrator: wires `useWiredResourceBoard` +
  * `useT()` + `ResourceBoardView`. `rename`/`duplicate`/`delete` are the
@@ -14,51 +15,9 @@ import { UNMATCHED_STATUS_BUCKET } from '../../constants.js';
  */
 export function ResourceBoard({ dependencies, statusOptions, toneMap, defaultStatus, normalizeStatus, sortOptions = [], initialSort, storageScopeKey, defaultViewMode, enableBulkDelete = true, onOpenItem, onCreate, createLabel, onRenameRequest, onCustomItemAction, refreshToken, renderBody, useWiredResourceBoard: useWiredResourceBoardHook = useWiredResourceBoard, }) {
     const t = useT();
-    const board = useWiredResourceBoardHook({
-        dependencies,
-        statusOrder: statusOptions.map((option) => option.value),
-        ...(defaultStatus !== undefined ? { defaultStatus } : {}),
-        ...(normalizeStatus !== undefined ? { normalizeStatus } : {}),
-        ...(initialSort !== undefined ? { initialSort } : {}),
-        ...(storageScopeKey !== undefined ? { storageScopeKey } : {}),
-        ...(defaultViewMode !== undefined ? { defaultViewMode } : {}),
-        ...(refreshToken !== undefined ? { refreshToken } : {}),
-    });
-    const kanbanColumns = statusOptions.map((option) => ({
-        status: option.value,
-        label: t(option.label),
-        // Non-null: `useWiredResourceBoard` above is called with `statusOrder:
-        // statusOptions.map(option => option.value)`, and `groupItemsByStatus`
-        // (rules.ts) always seeds one Map entry per `statusOrder` value before
-        // placing any item — so every `option.value` here is guaranteed present.
-        items: board.kanbanColumns.get(option.value),
-    }));
-    const unmatched = board.kanbanColumns.get(UNMATCHED_STATUS_BUCKET);
-    if (unmatched && unmatched.length > 0) {
-        kanbanColumns.push({ status: UNMATCHED_STATUS_BUCKET, label: t('Other'), items: unmatched });
-    }
-    const handleItemAction = (id, kind) => {
-        if (kind === 'delete') {
-            void board.remove(id);
-            return;
-        }
-        if (kind === 'duplicate') {
-            void board.duplicate(id);
-            return;
-        }
-        if (kind === 'rename') {
-            board.closeMenu();
-            // Non-null: `id` only ever reaches this handler via a menu-action
-            // click closure `ResourceBoardView` builds per rendered item (from
-            // this SAME render's `board.visibleItems`), so the item is always
-            // still present in the array at lookup time.
-            const current = board.visibleItems.find((item) => item.id === id);
-            onRenameRequest?.(id, current.title);
-            return;
-        }
-        board.closeMenu();
-        onCustomItemAction?.(id, kind);
-    };
-    return (_jsx(ResourceBoardView, { loading: board.loading, error: board.error, ...(board.error ? { errorLabel: t(board.error) } : {}), actionError: board.actionError, ...(board.actionError ? { actionErrorLabel: t(board.actionError) } : {}), hasAnyItems: board.totalCount > 0, items: board.visibleItems, kanbanColumns: kanbanColumns, sortOptions: sortOptions, ...(board.sort !== undefined ? { activeSort: board.sort } : {}), onSortChange: board.setSort, sortAriaLabel: t('Sort'), query: board.query, onQueryChange: board.setQuery, searchPlaceholder: t('Search…'), viewMode: board.viewMode, onViewModeChange: board.setViewMode, viewToggleAriaLabel: t('View'), gridViewLabel: t('Grid view'), kanbanViewLabel: t('Kanban view'), enableBulkDelete: enableBulkDelete, selectMode: board.selectMode, selected: board.selected, bulkDeleteBusy: board.bulkDeleteBusy, onEnterSelectMode: board.enterSelectMode, onExitSelectMode: board.exitSelectMode, onBulkDelete: () => void board.bulkDelete(), selectModeLabel: t('Select'), selectedCountLabel: (n) => t('{n} selected', { n }), deleteSelectedLabel: t('Delete selected'), cancelSelectLabel: t('Cancel'), ...(createLabel !== undefined || onCreate ? { createLabel: createLabel ?? t('New') } : {}), ...(onCreate ? { onCreate } : {}), openMenuId: board.openMenuId, menuContainerRef: board.menuContainerRef, onToggleMenu: board.toggleMenu, onItemAction: handleItemAction, isItemBusy: board.isItemBusy, onOpenItem: onOpenItem, onToggleSelected: board.toggleSelected, onKanbanDelete: (id) => void board.remove(id), statusLabel: (status) => t(statusOptions.find((option) => option.value === status)?.label ?? status), ...(toneMap ? { toneMap } : {}), menuActionLabel: (_kind, fallback) => t(fallback), moreLabel: t('More'), deleteLabel: t('Delete'), deleteAriaLabel: (title) => t('Delete {name}', { name: title }), emptyColumnLabel: t('No items in this column.'), emptyStateTitle: t('No items yet.'), emptyStateNoMatch: t('No items match your search.'), ...(renderBody ? { renderBody } : {}) }));
+    const board = useWiredResourceBoardHook(resourceBoardHookParams({ dependencies, statusOptions }, { defaultStatus, normalizeStatus, initialSort, storageScopeKey, defaultViewMode, refreshToken }));
+    const kanbanColumns = buildKanbanColumns({ statusOptions, columns: board.kanbanColumns, t });
+    const handleItemAction = createItemActionHandler({ board }, { onRenameRequest, onCustomItemAction });
+    return (_jsx(ResourceBoardView, { loading: board.loading, error: board.error, actionError: board.actionError, ...translatedErrorLabelProps({ t, error: board.error, actionError: board.actionError }), hasAnyItems: board.totalCount > 0, items: board.visibleItems, kanbanColumns: kanbanColumns, sortOptions: sortOptions, ...(board.sort !== undefined ? { activeSort: board.sort } : {}), onSortChange: board.setSort, sortAriaLabel: t('Sort'), query: board.query, onQueryChange: board.setQuery, searchPlaceholder: t('Search…'), viewMode: board.viewMode, onViewModeChange: board.setViewMode, viewToggleAriaLabel: t('View'), gridViewLabel: t('Grid view'), kanbanViewLabel: t('Kanban view'), enableBulkDelete: enableBulkDelete, selectMode: board.selectMode, selected: board.selected, bulkDeleteBusy: board.bulkDeleteBusy, onEnterSelectMode: board.enterSelectMode, onExitSelectMode: board.exitSelectMode, onBulkDelete: () => void board.bulkDelete(), selectModeLabel: t('Select'), selectedCountLabel: (n) => t('{n} selected', { n }), deleteSelectedLabel: t('Delete selected'), cancelSelectLabel: t('Cancel'), ...boardCreateProps({ t }, { createLabel, onCreate }), openMenuId: board.openMenuId, menuContainerRef: board.menuContainerRef, onToggleMenu: board.toggleMenu, onItemAction: handleItemAction, isItemBusy: board.isItemBusy, onOpenItem: onOpenItem, onToggleSelected: board.toggleSelected, onKanbanDelete: (id) => void board.remove(id), statusLabel: statusLabelFor({ t, statusOptions }), ...(toneMap ? { toneMap } : {}), menuActionLabel: (_kind, fallback) => t(fallback), moreLabel: t('More'), deleteLabel: t('Delete'), deleteAriaLabel: (title) => t('Delete {name}', { name: title }), emptyColumnLabel: t('No items in this column.'), emptyStateTitle: t('No items yet.'), emptyStateNoMatch: t('No items match your search.'), ...(renderBody ? { renderBody } : {}) }));
 }
 //# sourceMappingURL=ResourceBoard.js.map

package/dist/features/resource-dashboard/react/components/ResourceBoardView.d.ts

@@ -1,5 +1,5 @@
 import type { ReactNode, RefObject } from 'react';
-import { type ResourceKanbanColumn } from './ResourceKanbanBoard.js';
+import type { ResourceKanbanColumn } from './ResourceKanbanBoard.js';
 import type { ResourceBoardItem, ResourceBoardViewMode, ResourceSortOption, ResourceStatusToneMap } from '../../types.js';
 export interface ResourceBoardViewProps<TBody = unknown> {
     loading: boolean;
@@ -61,5 +61,5 @@ export interface ResourceBoardViewProps<TBody = unknown> {
  * state, the card grid, or the kanban board — DesignsTab's own
  * `filtered.length === 0 ? empty : view === "grid" ? grid : kanban` branch.
  */
-export declare function ResourceBoardView<TBody = unknown>({ loading, error, errorLabel, actionError, actionErrorLabel, hasAnyItems, items, kanbanColumns, sortOptions, activeSort, onSortChange, sortAriaLabel, query, onQueryChange, searchPlaceholder, viewMode, onViewModeChange, viewToggleAriaLabel, gridViewLabel, kanbanViewLabel, enableBulkDelete, selectMode, selected, bulkDeleteBusy, onEnterSelectMode, onExitSelectMode, onBulkDelete, selectModeLabel, selectedCountLabel, deleteSelectedLabel, cancelSelectLabel, createLabel, onCreate, openMenuId, menuContainerRef, onToggleMenu, onItemAction, isItemBusy, onOpenItem, onToggleSelected, onKanbanDelete, statusLabel, toneMap, menuActionLabel, moreLabel, deleteLabel, deleteAriaLabel, emptyColumnLabel, emptyStateTitle, emptyStateNoMatch, renderBody, }: ResourceBoardViewProps<TBody>): import("react").JSX.Element;
+export declare function ResourceBoardView<TBody = unknown>(props: ResourceBoardViewProps<TBody>): import("react").JSX.Element;
 //# sourceMappingURL=ResourceBoardView.d.ts.map