move lsp diagnostics routes to tui.ts to avoid type instantiation depth limit error in server.ts

lsp_diagnostics defaults to "true"

Author: itsrainingmani

packages/app/src/context/global-sync.tsx

@@ -52,7 +52,7 @@ type State = {
     [name: string]: McpStatus
   }
   lsp: LspStatus[]
-  lsp_diagnostics: boolean | undefined
+  lsp_diagnostics: boolean
   vcs: VcsInfo | undefined
   limit: number
   message: {
@@ -99,7 +99,7 @@ function createGlobalSync() {
         permission: {},
         mcp: {},
         lsp: [],
-        lsp_diagnostics: undefined,
+        lsp_diagnostics: true,
         vcs: undefined,
         limit: 5,
         message: {},
@@ -174,7 +174,7 @@ function createGlobalSync() {
           loadSessions(directory),
           sdk.mcp.status().then((x) => setStore("mcp", x.data!)),
           sdk.lsp.status().then((x) => setStore("lsp", x.data!)),
-          sdk.lsp.diagnostics.status().then((x) => setStore("lsp_diagnostics", x.data?.enabled)),
+          sdk.lsp.diagnostics.status().then((x) => setStore("lsp_diagnostics", x.data?.enabled ?? true)),
           sdk.vcs.get().then((x) => setStore("vcs", x.data)),
           sdk.permission.list().then((x) => {
             const grouped: Record<string, PermissionRequest[]> = {}

packages/app/src/pages/session.tsx

@@ -443,11 +443,13 @@ export default function Page() {
       onSelect: async () => {
         await sdk.client.lsp.diagnostics.toggle()
         const status = await sdk.client.lsp.diagnostics.status()
-        sync.set("lsp_diagnostics", status.data?.enabled)
-        showToast({
-          title: "LSP Diagnostics",
-          description: `Diagnostics ${status.data?.enabled ? "enabled" : "disabled"}`,
-        })
+        if (status.data) {
+          sync.set("lsp_diagnostics", status.data.enabled)
+          showToast({
+            title: "LSP Diagnostics",
+            description: `Diagnostics ${status.data.enabled ? "enabled" : "disabled"}`,
+          })
+        }
       },
     },
     {

packages/opencode/src/cli/cmd/tui/app.tsx

@@ -372,12 +372,14 @@ function App() {
       onSelect: async () => {
         await sdk.client.lsp.diagnostics.toggle()
         const status = await sdk.client.lsp.diagnostics.status()
-        sync.set("lsp_diagnostics", status.data?.enabled)
-        toast.show({
-          variant: "info",
-          message: `LSP diagnostics ${status.data?.enabled ? "enabled" : "disabled"}`,
-          duration: 2000,
-        })
+        if (status.data) {
+          sync.set("lsp_diagnostics", status.data.enabled)
+          toast.show({
+            variant: "info",
+            message: `LSP diagnostics ${status.data.enabled ? "enabled" : "disabled"}`,
+            duration: 2000,
+          })
+        }
       },
     },
     {

packages/opencode/src/cli/cmd/tui/context/sync.tsx

@@ -60,7 +60,7 @@ export const { use: useSync, provider: SyncProvider } = createSimpleContext({
         [messageID: string]: Part[]
       }
       lsp: LspStatus[]
-      lsp_diagnostics: boolean | undefined
+      lsp_diagnostics: boolean
       mcp: {
         [key: string]: McpStatus
       }
@@ -91,7 +91,7 @@ export const { use: useSync, provider: SyncProvider } = createSimpleContext({
       message: {},
       part: {},
       lsp: [],
-      lsp_diagnostics: undefined,
+      lsp_diagnostics: true,
       mcp: {},
       mcp_resource: {},
       formatter: [],
@@ -302,7 +302,7 @@ export const { use: useSync, provider: SyncProvider } = createSimpleContext({
             ...(args.continue ? [] : [sessionListPromise]),
             sdk.client.command.list().then((x) => setStore("command", reconcile(x.data ?? []))),
             sdk.client.lsp.status().then((x) => setStore("lsp", reconcile(x.data!))),
-            sdk.client.lsp.diagnostics.status().then((x) => setStore("lsp_diagnostics", x.data?.enabled)),
+            sdk.client.lsp.diagnostics.status().then((x) => setStore("lsp_diagnostics", x.data?.enabled ?? true)),
             sdk.client.mcp.status().then((x) => setStore("mcp", reconcile(x.data!))),
             sdk.client.experimental.resource.list().then((x) => setStore("mcp_resource", reconcile(x.data ?? {}))),
             sdk.client.formatter.status().then((x) => setStore("formatter", reconcile(x.data!))),

packages/opencode/src/server/server.ts

@@ -51,6 +51,7 @@ import { PermissionNext } from "@/permission/next"
 import { Installation } from "@/installation"
 import { MDNS } from "./mdns"
 import { Worktree } from "../worktree"
+import ignore from "ignore"
 
 // @ts-ignore This global is needed to prevent ai-sdk from logging warnings to stdout https://github.com/vercel/ai/blob/2dc67e0ef538307f21368db32d5a12345d98831b/packages/ai/src/logger/log-warnings.ts#L85
 globalThis.AI_SDK_LOG_WARNINGS = false
@@ -2403,49 +2404,6 @@ export namespace Server {
           return c.json(await LSP.status())
         },
       )
-      .get(
-        "/lsp/diagnostics/status",
-        describeRoute({
-          summary: "Get LSP diagnostics toggle status",
-          description: "Returns the current state of LSP diagnostics toggle",
-          operationId: "lsp.diagnostics.status",
-          responses: {
-            200: {
-              description: "LSP diagnostics toggle status",
-              content: {
-                "application/json": {
-                  schema: resolver(LSP.DiagnosticsStatus),
-                },
-              },
-            },
-          },
-        }),
-        async (c) => {
-          return c.json(await LSP.diagnosticsStatus())
-        },
-      )
-      .post(
-        "/lsp/diagnostics/toggle",
-        describeRoute({
-          summary: "Toggle LSP diagnostics",
-          description: "Toggle whether LSP diagnostics are sent to the model",
-          operationId: "lsp.diagnostics.toggle",
-          responses: {
-            200: {
-              description: "Updated diagnostics status",
-              content: {
-                "application/json": {
-                  schema: resolver(LSP.DiagnosticsStatus),
-                },
-              },
-            },
-          },
-        }),
-        async (c) => {
-          const enabled = await LSP.toggleDiagnostics()
-          return c.json({ enabled })
-        },
-      )
       .get(
         "/formatter",
         describeRoute({

packages/opencode/src/server/tui.ts

@@ -2,6 +2,7 @@ import { Hono, type Context } from "hono"
 import { describeRoute, resolver, validator } from "hono-openapi"
 import { z } from "zod"
 import { AsyncQueue } from "../util/queue"
+import { LSP } from "@/lsp"
 
 const TuiRequest = z.object({
   path: z.string(),
@@ -69,3 +70,46 @@ export const TuiRoute = new Hono()
       return c.json(true)
     },
   )
+  .get(
+    "/lsp/diagnostics/status",
+    describeRoute({
+      summary: "Get LSP diagnostics toggle status",
+      description: "Returns the current state of LSP diagnostics toggle",
+      operationId: "lsp.diagnostics.status",
+      responses: {
+        200: {
+          description: "LSP diagnostics toggle status",
+          content: {
+            "application/json": {
+              schema: resolver(LSP.DiagnosticsStatus),
+            },
+          },
+        },
+      },
+    }),
+    async (c) => {
+      return c.json(await LSP.diagnosticsStatus())
+    },
+  )
+  .post(
+    "/lsp/diagnostics/toggle",
+    describeRoute({
+      summary: "Toggle LSP diagnostics",
+      description: "Toggle whether LSP diagnostics are sent to the model",
+      operationId: "lsp.diagnostics.toggle",
+      responses: {
+        200: {
+          description: "Updated diagnostics status",
+          content: {
+            "application/json": {
+              schema: resolver(LSP.DiagnosticsStatus),
+            },
+          },
+        },
+      },
+    }),
+    async (c) => {
+      const enabled = await LSP.toggleDiagnostics()
+      return c.json({ enabled })
+    },
+  )

packages/sdk/js/src/v2/gen/sdk.gen.ts

@@ -2498,7 +2498,7 @@ export class Diagnostics extends HeyApiClient {
   ) {
     const params = buildClientParams([parameters], [{ args: [{ in: "query", key: "directory" }] }])
     return (options?.client ?? this.client).get<LspDiagnosticsStatusResponses, unknown, ThrowOnError>({
-      url: "/lsp/diagnostics/status",
+      url: "/tui/control/lsp/diagnostics/status",
       ...options,
       ...params,
     })
@@ -2517,7 +2517,7 @@ export class Diagnostics extends HeyApiClient {
   ) {
     const params = buildClientParams([parameters], [{ args: [{ in: "query", key: "directory" }] }])
     return (options?.client ?? this.client).post<LspDiagnosticsToggleResponses, unknown, ThrowOnError>({
-      url: "/lsp/diagnostics/toggle",
+      url: "/tui/control/lsp/diagnostics/toggle",
       ...options,
       ...params,
     })

packages/sdk/js/src/v2/gen/types.gen.ts

@@ -1980,16 +1980,16 @@ export type LspStatus = {
   status: "connected" | "error"
 }
 
-export type LspDiagnosticsStatus = {
-  enabled: boolean
-}
-
 export type FormatterStatus = {
   name: string
   extensions: Array<string>
   enabled: boolean
 }
 
+export type LspDiagnosticsStatus = {
+  enabled: boolean
+}
+
 export type OAuth = {
   type: "oauth"
   refresh: string
@@ -4238,42 +4238,6 @@ export type LspStatusResponses = {
 
 export type LspStatusResponse = LspStatusResponses[keyof LspStatusResponses]
 
-export type LspDiagnosticsStatusData = {
-  body?: never
-  path?: never
-  query?: {
-    directory?: string
-  }
-  url: "/lsp/diagnostics/status"
-}
-
-export type LspDiagnosticsStatusResponses = {
-  /**
-   * LSP diagnostics toggle status
-   */
-  200: LspDiagnosticsStatus
-}
-
-export type LspDiagnosticsStatusResponse = LspDiagnosticsStatusResponses[keyof LspDiagnosticsStatusResponses]
-
-export type LspDiagnosticsToggleData = {
-  body?: never
-  path?: never
-  query?: {
-    directory?: string
-  }
-  url: "/lsp/diagnostics/toggle"
-}
-
-export type LspDiagnosticsToggleResponses = {
-  /**
-   * Updated diagnostics status
-   */
-  200: LspDiagnosticsStatus
-}
-
-export type LspDiagnosticsToggleResponse = LspDiagnosticsToggleResponses[keyof LspDiagnosticsToggleResponses]
-
 export type FormatterStatusData = {
   body?: never
   path?: never
@@ -4586,6 +4550,42 @@ export type TuiControlResponseResponses = {
 
 export type TuiControlResponseResponse = TuiControlResponseResponses[keyof TuiControlResponseResponses]
 
+export type LspDiagnosticsStatusData = {
+  body?: never
+  path?: never
+  query?: {
+    directory?: string
+  }
+  url: "/tui/control/lsp/diagnostics/status"
+}
+
+export type LspDiagnosticsStatusResponses = {
+  /**
+   * LSP diagnostics toggle status
+   */
+  200: LspDiagnosticsStatus
+}
+
+export type LspDiagnosticsStatusResponse = LspDiagnosticsStatusResponses[keyof LspDiagnosticsStatusResponses]
+
+export type LspDiagnosticsToggleData = {
+  body?: never
+  path?: never
+  query?: {
+    directory?: string
+  }
+  url: "/tui/control/lsp/diagnostics/toggle"
+}
+
+export type LspDiagnosticsToggleResponses = {
+  /**
+   * Updated diagnostics status
+   */
+  200: LspDiagnosticsStatus
+}
+
+export type LspDiagnosticsToggleResponse = LspDiagnosticsToggleResponses[keyof LspDiagnosticsToggleResponses]
+
 export type AuthSetData = {
   body?: Auth
   path: {