Skip to content

feat(postcodes/PH): bulk-import 1,391 Philippines postcodes via PHLPost (#1039)#1455

Merged
dr5hn merged 1 commit into
masterfrom
feat/postcodes-philippines
Apr 27, 2026
Merged

feat(postcodes/PH): bulk-import 1,391 Philippines postcodes via PHLPost (#1039)#1455
dr5hn merged 1 commit into
masterfrom
feat/postcodes-philippines

Conversation

@dr5hn
Copy link
Copy Markdown
Owner

@dr5hn dr5hn commented Apr 27, 2026

Summary

Adds 1,391 Philippines postcodes spanning all 62 source provinces of
PHLPost's catalogue, redistributed via the
jayson-panganiban/phzipcodes Python package.

  • Coverage: 100% province resolution.
  • Granularity: city/municipality level — one record per
    (zip code, city) pair.

Source & licence

How it works

  • Walks Region -> Province -> City -> [zip codes] tree and emits
    one row per (code, city) pair.
  • Province name match is case-insensitive against states.json.
  • 5-entry STATE_ALIASES bridge handles parenthetical-suffix
    variants (Camarines Sur (Camsur), Davao De Oro (Formerly
    Compostela Valley), Cotabato (North)) and the Metro Manila ->
    National Capital Region exonym.

Validation

  • python3 -m py_compile clean.
  • 100% of 1,391 codes match country.postal_code_regex (^(\d{4})$).
  • 100% of records resolve a valid state_id whose country_id == 174
    and whose iso2 matches state_code.
  • No auto-managed fields (id, created_at, updated_at, flag).

Test plan

  • Importer compiles and runs on a clean checkout.
  • Cross-reference validator passes (regex + FK + state_code agreement).
  • Idempotent merge verified.
  • CI pipeline green.

Refs #1039.

🤖 Generated with Claude Code

…st (#1039)

Source: PHLPost catalogue redistributed via the
jayson-panganiban/phzipcodes Python package. All 62 source provinces
resolve at 100% via case-insensitive name match against states.json,
with a 5-entry STATE_ALIASES bridge for parenthetical-suffix variants
(Camarines Sur (Camsur), Davao De Oro (Formerly Compostela Valley),
Cotabato (North)) and the Metro Manila / Western Samar exonyms.

Refs #1039.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 27, 2026 13:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

CSC Validation Report

PR Format

  • ✅ Description provided
  • ✅ Data source linked
  • ✅ Issue linked (recommended for data changes)
  • ✅ Justification / context provided

Labels applied: data:postcodes, large-contribution

⚠️ Large Contribution

This PR contains 1391 records. Large contributions require manual review.

Schema Validation (1391 records)

✅ All records passed validation

Cross-Reference Validation

✅ 2782 reference(s) verified

Source URL Verification

✅ 2 source URL(s) accessible


All checks passed | Status: Ready for review

@dosubot dosubot Bot added the enhancement New feature or request label Apr 27, 2026
@dr5hn dr5hn merged commit d146c4a into master Apr 27, 2026
1 check passed
@dr5hn dr5hn deleted the feat/postcodes-philippines branch April 27, 2026 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:postcodes enhancement New feature or request large-contribution ready-for-review size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants