bcollazo / catanatron

94%

master: 93%

LAST BUILD ON BRANCH main

branch: main

CHANGE BRANCH

x

Reset

• main
• Deterministic_Discards
• action-record
• bryan/deterministic-discard
• catanatron_env
• chore/typescript
• chore/typescript-2
• claude/document-apply-function-01448BhkhbMCgL5jdPu51dJ8
• claude/mobile-friendly-homepage-xRYcx
• cli-banner
• copilot/fix-295
• feature/311-support
• feature/add-game-replay
• feature/add-step-by-step-replay-cli
• feature/catanatron-rust
• feature/mcts-testing
• feature/readme
• fix/adjacent-red-pips
• fix/build
• friendly-robber
• gym-improvements
• improve-rl-returns
• improvements
• master
• measuring-performance
• new-docs
• parquet-output
• patch-1
• ppo-example
• python312
• refactor/rename-adjacent-tiles
• restructure
• robber-action-bugfix
• server-updates
• staging
• ui-improvements
• updates

Build # 29432872710

Build Type

push

github

Committed by web-flow

Commit Message

fix: block road extension through opponent's settlement endpoint (#377)

* fix: block road extension through opponent's settlement endpoint

When a settlement is built on a node that is the endpoint of an
opponent's road (len == 1), the node was not removed from the opponent's connected components. This allowed the opponent to later build a road from that blocked endpoint, bypassing the "cannot build through a settlement" rule.

The len == 2 case (settlement in the middle of a road) was already handled correctly.

* test: add unit test for road extension past enemy settlement endpoint

Add test to ensure roads cannot extend past enemy settlements.

Coverage Stats

4 of 4 new or added lines in 1 file covered. (100.0%)

3297 of 3509 relevant lines covered (93.96%)

1.87 hits per line