Skip to content

Gonnect Variant: Cascading Connection Goals - #470

Merged
Perlkonig merged 4 commits into
AbstractPlay:developfrom
samtcifihi:claude/stoic-lamport-kdsc5q
Sep 14, 2026
Merged

Perlkonig merged 4 commits into
AbstractPlay:developfrom
samtcifihi:claude/stoic-lamport-kdsc5q

Conversation

@samtcifihi

Copy link
Copy Markdown
Contributor

If both players pass, ever-decreasing subboards are checked for valid connections. The biggest subboard where only one player connects across it determines the winner

Enables passing and a second end condition: two consecutive passes.
An outright edge-to-edge connection still wins immediately as before.
Otherwise the winner is found by checking, from the full board down
to the centre point, the largest centred subboard on which exactly
one player has an unbroken orthogonal connection between its two
opposite sides; if no such subboard exists, the game is a draw.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y6dm6EQVSpd9YtMQ8xjeMR
The version identifier is for incompatible saved-state shapes. This
change doesn't alter IMoveState/IGonnectState fields (pass already
reuses the existing "place"-less results/lastmove machinery other
games use), so no bump is warranted.

Also add a test playing out the exact tengen scenario via real moves
(place, pass, pass) rather than only via direct board manipulation.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y6dm6EQVSpd9YtMQ8xjeMR
@Perlkonig

Copy link
Copy Markdown
Member

Can you help me better understand the goal of the Gonnect changes here? Also, looks like there are some conflicts with your other Esperanto changes. You'll want to clear those up.

…port-kdsc5q

# Conflicts:
#	locales/eo/apgames.json
@samtcifihi

Copy link
Copy Markdown
Contributor Author

Okay, I think since this hasn't merged yet, I don't need to make a new PR, right?

@samtcifihi

Copy link
Copy Markdown
Contributor Author

@Perlkonig

Copy link
Copy Markdown
Member

Correct. You can just add commits to the PR.

@Perlkonig

Copy link
Copy Markdown
Member

If you don't mind, add experimental: true to the cascading variant definition. That way it won't get accidentally promoted to production until you've had a chance to validate. Everything else looks good.

Keeps it out of production selection until it's been validated on the
dev server.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y6dm6EQVSpd9YtMQ8xjeMR
@samtcifihi

Copy link
Copy Markdown
Contributor Author

Done

@Perlkonig

Copy link
Copy Markdown
Member

Cheers!

@Perlkonig
Perlkonig merged commit 135eee0 into AbstractPlay:develop Sep 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants