fix: include viewName in navigation href for object items

When a NavigationItem of type 'object' has a viewName (e.g. 'calendar',
'pipeline'), append /view/{viewName} to the generated href so the
correct list view is activated on navigation.

Changes:
- Add viewName property to NavigationItem interface
- Add viewName to Zod validation schema
- Update resolveHref in NavigationRenderer to append /view/{viewName}
- Update MobileBottomNav href generation in AppSchemaRenderer
- Add test for viewName href generation

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

Author: Copilot

packages/layout/src/AppSchemaRenderer.tsx

@@ -195,7 +195,10 @@ function MobileBottomNav({
       {leaves.map((item) => {
         const NavIcon = resolveIcon(item.icon);
         let href = '#';
-        if (item.type === 'object') href = `${basePath}/${item.objectName}`;
+        if (item.type === 'object') {
+          href = `${basePath}/${item.objectName}`;
+          if (item.viewName) href += `/view/${item.viewName}`;
+        }
         else if (item.type === 'dashboard') href = item.dashboardName ? `${basePath}/dashboard/${item.dashboardName}` : '#';
         else if (item.type === 'page') href = item.pageName ? `${basePath}/page/${item.pageName}` : '#';
         else if (item.type === 'report') href = item.reportName ? `${basePath}/report/${item.reportName}` : '#';

packages/layout/src/NavigationRenderer.tsx

@@ -184,8 +184,10 @@ const defaultPermission: PermissionChecker = () => true;
 
 function resolveHref(item: NavigationItem, basePath: string): { href: string; external: boolean } {
   switch (item.type) {
-    case 'object':
-      return { href: `${basePath}/${item.objectName ?? ''}`, external: false };
+    case 'object': {
+      const objectPath = `${basePath}/${item.objectName ?? ''}`;
+      return { href: item.viewName ? `${objectPath}/view/${item.viewName}` : objectPath, external: false };
+    }
     case 'dashboard':
       return { href: item.dashboardName ? `${basePath}/dashboard/${item.dashboardName}` : '#', external: false };
     case 'page':

packages/layout/src/__tests__/NavigationRenderer.test.tsx

@@ -110,6 +110,20 @@ describe('NavigationRenderer', () => {
     expect(link?.getAttribute('href')).toBe('/apps/test/account');
   });
 
+  it('renders object navigation item with viewName in href', () => {
+    const calendarItem: NavigationItem = {
+      id: 'nav-calendar',
+      type: 'object',
+      label: 'Calendar',
+      icon: 'Calendar',
+      objectName: 'event',
+      viewName: 'calendar',
+    };
+    renderNav([calendarItem]);
+    const link = screen.getByText('Calendar').closest('a');
+    expect(link?.getAttribute('href')).toBe('/apps/test/event/view/calendar');
+  });
+
   it('renders dashboard navigation item', () => {
     renderNav([dashboardItem]);
     expect(screen.getByText('Sales Dashboard')).toBeTruthy();

packages/types/src/app.ts

@@ -66,6 +66,9 @@ export interface NavigationItem {
   /** Target object name (for type: 'object') */
   objectName?: string;
 
+  /** Target view name (for type: 'object') — opens a specific named list view e.g. 'calendar', 'pipeline' */
+  viewName?: string;
+
   /** Target dashboard name (for type: 'dashboard') */
   dashboardName?: string;
 

packages/types/src/zod/app.zod.ts

@@ -41,6 +41,7 @@ export const NavigationItemSchema: z.ZodType<any> = z.lazy(() => z.object({
 
   // Type-specific target fields
   objectName: z.string().optional().describe('Target object name (type: object)'),
+  viewName: z.string().optional().describe('Target view name (type: object) — named list view e.g. calendar, pipeline'),
   dashboardName: z.string().optional().describe('Target dashboard name (type: dashboard)'),
   pageName: z.string().optional().describe('Target page name (type: page)'),
   reportName: z.string().optional().describe('Target report name (type: report)'),