diff options
| author | Maze Winther <mazewinther@gmail.com> | 2026-04-15 01:00:05 +0200 |
|---|---|---|
| committer | Maze Winther <mazewinther@gmail.com> | 2026-04-15 01:00:05 +0200 |
| commit | 1fa5442f211fb06548bc51ea3192b5e0b4ffe614 (patch) | |
| tree | 884abec94614860efd34eef385d9e66f3f64a504 /apps/web/src/components/ui/use-overlay-open-change.ts | |
| parent | dc0a8e7c963823432d8c89b6970ead024f06b091 (diff) | |
| download | opencut-1fa5442f211fb06548bc51ea3192b5e0b4ffe614.tar.gz | |
chore: remove debugging throughout codebase
Diffstat (limited to 'apps/web/src/components/ui/use-overlay-open-change.ts')
| -rw-r--r-- | apps/web/src/components/ui/use-overlay-open-change.ts | 133 |
1 files changed, 55 insertions, 78 deletions
diff --git a/apps/web/src/components/ui/use-overlay-open-change.ts b/apps/web/src/components/ui/use-overlay-open-change.ts index c5ef5331..dbc46cfc 100644 --- a/apps/web/src/components/ui/use-overlay-open-change.ts +++ b/apps/web/src/components/ui/use-overlay-open-change.ts @@ -1,78 +1,55 @@ -import { useCallback, useEffect, useId, useRef } from "react";
-import { useKeybindingsStore } from "@/stores/keybindings-store";
-
-export function useOverlayOpenChange({
- source,
- open,
- onOpenChange,
-}: {
- source: string;
- open?: boolean;
- onOpenChange?: (open: boolean) => void;
-}) {
- const { openOverlay, closeOverlay } = useKeybindingsStore();
- const isTrackedRef = useRef(false);
- const isControlled = typeof open === "boolean";
- const overlayId = useId();
-
- useEffect(() => {
- if (!isControlled) return;
-
- if (open && !isTrackedRef.current) {
- openOverlay(overlayId, source);
- isTrackedRef.current = true;
- return;
- }
-
- if (!open && isTrackedRef.current) {
- closeOverlay(overlayId, source);
- isTrackedRef.current = false;
- }
- }, [closeOverlay, isControlled, open, openOverlay, overlayId, source]);
-
- useEffect(() => {
- return () => {
- if (!isTrackedRef.current) return;
- // #region agent log
- fetch(
- "http://127.0.0.1:7245/ingest/669b22f8-172b-4e65-aa3f-1c702ede83f7",
- {
- method: "POST",
- headers: {
- "Content-Type": "application/json",
- "X-Debug-Session-Id": "3997d9",
- },
- body: JSON.stringify({
- sessionId: "3997d9",
- runId: "post-fix",
- hypothesisId: "H2",
- location: "use-overlay-open-change.ts:cleanup",
- message: "Overlay closed during unmount cleanup",
- data: { source, overlayId },
- timestamp: Date.now(),
- }),
- },
- ).catch(() => {});
- // #endregion
- closeOverlay(overlayId, source);
- isTrackedRef.current = false;
- };
- }, [closeOverlay, overlayId, source]);
-
- return useCallback(
- (nextOpen: boolean) => {
- if (!isControlled) {
- if (nextOpen && !isTrackedRef.current) {
- openOverlay(overlayId, source);
- isTrackedRef.current = true;
- } else if (!nextOpen && isTrackedRef.current) {
- closeOverlay(overlayId, source);
- isTrackedRef.current = false;
- }
- }
-
- onOpenChange?.(nextOpen);
- },
- [closeOverlay, isControlled, onOpenChange, openOverlay, overlayId, source],
- );
-}
+import { useCallback, useEffect, useId, useRef } from "react"; +import { useKeybindingsStore } from "@/stores/keybindings-store"; + +export function useOverlayOpenChange({ + open, + onOpenChange, +}: { + open?: boolean; + onOpenChange?: (open: boolean) => void; +}) { + const { openOverlay, closeOverlay } = useKeybindingsStore(); + const isTrackedRef = useRef(false); + const isControlled = typeof open === "boolean"; + const overlayId = useId(); + + useEffect(() => { + if (!isControlled) return; + + if (open && !isTrackedRef.current) { + openOverlay(overlayId); + isTrackedRef.current = true; + return; + } + + if (!open && isTrackedRef.current) { + closeOverlay(overlayId); + isTrackedRef.current = false; + } + }, [closeOverlay, isControlled, open, openOverlay, overlayId]); + + useEffect(() => { + return () => { + if (!isTrackedRef.current) return; + closeOverlay(overlayId); + isTrackedRef.current = false; + }; + }, [closeOverlay, overlayId]); + + return useCallback( + (nextOpen: boolean) => { + if (!isControlled) { + if (nextOpen && !isTrackedRef.current) { + openOverlay(overlayId); + isTrackedRef.current = true; + } else if (!nextOpen && isTrackedRef.current) { + closeOverlay(overlayId); + isTrackedRef.current = false; + } + } + + onOpenChange?.(nextOpen); + }, + [closeOverlay, isControlled, onOpenChange, openOverlay, overlayId], + ); +} |
