Update problems report when run type check tools is run (#2212)

e2e-tests/fixtures/engine/local-agent/run-type-checks.ts

+import type { LocalAgentFixture } from "../../../../testing/fake-llm-server/localAgentTypes";
+
+export const fixture: LocalAgentFixture = {
+  description:
+    "Write a file with TypeScript errors and then run type checks to update the Problems panel",
+  turns: [
+    {
+      text: "I'll create a file with some TypeScript errors for testing.",
+      toolCalls: [
+        {
+          name: "write_file",
+          args: {
+            path: "src/test-errors.ts",
+            content: `// File with TypeScript errors for testing
+const x: number = "not a number";
+const y: string = 123;
+nonExistentFunction();
+`,
+            description: "Create file with type errors",
+          },
+        },
+      ],
+    },
+    {
+      text: "Now let me run type checks to identify the errors.",
+      toolCalls: [
+        {
+          name: "run_type_checks",
+          args: {
+            paths: ["src/test-errors.ts"],
+          },
+        },
+      ],
+    },
+    {
+      text: "The type check found the errors. The Problems panel should now show these TypeScript errors.",
+    },
+  ],
+};

e2e-tests/local_agent_run_type_checks.spec.ts

+import { testSkipIfWindows, Timeout } from "./helpers/test_helper";
+import { expect } from "@playwright/test";
+
+/**
+ * E2E test for run_type_checks tool in local-agent mode
+ * Tests that running type checks updates the Problems panel in the UI
+ */
+
+testSkipIfWindows(
+  "local-agent - run_type_checks updates problems panel",
+  async ({ po }) => {
+    await po.setUpDyadPro({ localAgent: true });
+    await po.importApp("minimal");
+    await po.selectLocalAgentMode();
+
+    // Ensure pnpm install has run so TypeScript is available
+    await po.ensurePnpmInstall();
+
+    // Switch to Problems panel first to observe the update
+    await po.selectPreviewMode("problems");
+
+    // Initially there should be no problems
+    const fixButton = po.page.getByTestId("fix-all-button");
+    // The button may not exist if there are no problems, so we check for the "No problems found" text
+    await expect(
+      po.page.getByText(/No problems found|No Problems Report/),
+    ).toBeVisible({ timeout: Timeout.MEDIUM });
+
+    // Send prompt that triggers write_file with TS errors, then run_type_checks
+    await po.sendPrompt("tc=local-agent/run-type-checks");
+
+    // After the agent runs type checks, the Problems panel should show errors
+    // Wait for the fix button to be enabled and show errors
+    await expect(fixButton).toBeEnabled({ timeout: Timeout.LONG });
+    await expect(fixButton).toContainText(/Fix \d+ problem/);
+
+    // Verify the problems are displayed
+    const problemRows = po.page.getByTestId("problem-row");
+    await expect(problemRows.first()).toBeVisible({ timeout: Timeout.MEDIUM });
+
+    // Take a snapshot of the problems pane for regression testing
+    await po.snapshotProblemsPane();
+  },
+);

e2e-tests/snapshots/local_agent_run_type_checks.spec.ts_local-agent---run-type-checks-updates-problems-panel-1.aria.yml

+- img
+- text: 3 errors
+- button "Run checks":
+  - img
+- button "Clear all"
+- button "Fix 3 problems":
+  - img
+- checkbox "Select problem src/test-errors.ts 2:7 Type 'string' is not assignable to type 'number'." [checked]:
+  - checkbox "Select problem" [checked]:
+    - img
+  - img
+  - img
+  - paragraph: Type 'string' is not assignable to type 'number'.
+- checkbox "Select problem src/test-errors.ts 3:7 Type 'number' is not assignable to type 'string'." [checked]:
+  - checkbox "Select problem" [checked]:
+    - img
+  - img
+  - img
+  - paragraph: Type 'number' is not assignable to type 'string'.
+- checkbox "Select problem src/test-errors.ts 4:1 Cannot find name 'nonExistentFunction'." [checked]:
+  - checkbox "Select problem" [checked]:
+    - img
+  - img
+  - img
+  - paragraph: Cannot find name 'nonExistentFunction'.
\ No newline at end of file

src/ipc/ipc_client.ts

@@ -83,6 +83,7 @@ import type {
   AgentToolConsentRequestPayload,
   AgentToolConsentResponseParams,
   AgentTodosUpdatePayload,
+  AgentProblemsUpdatePayload,
   TelemetryEventPayload,
   GithubSyncOptions,
   ConsoleEntry,
@@ -145,6 +146,9 @@ export class IpcClient {
   private mcpConsentHandlers: Map<string, (payload: any) => void>;
   private agentConsentHandlers: Map<string, (payload: any) => void>;
   private agentTodosHandlers: Set<(payload: AgentTodosUpdatePayload) => void>;
+  private agentProblemsHandlers: Set<
+    (payload: AgentProblemsUpdatePayload) => void
+  >;
   private telemetryEventHandlers: Set<(payload: TelemetryEventPayload) => void>;
   // Global handlers called for any chat stream start (used for cleanup)
   private globalChatStreamStartHandlers: Set<(chatId: number) => void>;
@@ -158,6 +162,7 @@ export class IpcClient {
     this.mcpConsentHandlers = new Map();
     this.agentConsentHandlers = new Map();
     this.agentTodosHandlers = new Set();
+    this.agentProblemsHandlers = new Set();
     this.telemetryEventHandlers = new Set();
     this.globalChatStreamStartHandlers = new Set();
     this.globalChatStreamEndHandlers = new Set();
@@ -315,6 +320,13 @@ export class IpcClient {
       }
     });
 
+    // Agent problems update from main
+    this.ipcRenderer.on("agent-tool:problems-update", (payload) => {
+      for (const handler of this.agentProblemsHandlers) {
+        handler(payload as unknown as AgentProblemsUpdatePayload);
+      }
+    });
+
     // Telemetry events from main to renderer
     this.ipcRenderer.on("telemetry:event", (payload) => {
       if (payload && typeof payload === "object" && "eventName" in payload) {
@@ -1151,6 +1163,19 @@ export class IpcClient {
     };
   }
 
+  /**
+   * Subscribe to agent problems updates from the local agent.
+   * Called when the agent runs type checks and updates the problems report.
+   */
+  public onAgentProblemsUpdate(
+    handler: (payload: AgentProblemsUpdatePayload) => void,
+  ) {
+    this.agentProblemsHandlers.add(handler);
+    return () => {
+      this.agentProblemsHandlers.delete(handler);
+    };
+  }
+
   /**
    * Subscribe to be notified when any chat stream starts.
    * Useful for cleanup tasks like clearing pending consent requests.

src/ipc/ipc_types.ts

@@ -732,6 +732,11 @@ export interface AgentTodosUpdatePayload {
   todos: AgentTodo[];
 }
 
+export interface AgentProblemsUpdatePayload {
+  appId: number;
+  problems: ProblemReport;
+}
+
 export interface TelemetryEventPayload {
   eventName: string;
   properties?: Record<string, unknown>;

src/preload.ts

@@ -199,6 +199,8 @@ const validReceiveChannels = [
   "agent-tool:consent-request",
   // Agent todos update from main to renderer
   "agent-tool:todos-update",
+  // Agent problems update from main to renderer
+  "agent-tool:problems-update",
   // Telemetry events from main to renderer
   "telemetry:event",
 ] as const;

src/pro/main/ipc/handlers/local_agent/local_agent_handler.ts

@@ -164,6 +164,7 @@ export async function handleLocalAgentStream(
     // Build tool execute context
     const ctx: AgentContext = {
       event,
+      appId: chat.app.id,
       appPath,
       chatId: chat.id,
       supabaseProjectId: chat.app.supabaseProjectId,

src/pro/main/ipc/handlers/local_agent/tools/run_type_checks.ts

@@ -7,6 +7,7 @@ import {
 } from "./types";
 import { generateProblemReport } from "@/ipc/processors/tsc";
 import type { Problem } from "@/ipc/ipc_types";
+import { safeSend } from "@/ipc/utils/safe_sender";
 
 import { normalizePath } from "../../../../../../../shared/normalizePath";
 
@@ -98,6 +99,12 @@ export const runTypeChecksTool: ToolDefinition<
       appPath: ctx.appPath,
     });
 
+    // Send the full problem report to update the Problems panel in the UI
+    safeSend(ctx.event.sender, "agent-tool:problems-update", {
+      appId: ctx.appId,
+      problems: problemReport,
+    });
+
     let problems = problemReport.problems;
 
     // Filter by paths if specified

src/pro/main/ipc/handlers/local_agent/tools/types.ts

@@ -33,6 +33,7 @@ export type Todo = AgentTodo;
 
 export interface AgentContext {
   event: IpcMainInvokeEvent;
+  appId: number;
   appPath: string;
   chatId: number;
   supabaseProjectId: string | null;

src/renderer.tsx

@@ -198,6 +198,15 @@ function App() {
     return () => unsubscribe();
   }, []);
 
+  // Agent problems updates - update the TanStack Query cache when the agent runs type checks
+  useEffect(() => {
+    const ipc = IpcClient.getInstance();
+    const unsubscribe = ipc.onAgentProblemsUpdate((payload) => {
+      queryClient.setQueryData(["problems", payload.appId], payload.problems);
+    });
+    return () => unsubscribe();
+  }, []);
+
   return <RouterProvider router={router} />;
 }