Add pet and annoy animations for cats and sheep

[MeLoNaCo]

Cats now play a pet animation, while sheep play an annoy animation and turn away from the player.

Adds visual feedback for pet interactions. Make the sound effect also optional. If both are passed, the interact area is disabled until both finish playing. For that, move from using await to use signals with callbacks.

Document the PetInteraction class.

Add a configuration warning if the AnimatedSprite2D doesn't have an animation named "pet". Unfortunately this requires making the script a @tool.

Resolves #2220

---

Co-authored-by: Stephanie Vega tefsv.me@gmail.com

[github-actions]

Test build no longer available.

[MeLoNaCo]

@manuq @KarmDK

[Un-player0]

Hi!
I've checked your changes and can confirm the pet and annoy animation work as intended

2026-09-24.11-50-17.mp4

2026-09-24.11-52-06.mp4

I would suggest moving face_away_behavior to scenes/game_elements/characters/npcs/components/ next to pet_interaction to keep both in the same place, or alternatively inside sheep/components

[manuq]

Looks good!

In general I'm a bit worried of PetInteraction becoming too complex for the 2 existing scenes using it, the cat and the sheep (well, also a townie in Dev Archipelago!). Maybe we should pause and frame what this node about. A simple "no code" way to react to an interaction?

• Play sound effect.
• Play animation.
• Shake.
• Increment a fact counter.

If you agree, I will document the PetInteaction class with the above. For behaviors that require custom code like the sheep facing away, it would be easier to do them just by connecting to interaction_started of the InteractArea directly.

[manuq]

Check the function arguments, there is one called _from_right:

func _on_interact_area_interaction_started(player: Player, _from_right: bool) -> void:

You can use it to flip the sprite, instead of calculating it again in apply().

[manuq]

For consistency, maybe the animation name should be called "pet" even if we are annoying the sheep. The node name is PetInteraction.

[manuq]

I can see why you added this. Some sheeps are flipped horizontally by scale.x = -1. For example in Fray's End West:

I like that you made this a behavior, although one specific to the sheep. It should go inside a components folder: scenes/game_elements/props/decoration/sheep/components/

Also I find it a bit indirect that InteractArea emits a signal, then PetInteraction captures it and calls apply on this. This behavior could be connected to InteractArea.interaction_started directly. Please do so.

[manuq]

I would suggest moving face_away_behavior to scenes/game_elements/characters/npcs/components/ next to pet_interaction to keep both in the same place, or alternatively inside sheep/components

Thanks for your review, @Un-player0 ! Yes, I would say sheep/components.

[manuq]

@MeLoNaCo hi! Are you still around to do the changes suggested above?

[manuq]

I made the changes myself this time. And also:

• Changed the use of await in favor of signal callbacks. So the interact area is re-enabled after the sound or animation (or both) finish playing.
• Made the behaviors to disable a bit more generic.
• Documented the PetInteraction class.
• Added a configuration warning if the AnimatedSprite2D doesn't have an animation named "pet". Unfortunately this requires making the script a @tool.