@autobusal/order-process 1.4.0 → 1.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/order-process",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts",

package/services.ts

@@ -2,9 +2,39 @@ import { useQuery, UseQueryResult } from '@tanstack/react-query';
 import { apiClient } from '@autobusal/providers';
 import { BKTData, StripeData, MollieData, RaiffeisenData, NexiData } from './types';
 
+/**
+ * Starting a payment must happen EXACTLY ONCE per attempt.
+ *
+ * Claude - 2026-08-31, after Ferjolt hit "Too Many Attempts" retrying a
+ * card payment.
+ *
+ * These are useQuery, and react-query's defaults are built for reads: it
+ * retries a failure three times, and refetches on window focus, on reconnect
+ * and on mount. So ONE failed payment fired four POSTs, and alt-tabbing away
+ * and back fired more.
+ *
+ * Two things wrong with that, and the rate limit is the milder one. Each
+ * call creates an ORDER ENTRY at the gateway, and RaiAccept requires
+ * merchantOrderReference to be unique per attempt - it rejects a repeat as
+ * error 1007 - so the silent retries were manufacturing duplicate gateway
+ * orders for a single click, and the visible failure could be the retry
+ * being refused rather than the original problem.
+ *
+ * A payment is not a read. It fires once, when the component mounts, and a
+ * failure is reported rather than quietly attempted again.
+ */
+const ONCE = {
+  retry: false,
+  refetchOnWindowFocus: false,
+  refetchOnReconnect: false,
+  refetchOnMount: false,
+  staleTime: Infinity
+} as const;
+
 export const useMakeStandard = (type: string, hash: string): UseQueryResult<string> => (
   useQuery({
     queryKey: ['process-payment-standard', { type, hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/make', {
@@ -21,6 +51,7 @@ export const useMakeStandard = (type: string, hash: string): UseQueryResult<stri
 export const useMakeBkt = (hash: string): UseQueryResult<BKTData> => (
   useQuery({
     queryKey: ['process-payment-bkt', { hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/bkt', { hash })
@@ -34,6 +65,7 @@ export const useMakeBkt = (hash: string): UseQueryResult<BKTData> => (
 export const useMakeStripe = (hash: string): UseQueryResult<StripeData> => (
   useQuery({
     queryKey: ['process-payment-stripe', { hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/stripe', { hash })
@@ -47,6 +79,7 @@ export const useMakeStripe = (hash: string): UseQueryResult<StripeData> => (
 export const useMakeMollie = (hash: string): UseQueryResult<MollieData> => (
   useQuery({
     queryKey: ['process-payment-mollie', { hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/mollie', { hash })
@@ -60,6 +93,7 @@ export const useMakeMollie = (hash: string): UseQueryResult<MollieData> => (
 export const useMakeRaiffeisen = (hash: string): UseQueryResult<RaiffeisenData> => (
   useQuery({
     queryKey: ['process-payment-raiffeisen', { hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/raiffeisen', { hash })
@@ -73,6 +107,7 @@ export const useMakeRaiffeisen = (hash: string): UseQueryResult<RaiffeisenData>
 export const useMakeNexi = (hash: string): UseQueryResult<NexiData> => (
   useQuery({
     queryKey: ['process-payment-nexi', { hash }],
+    ...ONCE,
     queryFn: async () => (
       await apiClient
         .post('/api/payment/nexi', { hash })