@machinemetrics/mm-erp-sdk 0.1.7-beta.3 → 0.1.8-beta.1

package/dist/mm-erp-sdk.js

@@ -1,14 +1,14 @@
-import { C as CoreConfiguration, H as HashedCacheManager } from "./hashed-cache-manager-CzyFSt2B.js";
-import { E, g, a } from "./hashed-cache-manager-CzyFSt2B.js";
-import { l as logger } from "./logger-Db8CkwR6.js";
-import { g as getCachedMMToken, s as setCachedMMToken, a as setTimezoneOffsetInCache, b as getCachedTimezoneOffset, S as SQLiteCoordinator } from "./index-B9wo8pld.js";
-import { c, d } from "./index-B9wo8pld.js";
+import { C as CoreConfiguration, H as HashedCacheManager } from "./hashed-cache-manager-B6hTDLxU.js";
+import { E, g, a } from "./hashed-cache-manager-B6hTDLxU.js";
+import { l as logger } from "./logger-SqdNut1H.js";
+import { g as getCachedMMToken, s as setCachedMMToken, a as setTimezoneOffsetInCache, b as getCachedTimezoneOffset, S as SQLiteCoordinator } from "./index-Bg76oouR.js";
+import { c, d } from "./index-Bg76oouR.js";
 import axios, { AxiosError } from "axios";
 import knex from "knex";
 import { c as config } from "./knexfile-1qKKIORB.js";
 import fs from "fs";
 import path from "path";
-import "./connector-factory-C2czCs9v.js";
+import "./connector-factory-D8v6aQIt.js";
 import Bree from "bree";
 import Graceful from "@ladjs/graceful";
 import { fileURLToPath } from "url";
@@ -3607,8 +3607,6 @@ const runDataSyncService = async (connectorPath) => {
     const bree = new Bree({
       root: jobsPath,
       logger,
-      // Enable worker metadata for debugging
-      outputWorkerMetadata: true,
       worker: {
         env: {
           CONNECTOR_PATH: connectorPath,
@@ -3632,25 +3630,20 @@ const runDataSyncService = async (connectorPath) => {
         {
           name: "from-erp",
           timeout: "10s",
-          interval: config2.fromErpInterval,
-          // Ensure worker terminates completely after execution
-          closeWorkerAfterMs: 1e3
+          interval: config2.fromErpInterval
         },
         {
           name: "to-erp",
           //timeout: '3s',                  // Use timeout during development to see the job in action quickly
-          interval: config2.toErpInterval,
-          closeWorkerAfterMs: 1e3
+          interval: config2.toErpInterval
         },
         {
           name: "retry-failed-labor-tickets",
-          interval: config2.retryLaborTicketsInterval,
-          closeWorkerAfterMs: 1e3
+          interval: config2.retryLaborTicketsInterval
         },
         {
           name: "clean-up-expired-cache",
-          interval: config2.cacheExpirationCheckInterval,
-          closeWorkerAfterMs: 1e3
+          interval: config2.cacheExpirationCheckInterval
         }
       ]
     });
@@ -3660,34 +3653,14 @@ const runDataSyncService = async (connectorPath) => {
     const jobsConfig = bree.config.jobs.map((job) => ({
       name: job.name,
       interval: job.interval,
-      timeout: job.timeout,
-      closeWorkerAfterMs: job.closeWorkerAfterMs
+      timeout: job.timeout
     }));
     logger.info("JOBS CONFIGURATION:", { jobs: jobsConfig });
     const graceful = new Graceful({ brees: [bree] });
     graceful.listen();
-    bree.on("worker created", (name) => {
-      logger.debug(`Worker created for job: ${name}`);
-    });
-    bree.on("worker online", (name) => {
-      logger.debug(`Worker online for job: ${name}`);
-    });
-    bree.on("worker message", (name, message) => {
-      logger.debug(`Worker message from ${name}:`, message);
-    });
-    bree.on("worker deleted", (name) => {
-      logger.debug(`Worker deleted for job: ${name}`);
-    });
-    bree.on("worker exit", (name, code, signal) => {
-      if (code !== 0) {
-        logger.error(
-          `Worker ${name} exited with code ${code}, signal ${signal}`
-        );
-      } else {
-        logger.debug(`Worker ${name} exited successfully`);
-      }
-    });
-    await bree.start();
+    (async () => {
+      await bree.start();
+    })();
     bree.on("jobStarted", (job) => {
       console.log("Job " + job.name + " started");
     });
@@ -3995,14 +3968,14 @@ class SqlServerHelper {
   }
 }
 class PsqlService {
-  connection = null;
   config;
   constructor(config2) {
     this.config = config2;
   }
-  async dispose() {
-    await this.closeConnection();
-  }
+  // REMOVED: dispose() method - not needed anymore
+  // REMOVED: connection property - not needed anymore
+  // REMOVED: openConnection() method - not needed anymore
+  // REMOVED: closeConnection() method - not needed anymore
   /**
    * Build PSQL ODBC connection string
    * CRITICAL: ServerName must use IP.PORT format (e.g., 10.4.0.11.1583)
@@ -4020,7 +3993,7 @@ class PsqlService {
   }
   /**
    * Execute a query and return the results
-   * Interface matches SqlServerService for consistency
+   * Creates a fresh connection for each query to avoid handle corruption
    *
    * @param query The SQL query to execute
    * @param params Query parameters (currently unused for PSQL read operations)
@@ -4028,73 +4001,58 @@ class PsqlService {
    * @returns The entities fetched from the database, along with paging information
    */
   async executePreparedStatement(query, params = {}, paging) {
-    const connection = await this.openConnection();
-    let records;
+    let connection = null;
     try {
+      const connStr = this.buildConnectionString();
+      logger.debug("Creating fresh PSQL connection for query");
+      connection = await odbc.connect(connStr);
       if (Object.keys(params).length > 0) {
         logger.warn(
           "PsqlService: Query parameters provided but parameter binding not yet implemented. Using direct query execution. This is acceptable for Phase 1 read operations."
         );
       }
-      records = await connection.query(query);
+      const records = await connection.query(query);
+      const allRecords = PsqlService.recordsetToRecords(records);
+      const rowsFetched = allRecords.length;
+      const pagedData = paging?.offset !== void 0 || paging?.limit !== void 0 ? allRecords.slice(
+        paging.offset || 0,
+        (paging.offset || 0) + (paging.limit || allRecords.length)
+      ) : allRecords;
+      return {
+        data: pagedData,
+        paging: {
+          count: rowsFetched,
+          limit: paging?.limit || 0,
+          offset: paging?.offset || 0,
+          nextPage: paging?.limit && (paging.offset || 0) + paging.limit < rowsFetched ? String((paging.offset || 0) + paging.limit) : void 0,
+          previousPage: paging?.offset ? String(Math.max(0, (paging.offset || 0) - (paging.limit || 10))) : void 0
+        }
+      };
     } catch (error) {
       const errorInfo = error;
       logger.error("Error fetching data from PSQL", {
         error: errorInfo.message,
-        odbcErrors: errorInfo.odbcErrors
+        odbcErrors: errorInfo.odbcErrors,
+        query: query.substring(0, 200)
+        // Log first 200 chars of query
       });
       throw this.handleOdbcError(errorInfo);
-    }
-    const allRecords = PsqlService.recordsetToRecords(records);
-    const rowsFetched = allRecords.length;
-    const pagedData = paging?.offset !== void 0 || paging?.limit !== void 0 ? allRecords.slice(
-      paging.offset || 0,
-      (paging.offset || 0) + (paging.limit || allRecords.length)
-    ) : allRecords;
-    return {
-      data: pagedData,
-      paging: {
-        count: rowsFetched,
-        limit: paging?.limit || 0,
-        offset: paging?.offset || 0,
-        nextPage: paging?.limit && (paging.offset || 0) + paging.limit < rowsFetched ? String((paging.offset || 0) + paging.limit) : void 0,
-        previousPage: paging?.offset ? String(Math.max(0, (paging.offset || 0) - (paging.limit || 10))) : void 0
+    } finally {
+      if (connection) {
+        try {
+          await connection.close();
+          logger.debug("PSQL connection closed successfully");
+        } catch (err) {
+          logger.warn("Error closing PSQL connection (non-fatal)", {
+            error: err
+          });
+        }
       }
-    };
-  }
-  /**
-   * Opens a connection to PSQL database
-   * Caches the connection so that it can be reused.
-   * On failure to connect, throws
-   */
-  async openConnection() {
-    if (this.connection) {
-      logger.debug("Reusing existing PSQL connection");
-      return this.connection;
-    }
-    try {
-      const connStr = this.buildConnectionString();
-      logger.info("Opening new PSQL connection");
-      logger.debug(
-        "Connection string (password hidden):",
-        connStr.replace(/PWD=[^;]+/, "PWD=***")
-      );
-      this.connection = await odbc.connect(connStr);
-      logger.info("Successfully connected to PSQL database");
-      return this.connection;
-    } catch (error) {
-      logger.error("PsqlService>>openConnection>> Connection failed", {
-        error
-      });
-      throw this.handleOdbcError(error);
     }
   }
   /**
    * Transform ODBC result set to array of Record<string, string> instances.
    * IMPORTANT: PSQL CHAR fields are often padded with spaces - we trim them
-   *
-   * @param recordset Result set from ODBC query
-   * @returns array of Record<string, string> instances
    */
   static recordsetToRecords(recordset) {
     if (!Array.isArray(recordset)) {
@@ -4136,19 +4094,6 @@ class PsqlService {
         return new Error(`PSQL error (${errorCode || "unknown"}): ${message}`);
     }
   }
-  async closeConnection() {
-    if (this.connection) {
-      logger.info("Closing PSQL connection");
-      try {
-        await this.connection.close();
-      } catch (error) {
-        logger.error("PsqlService::closeConnection: Error closing connection", {
-          error
-        });
-      }
-      this.connection = null;
-    }
-  }
 }
 class PsqlLaborTicketOperations {
   constructor(service) {