refactor: remove patchDiscoveryEndpoints function and update ObjectForm test to access record properties

Author: hotlong

apps/console/src/App.tsx

@@ -19,22 +19,6 @@ import { PageView } from './components/PageView';
 import { ReportView } from './components/ReportView';
 import { ExpressionProvider } from './context/ExpressionProvider';
 
-/**
- * Patch: MSW discovery response uses 'routes' instead of 'endpoints'.
- * This normalizes the shape until @objectstack/plugin-msw is fixed upstream.
- */
-function patchDiscoveryEndpoints(adapter: ObjectStackAdapter) {
-  try {
-    const client = adapter.getClient();
-    const info = (client as any).discoveryInfo;
-    if (info?.routes && !info.endpoints) {
-      info.endpoints = info.routes;
-    }
-  } catch {
-    // Silently ignore — adapter may not expose getClient()
-  }
-}
-
 import { useParams } from 'react-router-dom';
 import { ThemeProvider } from './components/theme-provider';
 
@@ -83,10 +67,6 @@ export function AppContent() {
 
       await adapter.connect();
 
-      // Patch: MSW discovery returns 'routes' instead of 'endpoints'.
-      // TODO: Fix in @objectstack/plugin-msw to emit correct discovery shape.
-      patchDiscoveryEndpoints(adapter);
-
       setDataSource(adapter);
     } catch (err) {
       console.error('[Console] Failed to initialize:', err);

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

@@ -487,8 +487,8 @@ describe('ObjectForm with MSW Integration', () => {
 
       // Verify data is persisted by fetching it
       const result = await client.data.get('contact', createdId);
-      expect(result.name).toBe('Persist Test');
-      expect(result.email).toBe('persist@example.com');
+      expect(result.record.name).toBe('Persist Test');
+      expect(result.record.email).toBe('persist@example.com');
     });
   });
 });

apps/console/src/mocks/browser.ts

@@ -57,18 +57,7 @@ export async function startMockServer() {
     logRequests: true
   });
 
-  // WORKAROUND: ObjectKernel's PluginLoader uses object spread ({ ...plugin })
-  // which copies only own-enumerable properties, stripping prototype methods.
-  // This binds all prototype methods so they survive the spread.
-  // TODO: Remove once @objectstack/runtime fixes PluginLoader to preserve prototypes.
-  const fixedMswPlugin = { ...mswPlugin } as any;
-  for (const key of Object.getOwnPropertyNames(Object.getPrototypeOf(mswPlugin))) {
-    if (typeof (mswPlugin as any)[key] === 'function' && key !== 'constructor') {
-      fixedMswPlugin[key] = (mswPlugin as any)[key].bind(mswPlugin);
-    }
-  }
-
-  await kernel.use(fixedMswPlugin);
+  await kernel.use(mswPlugin);
 
   await kernel.bootstrap();