@finatic/client 0.0.140 → 0.0.142

package/src/core/client/ApiClient.ts

@@ -16,11 +16,22 @@ import {
   PositionsFilter,
   AccountsFilter,
   BalancesFilter,
+  OrderFill,
+  OrderEvent,
+  OrderGroup,
+  PositionLot,
+  PositionLotFill,
+  OrderFillsFilter,
+  OrderEventsFilter,
+  OrderGroupsFilter,
+  PositionLotsFilter,
+  PositionLotFillsFilter,
 } from '../../types/api/broker';
 import { TradingContext } from '../../types/api/orders';
 import { ApiPaginationInfo, PaginatedResult } from '../../types/common/pagination';
 import { ApiResponse } from '../../types/api/core';
 import { PortalUrlResponse } from '../../types/api/core';
+import { setupLogger, buildLoggerExtra, LoggerExtra } from '../../lib/logger';
 import {
   DeviceInfo,
   SessionState,
@@ -60,6 +71,17 @@ export class ApiClient {
   // Session and company context
   private companyId: string | null = null;
   private csrfToken: string | null = null;
+  private readonly logger = setupLogger('FinaticClientSDK.ApiClient', undefined, {
+    codebase: 'FinaticClientSDK',
+  });
+
+  private buildLoggerExtra(functionName: string, metadata?: Record<string, unknown>): LoggerExtra {
+    return {
+      module: 'ApiClient',
+      function: functionName,
+      ...(metadata ? buildLoggerExtra(metadata) : {}),
+    };
+  }
 
   constructor(baseUrl: string, deviceInfo?: DeviceInfo) {
     this.baseUrl = baseUrl;
@@ -213,15 +235,11 @@ export class ApiClient {
       }
     });
 
-    // Debug logging for development
-    if (typeof window !== 'undefined' && window.location.hostname === 'localhost') {
-      console.log('Request to:', url.toString());
-      console.log('Safari-safe headers:', safariSafeHeaders);
-      console.log('Browser:', navigator.userAgent);
-      console.log('Session ID:', this.currentSessionId);
-      console.log('Company ID:', this.companyId);
-      console.log('CSRF Token:', this.csrfToken);
-    }
+    this.logger.debug('Dispatching API request', this.buildLoggerExtra('request', {
+      url: url.toString(),
+      method: options.method,
+      has_body: Boolean(options.body),
+    }));
 
     const response = await fetch(url.toString(), {
       method: options.method,
@@ -229,16 +247,19 @@ export class ApiClient {
       body: options.body ? JSON.stringify(options.body) : undefined,
     });
 
-    // Debug logging for response
-    if (typeof window !== 'undefined' && window.location.hostname === 'localhost') {
-      console.log('Response status:', response.status);
-      console.log('Response headers:', Object.fromEntries(response.headers.entries()));
-      console.log('Request was made with headers:', safariSafeHeaders);
-    }
+    this.logger.debug('Received API response', this.buildLoggerExtra('request', {
+      url: url.toString(),
+      status: response.status,
+    }));
 
     if (!response.ok) {
       const error = await response.json();
-      throw this.handleError(response.status, error);
+      const apiError = this.handleError(response.status, error);
+      this.logger.exception('API request failed', apiError, this.buildLoggerExtra('request', {
+        url: url.toString(),
+        status: response.status,
+      }));
+      throw apiError;
     }
 
     const data = await response.json();
@@ -251,12 +272,22 @@ export class ApiClient {
         // Add context that this is an order-related error
         data._isOrderError = true;
       }
-      throw this.handleError(data.status_code || 500, data);
+      const apiError = this.handleError(data.status_code || 500, data);
+      this.logger.exception('API response indicated failure', apiError, this.buildLoggerExtra('request', {
+        url: url.toString(),
+        status: data.status_code || 500,
+      }));
+      throw apiError;
     }
 
     // Check if the response has a status_code field indicating an error (4xx or 5xx)
     if (data && typeof data === 'object' && 'status_code' in data && data.status_code >= 400) {
-      throw this.handleError(data.status_code, data);
+      const apiError = this.handleError(data.status_code, data);
+      this.logger.exception('API status code error', apiError, this.buildLoggerExtra('request', {
+        url: url.toString(),
+        status: data.status_code,
+      }));
+      throw apiError;
     }
 
     // Check if the response has errors field with content
@@ -268,7 +299,12 @@ export class ApiClient {
       Array.isArray(data.errors) &&
       data.errors.length > 0
     ) {
-      throw this.handleError(data.status_code || 500, data);
+      const apiError = this.handleError(data.status_code || 500, data);
+      this.logger.exception('API response contained errors', apiError, this.buildLoggerExtra('request', {
+        url: url.toString(),
+        status: data.status_code || 500,
+      }));
+      throw apiError;
     }
 
     return data;
@@ -1718,4 +1754,251 @@ export class ApiClient {
       },
     });
   }
+
+  /**
+   * Get order fills for a specific order
+   * @param orderId - The order ID
+   * @param filter - Optional filter parameters
+   * @returns Promise with order fills response
+   */
+  async getOrderFills(
+    orderId: string,
+    filter?: OrderFillsFilter
+  ): Promise<{
+    _id: string;
+    response_data: OrderFill[];
+    message: string;
+    status_code: number;
+    warnings: null;
+    errors: null;
+  }> {
+    const accessToken = await this.getValidAccessToken();
+    const params: Record<string, string> = {};
+
+    if (filter?.connection_id) {
+      params.connection_id = filter.connection_id;
+    }
+    if (filter?.limit) {
+      params.limit = filter.limit.toString();
+    }
+    if (filter?.offset) {
+      params.offset = filter.offset.toString();
+    }
+
+    return this.request<{
+      _id: string;
+      response_data: OrderFill[];
+      message: string;
+      status_code: number;
+      warnings: null;
+      errors: null;
+    }>(`/brokers/data/orders/${orderId}/fills`, {
+      method: 'GET',
+      headers: {
+        Authorization: `Bearer ${accessToken}`,
+      },
+      params,
+    });
+  }
+
+  /**
+   * Get order events for a specific order
+   * @param orderId - The order ID
+   * @param filter - Optional filter parameters
+   * @returns Promise with order events response
+   */
+  async getOrderEvents(
+    orderId: string,
+    filter?: OrderEventsFilter
+  ): Promise<{
+    _id: string;
+    response_data: OrderEvent[];
+    message: string;
+    status_code: number;
+    warnings: null;
+    errors: null;
+  }> {
+    const accessToken = await this.getValidAccessToken();
+    const params: Record<string, string> = {};
+
+    if (filter?.connection_id) {
+      params.connection_id = filter.connection_id;
+    }
+    if (filter?.limit) {
+      params.limit = filter.limit.toString();
+    }
+    if (filter?.offset) {
+      params.offset = filter.offset.toString();
+    }
+
+    return this.request<{
+      _id: string;
+      response_data: OrderEvent[];
+      message: string;
+      status_code: number;
+      warnings: null;
+      errors: null;
+    }>(`/brokers/data/orders/${orderId}/events`, {
+      method: 'GET',
+      headers: {
+        Authorization: `Bearer ${accessToken}`,
+      },
+      params,
+    });
+  }
+
+  /**
+   * Get order groups
+   * @param filter - Optional filter parameters
+   * @returns Promise with order groups response
+   */
+  async getOrderGroups(
+    filter?: OrderGroupsFilter
+  ): Promise<{
+    _id: string;
+    response_data: OrderGroup[];
+    message: string;
+    status_code: number;
+    warnings: null;
+    errors: null;
+  }> {
+    const accessToken = await this.getValidAccessToken();
+    const params: Record<string, string> = {};
+
+    if (filter?.broker_id) {
+      params.broker_id = filter.broker_id;
+    }
+    if (filter?.connection_id) {
+      params.connection_id = filter.connection_id;
+    }
+    if (filter?.limit) {
+      params.limit = filter.limit.toString();
+    }
+    if (filter?.offset) {
+      params.offset = filter.offset.toString();
+    }
+    if (filter?.created_after) {
+      params.created_after = filter.created_after;
+    }
+    if (filter?.created_before) {
+      params.created_before = filter.created_before;
+    }
+
+    return this.request<{
+      _id: string;
+      response_data: OrderGroup[];
+      message: string;
+      status_code: number;
+      warnings: null;
+      errors: null;
+    }>('/brokers/data/orders/groups', {
+      method: 'GET',
+      headers: {
+        Authorization: `Bearer ${accessToken}`,
+      },
+      params,
+    });
+  }
+
+  /**
+   * Get position lots (tax lots for positions)
+   * @param filter - Optional filter parameters
+   * @returns Promise with position lots response
+   */
+  async getPositionLots(
+    filter?: PositionLotsFilter
+  ): Promise<{
+    _id: string;
+    response_data: PositionLot[];
+    message: string;
+    status_code: number;
+    warnings: null;
+    errors: null;
+  }> {
+    const accessToken = await this.getValidAccessToken();
+    const params: Record<string, string> = {};
+
+    if (filter?.broker_id) {
+      params.broker_id = filter.broker_id;
+    }
+    if (filter?.connection_id) {
+      params.connection_id = filter.connection_id;
+    }
+    if (filter?.account_id) {
+      params.account_id = filter.account_id;
+    }
+    if (filter?.symbol) {
+      params.symbol = filter.symbol;
+    }
+    if (filter?.position_id) {
+      params.position_id = filter.position_id;
+    }
+    if (filter?.limit) {
+      params.limit = filter.limit.toString();
+    }
+    if (filter?.offset) {
+      params.offset = filter.offset.toString();
+    }
+
+    return this.request<{
+      _id: string;
+      response_data: PositionLot[];
+      message: string;
+      status_code: number;
+      warnings: null;
+      errors: null;
+    }>('/brokers/data/positions/lots', {
+      method: 'GET',
+      headers: {
+        Authorization: `Bearer ${accessToken}`,
+      },
+      params,
+    });
+  }
+
+  /**
+   * Get position lot fills for a specific lot
+   * @param lotId - The position lot ID
+   * @param filter - Optional filter parameters
+   * @returns Promise with position lot fills response
+   */
+  async getPositionLotFills(
+    lotId: string,
+    filter?: PositionLotFillsFilter
+  ): Promise<{
+    _id: string;
+    response_data: PositionLotFill[];
+    message: string;
+    status_code: number;
+    warnings: null;
+    errors: null;
+  }> {
+    const accessToken = await this.getValidAccessToken();
+    const params: Record<string, string> = {};
+
+    if (filter?.connection_id) {
+      params.connection_id = filter.connection_id;
+    }
+    if (filter?.limit) {
+      params.limit = filter.limit.toString();
+    }
+    if (filter?.offset) {
+      params.offset = filter.offset.toString();
+    }
+
+    return this.request<{
+      _id: string;
+      response_data: PositionLotFill[];
+      message: string;
+      status_code: number;
+      warnings: null;
+      errors: null;
+    }>(`/brokers/data/positions/lots/${lotId}/fills`, {
+      method: 'GET',
+      headers: {
+        Authorization: `Bearer ${accessToken}`,
+      },
+      params,
+    });
+  }
 }