diff --git a/_bmad-output/implementation-artifacts/1-4-location-resolution-icao-and-address.md b/_bmad-output/implementation-artifacts/1-4-location-resolution-icao-and-address.md index 7936e6f..91d4cac 100644 --- a/_bmad-output/implementation-artifacts/1-4-location-resolution-icao-and-address.md +++ b/_bmad-output/implementation-artifacts/1-4-location-resolution-icao-and-address.md @@ -1,6 +1,6 @@ # Story 1.4: Location Resolution (ICAO & Address) -Status: review +Status: done ## Story diff --git a/_bmad-output/implementation-artifacts/deferred-work.md b/_bmad-output/implementation-artifacts/deferred-work.md index dba47f4..3808828 100644 --- a/_bmad-output/implementation-artifacts/deferred-work.md +++ b/_bmad-output/implementation-artifacts/deferred-work.md @@ -40,3 +40,17 @@ Description: Actual captive portal detection behaviour (iOS/Android/Windows trig Story: `1-3-wifi-hotspot-and-captive-portal-form` Category: Infrastructure/environment Description: `provision.py` provisioning loop currently exits after one iteration (placeholder `provisioned = True`) — full sequence wired in Story 1.5. + +--- + +## Story 1.4: Location Resolution (ICAO & Address) + +### [1-4] Nominatim geocoding runtime verification +Story: `1-4-location-resolution-icao-and-address` +Category: Runtime verification +Description: Nominatim geocoding verified in tests with mocks only; real geocoding requires internet access and can only be confirmed on device at provisioning time. No automated test covers live HTTP to Nominatim. + +### [1-4] ICAO heuristic false-positive risk +Story: `1-4-location-resolution-icao-and-address` +Category: Technical debt +Description: ICAO heuristic (`len(query) == 4 and query.isalpha()`) may misclassify 4-letter words (e.g. "BATH", "YORK") as ICAO codes, causing them to be looked up in `airports.csv` before falling back to Nominatim. Acceptable for MVP given the provisioning context, but noted for future hardening (e.g. validate against a known ICAO prefix list). diff --git a/_bmad-output/implementation-artifacts/sprint-status.yaml b/_bmad-output/implementation-artifacts/sprint-status.yaml index 4bbe605..92e373d 100644 --- a/_bmad-output/implementation-artifacts/sprint-status.yaml +++ b/_bmad-output/implementation-artifacts/sprint-status.yaml @@ -35,7 +35,7 @@ # - Dev moves story to 'review', then runs code-review (fresh context, different LLM recommended) generated: 2026-04-22 -last_updated: 2026-04-22 +last_updated: 2026-04-22 # 1-4 marked done after Phase 5 review project: planeMapper project_key: NOKEY tracking_system: file-system @@ -47,7 +47,7 @@ development_status: 1-1-project-scaffold-and-verified-entry-points: done 1-2-configuration-read-write-wipe: done 1-3-wifi-hotspot-and-captive-portal-form: done - 1-4-location-resolution-icao-and-address: ready-for-dev + 1-4-location-resolution-icao-and-address: done 1-5-provisioning-execution-tile-download-cache-validation-and-wifi-kill: backlog epic-1-retrospective: optional