@tailor-platform/erp-kit 0.2.1 → 0.3.0

package/src/module.ts

@@ -1,17 +1,23 @@
-// shared/internal
-export { defineCommand, type Command } from "./modules/shared/defineCommand";
-export { defineQuery, type Query } from "./modules/shared/defineQuery";
-export { definePermissions } from "./modules/shared/definePermissions";
-export { requirePermission } from "./modules/shared/requirePermission";
-export { createDomainError, InsufficientPermissionError } from "./modules/shared/errors";
-export type { CommandContext, QueryContext, ReadonlyDB } from "./modules/shared/types";
+// shared
+export { defineCommand, type Command } from "./shared/defineCommand";
+export { defineQuery, type Query } from "./shared/defineQuery";
+export { definePermissions } from "./shared/definePermissions";
+export { requirePermission } from "./shared/requirePermission";
+export { createDomainError, InsufficientPermissionError } from "./shared/errors";
+export type {
+  CommandContext,
+  QueryContext,
+  ReadonlyDB,
+  PaginationInput,
+  PaginatedResult,
+} from "./shared/types";
 export type {
   InferSchema,
   Selectable,
   Insertable,
   Updateable,
   FieldsToInsertable,
-} from "./modules/shared/entityTypes";
+} from "./shared/entityTypes";
 
 // primitives
 export { defineModule as definePrimitivesModule } from "./modules/primitives/module";
@@ -20,40 +26,6 @@ export {
   own as primitivesOwn,
   all as primitivesAll,
 } from "./modules/primitives/lib/permissions.generated";
-export {
-  UomCategoryNotFoundError,
-  UnitNotFoundError,
-  IncompatibleUnitsError,
-  InactiveUnitError,
-  CurrencyNotFoundError,
-  InactiveCurrencyError,
-  ExchangeRateNotFoundError,
-  CannotDeactivateReferenceUnitError,
-  CategoryNotActiveError,
-  DuplicateUnitSymbolError,
-  InvalidConversionFactorError,
-  InvalidRoundingPrecisionError,
-  DuplicateCategoryNameError,
-  CategoryHasActiveUnitsError,
-  UnitNotInCategoryError,
-  InvalidIsoCodeError,
-  DuplicateCurrencyCodeError,
-  InvalidDecimalPlacesError,
-  CannotDeactivateBaseCurrencyError,
-  CannotSetInactiveAsBaseCurrencyError,
-  SameCurrencyPairError,
-  InvalidExchangeRateError,
-} from "./modules/primitives/lib/errors.generated";
-export { type ConvertQuantityInput } from "./modules/primitives/query/convertQuantity";
-export { type ActivateCategoryInput } from "./modules/primitives/command/activateCategory";
-export { type DeactivateCategoryInput } from "./modules/primitives/command/deactivateCategory";
-export { type SetReferenceUnitInput } from "./modules/primitives/command/setReferenceUnit";
-export { type ActivateUnitInput } from "./modules/primitives/command/activateUnit";
-export { type DeactivateUnitInput } from "./modules/primitives/command/deactivateUnit";
-export { type ConvertAmountInput } from "./modules/primitives/query/convertAmount";
-export { type ActivateCurrencyInput } from "./modules/primitives/command/activateCurrency";
-export { type DeactivateCurrencyInput } from "./modules/primitives/command/deactivateCurrency";
-export { type SetBaseCurrencyInput } from "./modules/primitives/command/setBaseCurrency";
 // user-management
 export { defineModule as defineUserManagementModule } from "./modules/user-management/module";
 export {
@@ -61,30 +33,6 @@ export {
   own as userManagementOwn,
   all as userManagementAll,
 } from "./modules/user-management/lib/permissions.generated";
-export { AuditEventEventType, UserStatus } from "./modules/user-management/generated/enums";
-export {
-  UserNotFoundError,
-  UserAlreadyExistsError,
-  UserNotActiveError,
-  InvalidEmailError,
-  MissingRequiredFieldError,
-  InvalidStatusTransitionError,
-  PermissionNotFoundError,
-  DuplicatePermissionKeyError,
-  InvalidPermissionKeyFormatError,
-  RoleNotFoundError,
-  DuplicateRoleNameError,
-  AssignmentNotFoundError,
-  InvalidEventTypeError,
-} from "./modules/user-management/lib/errors.generated";
-export { type ActivateUserInput } from "./modules/user-management/command/activateUser";
-export { type DeactivateUserInput } from "./modules/user-management/command/deactivateUser";
-export { type ReactivateUserInput } from "./modules/user-management/command/reactivateUser";
-export { type AssignPermissionToRoleInput } from "./modules/user-management/command/assignPermissionToRole";
-export { type RevokePermissionFromRoleInput } from "./modules/user-management/command/revokePermissionFromRole";
-export { type AssignRoleToUserInput } from "./modules/user-management/command/assignRoleToUser";
-export { type RevokeRoleFromUserInput } from "./modules/user-management/command/revokeRoleFromUser";
-export { type LogAuditEventInput } from "./modules/user-management/command/logAuditEvent";
 // item-management
 export { defineModule as defineItemManagementModule } from "./modules/item-management/module";
 export {
@@ -92,34 +40,3 @@ export {
   own as itemManagementOwn,
   all as itemManagementAll,
 } from "./modules/item-management/lib/permissions.generated";
-export { ItemStatus } from "./modules/item-management/generated/enums";
-export {
-  ItemNotFoundError,
-  DuplicateSkuError,
-  DuplicateBarcodeError,
-  UnitNotFoundError as ItemUnitNotFoundError,
-  SkuImmutableError,
-  UomLockedError,
-  NoFieldsToUpdateError,
-  InvalidStateTransitionError,
-  DeleteNonDraftError,
-  NodeNotFoundError,
-  DuplicateNodeCodeError,
-  ParentNodeNotFoundError,
-  MaxDepthExceededError,
-  CodeImmutableError,
-  MissingRequiredFieldsError,
-  CircularReferenceError,
-  NodeHasChildrenError,
-  NodeHasAssignmentsError,
-  DuplicateAssignmentError,
-  AssignmentNotFoundError as ItemAssignmentNotFoundError,
-} from "./modules/item-management/lib/errors.generated";
-export { type GetItemInput } from "./modules/item-management/query/getItem";
-export { type GetTaxonomyNodeInput } from "./modules/item-management/query/getTaxonomyNode";
-export { type CreateItemInput } from "./modules/item-management/command/createItem";
-export { type UpdateItemInput } from "./modules/item-management/command/updateItem";
-export { type ActivateItemInput } from "./modules/item-management/command/activateItem";
-export { type DeactivateItemInput } from "./modules/item-management/command/deactivateItem";
-export { type CreateTaxonomyNodeInput } from "./modules/item-management/command/createTaxonomyNode";
-export { type AssignItemToTaxonomyInput } from "./modules/item-management/command/assignItemToTaxonomy";

package/src/modules/item-management/README.md

@@ -33,6 +33,14 @@ Each item has its own lifecycle (DRAFT → ACTIVE ↔ INACTIVE), unique SKU iden
 - Bill of Materials (manufacturing module)
 - Product images and digital asset management
 
+### Scope Decision Rationale
+
+Item-management owns the **SKU-level transactional entity** — the unit that all downstream modules (sales, purchasing, inventory) actually reference. This is separated from product-management because a product template is a design-time concept, while an item is a runtime/operational entity with its own lifecycle.
+
+Product templates and variant generation stay in product-management because they represent the commercial catalog structure. Items can be created independently (without a product template) or generated from variants, so item-management must not depend on product-management's schema.
+
+Inventory, pricing, and supplier data are excluded because they attach to items but follow their own business rules, update frequencies, and access patterns. Taxonomy is included here (separate from product-management's categories) because item classification serves operational needs (warehouse organization, procurement grouping) that differ from commercial categorization.
+
 ## Module Dependencies
 
 - [primitives](../primitives/README.md) — UoM definitions for item unit of measure

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

package/src/modules/item-management/command/activateItem.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 "./activateItem";
 
 describe("activateItem", () => {
-  const ctx: CommandContext = {
-    actorId: "test-actor",
-    permissions: ["item-management:activateItem"],
-  };
-
   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("activateItem", () => {
   });
 
   it("returns error when item is not DRAFT", 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("activateItem", () => {
   });
 
   it("returns error when item is INACTIVE", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseInactiveItem);
 
-    const result = await run(db, { id: baseInactiveItem.id }, ctx);
+    const result = await run(db, { id: baseInactiveItem.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(InvalidStateTransitionError);
@@ -46,12 +40,12 @@ describe("activateItem", () => {
   });
 
   it("activates a DRAFT item", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activatedItem = { ...baseDraftItem, status: "ACTIVE" };
     spies.select.mockReturnValueOnce(baseDraftItem);
     spies.update.mockReturnValue(activatedItem);
 
-    const result = await run(db, { id: baseDraftItem.id }, ctx);
+    const result = await run(db, { id: baseDraftItem.id });
 
     expect(result.ok).toBe(true);
     if (result.ok) {
@@ -61,12 +55,12 @@ describe("activateItem", () => {
   });
 
   it("activates from custom status when from is overridden", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activatedItem = { ...baseInactiveItem, status: "ACTIVE" };
     spies.select.mockReturnValueOnce(baseInactiveItem);
     spies.update.mockReturnValue(activatedItem);
 
-    const result = await run(db, { id: baseInactiveItem.id, from: ["DRAFT", "INACTIVE"] }, ctx);
+    const result = await run(db, { id: baseInactiveItem.id, from: ["DRAFT", "INACTIVE"] });
 
     expect(result.ok).toBe(true);
     if (result.ok) {

package/src/modules/item-management/command/activateItem.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 ActivateItemInput {
   id: string;
@@ -14,11 +13,16 @@ export interface ActivateItemInput {
  * Transitions an item from DRAFT to ACTIVE status.
  * Requires name, SKU, and UoM to be set.
  */
-export async function run(db: DB, input: ActivateItemInput, ctx: CommandContext) {
+export async function run(db: Transaction, input: ActivateItemInput) {
   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/assignItemToTaxonomy.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 "./assignItemToTaxonomy";
 

package/src/modules/item-management/command/assignItemToTaxonomy.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 {
   DuplicateAssignmentError,
   ItemNotFoundError,
@@ -11,16 +10,11 @@ import { baseActiveItem, baseAssignment, baseChildNode } from "../testing/fixtur
 import { run } from "./assignItemToTaxonomy";
 
 describe("assignItemToTaxonomy", () => {
-  const ctx: CommandContext = {
-    actorId: "test-actor",
-    permissions: ["item-management:assignItemToTaxonomy"],
-  };
-
   it("returns error when item does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined); // Item not found
 
-    const result = await run(db, { itemId: "nonexistent", taxonomyNodeId: baseChildNode.id }, ctx);
+    const result = await run(db, { itemId: "nonexistent", taxonomyNodeId: baseChildNode.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(ItemNotFoundError);
@@ -28,12 +22,12 @@ describe("assignItemToTaxonomy", () => {
   });
 
   it("returns error when node does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseActiveItem) // Item exists
       .mockReturnValueOnce(undefined); // Node not found
 
-    const result = await run(db, { itemId: baseActiveItem.id, taxonomyNodeId: "nonexistent" }, ctx);
+    const result = await run(db, { itemId: baseActiveItem.id, taxonomyNodeId: "nonexistent" });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(NodeNotFoundError);
@@ -41,17 +35,13 @@ describe("assignItemToTaxonomy", () => {
   });
 
   it("returns error when assignment already exists", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseActiveItem) // Item exists
       .mockReturnValueOnce(baseChildNode) // Node exists
       .mockReturnValueOnce(baseAssignment); // Assignment exists
 
-    const result = await run(
-      db,
-      { itemId: baseActiveItem.id, taxonomyNodeId: baseChildNode.id },
-      ctx,
-    );
+    const result = await run(db, { itemId: baseActiveItem.id, taxonomyNodeId: baseChildNode.id });
     expect(result.ok).toBe(false);
     if (!result.ok) {
       expect(result.error).toBeInstanceOf(DuplicateAssignmentError);
@@ -59,7 +49,7 @@ describe("assignItemToTaxonomy", () => {
   });
 
   it("creates assignment successfully", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const createdAssignment = {
       ...baseAssignment,
       id: "new-assignment-id",
@@ -72,11 +62,7 @@ describe("assignItemToTaxonomy", () => {
       .mockReturnValueOnce(undefined); // No existing assignment
     spies.insert.mockReturnValue(createdAssignment);
 
-    const result = await run(
-      db,
-      { itemId: baseActiveItem.id, taxonomyNodeId: baseChildNode.id },
-      ctx,
-    );
+    const result = await run(db, { itemId: baseActiveItem.id, taxonomyNodeId: baseChildNode.id });
 
     expect(result.ok).toBe(true);
     if (result.ok) {

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

@@ -1,13 +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 {
   DuplicateAssignmentError,
   ItemNotFoundError,
   NodeNotFoundError,
 } from "../lib/errors.generated";
-import { getItem } from "../query/getItem.generated";
-import { getTaxonomyNode } from "../query/getTaxonomyNode.generated";
-import { getItemTaxonomyAssignment } from "../query/getItemTaxonomyAssignment.generated";
 
 export interface AssignItemToTaxonomyInput {
   itemId: string;
@@ -19,29 +16,35 @@ export interface AssignItemToTaxonomyInput {
  *
  * Creates a link between an item and a taxonomy node for many-to-many classification.
  */
-export async function run(db: DB, input: AssignItemToTaxonomyInput, ctx: CommandContext) {
+export async function run(db: Transaction, input: AssignItemToTaxonomyInput) {
   const { itemId, taxonomyNodeId } = input;
 
-  // 1. Check item exists
-  const { item } = await getItem(db, { id: itemId }, ctx);
+  // 1. Check item exists (validation only — not writing to Item)
+  const item = await db.selectFrom("Item").selectAll().where("id", "=", itemId).executeTakeFirst();
 
   if (!item) {
     return err(new ItemNotFoundError(itemId));
   }
 
-  // 2. Check node exists
-  const { node } = await getTaxonomyNode(db, { id: taxonomyNodeId }, ctx);
+  // 2. Check node exists (validation only — not writing to TaxonomyNode)
+  const node = await db
+    .selectFrom("TaxonomyNode")
+    .selectAll()
+    .where("id", "=", taxonomyNodeId)
+    .executeTakeFirst();
 
   if (!node) {
     return err(new NodeNotFoundError(taxonomyNodeId));
   }
 
-  // 3. Check for duplicate assignment
-  const { assignment: existing } = await getItemTaxonomyAssignment(
-    db,
-    { itemId, taxonomyNodeId },
-    ctx,
-  );
+  // 3. Check for duplicate assignment (uniqueness before insert — lock)
+  const existing = await db
+    .selectFrom("ItemTaxonomyAssignment")
+    .selectAll()
+    .where("itemId", "=", itemId)
+    .where("taxonomyNodeId", "=", taxonomyNodeId)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (existing) {
     return err(new DuplicateAssignmentError(`${itemId} in ${taxonomyNodeId}`));

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

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

@@ -1,14 +1,14 @@
 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,
   DuplicateSkuError,
   UnitNotFoundError,
 } from "../lib/errors.generated";
 import { baseActiveItem, baseDraftItem } from "../testing/fixtures";
-import { getUnit } from "../../primitives";
+import { getUnit } from "../../primitives/query/getUnit.generated";
 import { run } from "./createItem";
 
 describe("createItem", () => {
@@ -18,7 +18,7 @@ describe("createItem", () => {
   };
 
   it("returns error when SKU already exists", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseDraftItem); // SKU lookup finds existing
 
     const result = await run(db, { sku: "SKU-001", name: "Widget", unitId: "unit-kg" }, ctx, {
@@ -31,7 +31,7 @@ describe("createItem", () => {
   });
 
   it("returns error when barcode already exists", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(undefined) // SKU unique
       .mockReturnValueOnce(baseActiveItem); // Barcode lookup finds existing
@@ -49,7 +49,7 @@ describe("createItem", () => {
   });
 
   it("returns error when UoM does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(undefined) // SKU unique
       .mockReturnValueOnce(undefined); // UoM lookup fails
@@ -67,7 +67,7 @@ describe("createItem", () => {
   });
 
   it("creates item in DRAFT status by default", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activeUnit = { id: "unit-kg", isActive: true };
     const createdItem = { ...baseDraftItem, id: "new-item-id", sku: "SKU-NEW" };
 
@@ -89,7 +89,7 @@ describe("createItem", () => {
   });
 
   it("creates item in ACTIVE status", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activeUnit = { id: "unit-kg", isActive: true };
     const createdItem = { ...baseActiveItem, id: "new-item-id", sku: "SKU-NEW" };
 
@@ -113,7 +113,7 @@ describe("createItem", () => {
   });
 
   it("skips barcode uniqueness check when barcode is not provided", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activeUnit = { id: "unit-kg", isActive: true };
     const createdItem = { ...baseDraftItem, id: "new-item-id" };
 
@@ -134,7 +134,7 @@ describe("createItem", () => {
   });
 
   it("passes custom fields through to insert", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const activeUnit = { id: "unit-kg", isActive: true };
     const createdItem = { ...baseDraftItem, id: "new-item-id", priority: 5 };
 

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

@@ -1,11 +1,10 @@
-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,
   DuplicateSkuError,
   UnitNotFoundError,
 } from "../lib/errors.generated";
-import { getItem } from "../query/getItem.generated";
 import { type PrimitivesQueries } from "../module";
 
 export interface CreateItemInput {
@@ -22,7 +21,7 @@ export interface CreateItemInput {
  * DRAFT items allow review before activation; ACTIVE items are immediately available.
  */
 export async function run<CF extends Record<string, unknown>>(
-  db: DB,
+  db: Transaction,
   input: CreateItemInput & CF,
   ctx: CommandContext,
   primitivesQueries?: Pick<PrimitivesQueries, "getUnit">,
@@ -30,7 +29,12 @@ export async function run<CF extends Record<string, unknown>>(
   const { sku, name, barcode, unitId, status = "DRAFT", ...customFields } = input;
 
   // 1. Check SKU uniqueness
-  const { item: existingSku } = await getItem(db, { sku }, ctx);
+  const existingSku = await db
+    .selectFrom("Item")
+    .selectAll()
+    .where("sku", "=", sku)
+    .forUpdate()
+    .executeTakeFirst();
 
   if (existingSku) {
     return err(new DuplicateSkuError(sku));
@@ -38,7 +42,12 @@ export async function run<CF extends Record<string, unknown>>(
 
   // 2. Check barcode uniqueness (when provided)
   if (barcode) {
-    const { item: existingBarcode } = await getItem(db, { barcode }, ctx);
+    const existingBarcode = await db
+      .selectFrom("Item")
+      .selectAll()
+      .where("barcode", "=", barcode)
+      .forUpdate()
+      .executeTakeFirst();
 
     if (existingBarcode) {
       return err(new DuplicateBarcodeError(barcode));
@@ -46,7 +55,7 @@ export async function run<CF extends Record<string, unknown>>(
   }
 
   // 3. 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/createTaxonomyNode.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 "./createTaxonomyNode";
 

package/src/modules/item-management/command/createTaxonomyNode.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 {
   DuplicateNodeCodeError,
   MaxDepthExceededError,
@@ -17,7 +17,7 @@ describe("createTaxonomyNode", () => {
   };
 
   it("returns error when code already exists", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseRootNode); // Code lookup finds existing
 
     const result = await run(db, { code: "ELECTRONICS", name: "Electronics" }, ctx, {
@@ -30,7 +30,7 @@ describe("createTaxonomyNode", () => {
   });
 
   it("returns error when parent does not exist", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(undefined) // Code unique
       .mockReturnValueOnce(undefined); // Parent not found
@@ -48,7 +48,7 @@ describe("createTaxonomyNode", () => {
   });
 
   it("returns error when max depth would be exceeded", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(undefined) // getTaxonomyNode: code unique
       .mockReturnValueOnce(baseLeafNode) // getTaxonomyNode: parent exists (node-3)
@@ -70,7 +70,7 @@ describe("createTaxonomyNode", () => {
   });
 
   it("creates a root node", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const createdNode = { ...baseRootNode, id: "new-node-id", code: "CLOTHING" };
     spies.select.mockReturnValueOnce(undefined); // Code unique
     spies.insert.mockReturnValue(createdNode);
@@ -86,7 +86,7 @@ describe("createTaxonomyNode", () => {
   });
 
   it("creates a child node under existing parent", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const createdNode = {
       ...baseChildNode,
       id: "new-node-id",
@@ -114,7 +114,7 @@ describe("createTaxonomyNode", () => {
   });
 
   it("passes custom fields through to insert", async () => {
-    const { db, spies } = createMockDb<DB>();
+    const { db, spies } = createMockDb<Transaction>();
     const createdNode = { ...baseRootNode, id: "new-node-id", sortOrder: 1 };
     spies.select.mockReturnValueOnce(undefined); // Code unique
     spies.insert.mockReturnValue(createdNode);