@tailor-platform/erp-kit 0.2.1 → 0.3.0

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

@@ -1,22 +1,16 @@
 import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
-import { type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { createMockDb } from "../../../testing/index";
+import { Transaction } from "../generated/kysely-tailordb";
 import { InvalidStateTransitionError, ItemNotFoundError } from "../lib/errors.generated";
 import { baseActiveItem, baseDraftItem, baseInactiveItem } from "../testing/fixtures";
 import { run } from "./reactivateItem";
 
 describe("reactivateItem", () => {
-  const ctx: CommandContext = {
-    actorId: "test-actor",
-    permissions: ["item-management:reactivateItem"],
-  };
-
   it("returns error when item does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValue(undefined);
 
-    const result = await run(db, { id: "nonexistent" }, ctx);
+    const result = await run(db, { id: "nonexistent" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(ItemNotFoundError);
@@ -24,10 +18,10 @@ describe("reactivateItem", () => {
   });
 
   it("returns error when item is not INACTIVE", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseActiveItem);
 
-    const result = await run(db, { id: baseActiveItem.id }, ctx);
+    const result = await run(db, { id: baseActiveItem.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(InvalidStateTransitionError);
@@ -35,10 +29,10 @@ describe("reactivateItem", () => {
   });
 
   it("returns error when item is DRAFT", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseDraftItem);
 
-    const result = await run(db, { id: baseDraftItem.id }, ctx);
+    const result = await run(db, { id: baseDraftItem.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(InvalidStateTransitionError);
@@ -46,12 +40,12 @@ describe("reactivateItem", () => {
   });
 
   it("reactivates an INACTIVE item", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const reactivatedItem = { ...baseInactiveItem, status: "ACTIVE" };
     spies.select.mockReturnValueOnce(baseInactiveItem);
     spies.update.mockReturnValue(reactivatedItem);
 
-    const result = await run(db, { id: baseInactiveItem.id }, ctx);
+    const result = await run(db, { id: baseInactiveItem.id });
 
     expect(result.ok).toBe(true);
     if (result.ok) {
@@ -61,12 +55,12 @@ describe("reactivateItem", () => {
   });
 
   it("reactivates from custom status when from is overridden", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const reactivatedItem = { ...baseDraftItem, status: "ACTIVE" };
     spies.select.mockReturnValueOnce(baseDraftItem);
     spies.update.mockReturnValue(reactivatedItem);
 
-    const result = await run(db, { id: baseDraftItem.id, from: ["INACTIVE", "DRAFT"] }, ctx);
+    const result = await run(db, { id: baseDraftItem.id, from: ["INACTIVE", "DRAFT"] });
 
     expect(result.ok).toBe(true);
     if (result.ok) {

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

@@ -1,7 +1,6 @@
-import { ok, err, type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { ok, err } from "../../../shared";
+import { Transaction } from "../generated/kysely-tailordb";
 import { InvalidStateTransitionError, ItemNotFoundError } from "../lib/errors.generated";
-import { getItem } from "../query/getItem.generated";
 
 export interface ReactivateItemInput {
   id: string;
@@ -14,11 +13,16 @@ export interface ReactivateItemInput {
  * Transitions an item from INACTIVE back to ACTIVE status,
  * restoring a previously discontinued SKU for transactions.
  */
-export async function run(db: DB, input: ReactivateItemInput, ctx: CommandContext) {
+export async function run(db: Transaction, input: ReactivateItemInput) {
   const { id, from } = input;
 
   // 1. Check item exists
-  const { item } = await getItem(db, { id }, ctx);
+  const item = await db
+    .selectFrom("Item")
+    .selectAll()
+    .where("id", "=", id)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (!item) {
     return err(new ItemNotFoundError(id));

package/src/modules/item-management/command/removeItemFromTaxonomy.generated.ts

@@ -1,5 +1,5 @@
 // @generated — do not edit
-import { defineCommand } from "../../shared/internal";
+import { defineCommand } from "../../../shared";
 import { permissions } from "../lib/permissions.generated";
 import { run } from "./removeItemFromTaxonomy";
 

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

@@ -1,22 +1,16 @@
 import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
-import { type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { createMockDb } from "../../../testing/index";
+import { Transaction } from "../generated/kysely-tailordb";
 import { AssignmentNotFoundError } from "../lib/errors.generated";
 import { baseAssignment } from "../testing/fixtures";
 import { run } from "./removeItemFromTaxonomy";
 
 describe("removeItemFromTaxonomy", () => {
-  const ctx: CommandContext = {
-    actorId: "test-actor",
-    permissions: ["item-management:removeItemFromTaxonomy"],
-  };
-
   it("returns error when assignment does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined);
 
-    const result = await run(db, { itemId: "item-99", taxonomyNodeId: "node-99" }, ctx);
+    const result = await run(db, { itemId: "item-99", taxonomyNodeId: "node-99" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(AssignmentNotFoundError);
@@ -24,15 +18,14 @@ describe("removeItemFromTaxonomy", () => {
   });
 
   it("removes assignment successfully", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseAssignment);
     spies.delete.mockReturnValue(undefined);
 
-    const result = await run(
-      db,
-      { itemId: baseAssignment.itemId, taxonomyNodeId: baseAssignment.taxonomyNodeId },
-      ctx,
-    );
+    const result = await run(db, {
+      itemId: baseAssignment.itemId,
+      taxonomyNodeId: baseAssignment.taxonomyNodeId,
+    });
 
     expect(result.ok).toBe(true);
     if (result.ok) {

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

@@ -1,7 +1,6 @@
-import { ok, err, type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { ok, err } from "../../../shared";
+import { Transaction } from "../generated/kysely-tailordb";
 import { AssignmentNotFoundError } from "../lib/errors.generated";
-import { getItemTaxonomyAssignment } from "../query/getItemTaxonomyAssignment.generated";
 
 export interface RemoveItemFromTaxonomyInput {
   itemId: string;
@@ -13,11 +12,17 @@ export interface RemoveItemFromTaxonomyInput {
  *
  * Removes the link between an item and a taxonomy node.
  */
-export async function run(db: DB, input: RemoveItemFromTaxonomyInput, ctx: CommandContext) {
+export async function run(db: Transaction, input: RemoveItemFromTaxonomyInput) {
   const { itemId, taxonomyNodeId } = input;
 
-  // 1. Check assignment exists
-  const { assignment } = await getItemTaxonomyAssignment(db, { itemId, taxonomyNodeId }, ctx);
+  // 1. Check assignment exists (read-then-delete — lock)
+  const assignment = await db
+    .selectFrom("ItemTaxonomyAssignment")
+    .selectAll()
+    .where("itemId", "=", itemId)
+    .where("taxonomyNodeId", "=", taxonomyNodeId)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (!assignment) {
     return err(new AssignmentNotFoundError(`${itemId} in ${taxonomyNodeId}`));

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

@@ -1,5 +1,5 @@
 // @generated — do not edit
-import { defineCommand } from "../../shared/internal";
+import { defineCommand } from "../../../shared";
 import { permissions } from "../lib/permissions.generated";
 import { run } from "./updateItem";
 

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

@@ -1,7 +1,7 @@
 import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
-import { type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { createMockDb } from "../../../testing/index";
+import { type CommandContext } from "../../../shared";
+import { Transaction } from "../generated/kysely-tailordb";
 import {
   DuplicateBarcodeError,
   UnitNotFoundError,
@@ -11,7 +11,7 @@ import {
   UomLockedError,
 } from "../lib/errors.generated";
 import { baseActiveItem, baseDraftItem, baseInactiveItem } from "../testing/fixtures";
-import { getUnit } from "../../primitives";
+import { getUnit } from "../../primitives/query/getUnit.generated";
 import { run } from "./updateItem";
 
 describe("updateItem", () => {
@@ -21,7 +21,7 @@ describe("updateItem", () => {
   };
 
   it("returns error when item does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValue(undefined);
 
     const result = await run(db, { id: "nonexistent", name: "New Name" }, ctx, { getUnit });
@@ -32,7 +32,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when attempting to change SKU", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseDraftItem);
 
     const result = await run(db, { id: baseDraftItem.id, sku: "NEW-SKU" }, ctx, { getUnit });
@@ -43,7 +43,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when no fields to update", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseDraftItem);
 
     const result = await run(db, { id: baseDraftItem.id }, ctx, { getUnit });
@@ -54,7 +54,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when barcode is duplicate", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseDraftItem) // Item lookup
       .mockReturnValueOnce(baseActiveItem); // Barcode already taken
@@ -67,7 +67,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when changing UoM on non-DRAFT item", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseActiveItem);
 
     const result = await run(db, { id: baseActiveItem.id, unitId: "unit-g" }, ctx, { getUnit });
@@ -78,7 +78,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when changing UoM on INACTIVE item", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseInactiveItem);
 
     const result = await run(db, { id: baseInactiveItem.id, unitId: "unit-g" }, ctx, { getUnit });
@@ -89,7 +89,7 @@ describe("updateItem", () => {
   });
 
   it("returns error when new UoM does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseDraftItem) // Item lookup
       .mockReturnValueOnce(undefined); // UoM not found
@@ -104,7 +104,7 @@ describe("updateItem", () => {
   });
 
   it("updates name in any status", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedItem = { ...baseActiveItem, name: "Updated Name" };
     spies.select.mockReturnValueOnce(baseActiveItem);
     spies.update.mockReturnValue(updatedItem);
@@ -119,7 +119,7 @@ describe("updateItem", () => {
   });
 
   it("updates barcode in any status", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedItem = { ...baseActiveItem, barcode: "NEW-BAR" };
     spies.select
       .mockReturnValueOnce(baseActiveItem) // Item lookup
@@ -135,7 +135,7 @@ describe("updateItem", () => {
   });
 
   it("clears barcode by setting to null", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedItem = { ...baseActiveItem, barcode: null };
     spies.select.mockReturnValueOnce(baseActiveItem);
     spies.update.mockReturnValue(updatedItem);
@@ -149,7 +149,7 @@ describe("updateItem", () => {
   });
 
   it("updates UoM in DRAFT status", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activeUnit = { id: "unit-g", isActive: true };
     const updatedItem = { ...baseDraftItem, unitId: "unit-g" };
     spies.select
@@ -166,7 +166,7 @@ describe("updateItem", () => {
   });
 
   it("passes custom fields through to set", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedItem = { ...baseDraftItem, priority: 10 };
     spies.select.mockReturnValueOnce(baseDraftItem);
     spies.update.mockReturnValue(updatedItem);

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

@@ -1,5 +1,5 @@
-import { ok, err, type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { ok, err, type CommandContext } from "../../../shared";
+import { Transaction } from "../generated/kysely-tailordb";
 import {
   DuplicateBarcodeError,
   UnitNotFoundError,
@@ -8,7 +8,6 @@ import {
   SkuImmutableError,
   UomLockedError,
 } from "../lib/errors.generated";
-import { getItem } from "../query/getItem.generated";
 import { type PrimitivesQueries } from "../module";
 
 export interface UpdateItemInput {
@@ -26,7 +25,7 @@ export interface UpdateItemInput {
  * UoM can only be changed in DRAFT status.
  */
 export async function run<CF extends Record<string, unknown>>(
-  db: DB,
+  db: Transaction,
   input: UpdateItemInput & Partial<CF>,
   ctx: CommandContext,
   primitivesQueries?: Pick<PrimitivesQueries, "getUnit">,
@@ -34,7 +33,12 @@ export async function run<CF extends Record<string, unknown>>(
   const { id, sku, name, barcode, unitId, ...customFields } = input;
 
   // 1. Check item exists
-  const { item } = await getItem(db, { id }, ctx);
+  const item = await db
+    .selectFrom("Item")
+    .selectAll()
+    .where("id", "=", id)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (!item) {
     return err(new ItemNotFoundError(id));
@@ -62,6 +66,7 @@ export async function run<CF extends Record<string, unknown>>(
       .selectAll()
       .where("barcode", "=", barcode)
       .where("id", "!=", id)
+      .forUpdate()
       .executeTakeFirst();
 
     if (existingBarcode) {
@@ -76,7 +81,7 @@ export async function run<CF extends Record<string, unknown>>(
     }
 
     // Validate UoM exists and is active
-    const { unit } = await primitivesQueries!.getUnit(db, { id: unitId }, ctx);
+    const { unit } = (await primitivesQueries!.getUnit(db, { id: unitId }, ctx)).value;
 
     if (!unit?.isActive) {
       return err(new UnitNotFoundError(unitId));

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

@@ -1,5 +1,5 @@
 // @generated — do not edit
-import { defineCommand } from "../../shared/internal";
+import { defineCommand } from "../../../shared";
 import { permissions } from "../lib/permissions.generated";
 import { run } from "./updateTaxonomyNode";
 

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

@@ -1,7 +1,6 @@
 import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
-import { type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { createMockDb } from "../../../testing/index";
+import { Transaction } from "../generated/kysely-tailordb";
 import {
   CodeImmutableError,
   MissingRequiredFieldsError,
@@ -11,16 +10,11 @@ import { baseRootNode } from "../testing/fixtures";
 import { run } from "./updateTaxonomyNode";
 
 describe("updateTaxonomyNode", () => {
-  const ctx: CommandContext = {
-    actorId: "test-actor",
-    permissions: ["item-management:updateTaxonomyNode"],
-  };
-
   it("returns error when node does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValue(undefined);
 
-    const result = await run(db, { id: "nonexistent", name: "New Name" }, ctx);
+    const result = await run(db, { id: "nonexistent", name: "New Name" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(NodeNotFoundError);
@@ -28,10 +22,10 @@ describe("updateTaxonomyNode", () => {
   });
 
   it("returns error when attempting to change code", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseRootNode);
 
-    const result = await run(db, { id: baseRootNode.id, code: "NEW-CODE" }, ctx);
+    const result = await run(db, { id: baseRootNode.id, code: "NEW-CODE" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(CodeImmutableError);
@@ -39,10 +33,10 @@ describe("updateTaxonomyNode", () => {
   });
 
   it("returns error when name is empty", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseRootNode);
 
-    const result = await run(db, { id: baseRootNode.id, name: "" }, ctx);
+    const result = await run(db, { id: baseRootNode.id, name: "" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(MissingRequiredFieldsError);
@@ -50,10 +44,10 @@ describe("updateTaxonomyNode", () => {
   });
 
   it("returns error when name is not provided", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseRootNode);
 
-    const result = await run(db, { id: baseRootNode.id }, ctx);
+    const result = await run(db, { id: baseRootNode.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(MissingRequiredFieldsError);
@@ -61,12 +55,12 @@ describe("updateTaxonomyNode", () => {
   });
 
   it("updates node name", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedNode = { ...baseRootNode, name: "Consumer Electronics" };
     spies.select.mockReturnValueOnce(baseRootNode);
     spies.update.mockReturnValue(updatedNode);
 
-    const result = await run(db, { id: baseRootNode.id, name: "Consumer Electronics" }, ctx);
+    const result = await run(db, { id: baseRootNode.id, name: "Consumer Electronics" });
 
     expect(result.ok).toBe(true);
     if (result.ok) {
@@ -76,12 +70,12 @@ describe("updateTaxonomyNode", () => {
   });
 
   it("passes custom fields through to set", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const updatedNode = { ...baseRootNode, sortOrder: 5 };
     spies.select.mockReturnValueOnce(baseRootNode);
     spies.update.mockReturnValue(updatedNode);
 
-    await run(db, { id: baseRootNode.id, name: "Electronics", sortOrder: 5 }, ctx);
+    await run(db, { id: baseRootNode.id, name: "Electronics", sortOrder: 5 });
 
     expect(spies.set).toHaveBeenNthCalledWith(1, expect.objectContaining({ sortOrder: 5 }));
   });

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

@@ -1,11 +1,10 @@
-import { ok, err, type CommandContext } from "../../shared/internal";
-import { DB } from "../generated/kysely-tailordb";
+import { ok, err } from "../../../shared";
+import { Transaction } from "../generated/kysely-tailordb";
 import {
   CodeImmutableError,
   MissingRequiredFieldsError,
   NodeNotFoundError,
 } from "../lib/errors.generated";
-import { getTaxonomyNode } from "../query/getTaxonomyNode.generated";
 
 export interface UpdateTaxonomyNodeInput {
   id: string;
@@ -20,14 +19,18 @@ export interface UpdateTaxonomyNodeInput {
  * Node code is immutable.
  */
 export async function run<CF extends Record<string, unknown>>(
-  db: DB,
+  db: Transaction,
   input: UpdateTaxonomyNodeInput & Partial<CF>,
-  ctx: CommandContext,
 ) {
   const { id, code, name, ...customFields } = input;
 
   // 1. Check node exists
-  const { node } = await getTaxonomyNode(db, { id }, ctx);
+  const node = await db
+    .selectFrom("TaxonomyNode")
+    .selectAll()
+    .where("id", "=", id)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (!node) {
     return err(new NodeNotFoundError(id));

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

@@ -30,3 +30,11 @@ flowchart TD
 
 - **ITEM_NOT_FOUND**: Specified item ID does not exist
 - **INVALID_STATE_TRANSITION**: Item is not in DRAFT status
+
+## Test Cases
+
+- returns error when item does not exist
+- returns error when item is not DRAFT
+- returns error when item is INACTIVE
+- activates a DRAFT item
+- activates from custom status when from is overridden

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

@@ -36,3 +36,10 @@ flowchart TD
 - **ITEM_NOT_FOUND**: Specified item ID does not exist
 - **NODE_NOT_FOUND**: Specified taxonomy node ID does not exist
 - **DUPLICATE_ASSIGNMENT**: Item is already assigned to this taxonomy node
+
+## Test Cases
+
+- returns error when item does not exist
+- returns error when node does not exist
+- returns error when assignment already exists
+- creates assignment successfully

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

@@ -42,3 +42,13 @@ flowchart TD
 - **DUPLICATE_SKU**: An item with the same SKU already exists
 - **DUPLICATE_BARCODE**: An item with the same barcode already exists
 - **UNIT_NOT_FOUND**: Referenced unit does not exist or is inactive
+
+## Test Cases
+
+- returns error when SKU already exists
+- returns error when barcode already exists
+- returns error when UoM does not exist
+- creates item in DRAFT status by default
+- creates item in ACTIVE status
+- skips barcode uniqueness check when barcode is not provided
+- passes custom fields through to insert

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

@@ -42,3 +42,12 @@ flowchart TD
 - **DUPLICATE_NODE_CODE**: A node with the same code already exists
 - **PARENT_NODE_NOT_FOUND**: Specified parent node ID does not exist
 - **MAX_DEPTH_EXCEEDED**: Adding this child would exceed the configurable tree depth limit
+
+## Test Cases
+
+- returns error when code already exists
+- returns error when parent does not exist
+- returns error when max depth would be exceeded
+- creates a root node
+- creates a child node under existing parent
+- passes custom fields through to insert

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

@@ -32,3 +32,11 @@ flowchart TD
 
 - **ITEM_NOT_FOUND**: Specified item ID does not exist
 - **INVALID_STATE_TRANSITION**: Item is not in ACTIVE status
+
+## Test Cases
+
+- returns error when item does not exist
+- returns error when item is not ACTIVE
+- returns error when item is already INACTIVE
+- deactivates an ACTIVE item
+- deactivates from custom status when from is overridden

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

@@ -33,3 +33,10 @@ flowchart TD
 
 - **ITEM_NOT_FOUND**: Specified item ID does not exist
 - **DELETE_NON_DRAFT**: Item is in ACTIVE or INACTIVE status — only DRAFT items can be deleted
+
+## Test Cases
+
+- returns error when item does not exist
+- returns error when item is ACTIVE
+- returns error when item is INACTIVE
+- deletes a DRAFT item and removes taxonomy assignments

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

@@ -37,3 +37,10 @@ flowchart TD
 - **NODE_NOT_FOUND**: Specified node ID does not exist
 - **NODE_HAS_CHILDREN**: Node has one or more child nodes — children must be moved or deleted first
 - **NODE_HAS_ASSIGNMENTS**: Node has one or more item assignments — assignments must be removed first
+
+## Test Cases
+
+- returns error when node does not exist
+- returns error when node has children
+- returns error when node has item assignments
+- deletes a leaf node with no assignments

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

@@ -43,3 +43,13 @@ flowchart TD
 - **PARENT_NODE_NOT_FOUND**: Specified new parent node ID does not exist
 - **CIRCULAR_REFERENCE**: New parent is the node itself or one of its descendants
 - **MAX_DEPTH_EXCEEDED**: Moving this subtree would exceed the configurable tree depth limit
+
+## Test Cases
+
+- returns error when node does not exist
+- returns error when new parent does not exist
+- returns error when moving node under itself
+- returns error when moving node under its descendant
+- promotes node to root by setting parent to null
+- moves node to different parent
+- returns error when subtree depth + new parent depth exceeds maxDepth

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

@@ -32,3 +32,11 @@ flowchart TD
 
 - **ITEM_NOT_FOUND**: Specified item ID does not exist
 - **INVALID_STATE_TRANSITION**: Item is not in INACTIVE status
+
+## Test Cases
+
+- returns error when item does not exist
+- returns error when item is not INACTIVE
+- returns error when item is DRAFT
+- reactivates an INACTIVE item
+- reactivates from custom status when from is overridden

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

@@ -28,3 +28,8 @@ flowchart TD
 ## Error Scenarios
 
 - **ASSIGNMENT_NOT_FOUND**: No assignment exists for the specified item and taxonomy node combination
+
+## Test Cases
+
+- returns error when assignment does not exist
+- removes assignment successfully