@tailor-platform/erp-kit 0.2.0 → 0.2.2

package/src/generator/generate-code.ts

@@ -2,9 +2,14 @@ import fs from "node:fs";
 import path from "node:path";
 import { parseCommandDoc, errorCodeToClassName, type ParsedCommandDoc } from "./parse-command-doc";
 
-export function generateErrors(docs: ParsedCommandDoc[]): string {
+export function moduleNameToPrefix(moduleName: string): string {
+  return moduleName.toUpperCase().replace(/-/g, "_");
+}
+
+export function generateErrors(moduleName: string, docs: ParsedCommandDoc[]): string {
   const seen = new Set<string>();
   const lines: string[] = [];
+  const prefix = moduleNameToPrefix(moduleName);
 
   for (const doc of docs) {
     for (const error of doc.errors) {
@@ -12,8 +17,9 @@ export function generateErrors(docs: ParsedCommandDoc[]): string {
       seen.add(error.code);
 
       const className = errorCodeToClassName(error.code);
+      const prefixedCode = `${prefix}_${error.code}`;
       lines.push(`export const ${className} = createDomainError(`);
-      lines.push(`  "${className}", "${error.code}",`);
+      lines.push(`  "${className}", "${prefixedCode}",`);
       const escapedDesc = error.description.replace(/`/g, "'");
       lines.push(`  (identifier: string) => \`${escapedDesc}: \${identifier}\`,`);
       lines.push(`);`);
@@ -24,7 +30,7 @@ export function generateErrors(docs: ParsedCommandDoc[]): string {
   if (lines.length === 0) return "";
 
   return `// @generated — do not edit
-import { createDomainError } from "../../shared/internal";
+import { createDomainError } from "../../../shared";
 
 ${lines.join("\n")}`;
 }
@@ -33,14 +39,14 @@ export function generateCommandStub(doc: ParsedCommandDoc): string {
   const pascal = doc.commandName.charAt(0).toUpperCase() + doc.commandName.slice(1);
   const inputType = `${pascal}Input`;
 
-  return `import { ok, type CommandContext } from "../../shared/internal";
-import type { DB } from "../generated/kysely-tailordb";
+  return `import { ok, type CommandContext } from "../../../shared";
+import type { Transaction } from "../generated/kysely-tailordb";
 
 export interface ${inputType} {
   // TODO: define input fields
 }
 
-export async function run(db: DB, input: ${inputType}, ctx: CommandContext) {
+export async function run(db: Transaction, input: ${inputType}, ctx: CommandContext) {
   // TODO: implement
   return ok({});
 }
@@ -51,9 +57,9 @@ export function generateTestStub(doc: ParsedCommandDoc): string {
   const pascal = doc.commandName.charAt(0).toUpperCase() + doc.commandName.slice(1);
 
   return `import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
-import type { CommandContext } from "../../shared/internal";
-import type { DB } from "../generated/kysely-tailordb";
+import { createMockDb } from "../../../testing/index";
+import type { CommandContext } from "../../../shared";
+import type { Transaction } from "../generated/kysely-tailordb";
 import { run, ${pascal}Input } from "./${doc.commandName}";
 
 describe("${doc.commandName} - test scenario", () => {
@@ -63,7 +69,7 @@ describe("${doc.commandName} - test scenario", () => {
   };
 
   it("should be implemented", async () => {
-    const { db } = createMockDb<DB>();
+    const { db } = createMockDb<Transaction>();
     const result = await run(db, {} as ${pascal}Input, ctx);
     expect(result.ok).toBe(true);
   });
@@ -74,7 +80,7 @@ describe("${doc.commandName} - test scenario", () => {
 export function generateCommandShell(doc: ParsedCommandDoc): string {
   const lines: string[] = [
     `// @generated — do not edit`,
-    `import { defineCommand } from "../../shared/internal";`,
+    `import { defineCommand } from "../../../shared";`,
     `import { permissions } from "../lib/permissions.generated";`,
     `import { run } from "./${doc.commandName}";`,
     ``,
@@ -88,7 +94,7 @@ export function generateCommandShell(doc: ParsedCommandDoc): string {
 export function generateQueryShell(doc: ParsedCommandDoc): string {
   const lines: string[] = [
     `// @generated — do not edit`,
-    `import { defineQuery } from "../../shared/internal";`,
+    `import { defineQuery } from "../../../shared";`,
     `import { run } from "./${doc.commandName}";`,
     ``,
     `export const ${doc.commandName} = defineQuery(run);`,
@@ -102,7 +108,7 @@ export function generateQueryStub(doc: ParsedCommandDoc): string {
   const pascal = doc.commandName.charAt(0).toUpperCase() + doc.commandName.slice(1);
   const inputType = `${pascal}Input`;
 
-  return `import type { ReadonlyDB } from "../../shared/internal";
+  return `import type { ReadonlyDB } from "../../../shared";
 import type { DB } from "../generated/kysely-tailordb";
 
 export interface ${inputType} {
@@ -121,7 +127,7 @@ export function generateQueryTestStub(doc: ParsedCommandDoc): string {
   const inputType = `${pascal}Input`;
 
   return `import { describe, expect, it } from "vitest";
-import { createMockDb } from "../../testing/index";
+import { createMockDb } from "../../../testing/index";
 import type { DB } from "../generated/kysely-tailordb";
 import { run, type ${inputType} } from "./${doc.commandName}";
 
@@ -140,7 +146,7 @@ export function generatePermissions(moduleName: string, commandNames: string[]):
   const entries = sorted.map((name) => `  "${name}",`).join("\n");
 
   return `// @generated — do not edit
-import { definePermissions } from "../../shared/internal";
+import { definePermissions } from "../../../shared";
 
 export const { permissions, own, all } = definePermissions("${moduleName}", [
 ${entries}
@@ -186,7 +192,7 @@ export function runGenerateCode(modulePath: string, moduleName: string): number
 
   let generated = 0;
 
-  const errorsContent = generateErrors(allDocsForErrors);
+  const errorsContent = generateErrors(moduleName, allDocsForErrors);
   if (errorsContent) {
     fs.mkdirSync(libDir, { recursive: true });
     const errorsFile = path.join(libDir, "errors.generated.ts");

package/src/integration.test.ts

@@ -5,7 +5,7 @@ import path from "node:path";
 import { describe, it, expect } from "vitest";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const CLI_PATH = path.resolve(__dirname, "..", "dist", "cli.js");
+const CLI_PATH = path.resolve(__dirname, "..", "dist", "cli.mjs");
 const REPO_ROOT = path.resolve(__dirname, "..", "..", "..");
 
 const skipReason = fs.existsSync(CLI_PATH) ? undefined : "Run `pnpm build` first";

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,27 +33,10 @@ export {
   own as userManagementOwn,
   all as userManagementAll,
 } from "./modules/user-management/lib/permissions.generated";
-export { AuditEventEventType, UserStatus } from "./modules/user-management/generated/enums";
+// item-management
+export { defineModule as defineItemManagementModule } from "./modules/item-management/module";
 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";
+  permissions as itemManagementPermissions,
+  own as itemManagementOwn,
+  all as itemManagementAll,
+} from "./modules/item-management/lib/permissions.generated";

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