Skip to content

Add support for Lethal Dose support#9648

Merged
LocalIdentity merged 1 commit intoPathOfBuildingCommunity:devfrom
majochem:feat/lethalDose
Mar 16, 2026
Merged

Add support for Lethal Dose support#9648
LocalIdentity merged 1 commit intoPathOfBuildingCommunity:devfrom
majochem:feat/lethalDose

Conversation

@majochem
Copy link
Contributor

Description of the problem being solved:

Mod parsing for "additional_poisons_+_to_apply_vs_non_poisoned_enemies" on Lethal Dose support was still missing and is now added

Steps taken to verify a working solution:

  • Works with standard Viper Strike
  • Only applies if NonPoisonedOnly is true (e.g. via config)
  • Does not work with/benefit Viper Strike of the Mamba (because it can't apply multiple poisons)
  • Also works with spells

Link to a build that showcases this PR:

Lethal Dose Test

After screenshot:

Mod parsing
image

Working with standard Viper Strike
image

"Not working" with Mamba
image

@majochem majochem marked this pull request as ready for review March 16, 2026 10:59
@LocalIdentity LocalIdentity changed the title Add support for "Lethal Dose" support (for real this time) Add support for "Lethal Dose" support Mar 16, 2026
@LocalIdentity LocalIdentity added the enhancement New feature, calculation, or mod label Mar 16, 2026
@LocalIdentity LocalIdentity merged commit 3edc00a into PathOfBuildingCommunity:dev Mar 16, 2026
3 checks passed
@LocalIdentity LocalIdentity changed the title Add support for "Lethal Dose" support Add support for Lethal Dose support Mar 16, 2026
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