Freegle / Iznik / 3826

70%

Build Type

push

circleci

Committed by invalid-email-address

Commit Message

fix: sync all locations in polygon scope before postcode remap

When a location is created/updated, the remap task only synced the single
location_id to PostgreSQL. But that ID is the location being edited, not
necessarily the one whose polygon changed. Newly created locations (like
Millhouses) weren't in PostGIS yet, so the KNN query used stale data and
mapped postcodes to the wrong area.

Now syncs all locations intersecting the polygon scope before running KNN,
matching V1's behavior where setGeometry() synced inline before remapping.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

13100 of 20081 branches covered (65.24%)

Branch coverage included in aggregate %.

16 of 18 new or added lines in 1 file covered. (88.89%)

12 existing lines in 4 files now uncovered.

92502 of 128817 relevant lines covered (71.81%)

17.29 hits per line