@oxyhq/core 1.11.23 → 2.0.0

package/dist/esm/mixins/OxyServices.fedcm.js

@@ -33,12 +33,44 @@ function isUnknownModeEnumError(error) {
         ((message.includes('active') || message.includes('passive')) &&
             (message.includes('enum') || message.includes('not a valid'))));
 }
+/**
+ * Detect a `navigator.credentials.get` rejection that is consistent with
+ * "the supplied loginHint matched no account at the IdP".
+ *
+ * When an RP passes a `loginHint` and the IdP returns accounts but NONE of them
+ * declare that hint in their `login_hints`, Chrome filters every account out,
+ * greys it in the chooser ("You can't sign in using this account"), logs
+ * "Accounts were received, but none matched the login hint…", and ultimately
+ * rejects the credential request — surfacing as a `NotAllowedError` /
+ * `AbortError` (the same shape as a user-cancelled or timed-out request). A
+ * stale hint left over from a previously-signed-in/test account therefore hard
+ * -blocks sign-in.
+ *
+ * We can only safely apply the clear-and-retry recovery when a `loginHint` was
+ * actually supplied; without one this is just a normal cancel/timeout and must
+ * NOT be retried. Callers gate on `hadLoginHint` before calling this.
+ */
+function isPossibleHintMismatchError(error) {
+    if (!(error instanceof Error))
+        return false;
+    // FedCM surfaces a filtered-out / no-eligible-account outcome as
+    // NotAllowedError (current Chrome) or AbortError (our own timeout abort while
+    // the chooser had no selectable account). Both are indistinguishable from a
+    // genuine user cancel at the API level, so the gate on "a hint was supplied"
+    // (in the caller) is what makes the retry safe and targeted.
+    return error.name === 'NotAllowedError' || error.name === 'AbortError';
+}
 const FEDCM_LOGIN_HINT_KEY = 'oxy_fedcm_login_hint';
 // Global lock to prevent concurrent FedCM requests
 // FedCM only allows one navigator.credentials.get request at a time
 let fedCMRequestInProgress = false;
 let fedCMRequestPromise = null;
 let currentMediationMode = null;
+// AbortController of the in-flight request, exposed at module scope so an
+// arriving INTERACTIVE request can abort a slow/hung SILENT one instead of
+// blocking on it (see requestIdentityCredential). Set when a request starts,
+// cleared in that request's `finally`.
+let fedCMActiveController = null;
 /**
  * Federated Credential Management (FedCM) Authentication Mixin
  *
@@ -67,9 +99,12 @@ export function OxyServicesFedCMMixin(Base) {
                 super(...args);
             }
             resolveFedcmConfigUrl() {
+                // `DEFAULT_CONFIG_URL` is a static on the composed class; read it off the
+                // most-derived constructor through a typed cast (not `any`).
+                const configCtor = this.constructor;
                 return this.config.authWebUrl
                     ? `${this.config.authWebUrl}/fedcm.json`
-                    : this.constructor.DEFAULT_CONFIG_URL;
+                    : configCtor.DEFAULT_CONFIG_URL;
             }
             /**
              * Check if FedCM is supported in the current browser
@@ -113,67 +148,104 @@ export function OxyServicesFedCMMixin(Base) {
                 if (!this.isFedCMSupported()) {
                     throw new OxyAuthenticationError('FedCM not supported in this browser. Please update your browser or use an alternative sign-in method.');
                 }
+                // Use provided loginHint, or fall back to stored last-used account ID.
+                const initialLoginHint = options.loginHint || this.getStoredLoginHint();
                 try {
-                    // Prefer a server-minted, origin-bound nonce so the downstream
-                    // `/fedcm/exchange` can validate it. A caller-supplied nonce is
-                    // respected as-is for advanced use cases.
-                    const nonce = options.nonce || (await this.getFedcmNonce());
-                    const clientId = this.getClientId();
-                    // Use provided loginHint, or fall back to stored last-used account ID
-                    const loginHint = options.loginHint || this.getStoredLoginHint();
-                    debug.log('Interactive sign-in: Requesting credential for', clientId, loginHint ? `(hint: ${loginHint})` : '');
-                    // Request credential from browser's native identity flow.
-                    // mode: 'active' signals this is a user-gesture-initiated (button) flow.
-                    // 'active' is the current W3C spec value; requestIdentityCredential
-                    // transparently retries with the legacy 'button' value for Chrome 125–131.
-                    const credential = await this.requestIdentityCredential({
-                        configURL: this.resolveFedcmConfigUrl(),
-                        clientId,
-                        nonce,
-                        context: options.context,
-                        loginHint,
-                        mode: 'active',
-                    });
-                    if (!credential || !credential.token) {
-                        throw new OxyAuthenticationError('No credential received from browser');
-                    }
-                    debug.log('Interactive sign-in: Got credential, exchanging for session');
-                    // Exchange FedCM ID token for Oxy session
-                    const session = await this.exchangeIdTokenForSession(credential.token);
-                    // Store access token in HttpService. `accessToken`/`refreshToken` are
-                    // declared optional on SessionLoginResponse; default the refresh token to
-                    // an empty string when the exchange did not return one.
-                    if (session?.accessToken) {
-                        this.httpService.setTokens(session.accessToken, session.refreshToken ?? '');
-                    }
-                    // Store the user ID as loginHint for future FedCM requests
-                    if (session?.user?.id) {
-                        this.storeLoginHint(session.user.id);
-                    }
-                    debug.log('Interactive sign-in: Success!', { userId: session?.user?.id });
-                    return session;
+                    return await this.attemptInteractiveSignIn(options, initialLoginHint);
                 }
                 catch (error) {
-                    debug.log('Interactive sign-in failed:', error);
-                    const errorMessage = error instanceof Error ? error.message : String(error);
-                    // FedCM aborts/network failures surface as DOMException/Error instances,
-                    // both of which carry a `name`. Anything else has no meaningful name.
-                    const errorName = error instanceof Error ? error.name : '';
-                    if (errorName === 'AbortError') {
-                        throw new OxyAuthenticationError('Sign-in was cancelled by user');
-                    }
-                    if (errorName === 'NetworkError') {
-                        throw new OxyAuthenticationError('Network error during sign-in. Please check your connection.');
+                    // A STALE loginHint (e.g. left over from a previously-signed-in or test
+                    // account) that matches no account at the IdP makes Chrome filter out
+                    // every account and reject the request — indistinguishable from a user
+                    // cancel. When that happens AND we supplied a hint, clear the bad hint
+                    // and retry the credential request ONCE with no hint, which lets the
+                    // chooser surface the genuinely available account(s). We only do this for
+                    // a hint we pulled from storage (not a caller-supplied one), and only
+                    // once, so a real cancel never loops.
+                    const usedStoredHint = !!initialLoginHint && !options.loginHint;
+                    if (usedStoredHint && isPossibleHintMismatchError(error)) {
+                        debug.log('Interactive sign-in: stored loginHint matched no account; clearing it and retrying without a hint');
+                        this.clearLoginHint();
+                        return await this.attemptInteractiveSignIn(options, undefined);
                     }
-                    if (errorMessage.includes('multiple accounts')) {
-                        throw new OxyAuthenticationError('Please sign out and sign in again to use FedCM with a single account');
-                    }
-                    if (errorMessage.includes('retrieving a token') || errorMessage.includes('Error retrieving')) {
-                        debug.error('FedCM token retrieval error - this may be a browser or IdP configuration issue');
-                        throw new OxyAuthenticationError('Authentication failed. Please try again or use an alternative sign-in method.');
-                    }
-                    throw error;
+                    throw this.normalizeInteractiveSignInError(error);
+                }
+            }
+            /**
+             * Run a single interactive FedCM credential request + token exchange for the
+             * given (possibly undefined) loginHint. A successful exchange plants the
+             * access token and persists the user id as the future loginHint — the hint is
+             * therefore only ever stored after a GENUINELY successful sign-in, never
+             * speculatively.
+             *
+             * @private
+             */
+            async attemptInteractiveSignIn(options, loginHint) {
+                // Prefer a server-minted, origin-bound nonce so the downstream
+                // `/fedcm/exchange` can validate it. A caller-supplied nonce is
+                // respected as-is for advanced use cases.
+                const nonce = options.nonce || (await this.getFedcmNonce());
+                const clientId = this.getClientId();
+                debug.log('Interactive sign-in: Requesting credential for', clientId, loginHint ? `(hint: ${loginHint})` : '');
+                // Request credential from browser's native identity flow.
+                // mode: 'active' signals this is a user-gesture-initiated (button) flow.
+                // 'active' is the current W3C spec value; requestIdentityCredential
+                // transparently retries with the legacy 'button' value for Chrome 125–131.
+                const credential = await this.requestIdentityCredential({
+                    configURL: this.resolveFedcmConfigUrl(),
+                    clientId,
+                    nonce,
+                    context: options.context,
+                    loginHint,
+                    mode: 'active',
+                });
+                if (!credential || !credential.token) {
+                    throw new OxyAuthenticationError('No credential received from browser');
+                }
+                debug.log('Interactive sign-in: Got credential, exchanging for session');
+                // Exchange FedCM ID token for Oxy session
+                const session = await this.exchangeIdTokenForSession(credential.token);
+                // Store access token in HttpService. `accessToken`/`refreshToken` are
+                // declared optional on SessionLoginResponse; default the refresh token to
+                // an empty string when the exchange did not return one.
+                if (session?.accessToken) {
+                    this.httpService.setTokens(session.accessToken, session.refreshToken ?? '');
                 }
+                // Store the user ID as loginHint for future FedCM requests — only now, after
+                // a real successful exchange, so we never persist a hint that cannot resolve.
+                if (session?.user?.id) {
+                    this.storeLoginHint(session.user.id);
+                }
+                debug.log('Interactive sign-in: Success!', { userId: session?.user?.id });
+                return session;
+            }
+            /**
+             * Map a raw FedCM/exchange failure to a user-facing {@link OxyAuthenticationError}
+             * (or pass it through). Extracted so the clear-and-retry path can reuse the
+             * exact same error normalisation as the first attempt.
+             *
+             * @private
+             */
+            normalizeInteractiveSignInError(error) {
+                debug.log('Interactive sign-in failed:', error);
+                const errorMessage = error instanceof Error ? error.message : String(error);
+                // FedCM aborts/network failures surface as DOMException/Error instances,
+                // both of which carry a `name`. Anything else has no meaningful name.
+                const errorName = error instanceof Error ? error.name : '';
+                if (errorName === 'AbortError') {
+                    return new OxyAuthenticationError('Sign-in was cancelled by user');
+                }
+                if (errorName === 'NetworkError') {
+                    return new OxyAuthenticationError('Network error during sign-in. Please check your connection.');
+                }
+                if (errorMessage.includes('multiple accounts')) {
+                    return new OxyAuthenticationError('Please sign out and sign in again to use FedCM with a single account');
+                }
+                if (errorMessage.includes('retrieving a token') || errorMessage.includes('Error retrieving')) {
+                    debug.error('FedCM token retrieval error - this may be a browser or IdP configuration issue');
+                    return new OxyAuthenticationError('Authentication failed. Please try again or use an alternative sign-in method.');
+                }
+                return error;
             }
             /**
              * Silent sign-in using FedCM
@@ -315,16 +387,20 @@ export function OxyServicesFedCMMixin(Base) {
                 // If a request is already in progress...
                 if (fedCMRequestInProgress && fedCMRequestPromise) {
                     debug.log('Request already in progress, waiting...');
-                    // If current request is silent and new request is interactive,
-                    // wait for silent to finish, then make the interactive request
+                    // If the in-flight request is SILENT and this new one is INTERACTIVE,
+                    // abort the silent and proceed immediately. The silent round-trip can be
+                    // slow (it runs on page load and may stall in the browser), and a user who
+                    // just clicked "Sign In" must never be made to wait on — or be blocked by —
+                    // it. Awaiting the silent here is what previously let a hung silent
+                    // request deadlock the sign-in button, so we deliberately do NOT await it:
+                    // we abort it (its own `finally` resets the lock as it settles) and fall
+                    // through to start the interactive request synchronously below.
                     if (currentMediationMode === 'silent' && isInteractive) {
-                        try {
-                            await fedCMRequestPromise;
-                        }
-                        catch {
-                            // Ignore silent request errors
-                        }
-                        // Now fall through to make the interactive request
+                        debug.log('Aborting in-flight silent request to make way for interactive request');
+                        fedCMActiveController?.abort();
+                        // Fall through. The interactive request synchronously overwrites the
+                        // lock globals (below); the aborted silent's `finally` uses identity
+                        // guards so it cannot later clobber this interactive request's state.
                     }
                     else {
                         // Same type of request - wait for the existing one
@@ -339,10 +415,14 @@ export function OxyServicesFedCMMixin(Base) {
                 fedCMRequestInProgress = true;
                 currentMediationMode = requestedMediation;
                 const controller = new AbortController();
-                // Use shorter timeout for silent mediation since it should be quick
+                fedCMActiveController = controller;
+                // Use shorter timeout for silent mediation since it should be quick.
+                // The timeout constants are static on the composed class; read them off the
+                // most-derived constructor through a typed cast (not `any`).
+                const timeoutCtor = this.constructor;
                 const timeoutMs = requestedMediation === 'silent'
-                    ? this.constructor.FEDCM_SILENT_TIMEOUT
-                    : this.constructor.FEDCM_TIMEOUT;
+                    ? timeoutCtor.FEDCM_SILENT_TIMEOUT
+                    : timeoutCtor.FEDCM_TIMEOUT;
                 const timeout = setTimeout(() => {
                     debug.log('Request timed out after', timeoutMs, 'ms (mediation:', requestedMediation + ')');
                     controller.abort();
@@ -419,9 +499,20 @@ export function OxyServicesFedCMMixin(Base) {
                     }
                     finally {
                         clearTimeout(timeout);
-                        fedCMRequestInProgress = false;
-                        fedCMRequestPromise = null;
-                        currentMediationMode = null;
+                        // Only reset the shared lock if it still belongs to THIS request. When an
+                        // interactive request aborts a slow silent one, the silent settles (and
+                        // runs this `finally`) AFTER the interactive has already taken over the
+                        // lock and installed its own controller/promise. Guarding on identity
+                        // (`fedCMActiveController === controller`) ensures the settling silent
+                        // cannot null out the interactive request's in-progress state. The
+                        // request that still owns the lock clears it; the superseded one is a
+                        // no-op here.
+                        if (fedCMActiveController === controller) {
+                            fedCMRequestInProgress = false;
+                            fedCMRequestPromise = null;
+                            currentMediationMode = null;
+                            fedCMActiveController = null;
+                        }
                     }
                 })();
                 return fedCMRequestPromise;
@@ -606,11 +697,52 @@ export function OxyServicesFedCMMixin(Base) {
                     // Storage blocked
                 }
             }
+            /**
+             * List the authenticated user's authorized RP apps.
+             *
+             * Returns the intersection of the user's FedCM grants and the currently-
+             * approved RP catalog — what powers the "Connected apps" management UI in
+             * @oxyhq/services. Requires a real user session; service tokens are
+             * rejected by the underlying endpoint.
+             */
+            async listAuthorizedApps() {
+                try {
+                    const response = await this.makeRequest('GET', '/fedcm/me/authorized-apps', undefined, {
+                        cache: true,
+                        cacheTTL: 30 * 1000, // 30 second cache — short, this drives a manageable UI
+                    });
+                    return response.apps ?? [];
+                }
+                catch (error) {
+                    throw this.handleError(error);
+                }
+            }
+            /**
+             * Revoke the authenticated user's authorization for a specific RP origin.
+             *
+             * The next FedCM sign-in from that origin will require explicit re-consent.
+             * The corresponding cache entry is invalidated so a subsequent
+             * `listAuthorizedApps()` call sees fresh data.
+             */
+            async revokeAuthorizedApp(origin) {
+                try {
+                    await this.makeRequest('DELETE', `/fedcm/me/authorized-apps/${encodeURIComponent(origin)}`, undefined, { cache: false });
+                    this.clearCacheEntry('GET:/fedcm/me/authorized-apps');
+                }
+                catch (error) {
+                    throw this.handleError(error);
+                }
+            }
         },
         _a.DEFAULT_CONFIG_URL = 'https://auth.oxy.so/fedcm.json',
         _a.FEDCM_TIMEOUT = 15000 // 15 seconds for interactive
     ,
-        _a.FEDCM_SILENT_TIMEOUT = 3000 // 3 seconds for silent mediation
+        // Silent mediation runs on page load (e.g. re-signing-in a user whose stored
+        // session was cleared after a cold-boot token fetch 401'd). The real silent
+        // round-trip — mint nonce → navigator.credentials.get → /fedcm/exchange — was
+        // measured to take more than 3s for live users, so a 3s budget timed out and
+        // left them signed out on reload. 10s gives ample margin while staying bounded.
+        _a.FEDCM_SILENT_TIMEOUT = 10000 // 10 seconds for silent mediation
     ,
         _a;
 }

package/dist/esm/mixins/OxyServices.popup.js

@@ -77,7 +77,37 @@ export function OxyServicesPopupAuthMixin(Base) {
                     clientId: window.location.origin,
                     redirectUri: `${this.resolveAuthUrl()}/auth/callback`,
                 });
-                const popup = this.openCenteredPopup(authUrl, 'Oxy Sign In', width, height);
+                // If the caller pre-opened a popup on the raw user gesture (recommended
+                // path — see `openBlankPopup` and `PopupAuthOptions.popup`), navigate it
+                // to the auth URL instead of issuing a fresh `window.open` (which would
+                // be blocked once any prior `await` has consumed the user activation).
+                let popup;
+                const preOpened = options.popup ?? null;
+                if (preOpened) {
+                    if (preOpened.closed) {
+                        // The pre-opened popup is gone — distinguish a user cancel (they
+                        // closed the blank window before sign-in could navigate it) from a
+                        // blocker rejection. Lumping these together as "Popup blocked" is
+                        // misleading: the popup was NOT blocked, it was opened successfully
+                        // and then dismissed.
+                        throw new OxyAuthenticationError('Sign-in window was closed before authentication could start.');
+                    }
+                    try {
+                        preOpened.location.replace(authUrl);
+                    }
+                    catch (replaceError) {
+                        // `location.replace` can throw in sandboxed / cross-origin-locked
+                        // environments. Fall back to `href` assignment, which is more
+                        // permissive. Logged at debug-level so consumers can correlate
+                        // unusual sign-in behaviour without producing noise in normal flows.
+                        debug.warn('location.replace failed, falling back to location.href', replaceError);
+                        preOpened.location.href = authUrl;
+                    }
+                    popup = preOpened;
+                }
+                else {
+                    popup = this.openCenteredPopup(authUrl, 'Oxy Sign In', width, height);
+                }
                 if (!popup) {
                     throw new OxyAuthenticationError('Popup blocked. Please allow popups for this site and try again.');
                 }
@@ -220,6 +250,36 @@ export function OxyServicesPopupAuthMixin(Base) {
                     document.body.removeChild(iframe);
                 }
             }
+            /**
+             * Open a blank, centered popup window SYNCHRONOUSLY.
+             *
+             * Use this in a click (or other user-gesture) handler BEFORE any `await`
+             * to capture the transient user-activation. Pass the returned handle into
+             * `signInWithPopup({ popup })` once the async portion of the flow runs.
+             *
+             * Returns `null` if the browser's popup blocker rejected the open.
+             *
+             * @example
+             * ```typescript
+             * const onSignInClick = () => {
+             *   const popup = oxyServices.openBlankPopup();
+             *   (async () => {
+             *     const silent = await oxyServices.silentSignInWithFedCM();
+             *     if (silent) { popup?.close(); return; }
+             *     await oxyServices.signInWithPopup({ popup });
+             *   })();
+             * };
+             * ```
+             */
+            openBlankPopup(width, height) {
+                if (typeof window === 'undefined') {
+                    return null;
+                }
+                const ctor = this.constructor;
+                const w = width ?? ctor.POPUP_WIDTH;
+                const h = height ?? ctor.POPUP_HEIGHT;
+                return this.openCenteredPopup('about:blank', 'Oxy Sign In', w, h);
+            }
             /**
              * Open a centered popup window
              *

package/dist/esm/mixins/OxyServices.user.js

@@ -215,6 +215,24 @@ export function OxyServicesUserMixin(Base) {
                 throw this.handleError(error);
             }
         }
+        /**
+         * Update the authenticated user's notification preferences.
+         *
+         * Thin wrapper over `updateProfile` that constrains the patch to known
+         * notification channels — same persistence path, same cache invalidation,
+         * but type-safe at the call site.
+         */
+        async updateNotificationPreferences(preferences) {
+            return this.updateProfile({ notificationPreferences: preferences });
+        }
+        /**
+         * Update the authenticated user's general preferences (language, theme,
+         * reduce-motion, timezone). Persisted on the User document via
+         * `PUT /users/me` — same cache-invalidation behaviour as `updateProfile`.
+         */
+        async updateUserPreferences(preferences) {
+            return this.updateProfile({ userPreferences: preferences });
+        }
         /**
          * Request account verification
          */

package/dist/esm/utils/accountUtils.js

@@ -108,3 +108,64 @@ export const createQuickAccount = (sessionId, userData, existingAccount, getFile
         avatarUrl,
     };
 };
+/**
+ * Merge a fresh `/auth/refresh-all` snapshot into an existing QuickAccount
+ * list, preserving any cached fields (`avatarUrl`) for slots that didn't
+ * change. The fresh response is canonical: the resulting list contains EXACTLY
+ * the slots present in `fresh`, sorted by `authuser` ascending. Stale stored
+ * accounts that no longer appear in `fresh` are dropped (the server already
+ * authoritatively cleared the corresponding cookie).
+ *
+ * @param stored Previously persisted QuickAccount list (any order).
+ * @param fresh Server's authoritative refresh-all response.
+ * @returns Canonical merged list, sorted by `authuser` asc.
+ */
+export const mergeAccountsFromRefreshAll = (stored, fresh) => {
+    const storedByAuthuser = new Map();
+    if (stored) {
+        for (const account of stored) {
+            if (typeof account.authuser === 'number') {
+                storedByAuthuser.set(account.authuser, account);
+            }
+        }
+    }
+    const merged = fresh.map((entry) => {
+        const previous = storedByAuthuser.get(entry.authuser);
+        // `entry.user` is null on the SDK legacy-fallback path; preserve any
+        // previously cached identity for that slot rather than overwriting
+        // it with blanks, and let the AuthManager's getCurrentUser() hydration
+        // refresh it on the next snapshot.
+        const wireUser = entry.user;
+        const username = wireUser?.username ?? previous?.username ?? '';
+        const displayName = getAccountDisplayName({
+            name: wireUser?.name,
+            username,
+        });
+        const avatar = wireUser?.avatar ?? previous?.avatar ?? undefined;
+        const avatarUrl = previous && previous.avatar === avatar ? previous.avatarUrl : undefined;
+        return {
+            sessionId: entry.sessionId,
+            userId: wireUser?.id ?? previous?.userId,
+            username,
+            displayName,
+            avatar,
+            avatarUrl,
+            authuser: entry.authuser,
+            color: wireUser?.color ?? previous?.color ?? null,
+        };
+    });
+    merged.sort((a, b) => {
+        const aIdx = a.authuser ?? Number.POSITIVE_INFINITY;
+        const bIdx = b.authuser ?? Number.POSITIVE_INFINITY;
+        return aIdx - bIdx;
+    });
+    return merged;
+};
+/**
+ * Return the account's preferred Bloom color preset, or `null` if it has no
+ * preference. Centralises the `color ?? null` normalisation so consumers can
+ * drive per-account theming without duplicating the nullish-handling.
+ */
+export const getAccountColor = (account) => {
+    return account.color ?? null;
+};