refactor: update command execute methods to return CommandResult | undefined

1 files changed, 70 insertions, 69 deletions

diff --git a/apps/web/src/lib/commands/timeline/element/update-element-start-time.ts b/apps/web/src/lib/commands/timeline/element/update-element-start-time.ts
index ff81cff2..6232d539 100644
--- a/apps/web/src/lib/commands/timeline/element/update-element-start-time.ts
+++ b/apps/web/src/lib/commands/timeline/element/update-element-start-time.ts
@@ -1,69 +1,70 @@
-import { Command } from "@/lib/commands/base-command";

-import type { TimelineTrack } from "@/lib/timeline";

-import { EditorCore } from "@/core";

-import { enforceMainTrackStart } from "@/lib/timeline/placement";

-

-export class UpdateElementStartTimeCommand extends Command {

-	private savedState: TimelineTrack[] | null = null;

-	private readonly elements: { trackId: string; elementId: string }[];

-	private readonly startTime: number;

-

-	constructor({

-		elements,

-		startTime,

-	}: {

-		elements: { trackId: string; elementId: string }[];

-		startTime: number;

-	}) {

-		super();

-		this.elements = elements;

-		this.startTime = startTime;

-	}

-

-	execute(): void {

-		const editor = EditorCore.getInstance();

-		this.savedState = editor.timeline.getTracks();

-

-		const currentTracks = this.savedState;

-		const updatedTracks = currentTracks.map((track) => {

-			const hasElementsToUpdate = this.elements.some(

-				(elementEntry) => elementEntry.trackId === track.id,

-			);

-

-			if (!hasElementsToUpdate) {

-				return track;

-			}

-

-			const newElements = track.elements.map((element) => {

-				const shouldUpdate = this.elements.some(

-					(elementEntry) =>

-						elementEntry.elementId === element.id &&

-						elementEntry.trackId === track.id,

-				);

-				if (!shouldUpdate) {

-					return element;

-				}

-

-				const baseStartTime = Math.max(0, this.startTime);

-				const adjustedStartTime = enforceMainTrackStart({

-					tracks: currentTracks,

-					targetTrackId: track.id,

-					requestedStartTime: baseStartTime,

-					excludeElementId: element.id,

-				});

-

-				return { ...element, startTime: adjustedStartTime };

-			});

-			return { ...track, elements: newElements } as typeof track;

-		});

-

-		editor.timeline.updateTracks(updatedTracks);

-	}

-

-	undo(): void {

-		if (this.savedState) {

-			const editor = EditorCore.getInstance();

-			editor.timeline.updateTracks(this.savedState);

-		}

-	}

-}

+import { Command, type CommandResult } from "@/lib/commands/base-command";
+import type { TimelineTrack } from "@/lib/timeline";
+import { EditorCore } from "@/core";
+import { enforceMainTrackStart } from "@/lib/timeline/placement";
+
+export class UpdateElementStartTimeCommand extends Command {
+	private savedState: TimelineTrack[] | null = null;
+	private readonly elements: { trackId: string; elementId: string }[];
+	private readonly startTime: number;
+
+	constructor({
+		elements,
+		startTime,
+	}: {
+		elements: { trackId: string; elementId: string }[];
+		startTime: number;
+	}) {
+		super();
+		this.elements = elements;
+		this.startTime = startTime;
+	}
+
+	execute(): CommandResult | undefined {
+		const editor = EditorCore.getInstance();
+		this.savedState = editor.timeline.getTracks();
+
+		const currentTracks = this.savedState;
+		const updatedTracks = currentTracks.map((track) => {
+			const hasElementsToUpdate = this.elements.some(
+				(elementEntry) => elementEntry.trackId === track.id,
+			);
+
+			if (!hasElementsToUpdate) {
+				return track;
+			}
+
+			const newElements = track.elements.map((element) => {
+				const shouldUpdate = this.elements.some(
+					(elementEntry) =>
+						elementEntry.elementId === element.id &&
+						elementEntry.trackId === track.id,
+				);
+				if (!shouldUpdate) {
+					return element;
+				}
+
+				const baseStartTime = Math.max(0, this.startTime);
+				const adjustedStartTime = enforceMainTrackStart({
+					tracks: currentTracks,
+					targetTrackId: track.id,
+					requestedStartTime: baseStartTime,
+					excludeElementId: element.id,
+				});
+
+				return { ...element, startTime: adjustedStartTime };
+			});
+			return { ...track, elements: newElements } as typeof track;
+		});
+
+		editor.timeline.updateTracks(updatedTracks);
+		return undefined;
+	}
+
+	undo(): void {
+		if (this.savedState) {
+			const editor = EditorCore.getInstance();
+			editor.timeline.updateTracks(this.savedState);
+		}
+	}
+}