Skip to content

fix: add swarm conditions to the HGSS S.S. Anne dock - #1723

Open
serabi wants to merge 1 commit into
PokeAPI:masterfrom
serabi:fix/hgss-vermilion-swarms
Open

serabi wants to merge 1 commit into
PokeAPI:masterfrom
serabi:fix/hgss-vermilion-swarms

Conversation

@serabi

@serabi serabi commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

Change description

The goal of this PR is to add swarm conditions to the HeartGold and SoulSilver encounters at the S.S. Anne dock (location-area 349). The dock has the swarm slots, but nothing in the API currently marks them as swarm-only. Because of that, surfing adds up to 160%, and the extra Magikarp fishing slots mentioned in the issue show up all the time.

For each version, this adds swarm-yes to the swarm slot and swarm-no to the regular slot it replaces:

  • Surfing slot 1: Wingull (swarm) and Tentacool
  • Good Rod slots 3 and 4: Magikarp (swarm) and Chinchou or Shellder
  • Super Rod slots 1-5: Magikarp (swarm) and the regular species

Old Rod slot 3 and Good Rod slot 1 are already Magikarp, so the swarm doesn't change them and they stay as they are.

The main Vermilion City area (282) already tags its Wingull surfing swarm the same way.

Closes #1110.

References

Testing

Built the local database before and after the change and checked the S.S. Anne dock location-area endpoint.

Surfing and each rod now total 100% both with and without a swarm, in both versions.

All 69 tests passed, along with pre-commit, lint, formatting and type checks.

AI coding assistance disclosure

I used AI assistance from Claude Code (Claude Opus 5.5) to investigate the issue, compare PokeAPI's CSV records with HGSS's disassembled encounter data and code, and apply the fix. I then reviewed all of the code manually. All decisions made - and the PR text - are my own.

Contributor check list

  • I have written a description of the contribution and explained its motivation.
  • I have written tests for my code changes (if applicable).
  • I have read and understood the AI Assisted Contribution guidelines.
  • I will own this change in production, and I am prepared to fix any bugs caused by my code change.

@serabi
serabi marked this pull request as ready for review November 14, 2026 20:15
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.

HGSS S.S. Anne dock missing swarm conditions

1 participant