fix: remove wield value frequency, fix cache invalidation

[OrenAudeles]

Summary

SUMMARY: Bugfixes "Revert one per minute frequency addition to npc::regen_ai_cache and fix cache invalidation"

Purpose of change

Having wielded value to be calculated and cached only once per minute is a clunky solution caused by the bug in deactivate_weapon_cbm. The bug was forcing the cache to become invalid every turn so the following turn it was recalculated instead of actually being used.

Describe the solution

Removed the once_every wrapping condition introduced in #3283 and move the cache invalidator from outside the loop to inside so the cached value is invalidated only when the NPC is actually turning off weapon cbms. Now, instead of possibly calculating other values using up-to 1 minute old data the NPC will use the most accurate data it can. More accuracy, fewer calls to the expensive function.

Testing

Waited several hours in both pre-change and post-change configurations. Performance is approximately the same between the two. In profiling the NPCs calculate the cached value once then properly refer to that cached value every turn after.