@jini-ai/ui 0.6.15-rc.2 → 0.6.16

package/dist/features/browser-chrome/rules.js

@@ -1,4 +1,5 @@
 import { DEFAULT_HOME_NAVIGATION_ENTRY, EMPTY_URL, HISTORY_LIMIT } from './constants.js';
+import { trimTrailingChars } from '../../utils/trim-chars.js';
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
 }
@@ -46,7 +47,7 @@ export function formatAddressDisplayParts(url, title, homeLabel = DEFAULT_HOME_N
     const fallback = labelFromUrl(url, homeLabel);
     if (cleanTitle === fallback || cleanTitle === url)
         return { url };
-    return { url: url.replace(/\/+$/, ''), title: cleanTitle };
+    return { url: trimTrailingChars(url, '/'), title: cleanTitle };
 }
 export function formatAddressDisplay(url, title, homeLabel = DEFAULT_HOME_NAVIGATION_ENTRY.title) {
     const parts = formatAddressDisplayParts(url, title, homeLabel);
@@ -81,7 +82,7 @@ export function isHistoryUrl(url) {
     return url !== EMPTY_URL && (isHttpLikeUrl(url) || /^file:\/\//i.test(url));
 }
 export function sameUrl(left, right) {
-    return left.replace(/\/+$/, '') === right.replace(/\/+$/, '');
+    return trimTrailingChars(left, '/') === trimTrailingChars(right, '/');
 }
 function cleanIconUrl(url) {
     const value = url?.trim();
@@ -98,11 +99,11 @@ export function isHistoryEntry(value) {
     if (typeof value !== 'object' || value == null || Array.isArray(value))
         return false;
     const record = value;
-    return (typeof record.url === 'string'
-        && typeof record.title === 'string'
-        && typeof record.lastVisitedAt === 'number'
-        && typeof record.visitCount === 'number'
-        && (record.iconUrl === undefined || typeof record.iconUrl === 'string'));
+    if (typeof record.url !== 'string' || typeof record.title !== 'string')
+        return false;
+    if (typeof record.lastVisitedAt !== 'number' || typeof record.visitCount !== 'number')
+        return false;
+    return record.iconUrl === undefined || typeof record.iconUrl === 'string';
 }
 /** Namespace + scope key -> storage key. Namespace defaults to a generic Jini prefix (see `constants.ts`); scope key is host-supplied (e.g. a project or tab id). */
 export function historyStorageKey(namespace, scopeKey) {
@@ -234,7 +235,8 @@ function navigationTitle({ url }, { title, homeLabel }) {
 function findRejoinTarget({ state, url }, { replacePendingTarget, pendingTarget }) {
     const { navigationStack: stack, navigationIndex: index } = state;
     const current = stack[index];
-    if (current && (sameUrl(current.url, url) || (replacePendingTarget && pendingTarget && sameUrl(current.url, pendingTarget)))) {
+    const replacingCurrent = current && replacePendingTarget && pendingTarget && sameUrl(current.url, pendingTarget);
+    if (current && (sameUrl(current.url, url) || replacingCurrent)) {
         return { index, entry: current };
     }
     return [index - 1, index + 1]

package/dist/features/charts/browser.js

@@ -111,7 +111,8 @@ export function mountChart({ element, spec, registry }, options = {}) {
         async update({ spec: nextSpec, theme = input.theme }, _options = {}) {
             if (destroyed)
                 return;
-            const updateRevision = ++revision;
+            revision += 1;
+            const updateRevision = revision;
             input = { spec: nextSpec, theme };
             showData();
             const preferred = candidates()[0];

package/dist/features/charts/contract.d.ts

@@ -15,12 +15,12 @@ export declare const chartSpecSchema: z.ZodObject<{
         color: z.ZodOptional<z.ZodString>;
     }, "strict", z.ZodTypeAny, {
         key: string;
-        label?: string | undefined;
         color?: string | undefined;
+        label?: string | undefined;
     }, {
         key: string;
-        label?: string | undefined;
         color?: string | undefined;
+        label?: string | undefined;
     }>, "many">;
     title: z.ZodOptional<z.ZodString>;
     description: z.ZodOptional<z.ZodString>;
@@ -122,8 +122,8 @@ export declare const chartSpecSchema: z.ZodObject<{
     data: Record<string, ChartValue>[];
     series: {
         key: string;
-        label?: string | undefined;
         color?: string | undefined;
+        label?: string | undefined;
     }[];
     title?: string | undefined;
     provider?: string | undefined;
@@ -154,8 +154,8 @@ export declare const chartSpecSchema: z.ZodObject<{
     data: Record<string, ChartValue>[];
     series: {
         key: string;
-        label?: string | undefined;
         color?: string | undefined;
+        label?: string | undefined;
     }[];
     title?: string | undefined;
     provider?: string | undefined;

package/dist/features/charts/react.js

@@ -6,7 +6,7 @@ import { InteractiveUiRegistry } from '../interactive-ui/registry.js';
 import { chartSpecSchema, DEFAULT_CHART_THEME } from './contract.js';
 import { mountChart } from './browser.js';
 import { chartCapability, chartProvidersByKind } from './registry.js';
-import { chartWireManifests, legacyChartSpec } from './wire.js';
+import { chartWireManifests, legacyChartSpec, withSeriesLabels } from './wire.js';
 /** React owns only the container; mountChart owns its descendants and lifecycle. */
 export function ChartHost({ spec, registry, theme = DEFAULT_CHART_THEME, siteProvider, onProviderError }) {
     const element = useRef(null);
@@ -34,10 +34,12 @@ export function ChartHost({ spec, registry, theme = DEFAULT_CHART_THEME, sitePro
 /** Known wire kinds use the established shapes. Other providers register spec-based entries without
  * adding fields or kinds to the core grammar. This closure is built once per host snapshot.
  */
+/** Wire kinds that predate spec-based entries and keep their established prop shapes. */
+const LEGACY_CHART_KINDS = ['bar', 'line', 'pie'];
 function chartComponent(kind, providers, options) {
     return function RegisteredChart(props) {
-        const legacyKind = kind === 'bar' ? 'bar' : kind === 'line' ? 'line' : kind === 'pie' ? 'pie' : undefined;
-        const spec = legacyKind ? legacyChartSpec({ kind: legacyKind, props }) : chartSpecSchema.parse(props.spec);
+        const legacyKind = LEGACY_CHART_KINDS.find((legacy) => legacy === kind);
+        const spec = withSeriesLabels({ spec: legacyKind ? legacyChartSpec({ kind: legacyKind, props }) : chartSpecSchema.parse(props.spec) });
         return _jsx(ChartHost, { spec: spec, registry: providers, ...options });
     };
 }

package/dist/features/charts/wire.d.ts

@@ -8,32 +8,44 @@ export declare const rechartsBarChartPropsSchema: z.ZodObject<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, "passthrough", z.ZodTypeAny, z.objectOutputType<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, z.ZodTypeAny, "passthrough">, z.objectInputType<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, z.ZodTypeAny, "passthrough">>;
 export declare const rechartsLineChartPropsSchema: z.ZodObject<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, "passthrough", z.ZodTypeAny, z.objectOutputType<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, z.ZodTypeAny, "passthrough">, z.objectInputType<{
     data: z.ZodArray<z.ZodRecord<z.ZodString, z.ZodUnknown>, "many">;
     categoryKey: z.ZodString;
     valueKey: z.ZodString;
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.ZodOptional<z.ZodString>;
     color: z.ZodOptional<z.ZodString>;
 }, z.ZodTypeAny, "passthrough">>;
 export declare const rechartsPieChartPropsSchema: z.ZodObject<{
@@ -85,4 +97,11 @@ export declare function legacyChartSpec({ kind, props }: {
     kind: 'bar' | 'line' | 'pie';
     props: Record<string, unknown>;
 }, _options?: Record<string, never>): ChartSpec;
+/**
+ * Give every series a label, so a provider's legend and tooltip never show a raw data key
+ * (owner, 2026-10-30: a sign-ups chart's legend read "signups"). A label the caller passed wins.
+ */
+export declare function withSeriesLabels({ spec }: {
+    spec: ChartSpec;
+}, _options?: Record<string, never>): ChartSpec;
 //# sourceMappingURL=wire.d.ts.map

package/dist/features/charts/wire.js

@@ -4,7 +4,9 @@ import { chartSpecSchema } from './contract.js';
 /** Same passthrough/record reasoning as native data-table: A2UI action bindings and extra row data
  * belong to the wire protocol, not a renderer. Retain their established shapes during extraction. */
 export const rechartsBarChartPropsSchema = z.object({
-    data: z.array(z.record(z.unknown())).min(1), categoryKey: z.string(), valueKey: z.string(), color: z.string().optional(),
+    data: z.array(z.record(z.unknown())).min(1), categoryKey: z.string(), valueKey: z.string(),
+    /** The legend/tooltip name for `valueKey` ("Sign-ups"); omitted, the key is humanized. */
+    valueLabel: z.string().min(1).optional().describe('Human name for valueKey shown in the legend and tooltip, e.g. "Sign-ups". Defaults to the humanized key.'), color: z.string().optional(),
 }).passthrough();
 export const rechartsLineChartPropsSchema = rechartsBarChartPropsSchema;
 export const rechartsPieChartPropsSchema = z.object({
@@ -31,8 +33,24 @@ export function legacyChartSpec({ kind, props }, _options = {}) {
     const parsed = (kind === 'pie' ? rechartsPieChartPropsSchema : rechartsBarChartPropsSchema).parse(props);
     return chartSpecSchema.parse({
         kind, data: parsed.data, categoryKey: kind === 'pie' ? 'name' : parsed.categoryKey,
-        series: [{ key: kind === 'pie' ? 'value' : parsed.valueKey, ...(parsed.color !== undefined ? { color: parsed.color } : {}) }],
+        series: [{
+                key: kind === 'pie' ? 'value' : parsed.valueKey,
+                ...('valueLabel' in parsed && typeof parsed.valueLabel === 'string' ? { label: parsed.valueLabel } : {}),
+                ...(parsed.color !== undefined ? { color: parsed.color } : {}),
+            }],
         ...(typeof parsed.provider === 'string' ? { provider: parsed.provider } : {}),
     });
 }
+/** `signups` -> `Signups`, `sign_ups`/`signUps` -> `Sign ups`. Display only; the key still addresses the data. */
+function humanizeSeriesKey(key) {
+    const words = key.replace(/([a-z0-9])([A-Z])/g, '$1 $2').replace(/[_-]+/g, ' ').trim().toLowerCase();
+    return words ? words[0].toUpperCase() + words.slice(1) : key;
+}
+/**
+ * Give every series a label, so a provider's legend and tooltip never show a raw data key
+ * (owner, 2026-10-30: a sign-ups chart's legend read "signups"). A label the caller passed wins.
+ */
+export function withSeriesLabels({ spec }, _options = {}) {
+    return { ...spec, series: spec.series.map((series) => ({ ...series, label: series.label ?? humanizeSeriesKey(series.key) })) };
+}
 //# sourceMappingURL=wire.js.map

package/dist/features/connectors/ConnectorsBrowser.js

@@ -67,6 +67,8 @@ export function ConnectorsBrowser({ unlocked, providerTabs = DEFAULT_PROVIDER_TA
         onCancelAuthorization: (connectorId) => void authorization.cancelAuthorization(connectorId),
         onOpenExternalUrl: (url) => void deps.data.openExternalUrl(url),
     };
-    return (_jsxs("div", { className: "tab-panel connectors-panel connectors-panel-embedded", children: [_jsxs("div", { className: "tab-panel-toolbar", children: [_jsx("div", { className: "toolbar-left connectors-heading", children: _jsxs("div", { children: [_jsx("h2", { children: t('Connectors') }), _jsx("p", { children: t('Connect third-party tools and services.') })] }) }), _jsxs("div", { className: "toolbar-right", children: [_jsx(ProviderTabBar, { tabs: providerTabs, selectedId: selectedProvider, onSelect: handleProviderTabSelect, ...subHandleProp({ base: agentHandle, action: 'provider' }) }), _jsx(ConnectorSearchBar, { value: filter, onChange: setFilter, disabled: !unlocked, onFocus: () => onProviderTabClick?.('search_connectors'), ...subHandleProp({ base: agentHandle, action: 'search' }) })] })] }), _jsx(ConnectorAlertList, { alerts: alerts, onOpenDetails: detail.openDetails }), catalog.loading ? (_jsx(CenteredLoader, { label: t('Loading…') })) : (_jsx(ConnectorGrid, { connectors: filteredConnectors, locked: !unlocked, hasNoResults: hasNoResults, searchQuery: filter, pendingConnectorAction: authorization.pendingConnectorAction, authorizationPending: authorization.pending, authorizationCancelFailed: authorization.cancelFailed, toolsLoaded: catalog.enriched, ...actions, onOpenDetails: detail.openDetails, ...(getCategoryLabel ? { getCategoryLabel } : {}), onClearSearch: () => setFilter(''), ...(gate ? { gate: { ...gate, onClick: handleGateClick } } : {}), ...subHandleProp({ base: agentHandle, action: 'connector' }) })), _jsx(ConnectorsDetailPane, { detail: detail, authorization: authorization, unlocked: unlocked, actions: actions, getCategoryLabel: getCategoryLabel, getDisplayableAccountLabel: getDisplayableAccountLabel, agentHandle: agentHandle })] }));
+    const categoryLabelProps = getCategoryLabel ? { getCategoryLabel } : {};
+    const gateProps = gate ? { gate: { ...gate, onClick: handleGateClick } } : {};
+    return (_jsxs("div", { className: "tab-panel connectors-panel connectors-panel-embedded", children: [_jsxs("div", { className: "tab-panel-toolbar", children: [_jsx("div", { className: "toolbar-left connectors-heading", children: _jsxs("div", { children: [_jsx("h2", { children: t('Connectors') }), _jsx("p", { children: t('Connect third-party tools and services.') })] }) }), _jsxs("div", { className: "toolbar-right", children: [_jsx(ProviderTabBar, { tabs: providerTabs, selectedId: selectedProvider, onSelect: handleProviderTabSelect, ...subHandleProp({ base: agentHandle, action: 'provider' }) }), _jsx(ConnectorSearchBar, { value: filter, onChange: setFilter, disabled: !unlocked, onFocus: () => onProviderTabClick?.('search_connectors'), ...subHandleProp({ base: agentHandle, action: 'search' }) })] })] }), _jsx(ConnectorAlertList, { alerts: alerts, onOpenDetails: detail.openDetails }), catalog.loading ? (_jsx(CenteredLoader, { label: t('Loading…') })) : (_jsx(ConnectorGrid, { connectors: filteredConnectors, locked: !unlocked, hasNoResults: hasNoResults, searchQuery: filter, pendingConnectorAction: authorization.pendingConnectorAction, authorizationPending: authorization.pending, authorizationCancelFailed: authorization.cancelFailed, toolsLoaded: catalog.enriched, ...actions, onOpenDetails: detail.openDetails, ...categoryLabelProps, onClearSearch: () => setFilter(''), ...gateProps, ...subHandleProp({ base: agentHandle, action: 'connector' }) })), _jsx(ConnectorsDetailPane, { detail: detail, authorization: authorization, unlocked: unlocked, actions: actions, getCategoryLabel: getCategoryLabel, getDisplayableAccountLabel: getDisplayableAccountLabel, agentHandle: agentHandle })] }));
 }
 //# sourceMappingURL=ConnectorsBrowser.js.map

package/dist/features/connectors/hooks/useConnectorAuthorization.js

@@ -176,7 +176,7 @@ export function useConnectorAuthorization(port, authPendingStorage, authBridge,
     // Poll statuses while any authorization is in flight.
     useEffect(() => {
         if (Object.keys(pending).length === 0)
-            return;
+            return undefined;
         const interval = setInterval(() => {
             setPending((curr) => pruneConnectorAuthorizationPending(curr, Date.now()));
             void reloadStatuses();

package/dist/features/connectors/hooks/useConnectorCatalog.js

@@ -42,10 +42,10 @@ export function useConnectorCatalog(port, options) {
         if (!unlocked || !port.fetchConnectorEnrichment) {
             setEnriched(false);
             setEnriching(false);
-            return;
+            return undefined;
         }
         if (enriched)
-            return;
+            return undefined;
         let cancelled = false;
         setEnriching(true);
         void (async () => {

package/dist/features/connectors/rules.js

@@ -1,3 +1,4 @@
+import { omitKeys } from '../../utils/omit-keys.js';
 /** Restricts the catalog to a single provider tab's `match` predicate, defaulting to the first tab. */
 export function scopeConnectorsToProvider(connectors, providerTabs, selectedProviderId) {
     const tab = providerTabs.find((p) => p.id === selectedProviderId) ?? providerTabs[0];
@@ -176,8 +177,7 @@ export function applyConnectorStatuses(current, statuses) {
         const next = statuses[connector.id];
         if (!next)
             return connector;
-        const { accountLabel: _accountLabel, lastError: _lastError, ...base } = connector;
-        return { ...base, ...next };
+        return { ...omitKeys(connector, ['accountLabel', 'lastError']), ...next };
     });
 }
 export function getConnectorDisplayToolCount(connector) {
@@ -214,8 +214,7 @@ export function mergeConnectorToolPreview(current, next, append) {
     };
     if (next.toolsNextCursor !== undefined)
         return { ...merged, toolsNextCursor: next.toolsNextCursor };
-    const { toolsNextCursor: _toolsNextCursor, ...withoutCursor } = merged;
-    return withoutCursor;
+    return omitKeys(merged, ['toolsNextCursor']);
 }
 export function mergeConnectorActionResult(current, next) {
     const toolCount = next.toolCount ?? current.toolCount;
@@ -355,17 +354,14 @@ export function toolsBadgeTranslation(count) {
         return { key: '1 tool' };
     return { key: '{count} tools', vars: { count } };
 }
+const STATUS_LABELS = {
+    available: 'Available',
+    connected: 'Connected',
+    error: 'Error',
+    disabled: 'Disabled',
+};
 export function statusLabel(status) {
-    switch (status) {
-        case 'available':
-            return 'Available';
-        case 'connected':
-            return 'Connected';
-        case 'error':
-            return 'Error';
-        case 'disabled':
-            return 'Disabled';
-    }
+    return STATUS_LABELS[status];
 }
 export function defaultCategoryLabel(category) {
     return category;

package/dist/features/execution/constants.js

@@ -70,7 +70,6 @@ export const DEFAULT_AGENT_DESCRIPTIONS = {
     'cursor-agent': 'Cursor command line',
     opencode: 'Open-source agent CLI',
     qwen: 'Qwen coding CLI',
-    copilot: 'GitHub coding CLI',
     devin: 'Cognition terminal CLI',
     kimi: 'Moonshot Kimi CLI',
     qoder: 'Alibaba coding CLI',

package/dist/features/execution/react/components/AgentDiagnosticRow.js

@@ -2,6 +2,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { useT } from '../../../i18n/index.js';
 import { agentDiagnosticTooltip } from '../../rules.js';
 import { Icon } from '../../../../react/components/Icon.js';
+import { unhandledCase } from '../../../../utils/unhandled-case.js';
 /**
  * Maps one typed fix intent to a concrete icon button. `'setEnv'`/
  * `'clearEnv'`/`'launchOAuth'` intentionally resolve to no button — same as
@@ -31,6 +32,7 @@ function resolveAction(intent, handlers, labels) {
         case 'launchOAuth':
             return null;
     }
+    return unhandledCase(intent);
 }
 /**
  * Presents a single agent diagnostic as "one-line reason + fix button(s)".

package/dist/features/execution/react/components/ByokProviderForm.js

@@ -133,9 +133,7 @@ function ByokConnectionResult({ connectionTest, canTestConnection }) {
         return null;
     if (connectionTest.status !== 'ok' && connectionTest.status !== 'error')
         return null;
-    return (_jsx("div", { className: "jini-byok-test-row", children: _jsx("span", { className: `jini-byok-test-status is-${connectionTest.status}`, role: connectionTest.status === 'error' ? 'alert' : 'status', children: connectionTest.status === 'ok'
-                ? (connectionTest.message !== undefined ? t(connectionTest.message) : t('Connection succeeded'))
-                : t(connectionTest.message) }) }));
+    return (_jsx("div", { className: "jini-byok-test-row", children: _jsx("span", { className: `jini-byok-test-status is-${connectionTest.status}`, role: connectionTest.status === 'error' ? 'alert' : 'status', children: t(connectionTest.message ?? 'Connection succeeded') }) }));
 }
 /** Discovery failure, as an inline hint below the Model row. Renders nothing otherwise. */
 function ByokDiscoveryError({ modelDiscovery }) {

package/dist/features/execution/react/components/LocalCliAgentSettings.js

@@ -20,7 +20,8 @@ function LocalCliAgentModelControls({ agent, card, onModelChange, agentHandle })
     const { reasoning, source, sourceLabel, sourceHint } = card;
     const { hasModels, hasReasoning, reasoningOptions, reasoningValue, reasoningDisabled, allowCustomModel, customActive, modelChoices, selectValue, customModelInputValue, } = reasoning;
     const additionalOptions = allowCustomModel ? [{ value: CUSTOM_MODEL_SENTINEL, label: t('Custom…') }] : undefined;
-    return (_jsxs("div", { className: "jini-agent-settings-row", children: [hasModels ? (_jsxs("div", { className: "jini-agent-settings-field jini-agent-settings-model", children: [_jsxs("div", { className: "jini-agent-settings-label", children: [_jsx("span", { className: "jini-field-label", children: t('Model') }), sourceLabel ? (_jsx("span", { className: `jini-agent-model-source ${source}`, children: sourceLabel })) : null] }), _jsx(SearchableModelSelect, { className: "jini-input jini-agent-model-select", ariaLabel: t('Model'), searchPlaceholder: t('Search models'), testId: `jini-agent-model-${agent.id}`, searchInputTestId: `jini-agent-model-search-${agent.id}`, ...(agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'model' }) } : {}), value: selectValue, models: modelChoices, additionalOptions: additionalOptions, onChange: card.selectModel }), sourceHint ? _jsx("p", { className: "jini-field-hint", children: sourceHint }) : null] })) : null, customActive ? (_jsxs("label", { className: "jini-agent-settings-field", children: [_jsx("span", { className: "jini-field-label", children: t('Custom model id') }), _jsx("input", { className: "jini-input jini-agent-settings-custom", type: "text", "data-testid": `jini-agent-model-custom-${agent.id}`, value: customModelInputValue, placeholder: t('e.g. my-fine-tuned-model'), spellCheck: false, onChange: (event) => onModelChange(agent.id, event.target.value.trim()), ...agentHandleProps({}, { base: agentHandle, action: 'custom-model', role: 'field', label: t('Custom model id') }) })] })) : null, hasReasoning ? (_jsxs("label", { className: "jini-agent-settings-field", children: [_jsx("span", { className: "jini-field-label", children: t('Reasoning effort') }), _jsx("select", { className: "jini-input jini-agent-settings-reasoning", "data-testid": `jini-agent-reasoning-${agent.id}`, value: reasoningValue, disabled: reasoningDisabled, onChange: (event) => card.selectReasoning(event.target.value), ...agentHandleProps({}, { base: agentHandle, action: 'reasoning', role: 'field', label: t('Reasoning effort') }), children: reasoningOptions.map((option) => (_jsx("option", { value: option.id, children: option.label }, option.id))) })] })) : null] }));
+    const modelHandleProps = agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'model' }) } : {};
+    return (_jsxs("div", { className: "jini-agent-settings-row", children: [hasModels ? (_jsxs("div", { className: "jini-agent-settings-field jini-agent-settings-model", children: [_jsxs("div", { className: "jini-agent-settings-label", children: [_jsx("span", { className: "jini-field-label", children: t('Model') }), sourceLabel ? (_jsx("span", { className: `jini-agent-model-source ${source}`, children: sourceLabel })) : null] }), _jsx(SearchableModelSelect, { className: "jini-input jini-agent-model-select", ariaLabel: t('Model'), searchPlaceholder: t('Search models'), testId: `jini-agent-model-${agent.id}`, searchInputTestId: `jini-agent-model-search-${agent.id}`, ...modelHandleProps, value: selectValue, models: modelChoices, additionalOptions: additionalOptions, onChange: card.selectModel }), sourceHint ? _jsx("p", { className: "jini-field-hint", children: sourceHint }) : null] })) : null, customActive ? (_jsxs("label", { className: "jini-agent-settings-field", children: [_jsx("span", { className: "jini-field-label", children: t('Custom model id') }), _jsx("input", { className: "jini-input jini-agent-settings-custom", type: "text", "data-testid": `jini-agent-model-custom-${agent.id}`, value: customModelInputValue, placeholder: t('e.g. my-fine-tuned-model'), spellCheck: false, onChange: (event) => onModelChange(agent.id, event.target.value.trim()), ...agentHandleProps({}, { base: agentHandle, action: 'custom-model', role: 'field', label: t('Custom model id') }) })] })) : null, hasReasoning ? (_jsxs("label", { className: "jini-agent-settings-field", children: [_jsx("span", { className: "jini-field-label", children: t('Reasoning effort') }), _jsx("select", { className: "jini-input jini-agent-settings-reasoning", "data-testid": `jini-agent-reasoning-${agent.id}`, value: reasoningValue, disabled: reasoningDisabled, onChange: (event) => card.selectReasoning(event.target.value), ...agentHandleProps({}, { base: agentHandle, action: 'reasoning', role: 'field', label: t('Reasoning effort') }), children: reasoningOptions.map((option) => (_jsx("option", { value: option.id, children: option.label }, option.id))) })] })) : null] }));
 }
 /** Probe evidence and repair actions stay together beneath the model controls. */
 function LocalCliAgentFeedback({ agent, card, onEnvChange }) {

package/dist/features/execution/react/hooks/use-byok-field-validation.hooks.js

@@ -14,8 +14,12 @@ export function useByokFieldValidation(required, optional = {}) {
     return {
         missing,
         baseUrlInvalid,
-        isMissing: ({ field }, _options = {}) => (missing.has(field) || (field === 'baseUrl' && baseUrlInvalid)) &&
-            (!optional.deferMissingFeedback || (touched.providerId === config.providerId && touched.fields.has(field))),
+        isMissing: ({ field }, _options = {}) => {
+            const invalid = missing.has(field) || (field === 'baseUrl' && baseUrlInvalid);
+            if (!invalid || !optional.deferMissingFeedback)
+                return invalid;
+            return touched.providerId === config.providerId && touched.fields.has(field);
+        },
         markTouched: ({ field }, _options = {}) => {
             setTouched(current => ({
                 providerId: config.providerId,

package/dist/features/execution/react/hooks/useExecutionTab.js

@@ -48,7 +48,8 @@ export function useExecutionTab({ port, autoDetect = true, describeProbeError =
     }, []);
     const runDetection = useCallback((detect) => {
         clearTimeout(detectionTimer.current);
-        const ticket = ++detectionTicket.current;
+        detectionTicket.current += 1;
+        const ticket = detectionTicket.current;
         const isCurrent = () => alive.current && detectionTicket.current === ticket;
         setScan({ status: 'scanning' });
         detect().then((found) => {
@@ -60,7 +61,7 @@ export function useExecutionTab({ port, autoDetect = true, describeProbeError =
             if (found.some(agent => agent.discoveryPending)) {
                 detectionTimer.current = setTimeout(() => {
                     if (isCurrent())
-                        runDetection(() => port.detectLocalAgents());
+                        runDetection(port.detectLocalAgents.bind(port));
                 }, 1000);
             }
         }, (error) => {
@@ -81,7 +82,8 @@ export function useExecutionTab({ port, autoDetect = true, describeProbeError =
         runDetection(() => rescanFn.call(port));
     }, [port, runDetection]);
     const testConnection = useCallback((config) => {
-        const ticket = ++connectionTestTicket.current;
+        connectionTestTicket.current += 1;
+        const ticket = connectionTestTicket.current;
         const isCurrent = () => alive.current && connectionTestTicket.current === ticket;
         setConnectionTest({ status: 'testing' });
         port.testConnection(config).then((result) => {
@@ -100,7 +102,8 @@ export function useExecutionTab({ port, autoDetect = true, describeProbeError =
         const run = port.testAgent;
         if (!run)
             return;
-        const ticket = ++agentTestTicket.current;
+        agentTestTicket.current += 1;
+        const ticket = agentTestTicket.current;
         const isCurrent = () => alive.current && agentTestTicket.current === ticket;
         setAgentTest({ status: 'testing', agentId });
         run.call(port, agentId, model).then((result) => {
@@ -128,7 +131,8 @@ export function useExecutionTab({ port, autoDetect = true, describeProbeError =
         const listModels = port.listModels;
         if (!listModels)
             return;
-        const ticket = ++modelDiscoveryTicket.current;
+        modelDiscoveryTicket.current += 1;
+        const ticket = modelDiscoveryTicket.current;
         const isCurrent = () => alive.current && modelDiscoveryTicket.current === ticket;
         setModelDiscovery({ status: 'loading' });
         listModels.call(port, config).then((found) => {

package/dist/features/execution/react/hooks/useLocalCliAgentCard.js

@@ -53,14 +53,11 @@ export function useLocalCliAgentCard({ agent, config, agentTest, cliEnvFields, o
         }
         onReasoningChange(agent.id, nextValue);
     };
-    const hasSettings = reasoning.hasModels ||
-        reasoning.hasReasoning ||
-        Boolean(onTest) ||
-        result !== null ||
-        cliEnvFieldsForAgent(cliEnvFields, agent.id).length > 0;
+    const hasSettings = reasoning.hasModels || reasoning.hasReasoning || Boolean(onTest) || result !== null;
+    const hasEnvironmentSettings = cliEnvFieldsForAgent(cliEnvFields, agent.id).length > 0;
     return {
         reasoning,
-        hasSettings,
+        hasSettings: hasSettings || hasEnvironmentSettings,
         source,
         sourceLabel,
         sourceHint,

package/dist/features/execution/react/hooks/useReasoningControl.js

@@ -1,7 +1,7 @@
 import { useCallback, useEffect } from 'react';
 import { CUSTOM_MODEL_SENTINEL } from '../../constants.js';
 import { modelIdForReasoningLevel, pickerModelOptions, reasoningModelGroupFor, reasoningModelGroups, selectedAgentModel, selectedAgentReasoning, shouldShowCustomModelInput, splitReasoningModelId, } from '../../rules.js';
-export function useReasoningControl(optionsOrAgent, maybeConfig, maybeExplicitCustomMode) {
+export function useReasoningControl(optionsOrAgent, maybeConfig, maybeExplicitCustomMode = false) {
     const { agent, config, explicitCustomMode = false, onReasoningChange } = reasoningControlOptions(optionsOrAgent, maybeConfig, maybeExplicitCustomMode);
     const models = agent.models ?? [];
     const hasModels = models.length > 0;
@@ -23,7 +23,7 @@ export function useReasoningControl(optionsOrAgent, maybeConfig, maybeExplicitCu
             onReasoningChange('');
         // `reasoningOptionIdsKey` stands in for `reasoningOptions` (a fresh array every render) so this
         // only re-runs when the narrowed vocabulary actually changes, not on every unrelated re-render.
-        // eslint-disable-next-line react-hooks/exhaustive-deps
+        // Dependency list intentionally follows the lifecycle rationale above.
     }, [flatReasoningOptions.length, onReasoningChange, config.reasoningByAgentId, agent.id, reasoningOptionIdsKey]);
     const { allowCustomModel, knownModelIds, customActive } = customModelState({ agent, modelGroups, models, resolvedModel, explicitCustomMode });
     // A model-suffix agent picks a BASE model here; the effort control below supplies the rest of the
@@ -83,7 +83,7 @@ export function useReasoningControl(optionsOrAgent, maybeConfig, maybeExplicitCu
 }
 /** The two call shapes (options object, or the older positional `agent, config, explicitCustomMode`)
  *  as one options object. */
-function reasoningControlOptions(optionsOrAgent, maybeConfig, maybeExplicitCustomMode) {
+function reasoningControlOptions(optionsOrAgent, maybeConfig, maybeExplicitCustomMode = false) {
     if ('agent' in optionsOrAgent && 'config' in optionsOrAgent)
         return optionsOrAgent;
     return {

package/dist/features/execution/rules.js

@@ -118,10 +118,10 @@ function foreignKeyOwner(key, preset, presets) {
         const prefix = candidate.apiKeyPrefix;
         if (!prefix || candidate.custom || candidate.id === preset.id)
             continue;
-        if (prefix.length <= bestClaim || !key.startsWith(prefix))
-            continue;
-        owner = candidate;
-        bestClaim = prefix.length;
+        if (prefix.length > bestClaim && key.startsWith(prefix)) {
+            owner = candidate;
+            bestClaim = prefix.length;
+        }
     }
     return owner;
 }
@@ -328,9 +328,9 @@ export function cleanAgentVersionLabel(label, version) {
  * binary path, so the card stays one line without discarding either.
  */
 export function agentMetaLabel(agent, labels) {
-    const title = (agent.authStatus === 'missing' || agent.authStatus === 'unknown'
-        ? (agent.authMessage ?? agent.path ?? '')
-        : (agent.path ?? ''));
+    const authUnresolved = agent.authStatus === 'missing' || agent.authStatus === 'unknown';
+    const pathTitle = agent.path ?? '';
+    const title = authUnresolved ? (agent.authMessage ?? pathTitle) : pathTitle;
     if (!agent.installed)
         return { text: labels.notInstalled, title };
     if (agent.authStatus === 'missing')
@@ -505,10 +505,11 @@ export function pickerModelOptions({ models }, _optional = {}) {
         const existing = choices.get(id);
         if (!existing) {
             choices.set(id, option);
+            continue;
         }
-        else if (existing.label === id && label !== id) {
+        // A later duplicate only upgrades an entry whose label was just its id.
+        if (existing.label === id && label !== id)
             choices.set(id, { ...existing, label });
-        }
     }
     return [...choices.values()];
 }
@@ -578,21 +579,25 @@ export function splitReasoningModelId(modelId, levelIds) {
  * has no level, so nothing is stripped, and a vendor label that happens to end in an unrelated
  * parenthetical keeps it.
  */
-/** `Head (Inner)` with optional surrounding whitespace; group 1 is the head, group 2 the qualifier. */
-const TRAILING_PARENTHETICAL = /^(.*?)\s*\(([^()]*)\)\s*$/;
 function baseLabelForLevel(label, level) {
     if (!level)
         return label;
-    if (!TRAILING_PARENTHETICAL.test(label))
+    const trimmed = label.trimEnd();
+    if (!trimmed.endsWith(')'))
+        return label;
+    const open = trimmed.lastIndexOf('(');
+    if (open < 0)
+        return label;
+    const head = trimmed.slice(0, open).trimEnd();
+    const qualifier = trimmed.slice(open + 1, -1);
+    // A matched terminal group always supplies both parts; the old capture fallbacks were unreachable.
+    // The original head's dot did not span line breaks, and the qualifier excluded parentheses.
+    if (/[\r\n\u2028\u2029]/.test(head) || qualifier.includes(')'))
         return label;
-    // Both capture groups always participate once the pattern matches, so reading them through
-    // `replace` (which yields a plain string) replaces the old `match[n] ?? ''` fallbacks, which no
-    // matching label could ever reach.
-    const inner = label.replace(TRAILING_PARENTHETICAL, '$2').trim().toLowerCase();
+    const inner = qualifier.trim().toLowerCase();
     if (inner !== level.id.toLowerCase() && inner !== level.label.toLowerCase())
         return label;
-    const stripped = label.replace(TRAILING_PARENTHETICAL, '$1').trim();
-    return stripped || label;
+    return head.trim() || label;
 }
 /**
  * Groups an agent's model list by base model, recording for each base the effort levels it

package/dist/features/external-mcp/react/components/ExternalMcpTab.js

@@ -48,7 +48,16 @@ export function ExternalMcpTab({ dependencies, fieldSpecs = MCP_SOURCE_FIELD_SPE
     // A real load failure always wins over the host's static/decorative banner
     // text — showing both would just be two banners saying overlapping things.
     const banner = list.error ?? connectionError;
-    return (_jsxs("section", { className: "external-mcp-tab", children: [_jsxs("div", { className: "external-mcp-head", children: [_jsxs("div", { children: [_jsx("h3", { children: t('External MCP servers') }), _jsx("p", { className: "external-mcp-subtitle", children: t('Third-party tools for your coding agent.') })] }), _jsxs("button", { type: "button", className: "external-mcp-add-button", onClick: () => setFormOpen((open) => !open), "aria-expanded": formOpen, children: [_jsx(Icon, { name: "plus", size: 14 }), _jsx("span", { children: t('Add server') })] })] }), banner ? (_jsx("div", { className: "external-mcp-banner", role: "alert", children: t(banner) })) : null, formOpen ? (_jsx(SourceConfigAddForm, { fieldSpecs: fieldSpecs, values: addForm.values, validation: addForm.validation, submitAttempted: addForm.submitAttempted, submitting: addForm.submitting, submitError: addForm.submitError, onFieldChange: addForm.setField, onTrustChange: addForm.setTrust, onSubmit: () => void addForm.submit(), canTest: list.capabilities.canTest, testing: list.isPending(DRAFT_TEST_SCOPE, 'test'), ...(list.testResults[DRAFT_TEST_SCOPE] ? { testResult: list.testResults[DRAFT_TEST_SCOPE] } : {}), onTest: () => void list.test(undefined, addForm.values), addLabel: "Add server" })) : null, list.loading ? (_jsx("div", { className: "source-config-list-loading", role: "status", children: t('Loading…') })) : list.sources.length === 0 ? (_jsxs("div", { className: "external-mcp-empty", children: [_jsx("p", { className: "external-mcp-empty-title", children: t('No MCP servers configured.') }), _jsx("p", { className: "external-mcp-empty-hint", children: t('Click "Add server" to get started — pick a template or set up a custom stdio / HTTP server.') })] })) : (_jsx("div", { className: "source-config-list-items", children: list.sources.map((source) => (_jsx(SourceConfigItemCard, { source: source, fieldSpecs: fieldSpecs, ...(trustOptions ? { trustOptions } : {}), capabilities: list.capabilities, removing: list.isPending(source.id, 'remove'), refreshing: list.isPending(source.id, 'refresh'), settingTrust: list.isPending(source.id, 'trust'), testing: list.isPending(source.id, 'test'), updating: list.isPending(source.id, 'update'), onRefresh: () => void list.refresh(source.id), onRemove: () => void list.remove(source.id), onTrustChange: (trust) => void list.setTrust(source.id, trust), onTest: () => void list.test(source.id), onUpdate: (patch) => void list.update(source.id, patch), ...(list.testResults[source.id] ? { testResult: list.testResults[source.id] } : {}) }, source.id))) })), _jsx(ExternalMcpFooter, { saveStatusLabel: saveStatusLabel, configPath: configPath })] }));
+    const draftTestResultProps = list.testResults[DRAFT_TEST_SCOPE] ? { testResult: list.testResults[DRAFT_TEST_SCOPE] } : {};
+    return (_jsxs("section", { className: "external-mcp-tab", children: [_jsxs("div", { className: "external-mcp-head", children: [_jsxs("div", { children: [_jsx("h3", { children: t('External MCP servers') }), _jsx("p", { className: "external-mcp-subtitle", children: t('Third-party tools for your coding agent.') })] }), _jsxs("button", { type: "button", className: "external-mcp-add-button", onClick: () => setFormOpen((open) => !open), "aria-expanded": formOpen, children: [_jsx(Icon, { name: "plus", size: 14 }), _jsx("span", { children: t('Add server') })] })] }), banner ? (_jsx("div", { className: "external-mcp-banner", role: "alert", children: t(banner) })) : null, formOpen ? (_jsx(SourceConfigAddForm, { fieldSpecs: fieldSpecs, values: addForm.values, validation: addForm.validation, submitAttempted: addForm.submitAttempted, submitting: addForm.submitting, submitError: addForm.submitError, onFieldChange: addForm.setField, onTrustChange: addForm.setTrust, onSubmit: () => void addForm.submit(), canTest: list.capabilities.canTest, testing: list.isPending(DRAFT_TEST_SCOPE, 'test'), ...draftTestResultProps, onTest: () => void list.test(undefined, addForm.values), addLabel: "Add server" })) : null, list.loading ? (_jsx("div", { className: "source-config-list-loading", role: "status", children: t('Loading…') })) : (_jsx(ExternalMcpSources, { list: list, fieldSpecs: fieldSpecs, trustOptions: trustOptions })), _jsx(ExternalMcpFooter, { saveStatusLabel: saveStatusLabel, configPath: configPath })] }));
+}
+/** The configured servers, or the empty-state hint when there are none yet. */
+function ExternalMcpSources({ list, fieldSpecs, trustOptions, }) {
+    const t = useT();
+    if (list.sources.length === 0) {
+        return (_jsxs("div", { className: "external-mcp-empty", children: [_jsx("p", { className: "external-mcp-empty-title", children: t('No MCP servers configured.') }), _jsx("p", { className: "external-mcp-empty-hint", children: t('Click "Add server" to get started — pick a template or set up a custom stdio / HTTP server.') })] }));
+    }
+    return (_jsx("div", { className: "source-config-list-items", children: list.sources.map((source) => (_jsx(SourceConfigItemCard, { source: source, fieldSpecs: fieldSpecs, ...(trustOptions ? { trustOptions } : {}), capabilities: list.capabilities, removing: list.isPending(source.id, 'remove'), refreshing: list.isPending(source.id, 'refresh'), settingTrust: list.isPending(source.id, 'trust'), testing: list.isPending(source.id, 'test'), updating: list.isPending(source.id, 'update'), onRefresh: () => void list.refresh(source.id), onRemove: () => void list.remove(source.id), onTrustChange: (trust) => void list.setTrust(source.id, trust), onTest: () => void list.test(source.id), onUpdate: (patch) => void list.update(source.id, patch), ...(list.testResults[source.id] ? { testResult: list.testResults[source.id] } : {}) }, source.id))) }));
 }
 /** Renders the save-status footer as one unit; a path alone does not show it. */
 function ExternalMcpFooter({ saveStatusLabel, configPath }) {

package/dist/features/file-dropzone/rules.js

@@ -3,6 +3,7 @@
  * DOM-shaped value (`File`, event targets) arrives as a parameter.
  */
 import { FILE_DROPZONE_KIND_BY_EXTENSION } from './constants.js';
+import { trimChars } from '../../utils/trim-chars.js';
 /** Lowercased extension (no dot), or `''` for an extensionless name. */
 export function fileDropzoneExtension(file) {
     return (/\.([a-z0-9]+)$/i.exec(file.name)?.[1] ?? '').toLowerCase();
@@ -29,7 +30,7 @@ export function fileDropzoneKind(file) {
 }
 /** Kinds renderable straight from an object URL (`blob:`). Fonts also need one (`FontFace` loads from a URL); text is read via `FileReader`/`Blob.text()` instead. */
 export function fileDropzoneNeedsObjectUrl(kind) {
-    return kind === 'image' || kind === 'video' || kind === 'audio' || kind === 'pdf' || kind === 'html' || kind === 'font';
+    return ['image', 'video', 'audio', 'pdf', 'html', 'font'].includes(kind);
 }
 /** Uppercased, 4-char-capped extension for the glyph badge (e.g. `"a/b.jpeg"` -> `"JPEG"`, extensionless -> `"FILE"`). */
 export function fileDropzoneExtensionLabel(file) {
@@ -50,7 +51,7 @@ export function fileDropzoneStagingKey(file) {
 }
 /** CSS-safe, collision-resistant `@font-face` family name for a staged font at `index`. */
 export function fileDropzoneFontFamilyName(file, index) {
-    const slug = file.name.replace(/[^a-z0-9]+/gi, '-').replace(/^-+|-+$/g, '').slice(0, 24);
+    const slug = trimChars(file.name.replace(/[^a-z0-9]+/gi, '-'), '-').slice(0, 24);
     return `jini-file-dropzone-font-${index}-${slug || 'font'}`;
 }
 /** Whether a selection is large enough (by count or total bytes) to warrant a loading affordance while it's staged. */

package/dist/features/folder-path-drop/rules.d.ts

@@ -24,9 +24,8 @@ import type { CaptureFolderPathDropInput } from './types.js';
  */
 export declare function folderPathsFromDataTransfer(dataTransfer: DataTransfer, port: FolderPathDropPort): string[];
 /**
- * The text a folder drop inserts: every path, space-joined, in drop order. The one place the format
- * lives — a path that itself contains a space is ambiguous in this format, so any change to quoting
- * or separators belongs here and nowhere else.
+ * The text a folder drop inserts: every path, space-joined, in drop order. Quote whitespace-containing
+ * paths so text consumers can distinguish the path from adjacent prose or secrets.
  *
  * @complexity Time/space: O(total path length).
  */