Skip to content

feat(community-decks): read YGOPRODeck decks, lflists and card alternates - #149

Merged
diangogav merged 1 commit into
mainfrom
feat/community-decks-2-adapters
Oct 29, 2026
Merged

diangogav merged 1 commit into
mainfrom
feat/community-decks-2-adapters

Conversation

@diangogav

@diangogav diangogav commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • YGOPRODeck adapter for getDecks.php per category, paging 20 at a time until an empty/short page or a page cap, ≤1 request/second with an identifying User-Agent.
  • Format rules adapter: Edison and Genesys lists from evolution-assets, TCG from the newest … TCG section of the moecube base lflist; card alternates from evolution-assets cdb/alternates.json (feat(cdb): publish the original → alternate card id map evolution-assets#41).
  • Fixture-based tests, no network. Nothing wired yet.

Chain: community decks (2/4)

Depends on #148 (this PR targets its branch; retarget to main once it merges).

`main`
  └─ 1. domain rules #148
    └─ 2. adapters #149 📍
      └─ 3. storage #150
        └─ 4. endpoints + worker #151

Checks

bun test, bun run lint, bun run build pass.

Feature: a Community tab in the deck explorer with legal decklists synced from YGOPRODeck. Only evolution-api talks to YGOPRODeck; it normalizes ids, validates against our lists and serves a clean list.

@diangogav
diangogav changed the base branch from feat/community-decks-1-domain to main October 29, 2026 13:00
…ates

Page through YGOPRODeck categories at most one request per second with an
identifying User-Agent, parse the TCG, Genesys and Edison lflist sections the
client uses, and read the evolution-assets alternates map.
@diangogav
diangogav force-pushed the feat/community-decks-2-adapters branch from 870cb12 to 7376e7e Compare October 29, 2026 13:00
@diangogav
diangogav merged commit 0aa6e60 into main Oct 29, 2026
1 check passed
@diangogav
diangogav deleted the feat/community-decks-2-adapters branch October 29, 2026 13:01
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