Skip to content

feat: add 'skip til my turn' auto-reject for trade offers#184

Merged
njbrake merged 1 commit intomainfrom
feat/auto-reject-trades
Apr 8, 2026
Merged

feat: add 'skip til my turn' auto-reject for trade offers#184
njbrake merged 1 commit intomainfrom
feat/auto-reject-trades

Conversation

@njbrake
Copy link
Copy Markdown
Contributor

@njbrake njbrake commented Apr 8, 2026

Description

Adds a "skip til my turn" option when responding to incoming trade offers. Pressing s rejects the current offer and auto-rejects all subsequent trade offers until the player's next turn begins. This prevents repeated interruptions when the player isn't interested in trading.

The flag is stored on PlayingState and cleared automatically when ChooseAction (the player's turn) arrives. Auto-rejected trades are handled in apply_prompt() before the UI ever switches input modes, so the player stays in spectating mode uninterrupted.

Fixes #179

PR Type

  • New Feature
  • Bug Fix
  • Refactor
  • Documentation
  • Infrastructure / CI

Checklist

  • I understand the code I am submitting
  • New and existing tests pass
  • Documentation was updated where necessary

AI Usage

  • No AI was used
  • AI was used for drafting/refactoring
  • This is fully AI-generated

AI Model/Tool used: Claude Opus 4.6 (Claude Code)

Any Additional AI Details you'd like to share: All code, tests, and documentation written by Claude Code.

  • I am an AI Agent filling out this form (check box if true)

When responding to a trade, pressing `s` rejects the current offer and
auto-rejects all subsequent trade offers until the player's next turn.
This prevents repeated interruptions when the player isn't interested
in trading. The flag clears automatically when the player's turn starts.

Fixes #179

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@njbrake njbrake merged commit c9a2216 into main Apr 8, 2026
5 checks passed
@njbrake njbrake deleted the feat/auto-reject-trades branch April 8, 2026 14:27
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.

Add a "no til it's my turn" option

1 participant