meno-core 1.0.53 → 1.1.0

package/lib/shared/propResolver.ts

@@ -52,7 +52,7 @@ export function richTextMarkerToHtml(marker: RichTextMarker): string {
  */
 function wrapRichTextProps(
   props: Record<string, unknown>,
-  propDefs: Record<string, PropDefinition>
+  propDefs: Record<string, PropDefinition>,
 ): Record<string, unknown> {
   const result = { ...props };
 
@@ -100,39 +100,35 @@ export interface ResolvePropsOptions {
 /**
  * Resolves component props by applying defaults and merging with passed props
  * Now includes validation for type safety
- * 
+ *
  * @param options - Object containing prop definitions, passed props, and children
  * @returns Resolved props object with defaults applied and validated
  */
 export function resolveComponentProps(options: ResolvePropsOptions): Record<string, unknown> {
   const { propDefs, passedProps, children = [], locale, i18nConfig } = options;
-  
+
   // Validate props against interface definition
   // This provides runtime type checking while maintaining backward compatibility
   try {
     const validationResult = validateComponentProps(propDefs, passedProps);
-    
+
     if (!validationResult.valid && validationResult.errors.length > 0) {
       // Log validation errors but continue (graceful degradation)
     }
-    
+
     // Use validated props (includes coerced values and defaults)
     const validatedProps = validationResult.props;
-    
+
     // Handle children prop separately (not validated by prop validator)
     const resolvedProps: Record<string, unknown> = { ...validatedProps };
-    
+
     // Apply children handling
     if ('children' in propDefs) {
       const childrenPropDef = propDefs.children;
       if (childrenPropDef && typeof childrenPropDef === 'object' && 'default' in childrenPropDef) {
-        resolvedProps.children = Array.isArray(children) && children.length > 0 
-          ? children 
-          : childrenPropDef.default;
+        resolvedProps.children = Array.isArray(children) && children.length > 0 ? children : childrenPropDef.default;
       } else {
-        resolvedProps.children = Array.isArray(children) && children.length > 0 
-          ? children 
-          : undefined;
+        resolvedProps.children = Array.isArray(children) && children.length > 0 ? children : undefined;
       }
     } else if (Array.isArray(children) && children.length > 0) {
       // Children passed but not in interface - allow it (backward compatibility)
@@ -150,27 +146,21 @@ export function resolveComponentProps(options: ResolvePropsOptions): Record<stri
     // Wrap rich-text props in marker objects for HTML rendering
     // Must happen AFTER i18n resolution so HTML strings get properly wrapped
     return wrapRichTextProps(result, propDefs);
-  } catch (error) {
+  } catch (_error) {
     // Fallback to original behavior if validation fails (backward compatibility)
     const resolvedProps: Record<string, unknown> = {};
-    
+
     // Apply defaults and passed props (original logic)
     for (const [propName, propDef] of Object.entries(propDefs)) {
       if (propDef && typeof propDef === 'object' && 'default' in propDef) {
         if (propName === 'children') {
-          resolvedProps.children = Array.isArray(children) && children.length > 0 
-            ? children 
-            : propDef.default;
+          resolvedProps.children = Array.isArray(children) && children.length > 0 ? children : propDef.default;
         } else {
-          resolvedProps[propName] = passedProps[propName] !== undefined 
-            ? passedProps[propName] 
-            : propDef.default;
+          resolvedProps[propName] = passedProps[propName] !== undefined ? passedProps[propName] : propDef.default;
         }
       } else {
         if (propName === 'children') {
-          resolvedProps.children = Array.isArray(children) && children.length > 0 
-            ? children 
-            : undefined;
+          resolvedProps.children = Array.isArray(children) && children.length > 0 ? children : undefined;
         } else {
           resolvedProps[propName] = passedProps[propName];
         }
@@ -197,7 +187,7 @@ export function resolvePropsFromDefinition(
   passedProps: Record<string, unknown> = {},
   children: Array<ComponentNode | string> | string | ComponentNode | null | undefined = [],
   locale?: string,
-  i18nConfig?: I18nConfig
+  i18nConfig?: I18nConfig,
 ): Record<string, unknown> {
   const propDefs = componentDef.interface || {};
   return resolveComponentProps({
@@ -208,4 +198,3 @@ export function resolvePropsFromDefinition(
     i18nConfig,
   });
 }
-

package/lib/shared/pxToRem.test.ts

@@ -90,14 +90,14 @@ describe('applyRemConversion', () => {
   });
 
   test('handles multiple declarations', () => {
-    expect(applyRemConversion('padding: 16px; margin: 8px;', enabledConfig))
-      .toBe('padding: 1rem; margin: 0.5rem');
+    expect(applyRemConversion('padding: 16px; margin: 8px;', enabledConfig)).toBe('padding: 1rem; margin: 0.5rem');
   });
 
   test('preserves excluded properties', () => {
     expect(applyRemConversion('border-width: 1px;', enabledConfig)).toBe('border-width: 1px');
-    expect(applyRemConversion('box-shadow: 0 1px 2px rgba(0,0,0,0.1);', enabledConfig))
-      .toBe('box-shadow: 0 1px 2px rgba(0,0,0,0.1)');
+    expect(applyRemConversion('box-shadow: 0 1px 2px rgba(0,0,0,0.1);', enabledConfig)).toBe(
+      'box-shadow: 0 1px 2px rgba(0,0,0,0.1)',
+    );
   });
 
   test('returns unchanged when disabled', () => {
@@ -109,7 +109,8 @@ describe('applyRemConversion', () => {
   });
 
   test('handles mixed convertible and non-convertible', () => {
-    expect(applyRemConversion('padding: 16px; border-width: 1px; margin: 8px;', enabledConfig))
-      .toBe('padding: 1rem; border-width: 1px; margin: 0.5rem');
+    expect(applyRemConversion('padding: 16px; border-width: 1px; margin: 8px;', enabledConfig)).toBe(
+      'padding: 1rem; border-width: 1px; margin: 0.5rem',
+    );
   });
 });

package/lib/shared/pxToRem.ts

@@ -50,15 +50,12 @@ export function shouldConvertProperty(cssProperty: string): boolean {
  * Apply rem conversion to a CSS declarations string (e.g. "padding: 16px; font-size: 18px").
  * Checks each property against the exclusion list before converting.
  */
-export function applyRemConversion(
-  declarations: string,
-  remConfig?: RemConversionConfig
-): string {
+export function applyRemConversion(declarations: string, remConfig?: RemConversionConfig): string {
   if (!remConfig?.enabled) return declarations;
 
   return declarations
     .split(';')
-    .map(decl => {
+    .map((decl) => {
       const trimmed = decl.trim();
       if (!trimmed) return '';
       const colonIndex = trimmed.indexOf(':');

package/lib/shared/registry/BaseNodeTypeRegistry.test.ts

@@ -4,7 +4,6 @@
 
 import { describe, it, expect, beforeEach } from 'bun:test';
 import { ClientNodeTypeRegistry } from './ClientNodeTypeRegistry';
-import { SSRNodeTypeRegistry } from './SSRNodeTypeRegistry';
 import { NodeTypeManager, globalNodeTypeManager } from './NodeTypeManager';
 import { defineNodeType } from './defineNodeType';
 import { registerBuiltInNodeTypes, builtInNodeTypes } from './nodeTypes';
@@ -42,10 +41,11 @@ describe('BaseNodeTypeRegistry', () => {
   describe('registerAll', () => {
     it('should register multiple node type definitions', () => {
       registry.registerAll(builtInNodeTypes);
-      expect(registry.size).toBe(7);  // 7 node types after cms-list merged into list
+      expect(registry.size).toBe(10); // node, component, slot, embed, link, locale-list, list, island, markdown, custom
       expect(registry.has(NODE_TYPE.NODE)).toBe(true);
       expect(registry.has(NODE_TYPE.COMPONENT)).toBe(true);
       expect(registry.has(NODE_TYPE.EMBED)).toBe(true);
+      expect(registry.has(NODE_TYPE.ISLAND)).toBe(true);
     });
   });
 
@@ -136,14 +136,18 @@ describe('BaseNodeTypeRegistry', () => {
   describe('subscribe', () => {
     it('should notify listeners on registration', () => {
       let called = false;
-      registry.subscribe(() => { called = true; });
+      registry.subscribe(() => {
+        called = true;
+      });
       registry.registerAll(builtInNodeTypes);
       expect(called).toBe(true);
     });
 
     it('should allow unsubscribing', () => {
       let callCount = 0;
-      const unsubscribe = registry.subscribe(() => { callCount++; });
+      const unsubscribe = registry.subscribe(() => {
+        callCount++;
+      });
       registry.registerAll(builtInNodeTypes);
       expect(callCount).toBe(1);
 
@@ -161,7 +165,7 @@ describe('BaseNodeTypeRegistry', () => {
     it('should filter by category', () => {
       const coreTypes = registry.getByCategory('core');
       expect(coreTypes.length).toBeGreaterThan(0);
-      expect(coreTypes.every(t => t.category === 'core')).toBe(true);
+      expect(coreTypes.every((t) => t.category === 'core')).toBe(true);
     });
   });
 });
@@ -185,6 +189,8 @@ describe('globalNodeTypeManager', () => {
     registerBuiltInNodeTypes();
     expect(globalNodeTypeManager.has(NODE_TYPE.NODE)).toBe(true);
     expect(globalNodeTypeManager.has(NODE_TYPE.EMBED)).toBe(true);
-    expect(globalNodeTypeManager.getAll().length).toBe(7);  // 7 node types after cms-list merged into list
+    expect(globalNodeTypeManager.has(NODE_TYPE.ISLAND)).toBe(true);
+    expect(globalNodeTypeManager.has(NODE_TYPE.MARKDOWN)).toBe(true);
+    expect(globalNodeTypeManager.getAll().length).toBe(10); // node, component, slot, embed, link, locale-list, list, island, markdown, custom
   });
 });

package/lib/shared/registry/ClientRegistry.ts

@@ -4,7 +4,6 @@
  */
 
 import { BaseComponentRegistry } from './ComponentRegistry';
-import type { ComponentDefinition } from '../types';
 
 export class ClientRegistry extends BaseComponentRegistry {
   /**
@@ -12,4 +11,3 @@ export class ClientRegistry extends BaseComponentRegistry {
    * For now, it's the same as base registry
    */
 }
-

package/lib/shared/registry/ComponentRegistry.test.ts

@@ -15,7 +15,7 @@ describe('BaseComponentRegistry', () => {
   describe('register', () => {
     test('registers a component', () => {
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
 
       registry.register('Button', component);
@@ -26,10 +26,10 @@ describe('BaseComponentRegistry', () => {
 
     test('overwrites existing component', () => {
       const component1: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
       const component2: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'span' } }
+        component: { structure: { type: 'node' as const, tag: 'span' } },
       };
 
       registry.register('Button', component1);
@@ -42,7 +42,7 @@ describe('BaseComponentRegistry', () => {
   describe('get', () => {
     test('returns component by name', () => {
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
 
       registry.register('Button', component);
@@ -58,7 +58,7 @@ describe('BaseComponentRegistry', () => {
   describe('has', () => {
     test('returns true for registered component', () => {
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
 
       registry.register('Button', component);
@@ -74,10 +74,10 @@ describe('BaseComponentRegistry', () => {
   describe('clear', () => {
     test('removes all components', () => {
       const component1: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
       const component2: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'span' } }
+        component: { structure: { type: 'node' as const, tag: 'span' } },
       };
 
       registry.register('Button', component1);
@@ -94,8 +94,8 @@ describe('BaseComponentRegistry', () => {
   describe('merge', () => {
     test('merges multiple components', () => {
       const components: Record<string, ComponentDefinition> = {
-        'Button': { component: { structure: { type: 'node' as const, tag: 'button' } } },
-        'Card': { component: { structure: { type: 'node' as const, tag: 'div' } } }
+        Button: { component: { structure: { type: 'node' as const, tag: 'button' } } },
+        Card: { component: { structure: { type: 'node' as const, tag: 'div' } } },
       };
 
       registry.merge(components);
@@ -106,14 +106,14 @@ describe('BaseComponentRegistry', () => {
 
     test('overwrites existing components', () => {
       const component1: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
       const component2: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
 
       registry.register('Button', component1);
-      registry.merge({ 'Button': component2 });
+      registry.merge({ Button: component2 });
 
       expect(registry.get('Button')).toEqual(component2);
     });
@@ -122,10 +122,10 @@ describe('BaseComponentRegistry', () => {
   describe('getAll', () => {
     test('returns all registered components', () => {
       const component1: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
       const component2: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
 
       registry.register('Button', component1);
@@ -133,20 +133,20 @@ describe('BaseComponentRegistry', () => {
 
       const all = registry.getAll();
 
-      expect(all['Button']).toEqual(component1);
-      expect(all['Card']).toEqual(component2);
+      expect(all.Button).toEqual(component1);
+      expect(all.Card).toEqual(component2);
       expect(Object.keys(all).length).toBe(2);
     });
 
     test('returns copy of registry', () => {
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
 
       registry.register('Button', component);
 
       const all = registry.getAll();
-      delete all['Button'];
+      delete all.Button;
 
       expect(registry.has('Button')).toBe(true);
     });
@@ -159,10 +159,10 @@ describe('BaseComponentRegistry', () => {
   describe('getNames', () => {
     test('returns list of component names', () => {
       const component1: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
       const component2: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'div' } }
+        component: { structure: { type: 'node' as const, tag: 'div' } },
       };
 
       registry.register('Button', component1);
@@ -183,7 +183,7 @@ describe('BaseComponentRegistry', () => {
   describe('remove', () => {
     test('removes component and returns true', () => {
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
 
       registry.register('Button', component);
@@ -204,12 +204,14 @@ describe('BaseComponentRegistry', () => {
   describe('subscribe', () => {
     test('notifies listeners on register', () => {
       let called = false;
-      const listener = () => { called = true; };
+      const listener = () => {
+        called = true;
+      };
 
       registry.subscribe(listener);
 
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
       registry.register('Button', component);
 
@@ -218,7 +220,9 @@ describe('BaseComponentRegistry', () => {
 
     test('notifies listeners on clear', () => {
       let called = false;
-      const listener = () => { called = true; };
+      const listener = () => {
+        called = true;
+      };
 
       registry.subscribe(listener);
 
@@ -229,7 +233,9 @@ describe('BaseComponentRegistry', () => {
 
     test('notifies listeners on merge', () => {
       let called = false;
-      const listener = () => { called = true; };
+      const listener = () => {
+        called = true;
+      };
 
       registry.subscribe(listener);
 
@@ -241,12 +247,14 @@ describe('BaseComponentRegistry', () => {
     test('notifies listeners on remove', () => {
       let called = false;
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
 
       registry.register('Button', component);
 
-      const listener = () => { called = true; };
+      const listener = () => {
+        called = true;
+      };
       registry.subscribe(listener);
 
       registry.remove('Button');
@@ -256,12 +264,14 @@ describe('BaseComponentRegistry', () => {
 
     test('returns unsubscribe function', () => {
       let callCount = 0;
-      const listener = () => { callCount++; };
+      const listener = () => {
+        callCount++;
+      };
 
       const unsubscribe = registry.subscribe(listener);
 
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
       registry.register('Button', component);
 
@@ -275,14 +285,18 @@ describe('BaseComponentRegistry', () => {
     test('handles multiple listeners', () => {
       let count1 = 0;
       let count2 = 0;
-      const listener1 = () => { count1++; };
-      const listener2 = () => { count2++; };
+      const listener1 = () => {
+        count1++;
+      };
+      const listener2 = () => {
+        count2++;
+      };
 
       registry.subscribe(listener1);
       registry.subscribe(listener2);
 
       const component: ComponentDefinition = {
-        component: { structure: { type: 'node' as const, tag: 'button' } }
+        component: { structure: { type: 'node' as const, tag: 'button' } },
       };
       registry.register('Button', component);
 

package/lib/shared/registry/ComponentRegistry.ts

@@ -5,6 +5,7 @@
  */
 
 import type { ComponentDefinition } from '../types';
+import { logRuntimeError } from '../errorLogger';
 
 /**
  * Base registry class with common functionality
@@ -12,7 +13,7 @@ import type { ComponentDefinition } from '../types';
 export abstract class BaseComponentRegistry {
   protected registry: Record<string, ComponentDefinition> = {};
   private listeners: Set<() => void> = new Set();
-  
+
   /**
    * Register a component with the given name
    */
@@ -20,21 +21,21 @@ export abstract class BaseComponentRegistry {
     this.registry[name] = definition;
     this.notify();
   }
-  
+
   /**
    * Get a component by name
    */
   get(name: string): ComponentDefinition | undefined {
     return this.registry[name];
   }
-  
+
   /**
    * Check if a component exists
    */
   has(name: string): boolean {
     return name in this.registry;
   }
-  
+
   /**
    * Clear all registered components
    */
@@ -42,7 +43,7 @@ export abstract class BaseComponentRegistry {
     this.registry = {};
     this.notify();
   }
-  
+
   /**
    * Merge components into the registry
    * Existing components with the same name will be overwritten
@@ -51,21 +52,21 @@ export abstract class BaseComponentRegistry {
     Object.assign(this.registry, components);
     this.notify();
   }
-  
+
   /**
    * Get all registered components
    */
   getAll(): Record<string, ComponentDefinition> {
     return { ...this.registry };
   }
-  
+
   /**
    * Get list of registered component names
    */
   getNames(): string[] {
     return Object.keys(this.registry);
   }
-  
+
   /**
    * Remove a component by name
    */
@@ -93,8 +94,10 @@ export abstract class BaseComponentRegistry {
       try {
         listener();
       } catch (err) {
+        // One throwing subscriber must not stop the rest from being notified, but
+        // a swallowed listener error hides real store-update bugs — surface it.
+        logRuntimeError('ComponentRegistry.notify', err);
       }
     }
   }
 }
-

package/lib/shared/registry/NodeTypeDefinition.ts

@@ -6,12 +6,22 @@
 import type { ReactElement } from 'react';
 import type { ZodType } from 'zod';
 import type { ComponentNode } from '../types/nodes';
-import type { StyleValue } from '../types/styles';
 
 /**
  * Tree icon types for node display
  */
-export type TreeIcon = 'TEXT' | 'COMPONENT' | 'HTML_ELEMENT' | 'IMAGE' | 'OBJECT' | 'SLOT_MARKER' | 'ARRAY' | 'UNKNOWN' | 'FORM' | 'LINK';
+export type TreeIcon =
+  | 'TEXT'
+  | 'COMPONENT'
+  | 'HTML_ELEMENT'
+  | 'IMAGE'
+  | 'OBJECT'
+  | 'SLOT_MARKER'
+  | 'ARRAY'
+  | 'UNKNOWN'
+  | 'FORM'
+  | 'LINK'
+  | 'CODE';
 
 /**
  * Node category for grouping in command palette
@@ -83,13 +93,13 @@ export interface ClientRenderContext {
     children: Array<ComponentNode | string> | string | undefined,
     elementPath: (string | number)[],
     parentComponentName?: string | null,
-    componentContext?: string | null
+    componentContext?: string | null,
   ) => ReactElement | ReactElement[] | string | number | null;
   registerElement: (
     path: (string | number)[],
     element: HTMLElement | null,
     parentComponentName?: string | null,
-    isComponentRoot?: boolean
+    isComponentRoot?: boolean,
   ) => void;
   getComponentDefinition: (name: string) => unknown | undefined;
   processI18n: (value: unknown) => string;
@@ -121,16 +131,13 @@ export interface SSRRenderContext {
  */
 export type ClientNodeRenderer<TNode extends ComponentNode> = (
   node: TNode,
-  context: ClientRenderContext
+  context: ClientRenderContext,
 ) => ReactElement | null;
 
 /**
  * SSR renderer function type
  */
-export type SSRNodeRenderer<TNode extends ComponentNode> = (
-  node: TNode,
-  context: SSRRenderContext
-) => string;
+export type SSRNodeRenderer<TNode extends ComponentNode> = (node: TNode, context: SSRRenderContext) => string;
 
 /**
  * Props editor component props

package/lib/shared/registry/RegistryManager.ts

@@ -36,7 +36,7 @@ export class RegistryManager {
   sync(from: 'client' | 'ssr', to: 'client' | 'ssr'): void {
     const source = from === 'client' ? this.clientRegistry : this.ssrRegistry;
     const target = to === 'client' ? this.clientRegistry : this.ssrRegistry;
-    
+
     const components = source.getAll();
     target.merge(components);
   }
@@ -57,4 +57,3 @@ export class RegistryManager {
     this.ssrRegistry.clear();
   }
 }
-

package/lib/shared/registry/SSRRegistry.ts

@@ -4,7 +4,6 @@
  */
 
 import { BaseComponentRegistry } from './ComponentRegistry';
-import type { ComponentDefinition } from '../types';
 
 export class SSRRegistry extends BaseComponentRegistry {
   /**
@@ -12,4 +11,3 @@ export class SSRRegistry extends BaseComponentRegistry {
    * For now, it's the same as base registry
    */
 }
-

package/lib/shared/registry/createNodeType.ts

@@ -33,10 +33,7 @@ const DEFAULT_CAPABILITIES: NodeCapabilities = {
  */
 function createTypeGuard<T>(typeValue: string) {
   return (node: unknown): node is T =>
-    node !== null &&
-    typeof node === 'object' &&
-    'type' in node &&
-    (node as { type: unknown }).type === typeValue;
+    node !== null && typeof node === 'object' && 'type' in node && (node as { type: unknown }).type === typeValue;
 }
 
 /**
@@ -118,7 +115,7 @@ export interface CreateNodeTypeInput<TSchema extends ZodType> {
  * ```
  */
 export function createNodeType<TSchema extends ZodType>(
-  input: CreateNodeTypeInput<TSchema>
+  input: CreateNodeTypeInput<TSchema>,
 ): NodeTypeDefinition<z.infer<TSchema>> {
   type TNode = z.infer<TSchema>;
 
@@ -152,10 +149,11 @@ export function createNodeType<TSchema extends ZodType>(
     typeGuard: createTypeGuard<TNode>(input.type),
 
     // Auto-generated default factory
-    defaultFactory: (): TNode => ({
-      type: input.type,
-      ...input.defaultValues,
-    } as TNode),
+    defaultFactory: (): TNode =>
+      ({
+        type: input.type,
+        ...input.defaultValues,
+      }) as TNode,
 
     treeDisplay: {
       icon: input.treeDisplay.icon,