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

package/dist/agent-plugins/adapters/memory.js

@@ -17,9 +17,15 @@ export function createMemoryAgentPluginsApi({ plugins = [], files = {} }, _optio
             rows.set(pluginId, updated);
             return updated;
         },
-        async files({ pluginId }, options = {}) { options.signal?.throwIfAborted(); if (!rows.has(pluginId))
-            return notFound(); const listing = listings.get(pluginId); if (!listing)
-            return agentPluginFilesSnapshot({ listing: { pluginId, files: [], truncated: false, limits: { maxFiles: 200, maxEntries: 2000, maxFileBytes: 524288, maxTotalBytes: 4194304 } } }); return listing; },
+        async files({ pluginId }, options = {}) {
+            options.signal?.throwIfAborted();
+            if (!rows.has(pluginId))
+                return notFound();
+            const listing = listings.get(pluginId);
+            if (!listing)
+                return agentPluginFilesSnapshot({ listing: { pluginId, files: [], truncated: false, limits: { maxFiles: 200, maxEntries: 2000, maxFileBytes: 524288, maxTotalBytes: 4194304 } } });
+            return listing;
+        },
     };
 }
 //# sourceMappingURL=memory.js.map

package/dist/agent-plugins/conformance/agent-plugins-api.conformance.js

@@ -21,7 +21,7 @@ export async function runAgentPluginsApiConformance({ create, pluginId, otherPlu
     } check(rejected); };
     try {
         await test('lists installed packages independent of activation', () => { check(initial.length >= 2 && new Set(initial.map(p => p.pluginId)).size === initial.length); });
-        await test('list snapshots are immutable', () => { check(Object.isFrozen(initial) && Object.isFrozen(first) && Object.isFrozen(first.skills) && Object.isFrozen(first.keywords) && Object.isFrozen(first.mcpServerIds)); });
+        await test('list snapshots are immutable', () => { check([initial, first, first.skills, first.keywords, first.mcpServerIds].every(value => Object.isFrozen(value))); });
         await test('activation replies with the single updated row', async () => { const row = await api.setEnabled({ pluginId, enabled: !first.enabled }); check(row.pluginId === pluginId && row.enabled === !first.enabled && row.description === first.description); });
         await test('accepted activation is visible on reload', async () => { check((await api.list({})).find(p => p.pluginId === pluginId)?.enabled === !first.enabled); });
         await test('old snapshot retains its original enabled value', () => { check(initial.find(p => p.pluginId === pluginId)?.enabled === first.enabled); });

package/dist/agent-plugins/controllers/agent-plugins.controller.js

@@ -17,7 +17,8 @@ export function createAgentPluginsController({ api, permissions = [] }, _optiona
     async function load(_required, _optional = {}) {
         if (!canRead())
             return;
-        const ticket = ++generation, version = revision;
+        generation += 1;
+        const ticket = generation, version = revision;
         store.set({ patch: { loading: true, error: null } });
         try {
             const plugins = await api.list({}, { signal: store.signal });
@@ -67,8 +68,12 @@ export function createAgentPluginsController({ api, permissions = [] }, _optiona
             if (canWrite() && !store.getSnapshot().pendingDisable && rowFor(input.pluginId)?.enabled && !store.getSnapshot().busyIds.includes(input.pluginId))
                 store.set({ patch: { pendingDisable: { ...input } } });
         },
-        cancelDisable(_required, _optional = {}) { const p = store.getSnapshot().pendingDisable; if (!p || store.getSnapshot().busyIds.includes(p.pluginId))
-            return; store.set({ patch: { pendingDisable: null } }); },
+        cancelDisable(_required, _optional = {}) {
+            const p = store.getSnapshot().pendingDisable;
+            if (!p || store.getSnapshot().busyIds.includes(p.pluginId))
+                return;
+            store.set({ patch: { pendingDisable: null } });
+        },
         async confirmDisable(_required, _optional = {}) {
             const pending = store.getSnapshot().pendingDisable;
             if (!pending || !canWrite() || !rowFor(pending.pluginId)?.enabled) {
@@ -83,8 +88,12 @@ export function createAgentPluginsController({ api, permissions = [] }, _optiona
                 store.set({ patch: { pendingDisable: null } });
             return accepted;
         },
-        toggleExpanded({ pluginId }, _optional = {}) { if (!canRead() || !rowFor(pluginId))
-            return; const ids = store.getSnapshot().expandedIds; store.set({ patch: { expandedIds: ids.includes(pluginId) ? ids.filter(id => id !== pluginId) : [...ids, pluginId] } }); },
+        toggleExpanded({ pluginId }, _optional = {}) {
+            if (!canRead() || !rowFor(pluginId))
+                return;
+            const ids = store.getSnapshot().expandedIds;
+            store.set({ patch: { expandedIds: ids.includes(pluginId) ? ids.filter(id => id !== pluginId) : [...ids, pluginId] } });
+        },
         inspect({ pluginId }, _optional = {}) { if (canRead() && rowFor(pluginId))
             store.set({ patch: { inspectedId: pluginId } }); },
         closeInspector(_required, _optional = {}) { store.set({ patch: { inspectedId: null } }); },

package/dist/agent-plugins/package-file-tree.js

@@ -32,14 +32,12 @@ export function buildPackageFileTree({ paths }, optional = {}) {
     for (const path of paths) {
         const segments = path.split("/").filter(Boolean);
         const fileName = segments.pop();
-        if (!fileName)
-            continue;
-        const folder = folderAt({ root, segments });
-        const normalized = [...segments, fileName].join("/");
-        if (seen.has(normalized))
+        // A repeat of an already-listed path (after normalization) adds nothing; its folder exists.
+        const normalized = fileName ? [...segments, fileName].join("/") : null;
+        if (!fileName || normalized === null || seen.has(normalized))
             continue;
         seen.add(normalized);
-        folder.files.push({ name: fileName, path });
+        folderAt({ root, segments }).files.push({ name: fileName, path });
     }
     return freeze(root, "", optional.locale === undefined ? collator : new Intl.Collator(optional.locale, { numeric: true, sensitivity: "base" }));
 }

package/dist/agent-plugins/react/components/AgentPluginInspector.js

@@ -1,4 +1,4 @@
-import { jsx as _jsx, Fragment as _Fragment, jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { Dialog, Notice } from '@jini-ai/ui-kit/react';
 import { useAgentPluginInspector } from '../hooks/AgentPluginInspector.hooks.js';
 import { AgentPluginFileTree } from './AgentPluginFileTree.js';
@@ -6,6 +6,9 @@ import { AgentPluginFileContent } from './AgentPluginFileContent.js';
 import { agentPluginsMessagesEn as m } from '../../messages.en.js';
 export function AgentPluginInspector(props, _optional = {}) {
     const vm = useAgentPluginInspector(props);
-    return _jsx(Dialog, { open: true, title: vm.title, closeLabel: "Close package files", onClose: props.onClose, children: vm.denied ? _jsx(Notice, { tone: "danger", children: m.denied }) : _jsxs(_Fragment, { children: [vm.listNotice && _jsx("p", { role: "note", children: vm.listNotice }), _jsx(AgentPluginFileTree, { files: vm.files, selectedPath: vm.selectedPath, onOpen: vm.select }), vm.selected ? _jsx(AgentPluginFileContent, { file: vm.selected }, vm.selected.relativePath) : vm.status && _jsx("p", { role: vm.status.role, children: vm.status.text })] }) });
+    if (vm.denied)
+        return _jsx(Dialog, { open: true, title: vm.title, closeLabel: "Close package files", onClose: props.onClose, children: _jsx(Notice, { tone: "danger", children: m.denied }) });
+    const status = vm.status && _jsx("p", { role: vm.status.role, children: vm.status.text });
+    return _jsxs(Dialog, { open: true, title: vm.title, closeLabel: "Close package files", onClose: props.onClose, children: [vm.listNotice && _jsx("p", { role: "note", children: vm.listNotice }), _jsx(AgentPluginFileTree, { files: vm.files, selectedPath: vm.selectedPath, onOpen: vm.select }), vm.selected ? _jsx(AgentPluginFileContent, { file: vm.selected }, vm.selected.relativePath) : status] });
 }
 //# sourceMappingURL=AgentPluginInspector.js.map

package/dist/agent-plugins/react/hooks/AgentPluginInspector.hooks.js

@@ -27,6 +27,7 @@ function inspectorStatus({ state, hasFiles }, _optional = {}) {
     return hasFiles ? null : { text: m.filesEmpty, role: 'status' };
 }
 const fileGlyphs = { json: '{}', markdown: 'M↓', javascript: 'JS', typescript: 'TS', config: '⚙', html: '<>', css: '#', image: '▧', shell: '>', text: '¶', file: '▤' };
+const folderGlyph = (expanded) => (expanded ? '▾' : '▸');
 export function useAgentPluginFileTree({ files, selectedPath, onOpen }, _optional = {}) {
     const tree = buildPackageFileTree({ paths: files.map(f => f.relativePath) }), [toggled, setToggled] = useState(new Map());
     // Set by keyboard navigation to the tree that received the key, so the effect always has a real element.
@@ -36,16 +37,21 @@ export function useAgentPluginFileTree({ files, selectedPath, onOpen }, _optiona
     const isExpanded = ({ path }) => toggled.get(path) ?? ancestors.has(path);
     const rows = visiblePackageFileRows({ nodes: tree, isExpanded });
     const visible = (path) => path !== null && rows.some(r => r.node.path === path);
-    const activePath = visible(focused) ? focused : visible(selectedPath) ? selectedPath : rows[0]?.node.path ?? null;
+    const activePath = [focused, selectedPath].find(path => visible(path)) ?? rows[0]?.node.path ?? null;
     // Move DOM focus only after keyboard navigation, never when opening the inspector.
-    useEffect(() => { const tree = focusWithin.current; if (!tree)
-        return; focusWithin.current = null; Array.from(tree.querySelectorAll('[data-tree-path]')).find(el => el.dataset.treePath === activePath)?.focus(); });
+    useEffect(() => {
+        const tree = focusWithin.current;
+        if (!tree)
+            return;
+        focusWithin.current = null;
+        Array.from(tree.querySelectorAll('[data-tree-path]')).find(el => el.dataset.treePath === activePath)?.focus();
+    });
     const setExpanded = (path, expanded) => setToggled(current => new Map(current).set(path, expanded));
     return {
         rows: rows.map(row => {
             const { node } = row, folder = node.kind === 'folder', expanded = folder && isExpanded({ path: node.path });
             return { key: `${node.kind}:${node.path}`, name: node.name, path: node.path, depth: row.depth + 1, setSize: row.setSize, posInSet: row.posInSet, expanded: folder ? expanded : undefined, selected: folder ? undefined : node.path === selectedPath, tabIndex: node.path === activePath ? 0 : -1,
-                icon: folder ? expanded ? '▾' : '▸' : fileGlyphs[packageFileIconKind({ fileName: node.name })], indent: { paddingInlineStart: `${row.depth * 1.25}rem`, ...(node.kind === 'file' && node.path === selectedPath ? { backgroundColor: 'Highlight', color: 'HighlightText' } : {}) },
+                icon: folder ? folderGlyph(expanded) : fileGlyphs[packageFileIconKind({ fileName: node.name })], indent: { paddingInlineStart: `${row.depth * 1.25}rem`, ...(node.kind === 'file' && node.path === selectedPath ? { backgroundColor: 'Highlight', color: 'HighlightText' } : {}) },
                 onClick() { setFocused(node.path); if (folder)
                     setExpanded(node.path, !expanded);
                 else

package/dist/agent-plugins/react/hooks/AgentPluginPanel.hooks.js

@@ -38,11 +38,14 @@ function agentPluginRowView({ plugin, busy, expanded, controller, mode, canWrite
     const handle = `agent-plugin-${encodeURIComponent(plugin.pluginId)}`;
     const turnOff = () => controller?.requestDisable({ pluginId: plugin.pluginId, variant: mode === 'installed' ? 'disable' : 'remove' });
     // The switch and the button do the same thing: turning off asks first, enabling writes directly.
-    const activate = () => { if (!canWrite)
-        return; if (plugin.enabled)
-        turnOff();
-    else
-        void controller?.setEnabled({ pluginId: plugin.pluginId, enabled: true }); };
+    const activate = () => {
+        if (!canWrite)
+            return;
+        if (plugin.enabled)
+            turnOff();
+        else
+            void controller?.setEnabled({ pluginId: plugin.pluginId, enabled: true });
+    };
     const verb = plugin.enabled ? 'Turn off' : 'Enable';
     return {
         id: plugin.pluginId, name, version: plugin.version, description: plugin.description,

package/dist/agent-plugins/rules.js

@@ -23,7 +23,9 @@ export function selectAgentPluginFile({ files, path }, _optional = {}) {
 }
 const omissions = { binary: 'Binary file — not shown.', 'too-large': 'Too large to show here.', symlink: 'Symbolic link — not followed.', unreadable: 'This file could not be read.' };
 export function agentPluginFileNotice({ file }, _optional = {}) {
-    return file.omitted === null && file.content !== null ? null : omissions[file.omitted ?? 'unreadable'] ?? omissions.unreadable;
+    if (file.omitted === null && file.content !== null)
+        return null;
+    return omissions[file.omitted ?? 'unreadable'] ?? omissions.unreadable;
 }
 /** Confirmation names the target and workspace effect; Turn off never implies file deletion. */
 export function agentPluginDisableCopy({ pluginId, variant }, _optional = {}) {

package/dist/comments/adapters/http.js

@@ -1,4 +1,5 @@
 import { AdminApiError } from '../../core/transport/errors.js';
+import { querySuffix } from '../../core/transport/query-suffix.js';
 /** Preserve the host's decoded fields while adopting the shared core error owner. */
 async function request({ transport, ...required }, optional = {}) {
     try {
@@ -27,7 +28,7 @@ export function createHttpCommentsApi({ transport, basePath }, _optional = {}) {
             if (options.limit)
                 params.set('limit', String(options.limit));
             const qs = params.toString();
-            return request({ transport, path: `${base}/queue${qs ? `?${qs}` : ''}`, method: 'GET' });
+            return request({ transport, path: `${base}/queue${querySuffix(qs)}`, method: 'GET' });
         },
         async moderateComment({ commentId, action, expectedVersion }, options = {}) {
             await request({ transport, path: `${base}/${encodeURIComponent(commentId)}/${action}`, method: 'POST', body: { expectedVersion, note: options.note } });

package/dist/comments/adapters/memory.js

@@ -7,7 +7,9 @@ const DEFAULT_SETTINGS = {
 };
 const MODERATION_STATUSES = { approve: 'approved', spam: 'spam', trash: 'trash', restore: 'approved' };
 function isNumberIn({ value, min, max }, { integer = false } = {}) {
-    return Number.isFinite(value) && (!integer || Number.isInteger(value)) && value >= min && value <= max;
+    if (!Number.isFinite(value) || (integer && !Number.isInteger(value)))
+        return false;
+    return value >= min && value <= max;
 }
 /** Each settings field with its fixture-level shape check and the expectation a refusal names. */
 const SETTINGS_RULES = [

package/dist/comments/conformance/comments-api.conformance.js

@@ -14,8 +14,9 @@ function hasConflictVersion({ settled, version }, _optional = {}) {
     if (!settled.threw)
         return false;
     const e = settled.error;
-    return e instanceof Error && 'status' in e && e.status === 409 && 'body' in e
-        && e.body?.currentVersion === version;
+    if (!(e instanceof Error && 'status' in e && e.status === 409))
+        return false;
+    return 'body' in e && e.body?.currentVersion === version;
 }
 async function checkPaging({ check }, _optional = {}) {
     const { api, assert } = check;

package/dist/comments/controllers/comment-queue.controller.d.ts

@@ -8,7 +8,7 @@ import { type CommentsTranslator } from '../messages.en.js';
  * @param optional Translator and an invalidation callback for every cached status.
  * @returns An instance-scoped store and actions; dispose makes pending settlements inert.
  */
-export declare function createCommentQueueController({ api, permissions }: {
+export declare function createCommentQueueController({ api, permissions: initialPermissions }: {
     api: AdminCommentsPort;
     permissions?: readonly string[];
 }, { t, onInvalidate, loadPage }?: {

package/dist/comments/controllers/comment-queue.controller.js

@@ -9,11 +9,12 @@ import { englishComments } from '../messages.en.js';
  * @param optional Translator and an invalidation callback for every cached status.
  * @returns An instance-scoped store and actions; dispose makes pending settlements inert.
  */
-export function createCommentQueueController({ api, permissions = [] }, { t = englishComments, onInvalidate, loadPage } = {}) {
+export function createCommentQueueController({ api, permissions: initialPermissions = [] }, { t = englishComments, onInvalidate, loadPage } = {}) {
     const store = createControllerStore({ initial: {
             status: 'pending', items: null, nextCursor: null, error: null, loadingMore: false,
             rowState: {}, pendingPurge: null,
         } });
+    let permissions = initialPermissions;
     let firstPage = null;
     let morePages = [];
     let firstError = null;
@@ -95,7 +96,8 @@ export function createCommentQueueController({ api, permissions = [] }, { t = en
         if (store.signal.aborted || !nextCursor || loadingMore)
             return;
         loadingMore = true;
-        const generation = ++pageGeneration;
+        pageGeneration += 1;
+        const generation = pageGeneration;
         moreError = null; // A retry clears the prior failure before its request settles.
         store.set({ patch: { loadingMore: true } });
         publishPages();

package/dist/core/entities/rules.js

@@ -45,7 +45,8 @@ const MAX_JSON_NODES = 10_000;
 function describeNonJsonValue({ value, depth, budget }, _optional = {}) {
     if (depth > MAX_JSON_DEPTH)
         return `nests deeper than ${MAX_JSON_DEPTH} levels`;
-    if (++budget.nodes > MAX_JSON_NODES)
+    budget.nodes += 1;
+    if (budget.nodes > MAX_JSON_NODES)
         return `holds more than ${MAX_JSON_NODES} values`;
     if (value === null || typeof value !== 'object')
         return describeNonJsonScalar(value);
@@ -95,6 +96,16 @@ function describeNonJson(value) {
     const reason = describeNonJsonValue({ value, depth: 0, budget: { nodes: 0 } });
     return reason === null ? null : `is not a JSON value: it ${reason}`;
 }
+/** One describer per declared kind; the `Record` type makes a new kind without one fail typecheck. */
+const FIELD_MISMATCH_BY_KIND = {
+    text: (value) => describeWrongType({ value, type: 'string', label: 'a string' }),
+    relation: (value) => describeWrongType({ value, type: 'string', label: 'a string' }),
+    datetime: describeNonDatetime,
+    integer: (value) => describeNonNumber({ value, accepts: Number.isInteger, reason: 'is not an integer' }),
+    real: (value) => describeNonNumber({ value, accepts: Number.isFinite, reason: 'is not a finite number' }),
+    boolean: (value) => describeWrongType({ value, type: 'boolean', label: 'a boolean' }),
+    json: describeNonJson,
+};
 /**
  * Whether `value` is a legal runtime value for `field`.
  *
@@ -114,25 +125,7 @@ function describeNonJson(value) {
  * @overallScore 100
  */
 export function describeAdminEntityFieldMismatch(field, value) {
-    switch (field.kind) {
-        case 'text':
-        case 'relation':
-            return describeWrongType({ value, type: 'string', label: 'a string' });
-        case 'datetime':
-            return describeNonDatetime(value);
-        case 'integer':
-            return describeNonNumber({ value, accepts: Number.isInteger, reason: 'is not an integer' });
-        case 'real':
-            return describeNonNumber({
-                value,
-                accepts: Number.isFinite,
-                reason: 'is not a finite number',
-            });
-        case 'boolean':
-            return describeWrongType({ value, type: 'boolean', label: 'a boolean' });
-        case 'json':
-            return describeNonJson(value);
-    }
+    return FIELD_MISMATCH_BY_KIND[field.kind](value);
 }
 /**
  * Every way `row` fails to match `descriptor`.

package/dist/core/module/create-admin.js

@@ -1,7 +1,8 @@
+const bulletLine = (issue) => `- ${issue}`;
 export class AdminConfigError extends Error {
     issues;
     constructor({ issues }, _optional = {}) {
-        super(`Invalid admin configuration:\n${issues.map((issue) => `- ${issue}`).join('\n')}`);
+        super(`Invalid admin configuration:\n${issues.map(bulletLine).join('\n')}`);
         this.name = 'AdminConfigError';
         this.issues = Object.freeze([...issues]);
     }

package/dist/core/omit-keys.d.ts

@@ -0,0 +1,7 @@
+/**
+ * @file Copy an object without some of its keys — what `const { a: _a, ...rest } = source` does,
+ * without binding a throwaway name for every dropped key.
+ */
+/** A shallow copy of `source`'s own enumerable properties, minus `keys`. @complexity O(own keys). */
+export declare function omitKeys<T extends object, K extends keyof T>(source: T, keys: readonly K[]): Omit<T, K>;
+//# sourceMappingURL=omit-keys.d.ts.map

package/dist/core/omit-keys.js

@@ -0,0 +1,12 @@
+/**
+ * @file Copy an object without some of its keys — what `const { a: _a, ...rest } = source` does,
+ * without binding a throwaway name for every dropped key.
+ */
+/** A shallow copy of `source`'s own enumerable properties, minus `keys`. @complexity O(own keys). */
+export function omitKeys(source, keys) {
+    const copy = { ...source };
+    for (const key of keys)
+        delete copy[key];
+    return copy;
+}
+//# sourceMappingURL=omit-keys.js.map

package/dist/core/ports/forms.d.ts

@@ -63,6 +63,8 @@ export type FormFieldType = "text" | "email" | "textarea" | "checkbox" | (string
 /** Lifecycle of a form definition — open, see file header. There is no "deleted" state; see the
  *  file header on why this port has no delete for definitions at all. */
 export type FormDefinitionStatus = "active" | "disabled" | (string & {});
+/** How a form is authored: the field Builder or a hand-written HTML body. */
+export type AdminFormMode = "builder" | "html";
 export interface AdminFormField {
     readonly id: string;
     readonly label: string;
@@ -101,7 +103,7 @@ export interface AdminFormDefinition {
     readonly createdAt: string;
     readonly updatedAt: string;
     /** HTML authoring is explicit; absent mode means the existing Builder contract. */
-    readonly mode?: "builder" | "html" | undefined;
+    readonly mode?: AdminFormMode | undefined;
     readonly html?: string | undefined;
 }
 export interface AdminFormSubmission {
@@ -123,7 +125,7 @@ export interface AdminFormCreateInput {
     readonly name: string;
     readonly slug: string;
     readonly fields: readonly AdminFormField[];
-    readonly mode?: "builder" | "html" | undefined;
+    readonly mode?: AdminFormMode | undefined;
     readonly html?: string | undefined;
     readonly notify?: AdminFormNotifyConfig;
 }
@@ -131,7 +133,7 @@ export interface AdminFormCreateInput {
 export interface AdminFormUpdatePatch {
     readonly name?: string;
     /** A mode/body patch uses the authoring route; an ordinary patch keeps the CRUD route. */
-    readonly mode?: "builder" | "html" | undefined;
+    readonly mode?: AdminFormMode | undefined;
     readonly html?: string | undefined;
     /** Whole-array replace with an add/edit-only constraint — see file header. */
     readonly fields?: readonly AdminFormField[];

package/dist/core/transport/query-suffix.d.ts

@@ -0,0 +1,3 @@
+/** `"a=1"` -> `"?a=1"`; an empty query string adds nothing to the path. */
+export declare function querySuffix(query: string): string;
+//# sourceMappingURL=query-suffix.d.ts.map

package/dist/core/transport/query-suffix.js

@@ -0,0 +1,5 @@
+/** `"a=1"` -> `"?a=1"`; an empty query string adds nothing to the path. */
+export function querySuffix(query) {
+    return query ? `?${query}` : '';
+}
+//# sourceMappingURL=query-suffix.js.map

package/dist/forms/adapters/http.js

@@ -1,9 +1,10 @@
+import { querySuffix } from '../../core/transport/query-suffix.js';
 /** Preserve the existing forms wire, including its data envelopes and explicit authoring route.
  * `mediaBasePath` (the host's media collection path) enables file-answer download links. */
 export function createHttpFormsApi({ transport, basePath }, { mediaBasePath } = {}) {
     const base = basePath.replace(/\/$/, '');
     const formPath = (id) => `${base}/${encodeURIComponent(id)}`;
-    const submissionPath = (formId, submissionId) => `${formPath(formId)}/submissions${submissionId === undefined ? '' : `/${encodeURIComponent(submissionId)}`}`;
+    const submissionPath = (formId, submissionId) => `${formPath(formId)}/submissions${submissionId === undefined ? '' : '/' + encodeURIComponent(submissionId)}`;
     const mediaBase = mediaBasePath?.replace(/\/$/, '');
     return {
         ...(mediaBase === undefined ? {} : {
@@ -29,7 +30,7 @@ export function createHttpFormsApi({ transport, basePath }, { mediaBasePath } =
             if (options.limit)
                 params.set('limit', String(options.limit));
             const qs = params.toString();
-            const response = await transport.request({ path: `${submissionPath(formId)}${qs ? `?${qs}` : ''}`, method: 'GET' });
+            const response = await transport.request({ path: `${submissionPath(formId)}${querySuffix(qs)}`, method: 'GET' });
             return { items: response.data, nextCursor: response.nextCursor };
         },
         async getFormSubmission({ formId, submissionId }, _optional = {}) {

package/dist/forms/adapters/memory.js

@@ -19,6 +19,7 @@ export function createMemoryFormsApi(_required = {}, { forms = [], submissions =
     const definitions = structuredClone([...forms]);
     const answers = structuredClone([...submissions]);
     let sequence = definitions.length;
+    const nextFakeId = () => { sequence += 1; return `fake-${sequence}`; };
     const missing = (id, { kind = 'form definition' } = {}) => {
         throw new AdminApiError({ message: `${kind} '${id}' was not found`, status: 404 });
     };
@@ -47,10 +48,10 @@ export function createMemoryFormsApi(_required = {}, { forms = [], submissions =
         async createFormDefinition(input, options = {}) {
             if (definitions.some(form => form.slug === input.slug))
                 throw new AdminApiError({ message: 'form slug already exists', status: 409 });
-            let id = nextId?.() ?? `fake-${++sequence}`;
+            let id = nextId?.() ?? nextFakeId();
             if (!nextId)
                 while (definitions.some(form => form.id === id))
-                    id = `fake-${++sequence}`;
+                    id = nextFakeId();
             if (definitions.some(form => form.id === id))
                 throw new AdminApiError({ message: 'form id already exists', status: 409 });
             const created = { ...structuredClone(input), id, notify: structuredClone(options.notify ?? { enabled: false, recipients: [] }), status: 'active', createdAt: now(), updatedAt: now() };

package/dist/forms/controllers/field-attributes-dialog.controller.js

@@ -5,7 +5,21 @@ export function initialFieldAttributesDialogState({ field }, _optional = {}) {
     return { className: field.className ?? "", rows: attrRowsFromField({ field }), settings: settingsFormFromField({ field }), error: null };
 }
 /** Add or remove one value from a list setting (accepted uploads, default choices). */
-const toggled = (values, value, on) => on ? (values.includes(value) ? values : [...values, value]) : values.filter((item) => item !== value);
+const toggled = (values, value, on) => {
+    if (!on)
+        return values.filter((item) => item !== value);
+    return values.includes(value) ? values : [...values, value];
+};
+/** A single-select default holds at most the one chosen (non-empty) value. */
+const singleChoice = (value, on) => (on && value ? [value] : []);
+/** The first failure wins (settings before attributes); otherwise both patches merge. */
+function mergeSubmitPatches(settings, attributes) {
+    if (!settings.ok)
+        return settings;
+    if (!attributes.ok)
+        return attributes;
+    return { ok: true, patch: { ...settings.patch, ...attributes.patch } };
+}
 /** A headless section store. React owns subscriptions/effects, never this state. */
 export function createFieldAttributesDialogController({ field }, _optional = {}) {
     const store = createFormsDraftStore({ initial: initialFieldAttributesDialogState({ field }) });
@@ -18,7 +32,7 @@ export function createFieldAttributesDialogController({ field }, _optional = {})
         },
         /** `single` replaces the selection: a dropdown or radio group has one default. */
         toggleDefaultChoice({ value, on, single }, _options = {}) {
-            editSettings(settings => ({ ...settings, defaultChoices: single ? (on && value ? [value] : []) : toggled(settings.defaultChoices, value, on) }));
+            editSettings(settings => ({ ...settings, defaultChoices: single ? singleChoice(value, on) : toggled(settings.defaultChoices, value, on) }));
         },
         toggleAccept({ value, on }, _options = {}) {
             editSettings(settings => ({ ...settings, accept: toggled(settings.accept, value, on) }));
@@ -43,7 +57,7 @@ export function createFieldAttributesDialogController({ field }, _optional = {})
             const snapshot = store.getSnapshot();
             const settings = buildFieldSettingsPatch({ form: snapshot.settings, field });
             const attributes = buildFieldAttributesPatch(snapshot);
-            const result = !settings.ok ? settings : !attributes.ok ? attributes : { ok: true, patch: { ...settings.patch, ...attributes.patch } };
+            const result = mergeSubmitPatches(settings, attributes);
             store.update({ key: 'error', value: result.ok ? null : result.error });
             return result;
         },

package/dist/forms/controllers/form-fields-editor.controller.js

@@ -4,7 +4,6 @@ export function initialFormFieldsEditorState(_required, _optional = {}) {
 }
 /** A headless section store. React owns subscriptions/effects, never this state. */
 export function createFormFieldsEditorController(_required, _optional = {}) {
-    const store = createFormsDraftStore({ initial: initialFormFieldsEditorState({}) });
-    return store;
+    return createFormsDraftStore({ initial: initialFormFieldsEditorState({}) });
 }
 //# sourceMappingURL=form-fields-editor.controller.js.map

package/dist/forms/controllers/form-submission-detail.controller.js

@@ -4,7 +4,6 @@ export function initialFormSubmissionDetailState(_required, _optional = {}) {
 }
 /** A headless section store. React owns subscriptions/effects, never this state. */
 export function createFormSubmissionDetailController(_required, _optional = {}) {
-    const store = createFormsDraftStore({ initial: initialFormSubmissionDetailState({}) });
-    return store;
+    return createFormsDraftStore({ initial: initialFormSubmissionDetailState({}) });
 }
 //# sourceMappingURL=form-submission-detail.controller.js.map

package/dist/forms/controllers/form-submissions.controller.js

@@ -4,7 +4,6 @@ export function initialFormSubmissionsState(_required, _optional = {}) {
 }
 /** A headless section store. React owns subscriptions/effects, never this state. */
 export function createFormSubmissionsController(_required, _optional = {}) {
-    const store = createFormsDraftStore({ initial: initialFormSubmissionsState({}) });
-    return store;
+    return createFormsDraftStore({ initial: initialFormSubmissionsState({}) });
 }
 //# sourceMappingURL=form-submissions.controller.js.map

package/dist/forms/controllers/forms-list.controller.js

@@ -4,7 +4,6 @@ export function initialFormsListState(_required, _optional = {}) {
 }
 /** A headless section store. React owns subscriptions/effects, never this state. */
 export function createFormsListController(_required, _optional = {}) {
-    const store = createFormsDraftStore({ initial: initialFormsListState({}) });
-    return store;
+    return createFormsDraftStore({ initial: initialFormsListState({}) });
 }
 //# sourceMappingURL=forms-list.controller.js.map

package/dist/forms/field-settings.js

@@ -1,3 +1,4 @@
+import { trimChars } from "@jini-ai/core/text";
 import { FIELD_TYPE_REGISTRY, isBrowserSafeFieldPattern, MAX_FORM_FILE_BYTES, SAFE_FORM_FILE_TYPES } from "@jini-ai/cms/forms";
 const kindOf = (type) => Object.hasOwn(FIELD_TYPE_REGISTRY, type) ? FIELD_TYPE_REGISTRY[type] : undefined;
 /** Plain-language type names (translation keys). Exhaustive over the registry by type. */
@@ -86,8 +87,11 @@ export function fieldTypeGroups({ current }, _optional = {}) {
 }
 /** Lowercase letters, digits and `_`, starting with a letter: the CMS `FIELD_ID_PATTERN`. */
 export function fieldIdFromLabel({ label }, _optional = {}) {
-    const slug = label.normalize("NFKD").replace(/[̀-ͯ]/g, "").toLowerCase().replace(/[^a-z0-9]+/g, "_").replace(/^_+|_+$/g, "").slice(0, 60);
-    return slug === "" ? "" : /^[a-z]/.test(slug) ? slug : `field_${slug}`;
+    const underscored = label.normalize("NFKD").replace(/[̀-ͯ]/g, "").toLowerCase().replace(/[^a-z0-9]+/g, "_");
+    const slug = trimChars({ text: underscored, chars: "_" }).slice(0, 60);
+    if (slug === "")
+        return "";
+    return /^[a-z]/.test(slug) ? slug : `field_${slug}`;
 }
 /**
  * A label edit on an unsaved field also renames its id while the id still follows the label (empty
@@ -130,6 +134,12 @@ function retainedKeys({ field, before, after }) {
     ];
 }
 const keepsDefault = ({ before, after }) => after.defaultKind !== null && after.defaultKind === before.defaultKind && after.bounds === before.bounds;
+/** Arrays and objects come back as fresh one-level copies; anything else as is. */
+function shallowCopy(value) {
+    if (Array.isArray(value))
+        return [...value];
+    return typeof value === "object" && value !== null ? { ...value } : value;
+}
 /** The listed keys the field has set, in order; attribute maps and accept lists are copied, not shared. */
 function setKeys({ field, keys }) {
     const kept = {};
@@ -137,7 +147,7 @@ function setKeys({ field, keys }) {
         const value = field[key];
         if (value === undefined)
             continue;
-        Object.assign(kept, { [key]: Array.isArray(value) ? [...value] : typeof value === "object" && value !== null ? { ...value } : value });
+        Object.assign(kept, { [key]: shallowCopy(value) });
     }
     return kept;
 }
@@ -215,7 +225,8 @@ function optionalNumber(value, error, { integer = false } = {}) {
     if (isBlank(value))
         return { ok: true, value: undefined };
     const number = Number(value);
-    return Number.isFinite(number) && (!integer || (Number.isInteger(number) && number >= 0)) ? { ok: true, value: number } : { ok: false, error };
+    const wholeEnough = !integer || (Number.isInteger(number) && number >= 0);
+    return Number.isFinite(number) && wholeEnough ? { ok: true, value: number } : { ok: false, error };
 }
 /** Text length, pattern and placeholder settings. */
 function textPatch({ form, settings }) {