Slight expansion to the update earlier today, with another fix and a "feature"
The following files have been updated and must be reset for the changes to apply. These are NOT mandatory, but the knuckles trade bug will remain if you don't reset them. - trading/trades.json - trading/types.json
TWEAK While holding a named name tag, the custom trading system does not interact with villagers. This is to still try and allow you to rename them. Please let me know if I missed any weird interactions
TWEAK Added a "blockeritem" option to the /val merchants command, allowing you to make an item with which you can exclude an individual villager from the custom trading mechanics
TWEAK Daytime and light effect triggers not trigger appropriately, and added separate triggers for when the player is exposed to the outside daylight OR sheltered (meaning they have some block above them preventing some daylight)
FIXED /val merchants summonitem not working
FIXED Immersive recipes replacing the item in the player's hand even if the recipe failed
FIXED "on attack" effect trigger not working most of the time
FIXED Dummies getting default stats
FIXED Players often not being attributed responsibility for dealing custom damage types (primarily noticed with Hexblade) which caused them to be unaffected by damage modification stats
FIXED Royal knuckles being absent from trades