@machinemetrics/mm-erp-sdk 0.2.0-beta.0 → 0.2.0-beta.2

package/package.json

@@ -1,10 +1,10 @@
 {
   "name": "@machinemetrics/mm-erp-sdk",
   "description": "A library for syncing data between MachineMetrics and ERP systems",
-  "version": "0.2.0-beta.0",
+  "version": "0.2.0-beta.2",
   "license": "MIT",
   "author": "machinemetrics",
-  "main": "dist/index.js",
+  "main": "dist/mm-erp-sdk.js",
   "types": "dist/index.d.ts",
   "source": "src/index.ts",
   "type": "module",
@@ -12,9 +12,10 @@
     "node": ">=20"
   },
   "scripts": {
-    "build": "tsc --project tsconfig.build.json",
-    "start": "tsc --project tsconfig.build.json --watch",
-    "type-check": "tsc --noEmit"
+    "build": "vite build && tsc --project tsconfig.declarations.json",
+    "start": "vite build --watch",
+    "type-check": "tsc --noEmit",
+    "test:nats": "tsx src/services/nats-service/test-nats-subscriber.ts"
   },
   "dependencies": {
     "@azure/msal-node": "^2.12.0",
@@ -27,8 +28,8 @@
     "json-stable-stringify": "^1.3.0",
     "knex": "^3.1.0",
     "lodash": "^4.17.21",
-    "luxon": "^3.7.2",
     "mssql": "^11.0.1",
+    "nats": "^2.29.3",
     "winston": "^3.14.0",
     "winston-daily-rotate-file": "^5.0.0",
     "xxhashjs": "^0.2.2",
@@ -40,12 +41,13 @@
   "devDependencies": {
     "@types/json-stable-stringify": "^1.1.0",
     "@types/lodash": "^4.17.10",
-    "@types/luxon": "^3.7.1",
     "@types/mssql": "^9.1.6",
     "@types/node": "^20.0.0",
     "@types/xxhashjs": "^0.2.4",
     "prettier": "^3.6.2",
-    "typescript": "^5.8.3"
+    "tsx": "^4.20.6",
+    "typescript": "^5.8.3",
+    "vite": "^5.3.4"
   },
   "files": [
     "dist",

package/src/index.ts

@@ -4,24 +4,24 @@
  */
 
 // Core interfaces and types
-export type { IERPConnector } from "./types/erp-connector.js";
-export { ERPObjType } from "./types/erp-types.js";
-export type { ERPPagination, ERPResponse, ERPObject } from "./types/erp-types.js";
+export type { IERPConnector } from "./types/erp-connector";
+export { ERPObjType } from "./types/erp-types";
+export type { ERPPagination, ERPResponse, ERPObject } from "./types/erp-types";
 
 // Configuration interfaces
-export { ErpApiConnectionParams } from "./services/data-sync-service/configuration-manager.js";
-export { getErpApiConnectionParams } from "./services/data-sync-service/configuration-manager.js";
-export { buildLogicalCondition } from "./services/erp-api-services/rest/get-query-params.js";
-export type { ERPApiConfig } from "./services/erp-api-services/types.js";
+export { ErpApiConnectionParams } from "./services/data-sync-service/configuration-manager";
+export { getErpApiConnectionParams } from "./services/data-sync-service/configuration-manager";
+export { buildLogicalCondition } from "./services/erp-api-services/rest/get-query-params";
+export type { ERPApiConfig } from "./services/erp-api-services/types";
 
 // MM API client and types
-export { MMApiClient } from "./services/mm-api-service/index.js";
-export { MMReceiveLaborTicket } from "./services/mm-api-service/index.js";
+export { MMApiClient } from "./services/mm-api-service";
+export { MMReceiveLaborTicket } from "./services/mm-api-service";
 export type {
   MMReceiveLaborTicketReason,
   MMReceiveLaborTicketWorkOrderOperation,
   IToRESTApiObject,
-} from "./services/mm-api-service/index.js";
+} from "./services/mm-api-service";
 
 export {
   MMSendPerson,
@@ -32,9 +32,9 @@ export {
   MMSendWorkOrderOperation,
   MMSendReason,
   MMSendLaborTicket,
-} from "./services/mm-api-service/index.js";
+} from "./services/mm-api-service";
 
-export type { MMSendWorkOrderBatch } from "./services/mm-api-service/index.js";
+export type { MMSendWorkOrderBatch } from "./services/mm-api-service";
 
 // Utility functions that are safe for client use
 export {
@@ -49,69 +49,72 @@ export {
   applyTimezoneOffsetsToFields,
   convertToLocalTime,
   getErrorType,
-} from "./utils/index.js";
+  formatError,
+  formatErrorForLogging,
+} from "./utils";
+export type { FormattedError } from "./utils";
 
 // Local data store
-export { getInitialLoadComplete, setInitialLoadComplete } from "./utils/index.js";
+export { getInitialLoadComplete, setInitialLoadComplete } from "./utils";
 
 // HTTP client factory and types for custom API integrations
-export { HTTPClientFactory } from "./utils/http-client.js";
-export { HTTPError } from "./utils/http-client.js";
+export { HTTPClientFactory } from "./utils/http-client";
+export { HTTPError } from "./utils/http-client";
 export type {
   HTTPResponse,
   HTTPClient,
   HTTPRequestConfig,
-} from "./utils/http-client.js";
+} from "./utils/http-client";
 
 // Application initialization utilities
-export { ApplicationInitializer } from "./utils/application-initializer.js";
+export { ApplicationInitializer } from "./utils/application-initializer";
 
 // Data sync service
-export { runDataSyncService } from "./services/data-sync-service/index.js";
+export { runDataSyncService } from "./services/data-sync-service";
 
 // Logging service
-export { logger } from "./services/reporting-service/index.js";
+export { logger } from "./services/reporting-service";
 
 // Labor ticket handler interface
-export type { IERPLaborTicketHandler } from "./types/erp-connector.js";
+export type { IERPLaborTicketHandler } from "./types/erp-connector";
 
 // Process drivers and utilities
 export {
   StandardProcessDrivers,
   getCachedTimezoneOffset,
   getCachedTimezoneName,
-} from "./utils/index.js";
-export type { WriteEntitiesToMMResult } from "./utils/index.js";
-export { MMBatchValidationError } from "./utils/index.js";
+} from "./utils";
+export type { WriteEntitiesToMMResult } from "./utils";
+export { MMBatchValidationError } from "./utils";
 
 // API services
-export type { APIResponse } from "./services/erp-api-services/types.js";
-export { RestAPIService } from "./services/erp-api-services/rest/rest-api-service.js";
-export { ErrorHandler, GraphQLError } from "./services/erp-api-services/errors.js";
+export type { APIResponse } from "./services/erp-api-services/types";
+export { RestAPIService } from "./services/erp-api-services/rest/rest-api-service";
+export { ErrorHandler, GraphQLError } from "./services/erp-api-services/errors";
 export type {
   QueryParams,
   PaginatedAPIResponse,
-} from "./services/erp-api-services/rest/get-query-params.js";
-export { OAuthClient } from "./services/erp-api-services/oauth-client.js";
-export type { OAuthConfig } from "./services/erp-api-services/oauth-client.js";
-export { GraphQLService } from "./services/erp-api-services/graphql/graphql-service.js";
-export type { GraphQLServerError } from "./services/erp-api-services/graphql/types.js";
+} from "./services/erp-api-services/rest/get-query-params";
+export { OAuthClient } from "./services/erp-api-services/oauth-client";
+export type { OAuthConfig } from "./services/erp-api-services/oauth-client";
+export { GraphQLService } from "./services/erp-api-services/graphql/graphql-service";
+export type { GraphQLServerError } from "./services/erp-api-services/graphql/types";
 
 // Caching services
-export { BatchCacheManager } from "./services/caching-service/batch-cache-manager.js";
+export { BatchCacheManager } from "./services/caching-service/batch-cache-manager";
 
 // Configuration services
 export {
   CoreConfiguration,
   getSQLServerConfiguration,
-} from "./services/data-sync-service/configuration-manager.js";
+} from "./services/data-sync-service/configuration-manager";
 
 // SQL Server services
 export {
   SqlServerService,
   SqlServerHelper,
-} from "./services/sql-server-erp-service/index.js";
-export type { SQLInput } from "./services/sql-server-erp-service/index.js";
+} from "./services/sql-server-erp-service";
+export type { SQLInput } from "./services/sql-server-erp-service";
 
 // PSQL (Pervasive) services
 export {
@@ -121,15 +124,15 @@ export {
   combinePsqlDateTime,
   isPsqlDateEmpty,
   cleanPsqlCharField,
-} from "./services/psql-erp-service/index.js";
-export type { PsqlConfiguration } from "./services/psql-erp-service/index.js";
+} from "./services/psql-erp-service";
+export type { PsqlConfiguration } from "./services/psql-erp-service";
 
 // Record tracking services
-export { RecordTrackingManager } from "./services/caching-service/record-tracking-manager.js";
-export type { RecordTrackingObject } from "./services/caching-service/record-tracking-manager.js";
+export { RecordTrackingManager } from "./services/caching-service/record-tracking-manager";
+export type { RecordTrackingObject } from "./services/caching-service/record-tracking-manager";
 
 // Knex services
-export { default as knexDatabaseConfig } from "./knexfile.js";
+export { default as knexDatabaseConfig } from "./knexfile";
 
 // MM Connector Logging
 export {
@@ -138,4 +141,4 @@ export {
   LogEntry,
   type LogLevelString,
   type LogResponse,
-} from "./utils/index.js";
+} from "./utils";

package/src/knexfile.ts

@@ -14,7 +14,6 @@ const config: { [key: string]: Knex.Config } = {
     migrations: {
       directory: path.join(__dirname, "migrations"),
       extension: "js",
-      loadExtensions: [".js"],
     },
   },
 };

package/src/services/caching-service/batch-cache-manager.ts

@@ -1,5 +1,5 @@
-import { HashedCacheManager } from "./hashed-cache-manager.js";
-import { ERPObjType } from "../../types/erp-types.js";
+import { HashedCacheManager } from "./hashed-cache-manager";
+import { ERPObjType } from "../../types/erp-types";
 
 export type BatchResult<T> = {
   duplicates: T[];

package/src/services/caching-service/hashed-cache-manager.ts

@@ -1,11 +1,11 @@
 import knex, { Knex } from "knex";
-import config from "../../knexfile.js";
+import config from "../../knexfile";
 import stringify from "json-stable-stringify";
 import XXH from "xxhashjs";
-import { ERPObjType } from "../../types/erp-types.js";
-import { CacheMetrics } from "./index.js";
-import { CoreConfiguration } from "../data-sync-service/configuration-manager.js";
-import { logger } from "../reporting-service/index.js";
+import { ERPObjType } from "../../types/erp-types";
+import { CacheMetrics } from "./index";
+import { CoreConfiguration } from "../data-sync-service/configuration-manager";
+import { logger } from "../reporting-service";
 
 type HashedCacheManagerOptions = {
   ttl?: number;

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

@@ -1,4 +1,4 @@
-import { ERPObjType, ERPObject } from "../../types/erp-types.js";
+import { ERPObjType, ERPObject } from "../../types/erp-types";
 
 export interface CacheServiceConfig {
   tableName: string;

package/src/services/caching-service/record-tracking-manager.ts

@@ -1,6 +1,6 @@
 import knex, { Knex } from "knex";
-import config from "../../knexfile.js";
-import { ERPObjType } from "../../types/erp-types.js";
+import config from "../../knexfile";
+import { ERPObjType } from "../../types/erp-types";
 
 const DEFAULT_RECORD_TRACKING_TABLE_NAME = "record_tracking";
 

package/src/services/data-sync-service/configuration-manager.ts

@@ -1,7 +1,6 @@
 import "dotenv/config";
-import { configureLogger } from "../reporting-service/logger.js";
-import { SQLServerConfiguration } from "../sql-server-erp-service/configuration.js";
-import { getEnvBoolean, getEnvNumber, getEnvString } from "../../utils/env.js";
+import { configureLogger } from "../reporting-service/logger";
+import { SQLServerConfiguration } from "../sql-server-erp-service/configuration";
 
 export class CoreConfiguration {
   private static instance: CoreConfiguration;
@@ -30,66 +29,58 @@ export class CoreConfiguration {
   public readonly cacheExpirationCheckInterval: string;
 
   private constructor() {
-    this.logLevel = getEnvString("LOG_LEVEL", "info");
-    this.erpSystem = getEnvString("ERP_SYSTEM", "template");
-    this.nodeEnv = getEnvString("NODE_ENV", "development");
+    this.logLevel = process.env.LOG_LEVEL || "info";
+    this.erpSystem = process.env.ERP_SYSTEM || "template";
+    this.nodeEnv = process.env.NODE_ENV || "development";
 
     //#region MM API (aka "Mapping") Service
     /**
      * MM ERP Service REST API URL (typically https://erp-api.svc.machinemetrics.com)
      */
-    this.mmERPSvcApiBaseUrl = getEnvString("MM_MAPPING_SERVICE_URL", "");
+    this.mmERPSvcApiBaseUrl = process.env.MM_MAPPING_SERVICE_URL || "";
 
     /**
      * MM REST API URL  (typically https://api.machinemetrics.com)
      */
     console.log("=== CONFIG DEBUG ===");
-    const mmAuthSvcUrl = getEnvString("MM_MAPPING_AUTH_SERVICE_URL", "");
-    console.log("MM_MAPPING_AUTH_SERVICE_URL env var:", mmAuthSvcUrl);
-    this.mmApiBaseUrl = mmAuthSvcUrl;
+    console.log("MM_MAPPING_AUTH_SERVICE_URL env var:", process.env.MM_MAPPING_AUTH_SERVICE_URL);
+    this.mmApiBaseUrl = process.env.MM_MAPPING_AUTH_SERVICE_URL || "";
     console.log("mmApiBaseUrl set to:", this.mmApiBaseUrl);
     console.log("=== END CONFIG DEBUG ===");
 
     /**
      * Company Auth Token
      */
-    this.mmApiAuthToken = getEnvString("MM_MAPPING_SERVICE_TOKEN", "");
+    this.mmApiAuthToken = process.env.MM_MAPPING_SERVICE_TOKEN || "";
 
     /**
      * Number of retry attempts for MM API calls
      */
-    this.mmApiRetryAttempts = getEnvNumber("RETRY_ATTEMPTS", 0);
+    this.mmApiRetryAttempts = parseInt(process.env.RETRY_ATTEMPTS || "0");
     //#endregion MM API (aka "Mapping") Service
 
     /**
      * Default pagination limit for ERP API
      */
-    this.erpApiPagingLimit = getEnvNumber("ERP_PAGINATION_LIMIT", 0);
+    this.erpApiPagingLimit = parseInt(process.env.ERP_PAGINATION_LIMIT || "0");
     //#endregion ERP API Service
 
     /**
      * For how to define the intervals, see Bree's documentation: https://github.com/breejs/bree
      */
-    const pollInterval = getEnvString("POLL_INTERVAL", "");
-    this.fromErpInterval = getEnvString(
-      "FROM_ERP_INTERVAL",
-      pollInterval || "5 min"
-    );
-    this.toErpInterval = getEnvString("TO_ERP_INTERVAL", "5 min");
-    this.retryLaborTicketsInterval = getEnvString(
-      "RETRY_LABOR_TICKETS_INTERVAL",
-      "30 min"
-    );
-    this.cacheExpirationCheckInterval = getEnvString(
-      "CACHE_EXPIRATION_CHECK_INTERVAL",
-      "5 min"
-    );
+    this.fromErpInterval =
+      process.env.FROM_ERP_INTERVAL || process.env.POLL_INTERVAL || "5 min";
+    this.toErpInterval = process.env.TO_ERP_INTERVAL || "5 min";
+    this.retryLaborTicketsInterval =
+      process.env.RETRY_LABOR_TICKETS_INTERVAL || "30 min";
+    this.cacheExpirationCheckInterval =
+      process.env.CACHE_EXPIRATION_CHECK_INTERVAL || "5 min";
 
     /**
      * Cache TTL (in seconds)
      */
     const cacheTTLDef = 7 * 24 * 60 * 60; // 7 days
-    this.cacheTTL = getEnvNumber("CACHE_TTL", cacheTTLDef);
+    this.cacheTTL = parseInt(process.env.CACHE_TTL || cacheTTLDef.toString());
 
     // Configure the logger with our settings
     configureLogger(this.logLevel, this.nodeEnv);
@@ -141,24 +132,20 @@ export class CoreConfiguration {
  */
 export const getSQLServerConfiguration = (): SQLServerConfiguration => {
   return {
-    username: getEnvString("ERP_SQLSERVER_USERNAME", ""),
-    password: getEnvString("ERP_SQLSERVER_PASSWORD", ""),
-    database: getEnvString("ERP_SQLSERVER_DATABASE", ""),
-    host: getEnvString(
-      "ERP_SQLSERVER_HOST",
-      getEnvString("ERP_SQLSERVER_SERVER", "")
-    ),
-    port: getEnvString("ERP_SQLSERVER_PORT", "1433"),
-    connectionTimeout: getEnvString("ERP_SQLSERVER_CONNECTION_TIMEOUT", "30000"),
-    requestTimeout: getEnvString("ERP_SQLSERVER_REQUEST_TIMEOUT", "60000"),
-    poolMax: getEnvString("ERP_SQLSERVER_MAX", "10"),
-    poolMin: getEnvString("ERP_SQLSERVER_MIN", "0"),
-    idleTimeoutMillis: getEnvString(
-      "ERP_SQLSERVER_IDLE_TIMEOUT_MMILLIS",
-      "30000"
-    ),
-    encrypt: getEnvBoolean("ERP_SQLSERVER_ENCRYPT", false),
-    trustServer: getEnvBoolean("ERP_SQLSERVER_TRUST_SERVER", false),
+    username: process.env.ERP_SQLSERVER_USERNAME || "",
+    password: process.env.ERP_SQLSERVER_PASSWORD || "",
+    database: process.env.ERP_SQLSERVER_DATABASE || "",
+    host:
+      process.env.ERP_SQLSERVER_HOST || process.env.ERP_SQLSERVER_SERVER || "",
+    port: process.env.ERP_SQLSERVER_PORT || "1433",
+    connectionTimeout: process.env.ERP_SQLSERVER_CONNECTION_TIMEOUT || "30000",
+    requestTimeout: process.env.ERP_SQLSERVER_REQUEST_TIMEOUT || "60000",
+    poolMax: process.env.ERP_SQLSERVER_MAX || "10",
+    poolMin: process.env.ERP_SQLSERVER_MIN || "0",
+    idleTimeoutMillis:
+      process.env.ERP_SQLSERVER_IDLE_TIMEOUT_MMILLIS || "30000",
+    encrypt: process.env.ERP_SQLSERVER_ENCRYPT === "true",
+    trustServer: process.env.ERP_SQLSERVER_TRUST_SERVER === "true",
   };
 };
 
@@ -184,11 +171,11 @@ export class ErpApiConnectionParams {
  */
 export const getErpApiConnectionParams = (): ErpApiConnectionParams => {
   return new ErpApiConnectionParams(
-    getEnvString("ERP_API_URL", ""),
-    getEnvString("ERP_API_CLIENT_ID", ""),
-    getEnvString("ERP_API_CLIENT_SECRET", ""),
-    getEnvString("ERP_API_ORGANIZATION_ID", ""),
-    getEnvString("ERP_AUTH_BASE_URL", ""),
-    getEnvNumber("ERP_API_RETRY_ATTEMPTS", 3)
+    process.env.ERP_API_URL || "",
+    process.env.ERP_API_CLIENT_ID || "",
+    process.env.ERP_API_CLIENT_SECRET || "",
+    process.env.ERP_API_ORGANIZATION_ID || "",
+    process.env.ERP_AUTH_BASE_URL || "",
+    parseInt(process.env.ERP_API_RETRY_ATTEMPTS || "3")
   );
 };

package/src/services/data-sync-service/data-sync-service.ts

@@ -2,19 +2,23 @@ import Bree, { Job } from "bree";
 import Graceful from "@ladjs/graceful";
 import path from "path";
 import { fileURLToPath } from "url";
-import { CoreConfiguration } from "./configuration-manager.js";
-import logger from "../reporting-service/logger.js";
-
-type GracefulConstructor = new (config: { brees: Bree[] }) => {
-  listen: () => void;
-};
-
-const GracefulClass = Graceful as unknown as GracefulConstructor;
+import { CoreConfiguration } from "./configuration-manager";
+import logger from "../reporting-service/logger";
+import { createConnectorFromPath } from "../../utils/connector-factory";
+import { NatsLaborTicketListener } from "./nats-labor-ticket-listener";
 
 export const runDataSyncService = async (connectorPath: string) => {
   const config = CoreConfiguration.inst();
 
   try {
+    // Create connector instance
+    const connector = await createConnectorFromPath(connectorPath);
+
+    // Start NATS listener if enabled
+    if (process.env.NATS_ENABLED === "true") {
+      const natsListener = new NatsLaborTicketListener(connector);
+      await natsListener.start();
+    }
     // Find the SDK's node_modules directory and jobs path using ES module import
     const currentFileUrl = import.meta.url;
     const currentFilePath = fileURLToPath(currentFileUrl);
@@ -23,7 +27,12 @@ export const runDataSyncService = async (connectorPath: string) => {
     // Current: /path/to/sdk/dist/mm-erp-sdk.js.js
     // Target:  /path/to/sdk/dist/services/data-sync-service/jobs
     const sdkDistPath = path.dirname(currentFilePath);
-    const jobsPath = path.join(sdkDistPath, "jobs");
+    const jobsPath = path.join(
+      sdkDistPath,
+      "services",
+      "data-sync-service",
+      "jobs"
+    );
 
     const bree = new Bree({
       root: jobsPath,
@@ -79,7 +88,7 @@ export const runDataSyncService = async (connectorPath: string) => {
     }));
     logger.info("JOBS CONFIGURATION:", { jobs: jobsConfig });
 
-    const graceful = new GracefulClass({ brees: [bree] });
+    const graceful = new Graceful({ brees: [bree] });
     graceful.listen();
 
     (async () => {

package/src/services/data-sync-service/index.ts

@@ -1,11 +1,11 @@
 /**
  * Core data sync service functionality
  */
-export { runDataSyncService } from "./data-sync-service.js";
+export { runDataSyncService } from "./data-sync-service";
 
 /**
  * Configuration management for data sync
  */
-export { getSQLServerConfiguration } from "./configuration-manager.js";
+export { getSQLServerConfiguration } from "./configuration-manager";
 
-export { ErpApiConnectionParams, getErpApiConnectionParams } from "./configuration-manager.js";
+export { ErpApiConnectionParams, getErpApiConnectionParams } from "./configuration-manager";

package/src/services/data-sync-service/jobs/clean-up-expired-cache.ts

@@ -1,11 +1,10 @@
 import "dotenv/config";
-import { HashedCacheManager } from "../../caching-service/hashed-cache-manager.js";
-import { SQLiteCoordinator } from "../../sqlite-service/index.js";
-import logger from "../../../services/reporting-service/logger.js";
-import { getEnvString } from "../../../utils/env.js";
+import { HashedCacheManager } from "../../caching-service/hashed-cache-manager";
+import { SQLiteCoordinator } from "../../sqlite-service";
+import logger from "../../../services/reporting-service/logger";
 
 // Configure the logger with the correct log level
-logger.level = getEnvString("LOG_LEVEL", "info");
+logger.level = process.env.LOG_LEVEL || "info";
 
 const main = async () => {
   const cacheManager = new HashedCacheManager();

package/src/services/data-sync-service/jobs/from-erp.ts

@@ -1,12 +1,11 @@
 import "dotenv/config";
 
-import logger from "../../../services/reporting-service/logger.js";
-import { SQLiteCoordinator } from "../../sqlite-service/index.js";
-import { createConnectorFromPath } from "../../../utils/connector-factory.js";
-import { getEnvString } from "../../../utils/env.js";
+import logger from "../../../services/reporting-service/logger";
+import { SQLiteCoordinator } from "../../sqlite-service";
+import { createConnectorFromPath } from "../../../utils/connector-factory";
 
 // Configure the logger with the correct log level
-logger.level = getEnvString("LOG_LEVEL", "info");
+logger.level = process.env.LOG_LEVEL || "info";
 
 const main = async () => {
   try {
@@ -30,14 +29,9 @@ const main = async () => {
     await connector.syncFromERPCompleted();
     logger.info("==========Completed from-erp job cycle==========");
   } catch (error) {
-    const errorDetails = {
-      message: error instanceof Error ? error.message : String(error),
-      stack: error instanceof Error ? error.stack : undefined,
-      name: error instanceof Error ? error.name : undefined,
-      ...(error && typeof error === "object" ? error : {}), // Include all enumerable properties if it's an object
-    };
+    // Logger now handles circular references and error objects safely
     logger.error('Worker for job "from-erp" had an error', {
-      error: errorDetails,
+      error,
     });
 
     throw error; // Rethrow so Bree can handle it properly

package/src/services/data-sync-service/jobs/retry-failed-labor-tickets.ts

@@ -1,11 +1,10 @@
 import "dotenv/config";
 
-import logger from "../../../services/reporting-service/logger.js";
-import { getEnvString } from "../../../utils/env.js";
-import { createConnectorFromPath } from "../../../utils/connector-factory.js";
+import logger from "../../../services/reporting-service/logger";
+import { createConnectorFromPath } from "../../../utils/connector-factory";
 
 // Configure the logger with the correct log level
-logger.level = getEnvString("LOG_LEVEL", "info");
+logger.level = process.env.LOG_LEVEL || "info";
 
 const main = async () => {
   try {

package/src/services/data-sync-service/jobs/run-migrations.ts

@@ -1,10 +1,9 @@
 import knex, { Knex } from "knex";
-import logger from "../../reporting-service/logger.js";
-import config from "../../../knexfile.js";
-import { getEnvString } from "../../../utils/env.js";
+import logger from "../../reporting-service/logger";
+import config from "../../../knexfile";
 
 // Configure the logger with the correct log level
-logger.level = getEnvString("LOG_LEVEL", "info");
+logger.level = process.env.LOG_LEVEL || "info";
 
 // MLW TODO Consider the location of knexfile
 const db: Knex = knex(config.local);

package/src/services/data-sync-service/jobs/to-erp.ts

@@ -1,11 +1,11 @@
 import "dotenv/config";
 
-import logger from "../../reporting-service/logger.js";
-import { createConnectorFromPath } from "../../../utils/connector-factory.js";
-import { getEnvString } from "../../../utils/env.js";
+import logger from "../../reporting-service/logger";
+import { SQLiteCoordinator } from "../../sqlite-service";
+import { createConnectorFromPath } from "../../../utils/connector-factory";
 
 // Configure the logger with the correct log level
-logger.level = getEnvString("LOG_LEVEL", "info");
+logger.level = process.env.LOG_LEVEL || "info";
 
 const main = async () => {
   try {
@@ -22,7 +22,16 @@ const main = async () => {
     // Create a new connector instance for this job
     const connector = await createConnectorFromPath(connectorPath);
 
-    await connector.syncToERP();
+    // Only use lock if NATS is enabled (to prevent concurrent processing)
+    // Otherwise, no lock needed - just run directly
+    if (process.env.NATS_ENABLED === "true") {
+      await SQLiteCoordinator.executeWithLock("to-erp", async () => {
+        await connector.syncToERP();
+      });
+    } else {
+      await connector.syncToERP();
+    }
+
     await connector.syncToERPCompleted();
 
     logger.info("==========Completed to-erp job cycle==========");