@tailor-platform/erp-kit 0.7.0 → 0.9.0

package/src/modules/inventory/docs/commands/UpdateStorageLocation.md

@@ -2,13 +2,13 @@
 
 ## Overview
 
-updateStorageLocation modifies the mutable fields of a storage location, including name, code, storageCondition, and capacity. The warehouseId, parentId, and locationType are immutable after creation.
+updateStorageLocation modifies the mutable fields of a storage location, including name, storageCondition, and capacity. `warehouseId`, `parentId`, `locationType`, and `code` are fixed by the existing record and are not part of the mutable update payload.
 
 ## Business Rules
 
 - Storage location must exist
-- Name and code must remain unique within the warehouse if changed
-- warehouseId, parentId, and locationType are immutable after creation
+- Name must remain unique within the warehouse if changed
+- warehouseId, parentId, locationType, and code are not part of the mutable update payload
 
 ## Process Flow
 
@@ -16,8 +16,8 @@ updateStorageLocation modifies the mutable fields of a storage location, includi
 flowchart TD
     A[Receive update request] --> B{Location exists?}
     B -->|No| C[Return error: STORAGE_LOCATION_NOT_FOUND]
-    B -->|Yes| D{Name or code changed?}
-    D -->|Yes| E{New name/code unique in warehouse?}
+    B -->|Yes| D{Name changed?}
+    D -->|Yes| E{New name unique in warehouse?}
     E -->|No| F[Return error: DUPLICATE_LOCATION_NAME]
     E -->|Yes| G[Update location record]
     D -->|No| G
@@ -31,7 +31,7 @@ flowchart TD
 ## Error Scenarios
 
 - **STORAGE_LOCATION_NOT_FOUND**: Referenced storage location does not exist
-- **DUPLICATE_LOCATION_NAME**: A location with the new name or code already exists in the warehouse
+- **DUPLICATE_LOCATION_NAME**: A location with the new name already exists in the warehouse
 
 ## Test Cases
 

package/src/modules/inventory/docs/commands/UpdateWarehouse.md

@@ -2,13 +2,13 @@
 
 ## Overview
 
-updateWarehouse modifies the mutable fields of an existing warehouse, including name, storageCondition, and capacity. The siteId is immutable and cannot be changed after creation.
+updateWarehouse modifies the mutable fields of an existing warehouse, including name, storageCondition, and capacity. `siteId` is fixed by the existing record and is not part of the update input.
 
 ## Business Rules
 
 - Warehouse must exist
 - Name must remain unique within the same site if changed
-- siteId is immutable after creation
+- siteId is not part of the update input
 
 ## Process Flow
 

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/updateItem.test.ts

@@ -6,7 +6,6 @@ import {
   UnitNotFoundError,
   ItemNotFoundError,
   NoFieldsToUpdateError,
-  SkuImmutableError,
   UomLockedError,
 } from "../lib/errors.generated";
 import { baseActiveItem, baseDraftItem, baseInactiveItem } from "../testing/fixtures";
@@ -31,17 +30,6 @@ describe("updateItem", () => {
     }
   });
 
-  it("returns error when attempting to change SKU", async () => {
-    const { db, spies } = createMockDb<Transaction>();
-    spies.select.mockReturnValueOnce(baseDraftItem);
-
-    const result = await run(db, { id: baseDraftItem.id, sku: "NEW-SKU" }, ctx, { getUnit });
-    expect(result.ok).toBe(false);
-    if (!result.ok) {
-      expect(result.error).toBeInstanceOf(SkuImmutableError);
-    }
-  });
-
   it("returns error when no fields to update", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseDraftItem);
@@ -118,6 +106,22 @@ describe("updateItem", () => {
     expect(spies.update).toHaveBeenCalled();
   });
 
+  it("looks up item by sku and updates name", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    const updatedItem = { ...baseActiveItem, name: "Updated Name" };
+    spies.select.mockReturnValueOnce(baseActiveItem);
+    spies.update.mockReturnValue(updatedItem);
+
+    const result = await run(db, { sku: baseActiveItem.sku, name: "Updated Name" }, ctx, {
+      getUnit,
+    });
+
+    expect(result.ok).toBe(true);
+    if (result.ok) {
+      expect(result.value.item.name).toBe("Updated Name");
+    }
+  });
+
   it("updates barcode in any status", async () => {
     const { db, spies } = createMockDb<Transaction>();
     const updatedItem = { ...baseActiveItem, barcode: "NEW-BAR" };

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

@@ -4,68 +4,84 @@ import {
   UnitNotFoundError,
   ItemNotFoundError,
   NoFieldsToUpdateError,
-  SkuImmutableError,
   UomLockedError,
 } from "../lib/errors.generated";
 import { type PrimitivesQueries } from "../module";
 import { ok, err, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateItemInput {
-  id: string;
-  sku?: string;
+export type UpdateItemInput = (
+  | {
+      id: string;
+    }
+  | {
+      sku: string;
+    }
+) & {
   name?: string;
   barcode?: string | null;
   unitId?: string;
-}
+};
 
 /**
  * Function: updateItem
  *
- * Updates mutable fields of an existing item. SKU is immutable.
- * UoM can only be changed in DRAFT status.
+ * Updates mutable fields of an existing item.
+ * The item can be looked up by id or sku. UoM can only be changed in DRAFT status.
  */
 export async function run<CF extends Record<string, unknown>>(
   db: Transaction,
-  input: UpdateItemInput & Partial<CF>,
+  input: UpdateItemInput & Omit<Partial<CF>, "status">,
   ctx: CommandContext,
   primitivesQueries?: Pick<PrimitivesQueries, "getUnit">,
 ) {
-  const { id, sku, name, barcode, unitId, ...customFields } = input;
-
-  // 1. Check item exists
-  const item = await db
-    .selectFrom("Item")
-    .selectAll()
-    .where("id", "=", id)
-    .forUpdate()
-    .executeTakeFirst();
+  const { name, barcode, unitId } = input;
 
-  if (!item) {
-    return err(new ItemNotFoundError(id));
+  const ITEM_OWN_KEYS = new Set(["id", "sku", "name", "barcode", "unitId"]);
+  const customFields: Record<string, unknown> = {};
+  for (const [key, value] of Object.entries(input as Record<string, unknown>)) {
+    if (!ITEM_OWN_KEYS.has(key)) {
+      customFields[key] = value;
+    }
   }
 
-  // 2. SKU is immutable
-  if (sku !== undefined) {
-    return err(new SkuImmutableError(id));
+  // 1. Check item exists (lookup by id or sku)
+  const item =
+    "id" in input
+      ? await db
+          .selectFrom("Item")
+          .selectAll()
+          .where("id", "=", (input as { id: string }).id)
+          .forUpdate()
+          .executeTakeFirst()
+      : await db
+          .selectFrom("Item")
+          .selectAll()
+          .where("sku", "=", (input as { sku: string }).sku)
+          .forUpdate()
+          .executeTakeFirst();
+
+  if (!item) {
+    const key = "id" in input ? (input as { id: string }).id : (input as { sku: string }).sku;
+    return err(new ItemNotFoundError(key));
   }
 
-  // 3. Check at least one field provided
+  // 2. Check at least one field provided
   const hasName = name !== undefined;
   const hasBarcode = barcode !== undefined;
   const hasUnitId = unitId !== undefined;
   const hasCustomFields = Object.keys(customFields).length > 0;
 
   if (!hasName && !hasBarcode && !hasUnitId && !hasCustomFields) {
-    return err(new NoFieldsToUpdateError(id));
+    return err(new NoFieldsToUpdateError(item.id));
   }
 
-  // 4. Check barcode uniqueness when provided (non-null)
+  // 3. Check barcode uniqueness when provided (non-null)
   if (hasBarcode && barcode !== null) {
     const existingBarcode = await db
       .selectFrom("Item")
       .selectAll()
       .where("barcode", "=", barcode)
-      .where("id", "!=", id)
+      .where("id", "!=", item.id)
       .forUpdate()
       .executeTakeFirst();
 
@@ -74,10 +90,10 @@ export async function run<CF extends Record<string, unknown>>(
     }
   }
 
-  // 5. UoM can only be changed in DRAFT status
+  // 4. UoM can only be changed in DRAFT status
   if (hasUnitId) {
     if (item.status !== "DRAFT") {
-      return err(new UomLockedError(id));
+      return err(new UomLockedError(item.id));
     }
 
     // Validate UoM exists and is active
@@ -88,9 +104,9 @@ export async function run<CF extends Record<string, unknown>>(
     }
   }
 
-  // 6. Apply updates
+  // 5. Apply updates
   const updates: Record<string, unknown> = {
-    ...(customFields as Record<string, unknown>),
+    ...customFields,
     updatedAt: new Date(),
   };
   if (hasName) updates.name = name;
@@ -100,7 +116,7 @@ export async function run<CF extends Record<string, unknown>>(
   const updatedItem = await db
     .updateTable("Item")
     .set(updates)
-    .where("id", "=", id)
+    .where("id", "=", item.id)
     .returningAll()
     .executeTakeFirstOrThrow();
 

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

@@ -1,11 +1,7 @@
 import { describe, expect, it } from "vitest";
 import { createMockDb } from "../../../testing/index";
 import { Transaction } from "../generated/kysely-tailordb";
-import {
-  CodeImmutableError,
-  MissingRequiredFieldsError,
-  NodeNotFoundError,
-} from "../lib/errors.generated";
+import { MissingRequiredFieldsError, NodeNotFoundError } from "../lib/errors.generated";
 import { baseRootNode } from "../testing/fixtures";
 import { run } from "./updateTaxonomyNode";
 
@@ -21,17 +17,6 @@ describe("updateTaxonomyNode", () => {
     }
   });
 
-  it("returns error when attempting to change code", async () => {
-    const { db, spies } = createMockDb<Transaction>();
-    spies.select.mockReturnValueOnce(baseRootNode);
-
-    const result = await run(db, { id: baseRootNode.id, code: "NEW-CODE" });
-    expect(result.ok).toBe(false);
-    if (!result.ok) {
-      expect(result.error).toBeInstanceOf(CodeImmutableError);
-    }
-  });
-
   it("returns error when name is empty", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseRootNode);
@@ -64,11 +49,25 @@ 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();
   });
 
+  it("looks up node by code and updates name", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    const updatedNode = { ...baseRootNode, name: "Consumer Electronics" };
+    spies.select.mockReturnValueOnce(baseRootNode);
+    spies.update.mockReturnValue(updatedNode);
+
+    const result = await run(db, { code: baseRootNode.code, name: "Consumer Electronics" });
+
+    expect(result.ok).toBe(true);
+    if (result.ok) {
+      expect(result.value.taxonomyNode.name).toBe("Consumer Electronics");
+    }
+  });
+
   it("passes custom fields through to set", async () => {
     const { db, spies } = createMockDb<Transaction>();
     const updatedNode = { ...baseRootNode, sortOrder: 5 };

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

@@ -1,55 +1,68 @@
 import { Transaction } from "../generated/kysely-tailordb";
-import {
-  CodeImmutableError,
-  MissingRequiredFieldsError,
-  NodeNotFoundError,
-} from "../lib/errors.generated";
+import { MissingRequiredFieldsError, NodeNotFoundError } from "../lib/errors.generated";
 import { ok, err } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateTaxonomyNodeInput {
-  id: string;
-  code?: string;
+export type UpdateTaxonomyNodeInput = (
+  | {
+      id: string;
+    }
+  | {
+      code: string;
+    }
+) & {
   name?: string;
-}
+};
 
 /**
  * Function: updateTaxonomyNode
  *
  * Updates the display name of an existing taxonomy node.
- * Node code is immutable.
+ * The node can be looked up by id or code.
  */
 export async function run<CF extends Record<string, unknown>>(
   db: Transaction,
-  input: UpdateTaxonomyNodeInput & Partial<CF>,
+  input: UpdateTaxonomyNodeInput & Omit<Partial<CF>, "status">,
 ) {
-  const { id, code, name, ...customFields } = input;
-
-  // 1. Check node exists
-  const node = await db
-    .selectFrom("TaxonomyNode")
-    .selectAll()
-    .where("id", "=", id)
-    .forUpdate()
-    .executeTakeFirst();
+  const { name } = input;
 
-  if (!node) {
-    return err(new NodeNotFoundError(id));
+  const KNOWN_KEYS = new Set(["id", "code", "name"]);
+  const customFields: Record<string, unknown> = {};
+  for (const [key, value] of Object.entries(input as Record<string, unknown>)) {
+    if (!KNOWN_KEYS.has(key)) {
+      customFields[key] = value;
+    }
   }
 
-  // 2. Code is immutable
-  if (code !== undefined) {
-    return err(new CodeImmutableError(id));
+  // 1. Check node exists (lookup by id or code)
+  const node =
+    "id" in input
+      ? await db
+          .selectFrom("TaxonomyNode")
+          .selectAll()
+          .where("id", "=", (input as { id: string }).id)
+          .forUpdate()
+          .executeTakeFirst()
+      : await db
+          .selectFrom("TaxonomyNode")
+          .selectAll()
+          .where("code", "=", (input as { code: string }).code)
+          .forUpdate()
+          .executeTakeFirst();
+
+  if (!node) {
+    const key = "id" in input ? (input as { id: string }).id : (input as { code: string }).code;
+    return err(new NodeNotFoundError(key));
   }
 
-  // 3. Name must be provided and non-empty (unless custom fields provided)
+  // 2. Name must be provided and non-empty (unless custom fields provided)
   const hasCustomFields = Object.keys(customFields).length > 0;
   if (!name && !hasCustomFields) {
     return err(new MissingRequiredFieldsError("name"));
   }
 
-  // 4. Update fields
+  // 3. Update fields
   const updates: Record<string, unknown> = {
-    ...(customFields as Record<string, unknown>),
+    ...customFields,
     updatedAt: new Date(),
   };
   if (name) updates.name = name;
@@ -57,9 +70,9 @@ export async function run<CF extends Record<string, unknown>>(
   const updatedNode = await db
     .updateTable("TaxonomyNode")
     .set(updates)
-    .where("id", "=", id)
+    .where("id", "=", node.id)
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ node: updatedNode });
+  return ok({ taxonomyNode: updatedNode });
 }

package/src/modules/item-management/docs/commands/UpdateItem.md

@@ -2,14 +2,13 @@
 
 ## Overview
 
-updateItem modifies mutable fields of an existing item — name, barcode, and unit of measure (UoM). SKU is immutable and cannot be changed after creation. The set of updatable fields depends on the item's current lifecycle status.
+updateItem modifies mutable fields of an existing item — name, barcode, and unit of measure (UoM). The item can be looked up by `id` or `sku`. The set of updatable fields depends on the item's current lifecycle status.
 
 This command supports the DRAFT review workflow (correcting barcodes, setting names before activation) as well as post-activation updates to non-transactional fields.
 
 ## Business Rules
 
-- Target item must exist in the system
-- SKU is immutable — any attempt to change SKU returns an error
+- Target item must exist in the system (looked up by `id` or `sku`)
 - Name can be updated in any status (DRAFT, ACTIVE, INACTIVE)
 - Barcode can be updated in any status; must remain unique across all items when provided
 - Barcode can be cleared (set to null) to remove it from the item
@@ -21,11 +20,9 @@ This command supports the DRAFT review workflow (correcting barcodes, setting na
 
 ```mermaid
 flowchart TD
-    A[Receive update request] --> B{Item exists?}
-    B -->|No| C[Return error: not found]
-    B -->|Yes| D{SKU change attempted?}
-    D -->|Yes| E[Return error: SKU immutable]
-    D -->|No| F{Any field provided?}
+    A[Receive update request] --> B{Lookup by id or sku}
+    B -->|Not found| C[Return error: not found]
+    B -->|Found| F{Any field provided?}
     F -->|No| G[Return error: no fields to update]
     F -->|Yes| H{Barcode provided?}
     H -->|Yes| I{Barcode unique?}
@@ -47,8 +44,7 @@ flowchart TD
 
 ## Error Scenarios
 
-- **ITEM_NOT_FOUND**: Specified item ID does not exist
-- **SKU_IMMUTABLE**: Attempt to change the SKU field
+- **ITEM_NOT_FOUND**: Specified item does not exist
 - **DUPLICATE_BARCODE**: An item with the same barcode already exists
 - **UNIT_NOT_FOUND**: Referenced unit does not exist or is inactive
 - **UOM_LOCKED**: UoM cannot be changed after item has been activated (status is ACTIVE or INACTIVE)
@@ -57,13 +53,13 @@ flowchart TD
 ## Test Cases
 
 - returns error when item does not exist
-- returns error when attempting to change SKU
 - returns error when no fields to update
 - returns error when barcode is duplicate
 - returns error when changing UoM on non-DRAFT item
 - returns error when changing UoM on INACTIVE item
 - returns error when new UoM does not exist
 - updates name in any status
+- looks up item by sku and updates name
 - updates barcode in any status
 - clears barcode by setting to null
 - updates UoM in DRAFT status

package/src/modules/item-management/docs/commands/UpdateTaxonomyNode.md

@@ -2,12 +2,11 @@
 
 ## Overview
 
-updateTaxonomyNode modifies the display name of an existing taxonomy node. The node code is immutable and cannot be changed after creation — only the name (display label) can be updated. This supports scenarios such as renaming a classification branch to better reflect its contents or aligning terminology across the organization.
+updateTaxonomyNode modifies the display name of an existing taxonomy node. The node can be looked up by `id` or `code`. This supports scenarios such as renaming a classification branch to better reflect its contents or aligning terminology across the organization.
 
 ## Business Rules
 
-- Target node must exist in the system
-- Node code is immutable — any attempt to change the code returns an error
+- Target node must exist in the system (looked up by `id` or `code`)
 - Name is required and must not be empty
 - Name update does not affect the node's position in the tree, child nodes, or item assignments
 
@@ -15,14 +14,12 @@ updateTaxonomyNode modifies the display name of an existing taxonomy node. The n
 
 ```mermaid
 flowchart TD
-    A[Receive update request] --> B{Node exists?}
-    B -->|No| C[Return error: not found]
-    B -->|Yes| D{Code change attempted?}
-    D -->|Yes| E[Return error: code immutable]
-    D -->|No| F{Name provided and non-empty?}
-    F -->|No| G[Return error: name required]
-    F -->|Yes| H[Update node name]
-    H --> I[Return updated node]
+    A[Receive update request] --> B{Lookup by id or code}
+    B -->|Not found| C[Return error: not found]
+    B -->|Found| D{Name provided and non-empty?}
+    D -->|No| E[Return error: name required]
+    D -->|Yes| F[Update node name]
+    F --> G[Return updated node]
 ```
 
 ## External Dependencies
@@ -31,15 +28,14 @@ flowchart TD
 
 ## Error Scenarios
 
-- **NODE_NOT_FOUND**: Specified node ID does not exist
-- **CODE_IMMUTABLE**: Attempt to change the node code
+- **NODE_NOT_FOUND**: Specified node does not exist
 - **MISSING_REQUIRED_FIELDS**: Name not provided or empty
 
 ## Test Cases
 
 - returns error when node does not exist
-- returns error when attempting to change code
 - returns error when name is empty
 - returns error when name is not provided
 - updates node name
+- looks up node by code and updates name
 - passes custom fields through to set

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);