@machinemetrics/mm-erp-sdk 0.1.9-beta.5 → 0.1.9

package/src/services/erp-api-services/graphql/graphql-service.ts

@@ -2,11 +2,11 @@ import {
   HTTPClient,
   HTTPClientFactory,
   HTTPResponse,
-} from "../../../utils/http-client";
-import { GraphQLResponse } from "./types";
-import { ErrorHandler } from "../errors";
-import { APIResponse } from "../types";
-import { ERPApiConfig } from "../types";
+} from "../../../utils/http-client.js";
+import { GraphQLResponse } from "./types.js";
+import { ErrorHandler } from "../errors.js";
+import { APIResponse } from "../types.js";
+import { ERPApiConfig } from "../types.js";
 
 const DEFAULT_RETRY_ATTEMPTS = 0; // One try only, no retries
 

package/src/services/erp-api-services/index.ts

@@ -4,19 +4,19 @@
  */
 
 // REST API Service
-export { RestAPIService } from "./rest/rest-api-service";
-export type { QueryParams } from "./rest/get-query-params";
+export { RestAPIService } from "./rest/rest-api-service.js";
+export type { QueryParams } from "./rest/get-query-params.js";
 
 // GraphQL Service
-export { GraphQLService } from "./graphql/graphql-service";
-export type { GraphQLServerError } from './graphql/types';
+export { GraphQLService } from "./graphql/graphql-service.js";
+export type { GraphQLServerError } from './graphql/types.js';
 
 // OAuth Client
-export { OAuthClient } from './oauth-client'; 
-export type { OAuthConfig } from './oauth-client';
+export { OAuthClient } from './oauth-client.js'; 
+export type { OAuthConfig } from './oauth-client.js';
 
 // API Response
-export type { APIResponse } from './types';
+export type { APIResponse } from './types.js';
 
 // Error classes
-export { ErrorHandler, GraphQLError } from './errors';
+export { ErrorHandler, GraphQLError } from './errors.js';

package/src/services/erp-api-services/rest/rest-api-service.ts

@@ -1,11 +1,11 @@
-import { HTTPClient, HTTPClientFactory } from "../../../utils/http-client";
+import { HTTPClient, HTTPClientFactory } from "../../../utils/http-client.js";
 import {
   QueryParams,
   buildLogicalCondition,
   Pagination,
-} from "./get-query-params";
-import { ErrorHandler } from "../errors";
-import { APIResponse, ERPApiConfig } from "../types";
+} from "./get-query-params.js";
+import { ErrorHandler } from "../errors.js";
+import { APIResponse, ERPApiConfig } from "../types.js";
 
 const DEFAULT_RETRY_ATTEMPTS = 0; // One try only, no retries
 const DEFAULT_PAGINATION_LIMIT = 100; // Reasonable default for most APIs

package/src/services/erp-api-services/types.ts

@@ -1,5 +1,5 @@
-import { Pagination } from "./rest/get-query-params";
-import { GraphQLServerError } from "./graphql/types";
+import { Pagination } from "./rest/get-query-params.js";
+import { GraphQLServerError } from "./graphql/types.js";
 
 /**
  * Configuration for making API requests to an ERP system.

package/src/services/mm-api-service/index.ts

@@ -1,9 +1,9 @@
-import { MMApiClient } from "./mm-api-service";
+import { MMApiClient } from "./mm-api-service.js";
 import type {
   MMReceiveLaborTicket,
   MMReceiveLaborTicketReason,
   MMReceiveLaborTicketWorkOrderOperation,
-} from "./types/receive-types";
+} from "./types/receive-types.js";
 import {
   MMSendPerson,
   MMSendResource,
@@ -14,8 +14,8 @@ import {
   MMSendReason,
   MMSendLaborTicket,
   MMSendWorkOrderBatch,
-} from "./types/send-types";
-import type { IToRESTApiObject } from "./types/send-types";
+} from "./types/send-types.js";
+import type { IToRESTApiObject } from "./types/send-types.js";
 import type {
   MMApiBaseResponse,
   MM200NonLaborTicketResponse,
@@ -26,9 +26,7 @@ import type {
   MM500LaborTicketException,
   MMGraphQLResourceResponse,
   MMMachineGroupsResponse,
-} from "./types/mm-response-interfaces";
-import { getCompanyInfo } from "./company-info";
-import type { CompanyInfo } from "./company-info";
+} from "./types/mm-response-interfaces.js";
 
 /**
  * A class to manage interactions with the MM API
@@ -54,7 +52,7 @@ export type {
 /**
  * Types related of labor tickets received from the MM API
  */
-export { MMReceiveLaborTicket } from "./types/receive-types";
+export { MMReceiveLaborTicket } from "./types/receive-types.js";
 export type {
   MMReceiveLaborTicketReason,
   MMReceiveLaborTicketWorkOrderOperation,
@@ -83,9 +81,3 @@ export {
  * Batch interfaces for sending multiple related entities
  */
 export type { MMSendWorkOrderBatch };
-
-/**
- * Company information utilities
- */
-export { getCompanyInfo };
-export type { CompanyInfo };

package/src/services/mm-api-service/mm-api-service.ts

@@ -1,22 +1,22 @@
-import logger from "../reporting-service/logger";
-import { ERPObjType } from "../../types/erp-types";
-import { MMReceiveLaborTicket } from "./types/receive-types";
-import { MMTokenManager } from "./token-mgr";
-import * as MM_SEND_TYPES from "./types/send-types";
+import logger from "../reporting-service/logger.js";
+import { ERPObjType } from "../../types/erp-types.js";
+import { MMReceiveLaborTicket } from "./types/receive-types.js";
+import { MMTokenManager } from "./token-mgr.js";
+import * as MM_SEND_TYPES from "./types/send-types.js";
 import {
   HTTPClient,
   HTTPMethod,
   HTTPRequestConfig,
   HTTPClientFactory,
-} from "../../utils/http-client";
-import { Checkpoint } from "./types/checkpoint";
-import { CoreConfiguration } from "../data-sync-service/configuration-manager";
-import { ErrorHandler } from "../erp-api-services/errors";
+} from "../../utils/http-client.js";
+import { Checkpoint } from "./types/checkpoint.js";
+import { CoreConfiguration } from "../data-sync-service/configuration-manager.js";
+import { ErrorHandler } from "../erp-api-services/errors.js";
 import {
   MMApiBaseResponse,
   MMGraphQLResourceResponse,
   MMMachineGroupsResponse,
-} from "./types/mm-response-interfaces";
+} from "./types/mm-response-interfaces.js";
 
 const MAPPING = "MAPPING";
 const CHECKPOINTS = "CHECKPOINTS";
@@ -80,12 +80,9 @@ export class MMApiClient {
         return typeof err === "object" && err !== null && "status" in err;
       };
 
-      // Check if this looks like an auth error - retry on auth status codes or JWT expiration errors
+      // Check if this looks like an auth error - only retry on explicit auth status codes
       const isAuthError =
-        hasStatus(error) && (error.status === 401 || error.status === 403) ||
-        (hasStatus(error) && error.status === 500 && 
-         typeof (error as any).data?.error === 'string' && 
-         (error as any).data.error.includes('JWT'));
+        hasStatus(error) && (error.status === 401 || error.status === 403);
 
       if (isAuthError && !options.token) {
         // Only retry if we used cached token
@@ -509,20 +506,6 @@ export class MMApiClient {
     return updates.data.map((ticket) => new MMReceiveLaborTicket(ticket));
   }
 
-  /**
-   * Fetch a single labor ticket by reference from the MM API
-   * @param laborTicketRef The labor ticket reference to fetch
-   * @returns Promise with the labor ticket data
-   */
-  async fetchLaborTicketByRef(
-    laborTicketRef: string
-  ): Promise<MMReceiveLaborTicket> {
-    const response = (await this.getData(
-      `${this.resourceURLs[ERPObjType.LABOR_TICKETS]}/${laborTicketRef}`
-    )) as { data: MMReceiveLaborTicket };
-    return new MMReceiveLaborTicket(response.data);
-  }
-
   /**
    * Fetch a checkpoint for a specific system, table, and checkpoint type
    * @param checkpoint The checkpoint to fetch

package/src/services/mm-api-service/token-mgr.ts

@@ -1,10 +1,10 @@
 import {
   getCachedMMToken,
   setCachedMMToken,
-} from "../../utils/local-data-store/jobs-shared-data";
-import { HTTPClient, HTTPClientFactory } from "../../utils/http-client";
-import { CoreConfiguration } from "../data-sync-service/configuration-manager";
-import { logger } from "../reporting-service";
+} from "../../utils/local-data-store/jobs-shared-data.js";
+import { HTTPClient, HTTPClientFactory } from "../../utils/http-client.js";
+import { CoreConfiguration } from "../data-sync-service/configuration-manager.js";
+import { logger } from "../reporting-service/index.js";
 
 /**
  * Manages the MM API token with caching and expiration checking

package/src/services/mm-api-service/types/entity-transformer.ts

@@ -1,5 +1,5 @@
-import { ERPObjType } from "../../../types/erp-types";
-import { IToRESTApiObject } from "..";
+import { ERPObjType } from "../../../types/erp-types.js";
+import { IToRESTApiObject } from "../index.js";
 import {
   MMSendPerson,
   MMSendResource,
@@ -9,7 +9,7 @@ import {
   MMSendWorkOrderOperation,
   MMSendLaborTicket,
   MMSendReason,
-} from "..";
+} from "../index.js";
 
 /**
  * Utility class for transforming entities to/from non-typed and typed objects

package/src/services/psql-erp-service/index.ts

@@ -1,4 +1,4 @@
-import { PsqlService } from "./psql-service";
+import { PsqlService } from "./psql-service.js";
 
 /**
  * A class to manage interactions with PSQL (Pervasive) databases via ODBC
@@ -8,7 +8,7 @@ export { PsqlService };
 /**
  * Configuration interface for PSQL connections
  */
-export type { PsqlConfiguration } from "./configuration";
+export type { PsqlConfiguration } from "./configuration.js";
 
 /**
  * Helper functions for PSQL data formatting
@@ -19,4 +19,4 @@ export {
   combinePsqlDateTime,
   isPsqlDateEmpty,
   cleanPsqlCharField,
-} from "./psql-helpers";
+} from "./psql-helpers.js";

package/src/services/psql-erp-service/psql-service.ts

@@ -1,7 +1,7 @@
-import { PsqlConfiguration } from "./configuration";
-import { ERPResponse } from "../../types/erp-types";
-import { OdbcErrorResponse } from "./internal/types/psql-types";
-import logger from "../reporting-service/logger";
+import { PsqlConfiguration } from "./configuration.js";
+import { ERPResponse } from "../../types/erp-types.js";
+import { OdbcErrorResponse } from "./internal/types/psql-types.js";
+import logger from "../reporting-service/logger.js";
 
 type PagingParams = {
   limit?: number;

package/src/services/reporting-service/index.ts

@@ -1,4 +1,4 @@
 /**
  * Export the logger instance for use throughout the application
  */
-export { default as logger } from "./logger";
+export { default as logger } from "./logger.js";

package/src/services/reporting-service/logger.ts

@@ -3,6 +3,33 @@ import DailyRotateFile from "winston-daily-rotate-file";
 import path from "path";
 
 const logDirectory = "logs";
+const initialLogLevel = process.env.LOG_LEVEL || "info";
+
+const LOGGER_ERROR_PREFIX = "[mm-erp-sdk logger]";
+
+const serializeLoggerError = (error: unknown): string => {
+  if (error instanceof Error) {
+    return error.stack ?? error.message;
+  }
+  if (typeof error === "string") {
+    return error;
+  }
+  try {
+    return JSON.stringify(error);
+  } catch {
+    return String(error);
+  }
+};
+
+const handleLoggerError = (error: unknown) => {
+  const serialized = serializeLoggerError(error);
+  try {
+    // eslint-disable-next-line no-console
+    console.error(`${LOGGER_ERROR_PREFIX} transport error: ${serialized}`);
+  } catch {
+    /* ignore failures when reporting logger errors */
+  }
+};
 
 const MSG_MAX_LEN = 60;
 const MSG_MAX_CHARS = 2048;
@@ -23,45 +50,8 @@ const truncateString = (str: string): string => {
   return result;
 };
 
-/**
- * Safe JSON serializer that handles circular references and extracts meaningful error data
- */
-const safeStringify = (obj: any, indent: number = 2): string => {
-  const seen = new WeakSet();
-  
-  return JSON.stringify(
-    obj,
-    (key, value) => {
-      // Handle Error objects specially
-      if (value instanceof Error) {
-        return {
-          message: value.message,
-          name: value.name,
-          stack: value.stack,
-          ...(value.constructor.name === 'HTTPError' || value.constructor.name === 'AxiosError' ? {
-            status: (value as any).status,
-            code: (value as any).code,
-            data: (value as any).data,
-          } : {}),
-        };
-      }
-      
-      // Handle circular references
-      if (typeof value === "object" && value !== null) {
-        if (seen.has(value)) {
-          return "[Circular]";
-        }
-        seen.add(value);
-      }
-      
-      return value;
-    },
-    indent
-  );
-};
-
 const baseFormat = format.printf(({ timestamp, level, message, ...rest }) => {
-  let restString = safeStringify(rest, 2);
+  let restString = JSON.stringify(rest, undefined, 2);
   restString = restString === "{}" ? "" : restString;
 
   let formattedMessage: string;
@@ -97,25 +87,31 @@ const logFormat = format.combine(
   baseFormat
 );
 
+const createConsoleTransport = () => {
+  const consoleTransport = new transports.Console({
+    format: format.combine(
+      format.timestamp(),
+      format.splat(),
+      baseFormat,
+      format.colorize({ all: true })
+    ),
+  });
+  consoleTransport.on("error", handleLoggerError);
+  return consoleTransport;
+};
+
 // Create a basic logger with default configuration
 const logger = createLogger({
-  level: process.env.LOG_LEVEL || "info",
+  level: initialLogLevel,
   format: logFormat,
-  transports: [
-    new transports.Console({
-      format: format.combine(
-        format.timestamp(),
-        format.splat(),
-        baseFormat,
-        format.colorize({ all: true })
-      ),
-    }),
-  ],
+  transports: [createConsoleTransport()],
 });
 
+logger.on("error", handleLoggerError);
+
 // Helper function to create a file transport with shared configuration
 const createFileTransport = (): DailyRotateFile => {
-  return new DailyRotateFile({
+  const transport = new DailyRotateFile({
     filename: path.join(logDirectory, "%DATE%.log"),
     datePattern: "YYYY-MM-DD",
     zippedArchive: true,
@@ -123,6 +119,8 @@ const createFileTransport = (): DailyRotateFile => {
     maxFiles: "14d",
     format: logFormat,
   });
+  transport.on("error", handleLoggerError);
+  return transport;
 };
 
 // Rotate mitigation helper: attaches rotate handler to transport and recursively attaches to replacements
@@ -136,35 +134,76 @@ function attachRotateMitigation(
     if (isRefreshing) return;
     isRefreshing = true;
     let removalTimer: NodeJS.Timeout | null = null;
+    let next: DailyRotateFile | null = null;
 
-    // Create replacement first to avoid any logging gap
-    const next = createFileTransport();
-    // When the new file is created, remove the old transport
-    next.on("new", () => {
+    const cleanupRefresh = () => {
       if (removalTimer) {
         clearTimeout(removalTimer);
         removalTimer = null;
       }
-      try {
-        logger.remove(transport);
-      } catch {}
       isRefreshing = false;
-    });
-    attachRotateMitigation(next, opts);
-    logger.add(next);
-
-    // Fallback: if the "new" event doesn't fire, remove the old transport after a grace period
-    const REMOVAL_GRACE_MS = 30000;
-    removalTimer = setTimeout(() => {
+    };
+
+    try {
+      // Create replacement first to avoid any logging gap
+      next = createFileTransport();
+      // When the new file is created, remove the old transport
+      next.on("new", () => {
+        if (removalTimer) {
+          clearTimeout(removalTimer);
+          removalTimer = null;
+        }
+        try {
+          logger.remove(transport);
+        } catch (error) {
+          handleLoggerError(error);
+        }
+        cleanupRefresh();
+      });
+      attachRotateMitigation(next, opts);
       try {
-        logger.remove(transport);
-      } catch {}
-      isRefreshing = false;
-      removalTimer = null;
-    }, REMOVAL_GRACE_MS);
+        logger.add(next);
+      } catch (error) {
+        handleLoggerError(error);
+        cleanupRefresh();
+        try {
+          logger.remove(next);
+        } catch {}
+        try {
+          if (typeof next.close === "function") {
+            next.close();
+          }
+        } catch {}
+        return;
+      }
 
-    // Keep console and other transports intact; do not silence or clear
-    logger.level = logLevel;
+      // Fallback: if the "new" event doesn't fire, remove the old transport after a grace period
+      const REMOVAL_GRACE_MS = 30000;
+      removalTimer = setTimeout(() => {
+        try {
+          logger.remove(transport);
+        } catch (error) {
+          handleLoggerError(error);
+        }
+        cleanupRefresh();
+      }, REMOVAL_GRACE_MS);
+
+      // Keep console and other transports intact; do not silence or clear
+      logger.level = logLevel;
+    } catch (error) {
+      cleanupRefresh();
+      handleLoggerError(error);
+      if (next) {
+        try {
+          logger.remove(next);
+        } catch {}
+        try {
+          if (typeof next.close === "function") {
+            next.close();
+          }
+        } catch {}
+      }
+    }
   });
 }
 
@@ -194,16 +233,7 @@ export const configureLogger = (logLevel: string, nodeEnv: string) => {
 
   // Add console transport in non-production environments
   if (nodeEnv !== "production") {
-    logger.add(
-      new transports.Console({
-        format: format.combine(
-          format.timestamp(),
-          format.splat(),
-          baseFormat,
-          format.colorize({ all: true })
-        ),
-      })
-    );
+    logger.add(createConsoleTransport());
   }
 
   // Set the log level

package/src/services/sql-server-erp-service/index.ts

@@ -1,6 +1,6 @@
-import { SqlServerService } from "./sql-server-service";
-import { SqlServerHelper } from "./sql-server-helpers";
-import { SQLInput } from "./types/sql-input-param";
+import { SqlServerService } from "./sql-server-service.js";
+import { SqlServerHelper } from "./sql-server-helpers.js";
+import { SQLInput } from "./types/sql-input-param.js";
 
 /**
  * A class to manage interactions with SQL Server

package/src/services/sql-server-erp-service/internal/sql-labor-ticket-operations.ts

@@ -1,6 +1,6 @@
 import sql from "mssql";
-import { SQLInput } from "../types/sql-input-param";
-import { SQLTransactionOperation } from "./sql-transaction-manager";
+import { SQLInput } from "../types/sql-input-param.js";
+import { SQLTransactionOperation } from "./sql-transaction-manager.js";
 
 /**
  * Insert a new labor ticket into the database.

package/src/services/sql-server-erp-service/internal/sql-transaction-manager.ts

@@ -1,5 +1,5 @@
 import sql from "mssql";
-import logger from "../../reporting-service/logger";
+import logger from "../../reporting-service/logger.js";
 
 export interface SQLTransactionConfig {
   isolationLevel?: sql.IIsolationLevel;

package/src/services/sql-server-erp-service/sql-server-helpers.ts

@@ -1,12 +1,12 @@
-import { SQLTransactionManager } from "./internal/sql-transaction-manager";
+import { SQLTransactionManager } from "./internal/sql-transaction-manager.js";
 import {
   SQLLaborTicketInsertOperation,
   SQLLaborTicketUpdateOperation,
-} from "./internal/sql-labor-ticket-operations";
-import { MMReceiveLaborTicket } from "../mm-api-service/types/receive-types";
-import { SqlServerService } from "./sql-server-service";
-import logger from "../reporting-service/logger";
-import { SQLInput } from "./types/sql-input-param";
+} from "./internal/sql-labor-ticket-operations.js";
+import { MMReceiveLaborTicket } from "../mm-api-service/types/receive-types.js";
+import { SqlServerService } from "./sql-server-service.js";
+import logger from "../reporting-service/logger.js";
+import { SQLInput } from "./types/sql-input-param.js";
 
 export class SqlServerHelper {
   /**

package/src/services/sql-server-erp-service/sql-server-service.ts

@@ -1,8 +1,8 @@
 import sql, { ConnectionPool, IRecordSet } from "mssql";
-import { SQLServerConfigSchema } from "./internal/sql-server-config";
-import logger from "../reporting-service/logger";
-import { ERPResponse } from "../../types/erp-types";
-import { SQLServerConfiguration } from "./configuration";
+import { SQLServerConfigSchema } from "./internal/sql-server-config.js";
+import logger from "../reporting-service/logger.js";
+import { ERPResponse } from "../../types/erp-types.js";
+import { SQLServerConfiguration } from "./configuration.js";
 
 type PagingParams = {
   limit?: number;

package/src/services/sqlite-service/index.ts

@@ -1 +1 @@
-export { SQLiteCoordinator } from "./sqlite-coordinator";
+export { SQLiteCoordinator } from "./sqlite-coordinator.js";

package/src/services/sqlite-service/sqlite-coordinator.ts

@@ -2,8 +2,8 @@ import {
   acquireDatabaseLock,
   releaseDatabaseLock,
   getDatabaseLock,
-} from "../../utils/local-data-store/database-lock";
-import { logger } from "../reporting-service";
+} from "../../utils/local-data-store/database-lock.js";
+import { logger } from "../reporting-service/index.js";
 
 export class SQLiteCoordinator {
   private static readonly LOCK_TIMEOUT_MS = 30_000; // 30 seconds

package/src/types/erp-connector.ts

@@ -1,4 +1,4 @@
-import { MMReceiveLaborTicket } from "../services/mm-api-service/types/receive-types";
+import { MMReceiveLaborTicket } from "../services/mm-api-service/types/receive-types.js";
 
 /**
  * Interface for ERP connectors.

package/src/types/index.ts

@@ -3,10 +3,10 @@
  * This file re-exports all type definitions that are part of the public API.
  */
 
-export type { IERPConnector } from "./erp-connector";
+export type { IERPConnector } from "./erp-connector.js";
 export type {
   ERPObjType,
   ERPPagination,
   ERPResponse,
   ERPObject,
-} from "./erp-types";
+} from "./erp-types.js";

package/src/utils/application-initializer.ts

@@ -1,9 +1,9 @@
-import logger from "../services/reporting-service/logger";
-import { CoreConfiguration } from "../services/data-sync-service/configuration-manager";
-import { SQLiteCoordinator } from "../services/sqlite-service";
-import { getTimezoneOffsetAndPersist } from "./time-utils";
+import logger from "../services/reporting-service/logger.js";
+import { CoreConfiguration } from "../services/data-sync-service/configuration-manager.js";
+import { SQLiteCoordinator } from "../services/sqlite-service/index.js";
+import { getTimezoneOffsetAndPersist } from "./time-utils.js";
 import knex from "knex";
-import config from "../knexfile";
+import config from "../knexfile.js";
 
 /**
  * Utility class to handle application initialization tasks