Snapshot updates

Author: SteveSandersonMS

nodejs/test/client.test.ts

@@ -26,28 +26,6 @@ describe("CopilotClient", () => {
         );
     });
 
-    it("returns a standardized failure result when a tool is not registered", async () => {
-        const client = new CopilotClient();
-        await client.start();
-        onTestFinished(() => client.forceStop());
-
-        const session = await client.createSession({ onPermissionRequest: approveAll });
-
-        const response = await (
-            client as unknown as { handleToolCallRequest: (typeof client)["handleToolCallRequest"] }
-        ).handleToolCallRequest({
-            sessionId: session.sessionId,
-            toolCallId: "123",
-            toolName: "missing_tool",
-            arguments: {},
-        });
-
-        expect(response.result).toMatchObject({
-            resultType: "failure",
-            error: "tool 'missing_tool' not supported",
-        });
-    });
-
     it("forwards clientName in session.create request", async () => {
         const client = new CopilotClient();
         await client.start();

test/snapshots/builtin_tools/should_find_files_by_pattern.yaml

@@ -53,35 +53,3 @@ conversations:
           Found 2 TypeScript files:
           - `src/app.ts`
           - `src/index.ts`
-  # Windows returns files in alphabetical order
-  - messages:
-      - role: system
-        content: ${system}
-      - role: user
-        content: Find all .ts files in this directory (recursively). List the filenames you found.
-      - role: assistant
-        content: I'll search for all TypeScript files recursively in the current directory.
-        tool_calls:
-          - id: toolcall_0
-            type: function
-            function:
-              name: report_intent
-              arguments: '{"intent":"Finding TypeScript files"}'
-          - id: toolcall_1
-            type: function
-            function:
-              name: glob
-              arguments: '{"pattern":"**/*.ts"}'
-      - role: tool
-        tool_call_id: toolcall_0
-        content: Intent logged
-      - role: tool
-        tool_call_id: toolcall_1
-        content: |-
-          ./src/app.ts
-          ./src/index.ts
-      - role: assistant
-        content: |-
-          Found 2 TypeScript files:
-          - `src/app.ts`
-          - `src/index.ts`

test/snapshots/session_config/should_accept_message_attachments.yaml

@@ -8,6 +8,8 @@ conversations:
         content: |-
           Summarize the attached file
 
+
+
           <tagged_files>
           * ${workdir}/attached.txt (1 lines)
           </tagged_files>

test/snapshots/tool_results/should_handle_tool_result_with_failure_resulttype.yaml

@@ -15,6 +15,6 @@ conversations:
               arguments: "{}"
       - role: tool
         tool_call_id: toolcall_0
-        content: Service unavailable
+        content: '{"error":"API timeout","resultType":"failure","textResultForLlm":"Service unavailable"}'
       - role: assistant
         content: service is down