@wandelbots/nova-js 4.9.0 → 4.9.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@wandelbots/nova-js",
   "type": "module",
-  "version": "4.9.0",
+  "version": "4.9.1",
   "description": "Official JS client for the Wandelbots API",
   "sideEffects": false,
   "packageManager": "pnpm@11.9.0+sha512.bd682d5d03fe525ef7c9fd6780c6884d1e756ac4c9c9fe00c538782824310dcf90e3ddc4f53835f06dfaebd5085e41855e0bcbb3b60de2ac5bbab89e5036f03b",

package/src/lib/Nova.ts

@@ -3,7 +3,7 @@ import type { AxiosRequestConfig } from "axios"
 import axios, { isAxiosError } from "axios"
 import { AutoReconnectingWebsocket } from "./AutoReconnectingWebsocket.ts"
 import { availableStorage } from "./availableStorage.ts"
-import { isLocalhostDev } from "./context.ts"
+import { hasBrowserLocation, isLocalhostDev } from "./context.ts"
 import { parseNovaInstanceUrl } from "./converters.ts"
 import { guardedPageReload } from "./errorHandling.ts"
 import { loginWithAuth0 } from "./LoginWithAuth0.ts"
@@ -70,7 +70,10 @@ export class Nova {
       return request
     })
 
-    if (typeof window !== "undefined") {
+    // The handler below redirects the page on a 401/403/503, which only works
+    // where a location exists. React Native has a `window` without one, and
+    // there the errors bubble up to the caller as they do in node.
+    if (hasBrowserLocation) {
       axiosInstance.interceptors.response.use(
         (r) => r,
         async (error) => {

package/src/lib/context.ts

@@ -1,8 +1,15 @@
 // Some general checks about the context the nova-js code is running in
 
-const isBrowser = typeof window !== "undefined"
+// `typeof window` alone does not tell us whether a browser is present: React
+// Native defines `window` as an alias of `global`, and that object has no
+// `location`. Reading a property of it throws while this module loads, which
+// breaks the import in a native app. Code that needs the browser APIs
+// (`window.location`, `window.sessionStorage`) has to check this instead of
+// `typeof window`.
+export const hasBrowserLocation =
+  typeof window !== "undefined" && typeof window.location !== "undefined"
 
 export const isLocalhostDev =
-  isBrowser &&
+  hasBrowserLocation &&
   window.location.hostname === "localhost" &&
   process.env.NODE_ENV === "development"