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

package/dist/services/psql-erp-service/configuration.d.ts

@@ -1,10 +0,0 @@
-export interface PsqlConfiguration {
-    host: string;
-    port: string;
-    database: string;
-    username: string;
-    password: string;
-    connectionTimeout?: string;
-    requestTimeout?: string;
-}
-//# sourceMappingURL=configuration.d.ts.map

package/dist/services/psql-erp-service/configuration.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"configuration.d.ts","sourceRoot":"","sources":["../../../src/services/psql-erp-service/configuration.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB"}

package/dist/services/psql-erp-service/index.d.ts

@@ -1,19 +0,0 @@
-import { PsqlService } from "./psql-service";
-import { PsqlLaborTicketOperations } from "./internal/psql-labor-ticket-operations";
-/**
- * A class to manage interactions with PSQL (Pervasive) databases via ODBC
- */
-export { PsqlService };
-/**
- * Labor ticket operations for PSQL (Phase 2)
- */
-export { PsqlLaborTicketOperations };
-/**
- * Configuration interface for PSQL connections
- */
-export type { PsqlConfiguration } from "./configuration";
-/**
- * Helper functions for PSQL data formatting
- */
-export { formatPsqlDate, formatPsqlTime, combinePsqlDateTime, isPsqlDateEmpty, cleanPsqlCharField, } from "./psql-helpers";
-//# sourceMappingURL=index.d.ts.map

package/dist/services/psql-erp-service/index.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/services/psql-erp-service/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC;AAC7C,OAAO,EAAE,yBAAyB,EAAE,MAAM,yCAAyC,CAAC;AAEpF;;GAEG;AACH,OAAO,EAAE,WAAW,EAAE,CAAC;AAEvB;;GAEG;AACH,OAAO,EAAE,yBAAyB,EAAE,CAAC;AAErC;;GAEG;AACH,YAAY,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AAEzD;;GAEG;AACH,OAAO,EACL,cAAc,EACd,cAAc,EACd,mBAAmB,EACnB,eAAe,EACf,kBAAkB,GACnB,MAAM,gBAAgB,CAAC"}

package/dist/services/psql-erp-service/internal/psql-config.d.ts

@@ -1,28 +0,0 @@
-import { z } from "zod";
-export declare const PsqlConfigSchema: z.ZodObject<{
-    host: z.ZodString;
-    port: z.ZodString;
-    database: z.ZodString;
-    username: z.ZodString;
-    password: z.ZodString;
-    connectionTimeout: z.ZodDefault<z.ZodOptional<z.ZodString>>;
-    requestTimeout: z.ZodDefault<z.ZodOptional<z.ZodString>>;
-}, "strip", z.ZodTypeAny, {
-    password: string;
-    database: string;
-    port: string;
-    connectionTimeout: string;
-    requestTimeout: string;
-    host: string;
-    username: string;
-}, {
-    password: string;
-    database: string;
-    port: string;
-    host: string;
-    username: string;
-    connectionTimeout?: string | undefined;
-    requestTimeout?: string | undefined;
-}>;
-export type PsqlConfig = z.infer<typeof PsqlConfigSchema>;
-//# sourceMappingURL=psql-config.d.ts.map

package/dist/services/psql-erp-service/internal/psql-config.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"psql-config.d.ts","sourceRoot":"","sources":["../../../../src/services/psql-erp-service/internal/psql-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,eAAO,MAAM,gBAAgB;;;;;;;;;;;;;;;;;;;;;;;;EAQ3B,CAAC;AAEH,MAAM,MAAM,UAAU,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,gBAAgB,CAAC,CAAC"}

package/dist/services/psql-erp-service/internal/psql-labor-ticket-operations.d.ts

@@ -1,40 +0,0 @@
-/**
- * PSQL Labor Ticket Operations
- *
- * Phase 2: This will handle INSERT/UPDATE/DELETE operations for labor tickets
- * Phase 1: Placeholder - throws errors if called
- */
-import { PsqlService } from "../psql-service";
-import { MMReceiveLaborTicket } from "../../../services/mm-api-service";
-export declare class PsqlLaborTicketOperations {
-    private service;
-    constructor(service: PsqlService);
-    /**
-     * Create labor ticket in START_LABOR table
-     *
-     * Phase 2 Implementation Notes:
-     * - Will use prepared statements with parameter binding
-     * - Insert into START_LABOR table
-     * - Return GUID as erpUid
-     *
-     * @param laborTicket Labor ticket from MachineMetrics
-     * @returns Labor ticket and ERP unique ID
-     */
-    createLaborTicket(laborTicket: MMReceiveLaborTicket): Promise<{
-        laborTicket: MMReceiveLaborTicket;
-        erpUid: string;
-    }>;
-    /**
-     * Update labor ticket (move from START_LABOR to COMPLETED_LABOR)
-     *
-     * Phase 2 Implementation Notes:
-     * - Insert into COMPLETED_LABOR
-     * - Delete from START_LABOR
-     * - Should be done in a transaction
-     *
-     * @param laborTicket Labor ticket to update
-     * @returns Updated labor ticket
-     */
-    updateLaborTicket(laborTicket: MMReceiveLaborTicket): Promise<MMReceiveLaborTicket>;
-}
-//# sourceMappingURL=psql-labor-ticket-operations.d.ts.map

package/dist/services/psql-erp-service/internal/psql-labor-ticket-operations.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"psql-labor-ticket-operations.d.ts","sourceRoot":"","sources":["../../../../src/services/psql-erp-service/internal/psql-labor-ticket-operations.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAC9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,kCAAkC,CAAC;AAGxE,qBAAa,yBAAyB;IACxB,OAAO,CAAC,OAAO;gBAAP,OAAO,EAAE,WAAW;IAExC;;;;;;;;;;OAUG;IACG,iBAAiB,CACrB,WAAW,EAAE,oBAAoB,GAChC,OAAO,CAAC;QAAE,WAAW,EAAE,oBAAoB,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,CAAC;IASjE;;;;;;;;;;OAUG;IACG,iBAAiB,CACrB,WAAW,EAAE,oBAAoB,GAChC,OAAO,CAAC,oBAAoB,CAAC;CAQjC"}

package/dist/services/psql-erp-service/internal/types/psql-types.d.ts

@@ -1,15 +0,0 @@
-/**
- * PSQL-specific type definitions
- */
-export interface PsqlConnectionOptions {
-    connectionString: string;
-}
-export interface OdbcError {
-    state: string;
-    message: string;
-    code?: number;
-}
-export interface OdbcErrorResponse extends Error {
-    odbcErrors?: OdbcError[];
-}
-//# sourceMappingURL=psql-types.d.ts.map

package/dist/services/psql-erp-service/internal/types/psql-types.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"psql-types.d.ts","sourceRoot":"","sources":["../../../../../src/services/psql-erp-service/internal/types/psql-types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,MAAM,WAAW,qBAAqB;IACpC,gBAAgB,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,SAAS;IACxB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,iBAAkB,SAAQ,KAAK;IAC9C,UAAU,CAAC,EAAE,SAAS,EAAE,CAAC;CAC1B"}

package/dist/services/psql-erp-service/psql-helpers.d.ts

@@ -1,32 +0,0 @@
-/**
- * Helper functions for PSQL/Pervasive database operations
- */
-/**
- * Formats a date from PSQL YYMMDD format to ISO date string
- * @param psqlDate Date in YYMMDD format (e.g., "250105" for Jan 5, 2025)
- * @returns ISO date string (e.g., "2025-01-05") or null if invalid
- */
-export declare function formatPsqlDate(psqlDate: string): string | null;
-/**
- * Formats a time from PSQL HHMM format to HH:MM:SS
- * @param psqlTime Time in HHMM format (e.g., "1430" for 2:30 PM)
- * @returns Time string in HH:MM:SS format or null if invalid
- */
-export declare function formatPsqlTime(psqlTime: string): string | null;
-/**
- * Combines PSQL date and time into ISO datetime string
- * @param psqlDate Date in YYMMDD format
- * @param psqlTime Time in HHMM format
- * @returns ISO datetime string or null if invalid
- */
-export declare function combinePsqlDateTime(psqlDate: string, psqlTime: string): string | null;
-/**
- * Helper to check if a PSQL date is "empty" (000000 or blank)
- */
-export declare function isPsqlDateEmpty(psqlDate: string): boolean;
-/**
- * Clean and trim PSQL CHAR field (removes trailing spaces)
- * PSQL CHAR fields are fixed-width and padded with spaces
- */
-export declare function cleanPsqlCharField(value: string | null | undefined): string;
-//# sourceMappingURL=psql-helpers.d.ts.map

package/dist/services/psql-erp-service/psql-helpers.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"psql-helpers.d.ts","sourceRoot":"","sources":["../../../src/services/psql-erp-service/psql-helpers.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;;;GAIG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAuB9D;AAED;;;;GAIG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAY9D;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CACjC,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,MAAM,GACf,MAAM,GAAG,IAAI,CASf;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAEzD;AAED;;;GAGG;AACH,wBAAgB,kBAAkB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,MAAM,CAK3E"}

package/dist/services/psql-erp-service/psql-service.d.ts

@@ -1,49 +0,0 @@
-import odbc from "odbc";
-import { PsqlConfiguration } from "./configuration";
-import { ERPResponse } from "../../types/erp-types";
-type PagingParams = {
-    limit?: number;
-    offset?: number;
-};
-export declare class PsqlService {
-    private connection;
-    private config;
-    constructor(config: PsqlConfiguration);
-    dispose(): Promise<void>;
-    /**
-     * Build PSQL ODBC connection string
-     * CRITICAL: ServerName must use IP.PORT format (e.g., 10.4.0.11.1583)
-     */
-    private buildConnectionString;
-    /**
-     * Execute a query and return the results
-     * Interface matches SqlServerService for consistency
-     *
-     * @param query The SQL query to execute
-     * @param params Query parameters (currently unused for PSQL read operations)
-     * @param paging Optional paging parameters
-     * @returns The entities fetched from the database, along with paging information
-     */
-    executePreparedStatement(query: string, params?: Record<string, string>, paging?: PagingParams): Promise<ERPResponse | undefined>;
-    /**
-     * Opens a connection to PSQL database
-     * Caches the connection so that it can be reused.
-     * On failure to connect, throws
-     */
-    openConnection(): Promise<odbc.Connection>;
-    /**
-     * 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: any[]): Record<string, string>[];
-    /**
-     * Handle ODBC errors and provide meaningful messages
-     */
-    private handleOdbcError;
-    private closeConnection;
-}
-export {};
-//# sourceMappingURL=psql-service.d.ts.map

package/dist/services/psql-erp-service/psql-service.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"psql-service.d.ts","sourceRoot":"","sources":["../../../src/services/psql-erp-service/psql-service.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,EAAE,iBAAiB,EAAE,MAAM,iBAAiB,CAAC;AACpD,OAAO,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AAIpD,KAAK,YAAY,GAAG;IAClB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF,qBAAa,WAAW;IACtB,OAAO,CAAC,UAAU,CAAgC;IAClD,OAAO,CAAC,MAAM,CAAoB;gBAEtB,MAAM,EAAE,iBAAiB;IAI/B,OAAO;IAIb;;;OAGG;IACH,OAAO,CAAC,qBAAqB;IAgB7B;;;;;;;;OAQG;IACU,wBAAwB,CACnC,KAAK,EAAE,MAAM,EACb,MAAM,GAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAM,EACnC,MAAM,CAAC,EAAE,YAAY,GACpB,OAAO,CAAC,WAAW,GAAG,SAAS,CAAC;IAsDnC;;;;OAIG;IACG,cAAc,IAAI,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC;IA4BhD;;;;;;OAMG;WACW,kBAAkB,CAAC,SAAS,EAAE,GAAG,EAAE,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,EAAE;IAmB5E;;OAEG;IACH,OAAO,CAAC,eAAe;YA6BT,eAAe;CAa9B"}

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

@@ -1,9 +0,0 @@
-export interface PsqlConfiguration {
-  host: string;
-  port: string;
-  database: string;
-  username: string;
-  password: string;
-  connectionTimeout?: string;
-  requestTimeout?: string;
-}

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

@@ -1,28 +0,0 @@
-import { PsqlService } from "./psql-service";
-import { PsqlLaborTicketOperations } from "./internal/psql-labor-ticket-operations";
-
-/**
- * A class to manage interactions with PSQL (Pervasive) databases via ODBC
- */
-export { PsqlService };
-
-/**
- * Labor ticket operations for PSQL (Phase 2)
- */
-export { PsqlLaborTicketOperations };
-
-/**
- * Configuration interface for PSQL connections
- */
-export type { PsqlConfiguration } from "./configuration";
-
-/**
- * Helper functions for PSQL data formatting
- */
-export {
-  formatPsqlDate,
-  formatPsqlTime,
-  combinePsqlDateTime,
-  isPsqlDateEmpty,
-  cleanPsqlCharField,
-} from "./psql-helpers";

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

@@ -1,13 +0,0 @@
-import { z } from "zod";
-
-export const PsqlConfigSchema = z.object({
-  host: z.string().nonempty("Host is required."),
-  port: z.string().nonempty("Port is required."),
-  database: z.string().nonempty("Database name is required."),
-  username: z.string().nonempty("Username is required."),
-  password: z.string().nonempty("Password is required."),
-  connectionTimeout: z.string().optional().default("15000"),
-  requestTimeout: z.string().optional().default("15000"),
-});
-
-export type PsqlConfig = z.infer<typeof PsqlConfigSchema>;

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

@@ -1,58 +0,0 @@
-/**
- * PSQL Labor Ticket Operations
- *
- * Phase 2: This will handle INSERT/UPDATE/DELETE operations for labor tickets
- * Phase 1: Placeholder - throws errors if called
- */
-
-import { PsqlService } from "../psql-service";
-import { MMReceiveLaborTicket } from "../../../services/mm-api-service";
-import logger from "../../reporting-service/logger";
-
-export class PsqlLaborTicketOperations {
-  constructor(private service: PsqlService) {}
-
-  /**
-   * Create labor ticket in START_LABOR table
-   *
-   * Phase 2 Implementation Notes:
-   * - Will use prepared statements with parameter binding
-   * - Insert into START_LABOR table
-   * - Return GUID as erpUid
-   *
-   * @param laborTicket Labor ticket from MachineMetrics
-   * @returns Labor ticket and ERP unique ID
-   */
-  async createLaborTicket(
-    laborTicket: MMReceiveLaborTicket
-  ): Promise<{ laborTicket: MMReceiveLaborTicket; erpUid: string }> {
-    logger.warn(
-      "PsqlLaborTicketOperations.createLaborTicket not yet implemented (Phase 2)"
-    );
-    throw new Error(
-      "Labor ticket creation not implemented for PSQL. This is a Phase 2 feature."
-    );
-  }
-
-  /**
-   * Update labor ticket (move from START_LABOR to COMPLETED_LABOR)
-   *
-   * Phase 2 Implementation Notes:
-   * - Insert into COMPLETED_LABOR
-   * - Delete from START_LABOR
-   * - Should be done in a transaction
-   *
-   * @param laborTicket Labor ticket to update
-   * @returns Updated labor ticket
-   */
-  async updateLaborTicket(
-    laborTicket: MMReceiveLaborTicket
-  ): Promise<MMReceiveLaborTicket> {
-    logger.warn(
-      "PsqlLaborTicketOperations.updateLaborTicket not yet implemented (Phase 2)"
-    );
-    throw new Error(
-      "Labor ticket update not implemented for PSQL. This is a Phase 2 feature."
-    );
-  }
-}

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

@@ -1,17 +0,0 @@
-/**
- * PSQL-specific type definitions
- */
-
-export interface PsqlConnectionOptions {
-  connectionString: string;
-}
-
-export interface OdbcError {
-  state: string;
-  message: string;
-  code?: number;
-}
-
-export interface OdbcErrorResponse extends Error {
-  odbcErrors?: OdbcError[];
-}

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

@@ -1,90 +0,0 @@
-/**
- * Helper functions for PSQL/Pervasive database operations
- */
-
-/**
- * Formats a date from PSQL YYMMDD format to ISO date string
- * @param psqlDate Date in YYMMDD format (e.g., "250105" for Jan 5, 2025)
- * @returns ISO date string (e.g., "2025-01-05") or null if invalid
- */
-export function formatPsqlDate(psqlDate: string): string | null {
-  if (!psqlDate || psqlDate === "000000" || psqlDate.trim() === "") {
-    return null;
-  }
-
-  try {
-    const year = parseInt(psqlDate.substring(0, 2), 10);
-    const month = parseInt(psqlDate.substring(2, 4), 10);
-    const day = parseInt(psqlDate.substring(4, 6), 10);
-
-    // Convert 2-digit year to 4-digit (assuming 2000s)
-    const fullYear = year + 2000;
-
-    // Basic validation
-    if (month < 1 || month > 12 || day < 1 || day > 31) {
-      return null;
-    }
-
-    const date = new Date(fullYear, month - 1, day);
-    return date.toISOString().split("T")[0];
-  } catch (error) {
-    return null;
-  }
-}
-
-/**
- * Formats a time from PSQL HHMM format to HH:MM:SS
- * @param psqlTime Time in HHMM format (e.g., "1430" for 2:30 PM)
- * @returns Time string in HH:MM:SS format or null if invalid
- */
-export function formatPsqlTime(psqlTime: string): string | null {
-  if (!psqlTime || psqlTime.trim() === "") {
-    return null;
-  }
-
-  try {
-    const hours = psqlTime.substring(0, 2);
-    const minutes = psqlTime.substring(2, 4);
-    return `${hours}:${minutes}:00`;
-  } catch (error) {
-    return null;
-  }
-}
-
-/**
- * Combines PSQL date and time into ISO datetime string
- * @param psqlDate Date in YYMMDD format
- * @param psqlTime Time in HHMM format
- * @returns ISO datetime string or null if invalid
- */
-export function combinePsqlDateTime(
-  psqlDate: string,
-  psqlTime: string
-): string | null {
-  const date = formatPsqlDate(psqlDate);
-  const time = formatPsqlTime(psqlTime);
-
-  if (!date || !time) {
-    return null;
-  }
-
-  return `${date}T${time}`;
-}
-
-/**
- * Helper to check if a PSQL date is "empty" (000000 or blank)
- */
-export function isPsqlDateEmpty(psqlDate: string): boolean {
-  return !psqlDate || psqlDate === "000000" || psqlDate.trim() === "";
-}
-
-/**
- * Clean and trim PSQL CHAR field (removes trailing spaces)
- * PSQL CHAR fields are fixed-width and padded with spaces
- */
-export function cleanPsqlCharField(value: string | null | undefined): string {
-  if (value === null || value === undefined) {
-    return "";
-  }
-  return String(value).trim();
-}

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

@@ -1,215 +0,0 @@
-import odbc from "odbc";
-import { PsqlConfiguration } from "./configuration";
-import { ERPResponse } from "../../types/erp-types";
-import { OdbcErrorResponse } from "./internal/types/psql-types";
-import logger from "../reporting-service/logger";
-
-type PagingParams = {
-  limit?: number;
-  offset?: number;
-};
-
-export class PsqlService {
-  private connection: odbc.Connection | null = null;
-  private config: PsqlConfiguration;
-
-  constructor(config: PsqlConfiguration) {
-    this.config = config;
-  }
-
-  async dispose() {
-    await this.closeConnection();
-  }
-
-  /**
-   * Build PSQL ODBC connection string
-   * CRITICAL: ServerName must use IP.PORT format (e.g., 10.4.0.11.1583)
-   */
-  private buildConnectionString(): string {
-    // PSQL requires ServerName in format IP.PORT (not IP:PORT)
-    const serverName = `${this.config.host}.${this.config.port}`;
-
-    return (
-      [
-        "Driver={Pervasive ODBC Interface}",
-        `ServerName=${serverName}`,
-        `DBQ=${this.config.database}`,
-        `UID=${this.config.username}`,
-        `PWD=${this.config.password}`,
-        "AutoDoubleQuote=0",
-      ].join(";") + ";"
-    );
-  }
-
-  /**
-   * Execute a query and return the results
-   * Interface matches SqlServerService for consistency
-   *
-   * @param query The SQL query to execute
-   * @param params Query parameters (currently unused for PSQL read operations)
-   * @param paging Optional paging parameters
-   * @returns The entities fetched from the database, along with paging information
-   */
-  public async executePreparedStatement(
-    query: string,
-    params: Record<string, string> = {},
-    paging?: PagingParams
-  ): Promise<ERPResponse | undefined> {
-    const connection = await this.openConnection();
-
-    let records;
-    try {
-      // For Phase 1 (read-only), we execute queries directly
-      // Phase 2 will add proper parameter binding for INSERT/UPDATE/DELETE
-      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);
-    } catch (error) {
-      const errorInfo = error as OdbcErrorResponse;
-      logger.error("Error fetching data from PSQL", {
-        error: errorInfo.message,
-        odbcErrors: errorInfo.odbcErrors,
-      });
-
-      throw this.handleOdbcError(errorInfo);
-    }
-
-    const allRecords = PsqlService.recordsetToRecords(records);
-    const rowsFetched = allRecords.length;
-
-    // Apply paging if requested
-    const pagedData =
-      paging?.offset !== undefined || paging?.limit !== undefined
-        ? 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)
-            : undefined,
-        previousPage: paging?.offset
-          ? String(Math.max(0, (paging.offset || 0) - (paging.limit || 10)))
-          : undefined,
-      },
-    };
-  }
-
-  /**
-   * Opens a connection to PSQL database
-   * Caches the connection so that it can be reused.
-   * On failure to connect, throws
-   */
-  async openConnection(): Promise<odbc.Connection> {
-    // If we have a connection, reuse it
-    // Note: ODBC connections don't have a .connected property like SQL Server
-    // We'll keep it simple and reuse if not null
-    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 as OdbcErrorResponse);
-    }
-  }
-
-  /**
-   * 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
-   */
-  public static recordsetToRecords(recordset: any[]): Record<string, string>[] {
-    if (!Array.isArray(recordset)) {
-      return [];
-    }
-
-    const data: Record<string, string>[] = recordset.map((row) => {
-      const transformedRow: Record<string, string> = {};
-      Object.keys(row).forEach((key) => {
-        const value = row[key];
-        // Convert to string and trim (PSQL CHAR fields have trailing spaces)
-        transformedRow[key] =
-          value !== null && value !== undefined ? String(value).trim() : "";
-      });
-      return transformedRow;
-    });
-
-    return data;
-  }
-
-  /**
-   * Handle ODBC errors and provide meaningful messages
-   */
-  private handleOdbcError(error: OdbcErrorResponse): Error {
-    const odbcError = error.odbcErrors?.[0];
-    const errorCode = odbcError?.state;
-    const message = odbcError?.message || error.message;
-
-    switch (errorCode) {
-      case "08S01":
-        return new Error(
-          "PSQL connection failed. Check: " +
-            "1) PVSW environment variable set to /usr/local/psql/etc/pvsw.ini, " +
-            "2) Network connectivity to ports 1583/3351, " +
-            "3) ODBC configuration files in /usr/local/psql/etc/ and /etc/. " +
-            `Original error: ${message}`
-        );
-      case "28000":
-        return new Error(
-          `PSQL authentication failed. Check username/password. Original error: ${message}`
-        );
-      case "42000":
-        return new Error(`PSQL SQL syntax error. Original error: ${message}`);
-      case "42S02":
-        return new Error(
-          `PSQL table or view not found. Check table names in query. Original error: ${message}`
-        );
-      default:
-        return new Error(`PSQL error (${errorCode || "unknown"}): ${message}`);
-    }
-  }
-
-  private async closeConnection(): Promise<void> {
-    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;
-    }
-  }
-}