REMOVE DEAD CODE

Author: charlesvien

apps/array/src/renderer/components/MainLayout.tsx

@@ -13,25 +13,17 @@ import { MainSidebar } from "@features/sidebar/components/MainSidebar";
 import { useSidebarStore } from "@features/sidebar/stores/sidebarStore";
 import { TaskDetail } from "@features/task-detail/components/TaskDetail";
 import { TaskInput } from "@features/task-detail/components/TaskInput";
-import { TaskList } from "@features/task-list/components/TaskList";
 import { useIntegrations } from "@hooks/useIntegrations";
 import { Box, Flex } from "@radix-ui/themes";
 import { clearApplicationStorage } from "@renderer/lib/clearStorage";
-import type { Task } from "@shared/types";
 import { useNavigationStore } from "@stores/navigationStore";
 import { useCallback, useEffect, useState } from "react";
 import { useHotkeys } from "react-hotkeys-hook";
 import { Toaster } from "sonner";
 
 export function MainLayout() {
-  const {
-    view,
-    toggleSettings,
-    navigateToTask,
-    navigateToTaskInput,
-    goBack,
-    goForward,
-  } = useNavigationStore();
+  const { view, toggleSettings, navigateToTaskInput, goBack, goForward } =
+    useNavigationStore();
   const clearAllLayouts = usePanelLayoutStore((state) => state.clearAllLayouts);
   const toggleLeftSidebar = useSidebarStore((state) => state.toggle);
   const toggleRightSidebar = useRightSidebarStore((state) => state.toggle);
@@ -120,21 +112,13 @@ export function MainLayout() {
     };
   }, [goBack, goForward]);
 
-  const handleSelectTask = (task: Task) => {
-    navigateToTask(task);
-  };
-
   return (
     <Flex direction="column" height="100vh">
       <HeaderRow />
       <Flex flexGrow="1" overflow="hidden">
         <MainSidebar />
 
         <Box flexGrow="1" overflow="hidden">
-          {view.type === "task-list" && (
-            <TaskList onSelectTask={handleSelectTask} />
-          )}
-
           {view.type === "task-input" && <TaskInput />}
 
           {view.type === "task-detail" && view.data && (

apps/array/src/renderer/features/command/components/CommandMenu.tsx

@@ -1,7 +1,7 @@
 import { Command } from "@features/command/components/Command";
 import { CommandKeyHints } from "@features/command/components/CommandKeyHints";
 import { useTasks } from "@features/tasks/hooks/useTasks";
-import { FileTextIcon, ListBulletIcon } from "@radix-ui/react-icons";
+import { FileTextIcon } from "@radix-ui/react-icons";
 import { Flex, Text } from "@radix-ui/themes";
 import type { Task } from "@shared/types";
 import { useNavigationStore } from "@stores/navigationStore";
@@ -14,8 +14,7 @@ interface CommandMenuProps {
 }
 
 export function CommandMenu({ open, onOpenChange }: CommandMenuProps) {
-  const { navigateToTaskList, navigateToTask, navigateToTaskInput } =
-    useNavigationStore();
+  const { navigateToTask, navigateToTaskInput } = useNavigationStore();
   const { data: tasks = [] } = useTasks();
   const commandRef = useRef<HTMLDivElement>(null);
 
@@ -64,11 +63,6 @@ export function CommandMenu({ open, onOpenChange }: CommandMenuProps) {
     };
   }, [open, onOpenChange]);
 
-  const handleNavigateToTasks = () => {
-    navigateToTaskList();
-    onOpenChange(false);
-  };
-
   const handleCreateTask = () => {
     navigateToTaskInput();
     onOpenChange(false);
@@ -112,16 +106,6 @@ export function CommandMenu({ open, onOpenChange }: CommandMenuProps) {
               </Command.Item>
             </Command.Group>
 
-            <Command.Group heading="Navigation">
-              <Command.Item
-                value="Go to tasks"
-                onSelect={handleNavigateToTasks}
-              >
-                <ListBulletIcon className="mr-3 h-3 w-3 text-gray-11" />
-                <Text size="1">Go to tasks</Text>
-              </Command.Item>
-            </Command.Group>
-
             {tasks.length > 0 && (
               <Command.Group heading="Tasks">
                 {tasks.map((task) => (

apps/array/src/renderer/features/panels/constants/panelConstants.ts

@@ -23,7 +23,5 @@ export const DEFAULT_TAB_IDS = {
   LOGS: "logs",
   SHELL: "shell",
   FILES: "files",
-  TODO_LIST: "todo-list",
-  ARTIFACTS: "artifacts",
   CHANGES: "changes",
 } as const;

apps/array/src/renderer/features/panels/store/panelLayoutStore.test.ts

@@ -107,32 +107,6 @@ describe("panelLayoutStore", () => {
     });
   });
 
-  describe("openArtifact", () => {
-    beforeEach(() => {
-      usePanelLayoutStore.getState().initializeTask("task-1");
-    });
-
-    it("adds artifact tab to main panel", () => {
-      usePanelLayoutStore.getState().openArtifact("task-1", "plan.md");
-
-      const panel = findPanelById(getPanelTree("task-1"), "main-panel");
-      const artifactTab = panel?.content.tabs.find((t: { id: string }) =>
-        t.id.startsWith("artifact-"),
-      );
-      expect(artifactTab?.id).toBe("artifact-plan.md");
-    });
-
-    it("tracks open artifacts in metadata", () => {
-      usePanelLayoutStore.getState().openArtifact("task-1", "plan.md");
-      usePanelLayoutStore.getState().openArtifact("task-1", "notes.md");
-
-      const layout = getLayout("task-1");
-      expect(layout.openArtifacts).toContain("plan.md");
-      expect(layout.openArtifacts).toContain("notes.md");
-      expect(layout.openArtifacts).toHaveLength(2);
-    });
-  });
-
   describe("closeTab", () => {
     beforeEach(() => {
       usePanelLayoutStore.getState().initializeTask("task-1");

apps/array/src/renderer/features/panels/store/panelLayoutStore.ts

@@ -7,7 +7,6 @@ import {
 import {
   addNewTabToPanel,
   applyCleanupWithFallback,
-  createArtifactTabId,
   createDiffTabId,
   createFileTabId,
   generatePanelId,
@@ -32,7 +31,6 @@ import type { PanelNode, Tab } from "./panelTypes";
 export interface TaskLayout {
   panelTree: PanelNode;
   openFiles: string[];
-  openArtifacts: string[];
   draggingTabId: string | null;
   draggingTabPanelId: string | null;
   focusedPanelId: string | null;
@@ -48,14 +46,8 @@ export interface PanelLayoutStore {
     taskId: string,
     terminalLayoutMode?: "split" | "tabbed",
   ) => void;
-  openFile: (taskId: string, filePath: string, asPreview?: boolean) => void;
-  openArtifact: (taskId: string, fileName: string) => void;
-  openDiff: (
-    taskId: string,
-    filePath: string,
-    status?: string,
-    asPreview?: boolean,
-  ) => void;
+  openFile: (taskId: string, filePath: string) => void;
+  openDiff: (taskId: string, filePath: string, status?: string) => void;
   keepTab: (taskId: string, panelId: string, tabId: string) => void;
   closeTab: (taskId: string, panelId: string, tabId: string) => void;
   closeOtherTabs: (taskId: string, panelId: string, tabId: string) => void;
@@ -287,11 +279,6 @@ export const usePanelLayoutStore = createWithEqualityFn<PanelLayoutStore>()(
         set((state) => openTab(state, taskId, tabId, asPreview));
       },
 
-      openArtifact: (taskId, fileName) => {
-        const tabId = createArtifactTabId(fileName);
-        set((state) => openTab(state, taskId, tabId, false));
-      },
-
       openDiff: (taskId, filePath, status, asPreview = true) => {
         const tabId = createDiffTabId(filePath, status);
         set((state) => openTab(state, taskId, tabId, asPreview));

apps/array/src/renderer/features/panels/store/panelStoreHelpers.ts

@@ -7,7 +7,7 @@ import type { GroupPanel, LeafPanel, PanelNode, Tab } from "./panelTypes";
 export const DEFAULT_FALLBACK_TAB = DEFAULT_TAB_IDS.LOGS;
 
 // Tab ID utilities
-export type TabType = "file" | "artifact" | "diff" | "system";
+export type TabType = "file" | "diff" | "system";
 
 export interface ParsedTabId {
   type: TabType;
@@ -18,10 +18,6 @@ export function createFileTabId(filePath: string): string {
   return `file-${filePath}`;
 }
 
-export function createArtifactTabId(fileName: string): string {
-  return `artifact-${fileName}`;
-}
-
 export function createDiffTabId(filePath: string, status?: string): string {
   if (status) {
     return `diff-${status}:${filePath}`;
@@ -44,9 +40,6 @@ export function parseTabId(tabId: string): ParsedTabId & { status?: string } {
   if (tabId.startsWith("file-")) {
     return { type: "file", value: tabId.slice(5) };
   }
-  if (tabId.startsWith("artifact-")) {
-    return { type: "artifact", value: tabId.slice(9) };
-  }
   if (tabId.startsWith("diff-")) {
     const rest = tabId.slice(5);
     // Check for status:path format
@@ -184,12 +177,6 @@ export function createNewTab(
         status: (parsed.status || "modified") as GitFileStatus,
       };
       break;
-    case "artifact":
-      data = {
-        type: "artifact",
-        artifactId: parsed.value,
-      };
-      break;
     case "system":
       if (tabId === "logs") {
         data = { type: "logs" };
@@ -282,26 +269,18 @@ export function updateMetadataForTab(
   layout: TaskLayout,
   tabId: string,
   action: "add" | "remove",
-): Pick<TaskLayout, "openFiles" | "openArtifacts"> {
+): Pick<TaskLayout, "openFiles"> {
   const parsed = parseTabId(tabId);
 
   if (parsed.type === "file") {
     const openFiles =
       action === "add"
         ? [...layout.openFiles, parsed.value]
         : layout.openFiles.filter((f) => f !== parsed.value);
-    return { openFiles, openArtifacts: layout.openArtifacts };
-  }
-
-  if (parsed.type === "artifact") {
-    const openArtifacts =
-      action === "add"
-        ? [...layout.openArtifacts, parsed.value]
-        : layout.openArtifacts.filter((f) => f !== parsed.value);
-    return { openFiles: layout.openFiles, openArtifacts };
+    return { openFiles };
   }
 
-  return { openFiles: layout.openFiles, openArtifacts: layout.openArtifacts };
+  return { openFiles: layout.openFiles };
 }
 
 // Cleanup utilities

apps/array/src/renderer/features/panels/store/panelTypes.ts

@@ -33,10 +33,6 @@ export type TabData =
       command: string;
       scriptType: "init" | "start";
     }
-  | {
-      type: "artifact";
-      artifactId: string;
-    }
   | {
       type: "logs";
     }

apps/array/src/renderer/features/sidebar/hooks/useSidebarData.ts

@@ -57,7 +57,7 @@ export interface SidebarData {
 }
 
 interface ViewState {
-  type: "task-list" | "task-detail" | "task-input" | "settings";
+  type: "task-detail" | "task-input" | "settings";
   data?: Task;
 }
 
@@ -155,11 +155,8 @@ function createTaskViews(currentUser: Schemas.User | undefined): TaskView[] {
 
 function getActiveViewId(
   views: TaskView[],
-  activeView: ViewState,
   activeFilters: ActiveFilters,
 ): string | null {
-  if (activeView.type !== "task-list") return null;
-
   for (const view of views) {
     if (filtersMatch(activeFilters, view.filters)) {
       return view.id;
@@ -191,7 +188,7 @@ export function useSidebarData({
   const isHomeActive = activeView.type === "task-input";
 
   const views = createTaskViews(currentUser);
-  const activeViewId = getActiveViewId(views, activeView, activeFilters);
+  const activeViewId = getActiveViewId(views, activeFilters);
 
   const repositories = buildRepositoryMap(allTasks);
   const activeRepository = getActiveRepository(activeFilters);