🤖 fix: remove dynamic imports and fix linting issues

Author: kylecarbs

src/components/AppLoader.tsx

@@ -4,7 +4,7 @@ import { LoadingScreen } from "./LoadingScreen";
 import { useWorkspaceStoreRaw } from "../stores/WorkspaceStore";
 import { useGitStatusStoreRaw } from "../stores/GitStatusStore";
 import { AppProvider } from "../contexts/AppContext";
-import { ProjectProvider, useProjectContext } from "../contexts/ProjectContext";
+import { ProjectProvider } from "../contexts/ProjectContext";
 import { WorkspaceProvider, useWorkspaceContext } from "../contexts/WorkspaceContext";
 
 /**

src/contexts/WorkspaceContext.test.tsx

@@ -6,12 +6,12 @@
 /* eslint-disable @typescript-eslint/consistent-type-assertions */
 import type { FrontendWorkspaceMetadata } from "@/types/workspace";
 import type { IPCApi } from "@/types/ipc";
-import type { ProjectConfig } from "@/config";
 import { act, cleanup, render, waitFor } from "@testing-library/react";
 import { afterEach, describe, expect, mock, test } from "bun:test";
 import { GlobalWindow } from "happy-dom";
 import type { WorkspaceContext } from "./WorkspaceContext";
 import { WorkspaceProvider, useWorkspaceContext } from "./WorkspaceContext";
+import { ProjectProvider } from "@/contexts/ProjectContext";
 
 // Helper to create test workspace metadata with default runtime config
 const createWorkspaceMetadata = (
@@ -66,7 +66,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().workspaceMetadata.size).toBe(2));
@@ -85,7 +84,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     // Should have empty workspaces after failed load
@@ -131,7 +129,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().workspaceMetadata.size).toBe(1));
@@ -167,7 +164,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -198,7 +194,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -229,7 +224,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -276,7 +270,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -329,7 +322,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -382,7 +374,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -421,7 +412,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -440,7 +430,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -477,7 +466,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().loading).toBe(false));
@@ -530,7 +518,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().workspaceMetadata.has("ws-1")).toBe(true));
@@ -559,7 +546,6 @@ describe("WorkspaceContext", () => {
       list: () => Promise.resolve([]),
     });
 
-
     const ctx = await setup();
 
     await waitFor(() => expect(ctx().workspaceMetadata.size).toBe(1));
@@ -575,10 +561,8 @@ async function setup() {
     contextRef.current = useWorkspaceContext();
     return null;
   }
-  
+
   // WorkspaceProvider needs ProjectProvider to call useProjectContext
-  const { ProjectProvider } = await import("@/contexts/ProjectContext");
-  
   render(
     <ProjectProvider>
       <WorkspaceProvider>

src/contexts/WorkspaceContext.tsx

@@ -257,7 +257,9 @@ export function WorkspaceProvider(props: WorkspaceProviderProps) {
         throw new Error(result.error);
       }
     },
-    [props]
+    // refreshProjects is stable from context, doesn't need to be in deps
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+    [loadWorkspaceMetadata]
   );
 
   const removeWorkspace = useCallback(