ReviewOS

stacks/wildloop

Prevent claiming land that overlaps existing territories

#945
Closed glennmichael123 opened this 9 days ago ·
9 days ago

Problem

ClaimTerritoryAction validates closed-loop + area bounds but never checks overlap with existing territories. A player can claim a loop entirely inside (or overlapping) someone else's territory, producing undefined game state.

Acceptance

  • On claim, detect polygon overlap with existing active territories.
  • Reject (or clip) overlapping claims with a clear error.
  • Decide intended rule: claiming inside enemy land should route through conquest, not claim.
9 days ago

Done (commit a6d4488). ClaimTerritoryAction now rejects (409, code:'overlap') a loop whose polygon intersects any active territory — occupied land is taken via conquest, not by stacking overlapping territories. Added geo.polygonsOverlap (bbox quick-reject + both-direction intersect/containment). Verified via harness.

Sign in to comment on this issue.