@diagrammo/dgmo 0.21.1 → 0.22.0

package/src/map/geo-query.ts

@@ -21,14 +21,19 @@ import type { DecodedFeature } from './geo';
 import { pixelToLonLat, lonLatToPixel } from './invert';
 import type { MapData, GeoExtent } from './resolved-types';
 import type { Gazetteer } from './data/types';
+import type { DgmoError } from '../diagnostics';
 
 /** Nearest gazetteer city to a point: the real haversine distance, plus the
- *  canonical name + ISO + (US-only) subdivision for token shaping. */
+ *  canonical name + ISO + (US-only) subdivision for token shaping. `lon`/`lat`
+ *  are the city's own gazetteer coordinates (so callers can mark it on the map,
+ *  distinct from the inspected point). */
 export interface NearestCity {
   readonly name: string;
   readonly iso: string;
   readonly sub?: string;
   readonly distanceKm: number;
+  readonly lon: number;
+  readonly lat: number;
 }
 
 /** A region declaration with its canonical/primary form plus bare alternates
@@ -85,6 +90,10 @@ export interface MapGeoQuery {
   locate(px: number, py: number): ResultCard | null;
   /** Culled + projected cities for the all-cities layer (population-primary). */
   cities(extent?: GeoExtent): ProjectedCity[];
+  /** Layout-time, dimension-dependent diagnostics. They live on the geo-query
+   *  (bound to the rendered layout) rather than the resolver. Callers merge them
+   *  with `resolved.diagnostics`. (No producers currently — always empty.) */
+  readonly diagnostics: readonly DgmoError[];
 }
 
 export interface CreateMapGeoQueryOptions {
@@ -144,6 +153,8 @@ function nearestCity(
     iso: c[2],
     ...(c[5] !== undefined && { sub: c[5] }),
     distanceKm: best.dist,
+    lat: c[0],
+    lon: c[1],
   };
 }
 
@@ -206,7 +217,14 @@ const MAX_CITY_DOTS = 250;
 
 /** Construct a geo-query handle bound to the layout for `(content, width,
  *  height, data, palette, isDark)`. Deterministic: identical inputs ⇒ the same
- *  fitted projection the rendered SVG used, so inverted clicks align. */
+ *  fitted projection the rendered SVG used, so inverted clicks align.
+ *
+ *  INVARIANT: this is the PREVIEW path — it never passes `preferContain`, so its
+ *  layout matches the in-app preview (stretch-fill), where geo-query is used. It is
+ *  NOT valid against a content-aware EXPORT canvas (which may set `preferContain` →
+ *  contain-fit): the inverted positions would not match that export's pixels. If
+ *  geo-query is ever pointed at an export canvas, thread `preferContain` through
+ *  `CreateMapGeoQueryOptions` to keep the projection in sync. */
 export function createMapGeoQuery(opts: CreateMapGeoQueryOptions): MapGeoQuery {
   const { content, width, height, data, palette, isDark } = opts;
   const resolved = resolveMap(parseMap(content), data);
@@ -273,5 +291,5 @@ export function createMapGeoQuery(opts: CreateMapGeoQueryOptions): MapGeoQuery {
     return out;
   };
 
-  return { invert, project, locate, cities };
+  return { invert, project, locate, cities, diagnostics: layout.diagnostics };
 }

package/src/map/geo.ts

@@ -1,7 +1,7 @@
 // Geometry helpers for the resolver: topology indexing + antimeridian-correct
 // feature bounds (via d3-geo geoBounds — NOT naive min/max, which breaks on the
 // antimeridian and on multi-part features like US Alaska/Hawaii; R5/R6).
-import { feature } from 'topojson-client';
+import { feature, neighbors } from 'topojson-client';
 import { geoBounds, geoArea } from 'd3-geo';
 import type { BoundaryTopology } from './data/types';
 import type { GeoExtent } from './resolved-types';
@@ -47,6 +47,52 @@ export function idSet(topo: BoundaryTopology): Set<string> {
   return new Set(geomObject(topo).geometries.map((g) => g.id));
 }
 
+// Memoize adjacency on the RAW asset object (never the per-render-mutated
+// `worldLayer`). Keyed by topology identity — the assets are stable singletons
+// from load-data.ts, so one build per topology lasts the process (G13).
+const adjacencyCache = new WeakMap<BoundaryTopology, Map<string, string[]>>();
+
+/** Per-topology arc-adjacency: ISO → neighbour ISOs, from shared TopoJSON arcs
+ *  (`topojson-client.neighbors()` on the RAW topology geometries — arcs live on
+ *  the topology, independent of any `feature()` decode — F2/ADR-4). Computed
+ *  per-topology (a country never neighbours a state) and memoized.
+ *
+ *  DATA HYGIENE (G1): the raw geometry array can carry (a) `type: null`
+ *  sovereignty stubs with no arcs (e.g. "Ashmore & Cartier Is." tagged `AU`) and
+ *  (b) genuine duplicate ISO ids. Null stubs are skipped (no arcs → no
+ *  adjacency), and every geometry sharing one ISO is UNIONED into a single ISO
+ *  node — matching the merged layer `decodeLayer` actually draws. Without this
+ *  the ISO-keyed graph corrupts and the AC9 no-collision guarantee degrades. */
+export function buildAdjacency(topo: BoundaryTopology): Map<string, string[]> {
+  const cached = adjacencyCache.get(topo);
+  if (cached) return cached;
+  const geometries = geomObject(topo).geometries as Array<{
+    id: string;
+    type?: string;
+  }>;
+  // neighbors() returns, per geometry BY ARRAY POSITION, the indices of
+  // arc-sharing geometries. Null-geometry stubs share no arcs → empty lists.
+  const nb = neighbors(geometries as never);
+  const sets = new Map<string, Set<string>>();
+  geometries.forEach((g, i) => {
+    if (!g.type || g.type === 'null') return; // skip arc-less sovereignty stubs
+    let set = sets.get(g.id);
+    if (!set) {
+      set = new Set<string>();
+      sets.set(g.id, set);
+    }
+    for (const j of nb[i] ?? []) {
+      const nid = geometries[j]?.id;
+      if (nid && nid !== g.id) set.add(nid); // union; never self
+    }
+  });
+  const out = new Map<string, string[]>();
+  // Deterministic neighbour order (sorted) so downstream coloring is stable.
+  for (const [iso, set] of sets) out.set(iso, [...set].sort());
+  adjacencyCache.set(topo, out);
+  return out;
+}
+
 /** A decoded boundary feature: the GeoJSON geometry plus its ISO id and display
  *  name (carried straight from the topology's `properties.name`). Used for
  *  point-in-polygon reverse-geocoding by the geo-query. */