Skip to content

Conversation

@HammerGS
Copy link
Member

@HammerGS HammerGS commented Jan 3, 2026

Summary

  • Adds 0.21 damage per trooper against targets in the same hex
  • Applies +2 to-hit modifier (melee attack penalty)
  • Adds 0.2 BV per trooper to Offensive Battle Value
  • Conventional infantry only (not battle armor)

Changes

  • InfantryWeaponHandler.java: Tail damage calculation and reporting
  • ComputeAttackerToHitMods.java: +2 to-hit modifier for same-hex attacks
  • InfantryBVCalculator.java: BV calculation (+0.2 per trooper)
  • messages.properties: Added localized strings, removed "(Not Implemented)"

Testing

  • Verified damage appears in combat log: [3 base + 1 Prosthetic Tail]
  • Verified to-hit modifier displays correctly in attack breakdown
  • Build compiles successfully

Finally, your infantry can live out their dreams of being cat girls.
Nyan
🐱

  ## Summary
  - Adds 0.21 damage per trooper against targets in the same hex
  - Applies +2 to-hit modifier (melee attack penalty)
  - Adds 0.2 BV per trooper to Offensive Battle Value
  - Conventional infantry only (not battle armor)

  ## Changes
  - **InfantryWeaponHandler.java**: Tail damage calculation and reporting
  - **ComputeAttackerToHitMods.java**: +2 to-hit modifier for same-hex attacks
  - **InfantryBVCalculator.java**: BV calculation (+0.2 per trooper)
  - **messages.properties**: Added localized strings, removed "(Not Implemented)"

  ## Testing
  - Verified damage appears in combat log: `[3 base + 1 Prosthetic Tail]`
  - Verified to-hit modifier displays correctly in attack breakdown
  - Build compiles successfully

  ---

  *Finally, your infantry can live out their dreams of being cat girls.
  Nyan* 🐱
Copilot AI review requested due to automatic review settings January 3, 2026 01:37
@HammerGS HammerGS requested a review from a team as a code owner January 3, 2026 01:37
@HammerGS HammerGS added the AI Generated Fix AI-generated fix. Requires human testing and review before merging. label Jan 3, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements the Prosthetic Tail, Enhanced feature for conventional infantry units as specified in Interstellar Operations (IO) page 85. The enhancement adds melee combat capabilities with a damage bonus and to-hit penalty, along with appropriate BV adjustments.

Key Changes:

  • Adds 0.21 damage per trooper against targets in the same hex
  • Applies +2 to-hit modifier penalty for melee attacks at range 0
  • Adds 0.2 BV per trooper to offensive battle value

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
InfantryWeaponHandler.java Implements tail damage calculation at range 0 and integrates it into the existing damage reporting system
ComputeAttackerToHitMods.java Adds +2 to-hit modifier for same-hex attacks when conventional infantry has prosthetic tail
InfantryBVCalculator.java Calculates offensive BV bonus of 0.2 per trooper for units with prosthetic tail
messages.properties (common/options) Updates option description with accurate mechanics, removes "(Not Implemented)" tag
messages.properties (common) Adds localized strings for BV and damage reporting
messages.properties (client) Adds localized string for to-hit modifier display

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@HammerGS HammerGS added the AI ready for Review Indicates that is has been in game tested and is ready for review as it can be label Jan 4, 2026
Copy link
Collaborator

@Sleet01 Sleet01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Generated Fix AI-generated fix. Requires human testing and review before merging. AI ready for Review Indicates that is has been in game tested and is ready for review as it can be

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants