fix(map): resolve initialCoordinates even when IP lookup is skipped

Deep links (skip: true) would leave initialCoordinates null forever; combined
with a listing that has no coordinates or resolves to an error sentinel, the
<Map> never mounted. Fall back to DEFAULT_COORDINATES on skip so MapView can
always render.

Made-with: Cursor

Author: dnywh

src/features/map/components/MapView.tsx

@@ -181,8 +181,8 @@ export default function MapView({
   // MapLibre's `initialViewState` is only consumed once at mount, so we wait
   // for either a selected listing or the IP-based (or fallback) initial
   // centre to resolve before mounting the Map. `useIpInitialLocation`
-  // always resolves (to DEFAULT_COORDINATES on failure), so this cannot
-  // stall indefinitely.
+  // always resolves (to DEFAULT_COORDINATES on failure or when skipped), so
+  // this cannot stall indefinitely.
   const hasInitialPosition =
     hasValidCoordinates(selectedListing) || Boolean(initialCoordinates);
 

src/features/map/hooks/useIpInitialLocation.ts

@@ -30,9 +30,10 @@ function ensureMapTilerConfig() {
   hasConfiguredMapTiler = true;
 }
 
-// One-time IP-based initial centre. On timeout or error we still resolve to
-// `DEFAULT_COORDINATES` so MapView, which gates on `initialCoordinates`
-// being set, always eventually mounts.
+// One-time IP-based initial centre. On timeout, error, or when skipped we
+// still resolve to `DEFAULT_COORDINATES` so MapView, which gates on
+// `initialCoordinates` being set, always eventually mounts — including deep
+// links to listings with `coordinates: null` or that resolve to an error.
 export function useIpInitialLocation({
   skip = false,
 }: UseIpInitialLocationArgs = {}): UseIpInitialLocationResult {
@@ -42,7 +43,16 @@ export function useIpInitialLocation({
   const [countryCode, setCountryCode] = useState<string | null>(null);
 
   useEffect(() => {
-    if (skip) return;
+    if (skip) {
+      // Deep-linked: we're not running the IP lookup, but MapView still
+      // needs a non-null initial centre in case the listing itself has no
+      // valid coordinates (error sentinel or coordinates: null).
+      setInitialCoordinates({
+        ...DEFAULT_COORDINATES,
+        zoom: ZOOM_LEVEL_DEFAULT,
+      });
+      return;
+    }
 
     ensureMapTilerConfig();