refactor: in-process catalog sync + concurrent provider opens (MAPCO-11560)

[shimoncohen]

Summary

Stacked on #52. Replaces the CSW catalog-sync worker_threads Worker with an in-process, self-rescheduling setTimeout poll loop (CatalogSyncManager), and opens GeoTIFF providers concurrently instead of sequentially.

• src/heights/models/catalogSyncManager.ts (new) — owns the CSW client, fetch filter, isSame diff, and scheduling. Injects only CONFIG+LOGGER; receives the CatalogRecords + DEMTerrainCacheManager singletons via start(...) to avoid a circular import with containerConfig. Fetch errors are caught, logged, and the loop reschedules — self-healing where a dead worker used to freeze the cache silently.
• src/containerConfig.ts — worker removed; manager registered as a singleton, started after DI registration, and stopped in onSignal (graceful shutdown the worker never had).
• src/heights/models/DEMTerrainCacheManager.ts — initProviders sequential for loop → bounded PromisePool (concurrency = samplingConcurrency), preserving per-record error isolation.
• src/workerCatalogRecords.ts — deleted.

Why

Worker threads are for CPU-bound work; this job is pure network I/O (CSW fetch + GeotiffHeightProvider.fromUrl). The worker bought nothing, structured-cloned the record set across the thread boundary every cycle, and its exit/error handlers only logged — a dead worker silently froze the cache.

Behavior parity

/points API, response shape, and openapi3.yaml unchanged. Same CSW filter (GEOTIFF / PUBLISHED), same 1..1000 bounds, same isSame-gated update-only-on-change, same non-blocking startup.

Test Plan

• npm run test:unit — 25/25 (adds catalogSyncManager + concurrent-initProviders coverage)
• npm run test:integration — 8/8 (manager stubbed in the DI override so the real poll loop never runs in tests — no leaked timers, clean Jest exit)
• npm run build clean
• No new eslint errors vs the feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52 base (see notes)

Notes / accepted trade-offs

• Base branch is feat/geotiff-heights-migration (feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52), not master. Rebase onto master before merge if feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52 lands first.
• Pre-existing eslint errors on the feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52 base (DEMTerrainCacheManager.ts line 53, integration spec lines 127/158, other feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52 test files) are untouched — this branch adds none. CI (wearerequired/lint-action) already reflects the base state.
• samplingConcurrency is reused to bound provider-open concurrency (records are few; avoids a new config key across default/env/helm).
• onSignal resolves the manager from the global tsyringe container, matching the prior worker-handler pattern (useChild is never set true).
• PromisePool.withConcurrency throws on a NaN/0 samplingConcurrency; a bad value already breaks the query path on feat: replace QMesh terrain engine with direct GeoTIFF/COG sampling (MAPCO-11560) #52, so config validation is the right home for that — out of scope here.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ATQerkHAV3Vur82HVVFack