https://wildfiregames.com/forum/index.php?/topic/27214-borg-expansion-pack-mod-implementation-in-0ad-alpha-24-release/&do=findComment&comment=388029
We might check for this.isIdle in the LosRangeUpdate but I'm not sure that would not introduce other side effects. And setting up a MWE for this specific case is not too easy ;)

If this.idle is false in LosRangeUpdate, simply call FindNewTargets, but because there can be multiple events while this.idle is false on LosRangeUpdate, there need to be some boolean which would ensure we call FindNewTargets just once and then we are safe to fall back to default behaviour.

Or another more safer approach is to ingore range updates in IDLE state while unit is not true idle, what looks more likely what we want.

Notice that while that events would be ingored, entities will be taken into account when timer fires up.

I am glad we came to the same conclusion, albeit you expressed it clearly, and I had trouble transferring it from my mind to here ;)

Thanks for the fix!