XY Problems are much more common from the person asking the question though…

“Devops” original intent meant you don’t have a separate “operations” department separate from teams “developing” your product / software due to competing incentives. “Dev” wants to push new stuff out faster; “ops” wants to keep things stable. Or “dev” needs more resources; but “ops” blocks or doesn’t scale the same. The idea was to combine both “dev” and “ops” people onto projects to balance these incentives.

Then managers and cloud clowns repurposed it to apply to every person in a project so now every member is expected to perform both roles (badly). Or even more overloaded to somehow refer to “developer infrastructure” teams.

Google for “replace conditional with polymorphism”.

Just checked and it is in “Clean Code” - Chaper 17; Section G23 “Prefer Polymorphism to if/else or switch/case”.

There are several existing ways to regain 1 focus point in combat - however all of them are constrained with “Frequency once per day”.

I could see adding some kind of similar constraint - or making it a reaction where the trigger is if the previous focus spell failed it gives you a flat check to regain the focus point.

As for examples -

Fire Ray - fire domain clerics are now neverending flamethrowers.

Moonbeam - same for moon domain.

Ki Strike - every turn a monk can now do ki strike flurry of blows to have +1 and 1d6 extra on both strikes.

Hand of the Apprentice - wizards can become backline melee at range - thor hammers fly every round

Additional edit: The change to recover magic would also pretty much destroy any need for resource management - if I’m a caster why not just dump every spell slot I have in this current fight, rest an hour and then do it again? Clerics could just dump all the heal spells from their healing font after combat to heal everyone up, then rest an hour and regain them all and do it again next fight.

Extra Additional Edit: Both of these changes sound like fun cheat codes to make the players win, but it pretty much removes all challenge.

Parquet is a storage format; graphQL is a query language/transmission strategy.

This is also against clean code examples, because Uncle Bob seems to be allergic against function arguments and return values.

I think this is your strawman version of “Clean Code”… not anything that’s actually in it…

I “like” some parts of your example more than the previous one, but a lot of this depends on where exactly in the whole program this method is - if this method is on a “Salesman” class - does it make sense to pass the “Contract” in? If there’s a Contract class available, why doesn’t the “calculateCommission” method exist on it?

“NPC Core” was announced at PaizoCon as a core rulebook coming out in 2025. Not a lot of description yet, but I’d assume it would be an expanded version of what came with the original GMG.