@aguacerowx/react-native 0.0.50 → 0.0.52

package/lib/module/WeatherLayerManager.js

@@ -14,6 +14,9 @@ import { SatelliteAndroidController } from './satellite/satelliteAndroidControll
 import NwsAlertsOverlay from './nws/NwsAlertsOverlay';
 import { mapRegistry } from './MapRegistry';
 import { satBridgeWarn } from './satelliteBridgeDiag';
+import { augmentProcessFrameOptionsForDebug, aguaceroDebugWarn, configureAguaceroRnDebug, getAguaceroAuthDiagnosticSnapshot, isAguaceroRnDebugEnabled } from './aguaceroRnDebug';
+import { installAguaceroCoreDebugHooks, logProcessFrameAuthMismatch } from './aguaceroCoreDebugHooks';
+import { resolveGridRequestSiteOrigin } from './gridCdnAuth';
 const NEXRAD_NATIVE = Platform.OS === 'android' || Platform.OS === 'ios';
 const SATELLITE_NATIVE = Platform.OS === 'android' || Platform.OS === 'ios';
 satBridgeWarn('SDK fingerprint', {
@@ -22,9 +25,9 @@ satBridgeWarn('SDK fingerprint', {
   note: 'If you never see sat-bridge logs, the app bundle is not loading this WeatherLayerManager build.'
 });
 
-/**
- * Same filtering as {@link AguaceroCore#_getFilteredMrmsTimestampsForVariable} for older cores
- * where that helper (or {@code setMRMSDurationValue}) is missing from the prototype chain.
+/**
+ * Same filtering as {@link AguaceroCore#_getFilteredMrmsTimestampsForVariable} for older cores
+ * where that helper (or {@code setMRMSDurationValue}) is missing from the prototype chain.
  */
 function getFilteredMrmsTimestampsCompat(core, variable) {
   const raw = core.mrmsStatus?.[variable];
@@ -39,9 +42,9 @@ function getFilteredMrmsTimestampsCompat(core, variable) {
   return list;
 }
 
-/**
- * Older npm installs may resolve an {@link AguaceroCore} without {@code setMRMSDurationValue} /
- * {@code setNexradDurationValue}; mirror those methods here using the same logic as the SDK.
+/**
+ * Older npm installs may resolve an {@link AguaceroCore} without {@code setMRMSDurationValue} /
+ * {@code setNexradDurationValue}; mirror those methods here using the same logic as the SDK.
  */
 async function applyMrmsDurationValue(core, value) {
   if (typeof core.setMRMSDurationValue === 'function') {
@@ -106,12 +109,12 @@ const {
   InspectorModule
 } = NativeModules;
 
-/**
- * `state:change` payloads can briefly have {@code isNexrad} before {@code nexradSite} / {@code nexradTimestamp}
- * are populated; {@code core.state} is updated first. Merge so readouts match the active radar.
- *
- * @param {object | null} emitted
- * @param {object | null} coreState
+/**
+ * `state:change` payloads can briefly have {@code isNexrad} before {@code nexradSite} / {@code nexradTimestamp}
+ * are populated; {@code core.state} is updated first. Merge so readouts match the active radar.
+ *
+ * @param {object | null} emitted
+ * @param {object | null} coreState
  */
 function mergeNexradEmittedWithCore(emitted, coreState) {
   if (!emitted || !coreState) return null;
@@ -126,10 +129,10 @@ function mergeNexradEmittedWithCore(emitted, coreState) {
   };
 }
 
-/**
- * Compact timeline identity for deduping {@code state:change}: extending the NEXRAD/satellite window
- * often keeps the same selected unix — without this, {@link WeatherLayerManager} short-circuits and
- * never calls native {@code sync} / preload with the expanded frame list.
+/**
+ * Compact timeline identity for deduping {@code state:change}: extending the NEXRAD/satellite window
+ * often keeps the same selected unix — without this, {@link WeatherLayerManager} short-circuits and
+ * never calls native {@code sync} / preload with the expanded frame list.
  */
 function nexradObsTimelineSig(state) {
   const arr = [...(state?.availableNexradTimestamps || [])].map(Number).filter(t => Number.isFinite(t)).sort((a, b) => a - b);
@@ -142,8 +145,9 @@ function satelliteObsTimelineSig(state) {
   return `${keys.length}:${keys[0]}:${keys[keys.length - 1]}`;
 }
 
-/** True in __DEV__, or set `globalThis.__AGUACERO_WX_GRID_DEBUG__ = true` in the app entry for release builds. */
+/** True when {@link WeatherLayerManager} `debug` / {@link configureAguaceroRnDebug}, or legacy `__AGUACERO_WX_GRID_DEBUG__`. */
 function wxGridDebugEnabled() {
+  if (isAguaceroRnDebugEnabled()) return true;
   try {
     if (typeof __DEV__ !== 'undefined' && __DEV__) return true;
     return Boolean(typeof globalThis !== 'undefined' && globalThis.__AGUACERO_WX_GRID_DEBUG__);
@@ -169,37 +173,34 @@ function wxGridWarn(tag, detail) {
   }
 }
 
-/**
- * Native {@code processFrame} must mirror browser/AguaceroCore grid auth: encoded {@code apiKey}
- * in the query string and optional {@code Origin} / {@code Referer} (many CloudFront setups require them).
- *
- * @param {string} baseGridUrl
- * @param {string} resourcePath
- * @param {string} apiKey
- * @param {string | null | undefined} bundleId
- * @param {string | undefined} gridRequestSiteOrigin - e.g. production web origin your edge allowlists (no trailing slash)
+/**
+ * Native {@code processFrame} must mirror browser/AguaceroCore grid auth: encoded {@code apiKey}
+ * in the query string and optional {@code Origin} / {@code Referer} (many CloudFront setups require them).
+ *
+ * @param {string} baseGridUrl
+ * @param {string} resourcePath
+ * @param {string} apiKey
+ * @param {string | null | undefined} bundleId
+ * @param {string | undefined} gridRequestSiteOrigin - prop or core origin (see {@link resolveGridRequestSiteOrigin})
+ * @param {import('@aguacerowx/javascript-sdk').AguaceroCore | null | undefined} core
  */
-function buildGridFrameProcessOptions(baseGridUrl, resourcePath, apiKey, bundleId, gridRequestSiteOrigin) {
-  const url = `${baseGridUrl}${resourcePath}?apiKey=${encodeURIComponent(apiKey)}`;
+function buildGridFrameProcessOptions(baseGridUrl, resourcePath, apiKey, bundleId, gridRequestSiteOrigin, core) {
+  const trimmedKey = typeof apiKey === 'string' ? apiKey.trim() : apiKey;
+  const url = `${baseGridUrl}${resourcePath}?apiKey=${encodeURIComponent(trimmedKey || '')}`;
   const options = {
     url,
-    apiKey,
+    apiKey: trimmedKey,
     bundleId
   };
-  if (typeof gridRequestSiteOrigin === 'string') {
-    let origin = gridRequestSiteOrigin.trim();
-    while (origin.endsWith('/')) {
-      origin = origin.slice(0, -1);
-    }
-    if (origin.length > 0) {
-      options.gridRequestSiteOrigin = origin;
-    }
+  const origin = resolveGridRequestSiteOrigin(gridRequestSiteOrigin, core);
+  if (origin) {
+    options.gridRequestSiteOrigin = origin;
   }
   return options;
 }
 
-/**
- * A helper function to generate the raw RGBA byte buffer for the colormap texture.
+/**
+ * A helper function to generate the raw RGBA byte buffer for the colormap texture.
  */
 const _generateColormapBytes = colormap => {
   const width = 256;
@@ -261,8 +262,15 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
     onNwsAlertClick,
     /** Same semantics as mapsgl / AguaceroCore: Origin + Referer for grid CDN (required by many CloudFront rules). */
     gridRequestSiteOrigin,
+    /** When true, logs auth/HTTP diagnostics under `[AguaceroRN][debug]` (Metro / Logcat / Xcode). */
+    debug = false,
     ...restProps
   } = props;
+  useEffect(() => {
+    configureAguaceroRnDebug({
+      enabled: Boolean(debug)
+    });
+  }, [debug]);
   const context = useContext(AguaceroContext);
 
   // Create the core here instead of getting it from context
@@ -277,6 +285,24 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
     },
     autoRefresh: false // <-- add this
   }), [apiKey, gridRequestSiteOrigin]);
+  useEffect(() => {
+    if (!core) return;
+    installAguaceroCoreDebugHooks(core, {
+      gridRequestSiteOriginProp: gridRequestSiteOrigin ?? null,
+      debugProp: Boolean(debug)
+    });
+    if (isAguaceroRnDebugEnabled() && !apiKey) {
+      aguaceroDebugWarn('WeatherLayerManager.missingApiKey', {
+        hint: 'apiKey prop is empty — all CDN requests will fail or return 403'
+      });
+    }
+    if (isAguaceroRnDebugEnabled() && apiKey && !gridRequestSiteOrigin) {
+      aguaceroDebugWarn('WeatherLayerManager.missingGridOrigin', {
+        hint: 'gridRequestSiteOrigin is not set — CloudFront often returns 403 without Origin/Referer on React Native',
+        snapshot: getAguaceroAuthDiagnosticSnapshot(core)
+      });
+    }
+  }, [core, debug, gridRequestSiteOrigin, apiKey]);
   const [watchesWarningsOptions, setWatchesWarningsOptions] = useState(() => ({
     alertInteractionEnabled: true,
     ...(watchesWarningsProp ?? {})
@@ -380,6 +406,8 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
   // Track if we've done the initial load
   const hasInitialLoad = useRef(false);
   const hasPreloadedRef = useRef(false);
+  /** Bumped on {@code cancelAllFrames} / full reload so stale {@code processFrame} results are ignored. */
+  const preloadGenerationRef = useRef(0);
 
   // Track the last state we processed to avoid redundant updates
   const lastProcessedState = useRef(null);
@@ -460,9 +488,9 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
       refreshData: () => {
         _checkForUpdates();
       },
-      /**
-       * NWS watches/warnings (native map, iOS + Android): same options as mapsgl {@link WeatherLayerManager#configureWatchesWarnings}.
-       * @param {object} partial
+      /**
+       * NWS watches/warnings (native map, iOS + Android): same options as mapsgl {@link WeatherLayerManager#configureWatchesWarnings}.
+       * @param {object} partial
        */
       configureWatchesWarnings: partial => {
         setWatchesWarningsOptions(prev => ({
@@ -526,6 +554,8 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
 
     // Only mark as "has preloaded" after validation passes
     hasPreloadedRef.current = true;
+    const generation = preloadGenerationRef.current;
+    const isStaleGeneration = () => generation !== preloadGenerationRef.current;
     wxGridVerbose('preloadStart', {
       isMRMS,
       model,
@@ -642,7 +672,11 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
       } else {
         resourcePath = `/grids/${model}/${date}/${run}/${currentFrame}/${variable}/0`;
       }
-      const options = buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin);
+      const options = augmentProcessFrameOptionsForDebug(buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin, core), core);
+      logProcessFrameAuthMismatch(core, options, {
+        phase: 'preloadCurrent',
+        currentCacheKey
+      });
       try {
         wxGridVerbose('processFrameRequest', {
           currentCacheKey,
@@ -650,6 +684,13 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
           frame: currentFrame
         });
         const result = await WeatherFrameProcessorModule.processFrame(options);
+        if (isStaleGeneration()) {
+          wxGridVerbose('preloadCurrentFrameStale', {
+            currentCacheKey,
+            generation
+          });
+          return;
+        }
         if (!result || !result.filePath) {
           hasPreloadedRef.current = false;
           wxGridWarn('preloadAbort', {
@@ -747,12 +788,20 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
         const cancelled = e && (e.code === 'E_CANCELLED' || e?.userInfo?.code === 'E_CANCELLED' || typeof e?.message === 'string' && e.message.includes('superseded'));
         if (!cancelled) {
           hasPreloadedRef.current = false;
-          wxGridWarn('preloadProcessFrameError', {
+          const errDetail = {
             currentCacheKey,
             code: e?.code,
             message: e?.message,
             userInfo: e?.userInfo
-          });
+          };
+          wxGridWarn('preloadProcessFrameError', errDetail);
+          if (isAguaceroRnDebugEnabled()) {
+            aguaceroDebugWarn('preloadProcessFrameError', {
+              ...errDetail,
+              auth: getAguaceroAuthDiagnosticSnapshot(core),
+              is403: e?.code === 'HTTP_ERROR' && typeof e?.message === 'string' && e.message.includes('403')
+            });
+          }
         } else {
           wxGridVerbose('preloadProcessFrameCancelled', {
             currentCacheKey
@@ -777,8 +826,15 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
       } else {
         resourcePath = `/grids/${model}/${date}/${run}/${frame}/${variable}/0`;
       }
-      const options = buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin);
+      const options = augmentProcessFrameOptionsForDebug(buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin, core), core);
+      logProcessFrameAuthMismatch(core, options, {
+        phase: 'preloadBackground',
+        cacheKey
+      });
       WeatherFrameProcessorModule.processFrame(options).then(result => {
+        if (isStaleGeneration()) {
+          return;
+        }
         if (!result || !result.filePath) {
           return;
         }
@@ -1166,7 +1222,11 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
             m = (frameDate.getUTCMonth() + 1).toString().padStart(2, '0'),
             d = frameDate.getUTCDate().toString().padStart(2, '0');
           const resourcePath = `/grids/mrms/${y}${m}${d}/${frame}/0/${currentVariable}/0`;
-          const options = buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin);
+          const options = augmentProcessFrameOptionsForDebug(buildGridFrameProcessOptions(core.baseGridUrl, resourcePath, core.apiKey, core.bundleId, gridRequestSiteOrigin, core), core);
+          logProcessFrameAuthMismatch(core, options, {
+            phase: 'mrmsRefresh',
+            cacheKey
+          });
           WeatherFrameProcessorModule.processFrame(options).then(result => {
             if (!result || !result.filePath) return;
             const frameData = {
@@ -1431,6 +1491,7 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
           }
         }
         hasPreloadedRef.current = false;
+        preloadGenerationRef.current += 1;
         preloadedDataCache.current.clear();
         cachedGeometry.current = null;
         cachedColormap.current = null;
@@ -1438,7 +1499,8 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
         WeatherFrameProcessorModule.cancelAllFrames();
         wxGridVerbose('cancelAllFrames', {
           reason: 'needsFullLoad',
-          variable: newState.variable
+          variable: newState.variable,
+          generation: preloadGenerationRef.current
         });
         if (!newState.variable) {
           previousStateRef.current = newState;
@@ -1460,6 +1522,8 @@ export const WeatherLayerManager = /*#__PURE__*/forwardRef((props, ref) => {
             mrmsTimestamp: newState.mrmsTimestamp,
             forecastHour: newState.forecastHour
           });
+          hasPreloadedRef.current = false;
+          void preloadAllFramesToDisk(newState);
         }
         if (success && newState.opacity !== renderProps.opacity) {
           setRenderProps(prev => ({