test: unskip pending_work_resume; scope inproc-only skips conditionally

The 3 pending_work_resume 'continue pending ... after resume' tests are fixed
by the abort-scoping change (stop() no longer aborts external-server sessions),
so unskip them. The disconnect-driven capability tests (multi-client,
ui_elicitation) and 'should delete session by id' fail only over the in-process
transport (host-side auth resolved from process.env is dropped by the harness
afterEach before the delete-triggered resume runs), so gate them with
it.skipIf(isInProcessTransport) instead of skipping unconditionally.

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

Author: SteveSandersonMS

nodejs/test/e2e/client_api.e2e.test.ts

@@ -4,7 +4,7 @@
 
 import { describe, expect, it } from "vitest";
 import { approveAll } from "../../src/index.js";
-import { createSdkTestContext } from "./harness/sdkTestContext.js";
+import { createSdkTestContext, isInProcessTransport } from "./harness/sdkTestContext.js";
 
 describe("Client session management", async () => {
     const { copilotClient: client } = await createSdkTestContext();
@@ -36,7 +36,7 @@ describe("Client session management", async () => {
         expect(result).toBeFalsy();
     });
 
-    it.skip("should delete session by id", async () => {
+    it.skipIf(isInProcessTransport)("should delete session by id", async () => {
         const session = await client.createSession({ onPermissionRequest: approveAll });
         const sessionId = session.sessionId;
 

nodejs/test/e2e/multi-client.e2e.test.ts

@@ -6,7 +6,7 @@ import { describe, expect, it, afterAll } from "vitest";
 import { z } from "zod";
 import { CopilotClient, defineTool, approveAll, RuntimeConnection } from "../../src/index.js";
 import type { SessionEvent } from "../../src/index.js";
-import { createSdkTestContext } from "./harness/sdkTestContext";
+import { createSdkTestContext, isInProcessTransport } from "./harness/sdkTestContext";
 
 describe("Multi-client broadcast", async () => {
     // Use TCP mode so a second client can connect to the same CLI process
@@ -304,7 +304,7 @@ describe("Multi-client broadcast", async () => {
         }
     );
 
-    it.skip("disconnecting client removes its tools", { timeout: 90_000 }, async () => {
+    it.skipIf(isInProcessTransport)("disconnecting client removes its tools", { timeout: 90_000 }, async () => {
         const toolA = defineTool("stable_tool", {
             description: "A tool that persists across disconnects",
             parameters: z.object({ input: z.string() }),

nodejs/test/e2e/pending_work_resume.e2e.test.ts

@@ -180,7 +180,7 @@ describe("Pending work resume", async () => {
         return `localhost:${port}`;
     }
 
-    it.skip(
+    it(
         "should continue pending permission request after resume",
         { timeout: TEST_TIMEOUT_MS },
         async () => {
@@ -254,7 +254,7 @@ describe("Pending work resume", async () => {
         }
     );
 
-    it.skip(
+    it(
         "should continue pending external tool request after resume",
         { timeout: TEST_TIMEOUT_MS },
         async () => {
@@ -323,7 +323,7 @@ describe("Pending work resume", async () => {
         }
     );
 
-    it.skip(
+    it(
         "should continue parallel pending external tool requests after resume",
         { timeout: TEST_TIMEOUT_MS },
         async () => {

nodejs/test/e2e/ui_elicitation.e2e.test.ts

@@ -5,7 +5,7 @@
 import { afterAll, describe, expect, it } from "vitest";
 import { CopilotClient, approveAll, RuntimeConnection } from "../../src/index.js";
 import type { SessionEvent } from "../../src/index.js";
-import { createSdkTestContext } from "./harness/sdkTestContext.js";
+import { createSdkTestContext, isInProcessTransport } from "./harness/sdkTestContext.js";
 
 describe("UI Elicitation", async () => {
     const { copilotClient: client } = await createSdkTestContext();
@@ -116,7 +116,7 @@ describe("UI Elicitation Multi-Client Capabilities", async () => {
         }
     );
 
-    it.skip(
+    it.skipIf(isInProcessTransport)(
         "capabilities.changed fires when elicitation provider disconnects",
         { timeout: 60_000 },
         async () => {