fix: address code review - validate stored width, extract constants, dedupe label resolver

- sidebar.tsx: Validate localStorage width against min/max bounds, handle NaN
- sidebar.tsx: Extract SIDEBAR_DEFAULT_WIDTH_PX and SIDEBAR_CLICK_THRESHOLD_PX constants
- header-bar.tsx: Use shared resolveI18nLabel from @object-ui/react instead of local duplicate
- breadcrumb.tsx: Use shared resolveI18nLabel from @object-ui/react instead of local duplicate

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/components/src/renderers/data-display/breadcrumb.tsx

@@ -10,13 +10,7 @@ import { ComponentRegistry } from '@object-ui/core';
 import type { BreadcrumbSchema } from '@object-ui/types';
 import { Breadcrumb, BreadcrumbList, BreadcrumbItem, BreadcrumbLink, BreadcrumbPage, BreadcrumbSeparator } from '../../ui/breadcrumb';
 import { renderChildren } from '../../lib/utils';
-
-/** Resolve a label that may be a plain string or an I18nLabel object */
-function resolveItemLabel(label: string | { key?: string; defaultValue?: string } | undefined): string {
-  if (!label) return '';
-  if (typeof label === 'string') return label;
-  return label.defaultValue || label.key || '';
-}
+import { resolveI18nLabel } from '@object-ui/react';
 
 ComponentRegistry.register('breadcrumb', 
   ({ schema, ...props }: { schema: BreadcrumbSchema; [key: string]: any }) => {
@@ -38,9 +32,9 @@ ComponentRegistry.register('breadcrumb',
             <div key={idx} className="flex items-center">
               <BreadcrumbItem>
                 {idx === (schema.items?.length || 0) - 1 ? (
-                  <BreadcrumbPage>{resolveItemLabel(item.label)}</BreadcrumbPage>
+                  <BreadcrumbPage>{resolveI18nLabel(item.label) ?? ''}</BreadcrumbPage>
                 ) : (
-                  <BreadcrumbLink href={item.href}>{resolveItemLabel(item.label)}</BreadcrumbLink>
+                  <BreadcrumbLink href={item.href}>{resolveI18nLabel(item.label) ?? ''}</BreadcrumbLink>
                 )}
               </BreadcrumbItem>
               {idx < (schema.items?.length || 0) - 1 && <BreadcrumbSeparator />}

packages/components/src/renderers/navigation/header-bar.tsx

@@ -9,6 +9,7 @@
 import React from 'react';
 import { ComponentRegistry } from '@object-ui/core';
 import type { HeaderBarSchema } from '@object-ui/types';
+import { resolveI18nLabel } from '@object-ui/react';
 import {
   SidebarTrigger,
   Separator,
@@ -20,13 +21,6 @@ import {
   BreadcrumbPage
 } from '../../ui';
 
-/** Resolve a label that may be a plain string or an I18nLabel object */
-function resolveCrumbLabel(label: string | { key?: string; defaultValue?: string } | undefined): string {
-  if (!label) return '';
-  if (typeof label === 'string') return label;
-  return label.defaultValue || label.key || '';
-}
-
 ComponentRegistry.register('header-bar', 
   ({ schema }: { schema: HeaderBarSchema }) => (
     <header className="flex h-14 sm:h-16 shrink-0 items-center gap-2 border-b px-3 sm:px-4">
@@ -38,9 +32,9 @@ ComponentRegistry.register('header-bar',
             <React.Fragment key={idx}>
               <BreadcrumbItem>
                 {idx === schema.crumbs.length - 1 ? (
-                   <BreadcrumbPage>{resolveCrumbLabel(crumb.label)}</BreadcrumbPage>
+                   <BreadcrumbPage>{resolveI18nLabel(crumb.label) ?? ''}</BreadcrumbPage>
                 ) : (
-                   <BreadcrumbLink href={crumb.href || '#'}>{resolveCrumbLabel(crumb.label)}</BreadcrumbLink>
+                   <BreadcrumbLink href={crumb.href || '#'}>{resolveI18nLabel(crumb.label) ?? ''}</BreadcrumbLink>
                 )}
               </BreadcrumbItem>
               {idx < schema.crumbs.length - 1 && <BreadcrumbSeparator />}