chore(cleanup): move pure-math out of components/video-editor/videoPlayback/ into lib/zoomMath

[EtienneLescot]

What

Pure-rename cleanup. The folder src/components/video-editor/videoPlayback/ was a misnomer — it contained 11 files of pure zoom/spring/focus math (zero React), but the components/ location kept implying they were React components. They aren't. Move them into a clearer home: src/lib/zoomMath/.

Why

The directory has been importing across the codebase as if it were React, and types in components/ are being pulled into pure-math modules across the lib/ boundary. src/lib/zoomMath/ puts the math next to its peers (compositeLayout, cursor, exporter) and out of the components tree, where the contents actually belong.

Files moved (9 source + 2 co-located unit tests)

cameraFullscreenUtils.ts
cameraFullscreenUtils.test.ts
constants.ts
cursorFollowUtils.ts
focusUtils.ts
focusUtils.test.ts (does not exist — listed for completeness)
mathUtils.ts
motionSmoothing.ts
zoomRegionUtils.ts
zoomSpring.ts
zoomSpring.test.ts
zoomTransform.ts

git mv was used throughout — every change is a true rename, history is preserved.

Import sites updated (8 files, ~15 import specifiers)

src/native/sceneDescription.test.ts
src/lib/exporter/wgsl/evaluate.ts
src/lib/exporter/frameRenderer.ts
src/lib/cursor/pixiCursorRenderer.ts
src/lib/cursor/cursorPathSmoothing.ts
src/lib/ai-edition/timeline/zoom-preview.ts
src/components/ai-edition/ZoomFocusOverlay.tsx
src/components/ai-edition/PreviewCanvas.tsx

Plus 5 internal ../types -> @/components/video-editor/types fixes inside the moved files (cameraFullscreenUtils, constants, cursorFollowUtils, focusUtils, zoomRegionUtils) and the one inside the moved cameraFullscreenUtils.test.ts.

src/components/video-editor/types.ts was intentionally left in place — it is still a real (if misnamed) shared types module used by src/lib/ai-edition/. The directory src/components/video-editor/ itself stays; only the misnamed videoPlayback/ subfolder moved. types.ts is on the todo list for a future cleanup once the ai-edition rename is done.

The old src/components/video-editor/videoPlayback/ directory is removed; it was empty after the move.

Doc comments

Two prose references to the moved files were updated to the new paths:

• technical-documentation/architecture/cursor.md (cursorFollowUtils.ts reference)
• crates/compositor/src/cursor.rs (constants.ts reference)

Stale comments referencing files outside this move (VideoPlayback.tsx, videoPlayback/overlayUtils.ts, videoPlayback/videoEventHandlers.ts) were left alone — out of scope.

Classification

• Type: Refactor / maintenance
• Severity: Minor
• Platform specificity: Not platform-specific
• Risk: Low — pure rename, no behavior change

Testing

• npx tsc --noEmit — clean
• npm run test — 1144/1144 passing (baseline 1144/1144)
• npm run lint — 0 new warnings vs baseline (baseline has 7 pre-existing warnings in NewProjectModal.test.tsx, rafCoalesce.test.ts, and useNativeCompositorView.ts — none in files touched here)

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: eb2d0119-bae4-4a25-af02-8c899d7f6ad5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ponytail/move-zoom-math-to-lib

---

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.}