@stacksjs/commerce 0.75.75 → 0.75.77

package/dist/orders/index.d.ts

@@ -59,14 +59,13 @@ export {
   updateDeliveryInfo,
   updateStatus,
 } from './update';
-// Stripe webhook handlers (stacksjs/stacks#1879 Co-17). Call
-// registerCommerceWebhookHandlers() once at boot; the payments
-// package's processWebhook() verifies signatures + dispatches.
+// Payment webhook handling (stacksjs/stacks#1879 Co-17, #665): pass each event
+// the payment driver's verifyWebhook() returns to handleCommercePaymentEvent().
 export {
-  handleChargeRefunded,
-  handlePaymentIntentFailed,
-  handlePaymentIntentSucceeded,
-  registerCommerceWebhookHandlers,
+  handleCommercePaymentEvent,
+  handlePaymentFailed,
+  handlePaymentSucceeded,
+  handleRefundSucceeded,
 } from './webhook';
 // Server-side cart-to-order total recompute (stacksjs/stacks#1879 Co-13).
 // Apps pre-flight the cart through this before placeOrder to detect

package/dist/orders/webhook.d.ts

@@ -1,31 +1,35 @@
+import type { PaymentEvent } from '@stacksjs/payments';
 /**
- * Handler for `payment_intent.succeeded`. Marks the linked payment
- * row + order row as paid in a single transaction. Idempotent via
- * `recordEventOrSkip`.
+ * A payment succeeded. Marks the linked payment row and order row in a single
+ * transaction. Idempotent via `recordEventOrSkip`.
  */
-declare function handlePaymentIntentSucceeded(event: { id: string, data: { object: any } }): Promise<void>;
+declare function handlePaymentSucceeded(event: PaymentEvent): Promise<void>;
 /**
- * Handler for `payment_intent.payment_failed`. Marks the payment
- * as failed and cancels the linked order. Records the failure
- * reason on the payment row for ops triage.
+ * A payment failed. Marks the payment failed and cancels the linked order,
+ * recording the provider's reason on the payment row for ops triage.
  */
-declare function handlePaymentIntentFailed(event: { id: string, data: { object: any } }): Promise<void>;
+declare function handlePaymentFailed(event: PaymentEvent): Promise<void>;
 /**
- * Handler for `charge.refunded`. Marks the order REFUNDED and
- * records the refund amount on the linked payment.
+ * A refund succeeded. Adds it to the payment's refunded amount; a refund that
+ * covers what is left marks the payment `refunded` and the order REFUNDED,
+ * and a smaller one marks the payment `partiallyRefunded` and leaves the order
+ * where it is, since REFUNDED is terminal. It used to mark both refunded on any
+ * refund at all, and to overwrite the refunded amount rather than add to it.
+ *
+ * A refund larger than what is left is not recorded and is logged: the
+ * provider has refunded more than this database thinks was taken, which a
+ * person needs to look at - most likely the same refund recorded by hand in
+ * the dashboard as well.
  */
-declare function handleChargeRefunded(event: { id: string, data: { object: any } }): Promise<void>;
+declare function handleRefundSucceeded(event: PaymentEvent): Promise<void>;
 /**
- * Wire commerce's Stripe webhook handlers into the payments
- * package's event router. Call once at boot.
- *
- * Returns an unregister callback for tests that want to swap the
- * handler set between cases.
+ * Apply a verified payment event to the order it concerns. Answers whether
+ * commerce acted on it; events it has no use for are left alone.
  */
-export declare function registerCommerceWebhookHandlers(): Promise<() => void>;
+export declare function handleCommercePaymentEvent(event: PaymentEvent): Promise<boolean>;
 // Export the individual handlers for direct testing.
 export {
-  handleChargeRefunded,
-  handlePaymentIntentFailed,
-  handlePaymentIntentSucceeded,
+  handlePaymentFailed,
+  handlePaymentSucceeded,
+  handleRefundSucceeded,
 };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stacksjs/commerce",
   "type": "module",
-  "version": "0.75.75",
+  "version": "0.75.77",
   "description": "Stacks commerce utilities.",
   "author": "Chris Breuer",
   "contributors": [
@@ -57,17 +57,17 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/cache": "0.75.75",
-    "@stacksjs/config": "0.75.75",
-    "@stacksjs/database": "0.75.75",
-    "@stacksjs/env": "0.75.75",
-    "@stacksjs/error-handling": "0.75.75",
-    "@stacksjs/events": "0.75.75",
-    "@stacksjs/logging": "0.75.75",
-    "@stacksjs/orm": "0.75.75",
-    "@stacksjs/payments": "0.75.75",
-    "@stacksjs/realtime": "0.75.75",
-    "@stacksjs/strings": "0.75.75",
+    "@stacksjs/cache": "0.75.77",
+    "@stacksjs/config": "0.75.77",
+    "@stacksjs/database": "0.75.77",
+    "@stacksjs/env": "0.75.77",
+    "@stacksjs/error-handling": "0.75.77",
+    "@stacksjs/events": "0.75.77",
+    "@stacksjs/logging": "0.75.77",
+    "@stacksjs/orm": "0.75.77",
+    "@stacksjs/payments": "0.75.77",
+    "@stacksjs/realtime": "0.75.77",
+    "@stacksjs/strings": "0.75.77",
     "ts-spreadsheets": "^0.2.0"
   },
   "devDependencies": {