move hook

Author: jonathanlab

src/renderer/features/tasks/components/TaskDetail.tsx

@@ -4,7 +4,6 @@ import { RichTextEditor } from "@features/editor/components/RichTextEditor";
 import { TaskActions } from "@features/tasks/components/TaskActions";
 import { TaskHeader } from "@features/tasks/components/TaskHeader";
 import { TaskMetadata } from "@features/tasks/components/TaskMetadata";
-import { useTaskPanelLayout } from "@features/tasks/components/useTaskPanelLayout";
 import { useTasks, useUpdateTask } from "@features/tasks/hooks/useTasks";
 import { useTaskExecutionStore } from "@features/tasks/stores/taskExecutionStore";
 import { useBlurOnEscape } from "@hooks/useBlurOnEscape";
@@ -18,6 +17,7 @@ import { useTabStore } from "@stores/tabStore";
 import { expandTildePath } from "@utils/path";
 import { useEffect, useMemo } from "react";
 import { Controller, useForm } from "react-hook-form";
+import { useTaskPanelLayout } from "@/renderer/features/tasks/hooks/useTaskPanelLayout";
 
 interface TaskDetailProps {
   task: Task;

…/tasks/components/useTaskPanelLayout.tsx …tures/tasks/hooks/useTaskPanelLayout.tsxsrc/renderer/features/tasks/components/useTaskPanelLayout.tsx renamed to src/renderer/features/tasks/hooks/useTaskPanelLayout.tsx src/renderer/features/tasks/components/useTaskPanelLayout.tsx renamed to src/renderer/features/tasks/hooks/useTaskPanelLayout.tsx

@@ -7,7 +7,7 @@ import { ListIcon, NotePencilIcon, TerminalIcon } from "@phosphor-icons/react";
 import { Box, Flex, Text } from "@radix-ui/themes";
 import type { Task } from "@shared/types";
 import type { PanelNode, Tab } from "@stores/panelStore";
-import { useEffect, useMemo, useRef } from "react";
+import { useLayoutEffect, useMemo, useRef } from "react";
 
 interface UseTaskPanelLayoutParams {
   task: Task;
@@ -127,7 +127,7 @@ export const useTaskPanelLayout = ({
     [task.id, repoPath, activeArtifactId, onArtifactSelect],
   );
 
-  useEffect(() => {
+  useLayoutEffect(() => {
     // Check if any meaningful values have changed
     const currentValues = {
       taskId: task.id,
@@ -280,18 +280,5 @@ export const useTaskPanelLayout = ({
     };
 
     setRoot(panelStructure);
-  }, [
-    task.id,
-    repoPath,
-    openArtifacts,
-    activeArtifactId,
-    logsContent,
-    shellContent,
-    artifactsContent,
-    taskDetailContent,
-    onCloseArtifact,
-    onSavePlan,
-    panelId,
-    setRoot,
-  ]);
+  });
 };