fix(api): replace debug console.log with console.warn in production handlers

[NewCoder3294]

Summary

Cleans up console.log usage in server handler files under server/worldmonitor/. All 20 console.log calls were either upgraded to console.warn (for operationally significant messages) or removed (debug/trace noise).

Changes by file

server/worldmonitor/military/v1/get-usni-fleet-report.ts (4 calls):

• Removed [USNI Fleet] Fetching from WordPress API... — debug trace, fires every fetch
• Upgraded to console.warn [USNI Fleet] Parsed: N vessels, N CSGs, N regions — operational summary worth monitoring
• Removed [USNI Fleet] Cache hit — routine cache behavior, pure noise
• Upgraded to console.warn [USNI Fleet] Returning stale cached data — noteworthy fallback condition (serving stale data after an error)

server/worldmonitor/aviation/v1/_shared.ts (10 calls):

• Upgraded to console.warn [Aviation] No AVIATIONSTACK_API key — skipping — missing API key is noteworthy
• Removed [Aviation] Querying N airports (concurrency=N) — debug trace on every batch run
• Upgraded to console.warn [Aviation] Done: N ok, N failed, N alerts, healthy=bool — operational summary with failure counts
• Removed [Aviation] IATA: N flights, N cancelled, N delayed — per-airport trace, extremely chatty
• Upgraded to console.warn [Aviation] NOTAM: no ICAO_API_KEY — skipping — missing API key
• Removed [Aviation] NOTAM: fetching via relay for N airports — debug trace
• Removed [Aviation] NOTAM: fetching direct for N airports — debug trace
• Removed [Aviation] NOTAM: N raw NOTAMs received — debug trace
• Upgraded to console.warn [Aviation] NOTAM closures: ICAO, ICAO, ... — actual airport closures detected, operationally significant
• Removed [Aviation] NOTAM: no closures found — routine no-op, noise

server/worldmonitor/aviation/v1/list-airport-delays.ts (6 calls + 1 dead variable):

• Removed [Aviation] FAA: N alerts — debug trace
• Upgraded to console.warn [Aviation] No AVIATIONSTACK_API key — using simulation — fallback to simulation mode
• Removed [Aviation] AviationStack OK: N real alerts — debug trace
• Removed [Aviation] Intl: N alerts — debug trace
• Upgraded to console.warn [Aviation] NOTAM: N closures applied — operationally significant overlay
• Removed [Aviation] Total: N alerts (N normal) in Nms — debug summary; also removed the now-unused t0 timing variable
• Removed dead code block: if (flights.length > 0) block in fetchSingleAirport that only computed unused cancelled/delayed variables after their log was removed

Rationale

Decision	Criteria
Upgrade to console.warn	Operational condition worth monitoring: missing API keys, fallback behavior, stale data served, actual closures detected, batch completion summaries
Remove entirely	Debug/trace noise: "Fetching from...", "N results parsed", per-item traces, cache hits, routine totals

All existing console.warn and console.error calls were left untouched. No new logging was added. Log prefix format (e.g., [USNI Fleet], [Aviation]) is preserved on all upgraded messages.

Test plan

• Verify TypeScript compilation passes (tsc --noEmit)
• Confirm no remaining console.log calls in server/worldmonitor/
• Verify aviation and military endpoints still function with expected warn-level output

🤖 Generated with Claude Code

[vercel]

@NewCoder3294 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

[koala73]

Thank you - swamped, just got to it, but app got a big architecture update so #1097