Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
4653738
Add modelCapabilities override to Node SDK
SteveSandersonMS Apr 6, 2026
a2484dc
Add modelCapabilities override to all SDK languages
SteveSandersonMS Apr 6, 2026
2e943bf
Update to latest release and re-run codegen
SteveSandersonMS Apr 7, 2026
d189c79
Address PR review: use LINQ in C# test, fix formatting and lint
SteveSandersonMS Apr 7, 2026
cd5244c
Fix: forward modelCapabilities in Node.js resumeSession path
SteveSandersonMS Apr 7, 2026
62bf520
Fix Go formatting (gofmt)
SteveSandersonMS Apr 7, 2026
8d1af43
Revert Go session events regeneration to avoid breaking change
SteveSandersonMS Apr 7, 2026
9ec25ef
Regenerate all codegen including session events, fix Go DataContent
SteveSandersonMS Apr 7, 2026
f60b67c
Fix E2E test failures: replace fake-test-model, regenerate snapshots
SteveSandersonMS Apr 7, 2026
d895ad6
Fix C# test method names to match shared snapshot filenames
SteveSandersonMS Apr 7, 2026
e98a2c7
Fix fake-test-model in all languages, regenerate snapshots
SteveSandersonMS Apr 7, 2026
49f43fb
Add diagnostic logging to proxy snapshot mismatch errors
SteveSandersonMS Apr 7, 2026
21f0661
Add raw message content to proxy mismatch diagnostics
SteveSandersonMS Apr 7, 2026
be073f2
Fix proxy normalization for array-content user messages
SteveSandersonMS Apr 7, 2026
241b8d8
Fix test isolation: use sendAndWait in blob/attachment tests
SteveSandersonMS Apr 7, 2026
924ebbd
fix: write blob image to disk so model can view it
SteveSandersonMS Apr 7, 2026
400dae4
refactor: clean up proxy match-failure diagnostics for long-term use
SteveSandersonMS Apr 7, 2026
0b51a4d
fix: address PR review comments
SteveSandersonMS Apr 7, 2026
b81febb
fix: exclude arrays from DeepPartial recursion
SteveSandersonMS Apr 7, 2026
b089e0c
fix: use SendAndWait for blob tests in C#/Go/Python
SteveSandersonMS Apr 7, 2026
cf948da
fix: write blob PNG to disk in C#/Go/Python blob tests
SteveSandersonMS Apr 7, 2026
54cbab2
revert: undo unrelated Go scenario changes from codegen
SteveSandersonMS Apr 7, 2026
852c082
revert: restore generated_session_events.go to main
SteveSandersonMS Apr 7, 2026
0606475
revert: restore Go E2E tests that used DataContent.String
SteveSandersonMS Apr 7, 2026
a84b369
fix: revert .Content.String back to .Content in Go E2E tests
SteveSandersonMS Apr 7, 2026
09f7cee
ci: temporarily skip codegen check for this branch
SteveSandersonMS Apr 7, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: write blob image to disk so model can view it
The blob attachment test was sending an inline image but the model
would try to view it on disk, resulting in 'file not found' errors
in the snapshot. Write the PNG to workDir so the view tool succeeds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
  • Loading branch information
SteveSandersonMS and Copilot committed Apr 7, 2026
commit 924ebbd2cb639430ca2525ca3dd119f63d8822fd
11 changes: 8 additions & 3 deletions nodejs/test/e2e/session_config.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,16 +44,21 @@ describe("Session Configuration", async () => {
});

it("should accept blob attachments", async () => {
// Write the image to disk so the model can view it if it tries
const pngBase64 =
"iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNk+M9QDwADhgGAWjR9awAAAABJRU5ErkJggg==";
await writeFile(join(workDir, "pixel.png"), Buffer.from(pngBase64, "base64"));

const session = await client.createSession({ onPermissionRequest: approveAll });

await session.sendAndWait({
prompt: "Describe this image",
prompt: "What color is this pixel? Reply in one word.",
attachments: [
{
type: "blob",
data: "iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mNk+M9QDwADhgGAWjR9awAAAABJRU5ErkJggg==",
data: pngBase64,
mimeType: "image/png",
displayName: "test-pixel.png",
displayName: "pixel.png",
},
],
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,99 +6,22 @@ conversations:
content: ${system}
- role: user
content: |-
Describe this image
test-pixel.png
What color is this pixel? Reply in one word.
pixel.png
[image]
- role: assistant
content: I'll view the image for you.
- role: assistant
tool_calls:
- id: toolcall_0
type: function
function:
name: report_intent
arguments: '{"intent":"Viewing image file"}'
- role: assistant
tool_calls:
- id: toolcall_1
type: function
function:
name: view
arguments: '{"path":"${workdir}/test-pixel.png"}'
- messages:
- role: system
content: ${system}
- role: user
content: |-
Describe this image
test-pixel.png
[image]
- role: assistant
content: I'll view the image for you.
tool_calls:
- id: toolcall_0
type: function
function:
name: report_intent
arguments: '{"intent":"Viewing image file"}'
- id: toolcall_1
type: function
function:
name: view
arguments: '{"path":"${workdir}/test-pixel.png"}'
arguments: '{"path":"${workdir}/pixel.png"}'
- role: tool
tool_call_id: toolcall_0
content: Intent logged
- role: tool
tool_call_id: toolcall_1
content: Path ${workdir}/test-pixel.png does not exist. Please provide a valid path.
- role: assistant
content: Let me check the current directory to locate the image file.
- role: assistant
tool_calls:
- id: toolcall_2
type: function
function:
name: ${shell}
arguments: '{"command":"Get-ChildItem -Recurse -Filter \"test-pixel.png\"","description":"Find test-pixel.png file"}'
- messages:
- role: system
content: ${system}
content: Viewed image file successfully.
- role: user
content: |-
Describe this image
test-pixel.png
Image file at path ${workdir}/pixel.png
[image]
- role: assistant
content: I'll view the image for you.
tool_calls:
- id: toolcall_0
type: function
function:
name: report_intent
arguments: '{"intent":"Viewing image file"}'
- id: toolcall_1
type: function
function:
name: view
arguments: '{"path":"${workdir}/test-pixel.png"}'
- role: tool
tool_call_id: toolcall_0
content: Intent logged
- role: tool
tool_call_id: toolcall_1
content: Path ${workdir}/test-pixel.png does not exist. Please provide a valid path.
- role: assistant
content: Let me check the current directory to locate the image file.
tool_calls:
- id: toolcall_2
type: function
function:
name: ${shell}
arguments: '{"command":"Get-ChildItem -Recurse -Filter \"test-pixel.png\"","description":"Find test-pixel.png file"}'
- role: tool
tool_call_id: toolcall_2
content: <exited with exit code 0>
- role: assistant
content: The file doesn't appear to be in the working directory. Could you please upload or provide the image file
again? I'll be happy to describe it once I can access it.
content: Red
Loading