Commit Graph

2384 Commits

Author SHA1 Message Date
Zoinkwiz
61fa720012 fix: Use getIdx for potionstorage 2025-08-31 17:05:58 +01:00
Zoinkwiz
1f8650d59d fix: Allow vials to work for PotionStorage 2025-08-31 17:05:10 +01:00
Rasmus Karlsson
14b4313f25 maybe fix potion store? untested 2025-08-31 17:14:06 +02:00
Rasmus Karlsson
1282c1a468 fix: Quest Helper tab bank withdrawing
See 96ddec45d3
2025-08-31 17:13:52 +02:00
Zoinkwiz
5539626281 Polish enahkras lament (#2294)
* refactor: modernize

* fix: Correct Shadow Room locations and IDs

* Updated EL file with .editorconfig working
2025-08-28 19:25:52 +01:00
Zoinkwiz
fde001b830 fix: Corrected ordering for ardy hard's 'craft a death rune' step (#2290) 2025-08-27 12:30:48 +01:00
Zoinkwiz
2fb179db4e fix: hide mourner outfit req for ardy hard if SOTE completed (#2284)
* fix: hide mourner outfit req for ardy hard if SOTE completed

* fix: Hide new key if SOTE done for Ardy hard
2025-08-27 12:30:30 +01:00
Cooper Morris
241eaec29b Migrate varbits and varplayers to use gamevals (#2276)
* test

* Migrate varbits and varplayers to use gamevals

* More varbit gameval replacements

* A few more missed varbits

* Revert a few files already implemented in other PRs

* Add missing varbit/varplayer

* Revert changes to waterfall quest

* nit: remove unused requirements instead

* nit: remove unused item requirements that were part of previously unused requirements

---------

Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>
2025-08-27 12:30:20 +01:00
Pim Peters
b4b481aa1c polish: Creature of Fenkenstrain (#2275) 2025-08-27 10:45:07 +01:00
Miguel Brandão
9f8fa419d2 Clarify 65 fishing is only needed for Tai Bwo Wannai trio as iron if Raw Karambwan given is burned (#2274) 2025-08-27 10:45:00 +01:00
Frosty-J
64afeb620b Shadows of Custodia: Change "all" damage to "most" (#2273) 2025-08-27 10:35:39 +01:00
pajlada
6255f72e6f polish: Murder Mystery (#2270)
* feat(ConditionalStep): allow passing through current step text in sidebar

* refactor: modernize

* test: opt into sidebar test

* minimize start quest dialog

* refactor: gameval

* fix: don't use custom event/varbit checks

we now instead rely on ConditionalStep to be able to pass through the currect text

* fix: sidebar order

* nit: itemsteps

* guide user down for filling pot with flour

* nit: dialog id is not necessary

* fix: updated widget text requirement interface ID

* I _think_ I have to go through this dialog option to finish the quest

When following the quest normally, it would stop at the "talk to the suspect" step.

There is a chance that me talking about the customers again actually solved it, but without varbits it was difficult to
test, hence why the conditional steps I added are so guarded
2025-08-27 10:35:20 +01:00
pajlada
7fab185cb4 test: fix herb run CI (#2288)
* fix: correctly add the hosidius plant step as a substep to the hosidius patch step

* test: make all herb patches reachable

* test: slightly custmo logic for reorderableconditionalstep

we allow the default step (null req) to not have a sidebar step

to be able to do this, we'd essentially need to make reordreable sections also have one section that isn't reorderable
(and that you can't move things below), or a finish step with a static order( whic would need to constantly update since
		it can't allow gaps)
2025-08-27 10:34:56 +01:00
pajlada
f10b5d95e4 polish: Waterfall Quest (#2269)
* refactor: modernize

* test: opt into sidebar step

* remove todo that's already been implemented

* step1: direction

* fix: quest sart dialog highlight

* item req cleanup

* rope requirement when boarding raft

* step to leave home south of Almera

* check pebble in inv or bank instead of golrie conversation

* nit: reword usepebblestep

* equip amulet step

* fix "get glarial's stuff" steps

it didn't check for the correct amulet

* nit: reword west door entering step

* nit: highlight urn when prompted to use it on the chalice

* chore: license
2025-08-25 12:40:07 +01:00
pajlada
b3ab345383 polish: Dwarf Cannon (#2267)
* refactor: modernize

* test: opt into sidebar test

* fix: quest start dialog highlight

* better hammer tooltip / tips

* polish repair railing step

this will properly prompt the user to talk to Captain Lawgof if they need railings

6 -> six

* nit: no need to mention finishing the dialog

we don't prompt the user to continue until it's safe to do so

* nit: comma in location hint

* simplify "return to lawgof with remains" step

* nit: rephrase cave

* section rename

* update cave coords to play better with shortest path

* exit cave step after finding Lollk

* fix/polish/clean up cannon fixing step

* phrasing after fixing multicannon

* Nulodion phrasing

* mould/note names

* last step phrasing

* final reformat

* chore: recover original license

* chore: add myself to license
2025-08-25 12:38:20 +01:00
pajlada
0b04eec2d9 polish: Pirate's Treasure (#2265)
* refactor: modernize

* fix: sidebar steps

* polish rum smuggling steps

some sidebar steps missing,

* nit: key requirement & step order

* kill gardener step

* last reformat
2025-08-25 12:35:36 +01:00
pajlada
ab067d2de0 feat(PanelDetails): static locked panel creator (#2264)
* feat(PanelDetails): diary creator

* nit: rename from `diary` to `lockedPanel`
2025-08-25 12:31:08 +01:00
pajlada
fa5f219319 test(sidebar) support complex quests (#2263)
* test(sidebar): support complex helpers

* chore: remove x marks the spot exclusive
2025-08-25 12:30:51 +01:00
pajlada
9d1e4e300c polish: Daddy's Home (#2242)
* test: opt in Daddy's Home to sidebar check

* refactor: modernize

* refactor: gameval

* Better directions to Yarlo first time

* Consistent directions/direction format

* remove furniture: use a conditional step instead of substeps

No directions when last step was in Yarlo's house

* rebuild furniture: conditional step

* highlight build options in furniture

* reword search crate to include yarlo's full name

* reword operator talk to ensure directions all use dashes

* talk to operator teleport recommendation

* uncapitalize sawmill operator

* check for planks & logs in your bank

this ensures the user isn't told to wrongfully get the waxwood logs from the crates if they already have the logs or the
planks in their bank, and the user is now properly told at the item can be found in their bank

* directions to campbed

* campbed teleport item recommendation

* refactor: move last 2 steps down to the bottom

* highlight build campbed option

* remove direction to Old Man Yarlo since the previous step finished right next to him

* comment unreachable steps
2025-08-25 12:30:21 +01:00
Cooper Morris
c1969b11e4 Migrate sprites, objects, items, interfaces, and npcs to gamevals (#2277)
* Migrate sprites, objects, items, interfaces, and npcs to gamevals

* Lumbridge elite varbit comparison
2025-08-24 16:21:59 +01:00
Zoinkwiz
02b259fa22 Feat reorderable herb run (#2259)
* fix: Allow reordering of sidebar for hidden sections

* feat: Added default true RuneliteRequirement constructor

* fix: Remove unused orderedSteps from ConditionalStep

* feat: Propogate id for steps to substeps and panel steps

* feat: Allow multiple steps with the same id in a ReorderableConditionalStep

* feat: Improved herb run helper

- Allows for reorganising of sidebar steps
- Add Harmony teleport
- Modernised code
- Added barb training check for dibber requirement

* Added herb run to testable list
2025-08-20 13:02:50 +01:00
pajlada
fcc8f94829 feat: add option to always show debug overlay in developer mode (#2254)
essentially just saves me from typing ::questhelperdebug enable every time i restart the client
2025-08-15 11:48:08 +01:00
Zoinkwiz
49dd494b15 chore: Added Tree Gnome Village to list of testable helpers (#2258)
test: fix sidebar quest
2025-08-15 11:47:07 +01:00
pajlada
dc7e90c3f5 polish: Biohazard (#2257)
* test: opt Biohazard into sidebar test

* refactor: modernize

* nit: consolidate getBirdFeed/getBirdFeed2

nit: restructure step2

rework bird cage / bird feed steps

more bird cage cleanup

birdfeed -> bird feed

item step

* space

* Consolidate omart/go to west ardougne step

not worth another step just to change the phrasing to 'return' imo

also accept "Okay, let's do it." which apparently can show up.

* t: make rotten apple pickup step an ItemStep

* space

* Consolidate cupboard step

* add step for exiting headquarters backyard

* phrasing for sarah cupboard step

* kill mourner phrasing

* kilron phrasing

* add Distillator to requirement to the "Return to Elena" steps

* fix: tooltip for coins

* cleanup: medical gown equipped

* buy priest gown set step, just in case

* move chemical step in list

* remove stray notes that are no longer accurate

* add unlock reward

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:46:49 +01:00
pajlada
6f0af2638d polish: Plague City (#2255)
* test: opt into sidebar test

* refactor: modernize

* nit: fix Martha/Ron last name typo

* fix: quest experience reward

2425 mining xp, not 2426

* fix: Edmond NPC ID

* space

* nit: update phrasing when originally talking to Alrena with dwellberries

* varbit gameval

* improve bucket description

* nit: update underground edmond conversation

* nit: highlight both Martha and Ted Rehnison

* nit: remove warrant as requirement from the "Talk to Bravek" step

* nit: "the Bravek" -> "Bravek"

* nit: separate bravek hangover and warrant steps

* nit: phrasing

* add notes

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:44:39 +01:00
pajlada
f84b9119bb polish: Clock Tower (#2253)
* refactor: modernize

* test: opt in Clock Tower to sidebar test

* fix: quest start dialog highlight

* fix: Look for Quest Journal widget, not Quest Journal Overview widget

* nit: update quest start phrasing

* nit: update finish quest phrasing

it used to double the "Talk to Kojo for your reward" in the overlay

* nit: add bucket of water as a requirement to the "enter the basement" steps, ensuring you see it first thing after
placing the blue cog

* fix: add sync step to sidepanel so sidebar steps aren't all collapsed if a user relogs during quest

* feat: add step to get & fill an empty bucket during blue cog step if user doesn't have one

I'm happy to remove this if it feels too handholdy - the overlay text when entering basement for the black cog should be
enough imo

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:37:53 +01:00
Joseph Esmaail
61dc7f42bc polish: Barbarian training mithril dragon requirements (#2252)
* fix: add antifire shield and combat gear to Barbarian Training item requirements

* fix: add Mithril dragon to Barbarian Training combat requirements
2025-08-15 11:36:58 +01:00
David Phillips
feb01907e5 Fremennik Isles: Go to next step after getting Jester outfit (#2249)
Before this change, nothing happens after you get the Jester outfit from the chest. Only after equipping the Jester
outfit does it show the next step, but this is not explained to the player.

The `hasJesterOutfit` `ItemRequirement` now does not require the jester outfit to be equipped, and so now the next steps
will show correctly even when the Jester outfit is in your inventory.
2025-08-15 11:33:19 +01:00
pajlada
980a1e728d polish: Monk's Friend (#2248)
* test: opt in monk's friend to sidebar test

* refactor: modernize

* fix: direction to Monastery

* fix: quest start dialog highlights

* nit: add blanket to "Go up the ladder" step

* nit: "in the monastery"
2025-08-15 11:32:27 +01:00
pajlada
9e3c214756 polish: Tree Gnome Village (#2246)
* refactor: modernize

also add combat gear to general requirement, and food to general recommended

* fix: start quest "Yes." dialog highlight

* montai directions

* refactor: and() instead of new Conditions(LogicType.AND)

* refactor: gameval

* nit: north-west/south-west instead of northwest/southwest

* nit: more flavour text & direction

* add first orb as a requirement to elkoy skip step

* add dialog highlight to elkoy skip step

* fix: dupe message in overlay

* nit: reword orb of protection step

* nit: update name of warlord

* elkoy skip when returning the last time

* fix: add item reward

* refactor: final polish pass

* chore: license
2025-08-15 11:28:23 +01:00
pajlada
567e199c95 polish: Rune Mysteries (#2245)
* test: add rune mysteries to sidebar opt in test

* refactor: modernize

* fix: properly highlight Lumbridge castle south bottom stairs

* nit: update coords for Duke Horacio

Still the same room, just right inside the door so shortest path idnicator looks nicer

* fix: dialog step to climb down stairs

* nit: add airtalisman to "climb down" step

* fix: instruct directions to Sedridor when wanting to accept the package

* Make "get research package" its own step

* fix: ensure user is prompted to get research package from sedridor

* rename talkToAubury -> deliverPackageToAubury

* fix: don't tell the user to leave before they get the notes

* Clean up note delivery step

* fix: you can actually runecraft before this

* Segment into two sections, and rephrasing them to better match quest

* chore: license

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:25:49 +01:00
pajlada
712efb2cb3 polish: Hazeel Cult (#2244)
* test: add hazeel cult to sidebar step test

* refactor: modernize

* chore: add locking step notes

* nit: update directions

* nit: add another dialog highlight step when talking to Clivet

* reformat/reorganize

* Make the "catch step" part of the first valve turning step in case something goes out of sync

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:23:12 +01:00
pajlada
580f659b5e polish: Fight Arena (#2243)
* refactor: modernize

* fix: start quest dialog highlight

* test: opt in fight arena sidebar step

* nit: use .equipped() helper for khazard pieces

* nit: rephrase buy brew step

* fix: highlight step phrase

* reformat

* mark safespot tiles

* more safespots

* dialog step highlight on exiting the door

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:21:11 +01:00
pajlada
0efc16cf06 polish: Gertrude's Cat (#2241)
* test: add Gertrude's Cat to sidebar opt-in steps

* refactor: modernize

* polish quest

* chore: license
2025-08-15 11:16:43 +01:00
pajlada
b6bffe7c8e polish: Romeo & Juliet (#2240)
* refactor: reorganize

* refactor: modernize & reformat

* test: add Romeo & Juliet to sidebar step check

* refactor: potion -> cadavaPotion

* fix: add "Yes." dialog highlight to quest start

* fix: add step to go downstairs when handling letter to romeo

* refactor: and instead of new Conditions

* polish: highlight stairs down when about to finish quest

* final typing polish
2025-08-15 11:15:46 +01:00
pajlada
ee5b837f26 polish: Client of Kourend (#2237)
* refactor: restructure
// members
// Required items
// Recommended items
// Zones
// Miscellaneous requirements (e.g. mid-quest requirements)
// Steps

// ordered logically
setupZones
setupRequirements
setupSteps

loadSteps

// ordered based on order in the sidebar

getGeneralRequirements
getItemRequirements
getItemRecommended

getEnemiesToDefeat

getQuestPointReward
getExperienceRewards
getItemRewards
getUnlockRewards

getPanels

* refactor: modernize

* fix: add missing "Yes." option when starting the quest

* fix: scroll is needed for all steps

* nit: recommend 60 coints for use in minecart network

* fix: only highlight enchanted scroll in combining step

* fix: recommend minecart network travel

I've only done this with text, and no widget highlights

* fix: varbit id gameval

* increase minecart coins

* recommend minecart network for returning step

* chore: add missing unlock reward (ability to use kourend teleport spell)

* test: add client of kourend to opt-in-sidebar list

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:15:37 +01:00
pajlada
4db39435db polish: Children of the Sun (#2231)
* refactor: modernize

We are purposefully not using List.of in the line points since it does not accept null values.

* fix: gameval varbit

* fix: sidebar steps

* random note

* test: opt in children of the sun

* add more notes for post-quest-things
2025-08-15 11:13:14 +01:00
Crito
fb30622d43 Costume needle substitution exceptions (#2233)
* https://oldschool.runescape.wiki/w/Costume_needle

* Update src/main/java/com/questhelper/helpers/quests/elementalworkshopi/ElementalWorkshopI.java

Co-authored-by: pajlada <rasmus.karlsson+github@pajlada.com>

* Update src/main/java/com/questhelper/helpers/quests/spiritsoftheelid/SpiritsOfTheElid.java

Co-authored-by: pajlada <rasmus.karlsson+github@pajlada.com>

* Update src/main/java/com/questhelper/helpers/quests/spiritsoftheelid/SpiritsOfTheElid.java

Co-authored-by: pajlada <rasmus.karlsson+github@pajlada.com>

---------

Co-authored-by: pajlada <rasmus.karlsson+github@pajlada.com>
Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-15 11:10:12 +01:00
Dr Tom Goodman
2cc808a837 Change equip to requiredToBeEquipped (#2212)
* Fix build.gradle to reference sourceCompatibility properly

* Make 'equip required' property/access more descriptive

* Use custom getter so I can use the words I want

* shouldEquip -> mustBeEquipped

* Update ItemRequirements.java

* Update ItemRequirements.java
2025-08-15 11:07:30 +01:00
Zoinkwiz
e41a6f9029 polish: X Marks the Spot (#2226) 2025-08-15 11:00:26 +01:00
Zoinkwiz
666878df7e Merge pull request #2220 from pajlada/polish/misthalin-mystery
polish: Misthalin Mystery
2025-08-15 10:55:44 +01:00
Rasmus Karlsson
7942e8ef17 fix: don't test helpers that aren't basic helpers
some initialization is wonky and has to be done differently
2025-08-09 20:19:25 +02:00
Rasmus Karlsson
0eeec38783 chore: add myself to copyright 2025-08-09 16:26:08 +02:00
Rasmus Karlsson
b8bf8cfa02 fix: puzzlewrapper third & fourth puzzle 2025-08-09 16:26:03 +02:00
Rasmus Karlsson
c2c22b5d74 test: add misthalin mystery to "must have sidestep" list 2025-08-09 16:21:02 +02:00
Rasmus Karlsson
05e2edc976 refactor: reformat/reorganize 2025-08-09 16:21:02 +02:00
pajlada
b8ef742408 fix(The Final Dawn): step typo (#2234) 2025-08-09 14:04:24 +01:00
pajlada
d22285eb0c polish: Sheep Shearer (#2219)
* sheep shearer refactor: modernize

* refactor: reformat

* Define lumbridge south staircase in QHObjectID

* feat: add function to create widget highlight multiskill-by-id

* polish(Sheep Shearer): Gameval, use new convenience-functions, add final step

Previously, if the user turned in 19 balls of wool, the helper would close.

---------

Co-authored-by: Zoinkwiz <zoinkwiz@hotmail.co.uk>
2025-08-09 14:03:57 +01:00
Alexa Little
f985e5b45e feat(treeRun): Add the Nemus Retreat tree patch to the tree run quest helper (#2232)
* Add Nemus Retreat Tree Patch to TreeRun

* add states, add steps

* Add region, add missing step, fix coords

* Run runelite formatter

* Revert "Run runelite formatter"

This reverts commit 20063ce22c.
2025-08-09 14:03:16 +01:00
pajlada
df6b74e41c polish: Imp Catcher (#2230)
* refactor: modernize

* nit: merge "Climb the staircase again" step

* test: opt in imp catcher
2025-08-09 14:02:40 +01:00