@porulle/core 0.63.0 → 0.65.0

package/src/modules/fulfillment/service.ts

@@ -4,15 +4,13 @@ import { assertPermission } from "../../auth/permissions.js";
 import type { Actor } from "../../auth/types.js";
 import type { CommerceConfig } from "../../config/types.js";
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import type { TxContext } from "../../kernel/database/tx-context.js";
 import { CommerceNotFoundError, CommerceValidationError, toCommerceError } from "../../kernel/errors.js";
 import { runAfterHooks } from "../../kernel/hooks/executor.js";
-import { createHookContext } from "../../kernel/hooks/create-context.js";
+import { createModuleHookContext } from "../../kernel/hooks/create-context.js";
 import type { HookRegistry } from "../../kernel/hooks/registry.js";
 import type { AfterHook, HookContext } from "../../kernel/hooks/types.js";
 import { Err, Ok, type Result } from "../../kernel/result.js";
-import { createLogger } from "../../utils/logger.js";
 import type {
   FulfillmentRecord as FulfillmentDbRow,
   FulfillmentRecordInsert,
@@ -25,7 +23,6 @@ import type {
   FulfillmentStrategy,
   FulfillmentStrategyContext,
 } from "./types.js";
-import { makeId } from "../../utils/id.js";
 
 interface InventoryServiceLike {
   adjust(input: {
@@ -90,7 +87,7 @@ class PhysicalFulfillmentStrategy implements FulfillmentStrategy {
     _context: FulfillmentStrategyContext,
   ): Promise<Result<FulfillmentRecord>> {
     return Ok({
-      id: makeId(),
+      id: crypto.randomUUID(),
       orderId: lineItem.orderId,
       type: this.type,
       status: "pending",
@@ -122,7 +119,7 @@ class DigitalDownloadFulfillmentStrategy implements FulfillmentStrategy {
   ): Promise<Result<FulfillmentRecord>> {
     const expiresAt = new Date(Date.now() + 1000 * 60 * 60 * 24).toISOString();
     return Ok({
-      id: makeId(),
+      id: crypto.randomUUID(),
       orderId: lineItem.orderId,
       type: this.type,
       status: "fulfilled",
@@ -162,7 +159,7 @@ class DigitalAccessFulfillmentStrategy implements FulfillmentStrategy {
     _context: FulfillmentStrategyContext,
   ): Promise<Result<FulfillmentRecord>> {
     return Ok({
-      id: makeId(),
+      id: crypto.randomUUID(),
       orderId: lineItem.orderId,
       type: this.type,
       status: "fulfilled",
@@ -200,7 +197,7 @@ class InternalTransferFulfillmentStrategy implements FulfillmentStrategy {
     _context: FulfillmentStrategyContext,
   ): Promise<Result<FulfillmentRecord>> {
     return Ok({
-      id: makeId(),
+      id: crypto.randomUUID(),
       orderId: lineItem.orderId,
       type: this.type,
       status: "processing",
@@ -231,7 +228,7 @@ class AppointmentFulfillmentStrategy implements FulfillmentStrategy {
     _context: FulfillmentStrategyContext,
   ): Promise<Result<FulfillmentRecord>> {
     return Ok({
-      id: makeId(),
+      id: crypto.randomUUID(),
       orderId: lineItem.orderId,
       type: this.type,
       status: "pending",
@@ -328,15 +325,7 @@ export class FulfillmentService {
       const afterHooks = this.deps.hooks.resolve(
         "fulfillment.afterCreate",
       ) as AfterHook<FulfillmentRecord>[];
-      const hookCtx: HookContext = createHookContext({
-        actor: actor ?? ctx?.actor ?? null,
-        tx: ctx?.tx ?? null,
-        logger: createLogger("fulfillment"),
-        services: this.deps.services,
-        context: { moduleName: "fulfillment" },
-        database: { db: this.deps.database.db as PluginDb },
-        commerceConfig: this.deps.config,
-      });
+      const hookCtx: HookContext = createModuleHookContext("fulfillment", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
       await runAfterHooks(afterHooks, null, record, "create", hookCtx, (hook) => this.deps.hooks.runsInTransaction(hook));
 
       // Create a fulfillment line item linking this fulfillment to the order line item
@@ -487,15 +476,7 @@ export class FulfillmentService {
     const afterHooks = this.deps.hooks.resolve(
       "fulfillment.afterCreate",
     ) as AfterHook<FulfillmentRecord>[];
-    const hookCtx: HookContext = createHookContext({
-      actor: actor ?? ctx?.actor ?? null,
-      tx: ctx?.tx ?? null,
-      logger: createLogger("fulfillment"),
-      services: this.deps.services,
-      context: { moduleName: "fulfillment" },
-      database: { db: this.deps.database.db as PluginDb },
-      commerceConfig: this.deps.config,
-    });
+    const hookCtx: HookContext = createModuleHookContext("fulfillment", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, null, record, "create", hookCtx, (hook) => this.deps.hooks.runsInTransaction(hook));
 
     return Ok(record);

package/src/modules/inventory/service.ts

@@ -10,13 +10,11 @@ import {
   toCommerceError,
 } from "../../kernel/errors.js";
 import { runAfterHooks } from "../../kernel/hooks/executor.js";
-import { createHookContext } from "../../kernel/hooks/create-context.js";
+import { createModuleHookContext } from "../../kernel/hooks/create-context.js";
 import type { HookContext } from "../../kernel/hooks/types.js";
 import type { HookRegistry } from "../../kernel/hooks/registry.js";
 import { Err, Ok, type Result } from "../../kernel/result.js";
-import { createLogger } from "../../utils/logger.js";
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import { createTxContext, type TxContext } from "../../kernel/database/tx-context.js";
 import {
   InventoryRepository,
@@ -438,15 +436,7 @@ export class InventoryService {
         txCtx,
       );
 
-      const hookCtx: HookContext = createHookContext({
-        actor: actor ?? null,
-        tx: txCtx.tx,
-        logger: createLogger("inventory.adjust"),
-        services: this.deps.services,
-        context: { moduleName: "inventory" },
-        database: { db: this.deps.database.db as PluginDb },
-        commerceConfig: this.deps.config,
-      });
+      const hookCtx: HookContext = createModuleHookContext("inventory", this.deps, actor ?? null, txCtx.tx, { logScope: "inventory.adjust" });
 
       const afterHooks = this.deps.hooks.resolve("inventory.afterAdjust");
       await runAfterHooks(
@@ -622,15 +612,7 @@ export class InventoryService {
         organizationId: orgId,
         entities: [...byEntity].map(([entityId, levels]) => ({ entityId, levels })),
       };
-      const hookCtx: HookContext = createHookContext({
-        actor: actor ?? null,
-        tx: txCtx.tx,
-        logger: createLogger("inventory.adjustMany"),
-        services: this.deps.services,
-        context: { moduleName: "inventory" },
-        database: { db: this.deps.database.db as PluginDb },
-        commerceConfig: this.deps.config,
-      });
+      const hookCtx: HookContext = createModuleHookContext("inventory", this.deps, actor ?? null, txCtx.tx, { logScope: "inventory.adjustMany" });
       await runAfterHooks(
         this.deps.hooks.resolve("inventory.afterAdjustMany") as Parameters<typeof runAfterHooks>[0],
         null,

package/src/modules/media/service.ts

@@ -15,8 +15,8 @@ import type { CatalogRepository } from "../catalog/repository/index.js";
 import type { TxContext } from "../../kernel/database/tx-context.js";
 import { createTxContext } from "../../kernel/database/tx-context.js";
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import { makeId } from "../../utils/id.js";
 import type { MediaAssetOrigin } from "./schema.js";
+import { isUniqueViolation } from "../../kernel/database/unique-violation.js";
 
 export interface UploadMediaInput {
   filename: string;
@@ -66,12 +66,6 @@ export function assertMediaWritable(asset: Pick<MediaAsset, "origin">, actor: Ac
   if (asset.origin === "merchant") assertPermission(actor, "media:write");
 }
 
-function isUniqueViolation(error: unknown): boolean {
-  if (error == null || typeof error !== "object") return false;
-  const value = error as { code?: unknown; cause?: unknown };
-  if (value.code === "23505") return true;
-  return isUniqueViolation(value.cause);
-}
 
 function isSvgOrXml(buffer: Uint8Array): boolean {
   const prefix = new TextDecoder()
@@ -160,7 +154,7 @@ export class MediaService {
       );
     }
 
-    const id = makeId();
+    const id = crypto.randomUUID();
     const key = `${new Date().getFullYear()}/${id}-${input.filename}`;
     const uploaded = await this.deps.storage.upload(
       key,

package/src/modules/orders/quote.ts

@@ -12,7 +12,6 @@ import type { Actor } from "../../auth/types.js";
 import type { ServiceContainer } from "../../kernel/hooks/types.js";
 import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import type { ShippingAddress } from "../shipping/calculator.js";
-import { makeId } from "../../utils/id.js";
 
 export interface OrderPricingInput {
   currency: string;
@@ -65,7 +64,7 @@ export async function computeOrderPricing(
   tx?: unknown,
 ): Promise<OrderPricingBreakdown> {
   const data: CheckoutData = {
-    checkoutId: makeId(),
+    checkoutId: crypto.randomUUID(),
     cartId: "",
     currency: input.currency,
     paymentMethodId: "",

package/src/modules/orders/service.ts

@@ -13,13 +13,8 @@ import {
   toCommerceError,
 } from "../../kernel/errors.js";
 import { runAfterHooks, runBeforeHooks } from "../../kernel/hooks/executor.js";
-import { createHookContext } from "../../kernel/hooks/create-context.js";
-import type { JobsAdapter } from "../../kernel/jobs/adapter.js";
-import type {
-  AfterHook,
-  BeforeHook,
-  HookContext,
-} from "../../kernel/hooks/types.js";
+import { createModuleHookContext } from "../../kernel/hooks/create-context.js";
+import type { AfterHook, BeforeHook } from "../../kernel/hooks/types.js";
 import type { HookRegistry } from "../../kernel/hooks/registry.js";
 import {
   canTransition,
@@ -28,7 +23,6 @@ import {
   type StateDefinition,
 } from "../../kernel/state-machine/machine.js";
 import { Err, Ok, type Result } from "../../kernel/result.js";
-import { createLogger } from "../../utils/logger.js";
 import { makeIdempotencyScope } from "../../utils/id.js";
 import { paginate, type Pagination } from "../../utils/pagination.js";
 import {
@@ -36,7 +30,6 @@ import {
   type TxContext,
 } from "../../kernel/database/tx-context.js";
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import {
   OrdersRepository,
   type Order,
@@ -143,24 +136,6 @@ type BeforeStatusChangeHook = BeforeHook<StatusChangeHookInput>;
  */
 type AfterStatusChangeHook = AfterHook<HydratedOrder, StatusChangeHookInput>;
 
-function context(
-  actor: Actor | null,
-  services: Record<string, unknown>,
-  database: DatabaseAdapter,
-  config: CommerceConfig,
-  tx: unknown = null,
-): HookContext {
-  return createHookContext({
-    actor,
-    tx,
-    logger: createLogger("orders"),
-    services,
-    ...(services.jobs ? { jobs: services.jobs as JobsAdapter } : {}),
-    context: { moduleName: "orders" },
-    database: { db: database.db as PluginDb },
-    commerceConfig: config,
-  });
-}
 
 export class OrderService {
   private readonly repo: OrdersRepository;
@@ -522,7 +497,7 @@ export class OrderService {
     const afterHooks = this.deps.hooks.resolve(
       "orders.afterCreate",
     ) as AfterCreateOrderHook[];
-    const hookCtx = context(actor, this.deps.services, this.deps.database, this.deps.config, ctx?.tx);
+    const hookCtx = createModuleHookContext("orders", this.deps, actor, ctx?.tx);
 
     const processed = await runBeforeHooks(
       beforeHooks,
@@ -804,7 +779,7 @@ export class OrderService {
       "orders.afterGet",
     ) as AfterHook<HydratedOrder>[];
     if (afterGetHooks.length > 0) {
-      const hookCtx = context(actor, this.deps.services, this.deps.database, this.deps.config, ctx?.tx);
+      const hookCtx = createModuleHookContext("orders", this.deps, actor, ctx?.tx);
       await runAfterHooks(afterGetHooks, null, hydrated, "read", hookCtx, (hook) => this.deps.hooks.runsInTransaction(hook));
     }
 
@@ -1013,7 +988,7 @@ export class OrderService {
       "orders.afterStatusChange",
     ) as AfterStatusChangeHook[];
 
-    const hookCtx = context(actor, this.deps.services, this.deps.database, this.deps.config, ctx?.tx);
+    const hookCtx = createModuleHookContext("orders", this.deps, actor, ctx?.tx);
     const statusHookInput: StatusChangeHookInput = {
       orderId: order.id,
       fromStatus: order.status as OrderState,

package/src/modules/pricing/service.ts

@@ -2,18 +2,16 @@ import { resolveOrgIdForCommerce } from "../../auth/org.js";
 import type { Actor } from "../../auth/types.js";
 import type { CommerceConfig } from "../../config/types.js";
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import type { TxContext } from "../../kernel/database/tx-context.js";
 import {
   CommerceNotFoundError,
   CommerceValidationError,
 } from "../../kernel/errors.js";
 import { runAfterHooks } from "../../kernel/hooks/executor.js";
-import { createHookContext } from "../../kernel/hooks/create-context.js";
+import { createModuleHookContext } from "../../kernel/hooks/create-context.js";
 import type { HookRegistry } from "../../kernel/hooks/registry.js";
-import type { AfterHook, HookContext } from "../../kernel/hooks/types.js";
+import type { AfterHook } from "../../kernel/hooks/types.js";
 import { Err, Ok, type Result } from "../../kernel/result.js";
-import { createLogger } from "../../utils/logger.js";
 import type {
   PricingRepository,
   Price,
@@ -39,23 +37,6 @@ interface PricingServiceDeps {
   database: DatabaseAdapter;
 }
 
-function hookContext(
-  actor: Actor | null,
-  services: Record<string, unknown>,
-  database: DatabaseAdapter,
-  config: CommerceConfig,
-  tx: unknown,
-): HookContext {
-  return createHookContext({
-    actor,
-    tx,
-    logger: createLogger("pricing"),
-    services,
-    context: { moduleName: "pricing" },
-    database: { db: database.db as PluginDb },
-    commerceConfig: config,
-  });
-}
 
 export interface PriceResolutionContext {
   entityId: string;
@@ -92,6 +73,7 @@ export interface ResolvedPrice {
 
 export type { SetBasePriceInput, CreatePriceModifierInput } from "./schemas.js";
 import type { SetBasePriceInput, CreatePriceModifierInput } from "./schemas.js";
+import { normalizeCurrency } from "../../utils/money.js";
 
 function matchesQuantity(
   min: number | null | undefined,
@@ -190,9 +172,6 @@ function toGroupSet(context: PriceResolutionContext): Set<string> {
   return new Set(context.customerGroupIds ?? []);
 }
 
-function normalizeCurrency(currency: string): string {
-  return currency.trim().toUpperCase();
-}
 
 export class PricingService {
   private readonly repo: PricingRepository;
@@ -269,9 +248,7 @@ export class PricingService {
     const afterHooks = this.deps.hooks.resolve(
       "pricing.afterCreate",
     ) as AfterHook<Price>[];
-    const hctx = hookContext(
-      actor ?? ctx?.actor ?? null, this.deps.services, this.deps.database, this.deps.config, ctx?.tx ?? null,
-    );
+    const hctx = createModuleHookContext("pricing", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, null, record, "create", hctx);
 
     return Ok(record);
@@ -326,9 +303,7 @@ export class PricingService {
     const afterHooks = this.deps.hooks.resolve(
       "pricing.afterCreate",
     ) as AfterHook<PriceModifier>[];
-    const hctx = hookContext(
-      actor ?? ctx?.actor ?? null, this.deps.services, this.deps.database, this.deps.config, ctx?.tx ?? null,
-    );
+    const hctx = createModuleHookContext("pricing", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, null, modifier, "create", hctx);
 
     return Ok(modifier);
@@ -388,6 +363,13 @@ export class PricingService {
       },
       ctx,
     );
+
+    const afterHooks = this.deps.hooks.resolve(
+      "pricing.afterUpdate",
+    ) as AfterHook<PriceModifier>[];
+    const hctx = createModuleHookContext("pricing", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
+    await runAfterHooks(afterHooks, existing, updated!, "update", hctx);
+
     return Ok(updated!);
   }
 

package/src/modules/promotions/service.ts

@@ -1,16 +1,14 @@
 import type { DatabaseAdapter } from "../../kernel/database/adapter.js";
-import type { PluginDb } from "../../kernel/database/plugin-types.js";
 import type { TxContext } from "../../kernel/database/tx-context.js";
 import {
   CommerceNotFoundError,
   CommerceValidationError,
 } from "../../kernel/errors.js";
 import { runAfterHooks } from "../../kernel/hooks/executor.js";
-import { createHookContext } from "../../kernel/hooks/create-context.js";
+import { createModuleHookContext } from "../../kernel/hooks/create-context.js";
 import type { HookRegistry } from "../../kernel/hooks/registry.js";
-import type { AfterHook, HookContext } from "../../kernel/hooks/types.js";
+import type { AfterHook } from "../../kernel/hooks/types.js";
 import { Err, Ok, type Result } from "../../kernel/result.js";
-import { createLogger } from "../../utils/logger.js";
 import type {
   PromotionsRepository,
   Promotion,
@@ -41,23 +39,6 @@ interface PromotionServiceDeps {
   database: DatabaseAdapter;
 }
 
-function hookContext(
-  actor: Actor | null,
-  services: Record<string, unknown>,
-  database: DatabaseAdapter,
-  config: CommerceConfig,
-  tx: unknown,
-): HookContext {
-  return createHookContext({
-    actor,
-    tx,
-    logger: createLogger("promotions"),
-    services,
-    context: { moduleName: "promotions" },
-    database: { db: database.db as PluginDb },
-    commerceConfig: config,
-  });
-}
 
 export interface PromotionLineItem {
   entityId: string;
@@ -242,9 +223,7 @@ export class PromotionService {
     const afterHooks = this.deps.hooks.resolve(
       "promotions.afterCreate",
     ) as AfterHook<Promotion>[];
-    const hctx = hookContext(
-      actor ?? ctx?.actor ?? null, this.deps.services, this.deps.database, this.deps.config, ctx?.tx ?? null,
-    );
+    const hctx = createModuleHookContext("promotions", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, null, promotion, "create", hctx, (hook) => this.deps.hooks.runsInTransaction(hook));
 
     return Ok(promotion);
@@ -265,7 +244,7 @@ export class PromotionService {
       "promotions.afterUpdate",
     ) as AfterHook<Promotion>[];
     // Actor-less by design; resolves to the deployment's declared organization.
-    const hctx = hookContext(null, this.deps.services, this.deps.database, this.deps.config, ctx?.tx ?? null);
+    const hctx = createModuleHookContext("promotions", this.deps, null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, promotion, updated, "update", hctx, (hook) => this.deps.hooks.runsInTransaction(hook));
 
     return Ok(updated);
@@ -342,9 +321,7 @@ export class PromotionService {
     const afterHooks = this.deps.hooks.resolve(
       "promotions.afterUpdate",
     ) as AfterHook<Promotion>[];
-    const hctx = hookContext(
-      actor ?? ctx?.actor ?? null, this.deps.services, this.deps.database, this.deps.config, ctx?.tx ?? null,
-    );
+    const hctx = createModuleHookContext("promotions", this.deps, actor ?? ctx?.actor ?? null, ctx?.tx ?? null);
     await runAfterHooks(afterHooks, existing, updated, "update", hctx, (hook) => this.deps.hooks.runsInTransaction(hook));
 
     return Ok(updated);

package/src/modules/tax/service.ts

@@ -15,13 +15,13 @@ import type { TaxClass, TaxRate, TaxRatesRepository } from "./repository/index.j
 
 interface TaxServiceDeps {
   adapter: TaxAdapter | undefined;
-  repository?: TaxRatesRepository;
+  repository: TaxRatesRepository;
   config: CommerceConfig;
 }
 
 export class TaxService {
   private adapter: TaxAdapter | undefined;
-  private repository: TaxRatesRepository | undefined;
+  private repository: TaxRatesRepository;
   private config: CommerceConfig;
 
   constructor(deps: TaxServiceDeps) {
@@ -47,14 +47,14 @@ export class TaxService {
     orgId?: string,
     ctx?: TxContext,
   ): Promise<Result<TaxCalculationResult>> {
-    if (this.repository && orgId && params.lineItems.length > 0) {
+    if (orgId && params.lineItems.length > 0) {
       const classes = await this.repository.findActiveClasses(orgId, ctx);
       if (classes.length > 0) {
         return Ok(await this.calculateByClasses(params, classes, ctx));
       }
     }
 
-    if (this.repository && orgId && params.toAddress) {
+    if (orgId && params.toAddress) {
       const matched = await this.matchRuntimeRates(orgId, params.toAddress, ctx);
       if (matched.length > 0) {
         // Subtract the order-level discount from the taxable base too (audit
@@ -110,7 +110,7 @@ export class TaxService {
     const byName = new Map(classes.map((c) => [c.name, c]));
     const defaultClass = classes.find((c) => c.isDefault) ?? null;
 
-    const { byEntity, byVariant } = await this.repository!.resolveCatalogTaxClasses(
+    const { byEntity, byVariant } = await this.repository.resolveCatalogTaxClasses(
       params.lineItems.map((li) => ({ entityId: li.entityId, variantId: li.variantId })),
       ctx,
     );
@@ -175,14 +175,12 @@ export class TaxService {
     actor?: Actor | null,
     ctx?: TxContext,
   ): Promise<Result<TaxClass>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     if (input.rateBps < 0) {
       return Err(new CommerceValidationError("rateBps must be non-negative."));
     }
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    if (input.isDefault) await repo.value.clearDefaultClass(orgId, ctx);
-    const created = await repo.value.createClass(
+    if (input.isDefault) await this.repository.clearDefaultClass(orgId, ctx);
+    const created = await this.repository.createClass(
       {
         organizationId: orgId,
         name: input.name,
@@ -196,10 +194,8 @@ export class TaxService {
   }
 
   async listTaxClasses(actor?: Actor | null, ctx?: TxContext): Promise<Result<TaxClass[]>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    return Ok(await repo.value.findAllClasses(orgId, ctx));
+    return Ok(await this.repository.findAllClasses(orgId, ctx));
   }
 
   async updateTaxClass(
@@ -208,13 +204,11 @@ export class TaxService {
     actor?: Actor | null,
     ctx?: TxContext,
   ): Promise<Result<TaxClass>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    const existing = await repo.value.findClassById(orgId, id, ctx);
+    const existing = await this.repository.findClassById(orgId, id, ctx);
     if (!existing) return Err(new CommerceNotFoundError("Tax class not found."));
-    if (patch.isDefault) await repo.value.clearDefaultClass(orgId, ctx);
-    const updated = await repo.value.updateClass(
+    if (patch.isDefault) await this.repository.clearDefaultClass(orgId, ctx);
+    const updated = await this.repository.updateClass(
       id,
       {
         ...(patch.name !== undefined ? { name: patch.name } : {}),
@@ -228,12 +222,10 @@ export class TaxService {
   }
 
   async deleteTaxClass(id: string, actor?: Actor | null, ctx?: TxContext): Promise<Result<{ deleted: true }>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    const existing = await repo.value.findClassById(orgId, id, ctx);
+    const existing = await this.repository.findClassById(orgId, id, ctx);
     if (!existing) return Err(new CommerceNotFoundError("Tax class not found."));
-    await repo.value.deleteClass(id, ctx);
+    await this.repository.deleteClass(id, ctx);
     return Ok({ deleted: true });
   }
 
@@ -242,7 +234,7 @@ export class TaxService {
     toAddress: { country: string; state?: string },
     ctx?: TxContext,
   ): Promise<TaxRate[]> {
-    const rates = await this.repository!.findActive(orgId, ctx);
+    const rates = await this.repository.findActive(orgId, ctx);
     if (rates.length === 0) return [];
     const country = toAddress.country?.toUpperCase();
     const state = toAddress.state?.toUpperCase();
@@ -267,21 +259,9 @@ export class TaxService {
     return this.adapter.voidTransaction(params);
   }
 
-  requireConfigured(): Result<TaxAdapter> {
-    if (!this.adapter) {
-      return Err(new CommerceValidationError("Tax adapter is not configured."));
-    }
-    return Ok(this.adapter);
-  }
 
   // ── Runtime tax-rate management (issue #45) ────────────────────────────
 
-  private requireRepository(): Result<TaxRatesRepository> {
-    if (!this.repository) {
-      return Err(new CommerceValidationError("Tax rate persistence is not available."));
-    }
-    return Ok(this.repository);
-  }
 
   async createTaxRate(
     input: {
@@ -296,13 +276,11 @@ export class TaxService {
     actor?: Actor | null,
     ctx?: TxContext,
   ): Promise<Result<TaxRate>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     if (input.rateBps < 0) {
       return Err(new CommerceValidationError("rateBps must be non-negative."));
     }
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    const rate = await repo.value.create(
+    const rate = await this.repository.create(
       {
         organizationId: orgId,
         name: input.name,
@@ -319,10 +297,8 @@ export class TaxService {
   }
 
   async listTaxRates(actor?: Actor | null, ctx?: TxContext): Promise<Result<TaxRate[]>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    return Ok(await repo.value.findAll(orgId, ctx));
+    return Ok(await this.repository.findAll(orgId, ctx));
   }
 
   async updateTaxRate(
@@ -339,12 +315,10 @@ export class TaxService {
     actor?: Actor | null,
     ctx?: TxContext,
   ): Promise<Result<TaxRate>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    const existing = await repo.value.findById(orgId, id, ctx);
+    const existing = await this.repository.findById(orgId, id, ctx);
     if (!existing) return Err(new CommerceNotFoundError("Tax rate not found."));
-    const updated = await repo.value.update(
+    const updated = await this.repository.update(
       id,
       {
         ...(patch.name !== undefined ? { name: patch.name } : {}),
@@ -363,12 +337,10 @@ export class TaxService {
   }
 
   async deleteTaxRate(id: string, actor?: Actor | null, ctx?: TxContext): Promise<Result<{ deleted: true }>> {
-    const repo = this.requireRepository();
-    if (!repo.ok) return repo;
     const orgId = resolveOrgIdForCommerce(actor ?? ctx?.actor ?? null, this.config);
-    const existing = await repo.value.findById(orgId, id, ctx);
+    const existing = await this.repository.findById(orgId, id, ctx);
     if (!existing) return Err(new CommerceNotFoundError("Tax rate not found."));
-    await repo.value.delete(id, ctx);
+    await this.repository.delete(id, ctx);
     return Ok({ deleted: true });
   }
 }

package/src/modules/webhooks/hook.ts

@@ -1,6 +1,7 @@
 import { resolveOrgIdForCommerce } from "../../auth/org.js";
 import type { AfterHook } from "../../kernel/hooks/types.js";
 import type { InventoryAdjustManyResult } from "../inventory/service.js";
+import { webhookDeliveryTask } from "./tasks.js";
 
 /**
  * Webhook delivery hook — enqueues delivery jobs instead of blocking.
@@ -27,7 +28,7 @@ export const deliverWebhooks: AfterHook<unknown> = async ({ result, operation, c
   if (!endpoints.ok) return;
 
   for (const endpoint of endpoints.value) {
-    await context.jobs.enqueue("webhooks/deliver", {
+    await context.jobs.enqueue(webhookDeliveryTask.slug, {
       endpointId: endpoint.id,
       endpointUrl: endpoint.url,
       endpointSecret: endpoint.secret,
@@ -58,7 +59,7 @@ export const deliverWebhooksForAdjustMany: AfterHook<InventoryAdjustManyResult>
   if (!endpoints.ok) return;
   for (const endpoint of endpoints.value) {
     for (const level of levels) {
-      await context.jobs.enqueue("webhooks/deliver", {
+      await context.jobs.enqueue(webhookDeliveryTask.slug, {
         endpointId: endpoint.id,
         endpointUrl: endpoint.url,
         endpointSecret: endpoint.secret,