@ontemper/edi 1.2.0-beta.1 → 1.2.0-beta.2

package/README.md

@@ -1,11 +1,10 @@
 # Temper EDI Client
 
-A TypeScript EDI client with structured logging and distributed tracing capabilities, built for processing X12 (including HIPAA) and EDIFACT documents with comprehensive error handling and automatic span tracking.
+A TypeScript EDI client with structured logging and distributed tracing capabilities, built for processing X12 EDI documents with comprehensive error handling and automatic span tracking.
 
 ## Features
 
-- **X12 EDI Processing**: Complete X12 document parsing, validation, and generation — HIPAA guide models resolve automatically from the ST03 implementation reference
-- **EDIFACT Processing**: `fromEdifact` / `toEdifact` / `validateEdifact` / `acknowledgeEdifact` with the same error handling, tracing, and automatic control-reference stamping (UNB-0020 from the central counter service, UNH-0062 local sequence)
+- **X12 EDI Processing**: Complete X12 document parsing, validation, and generation
 - **Structured Logging**: Built-in integration with Temper Logger for comprehensive operation tracking
 - **Distributed Tracing**: Automatic span creation for all EDI operations with detailed payload information
 - **Type Safety**: Full TypeScript support with comprehensive type definitions
@@ -351,43 +350,83 @@ IEA*1*000001000`;
 run().catch(console.error);
 ```
 
-## Control Numbers
+## Control numbers
 
-The SDK provides functions for managing EDI control numbers (ISA13, GS06) via the centralized counter service:
+The SDK manages ISA13 and GS06 control numbers for each sender and receiver pair. `toX12()` automatically stamps
+empty or all-zero ISA13, GS06, and ST02 fields. It preserves valid non-zero values. ISA13 and GS06 use the
+centralized counters. ST02 is assigned `"0001"`.
+
+**By default, every environment draws from one shared sequence per trading partner. No setup is required.**
+Separating test traffic onto its own sequence is an optional, per-partner opt-in for agreements that demand it.
+
+### Separate test counters (optional)
+
+Skip this section unless a trading-partner agreement requires test traffic on an independent sequence. To opt in,
+configure the trading partner once, then seed and verify the lanes before sending test traffic. A seeded value is the
+last-used number, so the next allocation returns the seed plus one. To make the next ISA13 value 4601, seed 4600:
 
 ```typescript
-import { getNextControlNumber, getCurrentControlNumber, setControlNumber } from '@ontemper/edi';
+import {
+  getControlNumberPolicy,
+  getCurrentControlNumber,
+  setControlNumber,
+  setControlNumberPolicy,
+} from '@ontemper/edi';
 
-// Get next control number (atomic increment)
-const next = await getNextControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER');
+const senderId = 'ZZ:SENDER';
+const receiverId = 'ZZ:RECEIVER';
 
-// Peek at current value without incrementing
-const current = await getCurrentControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER');
+await setControlNumberPolicy(senderId, receiverId, { separateCounters: true });
 
-// Set a starting value before the counter has incremented (for migrations)
-await setControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER', 1000);
+// Seed the production lane explicitly before it starts.
+await setControlNumber('ISA13', senderId, receiverId, 4600, 9, { lane: 'production' });
+
+const policy = await getControlNumberPolicy(senderId, receiverId);
+const production = await getCurrentControlNumber('ISA13', senderId, receiverId, 9, { lane: 'production' });
+const test = await getCurrentControlNumber('ISA13', senderId, receiverId, 9, { lane: 'test' });
 ```
 
-In most cases you don't need these directly — `toX12()` automatically stamps any empty or all-zero
-ISA13/GS06/ST02 fields. ISA13 and GS06 use `getNextControlNumber`; ST02 is assigned `"0001"`. This includes the
-zero placeholders returned by acknowledgment generation.
-Once a counter has incremented, the API rejects later `setControlNumber` calls to avoid resetting active sequences.
-Retried `setControlNumber` calls with the same already-stored seed are treated as idempotent no-ops.
-The SDK surfaces that as a `ControlNumberError` with `code: "counter_already_started"`:
+`getControlNumberPolicy()` returns `configured` and `separateCounters`. After test traffic starts, you cannot turn
+separate counters off. `setControlNumberPolicy()` reports this as a `ControlNumberError` with the
+`counter_policy_locked` code.
+
+### Automatic environment routing
+
+The SDK reads `UNNBOUND_ENVIRONMENT` and sends it with counter operations only when its value is `sandbox`,
+`staging`, or `production`. The service selects the lane:
+
+| Policy | Production environment | Sandbox or staging |
+| --- | --- | --- |
+| No policy row | Production lane | Production lane |
+| `separateCounters: false` | Production lane | Production lane |
+| `separateCounters: true` | Production lane | Test lane |
+
+Older SDK versions do not send the environment, so their operations use the production lane. This is the historical
+sequence that all environments use when separation is off. ISA15
+(`UsageIndicator_15`) never selects a counter. Set ISA15 independently to the value required by the interchange.
+`toX12()` does not change it.
+
+### Manage counters directly
 
 ```typescript
-import { isControlNumberError, setControlNumber } from '@ontemper/edi';
+import { getCurrentControlNumber, getNextControlNumber, setControlNumber } from '@ontemper/edi';
 
-try {
-  await setControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER', 1000);
-} catch (error) {
-  if (isControlNumberError(error) && error.code === 'counter_already_started') {
-    // Counter is already active; do not reset it.
-  }
-}
+const next = await getNextControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER');
+const current = await getCurrentControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER');
+// The seed is last-used. To allocate 4601 next, seed 4600.
+await setControlNumber('ISA13', 'ZZ:SENDER', 'ZZ:RECEIVER', 4600, 9, { lane: 'production' });
 ```
 
-Requires `UNNBOUND_API_URL` environment variable (pre-configured in all workflow environments).
+Use the optional `{ lane: 'production' | 'test' }` argument on `getCurrentControlNumber()` and `setControlNumber()`
+to inspect or seed a specific lane. `getNextControlNumber()` always uses automatic environment routing and does not
+accept a lane.
+
+The seeded value is the last-used number, and the next allocation returns the seed plus one. For example, if you
+expect the next ISA13 value to be 4601, seed 4600. Set this value before the counter increments. The service treats a
+repeated seed with the same stored value as an idempotent operation. It rejects other updates after the counter
+starts with a `ControlNumberError` whose code is `counter_already_started`.
+
+These helpers require `UNNBOUND_API_URL`, which is preconfigured in workflow environments.
 
 ## Requirements
 

package/dist/control-numbers.d.ts

@@ -1,5 +1,27 @@
 /** @public */
-export type ControlNumberErrorCode = 'counter_already_started' | 'control_number_bad_request' | 'control_number_unauthorized' | 'control_number_unknown_error';
+export type ControlNumberCounterName = 'ISA13' | 'GS06';
+/** @public */
+export type ControlNumberLane = 'production' | 'test';
+/** @public */
+export interface ControlNumberLaneOptions {
+    readonly lane?: ControlNumberLane;
+}
+/** @public */
+export interface SetControlNumberPolicyOptions {
+    readonly separateCounters: boolean;
+}
+/** @public */
+export interface SetControlNumberPolicyResponse {
+    readonly tradingPartnerKey: string;
+    readonly separateCounters: boolean;
+}
+/** @public */
+export interface GetControlNumberPolicyResponse {
+    readonly configured: boolean;
+    readonly separateCounters: boolean;
+}
+/** @public */
+export type ControlNumberErrorCode = 'counter_already_started' | 'counter_policy_locked' | 'control_number_bad_request' | 'control_number_unauthorized' | 'control_number_unknown_error';
 interface ControlNumberErrorOptions extends ErrorOptions {
     code: ControlNumberErrorCode;
     message: string;
@@ -23,15 +45,19 @@ export declare const isControlNumberError: (error: unknown) => error is ControlN
  * @param maxDigits - Zero-pad to this width (default: 9 for ISA13)
  */
 /** @public */
-export declare function getNextControlNumber(counterName: string, senderId: string, receiverId: string, maxDigits?: number): Promise<string>;
+export declare function getNextControlNumber(counterName: ControlNumberCounterName, senderId: string, receiverId: string, maxDigits?: number): Promise<string>;
 /**
  * Get the current control number value without incrementing.
  */
 /** @public */
-export declare function getCurrentControlNumber(counterName: string, senderId: string, receiverId: string, maxDigits?: number): Promise<string>;
+export declare function getCurrentControlNumber(counterName: ControlNumberCounterName, senderId: string, receiverId: string, maxDigits?: number, options?: ControlNumberLaneOptions): Promise<string>;
 /**
  * Set a control number to a specific value before the counter starts (for migration/seeding).
  */
 /** @public */
-export declare function setControlNumber(counterName: string, senderId: string, receiverId: string, value: number, maxDigits?: number): Promise<string>;
+export declare function setControlNumber(counterName: ControlNumberCounterName, senderId: string, receiverId: string, value: number, maxDigits?: number, options?: ControlNumberLaneOptions): Promise<string>;
+/** @public */
+export declare function setControlNumberPolicy(senderId: string, receiverId: string, { separateCounters }: SetControlNumberPolicyOptions): Promise<SetControlNumberPolicyResponse>;
+/** @public */
+export declare function getControlNumberPolicy(senderId: string, receiverId: string): Promise<GetControlNumberPolicyResponse>;
 export {};

package/dist/control-numbers.js

@@ -37,6 +37,8 @@ exports.isControlNumberError = exports.ControlNumberError = void 0;
 exports.getNextControlNumber = getNextControlNumber;
 exports.getCurrentControlNumber = getCurrentControlNumber;
 exports.setControlNumber = setControlNumber;
+exports.setControlNumberPolicy = setControlNumberPolicy;
+exports.getControlNumberPolicy = getControlNumberPolicy;
 const axios_1 = __importStar(require("axios"));
 let _client = null;
 function getApiClient() {
@@ -78,10 +80,23 @@ exports.ControlNumberError = ControlNumberError;
 /** @public */
 const isControlNumberError = (error) => error instanceof ControlNumberError;
 exports.isControlNumberError = isControlNumberError;
-function unwrapControlNumberError(error) {
+const getEnvironment = () => {
+    const environment = process.env.UNNBOUND_ENVIRONMENT;
+    return environment === 'sandbox' || environment === 'staging' || environment === 'production'
+        ? environment
+        : undefined;
+};
+function unwrapControlNumberError(error, operation) {
     if (error instanceof ControlNumberError)
         throw error;
     if ((0, axios_1.isAxiosError)(error)) {
+        if (error.response?.status === 409 && operation === 'policy') {
+            throw new ControlNumberError({
+                code: 'counter_policy_locked',
+                message: 'Separate counters cannot be disabled after test traffic has started. Keep separate counters enabled or contact Temper support.',
+                cause: error,
+            });
+        }
         if (error.response?.status === 409) {
             throw new ControlNumberError({
                 code: 'counter_already_started',
@@ -92,7 +107,7 @@ function unwrapControlNumberError(error) {
         if (error.response?.status === 400) {
             throw new ControlNumberError({
                 code: 'control_number_bad_request',
-                message: 'Invalid EDI control number request. Counter name, trading partner IDs, and value are required.',
+                message: "Invalid EDI control number request. Use counter name 'ISA13' or 'GS06' with trading partner IDs and, for set, a value.",
                 cause: error,
             });
         }
@@ -110,6 +125,21 @@ function unwrapControlNumberError(error) {
         cause: error,
     });
 }
+async function requestControlNumber(operation, input) {
+    const environment = getEnvironment();
+    const request = { ...input, ...(environment ? { environment } : {}) };
+    try {
+        const response = operation === 'count'
+            ? await getApiClient().get(`/api/v2/internal/edi/control-numbers/${operation}`, {
+                params: request,
+            })
+            : await getApiClient().post(`/api/v2/internal/edi/control-numbers/${operation}`, request);
+        return response.data;
+    }
+    catch (error) {
+        return unwrapControlNumberError(error, operation);
+    }
+}
 /**
  * Get the next control number for an EDI counter.
  *
@@ -123,7 +153,7 @@ function unwrapControlNumberError(error) {
  */
 /** @public */
 async function getNextControlNumber(counterName, senderId, receiverId, maxDigits = 9) {
-    const { data } = await getApiClient().post('/api/v2/internal/edi/control-numbers/increment', {
+    const data = await requestControlNumber('increment', {
         tradingPartnerKey: `${senderId}:${receiverId}`,
         counterName,
     });
@@ -133,12 +163,11 @@ async function getNextControlNumber(counterName, senderId, receiverId, maxDigits
  * Get the current control number value without incrementing.
  */
 /** @public */
-async function getCurrentControlNumber(counterName, senderId, receiverId, maxDigits = 9) {
-    const { data } = await getApiClient().get('/api/v2/internal/edi/control-numbers/count', {
-        params: {
-            tradingPartnerKey: `${senderId}:${receiverId}`,
-            counterName,
-        },
+async function getCurrentControlNumber(counterName, senderId, receiverId, maxDigits = 9, options) {
+    const data = await requestControlNumber('count', {
+        tradingPartnerKey: `${senderId}:${receiverId}`,
+        counterName,
+        ...(options?.lane ? { lane: options.lane } : {}),
     });
     return String(data.value).padStart(maxDigits, '0');
 }
@@ -146,13 +175,40 @@ async function getCurrentControlNumber(counterName, senderId, receiverId, maxDig
  * Set a control number to a specific value before the counter starts (for migration/seeding).
  */
 /** @public */
-async function setControlNumber(counterName, senderId, receiverId, value, maxDigits = 9) {
-    const { data } = await getApiClient()
-        .post('/api/v2/internal/edi/control-numbers/set', {
+async function setControlNumber(counterName, senderId, receiverId, value, maxDigits = 9, options) {
+    const data = await requestControlNumber('set', {
         tradingPartnerKey: `${senderId}:${receiverId}`,
         counterName,
         value,
-    })
-        .catch(unwrapControlNumberError);
+        ...(options?.lane ? { lane: options.lane } : {}),
+    });
     return String(data.value).padStart(maxDigits, '0');
 }
+/** @public */
+async function setControlNumberPolicy(senderId, receiverId, { separateCounters }) {
+    try {
+        const response = await getApiClient().post('/api/v2/internal/edi/control-numbers/policy', {
+            tradingPartnerKey: `${senderId}:${receiverId}`,
+            separateCounters,
+        });
+        return response.data;
+    }
+    catch (error) {
+        return unwrapControlNumberError(error, 'policy');
+    }
+}
+/** @public */
+async function getControlNumberPolicy(senderId, receiverId) {
+    try {
+        const response = await getApiClient().get('/api/v2/internal/edi/control-numbers/policy', {
+            params: { tradingPartnerKey: `${senderId}:${receiverId}` },
+        });
+        return {
+            configured: response.data.configured,
+            separateCounters: response.data.separateCounters,
+        };
+    }
+    catch (error) {
+        return unwrapControlNumberError(error, 'policy');
+    }
+}

package/dist/index.d.ts

@@ -1,6 +1,6 @@
-import { type EdifactInterchange, type X12Interchange } from './edination-client';
-export type { ControlNumberErrorCode } from './control-numbers';
-export { ControlNumberError, getCurrentControlNumber, getNextControlNumber, isControlNumberError, setControlNumber, } from './control-numbers';
+import type { X12Interchange } from './edination-client';
+export type { ControlNumberCounterName, ControlNumberErrorCode, ControlNumberLane, ControlNumberLaneOptions, GetControlNumberPolicyResponse, SetControlNumberPolicyOptions, SetControlNumberPolicyResponse, } from './control-numbers';
+export { ControlNumberError, getControlNumberPolicy, getCurrentControlNumber, getNextControlNumber, isControlNumberError, setControlNumber, setControlNumberPolicy, } from './control-numbers';
 export * from './edination-client/model';
 interface UnnboundErrorOptions<C extends string = string> extends ErrorOptions {
     message: string;
@@ -35,21 +35,8 @@ export interface ValidateX12Options {
 export interface AcknolwedgeX12Options {
     input: X12Interchange;
 }
-export interface FromEdifactOptions {
-    input: unknown;
-}
-export interface ToEdifactOptions {
-    input: EdifactInterchange;
-}
-export interface ValidateEdifactOptions {
-    input: EdifactInterchange;
-}
-export interface AcknowledgeEdifactOptions {
-    input: EdifactInterchange;
-}
 export declare class TemperEdiClient {
     private X12;
-    private Edifact;
     constructor();
     private unwrap;
     private unwrapError;
@@ -64,19 +51,4 @@ export declare class TemperEdiClient {
     toX12({ input }: ToX12Options): Promise<unknown>;
     validateX12({ input }: ValidateX12Options): Promise<import("./edination-client").OperationResult>;
     acknowledgeX12({ input }: AcknolwedgeX12Options): Promise<X12Interchange[]>;
-    fromEdifact({ input }: FromEdifactOptions): Promise<EdifactInterchange[]>;
-    /**
-     * Stamp UNB-0020/UNH-0062 control references on the interchange if not already set.
-     *
-     * UNB-0020 (interchange control reference) comes from the same Restate-backed
-     * per-trading-partner counter service X12 uses, under the platform counter name
-     * `UNB0020` — unpadded, since EDIFACT 0020 is an..14 with no fixed width. The
-     * UNZ trailer echo is set to match. UNH-0062 (message reference number) only
-     * needs uniqueness within the interchange, so it is a local 1..n sequence with
-     * no service call, echoed into each UNT — the EDIFACT sibling of ST02="0001".
-     */
-    private stampEdifactControlNumbers;
-    toEdifact({ input }: ToEdifactOptions): Promise<unknown>;
-    validateEdifact({ input }: ValidateEdifactOptions): Promise<import("./edination-client").OperationResult>;
-    acknowledgeEdifact({ input }: AcknowledgeEdifactOptions): Promise<EdifactInterchange[]>;
 }

package/dist/index.js

@@ -36,17 +36,20 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.TemperEdiClient = exports.isEdiInfrastructureError = exports.EdiInfrastructureError = exports.isTemperEdiClientError = exports.TemperEdiClientError = exports.setControlNumber = exports.isControlNumberError = exports.getNextControlNumber = exports.getCurrentControlNumber = exports.ControlNumberError = void 0;
+exports.TemperEdiClient = exports.isEdiInfrastructureError = exports.EdiInfrastructureError = exports.isTemperEdiClientError = exports.TemperEdiClientError = exports.setControlNumberPolicy = exports.setControlNumber = exports.isControlNumberError = exports.getNextControlNumber = exports.getCurrentControlNumber = exports.getControlNumberPolicy = exports.ControlNumberError = void 0;
 const axios_1 = __importStar(require("axios"));
 const unnbound_logger_sdk_1 = require("unnbound-logger-sdk");
 const internal_1 = require("unnbound-logger-sdk/dist/internal");
+const control_numbers_1 = require("./control-numbers");
 const edination_client_1 = require("./edination-client");
-var control_numbers_1 = require("./control-numbers");
-Object.defineProperty(exports, "ControlNumberError", { enumerable: true, get: function () { return control_numbers_1.ControlNumberError; } });
-Object.defineProperty(exports, "getCurrentControlNumber", { enumerable: true, get: function () { return control_numbers_1.getCurrentControlNumber; } });
-Object.defineProperty(exports, "getNextControlNumber", { enumerable: true, get: function () { return control_numbers_1.getNextControlNumber; } });
-Object.defineProperty(exports, "isControlNumberError", { enumerable: true, get: function () { return control_numbers_1.isControlNumberError; } });
-Object.defineProperty(exports, "setControlNumber", { enumerable: true, get: function () { return control_numbers_1.setControlNumber; } });
+var control_numbers_2 = require("./control-numbers");
+Object.defineProperty(exports, "ControlNumberError", { enumerable: true, get: function () { return control_numbers_2.ControlNumberError; } });
+Object.defineProperty(exports, "getControlNumberPolicy", { enumerable: true, get: function () { return control_numbers_2.getControlNumberPolicy; } });
+Object.defineProperty(exports, "getCurrentControlNumber", { enumerable: true, get: function () { return control_numbers_2.getCurrentControlNumber; } });
+Object.defineProperty(exports, "getNextControlNumber", { enumerable: true, get: function () { return control_numbers_2.getNextControlNumber; } });
+Object.defineProperty(exports, "isControlNumberError", { enumerable: true, get: function () { return control_numbers_2.isControlNumberError; } });
+Object.defineProperty(exports, "setControlNumber", { enumerable: true, get: function () { return control_numbers_2.setControlNumber; } });
+Object.defineProperty(exports, "setControlNumberPolicy", { enumerable: true, get: function () { return control_numbers_2.setControlNumberPolicy; } });
 __exportStar(require("./edination-client/model"), exports);
 class UnnboundError extends Error {
     code;
@@ -79,7 +82,7 @@ exports.EdiInfrastructureError = EdiInfrastructureError;
 const isEdiInfrastructureError = (error) => error instanceof EdiInfrastructureError;
 exports.isEdiInfrastructureError = isEdiInfrastructureError;
 const buildEdiPayload = (edi) => ({ type: 'edi', edi });
-const buildEdiTransactionPayload = (operation, format, transaction) => buildEdiPayload({ operation, type: format, transaction });
+const buildEdiX12Payload = (operation, x12) => buildEdiPayload({ operation, type: 'x12', x12 });
 const needsControlNumber = (value) => {
     const normalized = value?.trim();
     return !normalized || /^0+$/.test(normalized);
@@ -87,7 +90,6 @@ const needsControlNumber = (value) => {
 const ediAxios = (0, unnbound_logger_sdk_1.traceAxios)(axios_1.default.create(), { getPayload: internal_1.internal });
 class TemperEdiClient {
     X12;
-    Edifact;
     constructor() {
         const apiKey = process.env.UNNBOUND_EDI_API_KEY;
         // When UNNBOUND_EDI_BASE_URL is set, use self-hosted EdiFabric InHouse API
@@ -100,7 +102,6 @@ class TemperEdiClient {
         // but the OpenAPI client requires a non-empty value — use a placeholder.
         const config = new edination_client_1.Configuration({ apiKey: apiKey ?? 'self-hosted', basePath });
         this.X12 = new edination_client_1.X12Api(config, undefined, ediAxios);
-        this.Edifact = new edination_client_1.EdifactApi(config, undefined, ediAxios);
     }
     unwrap(response) {
         return response.data;
@@ -147,7 +148,7 @@ class TemperEdiClient {
             return this.X12.x12ReadPost({ body: input })
                 .then(this.unwrap.bind(this))
                 .catch((error) => this.unwrapError(error, 'edi_read_error'));
-        }, (o) => buildEdiTransactionPayload('fromX12', 'x12', { input, output: o?.result }));
+        }, (o) => buildEdiX12Payload('fromX12', { input, output: o?.result }));
     }
     /**
      * Stamp ISA13/GS06/ST02 control numbers on the interchange if not already set.
@@ -156,7 +157,6 @@ class TemperEdiClient {
      * Zero-filled values from acknowledgment generation are placeholders, not manual overrides.
      */
     async stampControlNumbers(input) {
-        const { getNextControlNumber } = await import('./control-numbers.js');
         const isa = input.ISA;
         const senderQual = (isa.SenderIDQualifier_5 || '').trim();
         const senderId = (isa.InterchangeSenderID_6 || '').trim();
@@ -164,20 +164,18 @@ class TemperEdiClient {
         const receiverId = (isa.InterchangeReceiverID_8 || '').trim();
         const senderKey = `${senderQual}:${senderId}`;
         const receiverKey = `${receiverQual}:${receiverId}`;
-        // Phase 1: Fetch all needed control numbers (no mutations yet).
-        // If any fetch fails, no fields are mutated — avoids partial stamping.
+        // Allocate before mutating so a failed request cannot leave partial stamps.
         const isa13 = needsControlNumber(isa.InterchangeControlNumber_13)
-            ? await getNextControlNumber('ISA13', senderKey, receiverKey, 9)
+            ? await (0, control_numbers_1.getNextControlNumber)('ISA13', senderKey, receiverKey, 9)
             : null;
         const gs06Values = [];
         const groups = input.Groups ?? [];
         for (let i = 0; i < groups.length; i++) {
             if (needsControlNumber(groups[i].GS.GroupControlNumber_6)) {
-                const value = await getNextControlNumber('GS06', senderKey, receiverKey, 9);
+                const value = await (0, control_numbers_1.getNextControlNumber)('GS06', senderKey, receiverKey, 9);
                 gs06Values.push({ index: i, value });
             }
         }
-        // Phase 2: Apply all mutations (only reached if all fetches succeeded).
         if (isa13) {
             isa.InterchangeControlNumber_13 = isa13;
             if (input.IEATrailers?.length) {
@@ -191,7 +189,6 @@ class TemperEdiClient {
                 group.GETrailers[0].GroupControlNumber_2 = value;
             }
         }
-        // ST02 — always "0001" (pure assignment, no external calls)
         for (const group of groups) {
             for (const tx of group.Transactions) {
                 if (needsControlNumber(tx?.ST?.TransactionSetControlNumber_02)) {
@@ -204,20 +201,18 @@ class TemperEdiClient {
         }
     }
     async toX12({ input }) {
-        // Auto-stamp control numbers before writing X12
         try {
             await this.stampControlNumbers(input);
         }
         catch (error) {
             // Log but don't fail — workflows can still set control numbers manually
-            const { logger } = await import('unnbound-logger-sdk');
-            logger.warn({ err: error }, '[EDI SDK] Failed to auto-stamp control numbers, proceeding without');
+            unnbound_logger_sdk_1.logger.warn({ err: error }, '[EDI SDK] Failed to auto-stamp control numbers, proceeding without');
         }
         return (0, unnbound_logger_sdk_1.startSpan)('JSON to X12', () => {
             return this.X12.x12WritePost({ x12Interchange: input })
                 .then(this.unwrap.bind(this))
                 .catch((error) => this.unwrapError(error, 'edi_write_error'));
-        }, (o) => buildEdiTransactionPayload('toX12', 'x12', { input, output: o?.result }));
+        }, (o) => buildEdiX12Payload('toX12', { input, output: o?.result }));
     }
     validateX12({ input }) {
         return (0, unnbound_logger_sdk_1.startSpan)('Validate X12', () => {
@@ -225,7 +220,7 @@ class TemperEdiClient {
                 .then(this.unwrap.bind(this))
                 .catch((error) => this.unwrapError(error, 'edi_validate_error'));
         }, (o) => ({
-            ...buildEdiTransactionPayload('validateX12', 'x12', { input, output: o?.result }),
+            ...buildEdiX12Payload('validateX12', { input, output: o?.result }),
             ...(o?.result?.Status === 'error' && { level: 'warn' }),
         }));
     }
@@ -234,87 +229,7 @@ class TemperEdiClient {
             return this.X12.x12AckPost({ x12Interchange: input })
                 .then(this.unwrap.bind(this))
                 .catch((error) => this.unwrapError(error, 'edi_acknowledge_error'));
-        }, (o) => buildEdiTransactionPayload('acknowledgeX12', 'x12', { input, output: o?.result }));
-    }
-    fromEdifact({ input }) {
-        return (0, unnbound_logger_sdk_1.startSpan)('EDIFACT to JSON', () => {
-            return this.Edifact.edifactReadPost({ body: input })
-                .then(this.unwrap.bind(this))
-                .catch((error) => this.unwrapError(error, 'edi_read_error'));
-        }, (o) => buildEdiTransactionPayload('fromEdifact', 'edifact', { input, output: o?.result }));
-    }
-    /**
-     * Stamp UNB-0020/UNH-0062 control references on the interchange if not already set.
-     *
-     * UNB-0020 (interchange control reference) comes from the same Restate-backed
-     * per-trading-partner counter service X12 uses, under the platform counter name
-     * `UNB0020` — unpadded, since EDIFACT 0020 is an..14 with no fixed width. The
-     * UNZ trailer echo is set to match. UNH-0062 (message reference number) only
-     * needs uniqueness within the interchange, so it is a local 1..n sequence with
-     * no service call, echoed into each UNT — the EDIFACT sibling of ST02="0001".
-     */
-    async stampEdifactControlNumbers(input) {
-        const { getNextControlNumber } = await import('./control-numbers.js');
-        const sender = input.UNB?.INTERCHANGESENDER_2;
-        const recipient = input.UNB?.INTERCHANGERECIPIENT_3;
-        const senderKey = `${(sender?.IdentificationCodeQualifier_2 || '').trim()}:${(sender?.InterchangeSenderIdentification_1 || '').trim()}`;
-        const receiverKey = `${(recipient?.IdentificationCodeQualifier_2 || '').trim()}:${(recipient?.InterchangeRecipientIdentification_1 || '').trim()}`;
-        // Fetch before mutating — if the fetch fails, no fields change.
-        const unb0020 = needsControlNumber(input.UNB?.InterchangeControlReference_5)
-            ? await getNextControlNumber('UNB0020', senderKey, receiverKey, 0)
-            : null;
-        if (unb0020) {
-            input.UNB.InterchangeControlReference_5 = unb0020;
-            if (input.UNZTrailers?.length) {
-                input.UNZTrailers[0].InterchangeControlReference_2 = unb0020;
-            }
-        }
-        // UNH-0062 — local sequence across the interchange's messages (no external calls)
-        let messageReference = 0;
-        for (const group of input.Groups ?? []) {
-            for (const message of group.Transactions ?? []) {
-                messageReference += 1;
-                if (needsControlNumber(message?.UNH?.MessageReferenceNumber_01)) {
-                    if (message?.UNH)
-                        message.UNH.MessageReferenceNumber_01 = String(messageReference);
-                    if (message?.UNT)
-                        message.UNT.MessageReferenceNumber_02 = String(messageReference);
-                }
-            }
-        }
-    }
-    async toEdifact({ input }) {
-        // Auto-stamp control references before writing EDIFACT
-        try {
-            await this.stampEdifactControlNumbers(input);
-        }
-        catch (error) {
-            // Log but don't fail — workflows can still set control references manually
-            const { logger } = await import('unnbound-logger-sdk');
-            logger.warn({ err: error }, '[EDI SDK] Failed to auto-stamp control references, proceeding without');
-        }
-        return (0, unnbound_logger_sdk_1.startSpan)('JSON to EDIFACT', () => {
-            return this.Edifact.edifactWritePost({ edifactInterchange: input })
-                .then(this.unwrap.bind(this))
-                .catch((error) => this.unwrapError(error, 'edi_write_error'));
-        }, (o) => buildEdiTransactionPayload('toEdifact', 'edifact', { input, output: o?.result }));
-    }
-    validateEdifact({ input }) {
-        return (0, unnbound_logger_sdk_1.startSpan)('Validate EDIFACT', () => {
-            return this.Edifact.edifactValidatePost({ edifactInterchange: input })
-                .then(this.unwrap.bind(this))
-                .catch((error) => this.unwrapError(error, 'edi_validate_error'));
-        }, (o) => ({
-            ...buildEdiTransactionPayload('validateEdifact', 'edifact', { input, output: o?.result }),
-            ...(o?.result?.Status === 'error' && { level: 'warn' }),
-        }));
-    }
-    acknowledgeEdifact({ input }) {
-        return (0, unnbound_logger_sdk_1.startSpan)('Acknowledge EDIFACT', () => {
-            return this.Edifact.edifactAckPost({ edifactInterchange: input })
-                .then(this.unwrap.bind(this))
-                .catch((error) => this.unwrapError(error, 'edi_acknowledge_error'));
-        }, (o) => buildEdiTransactionPayload('acknowledgeEdifact', 'edifact', { input, output: o?.result }));
+        }, (o) => buildEdiX12Payload('acknowledgeX12', { input, output: o?.result }));
     }
 }
 exports.TemperEdiClient = TemperEdiClient;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ontemper/edi",
   "description": "An EDI client with structured logging.",
-  "version": "1.2.0-beta.1",
+  "version": "1.2.0-beta.2",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "author": "Unnbound Team",
@@ -15,8 +15,8 @@
     "url": "https://github.com/unnbounddev/unnbound-sdks/issues"
   },
   "dependencies": {
-    "axios": "1.16.0",
-    "unnbound-logger-sdk": "3.2.0-beta.1"
+    "axios": "1.18.0",
+    "unnbound-logger-sdk": "3.0.37"
   },
   "devDependencies": {
     "@types/jest": "^29.5.12",