🤖 fix: resolve eslint warnings in WorkspaceContext

- Add eslint disable comments for test file mocks (consistent with PR #600 pattern)
- Fix react-hooks/exhaustive-deps warnings by destructuring props in effects
- Change empty arrow function to use comment (no-op setWorkspaceMetadata)
- Remove await from non-promise expect call in tests

Author: kylecarbs

src/components/AppLoader.tsx

@@ -83,6 +83,7 @@ function AppLoaderInner(props: {
   const [hasRestoredFromHash, setHasRestoredFromHash] = useState(false);
 
   useEffect(() => {
+    const { setSelectedWorkspace } = props;
     // Wait until stores are synced before attempting restoration
     if (!storesSynced) return;
 
@@ -98,7 +99,7 @@ function AppLoaderInner(props: {
 
       if (metadata) {
         // Restore from hash (overrides localStorage)
-        props.setSelectedWorkspace({
+        setSelectedWorkspace({
           workspaceId: metadata.id,
           projectPath: metadata.projectPath,
           projectName: metadata.projectName,
@@ -108,21 +109,17 @@ function AppLoaderInner(props: {
     }
 
     setHasRestoredFromHash(true);
-  }, [
-    storesSynced,
-    workspaceContext.workspaceMetadata,
-    hasRestoredFromHash,
-    props.setSelectedWorkspace,
-  ]);
+  }, [storesSynced, workspaceContext.workspaceMetadata, hasRestoredFromHash, props]);
 
   // Check for launch project from server (for --add-project flag)
   // This only applies in server mode
   useEffect(() => {
+    const { selectedWorkspace, setSelectedWorkspace } = props;
     // Wait until stores are synced and hash restoration is complete
     if (!storesSynced || !hasRestoredFromHash) return;
 
     // Skip if we already have a selected workspace (from localStorage or URL hash)
-    if (props.selectedWorkspace) return;
+    if (selectedWorkspace) return;
 
     // Only check once
     const checkLaunchProject = async () => {
@@ -140,7 +137,7 @@ function AppLoaderInner(props: {
       if (projectWorkspaces.length > 0) {
         // Select the first workspace in the project
         const metadata = projectWorkspaces[0];
-        props.setSelectedWorkspace({
+        setSelectedWorkspace({
           workspaceId: metadata.id,
           projectPath: metadata.projectPath,
           projectName: metadata.projectName,
@@ -152,13 +149,7 @@ function AppLoaderInner(props: {
     };
 
     void checkLaunchProject();
-  }, [
-    storesSynced,
-    hasRestoredFromHash,
-    props.selectedWorkspace,
-    workspaceContext.workspaceMetadata,
-    props.setSelectedWorkspace,
-  ]);
+  }, [storesSynced, hasRestoredFromHash, workspaceContext.workspaceMetadata, props]);
 
   // Show loading screen until stores are synced
   if (workspaceContext.loading || !storesSynced) {
@@ -173,7 +164,9 @@ function AppLoaderInner(props: {
       addProject={props.addProject}
       removeProject={props.removeProject}
       workspaceMetadata={workspaceContext.workspaceMetadata}
-      setWorkspaceMetadata={() => {}} // This is a no-op now since WorkspaceContext handles it
+      setWorkspaceMetadata={() => {
+        /* no-op now since WorkspaceContext handles it */
+      }}
       createWorkspace={workspaceContext.createWorkspace}
       removeWorkspace={workspaceContext.removeWorkspace}
       renameWorkspace={workspaceContext.renameWorkspace}

src/contexts/WorkspaceContext.test.tsx

@@ -1,3 +1,9 @@
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-unsafe-return */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+/* eslint-disable @typescript-eslint/no-empty-function */
+/* eslint-disable @typescript-eslint/consistent-type-assertions */
 import type { FrontendWorkspaceMetadata } from "@/types/workspace";
 import type { IPCApi } from "@/types/ipc";
 import type { WorkspaceSelection } from "@/components/ProjectSidebar";
@@ -228,7 +234,7 @@ describe("WorkspaceContext", () => {
 
     await waitFor(() => expect(ctx().loading).toBe(false));
 
-    await expect(async () => {
+    expect(async () => {
       await act(async () => {
         await ctx().createWorkspace("/gamma", "feature", "main");
       });
@@ -606,7 +612,7 @@ describe("WorkspaceContext", () => {
   });
 
   test("ensureCreatedAt adds default timestamp when missing", async () => {
-    const workspaceWithoutTimestamp: FrontendWorkspaceMetadata = {
+    const workspaceWithoutTimestamp = {
       id: "ws-1",
       projectPath: "/alpha",
       projectName: "alpha",

src/contexts/WorkspaceContext.tsx

@@ -101,19 +101,21 @@ export function WorkspaceProvider(props: WorkspaceProviderProps) {
 
   // Load metadata once on mount
   useEffect(() => {
+    const { onProjectsUpdate } = props;
     void (async () => {
       await loadWorkspaceMetadata();
       // After loading metadata (which may trigger migration), reload projects
       // to ensure frontend has the updated config with workspace IDs
       const projectsList = await window.api.projects.list();
       const loadedProjects = new Map<string, ProjectConfig>(projectsList);
-      props.onProjectsUpdate(loadedProjects);
+      onProjectsUpdate(loadedProjects);
       setLoading(false);
     })();
-  }, [loadWorkspaceMetadata, props.onProjectsUpdate]);
+  }, [loadWorkspaceMetadata, props]);
 
   // Subscribe to metadata updates (for create/rename/delete operations)
   useEffect(() => {
+    const { onProjectsUpdate } = props;
     const unsubscribe = window.api.workspace.onMetadata(
       (event: { workspaceId: string; metadata: FrontendWorkspaceMetadata | null }) => {
         setWorkspaceMetadata((prev) => {
@@ -134,7 +136,7 @@ export function WorkspaceProvider(props: WorkspaceProviderProps) {
             void (async () => {
               const projectsList = await window.api.projects.list();
               const loadedProjects = new Map<string, ProjectConfig>(projectsList);
-              props.onProjectsUpdate(loadedProjects);
+              onProjectsUpdate(loadedProjects);
             })();
           }
 
@@ -146,7 +148,7 @@ export function WorkspaceProvider(props: WorkspaceProviderProps) {
     return () => {
       unsubscribe();
     };
-  }, [props.onProjectsUpdate]);
+  }, [props]);
 
   const createWorkspace = useCallback(
     async (
@@ -185,7 +187,7 @@ export function WorkspaceProvider(props: WorkspaceProviderProps) {
         throw new Error(result.error);
       }
     },
-    [loadWorkspaceMetadata, props.onProjectsUpdate]
+    [loadWorkspaceMetadata, props]
   );
 
   const removeWorkspace = useCallback(