refactor: extract enrichObject helper and add StackConfig type in handlers.ts

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

Author: Copilot

apps/console/src/mocks/handlers.ts

@@ -12,6 +12,11 @@ import { ObjectKernel } from '@objectstack/runtime';
 import { InMemoryDriver } from '@objectstack/driver-memory';
 import { http, HttpResponse } from 'msw';
 
+/** Minimal shape of the stack config used to enrich protocol responses. */
+interface StackConfig {
+  objects?: Array<{ name: string; listViews?: Record<string, unknown> }>;
+}
+
 /**
  * Create MSW request handlers for a given base URL.
  *
@@ -20,13 +25,13 @@ import { http, HttpResponse } from 'msw';
  * @param driver    - InMemoryDriver for direct data access
  * @param appConfig - Original stack config (used to enrich protocol responses with listViews)
  */
-export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: InMemoryDriver, appConfig?: any) {
+export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: InMemoryDriver, appConfig?: StackConfig) {
   const protocol = kernel.getService('protocol') as any;
 
   // Build a lookup of listViews by object name from the original config.
   // The runtime protocol strips listViews from object metadata, so we
   // re-attach them here to ensure the console can resolve named views.
-  const listViewsByObject: Record<string, Record<string, any>> = {};
+  const listViewsByObject: Record<string, Record<string, unknown>> = {};
   if (appConfig?.objects) {
     for (const obj of appConfig.objects) {
       if (obj.listViews && Object.keys(obj.listViews).length > 0) {
@@ -35,6 +40,12 @@ export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: In
     }
   }
 
+  /** Merge listViews from config into a single object metadata item. */
+  function enrichObject(obj: any): any {
+    const views = listViewsByObject[obj.name];
+    return views ? { ...obj, listViews: { ...(obj.listViews || {}), ...views } } : obj;
+  }
+
   // Determine whether we're in a browser (relative paths, wildcard prefix)
   // or in Node.js tests (absolute URLs)
   const isBrowser = !baseUrl.startsWith('http');
@@ -71,22 +82,15 @@ export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: In
       const response = await protocol.getMetaItems({ type: metadataType });
       // Enrich object metadata with listViews from stack config
       if ((metadataType === 'object' || metadataType === 'objects') && response?.items) {
-        response.items = response.items.map((obj: any) => {
-          const views = listViewsByObject[obj.name];
-          return views ? { ...obj, listViews: { ...(obj.listViews || {}), ...views } } : obj;
-        });
+        response.items = response.items.map(enrichObject);
       }
       return HttpResponse.json(response, { status: 200 });
     }),
     http.get(`${prefix}${baseUrl}/metadata/:type`, async ({ params }) => {
       const metadataType = params.type as string;
       const response = await protocol.getMetaItems({ type: metadataType });
-      // Enrich object metadata with listViews from stack config
       if ((metadataType === 'object' || metadataType === 'objects') && response?.items) {
-        response.items = response.items.map((obj: any) => {
-          const views = listViewsByObject[obj.name];
-          return views ? { ...obj, listViews: { ...(obj.listViews || {}), ...views } } : obj;
-        });
+        response.items = response.items.map(enrichObject);
       }
       return HttpResponse.json(response, { status: 200 });
     }),
@@ -99,12 +103,8 @@ export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: In
           name: params.name as string
         });
         let payload = (response && response.item) ? response.item : response;
-        // Enrich single object with listViews from stack config
-        if ((params.type === 'object' || params.type === 'objects') && payload && payload.name) {
-          const views = listViewsByObject[payload.name];
-          if (views) {
-            payload = { ...payload, listViews: { ...(payload.listViews || {}), ...views } };
-          }
+        if ((params.type === 'object' || params.type === 'objects') && payload?.name) {
+          payload = enrichObject(payload);
         }
         return HttpResponse.json(payload || { error: 'Not found' }, { status: payload ? 200 : 404 });
       } catch (e) {
@@ -118,12 +118,8 @@ export function createHandlers(baseUrl: string, kernel: ObjectKernel, driver: In
           name: params.name as string
         });
         let payload = (response && response.item) ? response.item : response;
-        // Enrich single object with listViews from stack config
-        if ((params.type === 'object' || params.type === 'objects') && payload && payload.name) {
-          const views = listViewsByObject[payload.name];
-          if (views) {
-            payload = { ...payload, listViews: { ...(payload.listViews || {}), ...views } };
-          }
+        if ((params.type === 'object' || params.type === 'objects') && payload?.name) {
+          payload = enrichObject(payload);
         }
         return HttpResponse.json(payload || { error: 'Not found' }, { status: payload ? 200 : 404 });
       } catch (e) {