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

package/src/utils/erp-timezone-utils.ts

@@ -0,0 +1,99 @@
+import { DateTime } from "luxon";
+import logger from "../services/reporting-service/logger.js";
+import { getCachedTimezoneName } from "./local-data-store/jobs-shared-data.js";
+
+/**
+ * Retrieves the ERP timezone name that should have been cached during initialization.
+ * @throws Error when the timezone cache has not been populated.
+ */
+export function getERPTimezone(): string {
+  const timezone = getCachedTimezoneName();
+  if (!timezone) {
+    throw new Error(
+      "ERP timezone is not cached. Run getTimezoneOffsetAndPersist() to populate it."
+    );
+  }
+  return timezone;
+}
+
+/**
+ * Converts a UTC ISO-8601 datetime string to the ERP's local timezone (also ISO-8601).
+ * @param utcDateTime ISO-8601 datetime string representing UTC time.
+ * @param timezone Optional timezone override; defaults to the cached ERP timezone.
+ * @returns ISO-8601 string in the ERP timezone.
+ */
+export function convertUtcDateTimeToErpLocal(
+  utcDateTime: string,
+  timezone?: string
+): string {
+  if (!utcDateTime) {
+    throw new Error("convertUtcDateTimeToErpLocal requires a non-empty UTC datetime string");
+  }
+  try {
+    const targetZone = timezone ?? getERPTimezone();
+    const parsedUtc = DateTime.fromISO(utcDateTime, { zone: "utc" });
+    if (!parsedUtc.isValid) {
+      throw new Error(
+        `Invalid UTC datetime "${utcDateTime}": ${parsedUtc.invalidReason}`
+      );
+    }
+
+    const localized = parsedUtc.setZone(targetZone);
+    if (!localized.isValid) {
+      throw new Error(
+        `Unable to convert UTC datetime "${utcDateTime}" to target zone "${targetZone}": ${localized.invalidReason}`
+      );
+    }
+
+    return localized.toISO();
+  } catch (error) {
+    logger.error("convertUtcDateTimeToErpLocal: Failed to convert datetime", {
+      utcDateTime,
+      error,
+    });
+    throw new Error(
+      "Failed to convert UTC datetime to ERP local timezone. See logs for details."
+    );
+  }
+}
+
+/**
+ * Converts an ERP-local ISO-8601 datetime string to UTC (still ISO-8601).
+ * @param localDateTime ISO-8601 datetime string representing ERP local time.
+ * @param timezone Optional timezone override; defaults to the cached ERP timezone.
+ * @returns ISO-8601 string in UTC.
+ */
+export function convertErpLocalDateTimeToUtc(
+  localDateTime: string,
+  timezone?: string
+): string {
+  if (!localDateTime) {
+    throw new Error("convertErpLocalDateTimeToUtc requires a non-empty local datetime string");
+  }
+  try {
+    const sourceZone = timezone ?? getERPTimezone();
+    const parsedLocal = DateTime.fromISO(localDateTime, { zone: sourceZone });
+    if (!parsedLocal.isValid) {
+      throw new Error(
+        `Invalid ERP local datetime "${localDateTime}" for zone "${sourceZone}": ${parsedLocal.invalidReason}`
+      );
+    }
+
+    const utcDateTime = parsedLocal.setZone("utc");
+    if (!utcDateTime.isValid) {
+      throw new Error(
+        `Unable to convert ERP local datetime "${localDateTime}" to UTC: ${utcDateTime.invalidReason}`
+      );
+    }
+
+    return utcDateTime.toISO();
+  } catch (error) {
+    logger.error("convertErpLocalDateTimeToUtc: Failed to convert datetime", {
+      localDateTime,
+      error,
+    });
+    throw new Error(
+      "Failed to convert ERP local datetime to UTC. See logs for details."
+    );
+  }
+}

package/src/utils/erp-type-from-entity.ts

@@ -1,4 +1,4 @@
-import { ERPObjType } from "../types/erp-types";
+import { ERPObjType } from "../types/erp-types.js";
 
 export function getERPAPITypeFromEntity(
   entity: string,

package/src/utils/http-client.ts

@@ -1,5 +1,5 @@
 import axios, { AxiosError, AxiosInstance, AxiosRequestConfig } from "axios";
-import logger from "../services/reporting-service/logger";
+import logger from "../services/reporting-service/logger.js";
 
 export class HTTPError extends Error {
   constructor(
@@ -64,10 +64,9 @@ class AxiosClient implements HTTPClient {
    *   It can be convenient to set `baseURL` for an instance of axios to pass relative URLs to methods of that instance.
    */
   constructor(baseUrl: string, retryAttempts: number) {
-    const timeout = parseInt(process.env.MM_API_TIMEOUT || "30000");
     this.client = axios.create({
       baseURL: baseUrl,
-      timeout: timeout,
+      timeout: 30000,
       headers: {
         "Content-Type": "application/json",
       },

package/src/utils/index.ts

@@ -1,93 +1,92 @@
 /**
  * Utility functions for data manipulation
  */
-export { getUniqueRows } from "./uniqueRows";
-export { removeExtraneousFields } from "./removeExtraneousFields";
-export { getPayloadWithoutIDField } from "./removeIdFieldFromPayload";
-export { trimObjectValues } from "./trimObjectValues";
-export { cleanupNumbers } from "./cleanup-numbers";
+export { getUniqueRows } from "./uniqueRows.js";
+export { removeExtraneousFields } from "./removeExtraneousFields.js";
+export { getPayloadWithoutIDField } from "./removeIdFieldFromPayload.js";
+export { trimObjectValues } from "./trimObjectValues.js";
+export { cleanupNumbers } from "./cleanup-numbers.js";
 export {
   addNewFieldFromExternalSource,
   addNewFieldFromLookupField,
-} from "./data-transformation";
+} from "./data-transformation.js";
 
 /**
  * Timezone and time-related utilities
  */
-export { getTimezoneOffsetAndPersist } from "./time-utils";
-export { formatDateWithTZOffset, convertToLocalTime, toISOWithOffset } from "./timezone";
-export { applyTimezoneOffsetsToFields } from "./time-utils";
-export * from "./time-utils";
+export { getTimezoneOffsetAndPersist } from "./time-utils.js";
+export { formatDateWithTZOffset, convertToLocalTime, toISOWithOffset } from "./timezone.js";
+export { applyTimezoneOffsetsToFields } from "./time-utils.js";
+export {
+  convertUtcDateTimeToErpLocal,
+  convertErpLocalDateTimeToUtc,
+  getERPTimezone,
+} from "./erp-timezone-utils.js";
+export * from "./time-utils.js";
 
 /**
  * Caching and record tracking utilities
  */
-export { RecordTrackingManager } from "../services/caching-service/record-tracking-manager";
-export type { RecordTrackingObject } from "../services/caching-service/record-tracking-manager";
-export { HashedCacheManager } from "../services/caching-service/hashed-cache-manager";
-export { BatchCacheManager } from "../services/caching-service/batch-cache-manager";
+export { RecordTrackingManager } from "../services/caching-service/record-tracking-manager.js";
+export type { RecordTrackingObject } from "../services/caching-service/record-tracking-manager.js";
+export { HashedCacheManager } from "../services/caching-service/hashed-cache-manager.js";
+export { BatchCacheManager } from "../services/caching-service/batch-cache-manager.js";
 
 /**
  * Process and data transformation utilities
  */
-export { StandardProcessDrivers } from "./standard-process-drivers/";
-export type { WriteEntitiesToMMResult } from "./standard-process-drivers/";
-export { MMBatchValidationError } from "./standard-process-drivers/";
+export { StandardProcessDrivers } from "./standard-process-drivers/index.js";
+export type { WriteEntitiesToMMResult } from "./standard-process-drivers/index.js";
+export { MMBatchValidationError } from "./standard-process-drivers/index.js";
 export {
   getCachedTimezoneOffset,
   getCachedTimezoneName,
-} from "./local-data-store/jobs-shared-data";
+} from "./local-data-store/jobs-shared-data.js";
 
 // Local data store
 export {
   getInitialLoadComplete,
   setInitialLoadComplete,
-} from "./local-data-store/jobs-shared-data";
+} from "./local-data-store/jobs-shared-data.js";
 
 /**
  * ERP API utilities
  */
-export * from "../services/erp-api-services/rest/rest-api-service";
-export * from "../services/erp-api-services/rest/get-query-params";
-export { getERPAPITypeFromEntity } from "./erp-type-from-entity";
+export * from "../services/erp-api-services/rest/rest-api-service.js";
+export * from "../services/erp-api-services/rest/get-query-params.js";
+export { getERPAPITypeFromEntity } from "./erp-type-from-entity.js";
 
 /**
  * HTTP Client utilities
  */
-export { HTTPClientFactory } from "./http-client";
-export { HTTPError } from "./http-client";
-export type { HTTPClient, HTTPRequestConfig, HTTPResponse } from "./http-client";
+export { HTTPClientFactory } from "./http-client.js";
+export { HTTPError } from "./http-client.js";
+export type { HTTPClient, HTTPRequestConfig, HTTPResponse } from "./http-client.js";
 
 /**
  * MM Labor Ticket utilities
  */
-export * from "./mm-labor-ticket-helpers";
-export { getErrorType } from './error-utils';
-
-/**
- * Error formatting utilities
- */
-export { formatError, formatErrorForLogging } from './error-formatter';
-export type { FormattedError } from './error-formatter';
+export * from "./mm-labor-ticket-helpers.js";
+export { getErrorType } from './error-utils.js';
 
 /**
  * MM Connector Logger utilities
  */
-export { MMConnectorLogger, LogEntry, type LogResponse, type LogLevelString } from './connector-log/mm-connector-logger';
-export { FileLogDeduper } from './connector-log/log-deduper';
+export { MMConnectorLogger, LogEntry, type LogResponse, type LogLevelString } from './connector-log/mm-connector-logger.js';
+export { FileLogDeduper } from './connector-log/log-deduper.js';
 
 /**
  * Application initializer
  */
-export * from "./application-initializer";
+export * from "./application-initializer.js";
 
 /**
  * GraphQL utilities
  */
-export { GraphQLService } from "../services/erp-api-services/graphql/graphql-service";
-export { OAuthClient } from "../services/erp-api-services/oauth-client";
+export { GraphQLService } from "../services/erp-api-services/graphql/graphql-service.js";
+export { OAuthClient } from "../services/erp-api-services/oauth-client.js";
 
 /**
  * Connector factory utilities
  */
-export { createConnectorFromPath } from "./connector-factory";
+export { createConnectorFromPath } from "./connector-factory.js";

package/src/utils/local-data-store/database-lock.ts

@@ -1,7 +1,7 @@
 import {
   readDatabaseLockState,
   writeDatabaseLockState,
-} from "./jobs-shared-data";
+} from "./jobs-shared-data.js";
 
 interface DatabaseLock {
   isLocked: boolean;

package/src/utils/local-data-store/jobs-shared-data.ts

@@ -72,6 +72,8 @@ export const setInitialLoadComplete = (complete: boolean): void => {
 };
 
 /**
+ * @deprecated The cached numeric offset cannot account for DST changes.
+ * Prefer using the cached timezone name with Luxon helpers for conversions.
  * Gets the company's cached current timezone offset (e.g., -5)
  * @returns The cached timezone offset or 0 if not found
  */

package/src/utils/mm-labor-ticket-helpers.ts

@@ -1,15 +1,13 @@
-import { convertToLocalTime, toISOWithOffset } from "./timezone";
-import { MMReceiveLaborTicket } from "../services/mm-api-service/types/receive-types";
+import { MMReceiveLaborTicket } from "../services/mm-api-service/types/receive-types.js";
+import { convertUtcDateTimeToErpLocal, getERPTimezone } from "./erp-timezone-utils.js";
 
 /**
- * Apply timezone offsets to datetime fields specifically for the MMApiReceiveLaborTicket object from the Zulu-based MM API
- * (presumably) before sending to the ERP
+ * Converts key datetime fields from UTC (MachineMetrics API) to the ERP's local timezone.
+ * All inputs/outputs are ISO-8601 strings, and invalid inputs throw errors.
  * @param laborTicket The MMApiReceiveLaborTicket object to convert
- * @param timezoneOffset The timezone offset to apply
  */
 export function convertLaborTicketToLocalTimezone(
-  laborTicket: MMReceiveLaborTicket,
-  timezoneOffset: number
+  laborTicket: MMReceiveLaborTicket
 ): MMReceiveLaborTicket {
   const timeFields = [
     "clockIn",
@@ -20,9 +18,14 @@ export function convertLaborTicketToLocalTimezone(
     "workOrderOperationClosedDate",
   ] as const;
 
+  const timezone = getERPTimezone();
   timeFields.forEach((field) => {
-    const localTime = convertToLocalTime(laborTicket[field], timezoneOffset);
-    laborTicket[field] = localTime ? toISOWithOffset(localTime, timezoneOffset) : null;
+    const value = laborTicket[field];
+    if (value) {
+      laborTicket[field] = convertUtcDateTimeToErpLocal(value, timezone);
+    } else {
+      laborTicket[field] = null;
+    }
   });
   return laborTicket;
 }

package/src/utils/removeExtraneousFields.ts

@@ -1,4 +1,4 @@
-import { ERPObject } from "../types/erp-types";
+import { ERPObject } from "../types/erp-types.js";
 
 /**
  * Retain only the fields listed in retainFields[] in the payload

package/src/utils/removeIdFieldFromPayload.ts

@@ -1,4 +1,4 @@
-import { ERPObject } from "../types/erp-types";
+import { ERPObject } from "../types/erp-types.js";
 
 /**
  * Remove all fields listed in idField[] the from the payload

package/src/utils/resource-group.ts

@@ -1,9 +1,9 @@
-import logger from "../services/reporting-service/logger";
+import logger from "../services/reporting-service/logger.js";
 import {
   MMApiClient,
   MMGraphQLResourceResponse,
   MMMachineGroupsResponse,
-} from "../services/mm-api-service";
+} from "../services/mm-api-service/index.js";
 
 // The GraphQLResourceTypes interface has been replaced by the centralized
 // MM response interfaces: MMGraphQLResourceResponse and MMMachineGroupsResponse

package/src/utils/standard-process-drivers/error-processor.ts

@@ -1,12 +1,12 @@
-import { ERPObjType } from "../../types/erp-types";
-import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager";
+import { ERPObjType } from "../../types/erp-types.js";
+import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager.js";
 import {
   IToRESTApiObject,
   MM207NonLaborTicketResponse,
   MM207LaborTicketResponse,
-} from "../../services/mm-api-service";
-import { EntityTransformer } from "../../services/mm-api-service/types/entity-transformer";
-import logger from "../../services/reporting-service/logger";
+} from "../../services/mm-api-service/index.js";
+import { EntityTransformer } from "../../services/mm-api-service/types/entity-transformer.js";
+import logger from "../../services/reporting-service/logger.js";
 
 /**
  * Handles error processing and record management utilities for MM API operations

package/src/utils/standard-process-drivers/index.ts

@@ -1,4 +1,4 @@
 // Public exports for standard-process-drivers
-export { StandardProcessDrivers } from "./standard-process-drivers";
-export type { WriteEntitiesToMMResult } from "./standard-process-drivers";
-export { MMBatchValidationError } from "./standard-process-drivers";
+export { StandardProcessDrivers } from "./standard-process-drivers.js";
+export type { WriteEntitiesToMMResult } from "./standard-process-drivers.js";
+export { MMBatchValidationError } from "./standard-process-drivers.js";

package/src/utils/standard-process-drivers/labor-ticket-erp-synchronizer.ts

@@ -1,9 +1,8 @@
-import { IERPLaborTicketHandler } from "../../types/erp-connector";
-import { MMApiClient } from "../../services/mm-api-service/mm-api-service";
-import { MMReceiveLaborTicket } from "../../services/mm-api-service/types/receive-types";
-import { convertLaborTicketToLocalTimezone } from "../mm-labor-ticket-helpers";
-import { getCachedTimezoneOffset } from "../local-data-store/jobs-shared-data";
-import logger from "../../services/reporting-service/logger";
+import { IERPLaborTicketHandler } from "../../types/erp-connector.js";
+import { MMApiClient } from "../../services/mm-api-service/mm-api-service.js";
+import { MMReceiveLaborTicket } from "../../services/mm-api-service/types/receive-types.js";
+import { convertLaborTicketToLocalTimezone } from "../mm-labor-ticket-helpers.js";
+import logger from "../../services/reporting-service/logger.js";
 
 /**
  * Handles synchronization of labor tickets between MachineMetrics and ERP systems
@@ -54,9 +53,12 @@ export class LaborTicketERPSynchronizer {
 
       // Find the most recent updatedAt timestamp from labor tickets. This will be used to update
       // the checkpoint to ensure there is no gap of time for the next sync.
+      // Consider, for instance, a time offset between an edge and the cloud, or that
+      // a DST change may have occurred between the last checkpoint and Now.
       const mostRecentUpdate = laborTicketsUpdates.reduce(
         (latest: string | null, ticket: MMReceiveLaborTicket) => {
-          if (!latest || !ticket.updatedAt) return latest;
+          if (!ticket.updatedAt) return latest;   // Skip tickets without updatedAt
+          if (!latest) return ticket.updatedAt;   // Initialize on first valid value
           return new Date(ticket.updatedAt) > new Date(latest)
             ? ticket.updatedAt
             : latest;
@@ -118,6 +120,9 @@ export class LaborTicketERPSynchronizer {
           timestamp: mostRecentUpdate || fallbackTimestamp,
         },
       });
+      logger.info("syncLaborTicketsToERP: Checkpoint saved:", {
+        checkpointTimestamp: (mostRecentUpdate || fallbackTimestamp)
+      });
     } catch (error) {
       logger.error("syncLaborTicketsToERP: Error:", error);
     }
@@ -214,10 +219,10 @@ export class LaborTicketERPSynchronizer {
   ): Promise<MMReceiveLaborTicket> {
     let laborTicketResult: MMReceiveLaborTicket;
 
-    laborTicketResult = convertLaborTicketToLocalTimezone(
-      laborTicket,
-      getCachedTimezoneOffset()
-    );
+    // Convert MM's UTC timestamps into the ERP's local timezone before invoking connector code.
+    // Connector implementations should treat the incoming values as localized wall time and avoid
+    // applying any further timezone shifts.
+    laborTicketResult = convertLaborTicketToLocalTimezone(laborTicket);
 
     logger.info(
       `processing laborTicket, id=${laborTicket.laborTicketId}, ref=${laborTicket.laborTicketRef}`

package/src/utils/standard-process-drivers/mm-entity-processor.ts

@@ -1,5 +1,5 @@
-import { ERPObjType } from "../../types/erp-types";
-import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager";
+import { ERPObjType } from "../../types/erp-types.js";
+import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager.js";
 import {
   IToRESTApiObject,
   MMSendPerson,
@@ -15,21 +15,27 @@ import {
   MM200LaborTicketResponse,
   MM207NonLaborTicketResponse,
   MM207LaborTicketResponse,
-} from "../../services/mm-api-service";
-import { MMApiClient } from "../../services/mm-api-service/mm-api-service";
-import { HTTPError } from "../http-client";
+} from "../../services/mm-api-service/index.js";
+import { MMApiClient } from "../../services/mm-api-service/mm-api-service.js";
+import { HTTPError } from "../http-client.js";
 import {
   WriteEntitiesToMMResult,
   MMBatchValidationError,
-} from "./standard-process-drivers";
-import { ErrorProcessor } from "./error-processor";
+} from "./standard-process-drivers.js";
+import { ErrorProcessor } from "./error-processor.js";
 
 /**
  * Handles processing of entities to the MachineMetrics API
  */
 export class MMEntityProcessor {
   /**
-   * Writes entities to MM API with deduplication and caching
+   * Writes entities to MM API with deduplication and caching.
+   *
+   * IMPORTANT: All datetime fields on `mmRecords` MUST already be expressed as
+   * ISO-8601 UTC strings (either trailing `Z` or an explicit offset). The SDK
+   * does not apply timezone conversion on this path. It forwards values directly
+   * to MachineMetrics and only reserializes them for validation. Connector code
+   * is responsible for converting local ERP times to UTC before invoking this API.
    */
   static async writeEntities(
     entityType: ERPObjType,

package/src/utils/standard-process-drivers/standard-process-drivers.ts

@@ -1,6 +1,7 @@
-import { ERPObjType } from "../../types/erp-types";
-import { IERPLaborTicketHandler } from "../../types/erp-connector";
-import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager";
+import { ERPObjType } from "../../types/erp-types.js";
+import { IERPLaborTicketHandler } from "../../types/erp-connector.js";
+import type { FlattenedWorkOrderRow } from "../../types/flattened-work-order.js";
+import { BatchCacheManager } from "../../services/caching-service/batch-cache-manager.js";
 import {
   IToRESTApiObject,
   MMSendWorkOrderBatch,
@@ -8,9 +9,9 @@ import {
   MMSendPartOperation,
   MMSendWorkOrder,
   MMSendWorkOrderOperation,
-} from "../../services/mm-api-service";
-import { LaborTicketERPSynchronizer } from "./labor-ticket-erp-synchronizer";
-import { MMEntityProcessor } from "./mm-entity-processor";
+} from "../../services/mm-api-service/index.js";
+import { LaborTicketERPSynchronizer } from "./labor-ticket-erp-synchronizer.js";
+import { MMEntityProcessor } from "./mm-entity-processor.js";
 
 /**
  * The result of writing entities to the MachineMetrics API on complete success.
@@ -248,7 +249,12 @@ export class StandardProcessDrivers {
    * parts, part operations, work orders, and work order operations, then process them in the correct order
    * to maintain referential integrity.
    *
-   * @param flattenedData Array of flattened rows containing both work order and operation data (camelCase fields)
+   * Timezone expectation: all datetime fields provided in `flattenedData` MUST already be expressed
+   * as ISO-8601 UTC strings (trailing `Z` or explicit offset). The SDK does not attempt to infer or
+   * convert local ERP timestamps on this path; it simply validates and forwards the values to MM.
+   * Connector implementations should convert ERP-local times to UTC before invoking this method.
+   *
+   * @param flattenedData Array of flattened rows containing both work order and operation data (see `FlattenedWorkOrderRow`)
    * @param batchCacheManager The batch cache manager instance; pass in null if caching is not desired
    *
    * @returns Combined results from all entity processing with detailed logging information
@@ -256,7 +262,7 @@ export class StandardProcessDrivers {
    * @throws Error on other underlying issues (network, authentication, etc.)
    */
   static async syncWorkOrderBatchFromFlattened(
-    flattenedData: any[],
+    flattenedData: ReadonlyArray<FlattenedWorkOrderRow>,
     batchCacheManager: BatchCacheManager | null
   ): Promise<{
     parts: WriteEntitiesToMMResult;
@@ -268,6 +274,11 @@ export class StandardProcessDrivers {
       throw new Error("No flattened work order data provided");
     }
 
+    const toStringOrFallback = (
+      value: string | number | null | undefined,
+      fallback = ""
+    ): string => (value === undefined || value === null ? fallback : String(value));
+
     // Process the flattened data - each row contains both work order and operation info
     const uniqueParts = new Map();
     const uniquePartOperations = new Map();
@@ -297,8 +308,8 @@ export class StandardProcessDrivers {
           resourceId: row.resourceId, // → resourceId
           cycleTimeMs: row.cycleTimeMs, // → cycleTimeMs
           setupTimeMs: row.setupTimeMs, // → setupTimeMs
-          description: row.operationDescription || "", // → description
-          quantityPerPart: row.quantityPerPart || 1, // → quantityPerPart
+          operationDescription: row.operationDescription, // → description
+          quantityPerPart: row.quantityPerPart ?? 1, // → quantityPerPart
         });
       }
 
@@ -351,8 +362,8 @@ export class StandardProcessDrivers {
     const parts = Array.from(uniqueParts.values()).map(
       (item) =>
         new MMSendPart(
-          item.partNumber || "", // partNumber
-          item.partRevision || "", // partRevision
+          toStringOrFallback(item.partNumber), // partNumber
+          toStringOrFallback(item.partRevision), // partRevision
           item.method || "Standard" // method
         )
     );
@@ -360,14 +371,14 @@ export class StandardProcessDrivers {
     const partOperations = Array.from(uniquePartOperations.values()).map(
       (item) =>
         new MMSendPartOperation(
-          item.partNumber || "", // partNumber
-          item.partRevision || "", // partRevision
+          toStringOrFallback(item.partNumber), // partNumber
+          toStringOrFallback(item.partRevision), // partRevision
           item.method || "Standard", // method
           item.sequenceNumber?.toString() || "", // sequenceNumber
           item.resourceId?.toString() || "", // resourceId
           item.cycleTimeMs || 0, // cycleTimeMs
           item.setupTimeMs || 0, // setupTimeMs
-          item.description || "", // description
+          item.operationDescription || "", // description
           item.quantityPerPart || 1 // quantityPerPart
         )
     );
@@ -376,10 +387,12 @@ export class StandardProcessDrivers {
       (item) =>
         new MMSendWorkOrder(
           item.workOrderId?.toString() || "", // workOrderId
-          item.lot || "", // lot
-          item.split || "", // split
-          item.sub || "", // sub
+          toStringOrFallback(item.lot), // lot
+          toStringOrFallback(item.split), // split
+          toStringOrFallback(item.sub), // sub
           item.status || "Open", // status
+          // Datetimes are assumed to already be UTC (or include an explicit offset).
+          // We reserialize via Date solely to validate the ISO format before sending to MM.
           item.dueDate ? new Date(item.dueDate).toISOString() : null, // dueDate
           item.description || "", // description
           item.scheduledStartDate
@@ -390,8 +403,8 @@ export class StandardProcessDrivers {
             : null, // scheduledEndDate
           item.closedDate ? new Date(item.closedDate).toISOString() : null, // closedDate
           item.quantityRequired || 0, // quantityRequired
-          item.partNumber || "", // partNumber
-          item.partRevision || "", // partRevision
+          toStringOrFallback(item.partNumber), // partNumber
+          toStringOrFallback(item.partRevision), // partRevision
           item.method || "Standard" // method
         )
     );
@@ -400,14 +413,15 @@ export class StandardProcessDrivers {
       (item) =>
         new MMSendWorkOrderOperation(
           item.workOrderId?.toString() || "", // workOrderId
-          item.lot || "", // lot
-          item.split || "", // split
-          item.sub || "", // sub
+          toStringOrFallback(item.lot), // lot
+          toStringOrFallback(item.split), // split
+          toStringOrFallback(item.sub), // sub
           item.sequenceNumber?.toString() || "", // sequenceNumber
           item.resourceId?.toString() || "", // resourceId
           item.startQuantity || 0, // startQuantity
           item.finishQuantity || 0, // finishQuantity
           item.expectedRejectRate || 0, // expectedRejectRate
+          // Same UTC expectation as above; reserialize to ensure valid ISO.
           item.scheduledStartDate
             ? new Date(item.scheduledStartDate).toISOString()
             : null, // scheduledStartDate
@@ -417,8 +431,8 @@ export class StandardProcessDrivers {
           item.closedDate ? new Date(item.closedDate).toISOString() : null, // closedDate
           item.cycleTimeMs || 0, // cycleTimeMs
           item.setupTimeMs || 0, // setupTimeMs
-          parseFloat(item.productionburdenRateHourly || "0"), // productionburdenRateHourly
-          parseFloat(item.setupburdenRatehourly || "0"), // setupburdenRatehourly
+          parseFloat(toStringOrFallback(item.productionburdenRateHourly, "0")), // productionburdenRateHourly
+          parseFloat(toStringOrFallback(item.setupburdenRatehourly, "0")), // setupburdenRatehourly
           item.operationType || "Production", // operationType
           item.quantityPerPart || 1, // quantityPerPart
           item.status || "Open" // status