@object-ui/app-shell 7.0.0 → 7.2.0

package/dist/views/metadata-admin/ResourceEditPage.js

@@ -48,6 +48,8 @@ import { getMetadataDefaultInspector } from './default-inspector-registry';
 import { detectLocale, t, tFormat, translateValidationMessage } from './i18n';
 import { JsonSourceEditor } from './JsonSourceEditor';
 import { validateMetadataDraft, hasClientValidator } from './clientValidation';
+import { describeIssuePath } from './issuePath';
+import { buildCreateModeBody } from './createBody';
 // react-resizable-panels' `direction` prop type does not always narrow
 // cleanly in our TS config; cast at the boundary (precedent:
 // packages/components/src/custom/navigation-overlay.tsx).
@@ -59,7 +61,7 @@ const PanelGroup = ResizablePanelGroup;
  * editable via the no-selection default inspector. Other types keep
  * the conventional "name it first, design after save" create flow.
  */
-const CREATE_MODE_CANVAS_TYPES = new Set(['object']);
+const CREATE_MODE_CANVAS_TYPES = new Set(['object', 'report', 'dataset']);
 /**
  * Top-level metadata keys that a type's canvas PreviewComponent owns and
  * edits visually (e.g. the object designer owns `fields` + `fieldGroups`).
@@ -310,6 +312,25 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
     // Issues to DISPLAY (banner + inline). Suppressed on a pristine create form
     // so a blank new item doesn't open covered in required-field errors.
     const displayIssues = React.useMemo(() => (createMode && !createDirty ? [] : issues), [createMode, createDirty, issues]);
+    // Server-computed diagnostics handed to a canvas Preview (e.g. the flow
+    // Problems panel + on-canvas badges). Errors prefer the live client-side Zod
+    // issues when a client validator exists (so they track every keystroke);
+    // warnings stay server-sourced. Mirrors the read-only banner's source
+    // selection, flattened to a path-keyed, severity-tagged list.
+    const previewDiagnostics = React.useMemo(() => {
+        const diag = layered?._diagnostics;
+        const errs = hasClientValidator(type)
+            ? displayIssues.map((i) => ({ path: i.path, message: translateValidationMessage(i.message, locale) }))
+            : (diag?.errors ?? []).map((i) => ({ path: i.path, message: translateValidationMessage(i.message, locale) }));
+        const warns = (diag?.warnings ?? []).map((i) => ({
+            path: i.path,
+            message: translateValidationMessage(i.message, locale),
+        }));
+        return [
+            ...errs.map((e) => ({ path: e.path || undefined, message: e.message, severity: 'error' })),
+            ...warns.map((w) => ({ path: w.path || undefined, message: w.message, severity: 'warning' })),
+        ];
+    }, [layered, displayIssues, type, locale]);
     // Per-item draft pending publish (mode=draft saves land here).
     // When non-null, the editor is "viewing the draft" and we surface
     // Publish / Discard-draft actions.
@@ -749,7 +770,10 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
         const PC = getMetadataPreview(type);
         if (!PC)
             return;
-        const isArtifact = layered?.code != null;
+        // See `isArtifactItem` below — a `sys_metadata`-tagged code layer is a
+        // published org object, NOT a packaged artifact, so it stays editable.
+        const isArtifact = layered?.code != null
+            && layered.code?._packageId !== 'sys_metadata';
         const cw = isArtifact
             ? !!entry?.allowOrgOverride
             : !!(entry?.allowOrgOverride || entry?.allowRuntimeCreate);
@@ -815,22 +839,29 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
         const key = path.split('.')[0];
         if (!key)
             return path;
-        const formForLabels = (createMode && config.createSchema ? undefined : entry?.form);
-        const sections = Array.isArray(formForLabels?.sections) ? formForLabels.sections : [];
-        for (const section of sections) {
-            const fields = Array.isArray(section?.fields) ? section.fields : [];
-            for (const field of fields) {
-                if (typeof field === 'string') {
-                    if (field === key)
-                        return field;
-                }
-                else if (field?.field === key) {
-                    return String(field.label ?? key);
+        // Resolve the human label for the HEAD segment from the form/schema.
+        const headLabel = (() => {
+            const formForLabels = (createMode && config.createSchema ? undefined : entry?.form);
+            const sections = Array.isArray(formForLabels?.sections) ? formForLabels.sections : [];
+            for (const section of sections) {
+                const fields = Array.isArray(section?.fields) ? section.fields : [];
+                for (const field of fields) {
+                    if (typeof field === 'string') {
+                        if (field === key)
+                            return field;
+                    }
+                    else if (field?.field === key) {
+                        return String(field.label ?? key);
+                    }
                 }
             }
-        }
-        const props = (schema?.properties ?? {});
-        return String(props[key]?.title ?? key);
+            const props = (schema?.properties ?? {});
+            return String(props[key]?.title ?? key);
+        })();
+        // For a NESTED path (e.g. `widgets.2.layout`) append a readable trail naming
+        // the offending element + sub-field, so a terse "Widgets: Invalid input"
+        // becomes "Widgets → priority_split → layout".
+        return describeIssuePath(headLabel, path, draft);
     }
     async function doSave(force) {
         setSaving(true);
@@ -842,10 +873,14 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
             // or `{ list: { data: { object } } }` for view) is present so
             // the saved body satisfies its JSONSchema. User-supplied values
             // always win over the defaults.
+            // Prefer the server's authoritative create seed (from /meta/types — the
+            // single source of truth in @objectstack/spec) over the locally hardcoded
+            // createDefaults, so the create shape can't drift from the spec's required
+            // fields (the dashboard-`layout` / action-`body` 422 family). `createSeed`
+            // is a runtime field absent from the bundled GetMetaTypes type, hence the cast.
+            const specCreateSeed = entry?.createSeed;
             let builtBody = createMode
-                ? (config.createBuildBody
-                    ? config.createBuildBody(draft)
-                    : { ...(config.createDefaults ?? {}), ...draft })
+                ? buildCreateModeBody(config, draft, specCreateSeed)
                 // Edit mode: serialise the editor draft back to the wire shape
                 // (inverse of `toDraft` — e.g. `view` folds the `{ list | form }`
                 // family key back into the ViewItem `config` wrapper).
@@ -929,7 +964,15 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
             if (!createMode && !stayInEditing)
                 setEditing(false);
             if (createMode) {
-                navigate(`../${encodeURIComponent(savedName)}`, { relative: 'path' });
+                // Preserve the active query string (notably `?package=…`) so the
+                // post-create navigation lands on the item in the SAME package the
+                // author was working in. Without this the param is dropped and the
+                // editor falls back to the user's default package, where the freshly
+                // saved draft doesn't exist — so it reloads a blank form.
+                const qs = searchParams.toString();
+                navigate(`../${encodeURIComponent(savedName)}${qs ? `?${qs}` : ''}`, {
+                    relative: 'path',
+                });
             }
         }
         catch (err) {
@@ -939,6 +982,14 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
                 setDestructiveIssues(Array.isArray(i) ? i : []);
                 setPendingItem(draft);
             }
+            // ADR-0070 D1/D3 — the kernel rejects authoring into a read-only
+            // code/installed package (`writable_package_required`, also HTTP 422,
+            // so this MUST precede the generic invalid_metadata branch below).
+            // Surface an actionable message guiding the author to pick or create a
+            // writable base rather than mangling it into phantom field issues.
+            else if (err?.code === 'writable_package_required') {
+                setError(t('engine.package.writableRequired', locale));
+            }
             // Map schema validation → inline field errors.
             else if (err?.status === 422 || err?.code === 'invalid_metadata' || err?.code === 'invalid_payload') {
                 const i = err?.body?.issues ?? [];
@@ -1101,7 +1152,15 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
     //   - artifact-backed items (layered.code != null) need allowOrgOverride
     //   - DB-only items (no artifact) need allowOrgOverride OR allowRuntimeCreate
     //   - createMode is always writable (the server will gate on intent)
-    const isArtifactItem = !createMode && layered?.code != null;
+    // A non-null `code` layer alone is NOT proof of a code (artifact) package:
+    // a published org object also surfaces its active version in `code`, but
+    // tagged with the `sys_metadata` provenance sentinel. Mirror the server's
+    // `isArtifactBacked` (which excludes `_packageId === 'sys_metadata'`) so an
+    // org-authored object stays editable after publish instead of being mis-read
+    // as a read-only packaged item.
+    const isArtifactItem = !createMode
+        && layered?.code != null
+        && layered.code?._packageId !== 'sys_metadata';
     // ADR-0010 — server-computed lock flags. undefined means "no opinion"
     // (older server / non-lockable item) → preserve legacy behaviour.
     const lockEditable = layered?.editable !== false;
@@ -1320,7 +1379,7 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
                     ? 'flex h-full min-h-0 flex-col'
                     : 'p-6 space-y-6 max-w-7xl', children: [(error || readOnly || hasDraft || isLocked) && (_jsxs("div", { className: PreviewComponent
                             ? 'px-6 pt-4 space-y-3'
-                            : 'space-y-3', children: [error && (_jsx("div", { className: "text-sm text-destructive border border-destructive/30 rounded p-3 bg-destructive/5", children: error })), isLocked && (_jsxs("div", { className: "text-xs text-amber-900 border border-amber-300/70 bg-amber-50/70 rounded-md px-3 py-2.5 dark:text-amber-200 dark:border-amber-700/40 dark:bg-amber-950/20 flex items-start gap-2.5", children: [_jsx(Lock, { className: "h-3.5 w-3.5 mt-0.5 shrink-0 opacity-80" }), _jsxs("div", { className: "flex-1 min-w-0", children: [_jsxs("div", { className: "font-medium", children: [layered?.lock === 'full' && t('engine.edit.lockFull', locale), layered?.lock === 'no-overlay' && t('engine.edit.lockNoOverlay', locale), layered?.lock === 'no-delete' && t('engine.edit.lockNoDelete', locale)] }), lockReason && _jsx("div", { className: "mt-0.5 opacity-90", children: lockReason }), layered?.lockDocsUrl && (_jsxs("a", { href: layered.lockDocsUrl, target: "_blank", rel: "noopener noreferrer", className: "mt-1 inline-flex items-center gap-1 text-amber-800 underline hover:text-amber-900 dark:text-amber-200 dark:hover:text-amber-100", children: [locale === 'zh-CN' ? '查看文档' : 'View docs', " \u2192"] })), layered?.packageId && (_jsx("div", { className: "mt-0.5 text-amber-700 dark:text-amber-300/80", children: _jsxs("code", { className: "font-mono", children: [layered.packageId, layered.packageVersion ? `@${layered.packageVersion}` : ''] }) }))] }), showArtifactLockedBanner && (_jsx(Button, { size: "sm", variant: "outline", className: "shrink-0 h-7 bg-background/60", onClick: () => navigate(`../new`, { relative: 'path' }), children: t('engine.list.create', locale) }))] })), hasDraft && !createMode && (_jsxs("div", { className: "text-xs text-emerald-900 border border-emerald-300 bg-emerald-50 rounded p-3 dark:text-emerald-200 dark:border-emerald-700/50 dark:bg-emerald-950/30 flex items-center gap-3", children: [_jsx(Send, { className: "h-3.5 w-3.5 shrink-0" }), _jsx("span", { className: "flex-1", children: t('engine.edit.draftPending', locale) }), canWrite && (_jsxs(_Fragment, { children: [_jsx(Button, { size: "sm", variant: "ghost", onClick: doDiscardDraft, disabled: saving || publishing, className: "h-7", children: t('engine.edit.discardDraft', locale) }), _jsx(Button, { size: "sm", onClick: doPublish, disabled: saving || publishing || isDirty, className: "h-7 bg-emerald-600 hover:bg-emerald-700 text-emerald-50", children: publishing ? (_jsx(Loader2, { className: "h-3.5 w-3.5 animate-spin" })) : (t('engine.edit.publish', locale)) })] }))] })), readOnly && !isLocked && (_jsxs("div", { className: "text-xs text-amber-800 border border-amber-300/70 bg-amber-50/70 rounded-md px-3 py-2.5 dark:text-amber-200 dark:border-amber-700/40 dark:bg-amber-950/20 flex items-start gap-3", children: [_jsx("div", { className: "flex-1", children: showArtifactLockedBanner ? (
+                            : 'space-y-3', children: [error && (_jsx("div", { className: "text-sm text-destructive border border-destructive/30 rounded p-3 bg-destructive/5", children: error })), isLocked && (_jsxs("div", { className: "text-xs text-amber-900 border border-amber-300/70 bg-amber-50/70 rounded-md px-3 py-2.5 dark:text-amber-200 dark:border-amber-700/40 dark:bg-amber-950/20 flex items-start gap-2.5", children: [_jsx(Lock, { className: "h-3.5 w-3.5 mt-0.5 shrink-0 opacity-80" }), _jsxs("div", { className: "flex-1 min-w-0", children: [_jsxs("div", { className: "font-medium", children: [layered?.lock === 'full' && t('engine.edit.lockFull', locale), layered?.lock === 'no-overlay' && t('engine.edit.lockNoOverlay', locale), layered?.lock === 'no-delete' && t('engine.edit.lockNoDelete', locale)] }), lockReason && _jsx("div", { className: "mt-0.5 opacity-90", children: lockReason }), layered?.lockDocsUrl && (_jsxs("a", { href: layered.lockDocsUrl, target: "_blank", rel: "noopener noreferrer", className: "mt-1 inline-flex items-center gap-1 text-amber-800 underline hover:text-amber-900 dark:text-amber-200 dark:hover:text-amber-100", children: [locale === 'zh-CN' ? '查看文档' : 'View docs', " \u2192"] })), layered?.packageId && (_jsx("div", { className: "mt-0.5 text-amber-700 dark:text-amber-300/80", children: _jsxs("code", { className: "font-mono", children: [layered.packageId, layered.packageVersion ? `@${layered.packageVersion}` : ''] }) }))] }), showArtifactLockedBanner && (_jsx(Button, { size: "sm", variant: "outline", className: "shrink-0 h-7 bg-background/60", onClick: () => navigate(`../new`, { relative: 'path' }), children: t('engine.list.create', locale) }))] })), hasDraft && !createMode && (_jsxs("div", { className: "text-xs text-emerald-900 border border-emerald-300 bg-emerald-50 rounded p-3 dark:text-emerald-200 dark:border-emerald-700/50 dark:bg-emerald-950/30 flex items-center gap-3", children: [_jsx(Send, { className: "h-3.5 w-3.5 shrink-0" }), _jsx("span", { className: "flex-1", children: t('engine.edit.draftPending', locale) }), canWrite && (_jsxs(_Fragment, { children: [_jsx(Button, { size: "sm", variant: "ghost", onClick: doDiscardDraft, disabled: saving || publishing, className: "h-7", children: t('engine.edit.discardDraft', locale) }), _jsx(Button, { size: "sm", onClick: doPublish, disabled: saving || publishing || isDirty, className: "h-7 bg-emerald-600 hover:bg-emerald-700 text-emerald-50", children: publishing ? (_jsx(Loader2, { className: "h-3.5 w-3.5 animate-spin" })) : (t('engine.edit.publish', locale)) })] }))] })), readOnly && !isLocked && (_jsxs("div", { "data-testid": "readonly-banner", className: "text-xs text-amber-800 border border-amber-300/70 bg-amber-50/70 rounded-md px-3 py-2.5 dark:text-amber-200 dark:border-amber-700/40 dark:bg-amber-950/20 flex items-start gap-3", children: [_jsx("div", { className: "flex-1", children: showArtifactLockedBanner ? (
                                         /* Type allows runtime-create but THIS item ships from
                                            a code package. Tell the user clearly and provide
                                            a CTA to author their own. */
@@ -1404,7 +1463,7 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
                                         const titleKey = kind === 'error'
                                             ? 'engine.edit.diagnostics.title'
                                             : 'engine.edit.diagnostics.warnTitle';
-                                        return (_jsxs("div", { className: `flex items-start gap-2 text-xs border rounded p-2.5 ${cls}`, children: [_jsx(AlertTriangle, { className: "h-4 w-4 mt-0.5 shrink-0" }), _jsxs("div", { className: "min-w-0 flex-1", children: [_jsx("div", { className: "font-medium", children: tFormat(titleKey, locale, { count: items.length }) }), _jsxs("ul", { className: "mt-1 space-y-0.5 font-mono text-[11px]", children: [head.map((e, i) => (_jsxs("li", { className: "truncate", children: [_jsx("span", { className: "opacity-70", children: e.path ? labelForIssuePath(e.path) : '(root)' }), ": ", e.message] }, i))), rest > 0 && (_jsx("li", { className: "opacity-70", children: tFormat('engine.edit.diagnostics.more', locale, { count: rest }) }))] })] })] }, kind));
+                                        return (_jsxs("div", { "data-testid": kind === 'error' ? 'metadata-validation-banner' : undefined, className: `flex items-start gap-2 text-xs border rounded p-2.5 ${cls}`, children: [_jsx(AlertTriangle, { className: "h-4 w-4 mt-0.5 shrink-0" }), _jsxs("div", { className: "min-w-0 flex-1", children: [_jsx("div", { className: "font-medium", children: tFormat(titleKey, locale, { count: items.length }) }), _jsxs("ul", { className: "mt-1 space-y-0.5 font-mono text-[11px]", children: [head.map((e, i) => (_jsxs("li", { className: "truncate", children: [_jsx("span", { className: "opacity-70", children: e.path ? labelForIssuePath(e.path) : '(root)' }), ": ", e.message] }, i))), rest > 0 && (_jsx("li", { className: "opacity-70", children: tFormat('engine.edit.diagnostics.more', locale, { count: rest }) }))] })] })] }, kind));
                                     };
                                     return (_jsxs("div", { className: "space-y-2", children: [hasErrs && renderBlock('error', errs), hasWarns && renderBlock('warning', warns)] }));
                                 })(), PreviewComponent ? (_jsx("div", { className: isFullscreen
@@ -1421,7 +1480,7 @@ function MetadataResourceEditPageImpl({ type, name, createMode, embedded, }) {
                                                                                 ? t('engine.edit.exitFullscreen', locale)
                                                                                 : t('engine.edit.fullscreen', locale), children: isFullscreen ? (_jsx(Minimize2, { className: "h-3.5 w-3.5" })) : (_jsx(Maximize2, { className: "h-3.5 w-3.5" })) })] })] }), _jsx("div", { className: "flex-1 min-h-0 overflow-auto p-4 bg-[radial-gradient(circle_at_1px_1px,theme(colors.border)_1px,transparent_0)] [background-size:16px_16px] bg-muted/30", children: _jsx(PreviewComponent, { type: type, name: name, draft: draft, baseline: !createMode
                                                                     ? (layered?.effective ?? undefined)
-                                                                    : undefined, editing: editing && !previewOnly, selection: previewOnly ? null : selection, onSelectionChange: setSelection, locale: locale, onPatch: (patch) => handleDraftChange((d) => ({ ...d, ...patch })) }) })] }) }), _jsx(ResizableHandle, { withHandle: true, className: inspectorCollapsed
+                                                                    : undefined, editing: editing && !previewOnly, selection: previewOnly ? null : selection, onSelectionChange: setSelection, locale: locale, diagnostics: previewDiagnostics, onPatch: (patch) => handleDraftChange((d) => ({ ...d, ...patch })) }) })] }) }), _jsx(ResizableHandle, { withHandle: true, className: inspectorCollapsed
                                                     ? 'hidden'
                                                     : 'w-1.5 bg-border/40 hover:bg-primary/40 active:bg-primary/60 transition-colors' }), _jsx(ResizablePanel, { panelRef: inspectorPanelRef, defaultSize: lastInspectorSizeRef.current, minSize: 22, collapsible: true, collapsedSize: 0, onResize: (size) => {
                                                     const pct = size.asPercentage;

package/dist/views/metadata-admin/ResourceListPage.js

@@ -21,9 +21,11 @@ import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue, } from '
 import { Empty, EmptyTitle, EmptyDescription } from '@object-ui/components';
 import { PageShell } from './PageShell';
 import { MetadataTypeActions } from './MetadataTypeActions';
+import { CreatePackageDialog } from './PackagesPage';
 import { useMetadataClient, useMetadataTypes, matchesQuery, } from './useMetadata';
 import { getMetadataResource, resolveResourceConfig, } from './registry';
 import { t, tFormat, translateMetadataType, detectLocale } from './i18n';
+import { buildPackageScopeOptions, LOCAL_PACKAGE_ID, isLocalScope } from './package-scope';
 /**
  * Derive provenance from item._packageId. The `loadMetaFromDb` path
  * tags objects with the synthetic packageId 'sys_metadata' (see
@@ -82,20 +84,7 @@ function DefaultMetadataList({ type, appName }) {
                 const list = await client.list('package');
                 if (cancelled)
                     return;
-                const SYSTEM_SCOPES = new Set(['system', 'cloud']);
-                const rows = (list ?? [])
-                    .map((raw) => {
-                    const item = raw && typeof raw === 'object' && 'item' in raw ? raw.item : raw;
-                    const m = (item?.manifest ?? item ?? {});
-                    return {
-                        id: m.id,
-                        scope: m.scope,
-                        name: m.name || m.id,
-                    };
-                })
-                    .filter((p) => p.id && !SYSTEM_SCOPES.has(p.scope));
-                rows.sort((a, b) => a.name.localeCompare(b.name));
-                setProjectPackages(rows.map((p) => ({ id: p.id, name: p.name })));
+                setProjectPackages(buildPackageScopeOptions(list));
             }
             catch {
                 if (!cancelled)
@@ -196,6 +185,22 @@ function DefaultMetadataList({ type, appName }) {
     const pkgSuffix = activePackage
         ? `?package=${encodeURIComponent(activePackage)}`
         : '';
+    // ADR-0070 D3 — never start a create that would orphan the item. When a real
+    // writable base exists, create into it (defaulting away from the Local/null
+    // scope); when none exists yet, prompt to create a base first.
+    const [showCreateBase, setShowCreateBase] = React.useState(false);
+    const handleCreate = React.useCallback(() => {
+        const realBases = (projectPackages ?? []).filter((p) => !isLocalScope(p.id));
+        if (projectPackages !== null && realBases.length === 0) {
+            setShowCreateBase(true);
+            return;
+        }
+        if (realBases.length > 0 && (!activePackage || isLocalScope(activePackage))) {
+            navigate(`./new?package=${encodeURIComponent(realBases[0].id)}`);
+            return;
+        }
+        navigate(`./new${pkgSuffix}`);
+    }, [projectPackages, activePackage, pkgSuffix, navigate]);
     React.useEffect(() => {
         let cancelled = false;
         setLoading(true);
@@ -249,8 +254,12 @@ function DefaultMetadataList({ type, appName }) {
         if (!activePackage)
             return false;
         const pkg = row.item?._packageId;
-        const effectivePkg = !pkg || pkg === 'sys_metadata' ? null : pkg;
-        return effectivePkg === activePackage;
+        const isLocal = !pkg || pkg === LOCAL_PACKAGE_ID;
+        // Local/Custom scope surfaces this environment's runtime-authored items
+        // (untagged / `sys_metadata` provenance); a code package shows its own.
+        if (activePackage === LOCAL_PACKAGE_ID)
+            return isLocal;
+        return !isLocal && pkg === activePackage;
     }), [items, activePackage, config]);
     // User-driven filters (search query + source provenance) on top of scope.
     const filtered = scopedItems.filter((row) => {
@@ -303,14 +312,14 @@ function DefaultMetadataList({ type, appName }) {
                     },
                 ]
                 : []),
-        ], actions: _jsxs(_Fragment, { children: [_jsx(MetadataTypeActions, { entry: entry, location: "list_toolbar", onAfter: () => setRefreshKey((k) => k + 1) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: () => setRefreshKey((k) => k + 1), title: t('engine.list.refresh', locale), children: _jsx(RefreshCw, { className: "h-4 w-4" }) }), (entry?.allowOrgOverride || entry?.allowRuntimeCreate) && (_jsxs(Button, { size: "sm", variant: config.createFields ? 'default' : 'outline', onClick: () => navigate(`./new${pkgSuffix}`), title: config.createFields
+        ], actions: _jsxs(_Fragment, { children: [_jsx(MetadataTypeActions, { entry: entry, location: "list_toolbar", onAfter: () => setRefreshKey((k) => k + 1) }), _jsx(Button, { variant: "ghost", size: "sm", onClick: () => setRefreshKey((k) => k + 1), title: t('engine.list.refresh', locale), children: _jsx(RefreshCw, { className: "h-4 w-4" }) }), (entry?.allowOrgOverride || entry?.allowRuntimeCreate) && (_jsxs(Button, { size: "sm", variant: config.createFields ? 'default' : 'outline', onClick: handleCreate, title: config.createFields
                         ? tFormat('engine.list.createHint', locale, { type: typeLabel })
-                        : undefined, children: [_jsx(Plus, { className: "h-4 w-4 mr-1" }), t('engine.list.create', locale)] }))] }), children: _jsxs("div", { className: "p-6 space-y-4", children: [_jsxs("div", { className: "flex items-center gap-3 flex-wrap", children: [_jsxs("div", { className: "relative flex-1 min-w-[200px] max-w-md", children: [_jsx(Search, { className: "absolute left-2.5 top-1/2 -translate-y-1/2 h-3.5 w-3.5 text-muted-foreground" }), _jsx(Input, { className: "pl-8", placeholder: t('engine.list.search', locale), value: query, onChange: (e) => setQuery(e.target.value) })] }), _jsxs(Select, { value: sourceFilter, onValueChange: setSourceFilter, children: [_jsx(SelectTrigger, { className: "w-[180px]", children: _jsx(SelectValue, {}) }), _jsxs(SelectContent, { children: [_jsxs(SelectItem, { value: "all", children: [t('engine.list.allSources', locale), " (", sourceCounts.all, ")"] }), _jsxs(SelectItem, { value: "artifact", children: [t('engine.list.source.artifact', locale), " (", sourceCounts.artifact, ")"] }), _jsxs(SelectItem, { value: "runtime", children: [t('engine.list.source.runtime', locale), " (", sourceCounts.runtime, ")"] })] })] })] }), (loading || projectPackages === null) && (_jsxs("div", { className: "text-sm text-muted-foreground", children: [t('engine.edit.loading', locale), " ", type, "\u2026"] })), error && (_jsx("div", { className: "text-sm text-destructive border border-destructive/30 rounded p-3 bg-destructive/5", children: error })), !loading && !error && projectPackages !== null && projectPackages.length === 0 && (_jsxs(Empty, { children: [_jsx(EmptyTitle, { children: "No project packages installed" }), _jsx(EmptyDescription, { children: "Studio only shows metadata that belongs to a project software package. Install or create a project package to manage its metadata here." })] })), !loading && !error && projectPackages !== null && projectPackages.length > 0 && filtered.length === 0 && (_jsxs(Empty, { children: [_jsx(EmptyTitle, { children: scopedItems.length === 0
+                        : undefined, children: [_jsx(Plus, { className: "h-4 w-4 mr-1" }), t('engine.list.create', locale)] }))] }), children: _jsxs("div", { className: "p-6 space-y-4", children: [_jsx(CreatePackageDialog, { open: showCreateBase, onOpenChange: setShowCreateBase, onCreated: (id) => navigate(`./new?package=${encodeURIComponent(id)}`) }), _jsxs("div", { className: "flex items-center gap-3 flex-wrap", children: [_jsxs("div", { className: "relative flex-1 min-w-[200px] max-w-md", children: [_jsx(Search, { className: "absolute left-2.5 top-1/2 -translate-y-1/2 h-3.5 w-3.5 text-muted-foreground" }), _jsx(Input, { className: "pl-8", placeholder: t('engine.list.search', locale), value: query, onChange: (e) => setQuery(e.target.value) })] }), _jsxs(Select, { value: sourceFilter, onValueChange: setSourceFilter, children: [_jsx(SelectTrigger, { className: "w-[180px]", children: _jsx(SelectValue, {}) }), _jsxs(SelectContent, { children: [_jsxs(SelectItem, { value: "all", children: [t('engine.list.allSources', locale), " (", sourceCounts.all, ")"] }), _jsxs(SelectItem, { value: "artifact", children: [t('engine.list.source.artifact', locale), " (", sourceCounts.artifact, ")"] }), _jsxs(SelectItem, { value: "runtime", children: [t('engine.list.source.runtime', locale), " (", sourceCounts.runtime, ")"] })] })] })] }), (loading || projectPackages === null) && (_jsxs("div", { className: "text-sm text-muted-foreground", children: [t('engine.edit.loading', locale), " ", type, "\u2026"] })), error && (_jsx("div", { className: "text-sm text-destructive border border-destructive/30 rounded p-3 bg-destructive/5", children: error })), !loading && !error && projectPackages !== null && projectPackages.length === 0 && (_jsxs(Empty, { children: [_jsx(EmptyTitle, { children: "No project packages installed" }), _jsx(EmptyDescription, { children: "Studio only shows metadata that belongs to a project software package. Install or create a project package to manage its metadata here." })] })), !loading && !error && projectPackages !== null && projectPackages.length > 0 && filtered.length === 0 && (_jsxs(Empty, { children: [_jsx(EmptyTitle, { children: scopedItems.length === 0
                                 ? tFormat('engine.list.emptyType', locale, { type: typeLabel })
                                 : tFormat('engine.list.emptyQuery', locale, { query }) }), _jsx(EmptyDescription, { children: config.emptyStateHint ??
                                 (entry?.allowOrgOverride || entry?.allowRuntimeCreate
                                     ? tFormat('engine.list.createHint', locale, { type: typeLabel })
-                                    : t('engine.list.readOnlyHint', locale)) }), scopedItems.length === 0 && (entry?.allowOrgOverride || entry?.allowRuntimeCreate) && (_jsx("div", { className: "mt-4", children: _jsxs(Button, { onClick: () => navigate(`./new${pkgSuffix}`), children: [_jsx(Plus, { className: "h-4 w-4 mr-1" }), t('engine.list.create', locale)] }) }))] })), !loading && filtered.length > 0 && (_jsx("div", { className: "border rounded-lg overflow-hidden", children: _jsxs("table", { className: "w-full text-sm", children: [_jsx("thead", { className: "bg-muted/40 text-xs uppercase tracking-wider text-muted-foreground", children: _jsxs("tr", { children: [columns.map((c) => (_jsx("th", { className: "px-3 py-2 text-left font-medium", style: c.width ? { width: c.width } : undefined, children: localizeColumnLabel(c) }, c.key))), _jsx("th", { className: "px-3 py-2 text-right font-medium w-[80px]", children: t('engine.list.col.source', locale) })] }) }), _jsx("tbody", { className: "divide-y", children: filtered.map((row, i) => {
+                                    : t('engine.list.readOnlyHint', locale)) }), scopedItems.length === 0 && (entry?.allowOrgOverride || entry?.allowRuntimeCreate) && (_jsx("div", { className: "mt-4", children: _jsxs(Button, { onClick: handleCreate, children: [_jsx(Plus, { className: "h-4 w-4 mr-1" }), t('engine.list.create', locale)] }) }))] })), !loading && filtered.length > 0 && (_jsx("div", { className: "border rounded-lg overflow-hidden", children: _jsxs("table", { className: "w-full text-sm", children: [_jsx("thead", { className: "bg-muted/40 text-xs uppercase tracking-wider text-muted-foreground", children: _jsxs("tr", { children: [columns.map((c) => (_jsx("th", { className: "px-3 py-2 text-left font-medium", style: c.width ? { width: c.width } : undefined, children: localizeColumnLabel(c) }, c.key))), _jsx("th", { className: "px-3 py-2 text-right font-medium w-[80px]", children: t('engine.list.col.source', locale) })] }) }), _jsx("tbody", { className: "divide-y", children: filtered.map((row, i) => {
                                     const pk = config.primaryKey ?? 'name';
                                     const name = String(row.item[pk] ?? `(unnamed-${i})`);
                                     // ADR-0048 — link to this row's OWNING package so the editor

package/dist/views/metadata-admin/StudioHomePage.js

@@ -28,6 +28,7 @@ import { useRecentItems } from '../../context/RecentItemsProvider';
 import { useMetadataClient, useMetadataTypes, useGlobalDiagnostics, } from './useMetadata';
 import { MetadataQuickFind } from './QuickFind';
 import { translateMetadataType, translateMetadataDomain, t, tFormat, detectLocale, } from './i18n';
+import { buildPackageScopeOptions, LOCAL_PACKAGE_ID } from './package-scope';
 const HIDDEN_TYPES = new Set(['field', 'package']);
 const DOMAIN_ICONS = {
     data: Database,
@@ -104,20 +105,7 @@ export function StudioHomePage() {
                 const list = await client.list('package');
                 if (cancelled)
                     return;
-                const SYSTEM_SCOPES = new Set(['system', 'cloud']);
-                const rows = (list ?? [])
-                    .map((raw) => {
-                    const item = raw && typeof raw === 'object' && 'item' in raw ? raw.item : raw;
-                    const m = (item?.manifest ?? item ?? {});
-                    return {
-                        id: m.id,
-                        scope: m.scope,
-                        name: m.name || m.id,
-                    };
-                })
-                    .filter((p) => p.id && !SYSTEM_SCOPES.has(p.scope));
-                rows.sort((a, b) => a.name.localeCompare(b.name));
-                setProjectPackages(rows.map((p) => ({ id: p.id, name: p.name })));
+                setProjectPackages(buildPackageScopeOptions(list));
             }
             catch {
                 if (!cancelled)
@@ -155,6 +143,10 @@ export function StudioHomePage() {
             return false;
         if (!activePackage)
             return false;
+        // Local/Custom scope: show every runtime-creatable type so the user can
+        // start authoring any kind of metadata here, even with zero items yet.
+        if (activePackage === LOCAL_PACKAGE_ID)
+            return e.allowOrgOverride || e.allowRuntimeCreate;
         return (packagesByType[e.type] ?? []).includes(activePackage);
     }), [activePackage, entries, packagesByType]);
     const writable = React.useMemo(() => visible.filter((e) => e.allowOrgOverride || e.allowRuntimeCreate), [visible]);

package/dist/views/metadata-admin/anchors.js

@@ -352,6 +352,15 @@ export function registerBuiltinAnchors() {
         createDerive: [
             { from: 'label', to: 'name', transform: 'slugify', untilUserEdits: true },
         ],
+        // A new action defaults to `type: 'script'` (ActionType.default), which the
+        // spec requires to carry an executable `body` or `target` — otherwise the
+        // draft fails validation on save (422) and AppPlugin registers no engine
+        // handler (the #2169 "Mark Done" runtime miss). Seed a no-op L2 body so
+        // "New action -> name -> Save" round-trips; the author edits the source after.
+        createDefaults: {
+            type: 'script',
+            body: { language: 'js', source: 'return { success: true };' },
+        },
     });
     // dashboard / report — surface ones bound to a specific object.
     // Many will not have an explicit anchor; those simply don't appear.
@@ -377,11 +386,20 @@ export function registerBuiltinAnchors() {
                 groupLabel: 'Reports',
                 order: 81,
             }],
-        createFields: ['label', 'name', 'objectName', 'description'],
+        // ADR-0021 single-form: a report is dataset-bound — `objectName` and the
+        // legacy `columns` array were removed from ReportSchema, so seeding them
+        // here produced a stub that failed server validation ("a report needs
+        // `dataset` + `values`"). Report-create now lights up the canvas +
+        // ReportDefaultInspector (see CREATE_MODE_CANVAS_TYPES), where the author
+        // picks the dataset/measures/dimensions directly; we just seed a sensible
+        // starting type.
+        createFields: ['label', 'name', 'description'],
         createDerive: [
             { from: 'label', to: 'name', transform: 'slugify', untilUserEdits: true },
         ],
-        createDefaults: { columns: [] },
+        // Seed `drilldown:true` so the inspector toggle reflects the schema default
+        // (otherwise it shows OFF on a fresh report while the spec default is true).
+        createDefaults: { type: 'summary', drilldown: true },
     });
     // Cosmetic defaults for the `object` type list page — gives Object the
     // same look as every other metadata type while still surfacing the

package/dist/views/metadata-admin/createBody.d.ts

@@ -0,0 +1,26 @@
+/**
+ * ObjectUI
+ * Copyright (c) 2024-present ObjectStack Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+import type { MetadataResourceConfig } from './registry';
+/**
+ * Build the create-mode save body for a new metadata item.
+ *
+ * Prefers the server's authoritative **create seed** (delivered per type on the
+ * `/meta/types` registry entry — the single source of truth in
+ * `@objectstack/spec`) over the locally hardcoded `createDefaults`. This is the
+ * drift-stop for the recurring "the designer emits a minimal shape the spec
+ * rejects, so create→save 422s" family (dashboard `layout`, action `body`):
+ * the structural defaults now come from the same place the spec validates
+ * against, so they cannot diverge. Falls back to `createDefaults` when the
+ * server provides no seed (older server, or canvas-create types whose shape is
+ * built interactively).
+ *
+ * User-supplied draft values always win over the seed's placeholders.
+ * `createBuildBody` (dynamic identity, e.g. a view's qualified name) still takes
+ * precedence — it incorporates user input the static seed cannot.
+ */
+export declare function buildCreateModeBody(config: Pick<MetadataResourceConfig, 'createBuildBody' | 'createDefaults'>, draft: Record<string, unknown>, specCreateSeed: Record<string, unknown> | undefined): Record<string, unknown>;

package/dist/views/metadata-admin/createBody.js

@@ -0,0 +1,30 @@
+/**
+ * ObjectUI
+ * Copyright (c) 2024-present ObjectStack Inc.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+/**
+ * Build the create-mode save body for a new metadata item.
+ *
+ * Prefers the server's authoritative **create seed** (delivered per type on the
+ * `/meta/types` registry entry — the single source of truth in
+ * `@objectstack/spec`) over the locally hardcoded `createDefaults`. This is the
+ * drift-stop for the recurring "the designer emits a minimal shape the spec
+ * rejects, so create→save 422s" family (dashboard `layout`, action `body`):
+ * the structural defaults now come from the same place the spec validates
+ * against, so they cannot diverge. Falls back to `createDefaults` when the
+ * server provides no seed (older server, or canvas-create types whose shape is
+ * built interactively).
+ *
+ * User-supplied draft values always win over the seed's placeholders.
+ * `createBuildBody` (dynamic identity, e.g. a view's qualified name) still takes
+ * precedence — it incorporates user input the static seed cannot.
+ */
+export function buildCreateModeBody(config, draft, specCreateSeed) {
+    if (config.createBuildBody) {
+        return config.createBuildBody(draft);
+    }
+    return { ...(specCreateSeed ?? config.createDefaults ?? {}), ...draft };
+}