Damage Reroll Feats for 5E

[jkeller]

I addressed it for the moment.

The base code only allows one handler, and there's no way for extensions to get any existing handler. So any handler that runs last will be the only one that runs.

My extension overrides the code that registers handlers, and I set it to run early. So, if another extension tries to register, I can capture it and the dispatch to it.

The devs said they will change the code to add a way for extensions to get any registered handler, so we can "chain" ourselves. Not sure when that will happen.

[bwatford]

The savage attacker is only rerolling one of the two weapon dice now. Not sure what changed?

Using Greatsword. it only offers to reroll the first of the two d6's rolled.

I fixed it my adding a bit of code to the CountD, also added a check to make sure it ignored other dice that might get added by CT effects, so it only rerolls the weapon dice.

local CountD = 0;
local nOrigClauses = tonumber(rRoll.nOrigClauses) or 1;
local nClauseCount = 0;
local sEffectsExpr = string.match(rRoll.sDesc, "%[EFFECTS (.-)%]") or "";
local aEffectsDice = DiceManager.convertStringToDice(sEffectsExpr) or {};
local nEffectsDice = #aEffectsDice;
for sDmgType, sDmgExpr in string.gmatch(rRoll.sDesc, "%[TYPE:%s*(.-)%s*%(([^)]*)%)%]") do
nClauseCount = nClauseCount + 1;
if nClauseCount > nOrigClauses then break; end
local lowerType = sDmgType:lower();
if string.match(lowerType, "slashing") or string.match(lowerType, "piercing") or string.match(lowerType, "bludgeoning") then
local sDiceExpr = string.match(sDmgExpr, "(.-)=%d*") or sDmgExpr; -- Strip =total if present
local aTempDice = DiceManager.convertStringToDice(sDiceExpr) or {};
CountD = CountD + math.max(0, #aTempDice - nEffectsDice); -- Subtract effects dice
end
end

[lprchaun]

The savage attacker is only rerolling one of the two weapon dice now. Not sure what changed?

Using Greatsword. it only offers to reroll the first of the two d6's rolled.

I fixed it my adding a bit of code to the CountD, also added a check to make sure it ignored other dice that might get added by CT effects, so it only rerolls the weapon dice.

local CountD = 0;
local nOrigClauses = tonumber(rRoll.nOrigClauses) or 1;
local nClauseCount = 0;
local sEffectsExpr = string.match(rRoll.sDesc, "%[EFFECTS (.-)%]") or "";
local aEffectsDice = DiceManager.convertStringToDice(sEffectsExpr) or {};
local nEffectsDice = #aEffectsDice;
for sDmgType, sDmgExpr in string.gmatch(rRoll.sDesc, "%[TYPE:%s*(.-)%s*%(([^)]*)%)%]") do
nClauseCount = nClauseCount + 1;
if nClauseCount > nOrigClauses then break; end
local lowerType = sDmgType:lower();
if string.match(lowerType, "slashing") or string.match(lowerType, "piercing") or string.match(lowerType, "bludgeoning") then
local sDiceExpr = string.match(sDmgExpr, "(.-)=%d*") or sDmgExpr; -- Strip =total if present
local aTempDice = DiceManager.convertStringToDice(sDiceExpr) or {};
CountD = CountD + math.max(0, #aTempDice - nEffectsDice); -- Subtract effects dice
end
end

the extension should be making sure it is just the weapon dice already.
and I know it was rolling all damage die before.

I know there were alot of changes in the last update, but I have been unable to test my extensions. I am in the process of selling my house and moving so my free time has been non-existent. I know a couple of problems popped up with great weapon master as well so I will be looking into fixing this extension As soon as I can.

Sorry to everybody being affected.

[lprchaun]

Are there currently only 2 reroll damage options for this ext right now? Or is there ways to customize others or are others being added?

I programmed these for 5E feats.
What other damage reroll options are you talking about?

[Frectorius]

Hello

I just bought and installed the extension and it works as inteded (5e, 2024).

If I may, i want to add a suggestion. It would be nice to have an option to select, if the feat is automatically used or if the promt should appear. It could be nice for if a player does not want to confirme eatch round the only attack it can make.

Have a nice day

Greetings
Frectorius

[lprchaun]

Updating. I am still in the process of moving so I am between two houses (not to mention fixing issues with house that is being sold, etc.) so I have not had time to address the issues that recently popped up.
I want everybody to know I have not forgotten will be getting to these items as soon as I can!

[nephranka]

Looks like on TEST:
When any damage is rolled:
[3/18/2026 7:10:56 AM] [ERROR] Handler error: [string "DamageRerollFeats:..nager_action_damage.lua"]:57: attempt to call global 'onDamageRoll' (a nil value)

Passing it along

[nephranka]

Just checking back in since TEST will be live soon.

[coyote670]

Hi,

With this active, every attempt at rolling damage comes up with this error (same as reported above):
[ERROR] Handler error: [string "DamageRerollFeats:..nager_action_damage.lua"]:57: attempt to call global 'onDamageRoll' (a nil value)Untitled.png

[sturtus]

Still broken on release. Do you have a repo? I could have a PR up pretty quick to fix.