@tailor-platform/erp-kit 0.7.0 → 0.8.0

package/src/commands/update.test.ts

@@ -74,7 +74,7 @@ describe("runUpdate", () => {
     fs.writeFileSync(path.join(wfDir, "erp-kit-check.yml"), "# Old");
     runUpdate(tmpDir, ["workflows"]);
     const content = fs.readFileSync(path.join(wfDir, "erp-kit-check.yml"), "utf-8");
-    expect(content).toContain("@tailor-platform/erp-kit");
+    expect(content).toContain("erp-kit Check");
   });
 
   it("refreshes .claude/skills when it is a copied directory (not a symlink)", () => {

package/src/generator/scaffold.ts

@@ -2,24 +2,33 @@ import fs from "node:fs";
 import path from "node:path";
 import { PACKAGE_ROOT, readErpKitVersion } from "../util";
 
+// Files that need renaming because they cause issues in the repo/pack.
 // npm renames .gitignore during pack (npm/cli#5756).
-// prepack copies .gitignore to __dot__gitignore to preserve it.
+// pnpm-workspace.yaml causes the template dir to be treated as a workspace member.
+// prepack copies these to their safe names; scaffold time restores the originals.
+const RENAME_MAP: Record<string, string> = {
+  __dot__gitignore: ".gitignore",
+  "__pnpm-workspace.yaml": "pnpm-workspace.yaml",
+};
+
 export function copyTemplateDir(
   srcDir: string,
   destDir: string,
   replacements: Record<string, string>,
   placeholderFiles: Set<string>,
+  excludeDirs = new Set<string>(),
 ): void {
   for (const entry of fs.readdirSync(srcDir, { withFileTypes: true })) {
     const srcPath = path.join(srcDir, entry.name);
-    const destName = entry.name === "__dot__gitignore" ? ".gitignore" : entry.name;
+    const destName = RENAME_MAP[entry.name] ?? entry.name;
     const destPath = path.join(destDir, destName);
 
     if (entry.name === "node_modules") continue;
+    if (excludeDirs.has(entry.name)) continue;
 
     if (entry.isDirectory()) {
       fs.mkdirSync(destPath, { recursive: true });
-      copyTemplateDir(srcPath, destPath, replacements, placeholderFiles);
+      copyTemplateDir(srcPath, destPath, replacements, placeholderFiles, excludeDirs);
     } else {
       if (fs.existsSync(destPath)) continue;
       fs.mkdirSync(path.dirname(destPath), { recursive: true });
@@ -57,5 +66,17 @@ export function scaffoldAppBoilerplate(appDir: string, appName: string): void {
     '"workspace:*"': `"${erpKitVersion}"`,
   };
   const placeholderFiles = new Set(["package.json", "tailor.config.ts", "index.html"]);
-  copyTemplateDir(templateDir, appDir, replacements, placeholderFiles);
+  // docs are authored by skills (app-1 through app-4), not scaffolded.
+  copyTemplateDir(templateDir, appDir, replacements, placeholderFiles, new Set(["docs"]));
+}
+
+export function scaffoldProjectBoilerplate(projectDir: string, projectName: string): void {
+  const templateDir = path.join(PACKAGE_ROOT, "templates", "scaffold", "project");
+  const erpKitVersion = readErpKitVersion();
+  const replacements = {
+    "template-project": projectName,
+    '"workspace:*"': `"${erpKitVersion}"`,
+  };
+  const placeholderFiles = new Set(["package.json"]);
+  copyTemplateDir(templateDir, projectDir, replacements, placeholderFiles);
 }

package/src/module.ts

@@ -44,7 +44,10 @@ export {
   all as itemManagementAll,
 } from "./modules/item-management/lib/permissions.generated";
 // audit
-export { defineModule as defineAuditModule } from "./modules/audit/module";
+export {
+  defineModule as defineAuditModule,
+  companyStub as auditCompanyStub,
+} from "./modules/audit/module";
 export {
   permissions as auditPermissions,
   own as auditOwn,

package/src/modules/accounting/README.md

@@ -0,0 +1,63 @@
+# README
+
+## Overview
+
+The accounting module provides management (internal) accounting capabilities for budgeting, cost allocation, profit center reporting, and variance analysis. It enables organizations to track, allocate, and analyze costs for internal decision-making, complementing the financial accounting capabilities provided by `finance-ledger` and `coa-management`.
+
+This module owns cost centers, profit centers, cost elements, budgets, internal orders, and allocation cycles — entities that exist solely for management accounting purposes and are distinct from the general ledger structure.
+
+## Key Features
+
+- **Cost Center Management** — Create and maintain cost centers with hierarchical grouping for overhead cost accumulation and reporting
+- **Cost Element Management** — Define primary cost elements (mapped to GL accounts) and secondary cost elements (internal allocation only)
+- **Profit Center Management** — Create and maintain profit centers with hierarchical grouping for internal P&L reporting
+- **Budget Management** — Create budgets with line items per cost element, manage approval workflows, track budget versions, and enforce budgetary control with commitment-aware availability checks
+- **Cost Allocation** — Execute allocation cycles (assessment/distribution) to distribute overhead costs from sender to receiver cost objects
+- **Internal Order Management** — Track costs for specific short-to-medium-term activities (projects, campaigns, events) with lifecycle management
+- **Variance Analysis** — Compare planned vs. actual costs by cost object and period, categorize variances, and generate management reports
+
+## Module Scope
+
+### In Scope
+
+- Cost center CRUD and hierarchy management
+- Profit center CRUD and hierarchy management
+- Primary and secondary cost element definition
+- Budget creation, approval workflow, and line item entry per cost element
+- Budgetary control with commitment-aware availability checks (Budget - Actual - Commitments - Encumbrances)
+- Commitment tracking via purchase module event consumption (PO confirmed, received, invoiced, cancelled)
+- Budget vs. actual comparison
+- Cost allocation cycles (assessment and distribution methods)
+- Allocation rule definition and execution
+- Internal order lifecycle (created → released → technically completed → closed)
+- Variance calculation and categorization (plan vs. actual) with cost-element-level detail lines
+- Multi-version planning (budget, forecast)
+- Audit trail for budget changes, allocation runs, and variance postings
+
+### Out of Scope
+
+- Financial (external) accounting — owned by `finance-ledger`
+- Chart of accounts and GL account management — owned by `coa-management`
+- Fiscal year and accounting period management — owned by `finance-ledger`
+- Activity-based costing (activity types, rate calculation) — future extension
+- Statistical key figures as allocation drivers — future extension
+- Multi-dimensional profitability analysis (CO-PA style) — future extension
+- What-if scenario modeling and simulation — future extension
+- Internal order settlement to receivers — future extension
+- Rolling forecasts beyond fiscal year boundaries — future extension
+- Manufacturing variance calculation — owned by `manufacturing`; this module consumes results
+- Transfer pricing between legal entities — future extension
+
+### Scope Decision Rationale
+
+The initial scope focuses on the six core entities identified in the roadmap (Budget, BudgetLine, ProfitCenter, CostAllocation, InternalOrder, Variance) plus foundational entities (CostCenter, CostElement) required by all management accounting features. Comprehensive features like activity-based costing and multi-dimensional profitability analysis are deferred as they add significant complexity without being prerequisites for the core budgeting-allocation-variance workflow. The module explicitly does not recreate entities already owned by other modules (fiscal calendar from finance-ledger, GL accounts from coa-management, manufacturing variances from manufacturing).
+
+## Module Dependencies
+
+- `primitives` — Currency definitions for multi-currency budget amounts and variance values
+- `organization` — Company scoping for all management accounting entities; Department/Site as optional mapping targets for cost centers and profit centers
+- `coa-management` — GL account references for primary cost elements, budget line items, and allocation target accounts
+- `finance-ledger` — Fiscal year/period alignment for budget periods; journal entry data as source of actuals; subledger integration pattern for posting allocation journal entries
+- `purchase` — Commitment event consumption for budgetary control: PO confirmation creates commitments, goods receipt and invoice match consume commitments, PO cancellation releases commitments
+- `user-management` — Permission definitions for budget creation, allocation execution, and report access
+- `audit` — Audit trail for budget lifecycle transitions, allocation runs, and variance postings

package/src/modules/accounting/command/activateBudget.generated.ts

@@ -0,0 +1,6 @@
+// @generated — do not edit
+import { permissions } from "../lib/permissions.generated";
+import { run } from "./activateBudget";
+import { defineCommand } from "@tailor-platform/erp-kit/module";
+
+export const activateBudget = defineCommand(permissions.activateBudget, run);

package/src/modules/accounting/command/activateBudget.test.ts

@@ -0,0 +1,119 @@
+import { describe, expect, it } from "vitest";
+import { createMockDb } from "../../../testing/index";
+import type { Transaction } from "../generated/kysely-tailordb";
+import {
+  BudgetNotFoundError,
+  InvalidStateError,
+  DuplicateActiveBudgetError,
+} from "../lib/errors.generated";
+import { approvedBudget, baseBudget, activeBudget, basePlanVersion } from "../testing/fixtures";
+import { commandCtx, expectErr, expectOk } from "../testing/commandTestUtils";
+import { run } from "./activateBudget";
+
+const forecastPlanVersion = { ...basePlanVersion, id: "plan-version-forecast", type: "FORECAST" };
+const simulationPlanVersion = { ...basePlanVersion, id: "plan-version-sim", type: "SIMULATION" };
+
+describe("activateBudget", () => {
+  it("activates an APPROVED budget transitioning it to ACTIVE status", async () => {
+    const activatedBudget = { ...approvedBudget, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select
+      .mockReturnValueOnce(approvedBudget)
+      .mockReturnValueOnce(basePlanVersion) // plan version type BUDGET
+      .mockReturnValueOnce(undefined); // no existing active
+    spies.update.mockReturnValueOnce(activatedBudget);
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.budget.status).toBe("ACTIVE");
+  });
+
+  it("enables funds checking upon activation", async () => {
+    const activatedBudget = { ...approvedBudget, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select
+      .mockReturnValueOnce(approvedBudget)
+      .mockReturnValueOnce(basePlanVersion)
+      .mockReturnValueOnce(undefined);
+    spies.update.mockReturnValueOnce(activatedBudget);
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.budget.status).toBe("ACTIVE");
+  });
+
+  it("returns error when budget does not exist", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(undefined);
+
+    const result = await run(db, { budgetId: "missing" }, commandCtx);
+
+    expectErr(result, BudgetNotFoundError);
+  });
+
+  it("returns error when budget is not in APPROVED status", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseBudget); // DRAFT
+
+    const result = await run(db, { budgetId: baseBudget.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("returns error when another ACTIVE BUDGET exists for the same cost object and fiscal year", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select
+      .mockReturnValueOnce(approvedBudget)
+      .mockReturnValueOnce(basePlanVersion) // type BUDGET
+      .mockReturnValueOnce(activeBudget) // existing active
+      .mockReturnValueOnce(basePlanVersion); // existing active's plan version is also BUDGET
+    // no update mock needed — should return error
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    expectErr(result, DuplicateActiveBudgetError);
+  });
+
+  it("allows multiple ACTIVE FORECAST versions for the same cost object and fiscal year", async () => {
+    const forecastApproved = { ...approvedBudget, planVersionId: forecastPlanVersion.id };
+    const activatedBudget = { ...forecastApproved, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(forecastApproved).mockReturnValueOnce(forecastPlanVersion); // type FORECAST — skip uniqueness check
+    spies.update.mockReturnValueOnce(activatedBudget);
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.budget.status).toBe("ACTIVE");
+  });
+
+  it("allows multiple ACTIVE SIMULATION versions for the same cost object and fiscal year", async () => {
+    const simApproved = { ...approvedBudget, planVersionId: simulationPlanVersion.id };
+    const activatedBudget = { ...simApproved, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(simApproved).mockReturnValueOnce(simulationPlanVersion); // type SIMULATION — skip uniqueness check
+    spies.update.mockReturnValueOnce(activatedBudget);
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.budget.status).toBe("ACTIVE");
+  });
+
+  it("emits audit event recording the status transition and acting user", async () => {
+    const activatedBudget = { ...approvedBudget, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select
+      .mockReturnValueOnce(approvedBudget)
+      .mockReturnValueOnce(basePlanVersion)
+      .mockReturnValueOnce(undefined);
+    spies.update.mockReturnValueOnce(activatedBudget);
+
+    const result = await run(db, { budgetId: approvedBudget.id }, commandCtx);
+
+    expectOk(result);
+    expect(spies.update).toHaveBeenCalled();
+  });
+});

package/src/modules/accounting/command/activateBudget.ts

@@ -0,0 +1,77 @@
+import type { Transaction } from "../generated/kysely-tailordb";
+import {
+  BudgetNotFoundError,
+  InvalidStateError,
+  DuplicateActiveBudgetError,
+} from "../lib/errors.generated";
+import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
+
+export interface ActivateBudgetInput {
+  budgetId: string;
+}
+
+export async function run(db: Transaction, input: ActivateBudgetInput, ctx: CommandContext) {
+  void ctx;
+
+  // Fetch budget with lock
+  const budget = await db
+    .selectFrom("Budget")
+    .where("id", "=", input.budgetId)
+    .selectAll()
+    .forUpdate()
+    .executeTakeFirst();
+
+  if (!budget) {
+    return err(new BudgetNotFoundError(input.budgetId));
+  }
+
+  if (budget.status !== "APPROVED") {
+    return err(new InvalidStateError(input.budgetId));
+  }
+
+  // Check plan version type — only BUDGET type needs uniqueness check
+  const planVersion = await db
+    .selectFrom("PlanVersion")
+    .where("id", "=", budget.planVersionId)
+    .selectAll()
+    .executeTakeFirst();
+
+  if (planVersion?.type === "BUDGET") {
+    // Check no other ACTIVE budget of type BUDGET exists for same cost object and fiscal year
+    const existingActive = await db
+      .selectFrom("Budget")
+      .where("costObjectType", "=", budget.costObjectType)
+      .where("costObjectId", "=", budget.costObjectId)
+      .where("fiscalYearId", "=", budget.fiscalYearId)
+      .where("status", "=", "ACTIVE")
+      .where("id", "!=", input.budgetId)
+      .selectAll()
+      .executeTakeFirst();
+
+    if (existingActive) {
+      // Verify the existing active budget is also of type BUDGET
+      const existingPlanVersion = await db
+        .selectFrom("PlanVersion")
+        .where("id", "=", existingActive.planVersionId)
+        .selectAll()
+        .executeTakeFirst();
+
+      if (existingPlanVersion?.type === "BUDGET") {
+        return err(new DuplicateActiveBudgetError(input.budgetId));
+      }
+    }
+  }
+
+  // Transition to ACTIVE (which enables funds checking)
+  const updated = await db
+    .updateTable("Budget")
+    .set({
+      status: "ACTIVE",
+      updatedAt: new Date(),
+    })
+    .where("id", "=", input.budgetId)
+    .returningAll()
+    .executeTakeFirst();
+
+  return ok({ budget: updated! });
+}

package/src/modules/accounting/command/activateCostElement.generated.ts

@@ -0,0 +1,6 @@
+// @generated — do not edit
+import { permissions } from "../lib/permissions.generated";
+import { run } from "./activateCostElement";
+import { defineCommand } from "@tailor-platform/erp-kit/module";
+
+export const activateCostElement = defineCommand(permissions.activateCostElement, run);

package/src/modules/accounting/command/activateCostElement.test.ts

@@ -0,0 +1,59 @@
+import { describe, expect, it } from "vitest";
+import { createMockDb } from "../../../testing/index";
+import type { Transaction } from "../generated/kysely-tailordb";
+import { CostElementNotFoundError, InvalidStateError } from "../lib/errors.generated";
+import { baseCostElement, activeCostElement, inactiveCostElement } from "../testing/fixtures";
+import { commandCtx, expectErr, expectOk } from "../testing/commandTestUtils";
+import { run } from "./activateCostElement";
+
+describe("activateCostElement", () => {
+  it("activates a DRAFT cost element transitioning it to ACTIVE status", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseCostElement);
+    const updated = { ...baseCostElement, status: "ACTIVE" };
+    spies.update.mockReturnValueOnce(updated);
+
+    const result = await run(db, { costElementId: baseCostElement.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.costElement.status).toBe("ACTIVE");
+  });
+
+  it("returns error when cost element does not exist", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(undefined);
+
+    const result = await run(db, { costElementId: "missing" }, commandCtx);
+
+    expectErr(result, CostElementNotFoundError);
+  });
+
+  it("returns error when cost element is already ACTIVE", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(activeCostElement);
+
+    const result = await run(db, { costElementId: activeCostElement.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("returns error when cost element is INACTIVE", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(inactiveCostElement);
+
+    const result = await run(db, { costElementId: inactiveCostElement.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("emits audit event recording the status transition and acting user", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseCostElement);
+    spies.update.mockReturnValueOnce({ ...baseCostElement, status: "ACTIVE" });
+
+    const result = await run(db, { costElementId: baseCostElement.id }, commandCtx);
+
+    expectOk(result);
+    expect(spies.update).toHaveBeenCalled();
+  });
+});

package/src/modules/accounting/command/activateCostElement.ts

@@ -0,0 +1,41 @@
+import type { Transaction } from "../generated/kysely-tailordb";
+import { CostElementNotFoundError, InvalidStateError } from "../lib/errors.generated";
+import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
+
+export interface ActivateCostElementInput {
+  costElementId: string;
+  from?: string[];
+}
+
+/** Command: activateCostElement
+ * Transitions a cost element from DRAFT status to ACTIVE status.
+ */
+export async function run(db: Transaction, input: ActivateCostElementInput, ctx: CommandContext) {
+  void ctx;
+
+  const validFromStatuses = input.from ?? ["DRAFT"];
+
+  const costElement = await db
+    .selectFrom("CostElement")
+    .selectAll()
+    .where("id", "=", input.costElementId)
+    .forUpdate()
+    .executeTakeFirst();
+
+  if (!costElement) {
+    return err(new CostElementNotFoundError(input.costElementId));
+  }
+
+  if (!validFromStatuses.includes(costElement.status)) {
+    return err(new InvalidStateError(input.costElementId));
+  }
+
+  const updated = await db
+    .updateTable("CostElement")
+    .set({ status: "ACTIVE", updatedAt: new Date() })
+    .where("id", "=", input.costElementId)
+    .returningAll()
+    .executeTakeFirst();
+
+  return ok({ costElement: updated! });
+}

package/src/modules/accounting/command/activateProfitCenter.generated.ts

@@ -0,0 +1,6 @@
+// @generated — do not edit
+import { permissions } from "../lib/permissions.generated";
+import { run } from "./activateProfitCenter";
+import { defineCommand } from "@tailor-platform/erp-kit/module";
+
+export const activateProfitCenter = defineCommand(permissions.activateProfitCenter, run);

package/src/modules/accounting/command/activateProfitCenter.test.ts

@@ -0,0 +1,60 @@
+import { describe, expect, it } from "vitest";
+import { createMockDb } from "../../../testing/index";
+import type { Transaction } from "../generated/kysely-tailordb";
+import { ProfitCenterNotFoundError, InvalidStateError } from "../lib/errors.generated";
+import { baseProfitCenter, activeProfitCenter, inactiveProfitCenter } from "../testing/fixtures";
+import { commandCtx, expectErr, expectOk } from "../testing/commandTestUtils";
+import { run } from "./activateProfitCenter";
+
+describe("activateProfitCenter", () => {
+  it("activates a DRAFT profit center transitioning it to ACTIVE status", async () => {
+    const activated = { ...baseProfitCenter, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseProfitCenter);
+    spies.update.mockReturnValueOnce(activated);
+
+    const result = await run(db, { profitCenterId: baseProfitCenter.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.profitCenter.status).toBe("ACTIVE");
+  });
+
+  it("returns error when profit center does not exist", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(undefined);
+
+    const result = await run(db, { profitCenterId: "missing-id" }, commandCtx);
+
+    expectErr(result, ProfitCenterNotFoundError);
+  });
+
+  it("returns error when profit center is already ACTIVE", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(activeProfitCenter);
+
+    const result = await run(db, { profitCenterId: activeProfitCenter.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("returns error when profit center is INACTIVE", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(inactiveProfitCenter);
+
+    const result = await run(db, { profitCenterId: inactiveProfitCenter.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("emits audit event recording the status transition and acting user", async () => {
+    const activated = { ...baseProfitCenter, status: "ACTIVE" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseProfitCenter);
+    spies.update.mockReturnValueOnce(activated);
+
+    const result = await run(db, { profitCenterId: baseProfitCenter.id }, commandCtx);
+
+    expectOk(result);
+    expect(spies.update).toHaveBeenCalled();
+  });
+});

package/src/modules/accounting/command/activateProfitCenter.ts

@@ -0,0 +1,38 @@
+import type { Transaction } from "../generated/kysely-tailordb";
+import { ProfitCenterNotFoundError, InvalidStateError } from "../lib/errors.generated";
+import { err, ok, type CommandContext } from "@tailor-platform/erp-kit/module";
+
+export interface ActivateProfitCenterInput {
+  profitCenterId: string;
+  from?: string[];
+}
+
+export async function run(db: Transaction, input: ActivateProfitCenterInput, ctx: CommandContext) {
+  void ctx;
+
+  const validFromStatuses = input.from ?? ["DRAFT"];
+
+  const profitCenter = await db
+    .selectFrom("ProfitCenter")
+    .selectAll()
+    .where("id", "=", input.profitCenterId)
+    .forUpdate()
+    .executeTakeFirst();
+
+  if (!profitCenter) {
+    return err(new ProfitCenterNotFoundError(input.profitCenterId));
+  }
+
+  if (!validFromStatuses.includes(profitCenter.status)) {
+    return err(new InvalidStateError(input.profitCenterId));
+  }
+
+  const updated = await db
+    .updateTable("ProfitCenter")
+    .set({ status: "ACTIVE", updatedAt: new Date() })
+    .where("id", "=", input.profitCenterId)
+    .returningAll()
+    .executeTakeFirst();
+
+  return ok({ profitCenter: updated! });
+}

package/src/modules/accounting/command/approveBudget.generated.ts

@@ -0,0 +1,6 @@
+// @generated — do not edit
+import { permissions } from "../lib/permissions.generated";
+import { run } from "./approveBudget";
+import { defineCommand } from "@tailor-platform/erp-kit/module";
+
+export const approveBudget = defineCommand(permissions.approveBudget, run);

package/src/modules/accounting/command/approveBudget.test.ts

@@ -0,0 +1,69 @@
+import { describe, expect, it } from "vitest";
+import { createMockDb } from "../../../testing/index";
+import type { Transaction } from "../generated/kysely-tailordb";
+import {
+  BudgetNotFoundError,
+  InvalidStateError,
+  SelfApprovalNotAllowedError,
+} from "../lib/errors.generated";
+import { submittedBudget, baseBudget, baseUser } from "../testing/fixtures";
+import { commandCtx, expectErr, expectOk } from "../testing/commandTestUtils";
+import { run } from "./approveBudget";
+
+// submittedBudget.createdByUserId === baseUser.id === "user-1"
+// commandCtx.actorId === "test-actor" (different from creator)
+
+describe("approveBudget", () => {
+  it("approves a SUBMITTED budget transitioning it to APPROVED status", async () => {
+    const approvedResult = { ...submittedBudget, status: "APPROVED" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(submittedBudget);
+    spies.update.mockReturnValueOnce(approvedResult);
+
+    const result = await run(db, { budgetId: submittedBudget.id }, commandCtx);
+
+    const value = expectOk(result);
+    expect(value.budget.status).toBe("APPROVED");
+  });
+
+  it("returns error when budget does not exist", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(undefined);
+
+    const result = await run(db, { budgetId: "missing" }, commandCtx);
+
+    expectErr(result, BudgetNotFoundError);
+  });
+
+  it("returns error when budget is not in SUBMITTED status", async () => {
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(baseBudget); // DRAFT
+
+    const result = await run(db, { budgetId: baseBudget.id }, commandCtx);
+
+    expectErr(result, InvalidStateError);
+  });
+
+  it("returns error when approver is the same as the budget creator", async () => {
+    // createdByUserId = "user-1", set actorId to match
+    const selfCtx = { ...commandCtx, actorId: baseUser.id };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(submittedBudget);
+
+    const result = await run(db, { budgetId: submittedBudget.id }, selfCtx);
+
+    expectErr(result, SelfApprovalNotAllowedError);
+  });
+
+  it("emits audit event recording the status transition and approver", async () => {
+    const approvedResult = { ...submittedBudget, status: "APPROVED" };
+    const { db, spies } = createMockDb<Transaction>();
+    spies.select.mockReturnValueOnce(submittedBudget);
+    spies.update.mockReturnValueOnce(approvedResult);
+
+    const result = await run(db, { budgetId: submittedBudget.id }, commandCtx);
+
+    expectOk(result);
+    expect(spies.update).toHaveBeenCalled();
+  });
+});