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

package/dist/services/data-sync-service/jobs/clean-up-expired-cache.js

@@ -1,7 +1,7 @@
-import "../../../config-C2Dse2g2.js";
-import { H as HashedCacheManager } from "../../../hashed-cache-manager-DbG72eRm.js";
-import { S as SQLiteCoordinator } from "../../../index-Cq9tNcJT.js";
-import { l as logger } from "../../../logger-DeKxCUPp.js";
+import "../../../config-Bax6Ofp5.js";
+import { H as HashedCacheManager } from "../../../hashed-cache-manager-C1u9jQgY.js";
+import { S as SQLiteCoordinator } from "../../../index-BkVlW0ZW.js";
+import { l as logger } from "../../../logger-DW5fyhVS.js";
 logger.level = process.env.LOG_LEVEL || "info";
 const main = async () => {
   const cacheManager = new HashedCacheManager();

package/dist/services/data-sync-service/jobs/from-erp.js

@@ -1,7 +1,7 @@
-import "../../../config-C2Dse2g2.js";
-import { l as logger } from "../../../logger-DeKxCUPp.js";
-import { S as SQLiteCoordinator } from "../../../index-Cq9tNcJT.js";
-import { c as createConnectorFromPath } from "../../../connector-factory-BGykiICT.js";
+import "../../../config-Bax6Ofp5.js";
+import { l as logger } from "../../../logger-DW5fyhVS.js";
+import { S as SQLiteCoordinator } from "../../../index-BkVlW0ZW.js";
+import { c as createConnectorFromPath } from "../../../connector-factory-BaMIlES8.js";
 logger.level = process.env.LOG_LEVEL || "info";
 const main = async () => {
   try {

package/dist/services/data-sync-service/jobs/retry-failed-labor-tickets.js

@@ -1,6 +1,6 @@
-import "../../../config-C2Dse2g2.js";
-import { l as logger } from "../../../logger-DeKxCUPp.js";
-import { c as createConnectorFromPath } from "../../../connector-factory-BGykiICT.js";
+import "../../../config-Bax6Ofp5.js";
+import { l as logger } from "../../../logger-DW5fyhVS.js";
+import { c as createConnectorFromPath } from "../../../connector-factory-BaMIlES8.js";
 logger.level = process.env.LOG_LEVEL || "info";
 const main = async () => {
   try {

package/dist/services/data-sync-service/jobs/run-migrations.js

@@ -1,5 +1,5 @@
 import knex from "knex";
-import { l as logger } from "../../../logger-DeKxCUPp.js";
+import { l as logger } from "../../../logger-DW5fyhVS.js";
 import { c as config } from "../../../knexfile-Bng2Ru9c.js";
 logger.level = process.env.LOG_LEVEL || "info";
 const db = knex(config.local);

package/dist/services/data-sync-service/jobs/to-erp.js

@@ -1,6 +1,6 @@
-import "../../../config-C2Dse2g2.js";
-import { l as logger } from "../../../logger-DeKxCUPp.js";
-import { c as createConnectorFromPath } from "../../../connector-factory-BGykiICT.js";
+import "../../../config-Bax6Ofp5.js";
+import { l as logger } from "../../../logger-DW5fyhVS.js";
+import { c as createConnectorFromPath } from "../../../connector-factory-BaMIlES8.js";
 logger.level = process.env.LOG_LEVEL || "info";
 const main = async () => {
   try {

package/dist/services/reporting-service/logger.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../../src/services/reporting-service/logger.ts"],"names":[],"mappings":"AA+DA,QAAA,MAAM,MAAM,0BAaV,CAAC;AA0DH,eAAO,MAAM,eAAe,GAAI,UAAU,MAAM,EAAE,SAAS,MAAM,SAuChE,CAAC;AAKF,eAAe,MAAM,CAAC"}
+{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../../src/services/reporting-service/logger.ts"],"names":[],"mappings":"AAuGA,QAAA,MAAM,MAAM,0BAIV,CAAC;AAuGH,eAAO,MAAM,eAAe,GAAI,UAAU,MAAM,EAAE,SAAS,MAAM,SA8BhE,CAAC;AAKF,eAAe,MAAM,CAAC"}

package/dist/utils/standard-process-drivers/labor-ticket-erp-synchronizer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"labor-ticket-erp-synchronizer.d.ts","sourceRoot":"","sources":["../../../src/utils/standard-process-drivers/labor-ticket-erp-synchronizer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AAOnE;;GAEG;AACH,qBAAa,0BAA0B;IACrC;;OAEG;WACU,SAAS,CACpB,aAAa,EAAE,MAAM,EACrB,SAAS,EAAE,sBAAsB,GAChC,OAAO,CAAC,IAAI,CAAC;IAgHhB;;OAEG;WACU,WAAW,CACtB,aAAa,EAAE,MAAM,EACrB,SAAS,EAAE,sBAAsB,GAChC,OAAO,CAAC,IAAI,CAAC;mBA+DK,sBAAsB;mBAetB,kBAAkB;CAkDxC"}
+{"version":3,"file":"labor-ticket-erp-synchronizer.d.ts","sourceRoot":"","sources":["../../../src/utils/standard-process-drivers/labor-ticket-erp-synchronizer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AAOnE;;GAEG;AACH,qBAAa,0BAA0B;IACrC;;OAEG;WACU,SAAS,CACpB,aAAa,EAAE,MAAM,EACrB,SAAS,EAAE,sBAAsB,GAChC,OAAO,CAAC,IAAI,CAAC;IA4GhB;;OAEG;WACU,WAAW,CACtB,aAAa,EAAE,MAAM,EACrB,SAAS,EAAE,sBAAsB,GAChC,OAAO,CAAC,IAAI,CAAC;mBA+DK,sBAAsB;mBAetB,kBAAkB;CAkDxC"}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@machinemetrics/mm-erp-sdk",
   "description": "A library for syncing data between MachineMetrics and ERP systems",
-  "version": "0.1.9-beta.0",
+  "version": "0.1.9-beta.2",
   "license": "MIT",
   "author": "machinemetrics",
   "main": "dist/mm-erp-sdk.js",

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

@@ -551,7 +551,7 @@ export class MMApiClient {
       logger.info("saveCheckpoint:", { saveCheckpoint });
     } else {
       logger.info("Existing checkpoint found:", {
-        checkpointTimestamp: checkpointResponse.timestamp,
+        timestamp: checkpointResponse.timestamp,
       });
     }
   }

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;
@@ -60,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,
@@ -86,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
@@ -99,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 {}
+      }
+    }
   });
 }
 
@@ -157,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/utils/standard-process-drivers/labor-ticket-erp-synchronizer.ts

@@ -56,8 +56,7 @@ export class LaborTicketERPSynchronizer {
       // the checkpoint to ensure there is no gap of time for the next sync.
       const mostRecentUpdate = laborTicketsUpdates.reduce(
         (latest: string | null, ticket: MMReceiveLaborTicket) => {
-          if (!ticket.updatedAt) return latest;   // Skip tickets without updatedAt
-          if (!latest) return ticket.updatedAt;   // Initialize on first valid value
+          if (!latest || !ticket.updatedAt) return latest;
           return new Date(ticket.updatedAt) > new Date(latest)
             ? ticket.updatedAt
             : latest;
@@ -119,9 +118,6 @@ export class LaborTicketERPSynchronizer {
           timestamp: mostRecentUpdate || fallbackTimestamp,
         },
       });
-      logger.info("syncLaborTicketsToERP: Checkpoint saved:", {
-        checkpointTimestamp: (mostRecentUpdate || fallbackTimestamp)
-      });
     } catch (error) {
       logger.error("syncLaborTicketsToERP: Error:", error);
     }