@tailor-platform/erp-kit 0.2.0 → 0.2.2

package/src/modules/item-management/command/moveTaxonomyNode.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 {
   CircularReferenceError,
   MaxDepthExceededError,
@@ -18,7 +18,7 @@ describe("moveTaxonomyNode", () => {
   };
 
   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", newParentId: null }, ctx, { maxDepth: 10 });
@@ -29,7 +29,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("returns error when new parent does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseChildNode) // getTaxonomyNode for node
       .mockReturnValueOnce(undefined); // getTaxonomyNode for new parent
@@ -44,7 +44,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("returns error when moving node under itself", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseRootNode) // getTaxonomyNode for node
       .mockReturnValueOnce(baseRootNode); // getTaxonomyNode for new parent
@@ -59,7 +59,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("returns error when moving node under its descendant", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseRootNode) // getTaxonomyNode for node (node-1)
       .mockReturnValueOnce(baseLeafNode) // getTaxonomyNode for new parent (node-3)
@@ -76,7 +76,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("promotes node to root by setting parent to null", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const movedNode = { ...baseChildNode, parentId: null };
     spies.select.mockReturnValueOnce(baseChildNode); // getTaxonomyNode for node
     spies.update.mockReturnValue(movedNode);
@@ -93,7 +93,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("moves node to different parent", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const movedNode = { ...baseChildNode, parentId: secondRootNode.id };
     spies.select
       .mockReturnValueOnce(baseChildNode) // getTaxonomyNode for node (node-2)
@@ -115,7 +115,7 @@ describe("moveTaxonomyNode", () => {
   });
 
   it("returns error when subtree depth + new parent depth exceeds maxDepth", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseChildNode) // getTaxonomyNode for node (node-2)
       .mockReturnValueOnce(secondRootNode) // getTaxonomyNode for new parent (node-4)

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

@@ -1,15 +1,11 @@
-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 {
   CircularReferenceError,
   MaxDepthExceededError,
   NodeNotFoundError,
   ParentNodeNotFoundError,
 } from "../lib/errors.generated";
-import { getTaxonomyNode } from "../query/getTaxonomyNode.generated";
-import { detectCircularReference } from "../query/detectCircularReference.generated";
-import { calculateNodeDepth } from "../query/calculateNodeDepth.generated";
-import { calculateSubtreeDepth } from "../query/calculateSubtreeDepth.generated";
 
 export interface MoveTaxonomyNodeInput {
   id: string;
@@ -23,16 +19,21 @@ export interface MoveTaxonomyNodeInput {
  * Validates circular references and depth limits.
  */
 export async function run(
-  db: DB,
+  db: Transaction,
   input: MoveTaxonomyNodeInput,
-  ctx: CommandContext,
+  _ctx: CommandContext,
   config?: { maxDepth: number },
 ) {
   const { maxDepth } = config!;
   const { id, newParentId } = input;
 
-  // 1. Check node exists
-  const { node } = await getTaxonomyNode(db, { id }, ctx);
+  // 1. Check node exists (read-then-update — lock)
+  const node = await db
+    .selectFrom("TaxonomyNode")
+    .selectAll()
+    .where("id", "=", id)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (!node) {
     return err(new NodeNotFoundError(id));
@@ -50,23 +51,66 @@ export async function run(
     return ok({ node: movedNode! });
   }
 
-  // 3. Check new parent exists
-  const { node: newParent } = await getTaxonomyNode(db, { id: newParentId }, ctx);
+  // 3. Check new parent exists (validation only — no lock)
+  const newParent = await db
+    .selectFrom("TaxonomyNode")
+    .selectAll()
+    .where("id", "=", newParentId)
+    .executeTakeFirst();
 
   if (!newParent) {
     return err(new ParentNodeNotFoundError(newParentId));
   }
 
-  // 4. Check circular reference
-  const { isCircular } = await detectCircularReference(db, { nodeId: id, newParentId }, ctx);
-
-  if (isCircular) {
+  // 4. Check circular reference — walk up from new parent to root
+  if (newParentId === id) {
     return err(new CircularReferenceError(`${id} -> ${newParentId}`));
   }
+  let ancestorId: string | null = newParentId;
+  while (ancestorId) {
+    const ancestor = await db
+      .selectFrom("TaxonomyNode")
+      .selectAll()
+      .where("id", "=", ancestorId)
+      .executeTakeFirst();
+    if (!ancestor) break;
+    ancestorId = ancestor.parentId;
+    if (ancestorId === id) {
+      return err(new CircularReferenceError(`${id} -> ${newParentId}`));
+    }
+  }
+
+  // 5. Check depth limit: calculate new parent depth
+  let newParentDepth = 1;
+  let currentId: string | null = newParentId;
+  while (currentId) {
+    const n = await db
+      .selectFrom("TaxonomyNode")
+      .selectAll()
+      .where("id", "=", currentId)
+      .executeTakeFirst();
+    if (!n?.parentId) break;
+    newParentDepth++;
+    currentId = n.parentId;
+  }
+
+  // Calculate subtree depth of moved node
+  async function getSubtreeDepth(nodeId: string): Promise<number> {
+    const children = await db
+      .selectFrom("TaxonomyNode")
+      .selectAll()
+      .where("parentId", "=", nodeId)
+      .execute();
+    if (!children || children.length === 0) return 1;
+    let maxChildDepth = 0;
+    for (const child of children) {
+      const childDepth = await getSubtreeDepth(child.id);
+      if (childDepth > maxChildDepth) maxChildDepth = childDepth;
+    }
+    return 1 + maxChildDepth;
+  }
 
-  // 5. Check depth limit: new parent depth + subtree depth of moved node
-  const { depth: newParentDepth } = await calculateNodeDepth(db, { nodeId: newParentId }, ctx);
-  const { depth: subtreeDepth } = await calculateSubtreeDepth(db, { nodeId: id }, ctx);
+  const subtreeDepth = await getSubtreeDepth(id);
   const totalDepth = newParentDepth + subtreeDepth;
 
   if (totalDepth > maxDepth) {

package/src/modules/item-management/command/reactivateItem.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 "./reactivateItem";
 

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