Skip to content

Conversation

@majochem
Copy link
Contributor

@majochem majochem commented Jun 20, 2025

Description of the problem being solved:

Adds support for:

  • "You are debilitated"
  • "Nearby enemies take 1% increased physical damage per two fortification on you"
  • "Your Warcries cover enemies in ash for 5 seconds"
  • "Skills used by your Traps and Mines Chain 2 additional times"

Also:

  • Adds some more generic support for "chain" related mods like "projectiles chain x additional times" and the Crusader mod "Socketed Skills chain 1 additional time"
  • Adds the missing 20% less movement speed penalty to "Debilitated" condition (was irrelevant for monsters, but can now apply to players)

Steps taken to verify a working solution:

  • Mods parse correctly and apply their effects only when conditions are met
  • Changes don't seem to break other mods based on ModCache check and some additional testing

Limitations:

  • The helmet mod that increases damage per fortification doesn't work properly when equipped on an AG (see commit msg), but that problem is not exclusive to this specific mod

Link to a build that showcases this PR:

https://pobb.in/OWX-JW52C4u0

After screenshot:

image

majochem and others added 10 commits June 20, 2025 11:07
Debilitated so far only cause 10% less damage
(presumably because it was only applied to players), but should also
apply 20% less movement speed

Also added the "globalEffect" / "debuff" tag in case of future
interactions
This enables "Your are debilitated", "Nearby enemies are debilitated",
"minions are debilitated" etc.

No effect on ModCache as no such mod exists on uniques or tree
(only "Infamous" mod atm)
NOTE: This doesn't correctly work when putting the item on an AG because
actor = "player" is necessary. actor = "enemy" did also not work and is
less safe. Not specifying an actor would make it based on monster
fortification stacks in some cases. "player" is the best solution until
we have actor = "self"
"Skills used by your Traps and Mines Chain 2 additional times"
- Added "skills used by your traps and mines" preFlag
- Added "chain " as modName (extra space to avoid match with "chaining")
- Added "(%d+) additional times?" to formList
This in combination with previous commit enables mods like:
"Projectiles chain 2 additional times"

Also included modCache to check it doesn't break any existing mods
Previous preFlag was limited to "have/deal/gain" etc. which did not work
with e.g. "chain" modName. So now "Socketed gems chain 1 additional time"
works.

Changing the lua pattern from "[hgd][ae][via][enl] "
to "([hgd]?[ae]?[via]?[enl]?%s?)" would probably work in a lot of places
, but I didn't want to do that much testing. It doesn't break any other
"socketed gems xyz" mods for now.
@LocalIdentity LocalIdentity added the enhancement New feature, calculation, or mod label Jun 26, 2025
@LocalIdentity LocalIdentity merged commit 18c0850 into PathOfBuildingCommunity:dev Jun 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, calculation, or mod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants