@xh/hoist 72.0.0-SNAPSHOT.1736809021133 → 72.0.0-SNAPSHOT.1736879763062

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## v72.0.0-SNAPSHOT - unreleased
 
+### ⚙️ Typescript API Adjustments
+
+* Improved signature of `HoistBase.markPersist`.
+
 ## v71.0.0 - 2025-01-08
 
 ### 💥 Breaking Changes (upgrade difficulty: 🟠 MEDIUM - Hoist core update, import adjustments)

package/build/types/core/HoistBase.d.ts

@@ -134,7 +134,7 @@ export declare abstract class HoistBase {
      * @param options - options governing the persistence of this object. These will be applied
      *      on top of any default persistWith options defined on the instance itself.
      */
-    markPersist(property: keyof this & string, options?: PersistOptions): void;
+    markPersist<P extends keyof this & string>(property: P, options?: PersistOptions): void;
     /** @returns true if this instance has been destroyed. */
     get isDestroyed(): boolean;
     /**

package/build/types/security/BaseOAuthClient.d.ts

@@ -142,6 +142,8 @@ export declare abstract class BaseOAuthClient<C extends BaseOAuthClientConfig<S>
      * @param key - key for re-accessing this state, as round-tripped with redirect.
      */
     protected restoreRedirectState(key: string): void;
+    /** Call after requesting the provider library redirect the user away for auth. */
+    protected maskAfterRedirect(): Promise<void>;
     protected fetchAllTokensAsync(useCache?: boolean): Promise<TokenMap>;
     protected getLocalStorage(key: string, defaultValue?: any): any;
     protected setLocalStorage(key: string, value: any): void;

package/core/HoistBase.ts

@@ -256,7 +256,7 @@ export abstract class HoistBase {
      * @param options - options governing the persistence of this object. These will be applied
      *      on top of any default persistWith options defined on the instance itself.
      */
-    markPersist(property: keyof this & string, options: PersistOptions = {}) {
+    markPersist<P extends keyof this & string>(property: P, options: PersistOptions = {}) {
         // Read from and attach to Provider, failing gently
         PersistenceProvider.create({
             persistOptions: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "72.0.0-SNAPSHOT.1736809021133",
+  "version": "72.0.0-SNAPSHOT.1736879763062",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",

package/security/BaseOAuthClient.ts

@@ -4,8 +4,11 @@
  *
  * Copyright © 2025 Extremely Heavy Industries Inc.
  */
+import {br, fragment} from '@xh/hoist/cmp/layout';
 import {HoistBase, managed, XH} from '@xh/hoist/core';
+import {Icon} from '@xh/hoist/icon';
 import {action, makeObservable} from '@xh/hoist/mobx';
+import {never, wait} from '@xh/hoist/promise';
 import {Token, TokenMap} from '@xh/hoist/security/Token';
 import {Timer} from '@xh/hoist/utils/async';
 import {MINUTES, olderThan, ONE_MINUTE, SECONDS} from '@xh/hoist/utils/datetime';
@@ -285,6 +288,32 @@ export abstract class BaseOAuthClient<C extends BaseOAuthClientConfig<S>, S> ext
         window.history.replaceState(null, '', pathname + search);
     }
 
+    /** Call after requesting the provider library redirect the user away for auth. */
+    protected async maskAfterRedirect() {
+        // We expect the tab to unload momentarily and be redirected to the provider's login page.
+        // Wait here to ensure we mask the app while the browser processes the redirect...
+        await wait(10 * SECONDS);
+
+        // ...but also handle an observed edge case where the browser decided to open a new tab
+        // instead of redirecting the current one (https://github.com/xh/hoist-react/issues/3899).
+        // Show message below if/when user swaps back to the original tab, vs. endless spinner.
+        await XH.alert({
+            title: 'Auth / Redirect Error',
+            message: fragment(
+                'Authentication did not complete as expected / tab was not redirected.',
+                br(),
+                br(),
+                'Please click below or refresh this tab in your browser to try again.'
+            ),
+            confirmProps: {text: 'Reload', icon: Icon.refresh(), intent: 'primary', minimal: false}
+        });
+
+        XH.reloadApp();
+
+        // Ensure stale init does not progress - this tab should *really* be on its way out now!
+        await never();
+    }
+
     protected async fetchAllTokensAsync(useCache = true): Promise<TokenMap> {
         const ret: TokenMap = {},
             {accessSpecs} = this;

package/security/authzero/AuthZeroClient.ts

@@ -7,7 +7,7 @@
 import type {Auth0ClientOptions} from '@auth0/auth0-spa-js';
 import {Auth0Client} from '@auth0/auth0-spa-js';
 import {XH} from '@xh/hoist/core';
-import {never, wait} from '@xh/hoist/promise';
+import {wait} from '@xh/hoist/promise';
 import {Token, TokenMap} from '@xh/hoist/security/Token';
 import {SECONDS} from '@xh/hoist/utils/datetime';
 import {mergeDeep, throwIf} from '@xh/hoist/utils/js';
@@ -99,11 +99,13 @@ export class AuthZeroClient extends BaseOAuthClient<AuthZeroClientConfig, AuthZe
 
     protected override async doLoginRedirectAsync(): Promise<void> {
         const appState = this.captureRedirectState();
+
         await this.client.loginWithRedirect({
             appState,
             authorizationParams: {scope: this.loginScope}
         });
-        await never();
+
+        await this.maskAfterRedirect();
     }
 
     protected override async doLoginPopupAsync(): Promise<void> {

package/security/msal/MsalClient.ts

@@ -10,11 +10,9 @@ import {
     IPublicClientApplication,
     LogLevel,
     PopupRequest,
-    RedirectRequest,
     SilentRequest
 } from '@azure/msal-browser';
 import {XH} from '@xh/hoist/core';
-import {never} from '@xh/hoist/promise';
 import {Token, TokenMap} from '@xh/hoist/security/Token';
 import {logDebug, logError, logInfo, logWarn, mergeDeep, throwIf} from '@xh/hoist/utils/js';
 import {flatMap, union, uniq} from 'lodash';
@@ -201,18 +199,18 @@ export class MsalClient extends BaseOAuthClient<MsalClientConfig, MsalTokenSpec>
     }
 
     protected override async doLoginRedirectAsync(): Promise<void> {
-        const {client} = this,
-            state = this.captureRedirectState(),
-            opts: RedirectRequest = {
-                state,
-                loginHint: this.getSelectedUsername(),
-                domainHint: this.config.domainHint,
-                scopes: this.loginScopes,
-                extraScopesToConsent: this.loginExtraScopesToConsent,
-                redirectUri: this.redirectUrl
-            };
-        await client.acquireTokenRedirect(opts);
-        await never();
+        const state = this.captureRedirectState();
+
+        await this.client.acquireTokenRedirect({
+            state,
+            loginHint: this.getSelectedUsername(),
+            domainHint: this.config.domainHint,
+            scopes: this.loginScopes,
+            extraScopesToConsent: this.loginExtraScopesToConsent,
+            redirectUri: this.redirectUrl
+        });
+
+        await this.maskAfterRedirect();
     }
 
     protected override async fetchIdTokenAsync(useCache: boolean = true): Promise<Token> {