Two bugs in the Opportunist bot strategy:
1. retreatMove: when a lone bot is surrounded by enemies with no nearby
allies, all passable directions scored below the -1 initial threshold
due to flat enemy penalties. Fixed by scoring distance-from-enemies
as a positive value (further = better) instead of a flat penalty,
ensuring the bot always picks the safest direction.
2. attackMove: BFS could never reach enemy targets because the passable
function excluded all enemy positions. The target IS an enemy, so the
path was unreachable. Fixed by wrapping passable to treat the target
position as passable during attack pathfinding.
All 19 tests now pass, including TestComputeMovesRetreat and
TestComputeMovesNearbyAdvantageAttack.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
cecbc4a2a0