Skip to content

test: align read_file image-content assertions with #488 #493

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wonderwhy-er merged 1 commit into from
Jun 5, 2026
Merged

test: align read_file image-content assertions with #488 #493

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
10 changes: 8 additions & 2 deletions test/test-file-handlers.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -327,7 +327,10 @@ async function testReadFilePreviewMetadata() {

const imageResult = await handleReadFile({ path: IMAGE_FILE });
assert.ok(Array.isArray(imageResult.content), 'Image result should include content array');
assert.ok(!imageResult.content.some((item) => item.type === 'image'), 'Image result should avoid image content item for host compatibility');
const imageContentItem = imageResult.content.find((item) => item.type === 'image');
assert.ok(imageContentItem, 'Image result should include an image content item so the host model can see the image');
assert.strictEqual(imageContentItem.mimeType, 'image/png', 'Image content item should carry the png mimeType');
assert.ok(typeof imageContentItem.data === 'string' && imageContentItem.data.length > 0, 'Image content item should carry non-empty base64 data');
assert.ok(imageResult.structuredContent, 'Image should include structuredContent');
assert.strictEqual(imageResult.structuredContent.fileType, 'image', 'Image fileType should map to image preview state');
assert.strictEqual(typeof imageResult.structuredContent.imageData, 'string', 'Image structured payload should include imageData');
Expand All @@ -339,7 +342,10 @@ async function testReadFilePreviewMetadata() {

const svgResult = await handleReadFile({ path: SVG_FILE });
assert.ok(Array.isArray(svgResult.content), 'SVG result should include content array');
assert.ok(!svgResult.content.some((item) => item.type === 'image'), 'SVG result should avoid image content item for host compatibility');
const svgContentItem = svgResult.content.find((item) => item.type === 'image');
assert.ok(svgContentItem, 'SVG result should include an image content item so the host model can see the image');
assert.strictEqual(svgContentItem.mimeType, 'image/svg+xml', 'SVG content item should carry the svg mimeType');
assert.ok(typeof svgContentItem.data === 'string' && svgContentItem.data.length > 0, 'SVG content item should carry non-empty base64 data');
assert.ok(svgResult.structuredContent, 'SVG should include structuredContent');
assert.strictEqual(svgResult.structuredContent.fileType, 'image', 'SVG should map to image preview state');
assert.strictEqual(svgResult.structuredContent.mimeType, 'image/svg+xml', 'SVG structured payload should include SVG mimeType');
Expand Down
Loading