Skip to content

fix: add targeting capability fields to UnitRenderState for battle si…#1367

Merged
braedonsaunders merged 1 commit into
mainfrom
claude/fix-fight-button-nDFoI
Feb 4, 2026
Merged

fix: add targeting capability fields to UnitRenderState for battle si…#1367
braedonsaunders merged 1 commit into
mainfrom
claude/fix-fight-button-nDFoI

Conversation

@braedonsaunders

Copy link
Copy Markdown
Owner

…mulator

The battle simulator's Fight button wasn't working because the findValidTarget() function needed isNaval, canAttackGround, and canAttackAir fields which weren't being sent in the render state.

  • Add isNaval, canAttackGround, canAttackAir to UnitRenderState interface
  • Update GameWorker to send these fields in the render state
  • Update UnitAdapter to expose these fields to the main thread
  • Remove debug logging from BattleSimulatorPanel

https://claude.ai/code/session_01RByrKsfTJ9jf3kCjztLrY5

…mulator

The battle simulator's Fight button wasn't working because the
findValidTarget() function needed isNaval, canAttackGround, and
canAttackAir fields which weren't being sent in the render state.

- Add isNaval, canAttackGround, canAttackAir to UnitRenderState interface
- Update GameWorker to send these fields in the render state
- Update UnitAdapter to expose these fields to the main thread
- Remove debug logging from BattleSimulatorPanel

https://claude.ai/code/session_01RByrKsfTJ9jf3kCjztLrY5
@braedonsaunders braedonsaunders merged commit efe33c4 into main Feb 4, 2026
4 checks passed
@braedonsaunders braedonsaunders deleted the claude/fix-fight-button-nDFoI branch February 4, 2026 19:24
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.

2 participants