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

package/dist/features/schedule-picker/rules.js

@@ -23,8 +23,12 @@ export function formatTime12h(time) {
     const h = Number(m[1]);
     const mm = m[2];
     const suffix = h >= 12 ? 'PM' : 'AM';
-    const h12 = h === 0 ? 12 : h > 12 ? h - 12 : h;
-    return `${h12}:${mm} ${suffix}`;
+    return `${to12Hour(h)}:${mm} ${suffix}`;
+}
+function to12Hour(h) {
+    if (h === 0)
+        return 12;
+    return h > 12 ? h - 12 : h;
 }
 function weekdayLongName(weekday, weekdays) {
     return weekdays.find((w) => w.value === weekday)?.long ?? weekdays[0]?.long ?? 'Sunday';
@@ -39,11 +43,13 @@ export function decomposeSchedule(schedule, weekdays = DEFAULT_WEEKDAYS) {
     if (schedule.kind === 'hourly') {
         return { kind: 'hourly', minute: String(schedule.minute).padStart(2, '0') };
     }
-    const freq = schedule.kind === 'daily'
-        ? 'Daily'
-        : schedule.kind === 'weekdays'
-            ? 'Weekdays'
-            : weekdayLongName(schedule.weekday, weekdays);
+    let freq;
+    if (schedule.kind === 'daily')
+        freq = 'Daily';
+    else if (schedule.kind === 'weekdays')
+        freq = 'Weekdays';
+    else
+        freq = weekdayLongName(schedule.weekday, weekdays);
     return { kind: 'timed', freq, time: formatTime12h(schedule.time), tz: schedule.timezone };
 }
 /** Plain-English one-line summary, e.g. `'Hourly at :05'` or

package/dist/features/sketch-editor/constants.d.ts

@@ -2,12 +2,6 @@ import type { SketchTooltipTarget } from './types.js';
 export declare const SAVED_VISIBLE_MS = 2000;
 export declare const EXPORTED_IMAGE_MIME_TYPE = "image/png";
 export declare const SKETCH_CONTEXT_MENU_MARGIN = 8;
-/**
- * Default tool/menu tooltip targets — these selectors and `data-testid`
- * values are Excalidraw's own toolbar markup, not host-product content.
- * Host-overridable via `SketchEditorProps.tooltipTargets` for a forked or
- * newer Excalidraw build with different markup.
- */
 export declare const DEFAULT_SKETCH_TOOLTIP_TARGETS: readonly SketchTooltipTarget[];
 /**
  * Default context-menu action allow-list + order. These are Excalidraw's

package/dist/features/sketch-editor/constants.js

@@ -7,23 +7,25 @@ export const SKETCH_CONTEXT_MENU_MARGIN = 8;
  * Host-overridable via `SketchEditorProps.tooltipTargets` for a forked or
  * newer Excalidraw build with different markup.
  */
+/** Anchor the tooltip on the toolbar button's wrapping `<label>`. */
+const CLOSEST_LABEL = 'closest-label';
 export const DEFAULT_SKETCH_TOOLTIP_TARGETS = [
     { selector: '[data-testid="main-menu-trigger"]', label: 'mainMenu', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-lock"]', target: 'closest-label', label: 'lock', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-hand"]', target: 'closest-label', label: 'hand', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-selection"]', target: 'closest-label', label: 'selection', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-rectangle"]', target: 'closest-label', label: 'rectangle', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-diamond"]', target: 'closest-label', label: 'diamond', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-ellipse"]', target: 'closest-label', label: 'ellipse', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-arrow"]', target: 'closest-label', label: 'arrow', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-line"]', target: 'closest-label', label: 'line', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-freedraw"]', target: 'closest-label', label: 'freedraw', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-text"]', target: 'closest-label', label: 'text', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-image"]', target: 'closest-label', label: 'image', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-eraser"]', target: 'closest-label', label: 'eraser', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-frame"]', target: 'closest-label', label: 'frame', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-embeddable"]', target: 'closest-label', label: 'embeddable', placement: 'bottom' },
-    { selector: '[data-testid="toolbar-laser"]', target: 'closest-label', label: 'laser', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-lock"]', target: CLOSEST_LABEL, label: 'lock', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-hand"]', target: CLOSEST_LABEL, label: 'hand', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-selection"]', target: CLOSEST_LABEL, label: 'selection', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-rectangle"]', target: CLOSEST_LABEL, label: 'rectangle', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-diamond"]', target: CLOSEST_LABEL, label: 'diamond', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-ellipse"]', target: CLOSEST_LABEL, label: 'ellipse', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-arrow"]', target: CLOSEST_LABEL, label: 'arrow', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-line"]', target: CLOSEST_LABEL, label: 'line', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-freedraw"]', target: CLOSEST_LABEL, label: 'freedraw', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-text"]', target: CLOSEST_LABEL, label: 'text', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-image"]', target: CLOSEST_LABEL, label: 'image', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-eraser"]', target: CLOSEST_LABEL, label: 'eraser', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-frame"]', target: CLOSEST_LABEL, label: 'frame', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-embeddable"]', target: CLOSEST_LABEL, label: 'embeddable', placement: 'bottom' },
+    { selector: '[data-testid="toolbar-laser"]', target: CLOSEST_LABEL, label: 'laser', placement: 'bottom' },
     { selector: '.App-toolbar__extra-tools-trigger', label: 'moreTools', placement: 'bottom' },
 ];
 /**

package/dist/features/sketch-editor/dom.js

@@ -109,18 +109,15 @@ function hasRecognizedContextMenuAction(menu, recognized) {
 function pruneContextMenuItems(menu, allowed) {
     const byAction = new Map();
     for (const child of Array.from(menu.children)) {
-        if (child instanceof HTMLHRElement) {
+        if (child instanceof HTMLHRElement)
             child.remove();
-            continue;
-        }
         if (!(child instanceof HTMLLIElement))
             continue;
         const actionName = child.getAttribute('data-testid') ?? '';
-        if (!allowed.has(actionName)) {
+        if (allowed.has(actionName))
+            byAction.set(actionName, child);
+        else
             child.remove();
-            continue;
-        }
-        byAction.set(actionName, child);
     }
     return byAction;
 }

package/dist/features/sketch-editor/react/components/SketchMainMenu.js

@@ -2,7 +2,8 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Icon } from '../../../../react/components/Icon.js';
 /** The composed `<MainMenu>` Excalidraw renders inside its own menu popover. */
 export function SketchMainMenu({ MainMenu, t, saving, showSaved, canSave, onSave, exportAvailable, exporting, canExport, onExportImage, canClear, onClear, }) {
-    const saveLabel = saving ? t('Saving…') : showSaved ? t('Saved') : t('Save');
+    const idleSaveLabel = showSaved ? t('Saved') : t('Save');
+    const saveLabel = saving ? t('Saving…') : idleSaveLabel;
     const exportLabel = exporting ? t('Exporting image…') : t('Export image');
     return (_jsxs(MainMenu, { children: [_jsx(MainMenu.Item, { "data-testid": "sketch-menu-save", icon: showSaved ? _jsx(Icon, { name: "check", size: 16 }) : undefined, onClick: onSave, disabled: saving || !canSave, "aria-label": saveLabel, children: saveLabel }), exportAvailable ? (_jsx(MainMenu.Item, { "data-testid": "sketch-menu-export-image", icon: _jsx(Icon, { name: "download", size: 16 }), onClick: onExportImage, disabled: exporting || !canExport, "aria-label": exportLabel, children: exportLabel })) : null, _jsx(MainMenu.Separator, {}), _jsx(MainMenu.DefaultItems.SearchMenu, {}), _jsx(MainMenu.DefaultItems.Help, {}), _jsx(MainMenu.Item, { "data-testid": "sketch-menu-clear", icon: _jsx(Icon, { name: "trash", size: 16 }), onClick: onClear, disabled: !canClear, children: t('Clear canvas') }), _jsx(MainMenu.Separator, {}), _jsx(MainMenu.DefaultItems.ChangeCanvasBackground, {})] }));
 }

package/dist/features/sketch-editor/react/components/SketchSaveStateBadge.js

@@ -1,7 +1,8 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Icon } from '../../../../react/components/Icon.js';
+const STATE_ICONS = { saving: 'spinner', dirty: 'alert-triangle' };
 /** The small save-state indicator rendered in Excalidraw's own top-right UI slot. */
 export function SketchSaveStateBadge({ state, label }) {
-    return (_jsx("div", { className: "sketch-excalidraw-actions", "data-testid": "sketch-save-state", children: _jsxs("span", { className: `sketch-save-state is-${state}`, role: "status", "aria-live": "polite", children: [_jsx(Icon, { name: state === 'saving' ? 'spinner' : state === 'dirty' ? 'alert-triangle' : 'check', size: 12, className: state === 'saving' ? 'icon-spin' : undefined }), _jsx("span", { children: label })] }) }));
+    return (_jsx("div", { className: "sketch-excalidraw-actions", "data-testid": "sketch-save-state", children: _jsxs("span", { className: `sketch-save-state is-${state}`, role: "status", "aria-live": "polite", children: [_jsx(Icon, { name: STATE_ICONS[state] ?? 'check', size: 12, className: state === 'saving' ? 'icon-spin' : undefined }), _jsx("span", { children: label })] }) }));
 }
 //# sourceMappingURL=SketchSaveStateBadge.js.map

package/dist/features/sketch-editor/react/hooks/useSketchDomEnhancements.js

@@ -16,7 +16,7 @@ export function useSketchDomEnhancements({ containerRef, t, domTextOverrides, to
     useEffect(() => {
         const root = containerRef.current;
         if (!root)
-            return;
+            return undefined;
         let frame = null;
         const applyEnhancements = () => {
             frame = null;

package/dist/features/sketch-editor/rules.js

@@ -136,12 +136,15 @@ export function exportedImageFileName(fileName, sourceExtension) {
     const slash = fileName.lastIndexOf('/');
     const baseName = slash >= 0 ? fileName.slice(slash + 1) : fileName;
     const stem = typeof sourceExtension === 'string'
-        ? baseName.toLowerCase().endsWith(sourceExtension.toLowerCase())
-            ? baseName.slice(0, baseName.length - sourceExtension.length)
-            : baseName
+        ? stripKnownExtension(baseName, sourceExtension)
         : baseName.replace(/\.[^./]+$/, '');
     return `${stem || 'sketch'}.png`;
 }
+function stripKnownExtension(baseName, extension) {
+    return baseName.toLowerCase().endsWith(extension.toLowerCase())
+        ? baseName.slice(0, baseName.length - extension.length)
+        : baseName;
+}
 export function exportedImageResultFileName(result, fallback) {
     if (result && typeof result === 'object' && typeof result.fileName === 'string') {
         return result.fileName;
@@ -197,15 +200,13 @@ export function buildSketchTooltipLabels(t) {
  * in `overrides` matches (caller should leave the node untouched).
  */
 export function translateDomTextValue(value, overrides) {
-    // Every group in this pattern is quantified with `*` (zero-or-more), so it
-    // matches every string input, including `''` — `.match()` can never return
-    // null here, and each capture group is always defined (possibly empty).
-    // The nullable return/`| undefined` group types are just `RegExp.match`'s
-    // and `RegExpMatchArray`'s general signatures, not a real runtime path.
-    const match = value.match(/^(\s*)([\s\S]*?)(\s*)$/);
-    const leading = match[1];
-    const core = match[2];
-    const trailing = match[3];
+    // Split into leading whitespace, core, trailing whitespace (an all-whitespace
+    // value is all leading). `trimStart`/`trimEnd` strip exactly regex `\s`.
+    const coreStart = value.length - value.trimStart().length;
+    const leading = value.slice(0, coreStart);
+    const rest = value.slice(coreStart);
+    const core = rest.trimEnd();
+    const trailing = rest.slice(core.length);
     const normalized = core.replace(/\s+/g, ' ').trim();
     if (!normalized)
         return null;

package/dist/features/skills/dependencies.js

@@ -1,3 +1,4 @@
+import { omitKeys } from '../../utils/omit-keys.js';
 const DEFAULT_FAKE_SKILLS = [
     {
         id: 'skill-summarize',
@@ -10,9 +11,13 @@ const DEFAULT_FAKE_SKILLS = [
     },
 ];
 let nextFakeId = 1;
+function takeFakeSkillId() {
+    const id = `fake-skill-${nextFakeId}`;
+    nextFakeId += 1;
+    return id;
+}
 function toSummary(detail) {
-    const { body: _body, ...summary } = detail;
-    return summary;
+    return omitKeys(detail, ['body']);
 }
 /**
  * An in-memory test/demo double. Per this package's established convention
@@ -39,7 +44,7 @@ export function createFakeSkillsPort(options = {}) {
             return delay([...(files.get(id) ?? [])]);
         },
         createSkill(payload) {
-            const id = `fake-skill-${nextFakeId++}`;
+            const id = takeFakeSkillId();
             const detail = {
                 id,
                 name: payload.name,
@@ -58,7 +63,7 @@ export function createFakeSkillsPort(options = {}) {
                 return Promise.reject(new Error(`Unknown skill: ${id}`));
             // A built-in edit writes a NEW user-owned shadow copy, same as the
             // origin — the requested id is never mutated in place.
-            const targetId = existing.source === 'user' ? id : `fake-skill-${nextFakeId++}`;
+            const targetId = existing.source === 'user' ? id : takeFakeSkillId();
             const detail = {
                 ...existing,
                 id: targetId,

package/dist/features/skills/react/components/SkillRow.js

@@ -18,7 +18,8 @@ export function SkillRow({ skill, locale, enabled, expanded, editing, body, body
     // Built-in skills are never edited in place — "editing" one creates a
     // user-owned override, so every edit affordance says so.
     const editActionLabel = builtIn ? l.overrideCreateLabel : l.editLabel;
-    return (_jsxs("div", { className: skillRowClassName({ enabled, expanded }), "data-testid": `skill-row-${skill.id}`, children: [_jsxs("div", { className: "jini-skills-row-head", children: [_jsx(SkillRowSummaryButton, { skill: skill, locale: locale, expanded: expanded, onToggleExpanded: onToggleExpanded, labels: l, agentHandle: agentHandle }), _jsxs("div", { className: "jini-skills-row-actions", children: [isDeletableSkill(skill) && confirmDelete ? (_jsx(SkillRowDeleteConfirm, { onCommitDelete: onCommitDelete, onCancelDelete: onCancelDelete, labels: l, agentHandle: agentHandle })) : (_jsx(SkillRowEditDeleteActions, { skill: skill, editActionLabel: editActionLabel, onStartEdit: onStartEdit, onArmDelete: onArmDelete, labels: l, agentHandle: agentHandle })), _jsxs("label", { className: "toggle-switch toggle-switch-sm jini-skills-row-enable", title: l.enableToggleLabel, children: [_jsx("input", { type: "checkbox", checked: enabled, onChange: (event) => onToggleEnabled(event.target.checked), "aria-label": l.enableToggleLabel, ...agentHandleProps({}, { base: agentHandle, action: 'enabled', role: 'checkbox', label: l.enableToggleLabel }) }), _jsx("span", { className: "toggle-slider" })] })] })] }), confirmBuiltInEdit ? (_jsx(SkillRowBuiltInEditWarning, { onCancelBuiltInEdit: onCancelBuiltInEdit, onConfirmBuiltInEdit: onConfirmBuiltInEdit, labels: l, agentHandle: agentHandle })) : null, expanded && !editing ? (_jsx(SkillRowDetail, { body: body, bodyLoading: bodyLoading, files: files, filesLoading: filesLoading, labels: l })) : null, editing && draft ? (_jsx(SkillDraftForm, { heading: editActionLabel, subheading: skill.id, draft: draft, onDraftChange: onDraftChange, error: draftError, saving: draftSaving, isEdit: true, isBuiltInOverride: builtIn, onCancel: onCancelEdit, onSubmit: onSubmitEdit, labels: labels, ...(agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'edit-form' }) } : {}) })) : null] }));
+    const editFormHandleProps = agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'edit-form' }) } : {};
+    return (_jsxs("div", { className: skillRowClassName({ enabled, expanded }), "data-testid": `skill-row-${skill.id}`, children: [_jsxs("div", { className: "jini-skills-row-head", children: [_jsx(SkillRowSummaryButton, { skill: skill, locale: locale, expanded: expanded, onToggleExpanded: onToggleExpanded, labels: l, agentHandle: agentHandle }), _jsxs("div", { className: "jini-skills-row-actions", children: [isDeletableSkill(skill) && confirmDelete ? (_jsx(SkillRowDeleteConfirm, { onCommitDelete: onCommitDelete, onCancelDelete: onCancelDelete, labels: l, agentHandle: agentHandle })) : (_jsx(SkillRowEditDeleteActions, { skill: skill, editActionLabel: editActionLabel, onStartEdit: onStartEdit, onArmDelete: onArmDelete, labels: l, agentHandle: agentHandle })), _jsxs("label", { className: "toggle-switch toggle-switch-sm jini-skills-row-enable", title: l.enableToggleLabel, children: [_jsx("input", { type: "checkbox", checked: enabled, onChange: (event) => onToggleEnabled(event.target.checked), "aria-label": l.enableToggleLabel, ...agentHandleProps({}, { base: agentHandle, action: 'enabled', role: 'checkbox', label: l.enableToggleLabel }) }), _jsx("span", { className: "toggle-slider" })] })] })] }), confirmBuiltInEdit ? (_jsx(SkillRowBuiltInEditWarning, { onCancelBuiltInEdit: onCancelBuiltInEdit, onConfirmBuiltInEdit: onConfirmBuiltInEdit, labels: l, agentHandle: agentHandle })) : null, expanded && !editing ? (_jsx(SkillRowDetail, { body: body, bodyLoading: bodyLoading, files: files, filesLoading: filesLoading, labels: l })) : null, editing && draft ? (_jsx(SkillDraftForm, { heading: editActionLabel, subheading: skill.id, draft: draft, onDraftChange: onDraftChange, error: draftError, saving: draftSaving, isEdit: true, isBuiltInOverride: builtIn, onCancel: onCancelEdit, onSubmit: onSubmitEdit, labels: labels, ...editFormHandleProps })) : null] }));
 }
 function skillRowClassName({ enabled, expanded }) {
     return `jini-skills-row${enabled ? '' : ' jini-skills-row-disabled'}${expanded ? ' jini-skills-row-expanded' : ''}`;

package/dist/features/skills/react/components/SkillsTab.js

@@ -21,7 +21,8 @@ export function SkillsTab({ port, disabledSkillIds, onToggleEnabled, locale = 'e
     const tab = useSkillsTab({ port, locale });
     const l = resolveLabels({ defaults: DEFAULT_SKILLS_TAB_LABELS, t }, { labels });
     const loadErrorLabel = labels?.loadErrorLabel ?? t('Could not load skills: {error}', { error: tab.loadError ?? '' });
-    return (_jsxs("section", { className: "jini-settings-section jini-settings-skills", children: [_jsx(SkillsTabToolbar, { tab: tab, labels: l, agentHandle: agentHandle }), tab.loadError ? (_jsx("div", { className: "jini-empty-card jini-empty-card-error", children: _jsx("strong", { children: loadErrorLabel }) })) : null, tab.creating ? (_jsx(SkillDraftForm, { heading: l.newSkillLabel, draft: tab.draft, onDraftChange: tab.setDraft, error: tab.draftError, saving: tab.draftSaving, isEdit: false, onCancel: tab.cancelDraft, onSubmit: tab.submitDraft, labels: labels, ...(agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'create-form' }) } : {}) })) : null, !tab.loading && tab.filteredSkills.length === 0 ? (_jsx("div", { className: "jini-empty-card", children: _jsx("strong", { children: l.noResultsLabel }) })) : (_jsx(SkillsTabRows, { tab: tab, disabledSkillIds: disabledSkillIds, onToggleEnabled: onToggleEnabled, locale: locale, labels: labels, agentHandle: agentHandle }))] }));
+    const createFormHandleProps = agentHandle ? { agentHandle: agentSubHandle({ base: agentHandle, action: 'create-form' }) } : {};
+    return (_jsxs("section", { className: "jini-settings-section jini-settings-skills", children: [_jsx(SkillsTabToolbar, { tab: tab, labels: l, agentHandle: agentHandle }), tab.loadError ? (_jsx("div", { className: "jini-empty-card jini-empty-card-error", children: _jsx("strong", { children: loadErrorLabel }) })) : null, tab.creating ? (_jsx(SkillDraftForm, { heading: l.newSkillLabel, draft: tab.draft, onDraftChange: tab.setDraft, error: tab.draftError, saving: tab.draftSaving, isEdit: false, onCancel: tab.cancelDraft, onSubmit: tab.submitDraft, labels: labels, ...createFormHandleProps })) : null, !tab.loading && tab.filteredSkills.length === 0 ? (_jsx("div", { className: "jini-empty-card", children: _jsx("strong", { children: l.noResultsLabel }) })) : (_jsx(SkillsTabRows, { tab: tab, disabledSkillIds: disabledSkillIds, onToggleEnabled: onToggleEnabled, locale: locale, labels: labels, agentHandle: agentHandle }))] }));
 }
 /** Search box, "New skill" button, and the source/mode/(category) selects. */
 function SkillsTabToolbar({ tab, labels: l, agentHandle, }) {

package/dist/features/skills/rules.js

@@ -146,16 +146,16 @@ const modeValue = (skill) => skill.mode;
 const categoryValue = (skill) => typeof skill.category === 'string' && skill.category ? skill.category : null;
 /** The data-derived pills (`mode`, `category`): option SET from every skill,
  *  COUNTS from `matching` — see `skillFilterOptions`. */
-function valueFilterOptions({ skills, matching, valueOf, }) {
+function valueFilterOptions({ skills, matching, valueOf: valueFor, }) {
     const values = new Set();
     for (const skill of skills) {
-        const value = valueOf(skill);
+        const value = valueFor(skill);
         if (value !== null)
             values.add(value);
     }
     const counts = new Map();
     for (const skill of matching) {
-        const value = valueOf(skill);
+        const value = valueFor(skill);
         if (value === null)
             continue;
         counts.set(value, (counts.get(value) ?? 0) + 1);

package/dist/features/source-config-list/agent-handles.js

@@ -39,6 +39,7 @@
  * why cards are keyed by id rather than list position.
  */
 import { agentHandle, buildAgentListHandles } from '@jini-ai/agentic';
+import { trimChars } from '../../utils/trim-chars.js';
 /** Segments of a derived handle: `<base>-field-<key>` and `<base>-<action>` share one joiner. */
 const HANDLE_SEPARATOR = '-';
 /** The namespace every host-described field sits under — see this module's doc for why. */
@@ -65,11 +66,11 @@ const LIST_ITEM_NAMESPACE = 'item';
  * @complexity O(n) in `segment.length`.
  */
 function sanitizeHandleSegment(segment) {
-    return segment
+    const separated = segment
         .replace(/([a-z0-9])([A-Z])/g, `$1${HANDLE_SEPARATOR}$2`)
         .toLowerCase()
-        .replace(/[^a-z0-9]+/g, HANDLE_SEPARATOR)
-        .replace(/^-+|-+$/g, '');
+        .replace(/[^a-z0-9]+/g, HANDLE_SEPARATOR);
+    return trimChars(separated, '-');
 }
 /**
  * The handle for one action or sub-control published under `base`.

package/dist/features/source-config-list/react/components/SourceConfigItemCard.js

@@ -122,7 +122,9 @@ function useItemCardEdit({ source, fieldSpecs, onUpdate }) {
 /** The expanded body: the Edit/Save/Cancel toggle, the editable or read-only fields, and the test control. */
 function ItemCardBody(props) {
     const { capabilities, testing, testResult, onTest, agentHandle, edit } = props;
-    return (_jsxs("div", { className: "source-config-item-card-body", children: [capabilities.canUpdate ? _jsx(ItemCardEditToggle, { ...props, edit: edit }) : null, edit.editing ? _jsx(ItemCardEditFields, { ...props, edit: edit }) : _jsx(ItemCardReadOnlyFields, { ...props }), capabilities.canTest ? (_jsx(SourceConfigTestControl, { running: testing, disabled: isAnyActionPending(props), onTest: onTest, ...(testResult ? { result: testResult } : {}), ...(agentHandle ? { agentHandle: sourceConfigActionHandle(agentHandle, 'test') } : {}) })) : null] }));
+    const testResultProps = testResult ? { result: testResult } : {};
+    const testHandleProps = agentHandle ? { agentHandle: sourceConfigActionHandle(agentHandle, 'test') } : {};
+    return (_jsxs("div", { className: "source-config-item-card-body", children: [capabilities.canUpdate ? _jsx(ItemCardEditToggle, { ...props, edit: edit }) : null, edit.editing ? _jsx(ItemCardEditFields, { ...props, edit: edit }) : _jsx(ItemCardReadOnlyFields, { ...props }), capabilities.canTest ? (_jsx(SourceConfigTestControl, { running: testing, disabled: isAnyActionPending(props), onTest: onTest, ...testResultProps, ...testHandleProps })) : null] }));
 }
 function ItemCardEditToggle(props) {
     const { updating, agentHandle, edit } = props;

package/dist/features/tab-strip/react/components/TabBar.hooks.js

@@ -138,9 +138,11 @@ export function useActiveTabInView({ activeId }, _optional = {}) {
             const tabBox = tab.getBoundingClientRect();
             // A host's end padding (e.g. room for an edge fade) counts as off-screen.
             const visibleRight = stripBox.right - (parseFloat(getComputedStyle(strip).paddingRight) || 0);
-            if (tabBox.left < stripBox.left)
+            if (tabBox.left < stripBox.left) {
                 strip.scrollLeft += tabBox.left - stripBox.left;
-            else if (tabBox.right > visibleRight)
+                return;
+            }
+            if (tabBox.right > visibleRight)
                 strip.scrollLeft += tabBox.right - visibleRight;
         }
         scrollActiveTabIntoView();

package/dist/features/version-manager/react/components/VersionPreviewFrame.js

@@ -11,6 +11,9 @@ export function VersionPreviewFrame({ previewDocument, frameReady, onFrameLoad,
     const [frameRef, canvasSize] = usePreviewCanvasSize();
     const preset = findViewportPreset(viewportPresets, viewport);
     const scale = effectivePreviewScale(preset, 1, canvasSize, DEFAULT_PREVIEW_CANVAS_PADDING);
-    return (_jsx("div", { className: "jini-version-preview", ref: frameRef, children: error ? (_jsx("div", { className: "jini-viewer-empty", role: "alert", children: error })) : (_jsxs(_Fragment, { children: [previewDocument ? (_jsx("div", { className: "jini-preview-viewport", style: previewViewportStyle(preset, scale, 1), children: _jsx("div", { className: "jini-preview-frame-clip", children: _jsx("div", { style: previewScaleShellStyle(preset, 1), children: _jsx("iframe", { title: title, sandbox: "allow-scripts allow-downloads", srcDoc: previewDocument, onLoad: onFrameLoad }) }) }) })) : !loading && !loadingContent ? (_jsx("div", { className: "jini-viewer-empty", children: t('No preview available.') })) : null, loading || loadingContent || (previewDocument && !frameReady) ? (_jsx("div", { className: "jini-version-preview-overlay", role: "status", "aria-label": t('Loading preview…'), children: _jsx("span", { className: "jini-version-preview-spinner", "aria-hidden": "true" }) })) : null] })) }));
+    const showEmpty = !previewDocument && !loading && !loadingContent;
+    const showLoading = loading || loadingContent || (previewDocument && !frameReady);
+    const emptyState = showEmpty ? _jsx("div", { className: "jini-viewer-empty", children: t('No preview available.') }) : null;
+    return (_jsx("div", { className: "jini-version-preview", ref: frameRef, children: error ? (_jsx("div", { className: "jini-viewer-empty", role: "alert", children: error })) : (_jsxs(_Fragment, { children: [previewDocument ? (_jsx("div", { className: "jini-preview-viewport", style: previewViewportStyle(preset, scale, 1), children: _jsx("div", { className: "jini-preview-frame-clip", children: _jsx("div", { style: previewScaleShellStyle(preset, 1), children: _jsx("iframe", { title: title, sandbox: "allow-scripts allow-downloads", srcDoc: previewDocument, onLoad: onFrameLoad }) }) }) })) : null, emptyState, showLoading ? (_jsx("div", { className: "jini-version-preview-overlay", role: "status", "aria-label": t('Loading preview…'), children: _jsx("span", { className: "jini-version-preview-spinner", "aria-hidden": "true" }) })) : null] })) }));
 }
 //# sourceMappingURL=VersionPreviewFrame.js.map

package/dist/features/version-manager/react/components/VersionSidebar.js

@@ -4,11 +4,13 @@ import { useT } from '../../../i18n/index.js';
  *  independent of whatever the preview pane is doing. */
 export function VersionSidebar({ countLabel, search, onSearchChange, showSearch, loading, versions, visibleVersions, selectedVersionId, onSelect, onPrefetch, formatDate, sourceLabel, sourceClassName, restoredFrom, }) {
     const t = useT();
-    return (_jsxs("div", { className: "jini-version-sidebar", children: [_jsx("div", { className: "jini-version-sidebar-head", children: _jsx("span", { className: "jini-version-count", children: countLabel }) }), showSearch ? (_jsxs("div", { className: "jini-version-search", children: [_jsx("input", { type: "search", value: search, onChange: (event) => onSearchChange(event.target.value), placeholder: t('Search…'), "aria-label": t('Search…') }), search ? (_jsx("button", { type: "button", className: "jini-version-search-clear", "aria-label": t('Clear'), onClick: () => onSearchChange(''), children: t('Clear') })) : null] })) : null, _jsx("div", { className: "jini-version-list", role: "listbox", "aria-label": t('Version history'), children: loading ? (_jsx("div", { className: "jini-version-skeleton-list", role: "status", "aria-label": t('Loading versions…'), children: [0, 1, 2, 3].map((row) => (_jsx("div", { className: "jini-version-skeleton-item", "aria-hidden": "true" }, row))) })) : versions.length === 0 ? (_jsx("div", { className: "jini-version-empty", children: t('No versions yet.') })) : visibleVersions.length === 0 ? (_jsx("div", { className: "jini-version-empty", children: t('No results for “{query}”.', { query: search.trim() }) })) : (visibleVersions.map((version) => {
-                    const selected = version.id === selectedVersionId;
-                    const itemRestoredFrom = restoredFrom(version);
-                    const prefetch = () => onPrefetch(version.id);
-                    return (_jsxs("button", { type: "button", className: `jini-version-item${selected ? ' active' : ''}`, role: "option", "aria-selected": selected, onClick: () => onSelect(version.id), onMouseEnter: prefetch, onFocus: prefetch, children: [_jsxs("span", { className: "jini-version-item-top", children: [version.current ? _jsx("span", { className: "jini-version-current-badge", children: t('Current') }) : null, _jsx("span", { className: `jini-version-source-badge ${sourceClassName(version)}`, children: sourceLabel(version) }), _jsx("span", { className: "jini-version-time", children: formatDate(version) })] }), _jsx("span", { className: "jini-version-item-title", children: version.prompt || version.label || t('Version {version}', { version: version.version }) }), _jsxs("span", { className: "jini-version-item-meta", children: [t('Version {version}', { version: version.version }), itemRestoredFrom ? (_jsx("span", { className: "jini-version-item-restored", children: t('Restored from v{version}', { version: itemRestoredFrom.version }) })) : null] })] }, version.id));
-                })) })] }));
+    const noResultsMessage = visibleVersions.length === 0 ? t('No results for “{query}”.', { query: search.trim() }) : null;
+    const emptyMessage = versions.length === 0 ? t('No versions yet.') : noResultsMessage;
+    return (_jsxs("div", { className: "jini-version-sidebar", children: [_jsx("div", { className: "jini-version-sidebar-head", children: _jsx("span", { className: "jini-version-count", children: countLabel }) }), showSearch ? (_jsxs("div", { className: "jini-version-search", children: [_jsx("input", { type: "search", value: search, onChange: (event) => onSearchChange(event.target.value), placeholder: t('Search…'), "aria-label": t('Search…') }), search ? (_jsx("button", { type: "button", className: "jini-version-search-clear", "aria-label": t('Clear'), onClick: () => onSearchChange(''), children: t('Clear') })) : null] })) : null, _jsxs("div", { className: "jini-version-list", role: "listbox", "aria-label": t('Version history'), children: [loading ? (_jsx("div", { className: "jini-version-skeleton-list", role: "status", "aria-label": t('Loading versions…'), children: [0, 1, 2, 3].map((row) => (_jsx("div", { className: "jini-version-skeleton-item", "aria-hidden": "true" }, row))) })) : null, !loading && emptyMessage ? _jsx("div", { className: "jini-version-empty", children: emptyMessage }) : null, !loading && !emptyMessage ? (visibleVersions.map((version) => {
+                        const selected = version.id === selectedVersionId;
+                        const itemRestoredFrom = restoredFrom(version);
+                        const prefetch = () => onPrefetch(version.id);
+                        return (_jsxs("button", { type: "button", className: `jini-version-item${selected ? ' active' : ''}`, role: "option", "aria-selected": selected, onClick: () => onSelect(version.id), onMouseEnter: prefetch, onFocus: prefetch, children: [_jsxs("span", { className: "jini-version-item-top", children: [version.current ? _jsx("span", { className: "jini-version-current-badge", children: t('Current') }) : null, _jsx("span", { className: `jini-version-source-badge ${sourceClassName(version)}`, children: sourceLabel(version) }), _jsx("span", { className: "jini-version-time", children: formatDate(version) })] }), _jsx("span", { className: "jini-version-item-title", children: version.prompt || version.label || t('Version {version}', { version: version.version }) }), _jsxs("span", { className: "jini-version-item-meta", children: [t('Version {version}', { version: version.version }), itemRestoredFrom ? (_jsx("span", { className: "jini-version-item-restored", children: t('Restored from v{version}', { version: itemRestoredFrom.version }) })) : null] })] }, version.id));
+                    })) : null] })] }));
 }
 //# sourceMappingURL=VersionSidebar.js.map

package/dist/features/version-manager/react/hooks/usePreviewCanvasSize.js

@@ -16,7 +16,7 @@ export function usePreviewCanvasSize() {
         // dead-branch-gets-refactored-away rule.
         const el = ref.current;
         if (!el)
-            return;
+            return undefined;
         const measure = () => {
             const rect = el.getBoundingClientRect();
             setSize({ width: rect.width, height: rect.height });

package/dist/features/version-manager/react/hooks/useVersionManager.js

@@ -108,10 +108,8 @@ export function useVersionManager(dependencies, options) {
         setVersions(nextVersions);
         const currentVersion = nextVersions.find((version) => version.current);
         seedCurrentVersionContent({ currentVersion, currentContent, contentCache: contentCacheRef.current });
-        const nextSelected = (preferredId ? nextVersions.find((version) => version.id === preferredId) : null) ??
-            currentVersion ??
-            nextVersions[0] ??
-            null;
+        const preferredVersion = preferredId ? nextVersions.find((version) => version.id === preferredId) : null;
+        const nextSelected = preferredVersion ?? currentVersion ?? nextVersions[0] ?? null;
         setSelectedId(nextSelected?.id ?? null);
         setLoading(false);
     }, [port, fileRef, currentContent]);
@@ -125,7 +123,7 @@ export function useVersionManager(dependencies, options) {
         if (!selectedId) {
             setSelectedContent(null);
             setSelectedContentVersionId(null);
-            return;
+            return undefined;
         }
         const cached = contentCacheRef.current.get(selectedId);
         if (cached !== undefined) {
@@ -138,7 +136,7 @@ export function useVersionManager(dependencies, options) {
             else {
                 setError(null);
             }
-            return;
+            return undefined;
         }
         // Cache miss: keep the previous preview mounted (don't clear
         // `selectedContent`) so switching versions never blanks to empty while
@@ -177,7 +175,7 @@ export function useVersionManager(dependencies, options) {
     // is ever missed.
     useEffect(() => {
         if (!previewDocument || loadedPreviewDocument === previewDocument)
-            return;
+            return undefined;
         const fallback = setTimeout(() => setLoadedPreviewDocument(previewDocument), PREVIEW_LOAD_FALLBACK_MS);
         return () => clearTimeout(fallback);
     }, [previewDocument, loadedPreviewDocument]);

package/dist/features/version-manager/rules.js

@@ -89,11 +89,9 @@ export function filterVersionsBySearch(versions, search, describe) {
  *  current, no restoring mid-fetch, no restoring stale/mismatched content). */
 export function isRestoreDisabled(input) {
     const { selectedVersion, restoring, loadingContent, selectedContentMatchesVersion } = input;
-    return (!selectedVersion ||
-        selectedVersion.current ||
-        restoring ||
-        loadingContent ||
-        !selectedContentMatchesVersion);
+    if (!selectedVersion || selectedVersion.current)
+        return true;
+    return restoring || loadingContent || !selectedContentMatchesVersion;
 }
 /** Whether the currently-loaded preview content actually corresponds to the
  *  currently-selected version (guards against acting on stale content

package/dist/features/viewer-shell/react/components/CommentSidePanel.js

@@ -113,7 +113,8 @@ function CommentSideSelectBar({ selectedCount, sending, sendDisabled, queueOnSen
     const t = useT();
     if (selectedCount === 0)
         return null;
-    return (_jsxs("div", { className: "comment-side-selectbar", "data-testid": "comment-side-selectbar", children: [_jsx("span", { className: "comment-side-selectcount", children: t('{n} selected', { n: selectedCount }) }), _jsx("button", { type: "button", className: "comment-side-clear", onClick: onClearSelection, children: t('Clear') }), _jsx("button", { type: "button", className: "comment-side-send", "data-testid": "comment-side-send-claude", disabled: sending || sendDisabled, onClick: () => void onSendSelected(), children: sending ? t('Sending…') : queueOnSend ? t('Queue') : t('Send to chat') })] }));
+    const idleSendLabel = queueOnSend ? t('Queue') : t('Send to chat');
+    return (_jsxs("div", { className: "comment-side-selectbar", "data-testid": "comment-side-selectbar", children: [_jsx("span", { className: "comment-side-selectcount", children: t('{n} selected', { n: selectedCount }) }), _jsx("button", { type: "button", className: "comment-side-clear", onClick: onClearSelection, children: t('Clear') }), _jsx("button", { type: "button", className: "comment-side-send", "data-testid": "comment-side-send-claude", disabled: sending || sendDisabled, onClick: () => void onSendSelected(), children: sending ? t('Sending…') : idleSendLabel })] }));
 }
 function CommentSideCreateForm({ renderCreateForm, draft, onDraftChange, onCreateComment, sending, sendDisabled, }) {
     const t = useT();

package/dist/features/viewer-shell/react/components/SegmentedToggle.js

@@ -1,5 +1,6 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { RemixIcon } from '../../../../react/components/RemixIcon.js';
+import { withClassName } from '../../../../utils/class-names.js';
 /**
  * A small, always-visible group of toggle buttons, one active at a time.
  * This is the shared primitive behind two call sites that turned out to be
@@ -14,7 +15,7 @@ import { RemixIcon } from '../../../../react/components/RemixIcon.js';
  * primitives for a minor ARIA-pattern difference.
  */
 export function SegmentedToggle({ options, value, onChange, ariaLabel, className, }) {
-    return (_jsx("div", { className: `viewer-segmented-toggle${className ? ` ${className}` : ''}`, role: "group", "aria-label": ariaLabel, children: options.map((option) => {
+    return (_jsx("div", { className: withClassName('viewer-segmented-toggle', className), role: "group", "aria-label": ariaLabel, children: options.map((option) => {
             const selected = option.value === value;
             const label = option.title ?? option.label;
             return (_jsxs("button", { type: "button", className: `viewer-segmented-toggle-button${selected ? ' active' : ''}`, "aria-pressed": selected, "aria-label": label, title: label, onClick: () => onChange(option.value), children: [option.icon ? _jsx(RemixIcon, { name: option.icon, size: 14 }) : null, _jsx("span", { children: option.label })] }, option.value));

package/dist/features/viewer-shell/react/components/ViewerShell.js

@@ -1,4 +1,5 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { withClassName } from '../../../../utils/class-names.js';
 /**
  * The "viewer-toolbar + viewer-body" chrome independently repeated across
  * 8 media-viewer components in the source file (`BinaryViewer`,
@@ -9,7 +10,7 @@ import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
  * layout wrapper.
  */
 export function ViewerShell({ kindClassName, toolbarLeft, toolbarActions, bodyClassName, children }) {
-    return (_jsxs("div", { className: `viewer${kindClassName ? ` ${kindClassName}` : ''}`, children: [_jsxs("div", { className: "viewer-toolbar", children: [_jsx("div", { className: "viewer-toolbar-left", children: toolbarLeft }), toolbarActions ? _jsx("div", { className: "viewer-toolbar-actions", children: toolbarActions }) : null] }), _jsx("div", { className: `viewer-body${bodyClassName ? ` ${bodyClassName}` : ''}`, children: children })] }));
+    return (_jsxs("div", { className: withClassName('viewer', kindClassName), children: [_jsxs("div", { className: "viewer-toolbar", children: [_jsx("div", { className: "viewer-toolbar-left", children: toolbarLeft }), toolbarActions ? _jsx("div", { className: "viewer-toolbar-actions", children: toolbarActions }) : null] }), _jsx("div", { className: withClassName('viewer-body', bodyClassName), children: children })] }));
 }
 /** Trivial "nothing to show" placeholder shared by every viewer body. */
 export function ViewerEmptyState({ children }) {

package/dist/features/viewer-shell/rules.js

@@ -97,13 +97,14 @@ export function hasPrecisionSensitiveJsonNumberText(text) {
         }
         numberTokenPattern.lastIndex = i;
         const match = numberTokenPattern.exec(text);
-        if (!match) {
+        if (match) {
+            if (isPrecisionSensitiveNumberLiteral(match[0]))
+                return true;
+            i = numberTokenPattern.lastIndex;
+        }
+        else {
             i += 1;
-            continue;
         }
-        if (isPrecisionSensitiveNumberLiteral(match[0]))
-            return true;
-        i = numberTokenPattern.lastIndex;
     }
     return false;
 }
@@ -111,10 +112,10 @@ export function hasPrecisionSensitiveJsonNumberText(text) {
  *  `start` (honouring `\` escapes), or `text.length` when it is unterminated. */
 function indexAfterJsonString(text, start) {
     for (let i = start; i < text.length; i += 1) {
+        if (text[i] === '"')
+            return i + 1;
         if (text[i] === '\\')
             i += 1;
-        else if (text[i] === '"')
-            return i + 1;
     }
     return text.length;
 }

package/dist/react/components/BrandLogo.js

@@ -18,12 +18,20 @@ export function brandMonogram(name) {
 }
 /** The stage to start at, given which sources are available. */
 export function firstLogoStage(opts) {
-    return opts.canUseBrandStage ? 'brand' : opts.logoSrc ? 'custom' : opts.host ? 'favicon' : 'letter';
+    if (opts.canUseBrandStage)
+        return 'brand';
+    return fallbackLogoStage(opts);
+}
+/** The first non-brand stage whose source is available. */
+function fallbackLogoStage(opts) {
+    if (opts.logoSrc)
+        return 'custom';
+    return opts.host ? 'favicon' : 'letter';
 }
 /** The next stage to try after the current one fails to load. */
 export function advanceLogoStage(stage, opts) {
     if (stage === 'brand')
-        return opts.logoSrc ? 'custom' : opts.host ? 'favicon' : 'letter';
+        return fallbackLogoStage(opts);
     if (stage === 'custom')
         return 'favicon';
     return 'letter';

package/dist/react/components/CustomSelect.js

@@ -233,7 +233,7 @@ export function useCustomSelectOriginTheme({ buttonRef, open, portal }, _optiona
     const [attributes, setAttributes] = useState({});
     useLayoutEffect(() => {
         if (!open || !portal)
-            return;
+            return undefined;
         const update = () => {
             const next = resolveCustomSelectOriginTheme({ trigger: buttonRef.current });
             setAttributes((current) => current['data-admin-theme'] === next['data-admin-theme']