Skip to content

fix: enable vision support for OpenAI-compatible client#153

Open
hazemahmedx0 wants to merge 2 commits into
mainfrom
fix/supports-vision
Open

fix: enable vision support for OpenAI-compatible client#153
hazemahmedx0 wants to merge 2 commits into
mainfrom
fix/supports-vision

Conversation

@hazemahmedx0
Copy link
Copy Markdown
Member

Summary

  • Sets supports_vision=True in the OpenAI-compatible client config (client.py:235)
  • Enables vision-capable models routed through this client to process image inputs

Test plan

  • Verify image inputs work end-to-end with an OpenAI-compatible vision model
  • Confirm non-vision model paths are unaffected

Set supports_vision=True so vision-capable models routed through
the OpenAI-compatible client can process image inputs.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 11, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Adds a vision_format option ("openai" | "anthropic") to OpenAIProvider
and the message translation functions. Endpoints like MDB.AI that speak
Anthropic content format over an OpenAI-compatible HTTP envelope need
image blocks kept as-is rather than converted to image_url format.
@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 5/5 - Safe to Merge

Safe to merge — this PR enables vision support for OpenAI-compatible clients, a focused and well-scoped feature addition. No review comments were generated and heuristic analysis found zero critical, significant, or medium issues across the changeset. The implementation appears clean with no logic bugs, missing validation, or security concerns identified.

Key Findings:

  • Heuristic analysis returned zero issues at any severity level (critical, significant, high-risk, medium, or low), indicating the code changes are structurally sound.
  • No new review comments were generated during automated review, suggesting the vision support integration follows existing patterns and conventions in the codebase.
  • The PR has a well-defined, narrow scope (enabling vision support for OpenAI-compatible client) which reduces the risk surface compared to broader refactors.
  • No pre-existing unresolved comments exist that could indicate lingering concerns carried over from prior review cycles.

@hazemahmedx0
Copy link
Copy Markdown
Member Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant