@ontemper/edi 1.2.0-beta.4 → 1.2.0-beta.6

package/README.md

@@ -362,32 +362,26 @@ sequence. Sandbox and staging allocations do not consume production control numb
 
 ### Automatic environment routing
 
-Temper injects `TEMPER_ENVIRONMENT` into workflows. The SDK uses `UNNBOUND_ENVIRONMENT` only when
-`TEMPER_ENVIRONMENT` is absent. The value must be `sandbox`, `staging`, or `production`. A missing or invalid value
-causes a `ControlNumberError` with the `control_number_environment_missing` code before any HTTP request.
+Current workflow deployments inject `UNNBOUND_ENVIRONMENT`. The SDK also accepts `TEMPER_ENVIRONMENT` as the
+future canonical name and prefers it when present. The value must be `sandbox`, `staging`, or `production`. A missing
+or invalid value causes a `ControlNumberError` with the `control_number_environment_missing` code before any HTTP request.
 
 `getNextControlNumber()`, `getCurrentControlNumber()`, and `toX12()` use this environment. They do not accept an
 environment override. ISA15 (`UsageIndicator_15`) does not select a counter, and `toX12()` does not change it.
 
 ```typescript
-import { getCurrentControlNumber, getNextControlNumber, setControlNumber } from '@ontemper/edi';
+import { getCurrentControlNumber, getNextControlNumber } from '@ontemper/edi';
 
 const senderId = 'ZZ:SENDER';
 const receiverId = 'ZZ:RECEIVER';
 
 const next = await getNextControlNumber('ISA13', senderId, receiverId);
 const current = await getCurrentControlNumber('ISA13', senderId, receiverId);
-
-// Legacy production migration: store 4599 as the last-used value.
-await setControlNumber('ISA13', senderId, receiverId, 4599);
 ```
 
-`setControlNumber()` is the legacy production setter. Its value is the last-used number, so setting `4599` makes the
-next allocation `4600`. It uses the flat production route and does not use the environment variables.
-
-Use the Builder agent `set_edi_control_number` tool for per-environment seeding. The tool value is the next number.
-For example, seeding `4600` makes the next allocation `4600`. Seeding is forward-only. Re-seeding the same value is
-a no-op.
+The SDK has no seeding helper. Seed a counter through the Builder agent `set_edi_control_number` tool, which asks
+the user to approve each seed. The tool value is the next number: seeding `4600` makes the next allocation `4600`.
+Seeding is forward-only. Re-seeding the same value is a no-op.
 
 These helpers require `UNNBOUND_API_URL`. Temper configures it in workflow environments.
 
@@ -397,7 +391,7 @@ These helpers require `UNNBOUND_API_URL`. Temper configures it in workflow envir
 - TypeScript (for TypeScript projects)
 - The EDI connection env — `UNNBOUND_EDI_BASE_URL`, plus `UNNBOUND_EDI_API_KEY` where the path needs a credential — is platform-injected per environment in Temper workflows; nothing to configure. Supply a key yourself only for the standalone legacy EDINation-cloud path.
 - `UNNBOUND_API_URL` (for control number auto-stamping, pre-configured in workflow environments)
-- `TEMPER_ENVIRONMENT` for counter routing, with `UNNBOUND_ENVIRONMENT` as the legacy fallback
+- `UNNBOUND_ENVIRONMENT` for current counter routing; `TEMPER_ENVIRONMENT` is the preferred future name
 
 ## Dependencies
 

package/dist/control-numbers.d.ts

@@ -3,7 +3,7 @@ export type TemperEnvironment = 'sandbox' | 'staging' | 'production';
 /** @public */
 export type ControlNumberCounterName = 'ISA13' | 'GS06';
 /** @public */
-export type ControlNumberErrorCode = 'counter_already_started' | 'control_number_environment_missing' | 'control_number_bad_request' | 'control_number_unauthorized' | 'control_number_unknown_error';
+export type ControlNumberErrorCode = 'control_number_environment_missing' | 'control_number_bad_request' | 'control_number_unauthorized' | 'control_number_unknown_error';
 interface ControlNumberErrorOptions extends ErrorOptions {
     code: ControlNumberErrorCode;
     message: string;
@@ -33,11 +33,4 @@ export declare function getNextControlNumber(counterName: ControlNumberCounterNa
  */
 /** @public */
 export declare function getCurrentControlNumber(counterName: ControlNumberCounterName, senderId: string, receiverId: string, maxDigits?: number): Promise<string>;
-/**
- * Set the last-used production control number through the legacy endpoint.
- *
- * Use the Builder agent `set_edi_control_number` tool for per-environment seeding.
- */
-/** @public */
-export declare function setControlNumber(counterName: ControlNumberCounterName, senderId: string, receiverId: string, value: number, maxDigits?: number): Promise<string>;
 export {};

package/dist/control-numbers.js

@@ -36,7 +36,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.isControlNumberError = exports.ControlNumberError = void 0;
 exports.getNextControlNumber = getNextControlNumber;
 exports.getCurrentControlNumber = getCurrentControlNumber;
-exports.setControlNumber = setControlNumber;
 const axios_1 = __importStar(require("axios"));
 let _client = null;
 function getApiClient() {
@@ -88,24 +87,17 @@ const requireEnvironment = () => {
         message: "TEMPER_ENVIRONMENT or UNNBOUND_ENVIRONMENT must be set to 'sandbox', 'staging', or 'production' because automatic control-number routing needs it.",
     });
 };
-function unwrapControlNumberError(error, environmentScoped = false) {
+function unwrapControlNumberError(error) {
     if (error instanceof ControlNumberError)
         throw error;
     if ((0, axios_1.isAxiosError)(error)) {
-        if (environmentScoped && error.response?.status === 404) {
+        if (error.response?.status === 404) {
             throw new ControlNumberError({
                 code: 'control_number_unknown_error',
                 message: 'The Temper API does not support environment-scoped EDI control numbers yet.',
                 cause: error,
             });
         }
-        if (error.response?.status === 409) {
-            throw new ControlNumberError({
-                code: 'counter_already_started',
-                message: 'Cannot set EDI control number after counter has started. Set the starting value before calling getNextControlNumber or toX12 for this trading partner.',
-                cause: error,
-            });
-        }
         if (error.response?.status === 400) {
             throw new ControlNumberError({
                 code: 'control_number_bad_request',
@@ -127,19 +119,6 @@ function unwrapControlNumberError(error, environmentScoped = false) {
         cause: error,
     });
 }
-async function requestControlNumber(operation, input) {
-    const environment = requireEnvironment();
-    const path = `/api/v2/internal/edi/control-numbers/${environment}/${operation}`;
-    try {
-        const response = operation === 'count'
-            ? await getApiClient().get(path, { params: input })
-            : await getApiClient().post(path, input);
-        return response.data;
-    }
-    catch (error) {
-        return unwrapControlNumberError(error, true);
-    }
-}
 /**
  * Get the next control number for an EDI counter.
  *
@@ -153,35 +132,30 @@ async function requestControlNumber(operation, input) {
  */
 /** @public */
 async function getNextControlNumber(counterName, senderId, receiverId, maxDigits = 9) {
-    const data = await requestControlNumber('increment', {
-        tradingPartnerKey: `${senderId}:${receiverId}`,
-        counterName,
-    });
-    return String(data.value).padStart(maxDigits, '0');
+    const environment = requireEnvironment();
+    try {
+        const { data } = await getApiClient().post(`/api/v2/internal/edi/control-numbers/${environment}/increment`, {
+            tradingPartnerKey: `${senderId}:${receiverId}`,
+            counterName,
+        });
+        return String(data.value).padStart(maxDigits, '0');
+    }
+    catch (error) {
+        return unwrapControlNumberError(error);
+    }
 }
 /**
  * Get the current control number value without incrementing.
  */
 /** @public */
 async function getCurrentControlNumber(counterName, senderId, receiverId, maxDigits = 9) {
-    const data = await requestControlNumber('count', {
-        tradingPartnerKey: `${senderId}:${receiverId}`,
-        counterName,
-    });
-    return String(data.value).padStart(maxDigits, '0');
-}
-/**
- * Set the last-used production control number through the legacy endpoint.
- *
- * Use the Builder agent `set_edi_control_number` tool for per-environment seeding.
- */
-/** @public */
-async function setControlNumber(counterName, senderId, receiverId, value, maxDigits = 9) {
+    const environment = requireEnvironment();
     try {
-        const { data } = await getApiClient().post('/api/v2/internal/edi/control-numbers/set', {
-            tradingPartnerKey: `${senderId}:${receiverId}`,
-            counterName,
-            value,
+        const { data } = await getApiClient().get(`/api/v2/internal/edi/control-numbers/${environment}/count`, {
+            params: {
+                tradingPartnerKey: `${senderId}:${receiverId}`,
+                counterName,
+            },
         });
         return String(data.value).padStart(maxDigits, '0');
     }

package/dist/index.d.ts

@@ -1,6 +1,6 @@
 import { type X12Interchange } from './edination-client';
 export type { ControlNumberErrorCode } from './control-numbers';
-export { ControlNumberError, getCurrentControlNumber, getNextControlNumber, isControlNumberError, setControlNumber, } from './control-numbers';
+export { ControlNumberError, getCurrentControlNumber, getNextControlNumber, isControlNumberError, } from './control-numbers';
 export * from './edination-client/model';
 interface UnnboundErrorOptions<C extends string = string> extends ErrorOptions {
     message: string;
@@ -41,12 +41,7 @@ export declare class TemperEdiClient {
     private unwrap;
     private unwrapError;
     fromX12({ input }: FromX12Options): Promise<X12Interchange[]>;
-    /**
-     * Stamp ISA13/GS06/ST02 control numbers on the interchange if not already set.
-     * Uses the centralized per-environment service for atomic partner sequencing.
-     * ST02 is always "0001" (one transaction per file per ticket).
-     * Zero-filled values from acknowledgment generation are placeholders, not manual overrides.
-     */
+    /** Allocate every remote number before mutating the interchange. */
     private stampControlNumbers;
     toX12({ input }: ToX12Options): Promise<File>;
     validateX12({ input }: ValidateX12Options): Promise<import("./edination-client").OperationResult>;

package/dist/index.js

@@ -36,7 +36,7 @@ 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.isControlNumberError = exports.getNextControlNumber = exports.getCurrentControlNumber = 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");
@@ -47,7 +47,6 @@ Object.defineProperty(exports, "ControlNumberError", { enumerable: true, get: fu
 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; } });
 __exportStar(require("./edination-client/model"), exports);
 class UnnboundError extends Error {
     code;
@@ -150,30 +149,24 @@ class TemperEdiClient {
                 .catch((error) => this.unwrapError(error, 'edi_read_error'));
         }, (o) => buildEdiX12Payload('fromX12', { input, output: o?.result }));
     }
-    /**
-     * Stamp ISA13/GS06/ST02 control numbers on the interchange if not already set.
-     * Uses the centralized per-environment service for atomic partner sequencing.
-     * ST02 is always "0001" (one transaction per file per ticket).
-     * Zero-filled values from acknowledgment generation are placeholders, not manual overrides.
-     */
+    /** Allocate every remote number before mutating the interchange. */
     async stampControlNumbers(input) {
         const isa = input.ISA;
-        const senderQual = (isa.SenderIDQualifier_5 || '').trim();
-        const senderId = (isa.InterchangeSenderID_6 || '').trim();
-        const receiverQual = (isa.ReceiverIDQualifier_7 || '').trim();
-        const receiverId = (isa.InterchangeReceiverID_8 || '').trim();
+        const senderQual = (isa.SenderIDQualifier_5 ?? '').trim();
+        const senderId = (isa.InterchangeSenderID_6 ?? '').trim();
+        const receiverQual = (isa.ReceiverIDQualifier_7 ?? '').trim();
+        const receiverId = (isa.InterchangeReceiverID_8 ?? '').trim();
         const senderKey = `${senderQual}:${senderId}`;
         const receiverKey = `${receiverQual}:${receiverId}`;
-        // Allocate before mutating so a failed request cannot leave partial stamps.
         const isa13 = needsControlNumber(isa.InterchangeControlNumber_13)
             ? await (0, control_numbers_1.getNextControlNumber)('ISA13', senderKey, receiverKey, 9)
             : null;
+        const groups = input.Groups;
         const gs06Values = [];
-        const groups = input.Groups ?? [];
-        for (let i = 0; i < groups.length; i++) {
-            if (needsControlNumber(groups[i].GS.GroupControlNumber_6)) {
+        for (const group of groups) {
+            if (needsControlNumber(group.GS.GroupControlNumber_6)) {
                 const value = await (0, control_numbers_1.getNextControlNumber)('GS06', senderKey, receiverKey, 9);
-                gs06Values.push({ index: i, value });
+                gs06Values.push({ group, value });
             }
         }
         if (isa13) {
@@ -182,8 +175,7 @@ class TemperEdiClient {
                 input.IEATrailers[0].InterchangeControlNumber_2 = isa13;
             }
         }
-        for (const { index, value } of gs06Values) {
-            const group = groups[index];
+        for (const { group, value } of gs06Values) {
             group.GS.GroupControlNumber_6 = value;
             if (group.GETrailers?.length) {
                 group.GETrailers[0].GroupControlNumber_2 = value;
@@ -201,14 +193,7 @@ class TemperEdiClient {
         }
     }
     async toX12({ input }) {
-        // Auto-stamp control numbers before writing X12
-        try {
-            await this.stampControlNumbers(input);
-        }
-        catch (error) {
-            // Workflows can still serialize documents whose control numbers were set manually.
-            unnbound_logger_sdk_1.logger.warn({ err: error }, '[EDI SDK] Failed to auto-stamp control numbers, proceeding without');
-        }
+        await this.stampControlNumbers(input);
         return (0, unnbound_logger_sdk_1.startSpan)('JSON to X12', () => {
             return this.X12.x12WritePost({ x12Interchange: input })
                 .then(this.unwrap.bind(this))

package/package.json

@@ -1,9 +1,21 @@
 {
   "name": "@ontemper/edi",
   "description": "An EDI client with structured logging.",
-  "version": "1.2.0-beta.4",
+  "version": "1.2.0-beta.6",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
+  "scripts": {
+    "build": "tsc",
+    "test": "vitest run src",
+    "typecheck": "tsc --noEmit",
+    "format": "oxfmt --write .",
+    "format:check": "oxfmt --check .",
+    "prepublishOnly": "pnpm run build",
+    "start:example": "tsx watch examples/node-edi.ts",
+    "version:bump": "npm version patch",
+    "release": "pnpm run build && pnpm publish --access public",
+    "codegen": "npx --yes @openapitools/openapi-generator-cli generate -c openapitools.json -o ./src/edination-client"
+  },
   "author": "Unnbound Team",
   "license": "MIT",
   "repository": {
@@ -16,7 +28,7 @@
   },
   "dependencies": {
     "axios": "1.18.0",
-    "unnbound-logger-sdk": "3.1.1"
+    "unnbound-logger-sdk": "workspace:3.1.1"
   },
   "devDependencies": {
     "@types/jest": "^29.5.12",
@@ -33,16 +45,5 @@
   "engines": {
     "node": ">=22"
   },
-  "sideEffects": false,
-  "scripts": {
-    "build": "tsc",
-    "test": "vitest run src",
-    "typecheck": "tsc --noEmit",
-    "format": "oxfmt --write .",
-    "format:check": "oxfmt --check .",
-    "start:example": "tsx watch examples/node-edi.ts",
-    "version:bump": "npm version patch",
-    "release": "pnpm run build && pnpm publish --access public",
-    "codegen": "npx --yes @openapitools/openapi-generator-cli generate -c openapitools.json -o ./src/edination-client"
-  }
-}
+  "sideEffects": false
+}

package/CHANGELOG.md

@@ -1,23 +0,0 @@
-## [1.1.7](https://github.com/ontemper/temper/releases/tag/%40ontemper%2Fedi-v1.1.7) (2026-07-28)
-
-
-### Bug Fixes
-
-* stamp zero acknowledgment envelope control numbers ([#2421](https://github.com/ontemper/temper/pull/2421))
-
-## [1.0.13](https://github.com/unnbounddev/unnbound-sdks/compare/unnbound-edi-sdk-v1.0.12...unnbound-edi-sdk-v1.0.13) (2025-11-25)
-
-
-### Bug Fixes
-
-* setup ci/cd ([dd2640f](https://github.com/unnbounddev/unnbound-sdks/commit/dd2640f01e39f1342f3e08dc882164b50331c7bb))
-* setup ci/cd ([520c03a](https://github.com/unnbounddev/unnbound-sdks/commit/520c03aacd978d7923b3bcf45c4f74a870be019d))
-
-## [1.0.12](https://github.com/unnbounddev/unnbound-sdks/compare/unnbound-edi-sdk-v1.0.11...unnbound-edi-sdk-v1.0.12) (2025-11-25)
-
-
-### Bug Fixes
-
-* remove unused files ([6dc00c2](https://github.com/unnbounddev/unnbound-sdks/commit/6dc00c20b092b17e9eb2108ef83dc5beafdef3f5))
-* setup ci/cd ([d3759c1](https://github.com/unnbounddev/unnbound-sdks/commit/d3759c14f7863e4c8d2f5e80bdbbf2446df79f1a))
-* setup ci/cd ([7e06fc8](https://github.com/unnbounddev/unnbound-sdks/commit/7e06fc84ae7bafba3e9c2c0225f9f9d614fa9207))