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

Pacifies all pets spawned by Xenobiology's Stabilized Gold #11350

Merged
merged 5 commits into from
Sep 1, 2024

Conversation

aramix273
Copy link
Contributor

@aramix273 aramix273 commented Aug 20, 2024

About The Pull Request

for all I can tell, the pet spawned by stabilized gold should be a pacifist
the pseudo-immortality is biggest argument in this sense
it's still a good question why there are hostile animals in friendly paths and why they have the FRIENDLY_SPAWN tag despite their attacks (perhaps referring to their default AI rather than capacity to harm, which is kind of an edge case considering sentience)

Why It's Good For The Game

Fixes: #11328

you can no longer have stabilized gold pets that can bite and poison you then come back each time you kill them
though they can still ventcrawl, pull and strip
took this route rather than removing the snake and lizard from the pool outright because:

  • it allows them to be pets still,
  • this should cover any other cases if pets were to be added to this

Testing Photographs and Procedure

Screenshots&Videos
pacifytesting2.mp4

Changelog

🆑 Aramix
tweak: pacified all pets spawned by Xenobiology's Stabilized Gold
/:cl:

@aramix273
Copy link
Contributor Author

should ideally solve #11328 too (testing evidence shows so) while keeping code/mechanic changes local to stabilized gold and without modifying simple_animal code

@EvilDragonfiend
Copy link
Member

why don't you just give them pacifist trait?

@aramix273
Copy link
Contributor Author

why don't you just give them pacifist trait?

did not come to mind to use it
apparently the parrot does not care about the proc that I registered that is supposed to cancel attacks
so, pacifism would help, albeit only for attacking mobs
apparently, even though the parrot has 0 obj_damage, it can still hurt walls and other structures while melee_damage is non-zero,, so that has to be set to 0 too

@aramix273
Copy link
Contributor Author

applied my last observations and had to handle a particular case of parrots being able to just change their melee_damage via "Toggle Mode"
this is getting a bit convoluted for the purpose of also disabling such a pet's capacity to damage structures, so if the PR's scope or implementation is getting a bit snowflakey, it could be reduced to pacification against just mobs, at expense of potential later abuse to break structures such as consoles or whatnot

@RDS88-beeman
Copy link
Contributor

RDS88-beeman commented Sep 1, 2024

As (I would like to imagine) the primary abuser of this bullshit, please hit burning yellow next.
Also, I approve this message.

@EvilDragonfiend EvilDragonfiend added this pull request to the merge queue Sep 1, 2024
Merged via the queue into BeeStation:master with commit 3d25f82 Sep 1, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug?]: Stabilized Gold Immortal Pet can attack
3 participants