@fantastic.dev/better-stripe 0.7.0 → 0.8.0

package/src/client/billing/checkout.ts

@@ -114,7 +114,7 @@ export async function createCheckoutSession(
     orgId?: string;
     stripePriceId: string;
     mode: "payment" | "subscription" | "setup";
-    uiMode?: "embedded" | "redirect";
+    uiMode?: "embedded" | "redirect" | "custom" | "form";
     quantity?: number;
     returnUrl: string;
     trialDays?: number;
@@ -295,8 +295,13 @@ export async function createCheckoutSession(
     sessionParams.payment_intent_data = paymentIntentData;
   }
 
-  if (uiMode === "embedded") {
-    sessionParams.ui_mode = "embedded_page";
+  if (uiMode === "embedded" || uiMode === "custom" || uiMode === "form") {
+    sessionParams.ui_mode =
+      uiMode === "custom"
+        ? "elements"
+        : uiMode === "form"
+          ? "form"
+          : "embedded_page";
     sessionParams.return_url = opts.returnUrl;
   } else {
     sessionParams.success_url = `${opts.returnUrl}?session_id={CHECKOUT_SESSION_ID}`;

package/src/client/billing/invoices.test.ts

@@ -75,7 +75,9 @@ describe("listInvoices", () => {
 
     expect(result).toEqual([]);
     const [ref, args] = ctx.runQuery.mock.calls[0];
-    expect(ref[TO_REF]).toBe(refPath(component, "billing/queries/listInvoices"));
+    expect(ref[TO_REF]).toBe(
+      refPath(component, "billing/queries/listInvoices"),
+    );
     expect(args).toEqual({});
   });
 
@@ -125,7 +127,9 @@ describe("listInvoicesByUser", () => {
 
     expect(result).toBe(rows);
     const [ref, args] = ctx.runQuery.mock.calls[0];
-    expect(ref[TO_REF]).toBe(refPath(component, "billing/queries/listInvoices"));
+    expect(ref[TO_REF]).toBe(
+      refPath(component, "billing/queries/listInvoices"),
+    );
     expect(args).toEqual({ userId: "u_9" });
   });
 });
@@ -177,9 +181,9 @@ describe("upsertInvoice", () => {
   it("throws when ctx has no runMutation (runMutationOrThrow guard)", async () => {
     const ctx = { runQuery: vi.fn() } as unknown as RunCtx;
 
-    await expect(
-      upsertInvoice(makeComponent(), ctx, baseOpts),
-    ).rejects.toThrow(/requires a Convex ctx with runMutation/);
+    await expect(upsertInvoice(makeComponent(), ctx, baseOpts)).rejects.toThrow(
+      /requires a Convex ctx with runMutation/,
+    );
   });
 });
 
@@ -189,7 +193,9 @@ describe("getInvoice", () => {
     const row = { stripeInvoiceId: "in_4", status: "paid" };
     const ctx = makeCtx(row);
 
-    const result = await getInvoice(component, ctx, { stripeInvoiceId: "in_4" });
+    const result = await getInvoice(component, ctx, {
+      stripeInvoiceId: "in_4",
+    });
 
     expect(result).toBe(row);
     const [ref, args] = ctx.runQuery.mock.calls[0];

package/src/client/connect/paymentMethods.test.ts

@@ -60,7 +60,9 @@ describe("payment methods are keyed to the customer_account (BTS-19 reuse)", ()
 
   it("detaches a payment method by id", async () => {
     const stripe = makeStripe();
-    await detachPaymentMethod(asStripe(stripe), ctx, { paymentMethodId: "pm_1" });
+    await detachPaymentMethod(asStripe(stripe), ctx, {
+      paymentMethodId: "pm_1",
+    });
     expect(stripe.paymentMethods.detach).toHaveBeenCalledWith("pm_1");
   });
 });

package/src/client/connect/paymentMethods.ts

@@ -51,5 +51,8 @@ export async function setDefaultPaymentMethod(
   _ctx: RunCtx,
   _opts: { stripeAccountId: string; paymentMethodId: string },
 ) {
-  throwStripeError("PAYMENT_METHOD_FAILED", "[better-stripe] setDefaultPaymentMethod is not yet implemented for V2 Accounts. Use createBillingPortalSession() to let users manage payment methods.");
+  throwStripeError(
+    "PAYMENT_METHOD_FAILED",
+    "[better-stripe] setDefaultPaymentMethod is not yet implemented for V2 Accounts. Use createBillingPortalSession() to let users manage payment methods.",
+  );
 }

package/src/client/connect/refundActor.ts

@@ -19,8 +19,7 @@ import type { SplitRecipientRole } from "../../component/lib/fees.js";
 
 /** Who is initiating a refund. `accountId` is the seller's `acct_…` id. */
 export type RefundActor =
-  | { type: "admin" }
-  | { type: "seller"; accountId: string };
+  { type: "admin" } | { type: "seller"; accountId: string };
 
 /**
  * The subset of a payment's routing needed to authorize a seller refund, read

package/src/client/connect/refunds.test.ts

@@ -53,7 +53,9 @@ function makeStripe(charge: Record<string, unknown>) {
         .mockResolvedValue({ id: "ch_1", payment_intent: "pi_1", ...charge }),
     },
     paymentIntents: {
-      retrieve: vi.fn().mockResolvedValue({ id: "pi_1", latest_charge: "ch_1" }),
+      retrieve: vi
+        .fn()
+        .mockResolvedValue({ id: "pi_1", latest_charge: "ch_1" }),
     },
   };
 }
@@ -62,12 +64,20 @@ const asStripe = (s: ReturnType<typeof makeStripe>) => s as unknown as Stripe;
 
 describe("createRefund — fee & transfer flags (BTS-34)", () => {
   it("defaults both flags on for a destination charge with an application fee", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
-
-    const result = await createRefund(asStripe(stripe), makeComponent(), noCtx, {
-      stripeChargeId: "ch_1",
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
     });
 
+    const result = await createRefund(
+      asStripe(stripe),
+      makeComponent(),
+      noCtx,
+      {
+        stripeChargeId: "ch_1",
+      },
+    );
+
     expect(result).toEqual({ stripeRefundId: "re_new" });
     expect(stripe.refunds.create).toHaveBeenCalledWith(
       {
@@ -80,7 +90,10 @@ describe("createRefund — fee & transfer flags (BTS-34)", () => {
   });
 
   it("keeps the flags on a partial refund (Stripe pro-rates both natively)", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
+    });
 
     await createRefund(asStripe(stripe), makeComponent(), noCtx, {
       stripeChargeId: "ch_1",
@@ -124,7 +137,10 @@ describe("createRefund — fee & transfer flags (BTS-34)", () => {
   });
 
   it("honours an explicit false even when the charge is capable", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
+    });
 
     await createRefund(asStripe(stripe), makeComponent(), noCtx, {
       stripeChargeId: "ch_1",
@@ -217,7 +233,10 @@ describe("createRefund — fee & transfer flags (BTS-34)", () => {
 
 describe("createRefund — actor scoping (BTS-35)", () => {
   it("lets a platform admin refund any sale without a ledger lookup", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
+    });
     const ctx = makeCtx();
 
     await createRefund(asStripe(stripe), makeComponent(), ctx, {
@@ -250,7 +269,10 @@ describe("createRefund — actor scoping (BTS-35)", () => {
   });
 
   it("rejects a seller refunding another seller's destination charge", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
+    });
     const ctx = makeCtx({
       stripePaymentIntentId: "pi_1",
       destinationAccountId: "acct_other",
@@ -307,7 +329,10 @@ describe("createRefund — actor scoping (BTS-35)", () => {
   });
 
   it("rejects a seller when no matching payment exists in the ledger (fail closed)", async () => {
-    const stripe = makeStripe({ transfer: "tr_auto", application_fee: "fee_1" });
+    const stripe = makeStripe({
+      transfer: "tr_auto",
+      application_fee: "fee_1",
+    });
     const ctx = makeCtx(null);
 
     await expect(

package/src/client/connect/refunds.ts

@@ -3,7 +3,10 @@ import type Stripe from "stripe";
 import type { RefundStatus } from "../../component/connect/validators.js";
 import { throwStripeError } from "../errors.js";
 import type { Component, RunCtx } from "../helpers.js";
-import type { StripeComponentPayment, StripeComponentRefund } from "../types.js";
+import type {
+  StripeComponentPayment,
+  StripeComponentRefund,
+} from "../types.js";
 import { componentRef } from "../webhooks/helpers.js";
 import { type RefundActor, isRefundAuthorized } from "./refundActor.js";
 

package/src/client/connect/transfers.test.ts

@@ -14,7 +14,9 @@ function makeComponent(): Component {
   const ref = (path: string) => ({ [TO_REF]: `betterStripe/${path}` });
   return {
     connect: {
-      queries: { listTransfersByCharge: ref("connect/queries/listTransfersByCharge") },
+      queries: {
+        listTransfersByCharge: ref("connect/queries/listTransfersByCharge"),
+      },
       mutations: {
         recordTransferReversal: ref("connect/mutations/recordTransferReversal"),
         upsertTransfer: ref("connect/mutations/upsertTransfer"),
@@ -56,7 +58,10 @@ function makeCtx(transfers: unknown[]) {
         legs.map((t) => ({
           stripeTransferId: t.stripeTransferId,
           amount: t.amount,
-          frontier: Math.max(t.reversedAmount ?? 0, t.reversalClaimedAmount ?? 0),
+          frontier: Math.max(
+            t.reversedAmount ?? 0,
+            t.reversalClaimedAmount ?? 0,
+          ),
         })),
         args.mode as Parameters<typeof computeReversalSlices>[1],
       );
@@ -117,7 +122,9 @@ describe("reverseTransfers (BTS-25)", () => {
       undefined,
     );
     // ledger updated with cumulative reversed amounts
-    const reversedAmounts = ctx.runMutation.mock.calls.map((c) => c[1].reversedAmount);
+    const reversedAmounts = ctx.runMutation.mock.calls.map(
+      (c) => c[1].reversedAmount,
+    );
     expect(reversedAmounts).toEqual([8000, 1000]);
   });
 
@@ -391,7 +398,12 @@ describe("createSplitTransfers idempotency (BTS-24)", () => {
     const stripe = makeStripe();
     // The store transfer already exists from a prior (partial) run.
     const ctx = makeCtx([
-      { stripeTransferId: "tr_store", destinationAccountId: "acct_store", role: "store", amount: 8000 },
+      {
+        stripeTransferId: "tr_store",
+        destinationAccountId: "acct_store",
+        role: "store",
+        amount: 8000,
+      },
     ]);
 
     await createSplitTransfers(asStripe(stripe), makeComponent(), ctx, {
@@ -414,8 +426,18 @@ describe("createSplitTransfers idempotency (BTS-24)", () => {
   it("creates nothing when all recipients are already in the ledger (full replay)", async () => {
     const stripe = makeStripe();
     const ctx = makeCtx([
-      { stripeTransferId: "tr_store", destinationAccountId: "acct_store", role: "store", amount: 8000 },
-      { stripeTransferId: "tr_aff", destinationAccountId: "acct_aff", role: "affiliate", amount: 1000 },
+      {
+        stripeTransferId: "tr_store",
+        destinationAccountId: "acct_store",
+        role: "store",
+        amount: 8000,
+      },
+      {
+        stripeTransferId: "tr_aff",
+        destinationAccountId: "acct_aff",
+        role: "affiliate",
+        amount: 1000,
+      },
     ]);
 
     await createSplitTransfers(asStripe(stripe), makeComponent(), ctx, {
@@ -436,8 +458,22 @@ describe("reinstateTransfers (BTS-29)", () => {
   it("re-creates transfers for the reversed amounts (dispute won)", async () => {
     const stripe = makeStripe();
     const ctx = makeCtx([
-      { stripeTransferId: "tr_store", destinationAccountId: "acct_store", role: "store", amount: 8000, currency: "usd", reversedAmount: 8000 },
-      { stripeTransferId: "tr_aff", destinationAccountId: "acct_aff", role: "affiliate", amount: 1000, currency: "usd", reversedAmount: 1000 },
+      {
+        stripeTransferId: "tr_store",
+        destinationAccountId: "acct_store",
+        role: "store",
+        amount: 8000,
+        currency: "usd",
+        reversedAmount: 8000,
+      },
+      {
+        stripeTransferId: "tr_aff",
+        destinationAccountId: "acct_aff",
+        role: "affiliate",
+        amount: 1000,
+        currency: "usd",
+        reversedAmount: 1000,
+      },
     ]);
 
     await reinstateTransfers(asStripe(stripe), makeComponent(), ctx, {
@@ -448,7 +484,11 @@ describe("reinstateTransfers (BTS-29)", () => {
 
     expect(stripe.transfers.create).toHaveBeenCalledTimes(2);
     expect(stripe.transfers.create).toHaveBeenCalledWith(
-      expect.objectContaining({ amount: 8000, currency: "usd", destination: "acct_store" }),
+      expect.objectContaining({
+        amount: 8000,
+        currency: "usd",
+        destination: "acct_store",
+      }),
       { idempotencyKey: "bs_reinstate_dp_1_acct_store_store" },
     );
     expect(stripe.transfers.create).toHaveBeenCalledWith(
@@ -457,14 +497,22 @@ describe("reinstateTransfers (BTS-29)", () => {
     );
     // Ledger rows are tagged reinstatement so they stay out of the charge's
     // split-leg view and can't be re-reversed.
-    expect(ctx.runMutation.mock.calls[0][1]).toMatchObject({ reinstatement: true });
+    expect(ctx.runMutation.mock.calls[0][1]).toMatchObject({
+      reinstatement: true,
+    });
   });
 
   it("uses a legacy row's fallback currency from the dispute, not USD", async () => {
     const stripe = makeStripe();
     const ctx = makeCtx([
       // Legacy row with no stored currency.
-      { stripeTransferId: "tr_x", destinationAccountId: "acct_x", role: "store", amount: 5000, reversedAmount: 5000 },
+      {
+        stripeTransferId: "tr_x",
+        destinationAccountId: "acct_x",
+        role: "store",
+        amount: 5000,
+        reversedAmount: 5000,
+      },
     ]);
     await reinstateTransfers(asStripe(stripe), makeComponent(), ctx, {
       sourceChargeId: "ch_1",
@@ -480,7 +528,13 @@ describe("reinstateTransfers (BTS-29)", () => {
   it("skips transfers that were never reversed", async () => {
     const stripe = makeStripe();
     const ctx = makeCtx([
-      { stripeTransferId: "tr_store", destinationAccountId: "acct_store", role: "store", amount: 8000, currency: "usd" },
+      {
+        stripeTransferId: "tr_store",
+        destinationAccountId: "acct_store",
+        role: "store",
+        amount: 8000,
+        currency: "usd",
+      },
     ]);
     await reinstateTransfers(asStripe(stripe), makeComponent(), ctx, {
       sourceChargeId: "ch_1",

package/src/client/connect/transfers.ts

@@ -78,7 +78,10 @@ async function executeClaimedSlices(
         await runMutationOrThrow(
           ctx,
           componentRef(component, "connect/mutations/recordTransferReversal"),
-          { stripeTransferId: slice.stripeTransferId, reversedAmount: slice.to },
+          {
+            stripeTransferId: slice.stripeTransferId,
+            reversedAmount: slice.to,
+          },
         );
         continue;
       }
@@ -242,7 +245,10 @@ export async function reclaimReversalClaim(
     ctx,
     componentRef(component, "connect/mutations/releaseReversalClaim"),
     { operationId: op.operationId, verified },
-  )) as { released: boolean; rewound: { stripeTransferId: string; to: number }[] };
+  )) as {
+    released: boolean;
+    rewound: { stripeTransferId: string; to: number }[];
+  };
   return { mode: "release", ...res };
 }
 
@@ -312,7 +318,13 @@ export async function reverseTransfers(
               : { kind: "full" as const },
       },
     )) as ReversalClaim;
-    return executeClaimedSlices(stripe, component, ctx, opts.operationId, claim);
+    return executeClaimedSlices(
+      stripe,
+      component,
+      ctx,
+      opts.operationId,
+      claim,
+    );
   }
 
   // Legacy one-shot path (no operationId): size against the claim frontier so
@@ -330,7 +342,10 @@ export async function reverseTransfers(
   const reversals: { stripeTransferId: string; amount: number }[] = [];
 
   for (const t of transfers) {
-    const already = Math.max(t.reversedAmount ?? 0, t.reversalClaimedAmount ?? 0);
+    const already = Math.max(
+      t.reversedAmount ?? 0,
+      t.reversalClaimedAmount ?? 0,
+    );
     const remaining = t.amount - already;
     if (remaining <= 0) continue; // already fully reversed — idempotent
 
@@ -338,7 +353,8 @@ export async function reverseTransfers(
     if (opts.percent !== undefined) {
       reversalAmt = Math.round((t.amount * opts.percent) / 100);
     } else if (amountBudget !== undefined) {
-      const prorata = total > 0 ? Math.round((opts.amount! * t.amount) / total) : 0;
+      const prorata =
+        total > 0 ? Math.round((opts.amount! * t.amount) / total) : 0;
       reversalAmt = Math.min(prorata, amountBudget);
     } else {
       reversalAmt = remaining; // full reversal
@@ -355,9 +371,15 @@ export async function reverseTransfers(
     await runMutationOrThrow(
       ctx,
       componentRef(component, "connect/mutations/recordTransferReversal"),
-      { stripeTransferId: t.stripeTransferId, reversedAmount: already + reversalAmt },
+      {
+        stripeTransferId: t.stripeTransferId,
+        reversedAmount: already + reversalAmt,
+      },
     );
-    reversals.push({ stripeTransferId: t.stripeTransferId, amount: reversalAmt });
+    reversals.push({
+      stripeTransferId: t.stripeTransferId,
+      amount: reversalAmt,
+    });
   }
 
   return { reversals };
@@ -461,7 +483,9 @@ export async function createSplitTransfers(
         metadata: { bsRole: t.role, bsSourceCharge: opts.sourceChargeId },
       },
       // Deterministic key so a webhook retry can't double-transfer (BTS-24).
-      { idempotencyKey: `bs_split_${opts.sourceChargeId}_${t.destinationAccountId}_${t.role}` },
+      {
+        idempotencyKey: `bs_split_${opts.sourceChargeId}_${t.destinationAccountId}_${t.role}`,
+      },
     );
     await runMutationOrThrow(
       ctx,

package/src/client/core/accountDefaults.ts

@@ -0,0 +1,15 @@
+/**
+ * Default V2 account configuration: merchant with card_payments.
+ */
+export const DEFAULT_ACCOUNT_CONFIGURATION: Record<string, unknown> = {
+  merchant: {
+    capabilities: { card_payments: { requested: true } },
+  },
+};
+
+export const DEFAULT_ACCOUNT_DEFAULTS: Record<string, unknown> = {
+  responsibilities: {
+    losses_collector: "application",
+    fees_collector: "application",
+  },
+};

package/src/client/core/accounts.ts

@@ -14,6 +14,15 @@ import { componentRef } from "../webhooks/helpers.js";
 import { validateStatementDescriptorSuffix } from "./descriptors.js";
 import { deriveAccountStatus } from "./accountStatus.js";
 
+import {
+  DEFAULT_ACCOUNT_CONFIGURATION,
+  DEFAULT_ACCOUNT_DEFAULTS,
+} from "./accountDefaults.js";
+export {
+  DEFAULT_ACCOUNT_CONFIGURATION,
+  DEFAULT_ACCOUNT_DEFAULTS,
+} from "./accountDefaults.js";
+
 // =============================================================================
 // Account CRUD
 // =============================================================================
@@ -76,22 +85,6 @@ export async function createAccount(
   };
 }
 
-/**
- * Default V2 account configuration: merchant with card_payments.
- */
-export const DEFAULT_ACCOUNT_CONFIGURATION: Record<string, unknown> = {
-  merchant: {
-    capabilities: { card_payments: { requested: true } },
-  },
-};
-
-export const DEFAULT_ACCOUNT_DEFAULTS: Record<string, unknown> = {
-  responsibilities: {
-    losses_collector: "application",
-    fees_collector: "application",
-  },
-};
-
 export async function createAccountWithOnboarding(
   stripe: Stripe,
   component: Component,
@@ -145,7 +138,11 @@ export async function createAccountWithOnboarding(
     } catch {
       // Best effort cleanup
     }
-    throwStripeError("ACCOUNT_CREATE_FAILED", "Failed to apply account configuration after create; account was rolled back", configError);
+    throwStripeError(
+      "ACCOUNT_CREATE_FAILED",
+      "Failed to apply account configuration after create; account was rolled back",
+      configError,
+    );
   }
 
   // Re-fetch account to get actual applied_configurations from Stripe

package/src/client/core/config.test.ts

@@ -37,7 +37,8 @@ describe("getPublishableKey", () => {
     const result = await getPublishableKey(component, ctx);
 
     expect(result).toBe("pk_test_123");
-    const [ref, args] = (ctx.runQuery as ReturnType<typeof vi.fn>).mock.calls[0];
+    const [ref, args] = (ctx.runQuery as ReturnType<typeof vi.fn>).mock
+      .calls[0];
     expect(ref[TO_REF]).toBe("betterStripe/core/queries/getPublishableKey");
     expect(args).toEqual({});
   });

package/src/client/core/descriptors.test.ts

@@ -38,15 +38,17 @@ describe("validateStatementDescriptorSuffix", () => {
   it.each(["<", ">", "\\", "'", '"', "*"])(
     "rejects the disallowed character %s",
     (char) => {
-      expect(() => validateStatementDescriptorSuffix(`STORE${char}NAME`)).toThrow(
-        /character/i,
-      );
+      expect(() =>
+        validateStatementDescriptorSuffix(`STORE${char}NAME`),
+      ).toThrow(/character/i);
     },
   );
 
   it("rejects a value with no letter (digits/punctuation only)", () => {
     expect(() => validateStatementDescriptorSuffix("12345")).toThrow(/letter/i);
-    expect(() => validateStatementDescriptorSuffix("---  9")).toThrow(/letter/i);
+    expect(() => validateStatementDescriptorSuffix("---  9")).toThrow(
+      /letter/i,
+    );
   });
 
   it("rejects non-ASCII (non-Latin) characters", () => {

package/src/client/core/fees.test.ts

@@ -14,7 +14,9 @@ describe("validatePlatformFee (BTS-13)", () => {
   });
 
   it("accepts percent + fixed", () => {
-    expect(() => validatePlatformFee({ percent: 2.9, fixed: 30 })).not.toThrow();
+    expect(() =>
+      validatePlatformFee({ percent: 2.9, fixed: 30 }),
+    ).not.toThrow();
   });
 
   it("accepts ascending tiers with a null final upper bound", () => {
@@ -219,14 +221,10 @@ describe("computeSplit (BTS-23)", () => {
   const aff = "acct_aff";
 
   it("splits with fixed recipient amounts; platform retains the rest", () => {
-    const r = computeSplit(
-      10000,
-      { percent: 10 },
-      [
-        { destinationAccountId: store, role: "store", amount: 8000 },
-        { destinationAccountId: aff, role: "affiliate", amount: 1000 },
-      ],
-    );
+    const r = computeSplit(10000, { percent: 10 }, [
+      { destinationAccountId: store, role: "store", amount: 8000 },
+      { destinationAccountId: aff, role: "affiliate", amount: 1000 },
+    ]);
     expect(r.transfers).toEqual([
       { destinationAccountId: store, role: "store", amount: 8000 },
       { destinationAccountId: aff, role: "affiliate", amount: 1000 },
@@ -239,14 +237,10 @@ describe("computeSplit (BTS-23)", () => {
   });
 
   it("supports percent-of-gross recipients", () => {
-    const r = computeSplit(
-      10000,
-      { percent: 10 },
-      [
-        { destinationAccountId: store, role: "store", percent: 80 },
-        { destinationAccountId: aff, role: "affiliate", percent: 5 },
-      ],
-    );
+    const r = computeSplit(10000, { percent: 10 }, [
+      { destinationAccountId: store, role: "store", percent: 80 },
+      { destinationAccountId: aff, role: "affiliate", percent: 5 },
+    ]);
     expect(r.transfers.map((t) => t.amount)).toEqual([8000, 500]);
     expect(r.platformRetained).toBe(1500); // 10000 - 8500
   });
@@ -269,15 +263,11 @@ describe("computeSplit (BTS-23)", () => {
 
   it("assigns the rounding remainder deterministically to the platform", () => {
     // 33% + 33% + 33% of 100 → 33 + 33 + 33 = 99; platform keeps the 1 remainder.
-    const r = computeSplit(
-      100,
-      undefined,
-      [
-        { destinationAccountId: store, role: "store", percent: 33 },
-        { destinationAccountId: aff, role: "affiliate", percent: 33 },
-        { destinationAccountId: "acct_c", role: "other", percent: 33 },
-      ],
-    );
+    const r = computeSplit(100, undefined, [
+      { destinationAccountId: store, role: "store", percent: 33 },
+      { destinationAccountId: aff, role: "affiliate", percent: 33 },
+      { destinationAccountId: "acct_c", role: "other", percent: 33 },
+    ]);
     const sum = r.transfers.reduce((s, t) => s + t.amount, 0);
     expect(sum).toBe(99);
     expect(r.platformRetained).toBe(1);
@@ -300,7 +290,7 @@ describe("computeSplit (BTS-23)", () => {
     expect(r.platformFee).toBe(0);
     expect(r.platformRetained).toBe(0);
   });
-})
+});
 
 describe("validateSplit (BTS-21)", () => {
   const r = (over = {}) => ({
@@ -354,7 +344,9 @@ describe("validateSplit (BTS-21)", () => {
   });
 
   it("rejects an empty destinationAccountId", () => {
-    expect(() => validateSplit([r({ destinationAccountId: "", amount: 100 })])).toThrow();
+    expect(() =>
+      validateSplit([r({ destinationAccountId: "", amount: 100 })]),
+    ).toThrow();
   });
 
   it("rejects duplicate destination+role legs (idempotency-key collision)", () => {

package/src/client/core/fees.ts

@@ -201,7 +201,11 @@ export function computeSplit(
         `Invalid split amount for ${r.destinationAccountId}: ${amt}`,
       );
     }
-    return { destinationAccountId: r.destinationAccountId, role: r.role, amount: amt };
+    return {
+      destinationAccountId: r.destinationAccountId,
+      role: r.role,
+      amount: amt,
+    };
   });
 
   const sumTransfers = transfers.reduce((s, t) => s + t.amount, 0);
@@ -247,7 +251,10 @@ export function validateSplit(recipients: SplitRecipient[]): void {
         `split recipient ${r.destinationAccountId} must set exactly one of amount or percent`,
       );
     }
-    if (hasAmount && (!Number.isInteger(r.amount) || (r.amount as number) < 0)) {
+    if (
+      hasAmount &&
+      (!Number.isInteger(r.amount) || (r.amount as number) < 0)
+    ) {
       throw new Error(
         `split recipient ${r.destinationAccountId} amount must be a non-negative integer (minor units)`,
       );