Skip to content

feat: Expand pay-for-protection to allotment and hops patches#81

Open
filipecouto wants to merge 4 commits intoSpeaax:masterfrom
filipecouto:feature/allotment-hops-protection
Open

feat: Expand pay-for-protection to allotment and hops patches#81
filipecouto wants to merge 4 commits intoSpeaax:masterfrom
filipecouto:feature/allotment-hops-protection

Conversation

@filipecouto
Copy link
Copy Markdown

@filipecouto filipecouto commented Apr 16, 2026

Note: Not all patch locations have been tested in-game due to limitations of my current OSRS account which is pretty new. Farmer names and dialog detection are based on wiki data and may need adjustments for untested locations.

Summary

Expands pay-for-protection support beyond tree/fruit tree patches to cover herb, flower, allotment, and hops patches.

Pay-for-protection: Herb patches

  • Highlights nearby allotment farmers when a herb patch is growing
  • Same dialog detection and protection flow as tree/fruit tree patches

Pay-for-protection: Flower patches

  • Highlights nearby allotment farmers when a flower patch is growing
  • Same dialog detection and protection flow as herb patches

Pay-for-protection: Allotment patches

  • Highlights nearby allotment farmers (Elstan, Dantaera, Kragen, Marisi, Lyra, Alan, Harminia) when a patch is growing
  • Detects "I'm already looking after that patch for you" NPC dialog to confirm protection
  • Tracks dialog consumption to prevent the same dialog satisfying both north and south allotment patches

Pay-for-protection: Hops patches

  • Highlights nearby hops farmers (Vasquen, Rhonen, Selena, Brother Althric, Ercos) when a patch is growing
  • Same dialog detection and protection flow as allotment patches

Other changes

  • Added Fairy Ring teleport option for Civitas illa Fortis
  • Added Castle Wars Minigame teleport option for Yanille hops
  • Fixed noted item detection by canonicalizing item IDs in inventory count

Test plan

  • Run a herb run with pay-for-protection enabled, verify farmer highlighting
  • Run a flower run with pay-for-protection enabled, verify farmer highlighting
  • Run an allotment run with pay-for-protection enabled, verify farmer highlighting and dialog detection at multiple locations
  • Run a hops run with pay-for-protection enabled, verify farmer highlighting
  • Verify north-to-south allotment transition works when both patches are already protected (dialog not consumed twice)

Adds pay-for-protection support for allotment patches (Elstan, Dantaera,
Kragen, Marisi, Lyra, Alan, Harminia) and hops patches (Vasquen, Rhonen,
Selena, Brother Althric, Ercos). Includes NPC dialog detection for
"already looking after that patch" message, noted item canonicalization
fix, and Civitas/Yanille teleport updates.
The Civitas farming area spans multiple regions. Walking from the AJP
fairy ring (region 6447) through region 6448 to the patches (region 6192)
caused the run to end because 6448 was not recognized as Civitas.
@filipecouto
Copy link
Copy Markdown
Author

Hi, I recently started playing OSRS again and noticed that the plugin wasn't supporting pay for protection in hops / allotments, so I added those and have been testing locally with my severely underleveled account.

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