Repository navigation
fix(log): resolve later anime cours before provider fan-out - #348
Conversation
|
Review findings from GPT-6, posted on behalf of Gian. Correctness review found that an unresolved later cour still allowed Simkl to receive a first-cour show identity. Fixed the planner to produce a reasoned skip, including when a resolved cour lacks a MAL id. Added owner-boundary coverage proving Simkl delivery never runs for either miss; both tests fail without the guard. The updated correctness review approved, and the independent ponytail complexity review found no issues. The final change resolves related AniList cours through matching ani.zip TVDB identity and episode mappings, shares the existing reverse mapper, and keeps Trakt/Serializd canonical episode numbers. All 1,560 tests, lint, typecheck, and diff checks pass. Live public provider reads confirm the Cyberpunk S2 identity and S2E3 → entry 3 mapping. No authenticated provider writes or device UI were tested. Layout-aware reverse mapping for other tracker season structures remains a follow-up. These JS changes hot reload without a native rebuild. |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
shinobu | 24988fd | Oct 28 2026, 04:06 AM |
Logging a later anime season from a TMDB or Trakt show could miss AniList and send Simkl a write against the first cour. Resolve the matching AniList cour through related entries and ani.zip before fan-out, retaining canonical episode numbers for Trakt and Serializd. Simkl skips with a reason when the cour or its MAL identity cannot be established.
Validated with 1,560 passing tests, lint, typecheck, and diff checks. The success regression fails against the old planner; both safety regressions fail without the skip guard. Live public AniList and ani.zip reads confirm Cyberpunk S2 maps to AniList 195539 / MAL 61990 and entry-relative episode 3.
Correctness and ponytail complexity reviews completed. No device UI or authenticated provider writes were tested. JS changes hot reload; no native rebuild is needed. Layout-aware reverse mapping for other tracker season structures remains a follow-up.