Frumple wrote:It would be a little redundant for archers, but it would also catch the tinkers...
Ah, so Embers classes inherit the changes to Cunning/Survival from the main module?
[sound F/X: source diving] And yes, all the Embers classes get the tree; that would let me back out the changes to Last Engineer Standing, which always felt like a slightly awkward fit.
Frumple wrote:and I guess just about anyone else that might use a ranged attack for some reason or another.
…well, for certain values of "ranged attack". As currently implemented, we only catch ranged attacks that go through Archery:archeryShoot(); that covers your Corpsebow example, but I don't know what else you had in mind. Throwing Knives, for instance, goes straight to ActorProject:projectile(), so it's a good thing we don't have to cover that case.