@jini-ai/ui 0.6.15-rc.1 → 0.6.15

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/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/html-editor/canvas-embed-placeholders.js

@@ -227,9 +227,8 @@ export function applyCanvasEmbedPlaceholders(body, describe) {
         if (existingCard && !canRefreshRuntimeCard({ el, card: existingCard }, {}))
             continue;
         const descriptor = describe(el);
-        if (!descriptor)
-            continue;
-        updatePlaceholder({ el, doc, descriptor, existingCard }, {});
+        if (descriptor)
+            updatePlaceholder({ el, doc, descriptor, existingCard }, {});
     }
 }
 function canRefreshRuntimeCard({ el, card }, _options = {}) {

package/dist/features/html-editor/css.js

@@ -1,3 +1,4 @@
+import { trimTrailingChars } from '../../utils/trim-chars.js';
 /**
  * `editor.getCss()` always returns single-line, semicolon-packed CSS — confirmed directly in
  * GrapesJS's own bundled source (`CssRule.prototype.toCSS`/`getDeclaration`): there is no
@@ -65,7 +66,7 @@ export function prettifyCss(css) {
         return false;
     };
     const openRule = (_input, _options = {}) => {
-        out = out.replace(/[ \t]+$/, '');
+        out = trimTrailingChars(out, ' \t');
         out += ' {';
         depth++;
         atDeclStart = true;
@@ -77,7 +78,7 @@ export function prettifyCss(css) {
     };
     const closeRule = (_input, _options = {}) => {
         depth = Math.max(0, depth - 1);
-        out = out.replace(/[ \t]+$/, '');
+        out = trimTrailingChars(out, ' \t');
         // A `}` immediately after `{` (an empty rule) needs no newline before it; otherwise close on
         // its own indented line.
         if (!out.endsWith('{'))
@@ -137,10 +138,11 @@ export function prettifyCss(css) {
             out += ' ';
             i++;
             skipWhitespace({}, {});
-            continue;
         }
-        out += ch;
-        i++;
+        else {
+            out += ch;
+            i++;
+        }
     }
     return out.trim();
 }

package/dist/features/html-editor/editor-blocks.js

@@ -34,11 +34,54 @@ const RUNTIME_EFFECTS = new Set([
 const TEXT_EFFECT_TAGS = new Set(['h1', 'h2', 'h3', 'h4', 'h5', 'h6', 'p', 'li', 'dt', 'dd', 'blockquote', 'figcaption', 'label']);
 const PHRASING_TAGS = new Set([...INLINE_TAGS, 'abbr', 'br', 'cite', 'code', 'data', 'dfn', 'img', 'kbd', 'q', 'samp', 'time', 'var', 'wbr']);
 const isPhrasingOnly = (el) => el.children.every(child => PHRASING_TAGS.has(child.tagName) && isPhrasingOnly(child));
+/** Closing quotes for each suffix, honoring the same backslash escapes as effect string literals.
+ * A failed quoted token can contain another valid token; suffix lookup preserves that recovery
+ * without repeatedly scanning an unterminated string. O(n) time and temporary memory. */
+function effectQuoteEnds({ text, quote }, _options = {}) {
+    const ends = new Int32Array(text.length + 2).fill(-1);
+    for (let i = text.length - 1; i >= 0; i--) {
+        const char = text[i];
+        if (char === quote)
+            ends[i] = i;
+        else if (char === '\\' && /[\r\n\u2028\u2029]/.test(text[i + 1] ?? ''))
+            ends[i] = -1;
+        else
+            ends[i] = ends[i + (char === '\\' ? 2 : 1)];
+    }
+    return ends;
+}
+/** Remove complete quoted strings and parenthesized arguments before splitting effect names.
+ * The first closing parenthesis wins, as in the existing effect grammar. O(n) time and memory. */
+function withoutEffectArguments({ text }, _options = {}) {
+    const double = effectQuoteEnds({ text, quote: '"' });
+    const single = effectQuoteEnds({ text, quote: "'" });
+    const lastClose = text.lastIndexOf(')');
+    const parts = [];
+    let start = 0;
+    let i = 0;
+    while (i < text.length) {
+        let end = -1;
+        if (text[i] === '"')
+            end = double[i + 1];
+        if (text[i] === "'")
+            end = single[i + 1];
+        if (text[i] === '(' && i < lastClose)
+            end = text.indexOf(')', i + 1);
+        if (end >= 0) {
+            parts.push(text.slice(start, i));
+            i = end;
+            start = end + 1;
+        }
+        i++;
+    }
+    parts.push(text.slice(start));
+    return parts.join('');
+}
 const isEditableEffect = (el) => {
     if (el.attributes['data-editor'] === 'block')
         return false;
     // Quoted selectors and timing/color functions may contain commas, which are not effect separators.
-    const effects = (el.attributes['data-kui'] ?? '').replace(/"(?:\\.|[^"\\])*"|'(?:\\.|[^'\\])*'|\([^)]*\)/g, '')
+    const effects = withoutEffectArguments({ text: el.attributes['data-kui'] ?? '' })
         .split(',').map(segment => segment.trim().split(/\s+/)[0]).filter(Boolean);
     if (effects.some(effect => RUNTIME_EFFECTS.has(effect)))
         return false;
@@ -72,10 +115,16 @@ function describeBlock({ el, isLive, hasLiveDescendants }, options = {}) {
         reason = 'embed';
     return { reason, host };
 }
+/** `.first.second` from an element's class attribute (first two classes), or undefined when it has none. */
+function classIdentity(className) {
+    if (!className)
+        return undefined;
+    return `.${className.trim().split(/\s+/).slice(0, 2).join('.')}`;
+}
 function createEditorBlock({ el, reason, host }, options = {}) {
     const t = options.translate ?? ((key) => key);
     const id = el.attributes.id;
-    const identity = id ? `#${id}` : el.attributes.class ? `.${el.attributes.class.trim().split(/\s+/).slice(0, 2).join('.')}` : el.tagName;
+    const identity = id ? `#${id}` : classIdentity(el.attributes.class) ?? el.tagName;
     return {
         ...el.outerRange, tagName: el.tagName, ...(id ? { id } : {}), reason, element: el,
         descriptor: reason === 'embed' && host ? host : {
@@ -155,8 +204,10 @@ export function prepareEditorSource({ html }, options = {}) {
     instances.sort((a, b) => a.outerStart - b.outerStart);
     let markerAttribute = 'data-jini-raw-source-0';
     let namespace = 0;
-    while (html.toLowerCase().includes(markerAttribute))
-        markerAttribute = `data-jini-raw-source-${++namespace}`;
+    while (html.toLowerCase().includes(markerAttribute)) {
+        namespace += 1;
+        markerAttribute = `data-jini-raw-source-${namespace}`;
+    }
     const prepared = renderSourceShells({ html, instances, blocks: detected.blocks, markerAttribute }, {});
     return { ok: true, html: prepared, markerAttribute, instances, canvasCss: css.join('\n') };
 }
@@ -175,7 +226,8 @@ function renderSourceShells({ html, instances, blocks, markerAttribute }, _optio
             .map(name => ` ${name}="${escapeAttribute(attrs[name])}"`).join('');
         const tag = instance.nonvisual ? 'span' : instance.tagName;
         const voidTag = instance.openingEnd === instance.outerEnd;
-        const shell = `<${tag} ${markerAttribute}="${i}"${layout}>${voidTag && !instance.nonvisual ? '' : `</${tag}>`}`;
+        const closingTag = voidTag && !instance.nonvisual ? '' : `</${tag}>`;
+        const shell = `<${tag} ${markerAttribute}="${i}"${layout}>${closingTag}`;
         prepared = prepared.slice(0, instance.outerStart) + shell + prepared.slice(instance.outerEnd);
     }
     return prepared;
@@ -186,7 +238,9 @@ export function findEditorBlockOpening({ html, target }, _options = {}) {
     let result;
     const walk = (el) => {
         if (el.tagName === target.tagName.toLowerCase() && (target.id === undefined || el.attributes.id === target.id)) {
-            if (occurrence++ === target.occurrenceIndex && el.outerRange) {
+            const index = occurrence;
+            occurrence += 1;
+            if (index === target.occurrenceIndex && el.outerRange) {
                 const { outerStart: start, openingEnd: end } = el.outerRange;
                 result = { start, end, openingTag: html.slice(start, end) };
             }

package/dist/features/html-editor/protected-elements.js

@@ -77,10 +77,12 @@ function selectedSourceStart({ wrapper, selected, original, html }, _options = {
     let copiesBefore = 0;
     let reached = false;
     visit(wrapper, component => {
-        if (component === selected)
+        if (component === selected) {
             reached = true;
-        else if (!reached && component.is(PROTECTED_ELEMENT_TYPE) && protectedElementHtml({ component }, {}) === original)
-            copiesBefore++;
+            return;
+        }
+        if (!reached && component.is(PROTECTED_ELEMENT_TYPE) && protectedElementHtml({ component }, {}) === original)
+            copiesBefore += 1;
     });
     let start = -1;
     for (let i = 0; i <= copiesBefore; i++)

package/dist/features/html-editor/react/hooks/useInteractiveHtmlEditor.js

@@ -227,7 +227,7 @@ export function useInteractiveHtmlEditor(html, onChange, isProtectedElement, can
     useEffect(() => {
         const wrapper = wrapperRef.current;
         if (!wrapper)
-            return;
+            return undefined;
         const sourcePlan = prepareEditorSource({ html }, { isProtectedElement, describeHostElement: describeEmbedPlaceholder, translate: options.translate });
         let integrityFailure;
         const reportSourceError = (error) => {
@@ -237,7 +237,7 @@ export function useInteractiveHtmlEditor(html, onChange, isProtectedElement, can
         };
         if (!sourcePlan.ok) {
             reportSourceError(new EditorSourceError({ message: sourcePlan.problem }, {}));
-            return;
+            return undefined;
         }
         const mountEl = document.createElement('div');
         mountEl.style.height = '100%';
@@ -249,7 +249,7 @@ export function useInteractiveHtmlEditor(html, onChange, isProtectedElement, can
         catch {
             reportSourceError(new EditorSourceError());
             mountEl.remove();
-            return;
+            return undefined;
         }
         const decorate = () => {
             protectTextAncestors({ editor }, {});

package/dist/features/html-editor/source-splice.js

@@ -34,9 +34,7 @@ function recordInnerRange({ node, currentPath, out }, _options = {}) {
 function indexChildren(children, parentPath, out) {
     let elementIndex = 0;
     for (const node of children) {
-        if (!isElement(node))
-            continue;
-        if (UNINDEXED_TAGS.has(node.tagName))
+        if (!isElement(node) || UNINDEXED_TAGS.has(node.tagName))
             continue;
         const currentPath = [...parentPath, elementIndex];
         recordInnerRange({ node, currentPath, out }, {});
@@ -134,7 +132,11 @@ export function indexSourceElements({ html }, _options = {}) {
         let slot = 0;
         const elements = nodes.filter(isElement).map(node => {
             const atom = UNINDEXED_TAGS.has(node.tagName);
-            const path = atom ? null : [...parentPath, slot++];
+            let path = null;
+            if (!atom) {
+                path = [...parentPath, slot];
+                slot += 1;
+            }
             const loc = node.sourceCodeLocation;
             const children = walk({ nodes: node.childNodes, parentPath: path ?? parentPath, parentStart: loc?.startTag?.endOffset ?? parentStart, parentEnd: loc?.endTag?.startOffset ?? parentEnd }, {});
             const outerRange = sourceOuterRange({ node, html, parentStart, parentEnd }, {});

package/dist/features/iframe-pool/react/components/IframeKeepAliveProvider.js

@@ -35,15 +35,15 @@ export function IframeKeepAliveProvider({ children, maxMounted = DEFAULT_MAX_MOU
     const pool = useMemo(() => ({
         attach(key, host, create) {
             let entry = entriesRef.current.get(key);
-            if (!entry) {
-                entry = { key, element: create(), lastUsedAt: Date.now() };
-                entriesRef.current.set(key, entry);
-            }
-            else if (!activeKeysRef.current.has(key)) {
+            if (entry && !activeKeysRef.current.has(key)) {
                 // Reattaching a parked entry — undo the hidden/inert markers
                 // `release()` set, or the reused iframe would stay invisible.
                 unparkIframeElement(entry.element);
             }
+            if (!entry) {
+                entry = { key, element: create(), lastUsedAt: Date.now() };
+                entriesRef.current.set(key, entry);
+            }
             entry.lastUsedAt = Date.now();
             activeKeysRef.current.add(key);
             host.appendChild(entry.element);

package/dist/features/iframe-pool/react/dom-sync.js

@@ -1,10 +1,10 @@
 export function setForwardedRef(ref, value) {
     if (typeof ref === 'function') {
         ref(value);
+        return;
     }
-    else if (ref) {
+    if (ref)
         ref.current = value;
-    }
 }
 export function propNameToAttributeName(name) {
     if (name === 'className')

package/dist/features/integrations/react/components/IntegrationsTab.js

@@ -96,7 +96,9 @@ function DeeplinkInstallRow({ deeplink, info, agentHandle }) {
 /** The generated snippet, or a loading / could-not-resolve placeholder while there is none. */
 function InstallSnippet({ resolved, loading, error }) {
     const t = useT();
-    return (_jsx(SnippetBlock, { snippet: resolved?.snippet ?? '', language: resolved?.language ?? 'json', placeholder: loading ? t('Loading install paths…') : error ? t('Could not resolve install paths.') : '' }));
+    const errorPlaceholder = error ? t('Could not resolve install paths.') : '';
+    const placeholder = loading ? t('Loading install paths…') : errorPlaceholder;
+    return (_jsx(SnippetBlock, { snippet: resolved?.snippet ?? '', language: resolved?.language ?? 'json', placeholder: placeholder }));
 }
 /** Build/Node warning, shown only once install info has loaded and reports a missing prerequisite. */
 function PrerequisiteWarning({ info }) {