fix(registry): prefix every namespaced key exactly once, in every namespace (#3037)

#3023 fixed eleven record:* blocks registered as
register('record:x', …, { namespace: 'record' }) — an already-prefixed name
handed to a registry that prefixes it again, landing the block at
record:record:x — and guarded that namespace alone. Twenty-two more were
sitting in action: (5), element: (10) and page: (7), two of them
(page:header, element:divider) curated public blocks.

Checking one namespace is exactly what let them keep sitting there, so the
guard now asks the whole registry rather than a prefix of it.

Same fix as before: register the bare name and let `namespace` do the
prefixing, with skipFallback: true so the fallback does not claim that bare
name globally. It would otherwise take over `header`, `footer`, `sidebar`,
`tabs`, `card`, `accordion`, `section`, `text`, `image`, `button`, `icon` —
every one of which belongs to ui:. All 22 stay reachable exactly as
<namespace>:<name>; the registry goes 522 keys to 500, and the contract is
unchanged at 42/42.

Found while probing why six curated Tier B primitives report no `inputs`. They
do declare them — vitest.setup.dom.tsx registers simplified text / image /
html / grid stubs that shadow the real registrations inside the test
environment only. That shadowing is a separate question, left alone here; the
doubled keys it turned up are not test-environment artifacts.


Claude-Session: https://claude.ai/code/session_01N4mrr1ihhwnfEHFSWmGoMp

Co-authored-by: Claude <noreply@anthropic.com>

Author: os-zhuang

.changeset/namespace-double-prefix-sweep.md

@@ -0,0 +1,30 @@
+---
+"@object-ui/components": patch
+"@object-ui/console": patch
+---
+
+fix(registry): prefix every namespaced key exactly once, in every namespace
+
+objectui#3023 fixed eleven `record:*` blocks registered as
+`register('record:x', …, { namespace: 'record' })` — an already-prefixed name
+handed to a registry that prefixes it again, landing the block at
+`record:record:x` — and guarded that namespace alone. Twenty-two more were
+sitting in `action:` (5), `element:` (10) and `page:` (7), two of them
+(`page:header`, `element:divider`) curated public blocks.
+
+Checking one namespace is exactly what let them keep sitting there, so the
+guard now asks the whole registry rather than a prefix of it.
+
+Same fix as before: register the bare name and let `namespace` do the
+prefixing, with `skipFallback: true` so the fallback does not claim that bare
+name globally. It would otherwise take over `header`, `footer`, `sidebar`,
+`tabs`, `card`, `accordion`, `section`, `text`, `image`, `button`, `icon` —
+every one of which belongs to `ui:`. All 22 stay reachable exactly as
+`<namespace>:<name>`; the registry goes 522 keys to 500, and the contract is
+unchanged at 42/42.
+
+Found while probing why six curated Tier B primitives report no `inputs`. They
+do declare them — `vitest.setup.dom.tsx` registers simplified `text` / `image` /
+`html` / `grid` stubs that shadow the real registrations inside the test
+environment only. That shadowing is a separate question, left alone here; the
+doubled keys it turned up are not test-environment artifacts.

apps/console/src/__tests__/public-contract.test.ts

@@ -202,14 +202,7 @@ describe('PUBLIC_BLOCKS ↔ console coverage (reverse direction)', () => {
   });
 
   it('registers each record:* block under one key, prefixed once', () => {
-    // `register('record:x', …, { namespace: 'record' })` prefixes an
-    // already-prefixed name: the block lands at `record:record:x` and stays
-    // reachable only through the un-namespaced fallback, which happens to spell
-    // `record:x`. Eleven blocks were registered that way. The contract hid it —
-    // `getPublicConfigs()` rewrites `type` to the curated tag — so nothing
-    // failed while the registry carried a phantom key per block.
     const keys = shippedRecordBlocks();
-    expect(keys.filter((k) => k.startsWith(`${NS}:${NS}:`))).toEqual([]);
     expect(keys.every((k) => k.startsWith(`${NS}:`))).toBe(true);
   });
 
@@ -224,6 +217,27 @@ describe('PUBLIC_BLOCKS ↔ console coverage (reverse direction)', () => {
     }
   });
 
+  it('prefixes every namespaced key exactly once, in every namespace', () => {
+    // `register('page:header', …, { namespace: 'page' })` hands an
+    // already-prefixed name to a registry that prefixes it again: the block
+    // lands at `page:page:header` and stays reachable only through the
+    // un-namespaced fallback, which happens to spell `page:header`. Nothing
+    // fails — `getPublicConfigs()` rewrites `type` to the curated tag — so the
+    // registry quietly carries a phantom key per block.
+    //
+    // objectui#3023 fixed the eleven in `record:` and guarded that namespace
+    // alone. Twenty-two more were sitting in `action:`, `element:` and `page:`,
+    // two of them (`page:header`, `element:divider`) curated public blocks.
+    // Checking one namespace is what let them keep sitting there, so this asks
+    // the whole registry.
+    const doubled = ComponentRegistry.getKnownTypes().filter((k) => {
+      const ns = ComponentRegistry.getMeta(k)?.namespace;
+      return !!ns && k.startsWith(`${ns}:${ns}:`);
+    });
+
+    expect(doubled).toEqual([]);
+  });
+
   it('keeps the chatter alias identical to the block it aliases', () => {
     // `record:chatter` is excluded because it duplicates `record:discussion`,
     // not because it is lesser. The moment the two configuration surfaces

packages/components/src/renderers/action/action-bar.tsx

@@ -282,8 +282,9 @@ const ActionBarRenderer = forwardRef<HTMLDivElement, { schema: ActionBarSchema;
 
 ActionBarRenderer.displayName = 'ActionBarRenderer';
 
-ComponentRegistry.register('action:bar', ActionBarRenderer, {
+ComponentRegistry.register('bar', ActionBarRenderer, {
   namespace: 'action',
+  skipFallback: true,
   label: 'Action Bar',
   inputs: [
     { name: 'actions', type: 'object', label: 'Actions' },

packages/components/src/renderers/action/action-button.tsx

@@ -175,8 +175,9 @@ const ActionButtonRenderer = forwardRef<HTMLButtonElement, ActionButtonProps>(
 
 ActionButtonRenderer.displayName = 'ActionButtonRenderer';
 
-ComponentRegistry.register('action:button', ActionButtonRenderer, {
+ComponentRegistry.register('button', ActionButtonRenderer, {
   namespace: 'action',
+  skipFallback: true,
   label: 'Action Button',
   inputs: [
     { name: 'name', type: 'string', label: 'Action Name' },

packages/components/src/renderers/action/action-group.tsx

@@ -294,8 +294,9 @@ const ActionGroupRenderer = forwardRef<HTMLDivElement, { schema: ActionGroupSche
 
 ActionGroupRenderer.displayName = 'ActionGroupRenderer';
 
-ComponentRegistry.register('action:group', ActionGroupRenderer, {
+ComponentRegistry.register('group', ActionGroupRenderer, {
   namespace: 'action',
+  skipFallback: true,
   label: 'Action Group',
   inputs: [
     { name: 'name', type: 'string', label: 'Group Name' },

packages/components/src/renderers/action/action-icon.tsx

@@ -130,8 +130,9 @@ const ActionIconRenderer = forwardRef<HTMLButtonElement, ActionIconProps>(
 
 ActionIconRenderer.displayName = 'ActionIconRenderer';
 
-ComponentRegistry.register('action:icon', ActionIconRenderer, {
+ComponentRegistry.register('icon', ActionIconRenderer, {
   namespace: 'action',
+  skipFallback: true,
   label: 'Action Icon',
   inputs: [
     { name: 'name', type: 'string', label: 'Action Name' },

packages/components/src/renderers/action/action-menu.tsx

@@ -220,8 +220,9 @@ const ActionMenuRenderer = forwardRef<HTMLButtonElement, { schema: ActionMenuSch
 
 ActionMenuRenderer.displayName = 'ActionMenuRenderer';
 
-ComponentRegistry.register('action:menu', ActionMenuRenderer, {
+ComponentRegistry.register('menu', ActionMenuRenderer, {
   namespace: 'action',
+  skipFallback: true,
   label: 'Action Menu',
   inputs: [
     { name: 'label', type: 'string', label: 'Trigger Label' },

packages/components/src/renderers/basic/data-list.tsx

@@ -77,8 +77,9 @@ function DefinitionListRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:definition-list', DefinitionListRenderer, {
+ComponentRegistry.register('definition-list', DefinitionListRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Definition List',
   category: 'content',
 });
@@ -172,8 +173,9 @@ function RepeaterRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:repeater', RepeaterRenderer, {
+ComponentRegistry.register('repeater', RepeaterRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Repeater',
   category: 'content',
 });

packages/components/src/renderers/basic/elements.tsx

@@ -92,8 +92,9 @@ function ElementTextRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:text', ElementTextRenderer, {
+ComponentRegistry.register('text', ElementTextRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Text',
   category: 'content',
 });
@@ -106,8 +107,9 @@ function ElementDividerRenderer({ schema }: { schema: any }) {
   return <Separator className={cn('my-4', schema?.className)} />;
 }
 
-ComponentRegistry.register('element:divider', ElementDividerRenderer, {
+ComponentRegistry.register('divider', ElementDividerRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Divider',
   category: 'content',
 });
@@ -155,8 +157,9 @@ function ElementImageRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:image', ElementImageRenderer, {
+ComponentRegistry.register('image', ElementImageRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Image',
   category: 'content',
 });
@@ -260,8 +263,9 @@ function ElementButtonRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:button', ElementButtonRenderer, {
+ComponentRegistry.register('button', ElementButtonRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Button',
   category: 'action',
 });
@@ -381,8 +385,9 @@ function ElementNumberRenderer({ schema }: { schema: any }) {
   );
 }
 
-ComponentRegistry.register('element:number', ElementNumberRenderer, {
+ComponentRegistry.register('number', ElementNumberRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Number',
   category: 'content',
 });

packages/components/src/renderers/basic/metadata-viewer.tsx

@@ -379,8 +379,9 @@ export function ElementMetadataViewerRenderer({ schema }: { schema: any }) {
   }
 }
 
-ComponentRegistry.register('element:metadata_viewer', ElementMetadataViewerRenderer, {
+ComponentRegistry.register('metadata_viewer', ElementMetadataViewerRenderer, {
   namespace: 'element',
+  skipFallback: true,
   label: 'Metadata Viewer',
   category: 'content',
 });