fix: address code review feedback

- Add comment clarifying empty path for notification list endpoint
- Remove misleading bodyTemplate from file upload endpoint
- Use toBeGreaterThanOrEqual in tests for catalog size assertions

Agent-Logs-Url: https://github.com/objectstack-ai/spec/sessions/35c2f5ae-4b72-4579-aab2-f9d563e61689

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

Author: Copilot

apps/studio/src/hooks/use-api-discovery.ts

@@ -110,7 +110,7 @@ export const SERVICE_ENDPOINT_CATALOG: Record<string, { group: string; defaultRo
     group: 'Notifications',
     defaultRoute: '/api/v1/notifications',
     endpoints: [
-      { method: 'GET', path: '', desc: 'List notifications' },
+      { method: 'GET', path: '', desc: 'List notifications' }, // empty path → hits the base route prefix
       { method: 'POST', path: '/devices', desc: 'Register device for push', bodyTemplate: { token: '', platform: 'web' } },
       { method: 'DELETE', path: '/devices/:deviceId', desc: 'Unregister device' },
       { method: 'GET', path: '/preferences', desc: 'Get notification preferences' },
@@ -166,7 +166,7 @@ export const SERVICE_ENDPOINT_CATALOG: Record<string, { group: string; defaultRo
     group: 'Storage',
     defaultRoute: '/api/v1/storage',
     endpoints: [
-      { method: 'POST', path: '/upload', desc: 'Upload file', bodyTemplate: {} },
+      { method: 'POST', path: '/upload', desc: 'Upload file (multipart/form-data)' },
       { method: 'GET', path: '/:fileId', desc: 'Download file' },
       { method: 'DELETE', path: '/:fileId', desc: 'Delete file' },
     ],

apps/studio/test/service-endpoint-catalog.test.ts

@@ -83,7 +83,7 @@ describe('buildServiceEndpoints', () => {
 
     it('should build workflow endpoints correctly', () => {
         const endpoints = buildServiceEndpoints('workflow', '/api/v1/workflow');
-        expect(endpoints.length).toBe(5);
+        expect(endpoints.length).toBeGreaterThanOrEqual(5);
 
         const methods = endpoints.map(e => e.method);
         expect(methods).toContain('GET');
@@ -96,7 +96,7 @@ describe('buildServiceEndpoints', () => {
 
     it('should build notification endpoints correctly', () => {
         const endpoints = buildServiceEndpoints('notification', '/api/v1/notifications');
-        expect(endpoints.length).toBe(7);
+        expect(endpoints.length).toBeGreaterThanOrEqual(7);
 
         // Should include list notifications at base path
         const listEp = endpoints.find(e => e.path === '/api/v1/notifications' && e.method === 'GET');