fix(timezones): correct 72 city rows with foreign IANA zones (#1085 follow-up)

Sweeps the residual scope from #1085 — city rows whose timezone field
points at a neighbouring country's zone instead of the correct local zone.

  VN  63  Asia/Bangkok    -> Asia/Ho_Chi_Minh
  PS   6  Asia/Jerusalem  -> Asia/Hebron (West Bank governates)
  UZ   2  Asia/Bishkek    -> Asia/Tashkent (Fergana Region)
  ID   1  Asia/Ho_Chi_Minh -> Asia/Pontianak (Sambas, KB)

TF/Tromelin (disputed admin) and FR/Nouméa (correct for state NC) excluded.
Idempotent fix script under bin/scripts/fixes/. Summary in .github/fixes-docs/.

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

Author: dr5hn

.github/fixes-docs/FIX_1085_FOREIGN_TIMEZONES_SUMMARY.md

@@ -0,0 +1,64 @@
+# Fix Summary: Foreign-zone city timezone cleanup (#1085 follow-up)
+
+Closes the residual scope from #1085. The original issue called out a handful
+of countries (ES, RO) where city timezone fields pointed at the wrong IANA zone.
+Master is clean for those. This PR sweeps every remaining city row whose
+timezone field belongs to a neighbouring country.
+
+## Scope
+
+72 rows across 4 country files:
+
+| File | Wrong zone | Correct zone | Rows |
+|------|------------|--------------|-----:|
+| `contributions/cities/VN.json` | `Asia/Bangkok` | `Asia/Ho_Chi_Minh` | 63 |
+| `contributions/cities/PS.json` | `Asia/Jerusalem` | `Asia/Hebron` | 6 |
+| `contributions/cities/UZ.json` | `Asia/Bishkek` | `Asia/Tashkent` | 2 |
+| `contributions/cities/ID.json` | `Asia/Ho_Chi_Minh` | `Asia/Pontianak` | 1 |
+
+## Per-file rationale
+
+- **VN (63 rows):** All 63 sit in Vietnamese provinces (state codes spanning
+  northern, central, and southern VN). Vietnam observes ICT (UTC+7) in a single
+  IANA zone: `Asia/Ho_Chi_Minh`. `Asia/Bangkok` is Thailand's zone — never
+  correct for a Vietnamese city.
+
+- **PS (6 rows):** All 6 sit in West Bank governates — Bethlehem (BTH),
+  Hebron (HBN), Jerusalem governate (JEM), Salfit (SLT). The IANA mapping for
+  the Palestinian territories is geographic: `Asia/Gaza` for Gaza Strip,
+  `Asia/Hebron` for the West Bank. None of the 6 are in the Gaza Strip,
+  so all map to `Asia/Hebron`. `Asia/Jerusalem` is Israel's zone.
+
+- **UZ (2 rows):** Both are in Fergana Region (state_code `FA`), eastern
+  Uzbekistan. Uzbekistan observes UZT (UTC+5) under `Asia/Tashkent` (the
+  most populous IANA zone for the country) or `Asia/Samarkand`. `Asia/Bishkek`
+  is Kyrgyzstan's zone — never correct for a Uzbek city.
+
+- **ID (1 row):** Sambas (West Kalimantan, state_code `KB`) observes WIB
+  (UTC+7) under `Asia/Pontianak`. `Asia/Ho_Chi_Minh` is Vietnam's zone.
+
+## Out of scope
+
+- **TF (Tromelin Island):** Currently tagged `Indian/Reunion`. Tromelin is
+  administered by France as part of the Scattered Islands (TAAF) but is
+  disputed with Mauritius. The current zone reflects de-facto administration —
+  left untouched.
+- **FR (Nouméa, id 160039):** Tagged `Pacific/Noumea`. State_code is `NC`
+  (New Caledonia, an FR overseas collectivity). This is correct, not a bug —
+  excluded from the sweep.
+
+## Verification
+
+```
+$ python3 bin/scripts/fixes/foreign_timezone_fixes.py
+Total rows fixed: 72
+$ python3 bin/scripts/fixes/foreign_timezone_fixes.py
+Total rows fixed: 0   # idempotent
+```
+
+After-counts: 0 VN rows tagged `Asia/Bangkok`, 0 PS rows tagged
+`Asia/Jerusalem`, 0 UZ rows tagged `Asia/Bishkek`, 0 ID rows tagged
+`Asia/Ho_Chi_Minh`.
+
+Schema, cross-reference, coordinate-bounds, and duplicate validators all
+pass — no FK or geometry changes, only the `timezone` text field.

bin/scripts/fixes/foreign_timezone_fixes.py

@@ -0,0 +1,72 @@
+#!/usr/bin/env python3
+"""Fix 72 city rows tagged with foreign IANA timezones.
+
+Issue #1085 close-out follow-up: VN/PS/UZ/ID rows whose timezone field
+points to a neighbouring country's zone instead of the correct local zone.
+
+Idempotent. Safe to re-run.
+"""
+
+import json
+import sys
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parents[3] / "contributions" / "cities"
+
+PLAN = [
+    {
+        "file": "VN.json",
+        "rule": "Asia/Bangkok -> Asia/Ho_Chi_Minh",
+        "match": lambda r: r.get("timezone") == "Asia/Bangkok",
+        "set_tz": "Asia/Ho_Chi_Minh",
+    },
+    {
+        "file": "PS.json",
+        "rule": "Asia/Jerusalem -> Asia/Hebron (all 6 rows are West Bank governates)",
+        "match": lambda r: r.get("timezone") == "Asia/Jerusalem",
+        "set_tz": "Asia/Hebron",
+    },
+    {
+        "file": "UZ.json",
+        "rule": "Asia/Bishkek -> Asia/Tashkent (Fergana Region rows)",
+        "match": lambda r: r.get("timezone") == "Asia/Bishkek",
+        "set_tz": "Asia/Tashkent",
+    },
+    {
+        "file": "ID.json",
+        "rule": "Asia/Ho_Chi_Minh -> Asia/Pontianak (Sambas, West Kalimantan)",
+        "match": lambda r: r.get("timezone") == "Asia/Ho_Chi_Minh",
+        "set_tz": "Asia/Pontianak",
+    },
+]
+
+
+def apply(plan):
+    path = ROOT / plan["file"]
+    if not path.exists():
+        sys.exit(f"missing: {path}")
+    rows = json.loads(path.read_text())
+    changed = 0
+    for r in rows:
+        if plan["match"](r) and r.get("timezone") != plan["set_tz"]:
+            r["timezone"] = plan["set_tz"]
+            changed += 1
+    path.write_text(json.dumps(rows, indent=2, ensure_ascii=False) + "\n")
+    return changed, len(rows)
+
+
+def main():
+    print("Foreign-timezone cleanup (issue #1085 follow-up)")
+    print("=" * 60)
+    total = 0
+    for plan in PLAN:
+        changed, n = apply(plan)
+        total += changed
+        print(f"  {plan['file']:>10}  {plan['rule']}")
+        print(f"  {'':>10}  changed {changed} of {n} rows")
+    print("=" * 60)
+    print(f"Total rows fixed: {total}")
+
+
+if __name__ == "__main__":
+    main()

contributions/cities/ID.json

@@ -32321,7 +32321,7 @@
     "longitude": "109.35000000",
     "native": "Sambas",
     "population": 57295,
-    "timezone": "Asia/Ho_Chi_Minh",
+    "timezone": "Asia/Pontianak",
     "translations": {
       "br": "Sambas",
       "ko": "삼바스",
@@ -32758,4 +32758,4 @@
     "flag": 1,
     "wikiDataId": "Q10116"
   }
-]
+]

contributions/cities/PS.json

@@ -13,7 +13,7 @@
     "longitude": "35.11743230",
     "native": "بتير",
     "population": 56746,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "Battir",
       "ko": "바티르",
@@ -300,7 +300,7 @@
     "longitude": "35.15014530",
     "native": "الخضر",
     "population": 10088,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "al-Khader",
       "ko": "알-카데르",
@@ -341,7 +341,7 @@
     "longitude": "35.09976230",
     "native": "نحالين",
     "population": 1021,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "Nahalin",
       "ko": "나할린",
@@ -1202,7 +1202,7 @@
     "longitude": "35.12317630",
     "native": "سعير",
     "population": 3151,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "Sa'ir",
       "ko": "사이르",
@@ -2391,7 +2391,7 @@
     "longitude": "35.24263130",
     "native": "حزما",
     "population": 8194,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "Hizma",
       "ko": "히즈마",
@@ -4605,7 +4605,7 @@
     "longitude": "35.06050810",
     "native": "كفر الديك",
     "population": 4280,
-    "timezone": "Asia/Jerusalem",
+    "timezone": "Asia/Hebron",
     "translations": {
       "br": "Kafr ad-Dik",
       "ko": "카프르 아드딕",
@@ -5411,4 +5411,4 @@
     "flag": 1,
     "wikiDataId": "Q985531"
   }
-]
+]

contributions/cities/UZ.json

@@ -3908,7 +3908,7 @@
     "longitude": "71.98026000",
     "native": "Quvasoy",
     "population": 96900,
-    "timezone": "Asia/Bishkek",
+    "timezone": "Asia/Tashkent",
     "translations": {
       "br": "Quvasoy",
       "ko": "쿠바소이",
@@ -4359,7 +4359,7 @@
     "longitude": "71.80512000",
     "native": "Shohimardon",
     "population": 5100,
-    "timezone": "Asia/Bishkek",
+    "timezone": "Asia/Tashkent",
     "translations": {
       "br": "Shohimardon",
       "ko": "쇼히마르돈",
@@ -5534,4 +5534,4 @@
     "flag": 1,
     "wikiDataId": "Q32059593"
   }
-]
+]