feat(player): split controller responsibilities and wire attack cadence

[f1cklepickle]

Why

Player input/combat flow needed structural cleanup to keep PlayerController as an orchestrator and to ensure attack speed changes affect real attack cadence, not only animation feel.

What changed

• Extracted repair and weapon-swap responsibilities out of PlayerController (#143)
• Kept PlayerController focused on orchestration and input routing
• Added authoritative attack cadence wiring using effective rate (baseAttackRate * weaponAttackSpeed) (#145)
• Added animator transition safeguards so hitbox event windows execute reliably
• Added EditMode + PlayMode regression tests for cadence contracts and real damage application
• Updated TEST_LOG with refactor, regression notes, and new test coverage

How to test

1. Open MainPrototype
2. Press Play → verify:
   • PC mouse attack works and deals damage
   • Android/touch attack works and deals damage
   • Attack speed changes affect actual attack cadence
3. Run tests:
   • EditMode
   • PlayMode

Checklist

• Unit tests (EditMode) added or updated
• PlayMode test or manual validation included
• Demo scene updated (if player-visible) (N/A)
• Prefab links, tags, and layers validated
• README / Docs touched

Related

• Refs refactor(player): split PlayerController — extract repair sensor and weapon swap handler #143
• Refs feat(combat): wire weapon attackSpeed to MobileInputDriver attack rate #145