Add extension setting

Author: royitaqi

lldb/tools/lldb-dap/package.json

@@ -106,6 +106,13 @@
             "markdownDescription": "Run lldb-dap in server mode.\n\nWhen enabled, lldb-dap will start a background server that will be reused between debug sessions. This allows caching of debug symbols between sessions and improves launch performance.",
             "default": false
           },
+          "lldb-dap.connectionTimeout": {
+            "order": 0,
+            "scope": "resource",
+            "type": "number",
+            "markdownDescription": "When running lldb-dap in server mode, the time in seconds to wait for new connections after the server has started and after all clients have disconnected. Each new connection will reset the timeout. When the timeout is reached, the server will be closed and the process will exit. Specifying non-positive values will cause the server to wait for new connections indefinitely.",
+            "default": 0
+          },
           "lldb-dap.arguments": {
             "scope": "resource",
             "type": "array",

lldb/tools/lldb-dap/src-ts/debug-configuration-provider.ts

@@ -207,10 +207,15 @@ export class LLDBDapConfigurationProvider
           config.get<boolean>("serverMode", false) &&
           (await isServerModeSupported(executable.command))
         ) {
+          const connectionTimeoutSeconds = config.get<number | undefined>(
+            "connectionTimeout",
+            undefined,
+          );
           const serverInfo = await this.server.start(
             executable.command,
             executable.args,
             executable.options,
+            connectionTimeoutSeconds,
           );
           if (!serverInfo) {
             return undefined;

lldb/tools/lldb-dap/src-ts/lldb-dap-server.ts

@@ -33,8 +33,19 @@ export class LLDBDapServer implements vscode.Disposable {
     dapPath: string,
     args: string[],
     options?: child_process.SpawnOptionsWithoutStdio,
+    connectionTimeoutSeconds?: number,
   ): Promise<{ host: string; port: number } | undefined> {
-    const dapArgs = [...args, "--connection", "listen://localhost:0"];
+    // Both the --connection and --connection-timeout arguments are subject to the shouldContinueStartup() check.
+    const connectionTimeoutArgs =
+      connectionTimeoutSeconds && connectionTimeoutSeconds > 0
+        ? ["--connection-timeout", `${connectionTimeoutSeconds}`]
+        : [];
+    const dapArgs = [
+      ...args,
+      "--connection",
+      "listen://localhost:0",
+      ...connectionTimeoutArgs,
+    ];
     if (!(await this.shouldContinueStartup(dapPath, dapArgs, options?.env))) {
       return undefined;
     }