Skip to content

Conversation

@ShepardToTheStars
Copy link
Contributor

@ShepardToTheStars ShepardToTheStars commented Jan 14, 2026

About the PR

Doing at least 5 damage to the ninja will reveal the ninja.

Why / Balance

The AttackedEvent only applies to melee attack events, not to projectiles or general damage.

Technical details

Using the much more generic DamageChangedEvent instead of AttackedEvent. I tried using ProjectileHit and other ranged attacking events but they just weren't working so I'm guessing they were marked handled or whatever.

But this does a few checks.

  • Is there a damage change? If no, return.

  • Is the change in damage healing only? If yes, return.

  • Does the damage have a source still? If no, it might be passive healing, airloss from space, bleed damage, etc, so let's check for those things.

    • Was there damage and healing? If so, return because its probably just an OD from 10u derma or whatever.
    • If the damage didn't have brute or burn types, then return.
  • Does it meet the minumum damage requirement? (4 damage)? If no, return

  • Then reveal the ninja (but only if its stealthed)

Media

simplescreenrecorder-2026-01-14_13.09.43.mp4

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an in-game showcase.

Licensing

Changelog

🆑

  • tweak: The ninja is now revealed with hit with attacks, projectiles, explosions, and grenades that do damage.
  • fix: The ninja is now revealed with hit with attacks, projectiles, explosions, and grenades that do damage.

@ShepardToTheStars ShepardToTheStars requested a review from a team as a code owner January 14, 2026 22:07
@github-actions github-actions bot added size/M 64-255 lines S: Needs Review Awaiting review from a Maintainer Changes: C# Changes any cs files and removed S: Needs Review Awaiting review from a Maintainer labels Jan 14, 2026
@ShepardToTheStars ShepardToTheStars changed the title Changed up how ninja's stealth is removed after they get hit. Ninja Stealth Fix?/Nerf? Jan 14, 2026
@Vapetastic-Gaming Vapetastic-Gaming added the No Direction Review Needed A review from the Direction team is unnecessary for this PR. label Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files No Direction Review Needed A review from the Direction team is unnecessary for this PR. size/M 64-255 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants