@vertesia/ui 1.5.0-dev.20260725.083715Z → 1.5.0-dev.20260806.125520Z

package/src/features/agent/chat/hooks/useFileProcessing.test.tsx

@@ -149,6 +149,11 @@ describe('useFileProcessing', () => {
 
         await act(async () => {
             resolveUpload?.({ url: 'https://example.test/upload', path: 'files/wrong.png' });
+            // biome 2.5.6 regression: the nursery rule mis-types `await` on a `Promise | undefined`
+            // variable assigned inside a callback, and its own fix suggests `await await`. It only
+            // fires on Linux CI, not on macOS, so the suppression may read as unused locally.
+            // Drop it once the rule is fixed upstream.
+            // biome-ignore lint/nursery/noFloatingPromises: false positive — already awaited
             await uploadPromise;
         });
 
@@ -188,4 +193,86 @@ describe('useFileProcessing', () => {
             vi.unstubAllGlobals();
         }
     });
+
+    it('excludes files already delivered via a sent message (survives reload)', () => {
+        const client = createClient();
+        const toast = vi.fn();
+        const delivered: ConversationFile = {
+            ...createReadyFile('file-1'),
+            artifact_path: 'files/wrong.png',
+            reference: 'artifact:files/wrong.png',
+        };
+        const serverFileUpdates = new Map([['file-1', delivered]]);
+
+        // No delivered refs → the file shows as a pending composer attachment.
+        const { result, rerender } = renderHook(
+            ({ refs }: { refs: Set<string> | undefined }) =>
+                useFileProcessing(client, 'agent-run-1', serverFileUpdates, toast, refs),
+            { initialProps: { refs: undefined as Set<string> | undefined } },
+        );
+        expect(result.current.processingFiles.has('file-1')).toBe(true);
+
+        // Once its artifact reference appears in message history, it is excluded — matching by
+        // either the full `artifact:` reference or the bare path.
+        rerender({ refs: new Set(['artifact:files/wrong.png']) });
+        expect(result.current.processingFiles.has('file-1')).toBe(false);
+
+        rerender({ refs: new Set(['files/wrong.png']) });
+        expect(result.current.processingFiles.has('file-1')).toBe(false);
+    });
+
+    it('excludes files marked consumed by the workflow without relying on message history', () => {
+        const client = createClient();
+        const toast = vi.fn();
+        const consumed: ConversationFile = {
+            ...createReadyFile('file-1'),
+            artifact_path: 'files/wrong.png',
+            reference: 'artifact:files/wrong.png',
+            consumed_at: 1_100,
+        };
+
+        const { result } = renderHook(() =>
+            useFileProcessing(client, 'agent-run-1', new Map([['file-1', consumed]]), toast),
+        );
+
+        expect(result.current.processingFiles.has('file-1')).toBe(false);
+    });
+
+    it('drops a staged local entry when the server marks that file consumed', async () => {
+        const client = createClient();
+        const toast = vi.fn();
+        const { result, rerender } = renderHook(
+            ({ updates }) => useFileProcessing(client, 'agent-run-1', updates, toast),
+            { initialProps: { updates: new Map<string, ConversationFile>() } },
+        );
+
+        const file = new File(['pdf'], 'report.pdf', { type: 'application/pdf' });
+        await act(async () => {
+            await result.current.handleFileUpload([file]);
+        });
+        const [fileId] = Array.from(result.current.processingFiles.entries())[0];
+        expect(result.current.processingFiles.has(fileId)).toBe(true);
+
+        // The turn was consumed elsewhere (another tab, or an agent-initiated turn), so
+        // clearProcessingFiles() never ran here and the optimistic entry is still staged.
+        // Merging local first means skipping the server update is not enough to hide it.
+        rerender({
+            updates: new Map<string, ConversationFile>([
+                [
+                    fileId,
+                    {
+                        id: fileId,
+                        name: 'report.pdf',
+                        content_type: 'application/pdf',
+                        size: 0,
+                        status: FileProcessingStatus.READY,
+                        started_at: 1_000,
+                        consumed_at: 1_100,
+                    },
+                ],
+            ]),
+        });
+
+        expect(result.current.processingFiles.has(fileId)).toBe(false);
+    });
 });

package/src/features/agent/chat/hooks/useFileProcessing.ts

@@ -51,6 +51,13 @@ export function useFileProcessing(
     agentRunId: string,
     serverFileUpdates: Map<string, ConversationFile>,
     toast: ToastFn,
+    /**
+     * Artifact references (`artifact:...` and bare paths) that already appear in sent user
+     * messages. Files matching these have been delivered to the agent, so they must not resurface
+     * as pending composer attachments after a reload — the durable, server-agnostic counterpart to
+     * the in-session `removedFileIds` suppression.
+     */
+    deliveredArtifactRefs?: Set<string>,
 ): UseFileProcessingResult {
     const t = i18nInstance.getFixedT(null, NAMESPACE);
     // Local optimistic file state (uploads initiated from the UI)
@@ -103,6 +110,17 @@ export function useFileProcessing(
             }
         });
         serverFileUpdates.forEach((file, id) => {
+            // consumed_at is the workflow's authoritative delivered marker. Filter it here in
+            // addition to message-history refs: older runs may lack the marker, while some
+            // server-originated messages may not preserve the rendered attachment block.
+            if (file.consumed_at) {
+                // Drop any optimistic local entry as well. clearProcessingFiles() normally
+                // retires it when this client sends the message, but a turn consumed
+                // elsewhere (another tab, or an agent-initiated turn) leaves this session's
+                // chip staged. Skipping the server update alone would keep it visible.
+                merged.delete(id);
+                return;
+            }
             if (!removedFileIds.has(id)) {
                 // Server updates are authoritative for status, but may omit fields the local
                 // optimistic entry already knows: preview_url is never sent by the server, and
@@ -117,8 +135,21 @@ export function useFileProcessing(
                 } as LocalConversationFile);
             }
         });
+        // Drop files already delivered via a sent message. Unlike removedFileIds (session-only),
+        // this is reconstructed from message history on every mount. It also covers runs created
+        // before the workflow began stamping consumed_at.
+        if (deliveredArtifactRefs && deliveredArtifactRefs.size > 0) {
+            for (const [id, file] of merged) {
+                if (
+                    (file.reference && deliveredArtifactRefs.has(file.reference)) ||
+                    (file.artifact_path && deliveredArtifactRefs.has(file.artifact_path))
+                ) {
+                    merged.delete(id);
+                }
+            }
+        }
         return merged;
-    }, [localFiles, removedFileIds, serverFileUpdates]);
+    }, [localFiles, removedFileIds, serverFileUpdates, deliveredArtifactRefs]);
 
     const hasProcessingFiles = useMemo(
         () =>

package/src/features/agent/chat/index.ts

@@ -39,6 +39,7 @@ export {
 } from './AskUserWidget';
 export * from './JumpingDots';
 export {
+    type AgentMessageFilter,
     ModernAgentConversation,
     type ModernAgentConversationProps,
     type SendAgentMessageFn,

package/src/features/facets/PromptsFacetsNav.tsx

@@ -1,4 +1,4 @@
-import type { FacetBucket } from '@vertesia/common';
+import type { ComputedFacetResponse, FacetBucket } from '@vertesia/common';
 import {
     type Filter as BaseFilter,
     FilterBar,
@@ -11,11 +11,13 @@ import { useState } from 'react';
 import { filterValueToQueryValue, type SearchInterface, setSearchQueryValue } from './utils/SearchInterface';
 
 interface PromptsFacetsNavProps {
-    facets: {
-        role?: FacetBucket[];
-        status?: FacetBucket[];
-        tags?: FacetBucket[];
-    };
+    /**
+     * The facet response as the endpoint publishes it: `total` plus one entry per facet the caller
+     * asked for. The names are chosen per request, so the buckets are narrowed below rather than
+     * declared here — the previous `{ role, status, tags }` shape named facets this component never
+     * requests and could not describe a caller that asks for a different one.
+     */
+    facets: ComputedFacetResponse;
     search: SearchInterface;
     /**
      * Optional controlled filter state. When provided, the parent owns the filter list (and is
@@ -28,8 +30,14 @@ interface PromptsFacetsNavProps {
     filterGroups?: FilterGroup[];
 }
 
+function facetBuckets(facets: ComputedFacetResponse, name: string): FacetBucket[] {
+    const value = facets[name];
+    return Array.isArray(value) ? (value as FacetBucket[]) : [];
+}
+
 // Hook to create filter groups for prompts
 export function usePromptsFilterGroups(facets: PromptsFacetsNavProps['facets']): FilterGroup[] {
+    const roleBuckets = facetBuckets(facets, 'role');
     const customFilterGroups: FilterGroup[] = [];
 
     // Add name filter as text type
@@ -42,12 +50,12 @@ export function usePromptsFilterGroups(facets: PromptsFacetsNavProps['facets']):
     customFilterGroups.push(nameFilterGroup);
 
     // Add role filter as select type if role facets are available
-    if (facets.role && facets.role.length > 0) {
+    if (roleBuckets.length > 0) {
         const rolesFilterGroup = {
             name: 'role',
             placeholder: 'Role',
             type: 'select' as const,
-            options: facets.role.map((facet) => ({
+            options: roleBuckets.map((facet) => ({
                 label: facet._id,
                 value: facet._id,
                 count: facet.count,
@@ -62,7 +70,7 @@ export function usePromptsFilterGroups(facets: PromptsFacetsNavProps['facets']):
         placeholder: 'Tags',
         type: 'stringList' as const,
         multiple: true,
-        options: (facets.tags ?? []).map((facet) => ({
+        options: facetBuckets(facets, 'tags').map((facet) => ({
             label: facet._id,
             value: facet._id,
         })),

package/src/features/store/collections/CollectionsTable.tsx

@@ -53,7 +53,7 @@ export function CollectionsTable() {
                 status: 'success',
                 duration: 3000,
             });
-            refetch();
+            void refetch();
         } catch (err: unknown) {
             console.error('Failed to delete collection:', err);
             toast({

package/src/features/store/objects/components/ContentOverview.tsx

@@ -843,7 +843,10 @@ function TextActions({
         () => (project ? client.projects.retrieve(project.id) : Promise.resolve(undefined)),
         [project?.id],
     );
-    const pdfTemplateObjectId = fullProject?.configuration?.pdf_template_object_id;
+    // `null` is how the setting is cleared, and it means the same thing here as an absent value: no
+    // custom template, so the export falls back to the built-in one. Collapsing the two spellings at
+    // the read keeps `templateObjectId` below a plain optional string.
+    const pdfTemplateObjectId = fullProject?.configuration?.pdf_template_object_id ?? undefined;
 
     const isMarkdown = isMarkdownContentType(content?.type);
     const isDocumentEditingContent = isDocumentEditingContentType(content?.type);

package/src/features/store/objects/selection/ObjectsActionContext.tsx

@@ -1,4 +1,4 @@
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { useToast } from '@vertesia/ui/core';
 import { useUserSession } from '@vertesia/ui/session';
 import { type ReactNode, useMemo } from 'react';
@@ -25,7 +25,7 @@ const DEFAULT_ACTIONS: ObjectsActionSpec[] = [
 
 interface ObjectsActionContextProps {
     children: ReactNode;
-    type?: ContentObjectTypeItem;
+    type?: ContentObjectTypeCatalogEntry;
 }
 export function ObjectsActionContextProvider({ children, type }: ObjectsActionContextProps) {
     const selection = useDocumentSelection();

package/src/features/store/objects/selection/ObjectsActionSpec.ts

@@ -1,5 +1,5 @@
 import type { VertesiaClient } from '@vertesia/client';
-import type { ColumnLayout, ContentObjectTypeItem } from '@vertesia/common';
+import type { ColumnLayout, ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import type { ToastFn } from '@vertesia/ui/core';
 import type { DocumentSelection } from '../DocumentSelectionProvider';
 import type { DocumentSearch } from '../search/DocumentSearchContext';
@@ -22,7 +22,7 @@ export interface ObjectsActionParams {
     toast: ToastFn;
     search: DocumentSearch;
     table_layout?: ColumnLayout[];
-    type?: ContentObjectTypeItem;
+    type?: ContentObjectTypeCatalogEntry;
 }
 
 export interface ActionComponentTypeProps {

package/src/features/store/objects/selection/SelectionActions.tsx

@@ -1,4 +1,4 @@
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { Button, Popover, PopoverContent, PopoverTrigger, SelectList } from '@vertesia/ui/core';
 import { useUITranslation } from '@vertesia/ui/i18n';
 import clsx from 'clsx';
@@ -13,7 +13,7 @@ import { useObjectsActionContext } from './ObjectsActionHooks';
 import type { ObjectsActionSpec } from './ObjectsActionSpec';
 
 interface SelectionActionsProps {
-    type?: ContentObjectTypeItem;
+    type?: ContentObjectTypeCatalogEntry;
     allowMutations?: boolean;
     allowDelete?: boolean;
     /**

package/src/features/store/objects/upload/DocumentUploadModal.tsx

@@ -1,4 +1,4 @@
-import type { Collection, ContentObjectTypeItem, DynamicCollection } from '@vertesia/common';
+import type { Collection, ContentObjectTypeCatalogEntry, DynamicCollection } from '@vertesia/common';
 import {
     Button,
     errorMessage,
@@ -90,7 +90,7 @@ export function DocumentUploadModal({
     const [files, setFiles] = useState<File[]>([]);
     const [processedFiles, setProcessedFiles] = useState<FileWithMetadata[]>([]);
     const [processingDone, setProcessingDone] = useState(false);
-    const [selectedType, setSelectedType] = useState<ContentObjectTypeItem | null>(null);
+    const [selectedType, setSelectedType] = useState<ContentObjectTypeCatalogEntry | null>(null);
     const [fileStatuses, setFileStatuses] = useState<FileUploadStatus[]>([]);
     const [isUploading, setIsUploading] = useState(false);
     const [uploadComplete, setUploadComplete] = useState(false);

package/src/features/store/types/IntakePolicyEditor.tsx

@@ -1,4 +1,8 @@
-import { type ContentObjectType, type ContentTypeIntakePolicy, ContentTypeIntakePolicySchema } from '@vertesia/common';
+import {
+    type ContentObjectTypeCatalogEntry,
+    type ContentTypeIntakePolicy,
+    ContentTypeIntakePolicySchema,
+} from '@vertesia/common';
 import {
     Badge,
     Button,
@@ -25,7 +29,7 @@ import { IntakePolicyForm, type IntakePolicyFormSection } from './IntakePolicyFo
 
 interface IntakePolicyEditorProps {
     /** Content type whose intake policy is edited; omit when editing a standalone policy. */
-    objectType?: ContentObjectType;
+    objectType?: ContentObjectTypeCatalogEntry;
     /** Initial policy value when no objectType is given (e.g. the project default policy). */
     value?: ContentTypeIntakePolicy;
     /** Custom persistence (e.g. save to project configuration). Returns the saved policy.
@@ -644,13 +648,17 @@ function stringifyPolicy(policy: ContentTypeIntakePolicy | undefined) {
     return JSON.stringify(policy ?? EMPTY_POLICY, null, 2);
 }
 
-function createPolicyValidator() {
+function createPolicyValidator(): ValidateFunction<ContentTypeIntakePolicy> {
     const ajv = new Ajv({
         strict: false,
         allErrors: true,
     });
     addFormats(ajv);
-    return ajv.compile(ContentTypeIntakePolicySchema);
+    // The schema is plain JSON — generated from the canonical Zod schema, and deliberately typed as
+    // opaque data so the package root can export it without pulling Zod into the browser bundle. AJV
+    // therefore infers `unknown`; the type argument states what compiling THIS schema validates, and
+    // `store.contract.test.ts` is what keeps the claim true.
+    return ajv.compile<ContentTypeIntakePolicy>(ContentTypeIntakePolicySchema);
 }
 
 function parseAndValidatePolicy(

package/src/features/store/types/IntakePolicyForm.tsx

@@ -152,6 +152,13 @@ export function IntakePolicyForm({ policy, section, onChange, readonly = false }
                             onChange={(value) => setValue(['text_conversion', 'scope'], value)}
                             readonly={readonly}
                         />
+                        <NumberField
+                            label={t('intakePolicy.field.renderDpi')}
+                            value={policy.text_conversion?.render_dpi}
+                            min={72}
+                            onChange={(value) => setValue(['text_conversion', 'render_dpi'], value)}
+                            readonly={readonly}
+                        />
                     </div>
                     <TextAreaField
                         label={t('intakePolicy.field.instructions')}
@@ -161,6 +168,15 @@ export function IntakePolicyForm({ policy, section, onChange, readonly = false }
                     />
                 </FormSection>
 
+                <FormSection title={t('intakePolicy.section.conversionModel')}>
+                    <ModelConfigFields
+                        title={t('intakePolicy.field.model')}
+                        config={policy.text_conversion?.config}
+                        onChange={(value) => setValue(['text_conversion', 'config'], value)}
+                        readonly={readonly}
+                    />
+                </FormSection>
+
                 {policy.text_conversion?.method === 'custom' && (
                     <FormSection title={t('intakePolicy.section.customConversion')}>
                         <div className="grid gap-4 md:grid-cols-2">
@@ -243,6 +259,15 @@ export function IntakePolicyForm({ policy, section, onChange, readonly = false }
                     />
                 </FormSection>
 
+                <FormSection title={t('intakePolicy.section.extractionModel')}>
+                    <ModelConfigFields
+                        title={t('intakePolicy.field.model')}
+                        config={policy.extraction?.config}
+                        onChange={(value) => setValue(['extraction', 'config'], value)}
+                        readonly={readonly}
+                    />
+                </FormSection>
+
                 <FormSection title={t('intakePolicy.section.visionBudget')}>
                     <div className="grid gap-4 md:grid-cols-2 xl:grid-cols-3">
                         <SelectField

package/src/features/store/types/ObjectSchemaEditor.tsx

@@ -1,4 +1,4 @@
-import type { ContentObjectType } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { Button, errorMessage, Panel, useTheme, useToast } from '@vertesia/ui/core';
 import { useUITranslation } from '@vertesia/ui/i18n';
 import { useUserSession } from '@vertesia/ui/session';
@@ -7,7 +7,7 @@ import { Ajv } from 'ajv';
 import { useMemo, useRef, useState } from 'react';
 
 interface ObjectSchemaEditorProps {
-    objectType: ContentObjectType;
+    objectType: ContentObjectTypeCatalogEntry;
     onSchemaUpdate: (jsonSchema: unknown) => void;
     readonly?: boolean;
 }

package/src/features/store/types/SelectContentType.tsx

@@ -1,4 +1,4 @@
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { SelectBox } from '@vertesia/ui/core';
 import { useUITranslation } from '@vertesia/ui/i18n';
 import { useEffect, useState } from 'react';
@@ -6,7 +6,7 @@ import { useTypeRegistry } from './TypeRegistryProvider.js';
 
 interface SelectContentTypeProps {
     defaultValue?: string | string[] | null; // the typeId
-    onChange: (type: ContentObjectTypeItem | null | ContentObjectTypeItem[]) => void;
+    onChange: (type: ContentObjectTypeCatalogEntry | null | ContentObjectTypeCatalogEntry[]) => void;
     className?: string;
     isClearable?: boolean;
     multiple?: boolean;
@@ -22,7 +22,7 @@ export function SelectContentType({
     const { registry: typeRegistry } = useTypeRegistry();
     const [isMounted, setIsMounted] = useState(false);
 
-    const optionLabel = (type: ContentObjectTypeItem | null) => {
+    const optionLabel = (type: ContentObjectTypeCatalogEntry | null) => {
         if (type === null) return t('store.none');
 
         return (
@@ -32,8 +32,8 @@ export function SelectContentType({
             </div>
         );
     };
-    const [selectedType, setSelectedType] = useState<ContentObjectTypeItem | undefined>();
-    const [selectedTypes, setSelectedTypes] = useState<ContentObjectTypeItem[]>([]);
+    const [selectedType, setSelectedType] = useState<ContentObjectTypeCatalogEntry | undefined>();
+    const [selectedTypes, setSelectedTypes] = useState<ContentObjectTypeCatalogEntry[]>([]);
 
     useEffect(() => {
         if (!isMounted) {
@@ -51,12 +51,12 @@ export function SelectContentType({
         }
     }, [isMounted, typeRegistry, defaultValue, multiple]);
 
-    const _onChange = (option: ContentObjectTypeItem | null) => {
+    const _onChange = (option: ContentObjectTypeCatalogEntry | null) => {
         setSelectedType(option || undefined);
         onChange(option);
     };
 
-    const _onChangeMultiple = (options: ContentObjectTypeItem[]) => {
+    const _onChangeMultiple = (options: ContentObjectTypeCatalogEntry[]) => {
         setSelectedTypes(options);
         onChange(options);
     };
@@ -64,7 +64,7 @@ export function SelectContentType({
     if (multiple) {
         return (
             <div className="flex flex-col gap-4 content-between">
-                <SelectBox<ContentObjectTypeItem>
+                <SelectBox<ContentObjectTypeCatalogEntry>
                     options={typeRegistry?.types || []}
                     value={selectedTypes}
                     onChange={_onChangeMultiple}
@@ -81,7 +81,7 @@ export function SelectContentType({
 
     return (
         <div className="flex flex-col gap-4 content-between">
-            <SelectBox<ContentObjectTypeItem>
+            <SelectBox<ContentObjectTypeCatalogEntry>
                 options={typeRegistry?.types || []}
                 value={selectedType}
                 onChange={_onChange}

package/src/features/store/types/SelectContentTypeModal.tsx

@@ -1,4 +1,4 @@
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { Button, Modal, ModalBody, ModalFooter, ModalTitle, SelectBox, VTooltip } from '@vertesia/ui/core';
 import { useUITranslation } from '@vertesia/ui/i18n';
 import { CheckCircleIcon, Info } from 'lucide-react';
@@ -18,7 +18,7 @@ interface SelectContentTypeModalProps {
     /** Children to render in the modal */
     children?: ReactNode;
     /** Optional initial selected type */
-    initialSelectedType?: ContentObjectTypeItem | null;
+    initialSelectedType?: ContentObjectTypeCatalogEntry | null;
     /** Optional description text */
     description?: string;
     /** Label for the confirm button */
@@ -45,7 +45,7 @@ export function SelectContentTypeModal({
     const { t } = useUITranslation();
     const resolvedTitle = title ?? t('store.selectContentType');
     const { registry: typeRegistry } = useTypeRegistry();
-    const [selectedType, setSelectedType] = useState<ContentObjectTypeItem | null>(initialSelectedType);
+    const [selectedType, setSelectedType] = useState<ContentObjectTypeCatalogEntry | null>(initialSelectedType);
 
     // Get available types from the registry
     const types = typeRegistry?.types || [];
@@ -79,7 +79,7 @@ export function SelectContentTypeModal({
                             value={selectedType}
                             optionLabel={(type) => (type ? type.name : t('store.selectContentTypeLabel'))}
                             placeholder={t('store.selectContentTypeAuto')}
-                            onChange={(selected: ContentObjectTypeItem | null | undefined) =>
+                            onChange={(selected: ContentObjectTypeCatalogEntry | null | undefined) =>
                                 setSelectedType(selected || null)
                             }
                             filterBy="name"
@@ -91,7 +91,7 @@ export function SelectContentTypeModal({
                             value={selectedType}
                             optionLabel={(type) => (type ? type.name : t('store.selectContentTypeLabel'))}
                             placeholder={t('store.selectContentTypeLabel')}
-                            onChange={(selected: ContentObjectTypeItem | null | undefined) =>
+                            onChange={(selected: ContentObjectTypeCatalogEntry | null | undefined) =>
                                 setSelectedType(selected || null)
                             }
                             filterBy="name"

package/src/features/store/types/TableLayoutEditor.tsx

@@ -1,4 +1,4 @@
-import type { ColumnLayout, ContentObjectType } from '@vertesia/common';
+import type { ColumnLayout, ContentObjectTypeCatalogEntry } from '@vertesia/common';
 import { Button, errorMessage, Panel, useTheme, useToast } from '@vertesia/ui/core';
 import { useUITranslation } from '@vertesia/ui/i18n';
 import { useUserSession } from '@vertesia/ui/session';
@@ -6,7 +6,7 @@ import { type EditorApi, MonacoEditor } from '@vertesia/ui/widgets';
 import { useMemo, useRef, useState } from 'react';
 
 interface TableLayoutEditorProps {
-    objectType: ContentObjectType;
+    objectType: ContentObjectTypeCatalogEntry;
     onLayoutUpdate: (value: ColumnLayout[] | undefined) => void;
     readonly?: boolean;
 }

package/src/features/store/types/TypeRegistry.ts

@@ -1,8 +1,13 @@
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 
+/**
+ * Holds the type catalog, which is `ContentObjectTypeCatalogEntry` rather than
+ * `ContentObjectTypeItem`: the listing includes system and app types, which are declared in code and
+ * therefore carry none of the four audit fields.
+ */
 export class TypeRegistry {
-    map: Record<string, ContentObjectTypeItem> = {};
-    constructor(public types: ContentObjectTypeItem[]) {
+    map: Record<string, ContentObjectTypeCatalogEntry> = {};
+    constructor(public types: ContentObjectTypeCatalogEntry[]) {
         //sort types
         types.sort((a, b) => a.name.localeCompare(b.name));
         for (const type of types) {

package/src/features/store/types/intake-policy-editor.logic.ts

@@ -34,13 +34,21 @@ export function updateIntakePolicy(
     return next as ContentTypeIntakePolicy;
 }
 
+/**
+ * `number[][]` in, `[number, number][]` out.
+ *
+ * The policy type says `number[][]` — JSON has no tuples, and the published component has always
+ * described a two-element array rather than a pair — so the editor takes what the policy holds and
+ * returns the pair shape the form rows render. The `[1, 1]` default is what makes the result honest:
+ * a short row read out of a stored policy is completed rather than assumed.
+ */
 export function replaceIntakeRange(
-    ranges: [number, number][] | undefined,
+    ranges: number[][] | undefined,
     index: number,
     endpoint: 0 | 1,
     value: number,
 ): [number, number][] {
-    const next = ranges?.map((range) => [...range] as [number, number]) ?? [];
+    const next: [number, number][] = ranges?.map((range) => [range[0] ?? 1, range[1] ?? 1]) ?? [];
     const current = next[index] ?? [1, 1];
     current[endpoint] = value;
     next[index] = current;

package/src/features/store/types/typeCatalogCache.ts

@@ -1,5 +1,5 @@
 import type { VertesiaClient } from '@vertesia/client';
-import type { ContentObjectTypeItem } from '@vertesia/common';
+import type { ContentObjectTypeCatalogEntry } from '@vertesia/common';
 
 /**
  * Browser-session cache for content-type catalog lookups, so render-time consumers
@@ -11,10 +11,17 @@ import type { ContentObjectTypeItem } from '@vertesia/common';
  * lookups self-evict so the next view retries. In-app type edits must call
  * `invalidateTypeCache` — edits made elsewhere (assistant, another tab) are picked up on
  * the next session.
+ *
+ * Entries are `ContentObjectTypeCatalogEntry`, not `ContentObjectTypeItem`: the catalog resolves
+ * in-code types contributed by plugins, whose audit fields are absent, so `created_by` and its three
+ * siblings are optional here.
  */
-const cachePerClient = new WeakMap<VertesiaClient, Map<string, Promise<ContentObjectTypeItem | undefined>>>();
+const cachePerClient = new WeakMap<VertesiaClient, Map<string, Promise<ContentObjectTypeCatalogEntry | undefined>>>();
 
-export function resolveTypeCached(client: VertesiaClient, typeId: string): Promise<ContentObjectTypeItem | undefined> {
+export function resolveTypeCached(
+    client: VertesiaClient,
+    typeId: string,
+): Promise<ContentObjectTypeCatalogEntry | undefined> {
     let cache = cachePerClient.get(client);
     if (!cache) {
         cache = new Map();