fix(studio): route rotation field edits through the animation like X/Y/W/H

[calcarazgre646]

Problem

In the PropertyPanel transform section, typing into X/Y or W/H on a GSAP-animated element routes the value into the animation:

if (onCommitAnimatedProperty && hasGsapAnimation) {
  void onCommitAnimatedProperty(element, axis, parsed);
  return;
}

commitManualRotation did not do this — it went straight to onSetManualRotation, so typing a rotation on an animated element behaved differently from every other transform field. Rotation is a supported animated property (SUPPORTED_PROPS includes "rotation"), and both the rotate-drag gesture (tryGsapRotationIntercept) and the rotation keyframe button (onCommitAnimatedProperty(element, "rotation", ...)) already route through the animation — the panel field was the one path that bypassed it.

Fix

Extract the shared animated-transform routing (animated property -> keyframe -> "no callbacks" toast) into commitAnimatedTransformValue, and have both commitManualOffset and commitManualRotation go through it. Non-animated elements still fall through to the manual setter unchanged. commitManualSize keeps its own keyframe-less shape.

Notes

• No test added: PropertyPanel.test.ts covers pure helpers only, and the X/Y/W/H routing has no render-test coverage either; this mirrors them. Full studio suite stays green (796 passing).

[miguel-heygen]

Strengths:

• PropertyPanel.tsx:164–183 — commitAnimatedTransformValue is a clean extraction. The boolean return / early-return pattern reads clearly and the three callers (onCommitAnimatedProperty, onAddKeyframe, hasGsapAnimation-only toast) are correctly unified.
• The bug is real. commitManualOffset already routed through the animation; commitManualRotation silently bypassed it. "rotation" is already in SUPPORTED_PROPS and the drag gesture already routes through onCommitAnimatedProperty, so this field was the one inconsistent path.

---

Blocker:

packages/studio/src/components/editor/PropertyPanel.tsx — 613 lines (max 600).

##[error]packages/studio/src/components/editor/PropertyPanel.tsx has 613 lines (max 600)

The File size check CI gate is failing. The refactor adds a net +27 lines, which pushed the file 13 lines over the limit. The fix logic is sound — the file just needs to shed those lines before merge.

Options:

1. Move commitAnimatedTransformValue (and possibly the existing commitManualSize helper) into a co-located util / hook (e.g., usePropertyPanelCommit.ts).
2. Or trim equivalent lines elsewhere in the panel if there's dead weight.

---

Verdict: REQUEST CHANGES
Reasoning: One required CI gate failing (File size check). The bug fix and refactor are correct; the only issue is the file now exceeds the 600-line limit.

— Magi