fix: update dataSource handling in components and tests for consistency

Author: hotlong

apps/console/src/App.tsx

@@ -17,7 +17,6 @@ import { RecordDetailView } from './components/RecordDetailView';
 import { DashboardView } from './components/DashboardView';
 import { PageView } from './components/PageView';
 import { ReportView } from './components/ReportView';
-import { MetadataToggle, MetadataPanel, useMetadataInspector } from './components/MetadataInspector';
 import { ExpressionProvider } from './context/ExpressionProvider';
 
 /**
@@ -164,7 +163,7 @@ export function AppContent() {
       <Routes>
         <Route path="/" element={
             // Redirect to first route within the app
-             <Navigate to={findFirstRoute(activeApp.navigation)} replace />
+             <Navigate to={findFirstRoute(activeApp.navigation || [])} replace />
         } />
 
         {/* List View */}
@@ -199,7 +198,7 @@ export function AppContent() {
             <ReportView dataSource={dataSource} />
         } />
         <Route path="page/:pageName" element={
-            <PageView dataSource={dataSource} />
+            <PageView />
         } />
       </Routes>
       </ErrorBoundary>

apps/console/src/__tests__/ObjectForm.test.tsx

@@ -25,10 +25,10 @@ describe('ObjectForm with MSW Integration', () => {
     await startMockServer();
 
     // Initialize client - use localhost to match MSW handlers
-    client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' });
+    client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' } as any);
     await client.connect();
 
-    dataSource = new ObjectStackDataSource(client);
+    dataSource = new ObjectStackDataSource(client as any);
 
     // Get a valid contact ID for edit tests
     const driver = getDriver();

apps/console/src/__tests__/ObjectGrid.test.tsx

@@ -18,9 +18,9 @@ describe('ObjectGrid MSW Integration', () => {
 
   beforeAll(async () => {
     await startMockServer();
-    client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' });
+    client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' } as any);
     await client.connect();
-    dataSource = new ObjectStackDataSource(client);
+    dataSource = new ObjectStackDataSource(client as any);
   });
 
   afterAll(() => {

apps/console/src/__tests__/ObjectGridInteractions.test.tsx

@@ -28,9 +28,9 @@ describe('ObjectGrid Interactions', () => {
 
     beforeAll(async () => {
         await startMockServer();
-        client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' });
+        client = new ObjectStackClient({ baseUrl: 'http://localhost:3000' } as any);
         await client.connect();
-        dataSource = new ObjectStackDataSource(client);
+        dataSource = new ObjectStackDataSource(client as any);
     });
 
     afterAll(() => {

apps/console/src/components/PageView.tsx

@@ -9,7 +9,7 @@ import { Empty, EmptyTitle, EmptyDescription } from '@object-ui/components';
 import { MetadataToggle, MetadataPanel, useMetadataInspector } from './MetadataInspector';
 import appConfig from '../../objectstack.shared';
 
-export function PageView({ dataSource }: { dataSource?: any }) {
+export function PageView() {
   const { pageName } = useParams<{ pageName: string }>();
   const [searchParams] = useSearchParams();
   const { showDebug, toggleDebug } = useMetadataInspector();
@@ -42,7 +42,7 @@ export function PageView({ dataSource }: { dataSource?: any }) {
                 schema={{
                     type: 'page',
                     ...page,
-                    context: { ...page.context, params }
+                    context: { ...(page as any).context, params }
                 }} 
              />
         </div>

apps/console/src/components/ReportView.tsx

@@ -18,7 +18,7 @@ const MOCK_FIELDS = [
   { name: 'amount', label: 'Amount', type: 'currency' },
 ];
 
-export function ReportView({ dataSource }: { dataSource?: any }) {
+export function ReportView({ dataSource: _dataSource }: { dataSource?: any }) {
   const { reportName } = useParams<{ reportName: string }>();
   const { showDebug, toggleDebug } = useMetadataInspector();
   const [isEditing, setIsEditing] = useState(false);
@@ -99,7 +99,7 @@ export function ReportView({ dataSource }: { dataSource?: any }) {
       <div className="flex-1 overflow-hidden flex flex-row relative">
          <div className="flex-1 overflow-auto p-8 bg-muted/5">
             <div className="max-w-5xl mx-auto shadow-sm border rounded-xl bg-background overflow-hidden min-h-150">
-                <ReportViewer schema={viewerSchema} dataSource={dataSource} />
+                <ReportViewer schema={viewerSchema} />
             </div>
          </div>
 

packages/data-objectstack/src/errors.test.ts

@@ -270,7 +270,7 @@ describe('Error Helpers', () => {
 
       expect(error).toBeInstanceOf(MetadataNotFoundError);
       expect(error.statusCode).toBe(404);
-      expect((error as MetadataNotFoundError).details.objectName).toBe('users');
+      expect((error as MetadataNotFoundError).details?.objectName).toBe('users');
     });
 
     it('should create generic error for 404 without metadata context', () => {

packages/data-objectstack/src/index.ts

@@ -599,7 +599,7 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
       const cacheKey = `view:${objectName}:${viewId}`;
       return await this.metadataCache.get(cacheKey, async () => {
         // Try meta.getItem for view metadata
-        const result: any = await this.client.meta.getItem(`${objectName}/views/${viewId}`);
+        const result: any = await this.client.meta.getItem(objectName, `views/${viewId}`);
         if (result && result.item) return result.item;
         return result ?? null;
       });
@@ -624,7 +624,7 @@ export class ObjectStackAdapter<T = unknown> implements DataSource<T> {
     try {
       const cacheKey = `app:${appId}`;
       return await this.metadataCache.get(cacheKey, async () => {
-        const result: any = await this.client.meta.getItem(`apps/${appId}`);
+        const result: any = await this.client.meta.getItem('apps', appId);
         if (result && result.item) return result.item;
         return result ?? null;
       });