feat(player): add post-aim facing grace before movement override

[f1cklepickle]

Why

Releasing attack aim while moving could immediately return facing to walking, causing a snap during the transition into block or parry. Facing needs a short retained-aim window and deterministic defense priority without coupling movement velocity to facing.

What changed

• Added a prefab-tunable 0.6-second post-aim facing grace for desktop and mobile input
• Made active defense aim the highest-priority facing source with retained/current fallback
• Separated facing execution from velocity-only movement orchestration
• Added EditMode, PlayMode, prefab-contract, and regression coverage

How to test

1. Open the main prototype scene and move away from a target while aiming toward it
2. Release attack aim, begin block/parry during the grace period, and verify movement continues while facing stays stable
3. Adjust defense aim while held and verify it overrides walking; release defense and verify walking immediately regains facing
4. Run tests:
   • EditMode
   • PlayMode

Checklist

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

Related

• Closes feat(player): add post-aim facing grace before movement override #289

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0bd88647b4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Clear stale aim when a new attack has no direction

When playing with a physical gamepad in an environment without a mouse, an aimed attack populates lastAttackAimDirection, but a later right-trigger attack with the right stick centered leaves that value unchanged because hasResolvedAim is false. Releasing the later attack then starts the grace period using the previous attack's direction, unexpectedly overriding current movement facing for 0.6 seconds. Reset or session-scope the stored direction when a new attack-aim session begins without valid aim.

Useful? React with 👍 / 👎.