fix: handle Zaprite upgrade review and irreversible statuses

Surface PAID_NEEDS_REVIEW copy and treat REFUNDED/REVOKED as terminal
so the client matches the billing-server money-path states.

Author: AnthonyRonning

frontend/src/billing/zapriteUpgrade.ts

@@ -4,7 +4,9 @@ export const ZAPRITE_UPGRADE_TERMINAL_STATUSES = [
   "COMPLETED",
   "EXPIRED",
   "CANCELED",
-  "FAILED"
+  "FAILED",
+  "REFUNDED",
+  "REVOKED"
 ] as const;
 
 export type ZapriteUpgradeStatus =
@@ -18,7 +20,10 @@ export type ZapriteUpgradeStatus =
   | "COMPLETED"
   | "EXPIRED"
   | "CANCELED"
-  | "FAILED";
+  | "FAILED"
+  | "PAID_NEEDS_REVIEW"
+  | "REFUNDED"
+  | "REVOKED";
 
 export type ZapriteUpgradePlanSummary = {
   product_id: string;

frontend/src/components/ZapriteUpgradeDialog.tsx

@@ -45,6 +45,11 @@ function statusCopy(status: string): string {
       return "This upgrade was canceled. Your current plan is unchanged.";
     case "FAILED":
       return "This upgrade could not be completed. Your current plan is unchanged.";
+    case "PAID_NEEDS_REVIEW":
+      return "Payment was received and needs a short review before the plan change is applied. Your current plan is unchanged.";
+    case "REFUNDED":
+    case "REVOKED":
+      return "This upgrade is no longer active. Your current plan is unchanged.";
     default:
       return "Preparing Bitcoin checkout.";
   }