Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(content): add huntmode to thieves in monkfriend quest #1438

Merged
merged 2 commits into from
Mar 11, 2025

Conversation

MattGould1
Copy link
Contributor

No description provided.

@@ -5302,6 +5302,8 @@ param=attack_anim,human_sword_stab
param=defend_anim,human_sword_defend2
param=attack_sound,stabsword_stab
param=retreat,8
huntmode=cowardly
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -0,0 +1,9 @@
[debugproc,setup_monks_friend]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are these welcome or is it just bloat?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

debugprocs are welcome if they make sense in some way to have, either way nobody really cares.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These "setup account to start a quest" debug procs could be useful, but should be more standardized maybe all in one place, or even added to the ::~quest cheat as a 3rd option (complete, start, reset)

Could be a task for someone to tackle later though

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay cool, I'll keep this in mind as I play test other quests

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These "setup account to start a quest" debug procs could be useful, but should be more standardized maybe all in one place, or even added to the ::~quest cheat as a 3rd option (complete, start, reset)

Could be a task for someone to tackle later though

If we'd like to continue what was added to d5cd8eb, I believe the quest.enum file would need be updated, and then we can add it to the queue.

@@ -5302,6 +5302,8 @@ param=attack_anim,human_sword_stab
param=defend_anim,human_sword_defend2
param=attack_sound,stabsword_stab
param=retreat,8
huntmode=cowardly
huntrange=2
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't really sure about hunt ranges, is there any info on general guidelines for this?

Copy link
Collaborator

@ultraviolet-jordan ultraviolet-jordan Mar 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only way to know is to login OSRS and test it out for yourself. There are no known leaks or dumps or collections for any type of npc ranges, well I guess we are the collection now...

So if this is a complete guess, then it should NOT be merged (it will be merged anyways)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's an educated guess based on https://youtu.be/rsKPx_gpXSs?si=_kpWqA-k6csJLOge&t=168

In the video, you see one thief follow until the door.

I decided to reduce the hunt range to 1. There are different thieves and now the 2 lv 14s will follow like the screenshots below

Screenshot 2568-03-08 at 13 46 26 Screenshot 2568-03-08 at 13 54 37

@MattGould1 MattGould1 changed the title feat: add huntmode to thieves in monkfriend quest feat(content): add huntmode to thieves in monkfriend quest Mar 11, 2025
@Pazaz Pazaz merged commit 65ef4d7 into 2004Scape:main Mar 11, 2025
2 checks passed
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.

6 participants