refactor: rename sessionIdleTimeoutMs to sessionIdleTimeoutSeconds

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: MackinnonBuck

docs/features/session-persistence.md

@@ -433,15 +433,15 @@ await client.deleteSession("user-123-task-456");
 
 ## Automatic Cleanup: Idle Timeout
 
-By default, sessions have **no idle timeout** and live indefinitely until explicitly disconnected or deleted. You can optionally configure a server-wide idle timeout via `CopilotClientOptions.sessionIdleTimeoutMs`:
+By default, sessions have **no idle timeout** and live indefinitely until explicitly disconnected or deleted. You can optionally configure a server-wide idle timeout via `CopilotClientOptions.sessionIdleTimeoutSeconds`:
 
 ```typescript
 const client = new CopilotClient({
-  sessionIdleTimeoutMs: 30 * 60 * 1000, // 30 minutes
+  sessionIdleTimeoutSeconds: 30 * 60, // 30 minutes
 });
 ```
 
-When a timeout is configured, sessions without activity for that duration are automatically cleaned up. The minimum value is 5 minutes (300,000ms). Set to `0` or omit to disable.
+When a timeout is configured, sessions without activity for that duration are automatically cleaned up. The minimum value is 300 seconds (5 minutes). Set to `0` or omit to disable.
 
 > **Note:** This option only applies when the SDK spawns the runtime process. When connecting to an existing server via `cliUrl`, the server's own timeout configuration applies.
 

nodejs/src/client.ts

@@ -339,7 +339,7 @@ export class CopilotClient {
             // Default useLoggedInUser to false when githubToken is provided, otherwise true
             useLoggedInUser: options.useLoggedInUser ?? (options.githubToken ? false : true),
             telemetry: options.telemetry,
-            sessionIdleTimeoutMs: options.sessionIdleTimeoutMs ?? 0,
+            sessionIdleTimeoutSeconds: options.sessionIdleTimeoutSeconds ?? 0,
         };
     }
 
@@ -1386,8 +1386,8 @@ export class CopilotClient {
                 args.push("--no-auto-login");
             }
 
-            if (this.options.sessionIdleTimeoutMs !== undefined && this.options.sessionIdleTimeoutMs > 0) {
-                args.push("--session-idle-timeout", this.options.sessionIdleTimeoutMs.toString());
+            if (this.options.sessionIdleTimeoutSeconds !== undefined && this.options.sessionIdleTimeoutSeconds > 0) {
+                args.push("--session-idle-timeout", this.options.sessionIdleTimeoutSeconds.toString());
             }
 
             // Suppress debug/trace output that might pollute stdout

nodejs/src/types.ts

@@ -184,13 +184,13 @@ export interface CopilotClientOptions {
     sessionFs?: SessionFsConfig;
 
     /**
-     * Server-wide idle timeout for sessions in milliseconds.
+     * Server-wide idle timeout for sessions in seconds.
      * Sessions without activity for this duration are automatically cleaned up.
      * Set to 0 or omit to disable (sessions live indefinitely).
-     * Minimum value: 300000 (5 minutes).
+     * Minimum value: 300 (5 minutes).
      * @default 0 (disabled)
      */
-    sessionIdleTimeoutMs?: number;
+    sessionIdleTimeoutSeconds?: number;
 }
 
 /**