@tailor-platform/erp-kit 0.8.0 → 0.9.0

package/src/modules/coa-management/docs/commands/UpdateAccountGroup.md

@@ -12,7 +12,6 @@ updateAccountGroup modifies the mutable fields (name, sort order, number range)
 - Group code is not part of the update input
 - Number range changes, when provided, require both start and end to be provided together, must remain a valid non-overlapping interval within the CoA
 - Changing a number range must not invalidate existing accounts already assigned to the group (their codes must still fall within the new range)
-- Emits an audit event via the audit module recording the acting user, timestamp, and previous and new field values for all changed attributes
 
 ## Process Flow
 
@@ -29,13 +28,12 @@ flowchart TD
     F -->|Yes| H{Existing accounts within new range?}
     H -->|No| I[Return error: accounts out of range]
     H -->|Yes| J[Update group fields]
-    J --> K[Emit audit event]
-    K --> L[Return updated group]
+    J --> K[Return updated group]
 ```
 
 ## External Dependencies
 
-- [audit](../../../audit/README.md) — Receives audit events emitted by this command
+- None
 
 ## Error Scenarios
 
@@ -59,5 +57,4 @@ flowchart TD
 - updates group name
 - updates sort order
 - updates number range
-- emits audit event recording previous and new field values
 - passes custom fields through to update

package/src/modules/coa-management/docs/commands/UpdateChartOfAccounts.md

@@ -10,7 +10,6 @@ updateChartOfAccounts modifies the mutable fields (name, description) of an exis
 - Target CoA must be in DRAFT or ACTIVE status; updating an ARCHIVED CoA is rejected
 - Name, when provided, must be non-empty
 - Description is optional and can be cleared
-- Emits an audit event via the audit module recording the acting user, timestamp, and previous and new field values for all changed attributes
 
 ## Process Flow
 
@@ -23,8 +22,7 @@ flowchart TD
     D -->|Yes| F{Name valid if provided?}
     F -->|No| G[Return error: name required]
     F -->|Yes| H[Update CoA fields]
-    H --> I[Emit audit event]
-    I --> J[Return updated CoA]
+    H --> I[Return updated CoA]
 ```
 
 ## External Dependencies
@@ -45,5 +43,4 @@ flowchart TD
 - updates name of a DRAFT CoA
 - updates description of an ACTIVE CoA
 - clears description by setting to null
-- emits audit event recording previous and new field values
 - passes custom fields through to update

package/src/modules/coa-management/module.ts

@@ -1,7 +1,7 @@
 import { type TailorAnyDBField } from "@tailor-platform/sdk";
 import type { defineModule as defineOrganizationModule } from "../organization";
 import type { defineModule as definePrimitivesModule } from "../primitives";
-import type { TemplateProvider, AuditCommands } from "./command/createChartOfAccounts";
+import type { TemplateProvider } from "./command/createChartOfAccounts";
 // Commands
 import { createChartOfAccounts } from "./command/createChartOfAccounts.generated";
 import { updateChartOfAccounts } from "./command/updateChartOfAccounts.generated";
@@ -60,11 +60,9 @@ export interface DefineModuleParams<
     queries: Pick<PrimitivesQueries, "getCurrency">;
   };
   templateProvider?: TemplateProvider;
-  audit?: {
-    commands: AuditCommands;
-  };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <
   const COAF extends Record<string, TailorAnyDBField> = EmptyFields,
   const AGF extends Record<string, TailorAnyDBField> = EmptyFields,
@@ -88,8 +86,6 @@ export const defineModule = <
     currencyType: params.primitives.db.currency,
   });
 
-  const auditCommands = params.audit?.commands;
-
   return {
     db: {
       chartOfAccounts: chartOfAccountsDb,
@@ -100,28 +96,21 @@ export const defineModule = <
       createChartOfAccounts: createChartOfAccounts<FieldsToInsertable<COAF>>(
         params.organization.queries,
         params.templateProvider,
-        auditCommands,
-      ),
-      updateChartOfAccounts: updateChartOfAccounts<FieldsToInsertable<COAF>>(auditCommands),
-      activateChartOfAccounts: activateChartOfAccounts(auditCommands),
-      deactivateChartOfAccounts: deactivateChartOfAccounts(auditCommands),
-      deleteChartOfAccounts: deleteChartOfAccounts(auditCommands),
-      createAccountGroup: createAccountGroup<FieldsToInsertable<AGF>>(auditCommands),
-      updateAccountGroup: updateAccountGroup<FieldsToInsertable<AGF>>(auditCommands),
-      moveAccountGroup: moveAccountGroup(auditCommands),
-      deleteAccountGroup: deleteAccountGroup(auditCommands),
-      createAccount: createAccount<FieldsToInsertable<AF>>(
-        params.primitives.queries,
-        auditCommands,
-      ),
-      updateAccount: updateAccount<FieldsToInsertable<AF>>(
-        params.primitives.queries,
-        auditCommands,
       ),
-      activateAccount: activateAccount(auditCommands),
-      deactivateAccount: deactivateAccount(auditCommands),
-      reactivateAccount: reactivateAccount(auditCommands),
-      deleteAccount: deleteAccount(auditCommands),
+      updateChartOfAccounts: updateChartOfAccounts<FieldsToInsertable<COAF>>(),
+      activateChartOfAccounts: activateChartOfAccounts(),
+      deactivateChartOfAccounts: deactivateChartOfAccounts(),
+      deleteChartOfAccounts: deleteChartOfAccounts(),
+      createAccountGroup: createAccountGroup<FieldsToInsertable<AGF>>(),
+      updateAccountGroup: updateAccountGroup<FieldsToInsertable<AGF>>(),
+      moveAccountGroup: moveAccountGroup(),
+      deleteAccountGroup: deleteAccountGroup(),
+      createAccount: createAccount<FieldsToInsertable<AF>>(params.primitives.queries),
+      updateAccount: updateAccount<FieldsToInsertable<AF>>(params.primitives.queries),
+      activateAccount: activateAccount(),
+      deactivateAccount: deactivateAccount(),
+      reactivateAccount: reactivateAccount(),
+      deleteAccount: deleteAccount(),
     },
     queries: {
       getChartOfAccounts,

package/src/modules/finance-ledger/module.ts

@@ -129,6 +129,7 @@ export interface DefineModuleParams<
   };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <
   const FyF extends Record<string, TailorAnyDBField> = EmptyFields,
   const ApF extends Record<string, TailorAnyDBField> = EmptyFields,

package/src/modules/inventory/command/approveInventoryAdjustment.test.ts

@@ -49,7 +49,7 @@ describe("approveInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("APPROVED");
+      expect(result.value.inventoryAdjustment.status).toBe("APPROVED");
     }
     expect(spies.update).toHaveBeenCalled();
   });

package/src/modules/inventory/command/approveInventoryAdjustment.ts

@@ -46,5 +46,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: updated! });
+  return ok({ inventoryAdjustment: updated! });
 }

package/src/modules/inventory/command/cancelStockMovement.test.ts

@@ -67,7 +67,7 @@ describe("cancelStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("CANCELLED");
+      expect(result.value.stockMovement.status).toBe("CANCELLED");
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -82,7 +82,7 @@ describe("cancelStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("CANCELLED");
+      expect(result.value.stockMovement.status).toBe("CANCELLED");
     }
   });
 

package/src/modules/inventory/command/cancelStockMovement.ts

@@ -52,5 +52,5 @@ export async function run(db: Transaction, input: CancelStockMovementInput, _ctx
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ movement: cancelled! });
+  return ok({ stockMovement: cancelled! });
 }

package/src/modules/inventory/command/confirmInventoryAdjustment.test.ts

@@ -81,7 +81,7 @@ describe("confirmInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("CONFIRMED");
+      expect(result.value.inventoryAdjustment.status).toBe("CONFIRMED");
     }
   });
 
@@ -104,7 +104,7 @@ describe("confirmInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("CONFIRMED");
+      expect(result.value.inventoryAdjustment.status).toBe("CONFIRMED");
     }
   });
 
@@ -133,7 +133,7 @@ describe("confirmInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("CONFIRMED");
+      expect(result.value.inventoryAdjustment.status).toBe("CONFIRMED");
     }
   });
 
@@ -163,7 +163,7 @@ describe("confirmInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("CONFIRMED");
+      expect(result.value.inventoryAdjustment.status).toBe("CONFIRMED");
     }
   });
 

package/src/modules/inventory/command/confirmInventoryAdjustment.ts

@@ -190,5 +190,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: confirmed! });
+  return ok({ inventoryAdjustment: confirmed! });
 }

package/src/modules/inventory/command/confirmStockMovement.test.ts

@@ -119,7 +119,7 @@ describe("confirmStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("CONFIRMED");
+      expect(result.value.stockMovement.status).toBe("CONFIRMED");
     }
     expect(spies.update).toHaveBeenCalled();
   });

package/src/modules/inventory/command/confirmStockMovement.ts

@@ -127,5 +127,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ movement: confirmed! });
+  return ok({ stockMovement: confirmed! });
 }

package/src/modules/inventory/command/createInventoryAdjustment.test.ts

@@ -135,8 +135,8 @@ describe("createInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.adjustmentType).toBe("CORRECTION");
-      expect(result.value.adjustment.quantityDelta).toBe(10);
+      expect(result.value.inventoryAdjustment.adjustmentType).toBe("CORRECTION");
+      expect(result.value.inventoryAdjustment.quantityDelta).toBe(10);
     }
     expect(spies.insert).toHaveBeenCalled();
   });
@@ -162,7 +162,7 @@ describe("createInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.quantityDelta).toBe(-5);
+      expect(result.value.inventoryAdjustment.quantityDelta).toBe(-5);
     }
     expect(spies.insert).toHaveBeenCalled();
   });
@@ -188,8 +188,8 @@ describe("createInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.adjustmentType).toBe("SCRAP");
-      expect(result.value.adjustment.quantityDelta).toBe(-3);
+      expect(result.value.inventoryAdjustment.adjustmentType).toBe("SCRAP");
+      expect(result.value.inventoryAdjustment.quantityDelta).toBe(-3);
     }
     expect(spies.insert).toHaveBeenCalled();
   });
@@ -216,7 +216,7 @@ describe("createInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.inventoryCountId).toBe("count-1");
+      expect(result.value.inventoryAdjustment.inventoryCountId).toBe("count-1");
     }
     expect(spies.insert).toHaveBeenCalled();
   });

package/src/modules/inventory/command/createInventoryAdjustment.ts

@@ -120,5 +120,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: adjustment! });
+  return ok({ inventoryAdjustment: adjustment! });
 }

package/src/modules/inventory/command/createStockMovement.test.ts

@@ -249,8 +249,8 @@ describe("createStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DRAFT");
-      expect(result.value.moveLines).toHaveLength(1);
+      expect(result.value.stockMovement.status).toBe("DRAFT");
+      expect(result.value.stockMoveLines).toHaveLength(1);
     }
   });
 
@@ -335,7 +335,7 @@ describe("createStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.moveLines).toHaveLength(2);
+      expect(result.value.stockMoveLines).toHaveLength(2);
     }
     expect(spies.insert).toHaveBeenCalledTimes(3); // 1 movement + 2 lines
   });

package/src/modules/inventory/command/createStockMovement.ts

@@ -168,5 +168,5 @@ export async function run(
     moveLines.push(moveLine!);
   }
 
-  return ok({ movement: movement!, moveLines });
+  return ok({ stockMovement: movement!, stockMoveLines: moveLines });
 }

package/src/modules/inventory/command/executeStockMovement.test.ts

@@ -186,7 +186,7 @@ describe("executeStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DONE");
+      expect(result.value.stockMovement.status).toBe("DONE");
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -217,7 +217,7 @@ describe("executeStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DONE");
+      expect(result.value.stockMovement.status).toBe("DONE");
     }
   });
 
@@ -256,7 +256,7 @@ describe("executeStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DONE");
+      expect(result.value.stockMovement.status).toBe("DONE");
     }
   });
 
@@ -290,7 +290,7 @@ describe("executeStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DONE");
+      expect(result.value.stockMovement.status).toBe("DONE");
     }
   });
 
@@ -324,7 +324,7 @@ describe("executeStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.status).toBe("DONE");
+      expect(result.value.stockMovement.status).toBe("DONE");
     }
   });
 

package/src/modules/inventory/command/executeStockMovement.ts

@@ -207,7 +207,7 @@ export async function run(db: Transaction, input: ExecuteStockMovementInput, _ct
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ movement: executed! });
+  return ok({ stockMovement: executed! });
 }
 
 // Helper: get stock level for item at location

package/src/modules/inventory/command/rejectInventoryAdjustment.test.ts

@@ -49,7 +49,7 @@ describe("rejectInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("REJECTED");
+      expect(result.value.inventoryAdjustment.status).toBe("REJECTED");
     }
     expect(spies.update).toHaveBeenCalled();
   });

package/src/modules/inventory/command/rejectInventoryAdjustment.ts

@@ -46,5 +46,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: updated! });
+  return ok({ inventoryAdjustment: updated! });
 }

package/src/modules/inventory/command/reviseInventoryAdjustment.test.ts

@@ -46,7 +46,7 @@ describe("reviseInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("DRAFT");
+      expect(result.value.inventoryAdjustment.status).toBe("DRAFT");
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -62,7 +62,7 @@ describe("reviseInventoryAdjustment", () => {
     expect(result.ok).toBe(true);
     if (result.ok) {
       // After revise, adjustment is back in DRAFT and can be modified/resubmitted
-      expect(result.value.adjustment.status).toBe("DRAFT");
+      expect(result.value.inventoryAdjustment.status).toBe("DRAFT");
     }
   });
 });

package/src/modules/inventory/command/reviseInventoryAdjustment.ts

@@ -44,5 +44,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: updated! });
+  return ok({ inventoryAdjustment: updated! });
 }

package/src/modules/inventory/command/submitInventoryAdjustment.test.ts

@@ -46,7 +46,7 @@ describe("submitInventoryAdjustment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.adjustment.status).toBe("PENDING_APPROVAL");
+      expect(result.value.inventoryAdjustment.status).toBe("PENDING_APPROVAL");
     }
     expect(spies.update).toHaveBeenCalled();
   });

package/src/modules/inventory/command/submitInventoryAdjustment.ts

@@ -43,5 +43,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ adjustment: updated! });
+  return ok({ inventoryAdjustment: updated! });
 }

package/src/modules/inventory/command/updateStockMovement.test.ts

@@ -71,7 +71,7 @@ describe("updateStockMovement", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.movement.reasonCode).toBe("new-reason");
+      expect(result.value.stockMovement.reasonCode).toBe("new-reason");
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -215,9 +215,9 @@ describe("updateStockMovement", () => {
       // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-explicit-any
       const value = result.value as any;
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      expect(value.moveLines).toHaveLength(1);
+      expect(value.stockMoveLines).toHaveLength(1);
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      expect(value.moveLines![0].quantity).toBe(30);
+      expect(value.stockMoveLines![0].quantity).toBe(30);
     }
   });
 
@@ -244,7 +244,7 @@ describe("updateStockMovement", () => {
       // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-explicit-any
       const value = result.value as any;
       // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      expect(value.moveLines).toHaveLength(0);
+      expect(value.stockMoveLines).toHaveLength(0);
     }
   });
 });

package/src/modules/inventory/command/updateStockMovement.ts

@@ -156,8 +156,8 @@ export async function run(
       newLines.push(moveLine!);
     }
 
-    return ok({ movement: updatedMovement!, moveLines: newLines });
+    return ok({ stockMovement: updatedMovement!, stockMoveLines: newLines });
   }
 
-  return ok({ movement: updatedMovement! });
+  return ok({ stockMovement: updatedMovement! });
 }

package/src/modules/inventory/module.ts

@@ -94,6 +94,7 @@ export interface DefineModuleParams<WF extends Record<string, TailorAnyDBField>>
   };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <const WF extends Record<string, TailorAnyDBField> = EmptyFields>(
   params: DefineModuleParams<WF>,
 ) => {

package/src/modules/item-management/command/createTaxonomyNode.test.ts

@@ -79,8 +79,8 @@ describe("createTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.code).toBe("CLOTHING");
-      expect(result.value.node.parentId).toBeNull();
+      expect(result.value.taxonomyNode.code).toBe("CLOTHING");
+      expect(result.value.taxonomyNode.parentId).toBeNull();
     }
     expect(spies.insert).toHaveBeenCalled();
   });
@@ -108,8 +108,8 @@ describe("createTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.code).toBe("TABLETS");
-      expect(result.value.node.parentId).toBe(baseRootNode.id);
+      expect(result.value.taxonomyNode.code).toBe("TABLETS");
+      expect(result.value.taxonomyNode.parentId).toBe(baseRootNode.id);
     }
   });
 

package/src/modules/item-management/command/createTaxonomyNode.ts

@@ -85,5 +85,5 @@ export async function run<CF extends Record<string, unknown>>(
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ node: node });
+  return ok({ taxonomyNode: node });
 }

package/src/modules/item-management/command/moveTaxonomyNode.test.ts

@@ -87,7 +87,7 @@ describe("moveTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.parentId).toBeNull();
+      expect(result.value.taxonomyNode.parentId).toBeNull();
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -110,7 +110,7 @@ describe("moveTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.parentId).toBe(secondRootNode.id);
+      expect(result.value.taxonomyNode.parentId).toBe(secondRootNode.id);
     }
   });
 

package/src/modules/item-management/command/moveTaxonomyNode.ts

@@ -48,7 +48,7 @@ export async function run(
       .returningAll()
       .executeTakeFirst();
 
-    return ok({ node: movedNode! });
+    return ok({ taxonomyNode: movedNode! });
   }
 
   // 3. Check new parent exists (validation only — no lock)
@@ -125,5 +125,5 @@ export async function run(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ node: movedNode! });
+  return ok({ taxonomyNode: movedNode! });
 }

package/src/modules/item-management/command/updateTaxonomyNode.test.ts

@@ -49,7 +49,7 @@ describe("updateTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.name).toBe("Consumer Electronics");
+      expect(result.value.taxonomyNode.name).toBe("Consumer Electronics");
     }
     expect(spies.update).toHaveBeenCalled();
   });
@@ -64,7 +64,7 @@ describe("updateTaxonomyNode", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.node.name).toBe("Consumer Electronics");
+      expect(result.value.taxonomyNode.name).toBe("Consumer Electronics");
     }
   });
 

package/src/modules/item-management/command/updateTaxonomyNode.ts

@@ -74,5 +74,5 @@ export async function run<CF extends Record<string, unknown>>(
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ node: updatedNode });
+  return ok({ taxonomyNode: updatedNode });
 }

package/src/modules/item-management/module.ts

@@ -49,6 +49,7 @@ export interface DefineModuleParams<
   };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <
   const IF extends Record<string, TailorAnyDBField> = EmptyFields,
   const TNF extends Record<string, TailorAnyDBField> = EmptyFields,

package/src/modules/manufacturing/command/createRouting.ts

@@ -124,5 +124,5 @@ export async function run<CF extends Record<string, unknown>>(
     createdOperations.push(operation!);
   }
 
-  return ok({ routing: routing!, operations: createdOperations });
+  return ok({ routing: routing!, routingOperations: createdOperations });
 }

package/src/modules/manufacturing/command/recordInventoryIssueOutcome.test.ts

@@ -61,7 +61,7 @@ describe("recordInventoryIssueOutcome", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.costSummary).toEqual(updatedSummary);
+      expect(result.value.manufacturingCostSummary).toEqual(updatedSummary);
     }
     expect(spies.insert).toHaveBeenCalled();
     expect(spies.update).toHaveBeenCalled();

package/src/modules/manufacturing/command/recordInventoryIssueOutcome.ts

@@ -135,5 +135,5 @@ export async function run<CF extends Record<string, unknown>>(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ costSummary: updatedSummary! });
+  return ok({ manufacturingCostSummary: updatedSummary! });
 }

package/src/modules/manufacturing/command/recordManufacturingCostSettlementAcknowledgment.test.ts

@@ -45,7 +45,7 @@ describe("recordManufacturingCostSettlementAcknowledgment", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.costSummary.status).toBe("SETTLED");
+      expect(result.value.manufacturingCostSummary.status).toBe("SETTLED");
     }
     expect(spies.insert).toHaveBeenCalled();
     expect(spies.update).toHaveBeenCalled();

package/src/modules/manufacturing/command/recordManufacturingCostSettlementAcknowledgment.ts

@@ -106,5 +106,5 @@ export async function run<CF extends Record<string, unknown>>(
     .returningAll()
     .executeTakeFirst();
 
-  return ok({ costSummary: updatedSummary! });
+  return ok({ manufacturingCostSummary: updatedSummary! });
 }

package/src/modules/manufacturing/command/reviewManufacturingCostSummary.test.ts

@@ -92,7 +92,7 @@ describe("reviewManufacturingCostSummary", () => {
 
     expect(result.ok).toBe(true);
     if (result.ok) {
-      expect(result.value.costSummary.status).toBe("VARIANCE_REVIEWED");
+      expect(result.value.manufacturingCostSummary.status).toBe("VARIANCE_REVIEWED");
     }
     // 8 variance lines inserted
     expect(spies.insert).toHaveBeenCalledTimes(8);