@tailor-platform/erp-kit 0.7.0 → 0.9.0

package/src/modules/product-management/docs/commands/UpdateProductAttribute.md

@@ -2,12 +2,11 @@
 
 ## Overview
 
-updateProductAttribute modifies the display name of an existing product attribute. In extensible module configurations, consumer-defined custom fields may also be updated through this command. Attribute code is immutable through this command.
+updateProductAttribute modifies the display name of an existing product attribute. The attribute can be looked up by `id` or `code`. In extensible module configurations, consumer-defined custom fields may also be updated through this command.
 
 ## Business Rules
 
-- Target attribute must exist
-- Attribute code is immutable — cannot be changed
+- Target attribute must exist (looked up by `id` or `code`)
 - At least one updatable field must be provided
 - Name must remain non-empty when provided
 - Consumer-defined custom fields may be updated
@@ -16,14 +15,12 @@ updateProductAttribute modifies the display name of an existing product attribut
 
 ```mermaid
 flowchart TD
-    A[Receive update request] --> B{Attribute exists?}
-    B -->|No| C[Return error: ATTRIBUTE_NOT_FOUND]
-    B -->|Yes| D{Code change attempted?}
-    D -->|Yes| E[Return error: CODE_IMMUTABLE]
-    D -->|No| F{Any updatable fields provided?}
-    F -->|No| G[Return error: MISSING_REQUIRED_FIELD]
-    F -->|Yes| H[Apply name and/or custom field updates]
-    H --> I[Return updated attribute]
+    A[Receive update request] --> B{Lookup by id or code}
+    B -->|Not found| C[Return error: ATTRIBUTE_NOT_FOUND]
+    B -->|Found| D{Any updatable fields provided?}
+    D -->|No| E[Return error: MISSING_REQUIRED_FIELD]
+    D -->|Yes| F[Apply name and/or custom field updates]
+    F --> G[Return updated attribute]
 ```
 
 ## External Dependencies
@@ -32,15 +29,14 @@ flowchart TD
 
 ## Error Scenarios
 
-- **ATTRIBUTE_NOT_FOUND**: Specified attribute ID does not exist
-- **CODE_IMMUTABLE**: Attempt to change the attribute code
+- **ATTRIBUTE_NOT_FOUND**: Specified attribute does not exist
 - **MISSING_REQUIRED_FIELD**: No updatable fields were provided, or name is empty when provided without any custom field updates
 
 ## Test Cases
 
 - returns error when attribute not found
-- returns error when trying to change code
 - returns error when no fields provided
 - returns error when name is empty string
 - passes custom fields through to update
 - updates name successfully
+- looks up attribute by code and updates name

package/src/modules/product-management/docs/commands/UpdateProductCategory.md

@@ -2,12 +2,11 @@
 
 ## Overview
 
-updateProductCategory modifies the display name of an existing product category. In extensible module configurations, consumer-defined custom fields may also be updated through this command. Category code and parent reference are immutable through this command — use moveProductCategory to change the parent.
+updateProductCategory modifies the display name of an existing product category. The category can be looked up by `id` or `code`. In extensible module configurations, consumer-defined custom fields may also be updated through this command. Use moveProductCategory to change the parent.
 
 ## Business Rules
 
-- Target category must exist
-- Category code is immutable — cannot be changed
+- Target category must exist (looked up by `id` or `code`)
 - At least one updatable field must be provided
 - Name must remain non-empty when provided
 - Consumer-defined custom fields may be updated
@@ -17,14 +16,12 @@ updateProductCategory modifies the display name of an existing product category.
 
 ```mermaid
 flowchart TD
-    A[Receive update request] --> B{Category exists?}
-    B -->|No| C[Return error: CATEGORY_NOT_FOUND]
-    B -->|Yes| D{Code change attempted?}
-    D -->|Yes| E[Return error: CODE_IMMUTABLE]
-    D -->|No| F{Any updatable fields provided?}
-    F -->|No| G[Return error: MISSING_REQUIRED_FIELD]
-    F -->|Yes| H[Apply name and/or custom field updates]
-    H --> I[Return updated category]
+    A[Receive update request] --> B{Lookup by id or code}
+    B -->|Not found| C[Return error: CATEGORY_NOT_FOUND]
+    B -->|Found| D{Any updatable fields provided?}
+    D -->|No| E[Return error: MISSING_REQUIRED_FIELD]
+    D -->|Yes| F[Apply name and/or custom field updates]
+    F --> G[Return updated category]
 ```
 
 ## External Dependencies
@@ -33,14 +30,13 @@ flowchart TD
 
 ## Error Scenarios
 
-- **CATEGORY_NOT_FOUND**: Specified category ID does not exist
-- **CODE_IMMUTABLE**: Attempt to change the category code
+- **CATEGORY_NOT_FOUND**: Specified category does not exist
 - **MISSING_REQUIRED_FIELD**: No updatable fields were provided, or name is empty when provided without any custom field updates
 
 ## Test Cases
 
 - should return CategoryNotFoundError when category does not exist
-- should return CodeImmutableError when code is provided
 - should return MissingRequiredFieldError when no fields are provided
 - should pass custom fields through to update
 - should update category name successfully
+- should look up category by code and update name

package/src/modules/product-management/module.ts

@@ -110,6 +110,7 @@ export interface DefineModuleParams<
   };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <
   const PF extends Record<string, TailorAnyDBField> = EmptyFields,
   const PAF extends Record<string, TailorAnyDBField> = EmptyFields,

package/src/modules/purchase/command/activatePurchasePaymentTerm.test.ts

@@ -13,9 +13,9 @@ describe("activatePurchasePaymentTerm", () => {
     spies.update.mockReturnValueOnce({ ...inactivePaymentTerm, status: "ACTIVE" });
 
     const result = await run(db, { id: inactivePaymentTerm.id }, commandCtx);
-    const value = expectOk<{ paymentTerm: typeof inactivePaymentTerm }>(result);
+    const value = expectOk<{ purchasePaymentTerm: typeof inactivePaymentTerm }>(result);
 
-    expect(value.paymentTerm.status).toBe("ACTIVE");
+    expect(value.purchasePaymentTerm.status).toBe("ACTIVE");
     expect(spies.set).toHaveBeenCalledWith(expect.objectContaining({ status: "ACTIVE" }));
   });
 
@@ -24,9 +24,9 @@ describe("activatePurchasePaymentTerm", () => {
     spies.select.mockReturnValueOnce(basePaymentTerm);
 
     const result = await run(db, { id: basePaymentTerm.id }, commandCtx);
-    const value = expectOk<{ paymentTerm: typeof basePaymentTerm }>(result);
+    const value = expectOk<{ purchasePaymentTerm: typeof basePaymentTerm }>(result);
 
-    expect(value.paymentTerm).toEqual(basePaymentTerm);
+    expect(value.purchasePaymentTerm).toEqual(basePaymentTerm);
     expect(spies.update).not.toHaveBeenCalled();
   });
 

package/src/modules/purchase/command/activatePurchasePaymentTerm.ts

@@ -18,7 +18,7 @@ export async function run(
     return err(new PaymentTermNotFoundError(input.id));
   }
   if (paymentTerm.status === "ACTIVE") {
-    return ok({ paymentTerm });
+    return ok({ purchasePaymentTerm: paymentTerm });
   }
 
   const validation = validateScheduleLines(
@@ -35,5 +35,5 @@ export async function run(
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ paymentTerm: activated });
+  return ok({ purchasePaymentTerm: activated });
 }

package/src/modules/purchase/command/deactivatePurchasePaymentTerm.test.ts

@@ -13,9 +13,9 @@ describe("deactivatePurchasePaymentTerm", () => {
     spies.update.mockReturnValueOnce({ ...basePaymentTerm, status: "INACTIVE" });
 
     const result = await run(db, { id: basePaymentTerm.id }, commandCtx);
-    const value = expectOk<{ paymentTerm: typeof basePaymentTerm }>(result);
+    const value = expectOk<{ purchasePaymentTerm: typeof basePaymentTerm }>(result);
 
-    expect(value.paymentTerm.status).toBe("INACTIVE");
+    expect(value.purchasePaymentTerm.status).toBe("INACTIVE");
   });
 
   it("returns success when the payment term is already inactive", async () => {
@@ -23,9 +23,9 @@ describe("deactivatePurchasePaymentTerm", () => {
     spies.select.mockReturnValueOnce(inactivePaymentTerm);
 
     const result = await run(db, { id: inactivePaymentTerm.id }, commandCtx);
-    const value = expectOk<{ paymentTerm: typeof inactivePaymentTerm }>(result);
+    const value = expectOk<{ purchasePaymentTerm: typeof inactivePaymentTerm }>(result);
 
-    expect(value.paymentTerm).toEqual(inactivePaymentTerm);
+    expect(value.purchasePaymentTerm).toEqual(inactivePaymentTerm);
     expect(spies.update).not.toHaveBeenCalled();
   });
 

package/src/modules/purchase/command/deactivatePurchasePaymentTerm.ts

@@ -18,7 +18,7 @@ export async function run(
     return err(new PaymentTermNotFoundError(input.id));
   }
   if (paymentTerm.status === "INACTIVE") {
-    return ok({ paymentTerm });
+    return ok({ purchasePaymentTerm: paymentTerm });
   }
 
   const supplierProfile = await db
@@ -37,5 +37,5 @@ export async function run(
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ paymentTerm: deactivated });
+  return ok({ purchasePaymentTerm: deactivated });
 }

package/src/modules/purchase/command/updateGoodsReceipt.ts

@@ -17,12 +17,11 @@ import {
 } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateGoodsReceiptInput {
+export type UpdateGoodsReceiptInput = {
   id: string;
   receiptDate?: Date;
   lines?: GoodsReceiptLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdateGoodsReceiptInput, ctx: CommandContext) {
   void ctx;
   const goodsReceipt = await selectOne(db, "GoodsReceipt", "id", input.id, true);

package/src/modules/purchase/command/updatePurchaseBill.ts

@@ -17,13 +17,12 @@ import {
 } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdatePurchaseBillInput {
+export type UpdatePurchaseBillInput = {
   id: string;
   invoiceNumber?: string;
   invoiceDate?: Date;
   lines?: PurchaseBillLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdatePurchaseBillInput, ctx: CommandContext) {
   void ctx;
   const purchaseBill = await db

package/src/modules/purchase/command/updatePurchaseOrder.ts

@@ -22,15 +22,14 @@ import {
 } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdatePurchaseOrderInput {
+export type UpdatePurchaseOrderInput = {
   id: string;
   orderDate?: Date;
   documentReceivingSiteId?: string | null;
   paymentTermId?: string | null;
   externalSupplierOrderReference?: string | null;
   lines?: PurchaseOrderLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdatePurchaseOrderInput, ctx: CommandContext) {
   void ctx;
   const existing = await selectOne(db, "PurchaseOrder", "id", input.id, true);

package/src/modules/purchase/command/updatePurchasePaymentTerm.test.ts

@@ -30,7 +30,7 @@ describe("updatePurchasePaymentTerm", () => {
       commandCtx,
     );
 
-    expect(expectOk(result).paymentTerm.name).toBe("Net 45");
+    expect(expectOk(result).purchasePaymentTerm.name).toBe("Net 45");
   });
 
   it("updates an inactive payment term without reactivating it", async () => {
@@ -45,7 +45,7 @@ describe("updatePurchasePaymentTerm", () => {
       commandCtx,
     );
 
-    expect(expectOk(result).paymentTerm.status).toBe("INACTIVE");
+    expect(expectOk(result).purchasePaymentTerm.status).toBe("INACTIVE");
   });
 
   it("preserves historical purchase-order snapshots after update", async () => {

package/src/modules/purchase/command/updatePurchasePaymentTerm.ts

@@ -17,15 +17,14 @@ import {
 } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdatePurchasePaymentTermInput {
+export type UpdatePurchasePaymentTermInput = {
   id: string;
   companyId?: string;
   name?: string;
   code?: string;
   scheduleLines?: PaymentTermScheduleLineInput[];
   notes?: string | null;
-}
-
+};
 export async function run(
   db: Transaction,
   input: UpdatePurchasePaymentTermInput,
@@ -69,5 +68,5 @@ export async function run(
     .returningAll()
     .executeTakeFirstOrThrow();
 
-  return ok({ paymentTerm });
+  return ok({ purchasePaymentTerm: paymentTerm });
 }

package/src/modules/purchase/command/updatePurchasePriceList.ts

@@ -8,7 +8,7 @@ import {
 import { selectOne } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdatePurchasePriceListInput {
+export type UpdatePurchasePriceListInput = {
   id: string;
   companyId?: string;
   supplierId?: string;
@@ -16,8 +16,7 @@ export interface UpdatePurchasePriceListInput {
   code?: string;
   priority?: number;
   notes?: string | null;
-}
-
+};
 export async function run(
   db: Transaction,
   input: UpdatePurchasePriceListInput,

package/src/modules/purchase/command/updatePurchasePriceRule.ts

@@ -14,7 +14,7 @@ import {
 import { findRuleConflict, isActiveStatus, selectOne } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdatePurchasePriceRuleInput {
+export type UpdatePurchasePriceRuleInput = {
   id: string;
   itemId?: string;
   currencyId?: string;
@@ -25,8 +25,7 @@ export interface UpdatePurchasePriceRuleInput {
   effectiveTo?: Date | null;
   priority?: number;
   notes?: string | null;
-}
-
+};
 export async function run(
   db: Transaction,
   input: UpdatePurchasePriceRuleInput,

package/src/modules/purchase/module.ts

@@ -111,6 +111,7 @@ export interface DefineModuleParams<
   };
 }
 
+/* @__NO_SIDE_EFFECTS__ */
 export const defineModule = <
   const SPF extends Record<string, TailorAnyDBField> = Record<never, never>,
   const PTF extends Record<string, TailorAnyDBField> = Record<never, never>,

package/src/modules/sales/command/createSalesOrder.ts

@@ -136,5 +136,5 @@ export async function run(db: Transaction, input: CreateSalesOrderInput, ctx: Co
     insertedLines.push(inserted);
   }
 
-  return ok({ salesOrder: salesOrder!, lines: insertedLines });
+  return ok({ salesOrder: salesOrder!, salesOrderLines: insertedLines });
 }

package/src/modules/sales/command/updateChannel.test.ts

@@ -31,7 +31,7 @@ describe("updateChannel", () => {
     expectOk(result);
   });
 
-  it("throws when Channel does not exist", async () => {
+  it("returns error when Channel does not exist", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined);
 
@@ -40,7 +40,7 @@ describe("updateChannel", () => {
     expectErr(result, ChannelNotFoundError);
   });
 
-  it("throws when Channel is not editable", async () => {
+  it("returns error when Channel is not editable", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce({ ...baseChannel, status: "ARCHIVED" });
 
@@ -49,7 +49,7 @@ describe("updateChannel", () => {
     expectErr(result, ChannelNotEditableError);
   });
 
-  it("throws when the resulting channel type is empty", async () => {
+  it("returns error when the resulting channel type is empty", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseChannel);
 
@@ -58,7 +58,7 @@ describe("updateChannel", () => {
     expectErr(result, InvalidChannelTypeError);
   });
 
-  it("throws when the updated channel name conflicts within the company", async () => {
+  it("returns error when the updated channel name conflicts within the company", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseChannel)

package/src/modules/sales/command/updateChannel.ts

@@ -8,14 +8,13 @@ import {
 import { encodeJson } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateChannelInput {
+export type UpdateChannelInput = {
   channelId: string;
   type?: string;
   name?: string;
   code?: string | null;
   settings?: Record<string, unknown> | null;
-}
-
+};
 export async function run(db: Transaction, input: UpdateChannelInput, ctx: CommandContext) {
   void ctx;
   const channel = await db

package/src/modules/sales/command/updateListing.test.ts

@@ -23,7 +23,7 @@ describe("updateListing", () => {
     expect(spies.set).toHaveBeenCalledWith(expect.objectContaining({ title: "Updated Title" }));
   });
 
-  it("throws when listing is retired", async () => {
+  it("returns error when listing is retired", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(retiredListing);
 
@@ -36,7 +36,7 @@ describe("updateListing", () => {
     expectErr(result, ListingNotEditableError);
   });
 
-  it("throws when updated external identifier conflicts with another listing", async () => {
+  it("returns error when updated external identifier conflicts with another listing", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select
       .mockReturnValueOnce(baseListing)

package/src/modules/sales/command/updateListing.ts

@@ -6,15 +6,14 @@ import {
 } from "../lib/errors.generated";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateListingInput {
+export type UpdateListingInput = {
   listingId: string;
   sku?: string | null;
   externalIdentifier?: string | null;
   title?: string | null;
   requiresPriceSync?: boolean;
   salesPriceListId?: string | null;
-}
-
+};
 export async function run(db: Transaction, input: UpdateListingInput, ctx: CommandContext) {
   void ctx;
   const listing = await db

package/src/modules/sales/command/updateSalesCreditNote.test.ts

@@ -37,7 +37,7 @@ describe("updateSalesCreditNote", () => {
     expect(spies.set).toHaveBeenCalledWith(expect.objectContaining({ reason: "Updated" }));
   });
 
-  it("throws when updating an issued credit note", async () => {
+  it("returns error when updating an issued credit note", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(issuedCreditNote);
 
@@ -50,7 +50,7 @@ describe("updateSalesCreditNote", () => {
     expectErr(result, CreditNoteNotEditableError);
   });
 
-  it("throws when correction content is invalid", async () => {
+  it("returns error when correction content is invalid", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseCreditNote);
 

package/src/modules/sales/command/updateSalesCreditNote.ts

@@ -7,12 +7,11 @@ import {
 import { type SalesCreditNoteLineInput } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateSalesCreditNoteInput {
+export type UpdateSalesCreditNoteInput = {
   salesCreditNoteId: string;
   reason?: string | null;
   lines?: SalesCreditNoteLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdateSalesCreditNoteInput, ctx: CommandContext) {
   void ctx;
   const salesCreditNote = await db

package/src/modules/sales/command/updateSalesInvoice.test.ts

@@ -42,7 +42,7 @@ describe("updateSalesInvoice", () => {
     expect(spies.set).toHaveBeenCalledWith(expect.objectContaining({ notes: "Updated" }));
   });
 
-  it("throws when updating an issued invoice", async () => {
+  it("returns error when updating an issued invoice", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(issuedSalesInvoice);
 
@@ -55,7 +55,7 @@ describe("updateSalesInvoice", () => {
     expectErr(result, InvoiceNotEditableError);
   });
 
-  it("throws when updated line content is invalid", async () => {
+  it("returns error when updated line content is invalid", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseSalesInvoice);
 

package/src/modules/sales/command/updateSalesInvoice.ts

@@ -7,13 +7,12 @@ import {
 import { encodeJson, type SalesInvoiceLineInput } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateSalesInvoiceInput {
+export type UpdateSalesInvoiceInput = {
   salesInvoiceId: string;
   invoiceDate?: Date;
   notes?: string | null;
   lines?: SalesInvoiceLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdateSalesInvoiceInput, ctx: CommandContext) {
   void ctx;
   const salesInvoice = await db

package/src/modules/sales/command/updateSalesOrder.test.ts

@@ -64,7 +64,7 @@ describe("updateSalesOrder", () => {
     );
   });
 
-  it("throws when the order does not exist", async () => {
+  it("returns error when the order does not exist", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined);
 
@@ -73,7 +73,7 @@ describe("updateSalesOrder", () => {
     expectErr(result, SalesOrderNotFoundError);
   });
 
-  it("throws when updating a non-draft order", async () => {
+  it("returns error when updating a non-draft order", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce({ ...baseSalesOrder, status: "SUBMITTED" });
 
@@ -82,7 +82,7 @@ describe("updateSalesOrder", () => {
     expectErr(result, OrderNotEditableError);
   });
 
-  it("throws when the update would remove all lines", async () => {
+  it("returns error when the update would remove all lines", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(baseSalesOrder);
 

package/src/modules/sales/command/updateSalesOrder.ts

@@ -11,7 +11,7 @@ import {
 import { encodeJson, type SalesOrderLineInput } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateSalesOrderInput {
+export type UpdateSalesOrderInput = {
   salesOrderId: string;
   customerId?: string;
   paymentTermId?: string | null;
@@ -20,8 +20,7 @@ export interface UpdateSalesOrderInput {
   shippingAddressSnapshot?: Record<string, unknown> | null;
   billingAddressSnapshot?: Record<string, unknown> | null;
   lines?: SalesOrderLineInput[];
-}
-
+};
 export async function run(db: Transaction, input: UpdateSalesOrderInput, ctx: CommandContext) {
   void ctx;
   const salesOrder = await db

package/src/modules/sales/command/updateSalesPaymentTerm.test.ts

@@ -38,7 +38,7 @@ describe("updateSalesPaymentTerm", () => {
     expect(spies.set).toHaveBeenCalledWith(expect.objectContaining({ name: "Installments" }));
   });
 
-  it("throws when SalesPaymentTerm does not exist", async () => {
+  it("returns error when SalesPaymentTerm does not exist", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined);
 
@@ -51,7 +51,7 @@ describe("updateSalesPaymentTerm", () => {
     expectErr(result, SalespaymenttermNotFoundError);
   });
 
-  it("throws when SalesPaymentTerm is not editable", async () => {
+  it("returns error when SalesPaymentTerm is not editable", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(inactivePaymentTerm);
 

package/src/modules/sales/command/updateSalesPaymentTerm.ts

@@ -11,14 +11,13 @@ import {
 } from "../lib/domain";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateSalesPaymentTermInput {
+export type UpdateSalesPaymentTermInput = {
   salesPaymentTermId: string;
   name?: string;
   code?: string | null;
   scheduleLines?: PaymentTermScheduleLineInput[];
   notes?: string | null;
-}
-
+};
 export async function run(
   db: Transaction,
   input: UpdateSalesPaymentTermInput,

package/src/modules/sales/command/updateSalesPriceList.test.ts

@@ -30,7 +30,7 @@ describe("updateSalesPriceList", () => {
     expectOk(result);
   });
 
-  it("throws when SalesPriceList does not exist", async () => {
+  it("returns error when SalesPriceList does not exist", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(undefined);
 
@@ -43,7 +43,7 @@ describe("updateSalesPriceList", () => {
     expectErr(result, SalespricelistNotFoundError);
   });
 
-  it("throws when SalesPriceList is not editable", async () => {
+  it("returns error when SalesPriceList is not editable", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce({ ...basePriceList, status: "INACTIVE" });
 
@@ -56,7 +56,7 @@ describe("updateSalesPriceList", () => {
     expectErr(result, SalespricelistNotEditableError);
   });
 
-  it("throws when the company backing the price list does not exist", async () => {
+  it("returns error when the company backing the price list does not exist", async () => {
     const { db, spies } = createMockDb<Transaction>();
     spies.select.mockReturnValueOnce(draftPriceList).mockReturnValueOnce(undefined);
 

package/src/modules/sales/command/updateSalesPriceList.ts

@@ -6,14 +6,13 @@ import {
 } from "../lib/errors.generated";
 import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
 
-export interface UpdateSalesPriceListInput {
+export type UpdateSalesPriceListInput = {
   salesPriceListId: string;
   name?: string;
   code?: string | null;
   priority?: number;
   notes?: string | null;
-}
-
+};
 export async function run(db: Transaction, input: UpdateSalesPriceListInput, ctx: CommandContext) {
   void ctx;
   const salesPriceList = await db