@birtalanrobert/commerce 2.0.0 → 3.0.0

package/README.md

@@ -27,7 +27,7 @@ canTakeMoney('pending'); // false — the provider has not verified them yet
 ## Using it in a NestJS application
 
 ```ts
-import { StripeConnect } from '@birtalanrobert/commerce';
+import { StripeConnect } from '@birtalanrobert/commerce/stripe';
 import {
   COMMERCE_PROVIDER,
   CommerceService,
@@ -55,6 +55,14 @@ export class PaymentsModule {}
 Register `commerceEntities` and `commerceMigrations` with the data source, as
 with every other package here.
 
+**Three entry points, and the split is deliberate.** The root is pure — what a
+deposit comes to, whether a business may sell yet, whether a payment can still
+be refunded — because a console decides all three while somebody drags a
+slider. `/nestjs` holds the service, the entities and the migrations.
+`/stripe` holds the vendor's client on its own, because importing it pulls in
+the whole SDK, and a barrel that re-exported it once put a payments SDK into a
+booking console's first load.
+
 ## The parts worth knowing before using it
 
 - **A business cannot be charged for until the provider says it may be paid

package/dist/index.d.ts

@@ -12,13 +12,14 @@
  * the mistake that makes both hard to reason about — they differ in who pays
  * whom, in which Stripe account, and in what happens when one fails.
  *
- * **This entry point is pure.** What a deposit comes to and whether a business
- * may sell yet are decided without a database or a provider, because a console
- * shows both while somebody drags a slider. Everything needing storage or
- * somebody else's money-moving licence is behind `/nestjs`.
+ * **This entry point is pure**, and stays that way by leaving things out. What
+ * a deposit comes to and whether a business may sell yet are decided without a
+ * database or a provider, because a console shows both while somebody drags a
+ * slider. Storage and the container are behind `/nestjs`; the Stripe client is
+ * behind `/stripe`, because a barrel that re-exports it puts the vendor's whole
+ * SDK into every bundle that wanted to divide a price by three.
  */
 export { balanceAfter, canTakeMoney, depositFor, payoutBlockReason, type DepositKind, type DepositPolicy, type PayoutStatus, } from './deposits';
 export { isRefundable, refundableAmount, type PaymentKind, type PaymentMethod, type PaymentState, } from './payments';
 export type { ChargeRequest, ChargeResult, OnboardingLink, PaymentProvider, ProviderAccount, ProviderEvent, RefundRequest, SaveCardRequest, SaveCardResult, StoredCard, } from './providers/port';
-export { StripeConnect, type StripeConnectOptions } from './providers/stripe';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AACH,OAAO,EACL,YAAY,EACZ,YAAY,EACZ,UAAU,EACV,iBAAiB,EACjB,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,YAAY,GAClB,MAAM,YAAY,CAAC;AAEpB,OAAO,EACL,YAAY,EACZ,gBAAgB,EAChB,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,YAAY,GAClB,MAAM,YAAY,CAAC;AAEpB,YAAY,EACV,aAAa,EACb,YAAY,EACZ,cAAc,EACd,eAAe,EACf,eAAe,EACf,aAAa,EACb,aAAa,EACb,eAAe,EACf,cAAc,EACd,UAAU,GACX,MAAM,kBAAkB,CAAC;AAE1B,OAAO,EAAE,aAAa,EAAE,KAAK,oBAAoB,EAAE,MAAM,oBAAoB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EACL,YAAY,EACZ,YAAY,EACZ,UAAU,EACV,iBAAiB,EACjB,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,YAAY,GAClB,MAAM,YAAY,CAAC;AAEpB,OAAO,EACL,YAAY,EACZ,gBAAgB,EAChB,KAAK,WAAW,EAChB,KAAK,aAAa,EAClB,KAAK,YAAY,GAClB,MAAM,YAAY,CAAC;AAEpB,YAAY,EACV,aAAa,EACb,YAAY,EACZ,cAAc,EACd,eAAe,EACf,eAAe,EACf,aAAa,EACb,aAAa,EACb,eAAe,EACf,cAAc,EACd,UAAU,GACX,MAAM,kBAAkB,CAAC"}

package/dist/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.StripeConnect = exports.refundableAmount = exports.isRefundable = exports.payoutBlockReason = exports.depositFor = exports.canTakeMoney = exports.balanceAfter = void 0;
+exports.refundableAmount = exports.isRefundable = exports.payoutBlockReason = exports.depositFor = exports.canTakeMoney = exports.balanceAfter = void 0;
 /**
  * Taking money on a business's behalf.
  *
@@ -15,10 +15,12 @@ exports.StripeConnect = exports.refundableAmount = exports.isRefundable = export
  * the mistake that makes both hard to reason about — they differ in who pays
  * whom, in which Stripe account, and in what happens when one fails.
  *
- * **This entry point is pure.** What a deposit comes to and whether a business
- * may sell yet are decided without a database or a provider, because a console
- * shows both while somebody drags a slider. Everything needing storage or
- * somebody else's money-moving licence is behind `/nestjs`.
+ * **This entry point is pure**, and stays that way by leaving things out. What
+ * a deposit comes to and whether a business may sell yet are decided without a
+ * database or a provider, because a console shows both while somebody drags a
+ * slider. Storage and the container are behind `/nestjs`; the Stripe client is
+ * behind `/stripe`, because a barrel that re-exports it puts the vendor's whole
+ * SDK into every bundle that wanted to divide a price by three.
  */
 var deposits_1 = require("./deposits");
 Object.defineProperty(exports, "balanceAfter", { enumerable: true, get: function () { return deposits_1.balanceAfter; } });
@@ -28,6 +30,14 @@ Object.defineProperty(exports, "payoutBlockReason", { enumerable: true, get: fun
 var payments_1 = require("./payments");
 Object.defineProperty(exports, "isRefundable", { enumerable: true, get: function () { return payments_1.isRefundable; } });
 Object.defineProperty(exports, "refundableAmount", { enumerable: true, get: function () { return payments_1.refundableAmount; } });
-var stripe_1 = require("./providers/stripe");
-Object.defineProperty(exports, "StripeConnect", { enumerable: true, get: function () { return stripe_1.StripeConnect; } });
+/*
+ * `StripeConnect` is deliberately NOT re-exported here.
+ *
+ * It lives at `@birtalanrobert/commerce/stripe`, because importing it pulls in
+ * the vendor's whole SDK — and this entry point is imported by browser code
+ * that only wants to know what a deposit comes to. Re-exporting it put 30 KB of
+ * a payments SDK into a console's first load for a screen that never calls it:
+ * a barrel is a bundling decision, and the only way to keep this one pure is to
+ * leave the heavy thing out of it.
+ */
 //# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;AAAA;;;;;;;;;;;;;;;;;;GAkBG;AACH,uCAQoB;AAPlB,wGAAA,YAAY,OAAA;AACZ,wGAAA,YAAY,OAAA;AACZ,sGAAA,UAAU,OAAA;AACV,6GAAA,iBAAiB,OAAA;AAMnB,uCAMoB;AALlB,wGAAA,YAAY,OAAA;AACZ,4GAAA,gBAAgB,OAAA;AAmBlB,6CAA8E;AAArE,uGAAA,aAAa,OAAA"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,uCAQoB;AAPlB,wGAAA,YAAY,OAAA;AACZ,wGAAA,YAAY,OAAA;AACZ,sGAAA,UAAU,OAAA;AACV,6GAAA,iBAAiB,OAAA;AAMnB,uCAMoB;AALlB,wGAAA,YAAY,OAAA;AACZ,4GAAA,gBAAgB,OAAA;AAmBlB;;;;;;;;;GASG"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@birtalanrobert/commerce",
-  "version": "2.0.0",
+  "version": "3.0.0",
   "description": "Taking money on a business's behalf: payout onboarding, payments, refunds and the record of both",
   "license": "AGPL-3.0-only",
   "author": "Robert Birtalan",
@@ -20,6 +20,10 @@
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
     },
+    "./stripe": {
+      "types": "./dist/providers/stripe.d.ts",
+      "default": "./dist/providers/stripe.js"
+    },
     "./nestjs": {
       "types": "./dist/nestjs/index.d.ts",
       "default": "./dist/nestjs/index.js"
@@ -34,16 +38,17 @@
     "README.md",
     "LICENSE",
     "NOTICE",
+    "stripe",
     "nestjs",
     "CHANGELOG.md"
   ],
   "dependencies": {
     "stripe": "^22.6.0",
-    "@birtalanrobert/database": "^1.0.0",
     "@birtalanrobert/context": "^1.1.0",
     "@birtalanrobert/http": "^2.0.0",
-    "@birtalanrobert/tenancy": "^1.1.0",
-    "@birtalanrobert/money": "^1.0.0"
+    "@birtalanrobert/money": "^1.0.0",
+    "@birtalanrobert/database": "^1.0.0",
+    "@birtalanrobert/tenancy": "^1.1.0"
   },
   "peerDependencies": {
     "@nestjs/common": "^11.0.0",

package/src/index.ts

@@ -12,10 +12,12 @@
  * the mistake that makes both hard to reason about — they differ in who pays
  * whom, in which Stripe account, and in what happens when one fails.
  *
- * **This entry point is pure.** What a deposit comes to and whether a business
- * may sell yet are decided without a database or a provider, because a console
- * shows both while somebody drags a slider. Everything needing storage or
- * somebody else's money-moving licence is behind `/nestjs`.
+ * **This entry point is pure**, and stays that way by leaving things out. What
+ * a deposit comes to and whether a business may sell yet are decided without a
+ * database or a provider, because a console shows both while somebody drags a
+ * slider. Storage and the container are behind `/nestjs`; the Stripe client is
+ * behind `/stripe`, because a barrel that re-exports it puts the vendor's whole
+ * SDK into every bundle that wanted to divide a price by three.
  */
 export {
   balanceAfter,
@@ -48,4 +50,13 @@ export type {
   StoredCard,
 } from './providers/port';
 
-export { StripeConnect, type StripeConnectOptions } from './providers/stripe';
+/*
+ * `StripeConnect` is deliberately NOT re-exported here.
+ *
+ * It lives at `@birtalanrobert/commerce/stripe`, because importing it pulls in
+ * the vendor's whole SDK — and this entry point is imported by browser code
+ * that only wants to know what a deposit comes to. Re-exporting it put 30 KB of
+ * a payments SDK into a console's first load for a screen that never calls it:
+ * a barrel is a bundling decision, and the only way to keep this one pure is to
+ * leave the heavy thing out of it.
+ */

package/stripe/package.json

@@ -0,0 +1,5 @@
+{
+  "main": "../dist/providers/stripe.js",
+  "types": "../dist/providers/stripe.d.ts",
+  "sideEffects": false
+}