Skip to content

Introduce provider-aware chart processing foundation - #38

Open
bkielczewski wants to merge 10 commits into
iccowan:mainfrom
bkielczewski:feature/source-aware-chart-processors
Open

Introduce provider-aware chart processing foundation#38
bkielczewski wants to merge 10 commits into
iccowan:mainfrom
bkielczewski:feature/source-aware-chart-processors

Conversation

@bkielczewski

Copy link
Copy Markdown

Summary

This PR introduces a provider-aware foundation for chart processing while preserving the existing FAA behavior and public API shape. faa_tpp remains the only implemented provider in this PR. The purpose here is to make the current ingestion/retrieval pipeline extensible enough to support additional providers, such as UK AIP, in follow-up work.

Changes

  • Added internal provider routing across AIRAC state, repositories, and SNS messaging
  • Made airport and AIRAC storage keys provider-aware
  • Introduced a provider registry in the chart processor
  • Wrapped current FAA behavior in a faa_tpp provider
  • Let providers define expected processing jobs
  • Kept chart API routes unchanged; collectors now resolve across registered providers internally
  • Removed the implicit USA default from AirportData; FAA processing now sets country explicitly

Compatibility

  • Existing API routes remain unchanged
  • Existing FAA behavior is preserved
  • Legacy FAA storage keys remain readable

@bkielczewski
bkielczewski requested a review from iccowan as a code owner April 30, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant