Repository navigation
feat(tappable): add pokemon tappables and Golbat API support - #1277
unseenmagik wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
@codex review |
|
Codex Review: Something went wrong. Try again later by commenting “@codex review”. ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Mind explaining yourself? |
|
Not going to merge this. |
Closes #1276. Follows the candy rewards work (9b78da7).
What
Pokémon tappables in the tappable filters
Encounter tappables (e.g. the costume Gengar tappable) were excluded by
whereNull('pokemon_id'). They now show on the tappable layer and in its filter list.94), matching what Golbat stores (tappable.pokemon_idis the species) and what its scan filter can match.pokemonrow that shares the tappable's encounter id. This is a correlated subquery on the primary key, so it is still a single query.defaultFilters.tappables.pokemondefault (env:DEFAULT_FILTERS_TAPPABLES_POKEMON).Golbat tappable API (#1276)
When a source has an
endpointand Golbat advertisesfeatures.tappable_scanin/api/status:Tappable.getAllPOST /api/tappable/scanTappable.getByIdGET /api/tappable/id/{id}Tappable.getAvailableGET /api/tappable/availableareaRestrictionsDenyAll+filterRTree), the manual-id deep link andqueryLimits.tappablesbehave as on the SQL path./api/tappable/availablehas no reward amount, which mega energy keys need (m<id>-<amount>[-<evo>]). When mega energy is listed, one extra scan over the whole map, limited to mega energy tappables, gets the amounts and temp evolutions.[TAPPABLES]logger tag.Screenshots
Notes
tappablestays on the Golbat DB entry. Listing it on an endpoint-only entry gives no SQL fallback on older Golbat builds. To use the API with fallback, addendpointandsecretto the Golbat DB entry.PokemonDisplaywhen it decodes the encounter, so exposing it there would be a small follow-up.max_tappable_resultscap applies before the local temp-evolution check.Testing
yarn lint: passesyarn prettier: changed files are clean.node --test server/test/*.test.js: 156/156 pass. Added atappablescase for Pokémon keys inclientFilterSelection.test.js.getAllandgetAvailable, both with and without the reward columns.🤖 Generated with Claude Code