ship: iteration 1 - remove legacy Files tab

Author: arul28

apps/desktop/src/renderer/components/app/App.tsx

@@ -54,8 +54,8 @@ const preloadLanesPage = lanesRoute.preload;
 const filesRoute = createPreloadableRoute<{ active?: boolean }>(() =>
   import("../files/FilesTab").then((m) => ({ default: m.FilesTab }))
 );
-const FilesPage = filesRoute.Component;
-const preloadFilesPage = filesRoute.preload;
+const FilesTab = filesRoute.Component;
+const preloadFilesTab = filesRoute.preload;
 const workRoute = createPreloadableRoute<{ active?: boolean }>(() =>
   import("../terminals/TerminalsPage").then((m) => ({ default: m.TerminalsPage }))
 );
@@ -425,7 +425,7 @@ function ProjectRouteContent({ active, route }: { active: boolean; route: string
           <Route path="/glossary" element={<PageErrorBoundary><GlossaryPage /></PageErrorBoundary>} />
           <Route path="/files" element={
             <PageErrorBoundary>
-              <React.Suspense fallback={LazyFallback}>{React.createElement(FilesPage as React.ComponentType<{ active?: boolean }>, routeProps)}</React.Suspense>
+              <React.Suspense fallback={LazyFallback}>{React.createElement(FilesTab as React.ComponentType<{ active?: boolean }>, routeProps)}</React.Suspense>
             </PageErrorBoundary>
           } />
           <Route path="/graph" element={
@@ -614,7 +614,7 @@ function ProjectTabHost() {
     const preload = () => {
       void preloadTerminalsPage().catch(() => undefined);
       void preloadLanesPage().catch(() => undefined);
-      void preloadFilesPage().catch(() => undefined);
+      void preloadFilesTab().catch(() => undefined);
       void preloadCtoPage().catch(() => undefined);
     };
     const idleWindow = window as Window & {

apps/desktop/src/renderer/components/app/App.workKeepAlive.test.tsx

@@ -117,23 +117,6 @@ vi.mock("../terminals/TerminalsPage", async () => {
   };
 });
 
-vi.mock("../files/FilesPage", async () => {
-  const Router = await vi.importActual("react-router-dom") as typeof RouterNamespace;
-
-  return {
-    FilesPage: () => {
-      const navigate = Router.useNavigate();
-      return (
-        <div data-testid="files-page">
-          <button type="button" onClick={() => navigate("/work")}>
-            Open work
-          </button>
-        </div>
-      );
-    },
-  };
-});
-
 vi.mock("../files/FilesTab", async () => {
   const Router = await vi.importActual("react-router-dom") as typeof RouterNamespace;
 

apps/desktop/src/renderer/components/app/LinearIssueBrowser.tsx

@@ -439,7 +439,9 @@ const LINEAR_MARKDOWN_COMPONENTS: Components = buildChatMarkdownComponents("neut
       href={href}
       onClick={(event) => {
         event.preventDefault();
-        openUrlInAdeBrowser(href);
+        if (typeof href === "string" && href.trim() !== "") {
+          openUrlInAdeBrowser(href);
+        }
       }}
       className="font-medium text-[color:var(--color-accent,#A78BFA)] underline underline-offset-2 transition-opacity hover:opacity-80"
     >
@@ -1184,10 +1186,11 @@ export function LinearIssueBrowser({
                 ) : pageInfo.hasNextPage && (!canAutoLoadIssues || issues.length >= AUTO_LOAD_MAX_ISSUES) ? (
                   <button
                     type="button"
+                    disabled={loadingIssues}
                     className="flex w-full items-center justify-center gap-2 px-3 py-2.5 text-[12px] text-muted-fg/70 transition-colors hover:bg-white/[0.04] hover:text-fg"
                     onClick={() => searchIssues(true)}
                   >
-                    Load more issues
+                    Load more
                   </button>
                 ) : null}
               </>