-
Notifications
You must be signed in to change notification settings - Fork 472
Document image attachment support in SDK READMEs #150
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.
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
Conversation
Co-authored-by: friggeri <106686+friggeri@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR documents the previously undocumented image attachment feature in all SDK READMEs. The feature allows users to attach images to prompts either explicitly via the attachments parameter or implicitly through the agent's built-in view tool.
Changes:
- Added "Image Support" section to Python, TypeScript, Go, and C# SDK READMEs
- Documented image attachment via
attachmentsparameter with code examples - Documented alternative approach using agent's
viewtool for implicit image access - Listed supported image formats (JPG, PNG, GIF, etc.)
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| python/README.md | Added Image Support section with Python-specific syntax for image attachments |
| nodejs/README.md | Added Image Support section with TypeScript-specific syntax for image attachments |
| go/README.md | Added Image Support section with Go-specific syntax for image attachments |
| dotnet/README.md | Added Image Support section with C#-specific syntax for image attachments |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
✅ Cross-SDK Consistency ReviewThis PR maintains excellent consistency across all four SDK implementations. Here's what I verified: ✅ All SDKs UpdatedAll four SDK READMEs have been updated with parallel "Image Support" sections:
✅ Consistent Documentation StructureEach SDK's documentation follows the same pattern:
✅ Respects Language ConventionsThe examples correctly adapt to each language's idioms:
✅ API ConsistencyAll SDKs use the same underlying structure:
SummaryNo consistency issues found. This PR documents an existing feature uniformly across all SDK implementations without creating any feature parity gaps. The documentation is well-aligned and respects each language's conventions while maintaining semantic equivalence.
|
Image attachment support exists in all SDKs but was undocumented, leading users to believe the feature was missing.
Changes
Added "Image Support" section to each SDK README documenting:
attachmentsparameterviewtoolExample
Python:
Alternative using agent's view tool:
Examples adapted to each SDK's language conventions (TypeScript, Go, C#).
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.