@djangocfg/api 2.1.548 → 2.1.550

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@djangocfg/api",
-  "version": "2.1.548",
+  "version": "2.1.550",
   "description": "Auto-generated TypeScript API client with React hooks, SWR integration, and Zod validation for Django REST Framework backends",
   "keywords": [
     "django",
@@ -85,7 +85,7 @@
     "zod": "^4.3.6"
   },
   "devDependencies": {
-    "@djangocfg/typescript-config": "^2.1.548",
+    "@djangocfg/typescript-config": "^2.1.550",
     "@testing-library/react": "^16.3.2",
     "@types/node": "^25.9.5",
     "@types/react": "19.2.15",

package/src/auth/context/AccountsContext.tsx

@@ -101,6 +101,9 @@ export function AccountsProvider({ children }: AccountsProviderProps) {
 
   const profileRef = useRef<User | undefined>(profile);
   const isLoadingRef = useRef(false);
+  // Monotonic id per fetch. A forced re-read can now overlap the bootstrap
+  // one, and the slower response must not overwrite the newer profile.
+  const fetchSeqRef = useRef(0);
 
   useEffect(() => {
     profileRef.current = profile;
@@ -129,11 +132,16 @@ export function AccountsProvider({ children }: AccountsProviderProps) {
       return undefined;
     }
 
-    if (isLoadingRef.current) {
+    // `force` is how a write says "I just changed the server, re-read it".
+    // Honouring the in-flight guard there would hand the caller back the
+    // pre-write profile and never fetch again, so the UI keeps showing the
+    // old value indefinitely.
+    if (isLoadingRef.current && !force) {
       authLogger.debug(`Profile loading in progress, skipping (caller: ${callerId})`);
       return profileRef.current;
     }
 
+    const seq = ++fetchSeqRef.current;
     setIsLoadingProfile(true);
     isLoadingRef.current = true;
     setProfileError(null);
@@ -141,17 +149,22 @@ export function AccountsProvider({ children }: AccountsProviderProps) {
       authLogger.debug(`Fetching profile from API (caller: ${callerId}, force: ${force})`);
       const res = await CfgAccountsProfile.cfgAccountsProfileRetrieve({ throwOnError: true });
       const result = res.data;
+      // A newer fetch already landed — its profile is the current one, and
+      // this older response would roll it back.
+      if (seq !== fetchSeqRef.current) return result;
       setProfile(result);
       profileRef.current = result;
       setCachedProfile(result);
       return result;
     } catch (error) {
       const err = error instanceof Error ? error : new Error('Failed to fetch profile');
-      setProfileError(err);
+      if (seq === fetchSeqRef.current) setProfileError(err);
       throw err;
     } finally {
-      setIsLoadingProfile(false);
-      isLoadingRef.current = false;
+      if (seq === fetchSeqRef.current) {
+        setIsLoadingProfile(false);
+        isLoadingRef.current = false;
+      }
     }
   }, []);