fix(stripe-a-posteriori): demande l'email du restaurant avant /v1/accounts

La page « Stripe Connect a posteriori » hardcodait `email: ""` dans le
prefill du compte Stripe Connect (commentaire : « Stripe Express collecte
l'email du gérant pendant le KYC »). C'est faux : Stripe REJETTE un email
vide sur `POST /v1/accounts` avec « Invalid email address: ». Le wizard
s'en sortait parce que `prospect.email` existait à l'étape 1 ; pour un
tenant créé hors wizard (a posteriori), il n'y a pas de prospect, donc
pas de source d'email.

Fix : ajout d'un champ « Email du restaurant » dans la view (Input
contrôlé via `email` + `onEmailChange` props), avec validation regex
client. Le bouton « Générer un lien Stripe Connect » reste désactivé
tant que la syntaxe email est invalide.

Comportement Régénérer : préserve le UX existant — `/v1/account_links`
ne recrée pas le compte, donc l'email n'est pas relu. Le bouton
« Régénérer » est enabled même avec `email = ""` (cf. test pinned).

Tests : +6 cases pinning la nouvelle logique (email input rendu /
masqué selon `hasAccount`, validation regex, callback `onEmailChange`,
disable/enable du bouton, regenerate sans email). 22/22 green.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: alexpmichelet

apps/admin/src/app/(app)/t/[tenantId]/parametres/stripe/page.tsx

@@ -96,6 +96,10 @@ export default function StripeSettingsPage() {
   const [accountLink, setAccountLink] = useState<string | null>(null);
   const [isGenerating, setIsGenerating] = useState(false);
   const [genError, setGenError] = useState<string | null>(null);
+  // Email du restaurant — requis par Stripe `/v1/accounts` (champ `email`).
+  // Saisi par l'opérateur ici parce qu'aucun « prospect » n'existe pour un
+  // tenant créé hors wizard (a posteriori).
+  const [email, setEmail] = useState("");
 
   // Loading sentinel — the view requires a tenantName + the read state.
   if (stripeState === undefined) {
@@ -127,12 +131,12 @@ export default function StripeSettingsPage() {
         returnUrl,
         prefill: {
           siret: stripeState.siret,
-          // Stripe Express collects the legal representative's email
-          // during the KYC UI itself; we forward an empty string when
-          // we don't have a manager email handy (matches the wizard
-          // `Step3Form` discipline). The `business_type: "company"`
-          // branch ignores `individual[*]` params anyway.
-          email: "",
+          // Email saisi par l'opérateur dans le formulaire (la view garde
+          // le bouton désactivé tant que la syntaxe email est invalide).
+          // Stripe `/v1/accounts` rejette un email vide ("Invalid email
+          // address: "), donc on ne se permet PAS de pousser "" comme le
+          // faisait le wizard quand `prospect.email` était null.
+          email: email.trim(),
         },
       });
       setAccountLink(result.url);
@@ -165,6 +169,8 @@ export default function StripeSettingsPage() {
   return (
     <StripeSettingsView
       tenantName={stripeState.name}
+      email={email}
+      onEmailChange={setEmail}
       stripeAccountId={stripeState.stripeAccountId ?? undefined}
       stripeStatus={
         (stripeState.stripeStatus ?? undefined) as StripeStatus | undefined

apps/admin/src/app/(app)/t/[tenantId]/parametres/stripe/stripe-settings-view.test.tsx

@@ -196,6 +196,8 @@ function defaultProps(
 ): StripeSettingsViewProps {
   return {
     tenantName: "Le Petit Bistrot",
+    email: "contact@le-petit-bistrot.fr",
+    onEmailChange: vi.fn(),
     stripeAccountId: undefined,
     stripeStatus: undefined,
     accountLink: null,
@@ -327,6 +329,86 @@ describe("StripeSettingsView — generate / regenerate CTA", () => {
   });
 });
 
+describe("StripeSettingsView — email du restaurant (a posteriori)", () => {
+  it("renders the email input when no Stripe account exists yet", () => {
+    const tree = serialize(StripeSettingsView(defaultProps({ email: "" })));
+    const emailInput = findBySlot(tree, "stripe-settings-email") as {
+      props: { value?: string; type?: string };
+    } | null;
+    expect(emailInput).not.toBeNull();
+    expect(emailInput?.props.type).toBe("email");
+  });
+
+  it("hides the email input once an account exists (regenerate path)", () => {
+    const tree = serialize(
+      StripeSettingsView(
+        defaultProps({
+          stripeAccountId: ACCOUNT_ID,
+          stripeStatus: "pending",
+          email: "",
+        }),
+      ),
+    );
+    expect(findBySlot(tree, "stripe-settings-email")).toBeNull();
+  });
+
+  it("empty email → Générer button is disabled (Stripe rejects empty)", () => {
+    const tree = serialize(StripeSettingsView(defaultProps({ email: "" })));
+    const btn = findBySlot(tree, "stripe-settings-generate") as {
+      props: { disabled?: boolean };
+    } | null;
+    expect(btn?.props.disabled).toBe(true);
+  });
+
+  it("syntactically invalid email → Générer button is disabled", () => {
+    const tree = serialize(
+      StripeSettingsView(defaultProps({ email: "not-an-email" })),
+    );
+    const btn = findBySlot(tree, "stripe-settings-generate") as {
+      props: { disabled?: boolean };
+    } | null;
+    expect(btn?.props.disabled).toBe(true);
+  });
+
+  it("valid email → Générer button is enabled", () => {
+    const tree = serialize(
+      StripeSettingsView(defaultProps({ email: "alex@example.com" })),
+    );
+    const btn = findBySlot(tree, "stripe-settings-generate") as {
+      props: { disabled?: boolean };
+    } | null;
+    expect(btn?.props.disabled).toBe(false);
+  });
+
+  it("typing in the email input calls `onEmailChange` with the new value", () => {
+    const onEmailChange = vi.fn();
+    const tree = serialize(
+      StripeSettingsView(defaultProps({ email: "", onEmailChange })),
+    );
+    const emailInput = findBySlot(tree, "stripe-settings-email") as {
+      props: { onChange?: (e: { target: { value: string } }) => void };
+    } | null;
+    emailInput?.props.onChange?.({ target: { value: "new@example.com" } });
+    expect(onEmailChange).toHaveBeenCalledWith("new@example.com");
+  });
+
+  it("regenerate path: empty email still allowed (account exists, no `/v1/accounts` re-call)", () => {
+    const tree = serialize(
+      StripeSettingsView(
+        defaultProps({
+          stripeAccountId: ACCOUNT_ID,
+          stripeStatus: "pending",
+          email: "",
+        }),
+      ),
+    );
+    const btn = findBySlot(tree, "stripe-settings-regenerate") as {
+      props: { disabled?: boolean };
+    } | null;
+    expect(btn?.props.disabled).toBe(false);
+  });
+});
+
 describe("StripeSettingsView — generated URL display", () => {
   it("when `accountLink` is set, renders the URL in a read-only input", () => {
     const tree = serialize(

apps/admin/src/app/(app)/t/[tenantId]/parametres/stripe/stripe-settings-view.tsx

@@ -70,6 +70,16 @@ export type StripeStatus = "pending" | "ready" | "disabled";
 export type StripeSettingsViewProps = {
   /** Tenant name for the page header. */
   tenantName: string;
+  /**
+   * Email du restaurant utilisé pour pré-remplir le compte Stripe Connect
+   * Express (champ `email` requis par l'API `/v1/accounts`). Saisi par
+   * l'opérateur sur cette page parce qu'aucun « prospect » n'existe pour
+   * un tenant créé hors wizard. Le bouton « Générer » reste désactivé tant
+   * que `email` est vide ou syntaxiquement invalide.
+   */
+  email: string;
+  /** Setter contrôlé de l'email (parent owns state). */
+  onEmailChange: (value: string) => void;
   /**
    * The Stripe Connect Express account id (`acct_xxx`) stamped on the
    * tenant once the action has been fired at least once. `undefined`
@@ -179,11 +189,15 @@ function statusCopy(
 // Component
 // ---------------------------------------------------------------------------
 
+const EMAIL_REGEX = /^[^\s@]+@[^\s@]+\.[^\s@]+$/;
+
 export function StripeSettingsView(
   props: StripeSettingsViewProps,
 ): React.JSX.Element {
   const {
     tenantName,
+    email,
+    onEmailChange,
     stripeAccountId,
     stripeStatus,
     accountLink,
@@ -196,6 +210,12 @@ export function StripeSettingsView(
   const status = statusCopy(stripeAccountId, stripeStatus);
   const hasAccount = stripeAccountId !== undefined;
   const hasUrl = accountLink !== null;
+  const isEmailValid = EMAIL_REGEX.test(email.trim());
+  // Email n'est requis qu'à la PREMIÈRE génération (création du compte
+  // Stripe). Une régénération ne crée qu'un nouveau `account_link` sur
+  // un compte existant — pas de re-passage par `/v1/accounts`, donc
+  // l'email du tenant n'est pas relu.
+  const disableGenerate = isGenerating || (!hasAccount && !isEmailValid);
 
   return (
     <div
@@ -251,6 +271,30 @@ export function StripeSettingsView(
         ) : null}
       </Card>
 
+      {/* Email du restaurant — requis par Stripe `/v1/accounts` (`business
+          email`). Pas de prospect ici (a posteriori), donc on demande à
+          l'opérateur. Le bouton « Générer » reste désactivé tant que la
+          syntaxe email n'est pas valide. */}
+      {!hasAccount ? (
+        <div className="flex flex-col gap-1.5">
+          <Label htmlFor="stripe-settings-email">Email du restaurant</Label>
+          <Input
+            id="stripe-settings-email"
+            data-slot="stripe-settings-email"
+            type="email"
+            value={email}
+            onChange={(e) => onEmailChange(e.target.value)}
+            placeholder="contact@le-petit-bistrot.fr"
+            autoComplete="email"
+          />
+          <p className="text-xs text-muted-foreground">
+            Cet email sera attaché au compte Stripe Connect (reçus,
+            notifications Stripe). Le KYC personnel du gérant sera collecté plus
+            tard par Stripe lors de l&apos;onboarding.
+          </p>
+        </div>
+      ) : null}
+
       {/* Generate / Regenerate CTA */}
       {hasAccount ? (
         <div className="flex items-center justify-start">
@@ -259,7 +303,7 @@ export function StripeSettingsView(
             variant="outline"
             data-slot="stripe-settings-regenerate"
             onClick={onGenerate}
-            disabled={isGenerating}
+            disabled={disableGenerate}
           >
             {isGenerating ? "Génération…" : "Régénérer un lien Stripe Connect"}
           </Button>
@@ -270,7 +314,7 @@ export function StripeSettingsView(
             type="button"
             data-slot="stripe-settings-generate"
             onClick={onGenerate}
-            disabled={isGenerating}
+            disabled={disableGenerate}
           >
             {isGenerating ? "Génération…" : "Générer un lien Stripe Connect"}
           </Button>