Align client discovery interface with spec API protocol

Co-authored-by: xuyushun441-sys <255036401+xuyushun441-sys@users.noreply.github.com>

Author: Copilot

packages/client/src/client.test.ts

@@ -15,7 +15,12 @@ describe('ObjectStackClient', () => {
     it('should make discovery request on connect', async () => {
         const fetchMock = vi.fn().mockResolvedValue({
             ok: true,
-            json: async () => ({ routes: { data: '/api/v1/data' } })
+            json: async () => ({ 
+                version: 'v1', 
+                apiName: 'ObjectStack',
+                capabilities: ['metadata', 'data', 'ui'],
+                endpoints: {}
+            })
         });
 
         const client = new ObjectStackClient({ 

packages/client/src/index.ts

@@ -8,7 +8,8 @@ import {
   MetadataCacheRequest,
   MetadataCacheResponse,
   StandardErrorCode,
-  ErrorCategory
+  ErrorCategory,
+  GetDiscoveryResponse
 } from '@objectstack/spec/api';
 import { Logger, createLogger } from '@objectstack/core';
 
@@ -29,16 +30,11 @@ export interface ClientConfig {
   debug?: boolean;
 }
 
-export interface DiscoveryResult {
-  routes: {
-    discovery: string;
-    metadata: string;
-    data: string;
-    auth: string;
-    ui: string;
-  };
-  capabilities?: Record<string, boolean>;
-}
+/**
+ * Discovery Result
+ * Re-export from @objectstack/spec/api for convenience
+ */
+export type DiscoveryResult = GetDiscoveryResponse;
 
 export interface QueryOptions {
   select?: string[]; // Simplified Selection
@@ -69,7 +65,7 @@ export class ObjectStackClient {
   private baseUrl: string;
   private token?: string;
   private fetchImpl: (input: RequestInfo | URL, init?: RequestInit) => Promise<Response>;
-  private routes?: DiscoveryResult['routes'];
+  private discoveryInfo?: DiscoveryResult;
   private logger: Logger;
 
   constructor(config: ClientConfig) {
@@ -87,21 +83,21 @@ export class ObjectStackClient {
   }
 
   /**
-   * Initialize the client by discovering server capabilities and routes.
+   * Initialize the client by discovering server capabilities.
    */
   async connect() {
     this.logger.debug('Connecting to ObjectStack server', { baseUrl: this.baseUrl });
 
     try {
-      // Connect to the discovery endpoint
-      // During boot, we might not know routes, so we check convention /api/v1 first
+      // Connect to the discovery endpoint at /api/v1
       const res = await this.fetch(`${this.baseUrl}/api/v1`);
 
       const data = await res.json();
-      this.routes = data.routes;
+      this.discoveryInfo = data;
 
       this.logger.info('Connected to ObjectStack server', { 
-        routes: Object.keys(data.routes || {}),
+        version: data.version,
+        apiName: data.apiName,
         capabilities: data.capabilities 
       });
 
@@ -407,16 +403,20 @@ export class ObjectStackClient {
     return res;
   }
 
-  private getRoute(key: keyof DiscoveryResult['routes']): string {
-    if (!this.routes) {
-        // Fallback for strictness, but we allow bootstrapping
-        this.logger.warn('Accessing route before connect()', { 
-          route: key, 
-          fallback: `/api/v1/${key}` 
-        });
-        return `/api/v1/${key}`;
-    }
-    return this.routes[key] || `/api/v1/${key}`; 
+  /**
+   * Get the conventional route path for a given API endpoint type
+   * ObjectStack uses standard conventions: /api/v1/data, /api/v1/meta, /api/v1/ui
+   */
+  private getRoute(type: 'data' | 'metadata' | 'ui' | 'auth'): string {
+    // Use conventional ObjectStack API paths
+    const routeMap: Record<string, string> = {
+      data: '/api/v1/data',
+      metadata: '/api/v1/meta',
+      ui: '/api/v1/ui',
+      auth: '/api/v1/auth'
+    };
+    
+    return routeMap[type] || `/api/v1/${type}`;
   }
 }
 
@@ -435,5 +435,6 @@ export type {
   MetadataCacheRequest,
   MetadataCacheResponse,
   StandardErrorCode,
-  ErrorCategory
+  ErrorCategory,
+  GetDiscoveryResponse
 } from '@objectstack/spec/api';