NixReaperandClaude Sonnet 5
892f78016c
CI / lint (push) Canceled after 0s
CI / typecheck (push) Canceled after 0s
CI / unit-tests (push) Canceled after 0s
CI / integration-tests (push) Canceled after 0s
CI / container-build (push) Canceled after 0s
CI / dependency-audit (push) Canceled after 0s
Collections were bare stubs (name only) lazily created the moment a
movie's belongs_to_collection reference was seen. Adds a real
ingest_collection pipeline: TMDb's /collection/{id} for overview and
poster/backdrop artwork, plus ordered CollectionMovie membership --
linking only movies already ingested into Aurora, the same
never-fabricate-a-stub rule movie recommendations already follow.
get_or_create_collection now refreshes name/overview on an existing
stub instead of only creating, guarded so a bare movie-stub call
(overview=None) never clobbers real data a full ingestion already
filled in.
New POST /api/v1/admin/ingest/collection endpoint and
ingest_collection_task; GET /api/v1/collections/{id} needed no changes
since its read path (collection_service.py) already assembled the full
detail shape.
Co-Authored-By: Claude Sonnet 5 <[email protected]>
64 lines
2.3 KiB
Python
64 lines
2.3 KiB
Python
"""Provider abstraction (section 13).
|
|
|
|
A `MetadataProvider` fetches **raw**, unnormalized data from one external
|
|
source. It never writes to the database and never returns Aurora entities —
|
|
that's the normalizer/matcher/ingestion pipeline's job (see
|
|
`app/services/metadata/` and `app/services/ingestion/`). This keeps adapters
|
|
replaceable: swapping TMDb for another provider means writing a new class
|
|
here, not touching ingestion logic.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from abc import ABC, abstractmethod
|
|
from typing import Any
|
|
|
|
|
|
class ProviderError(Exception):
|
|
"""Raised for any provider-side failure (auth, not-found, rate limit,
|
|
network). Ingestion jobs catch this and record it, they don't crash."""
|
|
|
|
|
|
class ProviderNotConfiguredError(ProviderError):
|
|
"""Raised when a required credential (e.g. TMDB_API_KEY) is missing."""
|
|
|
|
|
|
class ProviderNotFoundError(ProviderError):
|
|
"""Raised when the provider has no record for the given external ID."""
|
|
|
|
|
|
class MetadataProvider(ABC):
|
|
slug: str
|
|
name: str
|
|
|
|
@abstractmethod
|
|
async def get_movie(self, external_id: str) -> dict[str, Any]:
|
|
"""Fetch raw movie data, including credits/images/videos/keywords/
|
|
alternate titles/external IDs/release dates where the provider
|
|
supports fetching them in one call."""
|
|
|
|
@abstractmethod
|
|
async def get_series(self, external_id: str) -> dict[str, Any]:
|
|
"""Fetch raw series data (without season/episode bodies)."""
|
|
|
|
@abstractmethod
|
|
async def get_season(self, series_external_id: str, season_number: int) -> dict[str, Any]:
|
|
"""Fetch raw season data, including its episodes."""
|
|
|
|
@abstractmethod
|
|
async def get_person(self, external_id: str) -> dict[str, Any]:
|
|
"""Fetch raw person data, including external IDs and images."""
|
|
|
|
@abstractmethod
|
|
async def get_collection(self, external_id: str) -> dict[str, Any]:
|
|
"""Fetch raw collection/franchise data: name, overview, poster/
|
|
backdrop, and its member movies (TMDb's "parts")."""
|
|
|
|
@abstractmethod
|
|
async def search_movie(self, query: str) -> list[dict[str, Any]]:
|
|
"""Search this provider's catalog for movies matching `query`. Used
|
|
to seed ingestion, not exposed directly to API clients."""
|
|
|
|
async def aclose(self) -> None:
|
|
"""Release any held resources (HTTP clients, ...). Optional."""
|