Skip to content

feat(rubyllm): generate images via Context#paint behind use_ruby_llm flag - #799

Draft
fluxgravity wants to merge 1 commit into
AllYourBot:mainfrom
strivedi183:add-rubyllm-generate-image-override
Draft

fluxgravity wants to merge 1 commit into
AllYourBot:mainfrom
strivedi183:add-rubyllm-generate-image-override

Conversation

@fluxgravity

@fluxgravity fluxgravity commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RubyLLM image generation behind the use_ruby_llm flag

Completes the optional Phase 6 item skipped in #775: AIBackend::RubyLLM.generate_image wraps a keyed RubyLLM.context.paint instead of inheriting the #780 delegation to AIBackend::OpenAI.generate_image. Same { b64_json:, model: } contract and dispatch seam — flag off keeps the ruby-openai path byte-identical.

Related PRs

Known trade-off (deferred to #797)

A renamed OpenAI service keeps image gen flag-off but gets the "OpenAI API key not found" error flag-on. The URL-based reconciliation above resolves it.

Testing

  • bin/rails test: 912 runs, 0 failures; old-path tests untouched.
  • Live dogfood: image generated via a Claude chat; images/generations POST confirmed.
  • Phase 7 (if implemented) can remove ruby-openai.

@fluxgravity

Copy link
Copy Markdown
Contributor Author

@strivedi183 do you have any feedback on this? I deferred to your #797 for the URL-based selection refactor. If that gets merged first, then this branch will likely need a minor rebase tweak.

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