Merge branch 'pr/116'

Author: realfishsam

core/specs/polymarket/PolymarketClobAPI.yaml

@@ -994,6 +994,7 @@ paths:
   # ----------------------------------------------------------------------------
   /geoblock:
     get:
+      operationId: getGeoblock
       summary: Check Geoblock Status
       description: Check the geographic eligibility of the requesting IP address.
       tags: [System]

core/src/BaseExchange.ts

@@ -45,6 +45,11 @@ export interface ApiEndpoint {
     isPrivate?: boolean;
     /** Identifier used to generate the implicit API method name. */
     operationId?: string;
+    /**
+     * When set, requests use this base URL instead of the descriptor default
+     * (OpenAPI path- or operation-level `servers` override).
+     */
+    baseUrl?: string;
 }
 
 export interface ApiDescriptor {
@@ -1481,7 +1486,11 @@ export abstract class PredictionMarketExchange {
             if (name in this) {
                 continue;
             }
-            (this as any)[name] = this.createImplicitMethod(name, endpoint, descriptor.baseUrl);
+            (this as any)[name] = this.createImplicitMethod(
+                name,
+                endpoint,
+                endpoint.baseUrl ?? descriptor.baseUrl
+            );
         }
     }
 
@@ -1507,7 +1516,7 @@ export abstract class PredictionMarketExchange {
     private createImplicitMethod(
         name: string,
         endpoint: ApiEndpoint,
-        baseUrl: string
+        resolvedBaseUrl: string
     ): (params?: Record<string, any>) => Promise<any> {
         return async (params?: Record<string, any>): Promise<any> => {
             const allParams = { ...(params || {}) };
@@ -1531,7 +1540,7 @@ export abstract class PredictionMarketExchange {
                 headers = this.sign(endpoint.method, resolvedPath, allParams);
             }
 
-            const url = `${baseUrl}${resolvedPath}`;
+            const url = `${resolvedBaseUrl}${resolvedPath}`;
             const method = endpoint.method.toUpperCase();
 
             try {

core/src/exchanges/polymarket/api-clob.ts

@@ -517,6 +517,7 @@ export const polymarketClobSpec = {
         },
         "/geoblock": {
             "get": {
+                "operationId": "getGeoblock",
                 "summary": "Check Geoblock Status",
                 "tags": [
                     "System"

core/src/utils/openapi.ts

@@ -42,12 +42,27 @@ export function parseOpenApiSpec(spec: any, baseUrl?: string): ApiDescriptor {
     const topLevelSecurity = !!(spec.security && spec.security.length > 0);
 
     const paths = spec.paths || {};
-    for (const [path, methods] of Object.entries<any>(paths)) {
-        for (const [httpMethod, operation] of Object.entries<any>(methods)) {
-            // Skip non-HTTP-method keys like "parameters"
+    for (const [path, pathItem] of Object.entries<any>(paths)) {
+        if (!pathItem || typeof pathItem !== 'object') {
+            continue;
+        }
+        const pathServerUrl = Array.isArray(pathItem.servers) && pathItem.servers[0]?.url
+            ? String(pathItem.servers[0].url).replace(/\/$/, '')
+            : undefined;
+
+        for (const [httpMethod, operation] of Object.entries<any>(pathItem)) {
+            // Skip non-HTTP-method keys like "parameters", "servers"
             if (!['get', 'post', 'put', 'patch', 'delete'].includes(httpMethod.toLowerCase())) {
                 continue;
             }
+            if (!operation || typeof operation !== 'object') {
+                continue;
+            }
+
+            const opServerUrl = Array.isArray(operation.servers) && operation.servers[0]?.url
+                ? String(operation.servers[0].url).replace(/\/$/, '')
+                : undefined;
+            const endpointBaseUrl = opServerUrl || pathServerUrl;
 
             const name = operation.operationId || generateMethodName(httpMethod, path);
             const isPrivate = operation.security !== undefined
@@ -59,6 +74,7 @@ export function parseOpenApiSpec(spec: any, baseUrl?: string): ApiDescriptor {
                 path,
                 isPrivate,
                 operationId: operation.operationId,
+                ...(endpointBaseUrl ? { baseUrl: endpointBaseUrl } : {}),
             };
         }
     }

core/test/unit/openapi.test.ts

@@ -0,0 +1,45 @@
+import { polymarketClobSpec } from '../../src/exchanges/polymarket/api-clob';
+import { parseOpenApiSpec } from '../../src/utils/openapi';
+
+describe('parseOpenApiSpec', () => {
+    it('Polymarket getGeoblock uses website API host (not CLOB)', () => {
+        const d = parseOpenApiSpec(polymarketClobSpec);
+        expect(d.endpoints.getGeoblock.baseUrl).toBe('https://polymarket.com/api');
+        expect(d.endpoints.getGeoblock.path).toBe('/geoblock');
+    });
+
+    it('uses operation-level servers as endpoint baseUrl override', () => {
+        const spec = {
+            openapi: '3.0.3',
+            servers: [{ url: 'https://primary.example.com' }],
+            paths: {
+                '/x': {
+                    get: {
+                        operationId: 'getX',
+                        servers: [{ url: 'https://override.example.com/' }],
+                    },
+                },
+            },
+        };
+
+        const d = parseOpenApiSpec(spec);
+        expect(d.baseUrl).toBe('https://primary.example.com');
+        expect(d.endpoints.getX.baseUrl).toBe('https://override.example.com');
+    });
+
+    it('inherits path-level servers when the operation omits servers', () => {
+        const spec = {
+            openapi: '3.0.3',
+            servers: [{ url: 'https://primary.example.com' }],
+            paths: {
+                '/y': {
+                    servers: [{ url: 'https://path-level.example.com' }],
+                    get: { operationId: 'getY' },
+                },
+            },
+        };
+
+        const d = parseOpenApiSpec(spec);
+        expect(d.endpoints.getY.baseUrl).toBe('https://path-level.example.com');
+    });
+});