Address PR review comments from #1357

- Add missing RuntimeConnection imports to 26 test/scenarios TypeScript
  fixtures. The earlier batch rewrite added the .forStdio/.forUri call
  sites but missed the corresponding import statement, so each scenario
  failed to compile until now.
- nodejs/test/e2e/harness/sdkTestContext.ts: strip the 'kind' property
  before spreading a user-supplied RuntimeConnection back through the
  forStdio/forTcp factory opts. The factory opt types don't accept
  'kind' so the spread was producing excess-property type errors.
- nodejs/src/client.ts: rewrite the 'Path to Copilot CLI is required'
  error message to point at the new connection options
  (RuntimeConnection.forStdio({ path }), forTcp({ path }), forUri(...),
  or the COPILOT_CLI_PATH environment variable). The old message
  referenced removed cliPath / cliUrl options.
- nodejs/src/client.ts: change the default logLevel from 'debug' to
  'info'. 'debug' was a TS-only outlier; Python and Rust default to
  'info', and the README has always claimed 'info'. Go and .NET don't
  pass --log-level at all when omitted (CLI defaults to info anyway),
  so 'info' is consistent with every other SDK's effective default.
- nodejs/src/types.ts: fix MCPServerConfigBase.tools doc comment to
  spell the all-tools sentinel as ['*'] (the actual type is string[],
  so a bare '*' string can't be passed).

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

Author: SteveSandersonMS

nodejs/src/client.ts

@@ -375,7 +375,7 @@ export class CopilotClient {
 
         this.options = {
             cwd: options.cwd ?? process.cwd(),
-            logLevel: options.logLevel || "debug",
+            logLevel: options.logLevel || "info",
             gitHubToken: options.gitHubToken,
             // Default useLoggedInUser to false when gitHubToken is provided, otherwise true.
             useLoggedInUser: options.useLoggedInUser ?? (options.gitHubToken ? false : true),
@@ -1550,7 +1550,11 @@ export class CopilotClient {
 
             if (!this.resolvedCliPath) {
                 throw new Error(
-                    "Path to Copilot CLI is required. Please provide it via the cliPath option, or use cliUrl to rely on a remote CLI."
+                    "Path to Copilot CLI is required. Please supply it via " +
+                        "`RuntimeConnection.forStdio({ path })` or " +
+                        "`RuntimeConnection.forTcp({ path })`, set the COPILOT_CLI_PATH " +
+                        "environment variable, or use `RuntimeConnection.forUri(...)` to " +
+                        "connect to an already-running runtime."
                 );
             }
 

nodejs/src/types.ts

@@ -1211,7 +1211,7 @@ export interface SessionHooks {
 interface MCPServerConfigBase {
     /**
      * List of tools to include from this server.
-     * `undefined` (the default) or `"*"` means include all tools.
+     * `undefined` (the default) or `["*"]` means include all tools.
      * `[]` means include none.
      */
     tools?: string[];

nodejs/test/e2e/harness/sdkTestContext.ts

@@ -70,16 +70,20 @@ export async function createSdkTestContext({
     let connection: RuntimeConnection;
     if (userConn) {
         // Caller supplied a RuntimeConnection — merge in the harness-managed
-        // CLI path (and stay on TCP if the caller asked for that variant).
+        // CLI path (and stay on the same transport variant). Strip `kind`
+        // before forwarding to the factory opts since the factories don't
+        // accept it in their argument shape.
         if (userConn.kind === "tcp") {
+            const { kind: _k, ...tcp } = userConn;
             connection = RuntimeConnection.forTcp({
-                ...userConn,
-                path: userConn.path ?? process.env.COPILOT_CLI_PATH,
+                ...tcp,
+                path: tcp.path ?? process.env.COPILOT_CLI_PATH,
             });
         } else if (userConn.kind === "stdio") {
+            const { kind: _k, ...stdio } = userConn;
             connection = RuntimeConnection.forStdio({
-                ...userConn,
-                path: userConn.path ?? process.env.COPILOT_CLI_PATH,
+                ...stdio,
+                path: stdio.path ?? process.env.COPILOT_CLI_PATH,
             });
         } else {
             connection = userConn;

test/scenarios/auth/byok-anthropic/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 async function main() {
   const apiKey = process.env.ANTHROPIC_API_KEY;

test/scenarios/auth/byok-azure/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 async function main() {
   const endpoint = process.env.AZURE_OPENAI_ENDPOINT;

test/scenarios/auth/byok-ollama/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 const OLLAMA_BASE_URL = process.env.OLLAMA_BASE_URL ?? "http://localhost:11434/v1";
 const OLLAMA_MODEL = process.env.OLLAMA_MODEL ?? "llama3.2:3b";

test/scenarios/auth/byok-openai/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 const OPENAI_BASE_URL = process.env.OPENAI_BASE_URL ?? "https://api.openai.com/v1";
 const OPENAI_MODEL = process.env.OPENAI_MODEL ?? "claude-haiku-4.5";

test/scenarios/auth/gh-app/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 import readline from "node:readline/promises";
 import { stdin as input, stdout as output } from "node:process";
 

test/scenarios/bundling/fully-bundled/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 async function main() {
   const client = new CopilotClient({

test/scenarios/callbacks/hooks/typescript/src/index.ts

@@ -1,4 +1,4 @@
-import { CopilotClient } from "@github/copilot-sdk";
+import { CopilotClient , RuntimeConnection } from "@github/copilot-sdk";
 
 async function main() {
   const hookLog: string[] = [];