Fix discovery endpoint returning routes without /api/v1 prefix on Vercel

Root cause: all adapters registered discovery only at GET prefix (root),
not at GET prefix/discovery. When frontend fetched /api/v1/discovery, the
catch-all stripped the prefix and called dispatch('/discovery') which called
getDiscoveryInfo('') with empty prefix, returning /data instead of /api/v1/data.

Locally this was hidden because MSW mode doesn't register real services,
so the wrong routes were never used. On Vercel, all services are registered,
breaking the API console.

Fixes:
- http-dispatcher: add optional prefix param to dispatch() so discovery
  handler uses the correct prefix instead of empty string
- hono adapter: add explicit GET prefix/discovery route (Vercel fix)
- fastify, nuxt adapters: add explicit GET prefix/discovery route
- sveltekit, nextjs adapters: handle 'discovery' segment before catch-all
- express adapter: pass prefix to dispatch() (already had explicit route)
- MSW plugin: add explicit baseUrl/discovery handler + pass baseUrl to dispatch()
- Revert wrong protocol.ts/rest-server.ts/spec changes from previous commit
- Update hono test assertions for new 6th prefix parameter

Agent-Logs-Url: https://github.com/objectstack-ai/framework/sessions/ae9fa3e0-4bf7-45e8-b4ee-4816ad2f6917

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

Author: Copilot

packages/adapters/express/src/index.ts

@@ -167,7 +167,7 @@ export function createExpressRouter(options: ExpressAdapterOptions): Router {
       const subPath = '/' + (req.params as any).path;
       const method = req.method;
       const body = (method === 'POST' || method === 'PUT' || method === 'PATCH') ? req.body : undefined;
-      const result = await dispatcher.dispatch(method, subPath, body, req.query, { request: req, response: res });
+      const result = await dispatcher.dispatch(method, subPath, body, req.query, { request: req, response: res }, prefix);
       return sendResult(result, res);
     } catch (err: any) {
       return errorResponse(err, res);

packages/adapters/fastify/src/index.ts

@@ -80,6 +80,10 @@ export async function objectStackPlugin(fastify: FastifyInstance, options: Fasti
     return reply.send({ data: await dispatcher.getDiscoveryInfo(prefix) });
   });
 
+  fastify.get(`${prefix}/discovery`, async (_request: FastifyRequest, reply: FastifyReply) => {
+    return reply.send({ data: await dispatcher.getDiscoveryInfo(prefix) });
+  });
+
   // --- .well-known ---
   fastify.get('/.well-known/objectstack', async (_request: FastifyRequest, reply: FastifyReply) => {
     return reply.redirect(prefix);
@@ -172,7 +176,7 @@ export async function objectStackPlugin(fastify: FastifyInstance, options: Fasti
       const subPath = urlPath.substring(prefix.length);
       const method = request.method;
       const body = (method === 'POST' || method === 'PUT' || method === 'PATCH') ? request.body : undefined;
-      const result = await dispatcher.dispatch(method, subPath, body, request.query, { request: request.raw });
+      const result = await dispatcher.dispatch(method, subPath, body, request.query, { request: request.raw }, prefix);
       return sendResult(result, reply);
     } catch (err: any) {
       return errorResponse(err, reply);

packages/adapters/hono/src/hono.test.ts

@@ -97,6 +97,14 @@ describe('createHonoApp', () => {
       expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/api');
     });
 
+    it('GET /api/discovery returns discovery info with correct prefix', async () => {
+      const res = await app.request('/api/discovery');
+      expect(res.status).toBe(200);
+      const json = await res.json();
+      expect(json.data).toBeDefined();
+      expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/api');
+    });
+
     it('uses custom prefix for discovery', async () => {
       const customApp = createHonoApp({ kernel: mockKernel, prefix: '/v2' });
       const res = await customApp.request('/v2');
@@ -105,6 +113,15 @@ describe('createHonoApp', () => {
       expect(json.data).toBeDefined();
       expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/v2');
     });
+
+    it('uses custom prefix for /discovery route', async () => {
+      const customApp = createHonoApp({ kernel: mockKernel, prefix: '/v2' });
+      const res = await customApp.request('/v2/discovery');
+      expect(res.status).toBe(200);
+      const json = await res.json();
+      expect(json.data).toBeDefined();
+      expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/v2');
+    });
   });
 
   describe('.well-known Endpoint', () => {
@@ -277,6 +294,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -294,6 +312,7 @@ describe('createHonoApp', () => {
         body,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -306,6 +325,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -318,6 +338,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.objectContaining({ package: 'com.acme.crm' }),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -330,6 +351,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -347,6 +369,7 @@ describe('createHonoApp', () => {
         body,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -364,6 +387,7 @@ describe('createHonoApp', () => {
         body,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -384,6 +408,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -396,6 +421,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -413,6 +439,7 @@ describe('createHonoApp', () => {
         body,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -425,6 +452,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.objectContaining({ status: 'active' }),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -446,6 +474,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -458,6 +487,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -470,6 +500,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
 
@@ -482,6 +513,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api',
       );
     });
   });
@@ -571,6 +603,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api/v1',
       );
     });
 
@@ -585,6 +618,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api/v1',
       );
     });
 
@@ -598,6 +632,16 @@ describe('createHonoApp', () => {
       expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/api/v1');
     });
 
+    it('routes /api/v1/discovery through outer→inner delegation with correct prefix', async () => {
+      const outerApp = createVercelApp();
+
+      const res = await outerApp.request('/api/v1/discovery');
+      expect(res.status).toBe(200);
+      const json = await res.json();
+      expect(json.data).toBeDefined();
+      expect(mockDispatcher.getDiscoveryInfo).toHaveBeenCalledWith('/api/v1');
+    });
+
     it('routes /api/v1/data/account through outer→inner delegation', async () => {
       const outerApp = createVercelApp();
 
@@ -609,6 +653,7 @@ describe('createHonoApp', () => {
         undefined,
         expect.any(Object),
         expect.objectContaining({ request: expect.anything() }),
+        '/api/v1',
       );
     });
 

packages/adapters/hono/src/index.ts

@@ -112,6 +112,10 @@ export function createHonoApp(options: ObjectStackHonoOptions): Hono {
     return c.json({ data: await dispatcher.getDiscoveryInfo(prefix) });
   });
 
+  app.get(`${prefix}/discovery`, async (c) => {
+    return c.json({ data: await dispatcher.getDiscoveryInfo(prefix) });
+  });
+
   // --- .well-known ---
   app.get('/.well-known/objectstack', (c) => {
     return c.redirect(prefix);
@@ -202,7 +206,7 @@ export function createHonoApp(options: ObjectStackHonoOptions): Hono {
       const url = new URL(c.req.url);
       url.searchParams.forEach((val, key) => { queryParams[key] = val; });
 
-      const result = await dispatcher.dispatch(method, subPath, body, queryParams, { request: c.req.raw });
+      const result = await dispatcher.dispatch(method, subPath, body, queryParams, { request: c.req.raw }, prefix);
       return toResponse(c, result);
     } catch (err: any) {
       return errorJson(c, err.message || 'Internal Server Error', err.statusCode || 500);

packages/adapters/nextjs/src/index.ts

@@ -66,6 +66,10 @@ export function createRouteHandler(options: NextAdapterOptions) {
       return NextResponse.json({ data: await dispatcher.getDiscoveryInfo(options.prefix || '/api') });
     }
 
+    if (segments.length === 1 && segments[0] === 'discovery' && method === 'GET') {
+      return NextResponse.json({ data: await dispatcher.getDiscoveryInfo(options.prefix || '/api') });
+    }
+
     try {
         const rawRequest = req;
 
@@ -135,7 +139,7 @@ export function createRouteHandler(options: NextAdapterOptions) {
         const queryParams: Record<string, any> = {};
         url.searchParams.forEach((val, key) => queryParams[key] = val);
 
-        const result = await dispatcher.dispatch(method, path, body, queryParams, { request: rawRequest });
+        const result = await dispatcher.dispatch(method, path, body, queryParams, { request: rawRequest }, options.prefix || '/api');
         return toResponse(result);
 
     } catch (err: any) {

packages/adapters/nuxt/src/index.ts

@@ -95,6 +95,13 @@ export function createH3Router(options: NuxtAdapterOptions): Router {
     }),
   );
 
+  router.get(
+    `${prefix}/discovery`,
+    defineEventHandler(async () => {
+      return { data: await dispatcher.getDiscoveryInfo(prefix) };
+    }),
+  );
+
   // --- .well-known ---
   router.get(
     '/.well-known/objectstack',
@@ -210,7 +217,7 @@ export function createH3Router(options: NuxtAdapterOptions): Router {
           ? await readBody(event)
           : undefined;
         const query = getQuery(event);
-        const result = await dispatcher.dispatch(method, subPath, body, query, { request: event.node.req });
+        const result = await dispatcher.dispatch(method, subPath, body, query, { request: event.node.req }, prefix);
         return toResponse(event, result);
       } catch (err: any) {
         return errorJson(event, err.message || 'Internal Server Error', err.statusCode || 500);

packages/adapters/sveltekit/src/index.ts

@@ -106,6 +106,13 @@ export function createRequestHandler(options: SvelteKitAdapterOptions) {
       });
     }
 
+    if (segments.length === 1 && segments[0] === 'discovery' && method === 'GET') {
+      return new Response(JSON.stringify({ data: await dispatcher.getDiscoveryInfo(prefix) }), {
+        status: 200,
+        headers: { 'Content-Type': 'application/json' },
+      });
+    }
+
     try {
       // --- Auth (needs auth service integration) ---
       if (segments[0] === 'auth') {
@@ -176,7 +183,7 @@ export function createRequestHandler(options: SvelteKitAdapterOptions) {
       const queryParams: Record<string, any> = {};
       url.searchParams.forEach((val, key) => { queryParams[key] = val; });
 
-      const result = await dispatcher.dispatch(method, subPath, body, queryParams, { request });
+      const result = await dispatcher.dispatch(method, subPath, body, queryParams, { request }, prefix);
       return toResponse(result);
     } catch (err: any) {
       return errorJson(err.message || 'Internal Server Error', err.statusCode || 500);