Cleanup: remove unused code from four files

[VariableVince]

Description:

PR 2/x in effort to break up PR #3220. Follows on #3233. Precedes #3235.

Please see if these can be merged for v30.

Removes unused code and properties.

• Game.ts and DefaultConfig unitInfo: removed canBuildTrainStation and expirimental properties, as they weren't used anywhere anymore.

• PlayerActionHandler: remove unused getPlayerActions, the only potential caller MainRadialMenu already just calls myPlayer.actions via GameView directly.

• StructureIconsLayer: remove unused PlayerActions

Please complete the following:

• I have added screenshots for all UI updates
• I process any text displayed to the user through translateText() and I've added it to the en.json file
• I have added relevant tests to the test directory
• I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced

Please put your Discord username so you can be contacted if a bug or regression is found:

tryout33

[coderabbitai]

Walkthrough

This pull request removes PlayerActions-related code from graphics layers and eliminates two optional metadata properties (canBuildTrainStation and experimental) from the UnitInfo interface and its default configuration, simplifying the unit system.

Changes

Cohort / File(s)	Summary
Player Action Removal src/client/graphics/layers/PlayerActionHandler.ts, src/client/graphics/layers/StructureIconsLayer.ts	Removed PlayerActions import, public playerActions property, and getPlayerActions() method from graphics layers.
UnitInfo Property Cleanup src/core/game/Game.ts, src/core/configuration/DefaultConfig.ts	Removed canBuildTrainStation and experimental optional properties from UnitInfo interface and removed corresponding property assignments from City, Factory, Port, and Train unit configurations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🧹 Clean slate for units, no trains to explore,
Experimental flags fade out the door,
Layers grow lighter, actions removed,
Simple and focused, the codebase improved. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: removing unused code from multiple files as part of a refactoring effort.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check	✅ Passed	The description clearly explains the purpose of removing unused code and properties across four files, matching the actual changeset.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[DevelopingTom]

Thanks!