Phase C: CopilotClientOptions / MCP / streaming shape changes

- Remove autoStart and autoRestart from CopilotClientOptions. The client
  now always starts on first createSession/resumeSession; users can still
  call client.start() explicitly for eager startup.
- Make MCPServerConfigBase.tools optional (undefined = all, [] = none).
- Fix streaming JSDoc block comment that wasn't attached due to single-star.

Mirrors C# PR #1343 Phase 4c.

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

Author: SteveSandersonMS

nodejs/README.md

@@ -415,7 +415,7 @@ Note: `assistant.message` and `assistant.reasoning` (final events) are always se
 ### Manual Server Control
 
 ```typescript
-const client = new CopilotClient({ autoStart: false });
+const client = new CopilotClient({ });
 
 // Start manually
 await client.start();

nodejs/src/client.ts

@@ -379,8 +379,6 @@ export class CopilotClient {
             isChildProcess: options.isChildProcess ?? false,
             cliUrl: options.cliUrl,
             logLevel: options.logLevel || "debug",
-            autoStart: options.autoStart ?? true,
-            autoRestart: false,
 
             env: effectiveEnv,
             gitHubToken: options.gitHubToken,
@@ -465,14 +463,14 @@ export class CopilotClient {
      * If connecting to an external server (via cliUrl), only establishes the connection.
      * Otherwise, spawns the CLI server process and then connects.
      *
-     * This method is called automatically when creating a session if `autoStart` is true (default).
+     * This method is called automatically the first time you create or resume a session.
      *
      * @returns A promise that resolves when the connection is established
      * @throws Error if the server fails to start or the connection fails
      *
      * @example
      * ```typescript
-     * const client = new CopilotClient({ autoStart: false });
+     * const client = new CopilotClient();
      * await client.start();
      * // Now ready to create sessions
      * ```
@@ -707,12 +705,11 @@ export class CopilotClient {
      * Creates a new conversation session with the Copilot CLI.
      *
      * Sessions maintain conversation state, handle events, and manage tool execution.
-     * If the client is not connected and `autoStart` is enabled, this will automatically
-     * start the connection.
+     * If the client is not connected, this method automatically starts the connection.
      *
      * @param config - Optional configuration for the session
      * @returns A promise that resolves with the created session
-     * @throws Error if the client is not connected and autoStart is disabled
+     * @throws Error if the client fails to start
      *
      * @example
      * ```typescript
@@ -734,11 +731,7 @@ export class CopilotClient {
      */
     async createSession(config: SessionConfig): Promise<CopilotSession> {
         if (!this.connection) {
-            if (this.options.autoStart) {
-                await this.start();
-            } else {
-                throw new Error("Client not connected. Call start() first.");
-            }
+            await this.start();
         }
 
         const sessionId = config.sessionId ?? randomUUID();
@@ -874,11 +867,7 @@ export class CopilotClient {
      */
     async resumeSession(sessionId: string, config: ResumeSessionConfig): Promise<CopilotSession> {
         if (!this.connection) {
-            if (this.options.autoStart) {
-                await this.start();
-            } else {
-                throw new Error("Client not connected. Call start() first.");
-            }
+            await this.start();
         }
 
         // Create and register the session before issuing the RPC so that

nodejs/src/types.ts

@@ -119,17 +119,6 @@ export interface CopilotClientOptions {
      */
     logLevel?: "none" | "error" | "warning" | "info" | "debug" | "all";
 
-    /**
-     * Auto-start the CLI server on first use
-     * @default true
-     */
-    autoStart?: boolean;
-
-    /**
-     * @deprecated This option has no effect and will be removed in a future release.
-     */
-    autoRestart?: boolean;
-
     /**
      * Environment variables to pass to the CLI process. If not set, inherits process.env.
      */
@@ -1145,9 +1134,11 @@ export interface SessionHooks {
  */
 interface MCPServerConfigBase {
     /**
-     * List of tools to include from this server. [] means none. "*" means all.
+     * List of tools to include from this server.
+     * `undefined` (the default) or `"*"` means include all tools.
+     * `[]` means include none.
      */
-    tools: string[];
+    tools?: string[];
     /**
      * Indicates the server type: "stdio" for local/subprocess servers, "http"/"sse" for remote servers.
      * If not specified, defaults to "stdio".
@@ -1433,7 +1424,7 @@ export interface SessionConfig {
      */
     workingDirectory?: string;
 
-    /*
+    /**
      * Enable streaming of assistant message and reasoning chunks.
      * When true, ephemeral assistant.message_delta and assistant.reasoning_delta
      * events are sent as the response is generated. Clients should accumulate

nodejs/test/cjs-compat.test.ts

@@ -43,7 +43,7 @@ describe("Dual ESM/CJS build (#528)", () => {
     it("CJS build resolves bundled CLI path", () => {
         const script = `
             const sdk = require(${JSON.stringify(join(distDir, "cjs/index.js"))});
-            const client = new sdk.CopilotClient({ autoStart: false });
+            const client = new sdk.CopilotClient({ });
             console.log('CJS CLI resolved: OK');
         `;
         const output = execFileSync(process.execPath, ["--eval", script], {
@@ -59,7 +59,7 @@ describe("Dual ESM/CJS build (#528)", () => {
         const script = `
             import { pathToFileURL } from 'node:url';
             const sdk = await import(pathToFileURL(${JSON.stringify(esmPath)}).href);
-            const client = new sdk.CopilotClient({ autoStart: false });
+            const client = new sdk.CopilotClient({ });
             console.log('ESM CLI resolved: OK');
         `;
         const output = execFileSync(process.execPath, ["--input-type=module", "--eval", script], {

nodejs/test/client.test.ts

@@ -8,13 +8,13 @@ import { defaultJoinSessionPermissionHandler } from "../src/types.js";
 
 describe("CopilotClient", () => {
     it("allows createSession without onPermissionRequest", async () => {
-        const client = new CopilotClient({ autoStart: false });
+        const client = new CopilotClient({ });
 
         await expect(client.createSession({})).rejects.toThrow(/Client not connected/);
     });
 
     it("allows resumeSession without onPermissionRequest", async () => {
-        const client = new CopilotClient({ autoStart: false });
+        const client = new CopilotClient({ });
 
         await expect(client.resumeSession("session-1", {})).rejects.toThrow(/Client not connected/);
     });

nodejs/test/e2e/client_options.e2e.test.ts

@@ -144,8 +144,7 @@ describe("Client options", async () => {
         const client = new CopilotClient({
             cwd: workDir,
             env,
-            cliPath: process.env.COPILOT_CLI_PATH,
-            autoStart: false,
+            cliPath: process.env.COPILOT_CLI_PATH
         });
         onTestFinished(async () => {
             try {
@@ -247,7 +246,6 @@ describe("Client options", async () => {
         const client = new CopilotClient({
             cwd: workDir,
             env: { ...env, COPILOT_HOME: copilotHomeFromEnv },
-            autoStart: false,
             cliPath,
             cliArgs: ["--capture-file", capturePath],
             copilotHome: copilotHomeFromOption,