@inwink-mcp/backoffice-extension-helpers 1.0.6 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inwink-mcp/backoffice-extension-helpers",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "author": "inwink",
   "license": "ISC",
   "description": "CLI pour bootstrapper, packager et vérifier les extensions backoffice inwink.",

package/templates/test-app/src/components/EventApiTester.tsx

@@ -0,0 +1,54 @@
+import { useEffect, useState } from 'react';
+import { FiRefreshCw } from 'react-icons/fi';
+import { useInwinkEventApi } from '../hooks/useInwinkEventApi';
+
+// Composant de test (overlay "test-app") : exerce useInwinkEventApi en lançant
+// l'appel automatiquement au montage. N'est monté que lorsque l'utilisateur est
+// authentifié (cf. HomePage), mais l'appel passe de toute façon par le gate
+// `whenReady` du hook, qui attend la résolution d'auth avant de lire le token.
+export default function EventApiTester() {
+  const eventApi = useInwinkEventApi();
+  const [result, setResult] = useState<string>('');
+  const [error, setError] = useState<string>('');
+  const [isLoading, setIsLoading] = useState(false);
+
+  async function callEventApi(signal?: { aborted: boolean }) {
+    setError('');
+    setResult('');
+    setIsLoading(true);
+    try {
+      const data = await eventApi.post('/dae274d9-e86e-f011-8dca-0022488a3cad/session/query', {
+        "selects": { "id": true, "title": true, "status": true, "moderators": { "person": { "id": true, "firstname": true, "lastname": true, "company": true, "mail": true } }, "startDate": true, "endDate": true, "room": { "name": true, "id": true }, "speakers": { "id": true, "firstname": true, "lastname": true, "company": true, "mail": true }, "exhibitors": { "id": true, "name": true }, "infos": { "registered": true, "attended": true, "attendedOnsite": true, "attendedOnlineLive": true, "attendedOnlineReplay": true } },
+        "orders": [{ "desc": true, "value": { "validFrom": {} } }],
+        "page": { "index": 0, "size": 200 }
+      });
+      if (!signal?.aborted) setResult(JSON.stringify(data, null, 2));
+    } catch (err) {
+      if (!signal?.aborted) setError(err instanceof Error ? err.message : String(err));
+    } finally {
+      if (!signal?.aborted) setIsLoading(false);
+    }
+  }
+
+  // Appel automatique au montage ; on ignore le résultat si le composant est
+  // démonté entre-temps (évite un setState sur composant démonté).
+  useEffect(() => {
+    const signal = { aborted: false };
+    void callEventApi(signal);
+    return () => {
+      signal.aborted = true;
+    };
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, []);
+
+  return (
+    <section>
+      <button disabled={isLoading} onClick={() => void callEventApi()}>
+        <FiRefreshCw aria-hidden />
+        {isLoading ? 'Appel en cours…' : "Rappeler l'API Event"}
+      </button>
+      {result && <pre>{result}</pre>}
+      {error && <p role="alert">Erreur : {error}</p>}
+    </section>
+  );
+}

package/templates/test-app/src/pages/HomePage.tsx.tpl

@@ -1,31 +1,12 @@
-import { useState } from 'react';
-import { FiLogIn, FiLogOut, FiRefreshCw } from 'react-icons/fi';
+import { FiLogIn, FiLogOut } from 'react-icons/fi';
 import { useAuth } from '../hooks/useAuth';
-import { useInwinkEventApi } from '../hooks/useInwinkEventApi';
+import EventApiTester from '../components/EventApiTester';
 
 // Version de HomePage générée par le template overlay "test-app".
 // Sert à vérifier que le squelette bootstrappé compile et que les hooks API
-// (ici useInwinkEventApi) sont utilisables.
+// (ici useInwinkEventApi, via EventApiTester) sont utilisables.
 export default function HomePage() {
   const { isAuthenticated, isLoading, login, logout, user } = useAuth();
-  const eventApi = useInwinkEventApi();
-  const [result, setResult] = useState<string>('');
-  const [error, setError] = useState<string>('');
-
-  async function callEventApi() {
-    setError('');
-    setResult('');
-    try {
-      const data = await eventApi.post('/dae274d9-e86e-f011-8dca-0022488a3cad/session/query', {
-        "selects": { "id": true, "title": true, "status": true, "moderators": { "person": { "id": true, "firstname": true, "lastname": true, "company": true, "mail": true } }, "startDate": true, "endDate": true, "room": { "name": true, "id": true }, "speakers": { "id": true, "firstname": true, "lastname": true, "company": true, "mail": true }, "exhibitors": { "id": true, "name": true }, "infos": { "registered": true, "attended": true, "attendedOnsite": true, "attendedOnlineLive": true, "attendedOnlineReplay": true } },
-        "orders": [{ "desc": true, "value": { "validFrom": {} } }],
-        "page": { "index": 0, "size": 200 }
-      });
-      setResult(JSON.stringify(data, null, 2));
-    } catch (err) {
-      setError(err instanceof Error ? err.message : String(err));
-    }
-  }
 
   return (
     <main>
@@ -38,16 +19,11 @@ export default function HomePage() {
       ) : (
         <>
           <p>Connecté en tant que : {user?.profile.email ?? 'Inconnu'}</p>
-          <button onClick={() => void callEventApi()}>
-            <FiRefreshCw aria-hidden />
-            Appeler l'API Event
-          </button>
           <button onClick={() => void logout()}>
             <FiLogOut aria-hidden />
             Se déconnecter
           </button>
-          {result && <pre>{result}</pre>}
-          {error && <p role="alert">Erreur : {error}</p>}
+          <EventApiTester />
         </>
       )}
     </main>

package/templates/vite-extension/src/auth/AuthProvider.tsx

@@ -12,6 +12,13 @@ type AuthContextValue = {
   getAccessToken: () => string | null;
   /** Renouvelle le token silencieusement. Retourne le nouvel access_token, ou null si échec. */
   renewToken: () => Promise<string | null>;
+  /**
+   * Résout une fois la résolution d'auth initiale terminée (callback OIDC ou
+   * `getUser()`). À attendre avant de lire le token dans un appel API : évite la
+   * course où un effet enfant part avant que la session soit chargée et
+   * déclenche un login interactif intempestif.
+   */
+  whenReady: () => Promise<void>;
 };
 
 export const AuthContext = createContext<AuthContextValue | null>(null);
@@ -29,8 +36,18 @@ export function AuthProvider({ children }: PropsWithChildren) {
   const [isLoading, setIsLoading] = useState(true);
   const userRef = useRef(user);
   const loginInFlightRef = useRef(false);
+  const renewInFlightRef = useRef<Promise<string | null> | null>(null);
   useEffect(() => { userRef.current = user; }, [user]);
 
+  // Deferred résolu à la fin de la résolution d'auth initiale (cf. `whenReady`).
+  // Créé en lazy-init pour exister dès le premier rendu (avant tout effet enfant).
+  const readyRef = useRef<{ promise: Promise<void>; resolve: () => void } | null>(null);
+  if (!readyRef.current) {
+    let resolve!: () => void;
+    const promise = new Promise<void>((r) => { resolve = r; });
+    readyRef.current = { promise, resolve };
+  }
+
   useEffect(() => {
     let isMounted = true;
 
@@ -49,7 +66,10 @@ export function AuthProvider({ children }: PropsWithChildren) {
       }
     }
 
-    void init();
+    // Le gate `ready` est résolu quelle que soit l'issue de init() (succès,
+    // absence de session, ou erreur) : les appels en attente reprennent alors,
+    // token présent ou non — le cas « réellement non authentifié » mène à login().
+    void init().finally(() => readyRef.current?.resolve());
 
     const handleUserLoaded = (u: User) => { setUser(u); setIsLoading(false); };
     const handleUserUnloaded = () => { setUser(null); setIsLoading(false); };
@@ -92,14 +112,27 @@ export function AuthProvider({ children }: PropsWithChildren) {
 
   const getAccessToken = useCallback(() => userRef.current?.access_token ?? null, []);
 
+  const whenReady = useCallback(() => readyRef.current!.promise, []);
+
   const renewToken = useCallback(async () => {
-    try {
-      const renewed = await userManager.signinSilent();
-      setUser(renewed);
-      return renewed?.access_token ?? null;
-    } catch {
-      return null;
+    // Déduplication : des appels concurrents recevant un 401 partagent un seul
+    // signinSilent() en vol plutôt que d'en lancer un chacun.
+    if (renewInFlightRef.current) {
+      return renewInFlightRef.current;
     }
+    const renewal = (async () => {
+      try {
+        const renewed = await userManager.signinSilent();
+        setUser(renewed);
+        return renewed?.access_token ?? null;
+      } catch {
+        return null;
+      } finally {
+        renewInFlightRef.current = null;
+      }
+    })();
+    renewInFlightRef.current = renewal;
+    return renewal;
   }, []);
 
   const value = useMemo(() => ({
@@ -110,6 +143,7 @@ export function AuthProvider({ children }: PropsWithChildren) {
     user,
     getAccessToken,
     renewToken,
+    whenReady,
   }), [user, isLoading]);
 
   return (

package/templates/vite-extension/src/hooks/useInwinkApi.ts

@@ -59,7 +59,12 @@ export function useInwinkApi(baseUrl: string): InwinkApi {
     }
 
     async function request<T = unknown>(path: string, init?: RequestInit): Promise<T> {
-      const { getAccessToken, renewToken, login } = authRef.current;
+      const { getAccessToken, renewToken, login, whenReady } = authRef.current;
+      // Attend la fin de la résolution d'auth initiale avant de lire le token :
+      // sans ce gate, un appel parti depuis l'effet de montage d'un composant
+      // (hors ProtectedRoute) verrait token=null et déclencherait un login()
+      // intempestif alors que la session est en cours de chargement.
+      await whenReady();
       let token = getAccessToken();
       if (!token) {
         await login();