Skip to content

Update Image Generation Models list - #1722

Closed
backryun wants to merge 1 commit into
cheaper-inference:release/v3.7.3from
backryun:main
Closed

backryun wants to merge 1 commit into
cheaper-inference:release/v3.7.3from
backryun:main

Conversation

@backryun

Copy link
Copy Markdown
Contributor

Summary

  • Updated the image generation model registry with refreshed provider model lists.
  • Replaced stale FLUX alias targets with current FLUX Kontext / FLUX.2 mappings.
  • Removed deprecated FLUX Redux / Depth / Canny / Dev LoRA aliases that no longer point to registered models.
  • Updated image routing helpers so current FLUX.2 and Black Forest Labs models are recognized consistently.

Validation

  • node --import tsx/esm --test tests/unit/image-generation-handler.test.ts
  • node --import tsx/esm --test tests/unit/models-catalog-route.test.ts
  • npm run typecheck:core

Tests Added Or Updated

  • tests/unit/image-generation-handler.test.ts
  • tests/unit/models-catalog-route.test.ts

Coverage Notes

  • open-sse/config/imageRegistry.ts changes are covered by image registry alias resolution and catalog exposure assertions in tests/unit/image-generation-handler.test.ts.
  • open-sse/handlers/imageGeneration.ts routing helper updates are covered by existing image handler tests and updated alias/catalog expectations.
  • Unified model catalog output is covered by tests/unit/models-catalog-route.test.ts.
  • Full coverage was not run in this local validation pass.

Reviewer Notes

  • This PR only updates image model registry/routing metadata and related tests.
  • Stability AI multipart/form-data handling for Core/Ultra remains a separate follow-up task.
  • No migrations or feature flags are included.

@backryun
backryun requested a review from diegosouzapw as a code owner April 28, 2026 11:04

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the image model registry and generation handlers to transition from FLUX.1 and SD3 models to FLUX.2 and SD3.5, while adding support for several new models including Wan 2.6 and NanoBanana. The review feedback highlights a critical omission of the new FLUX.2 models from the image-to-image processing logic and identifies several opportunities to improve the consistency of model naming and the handling of inactive providers within the registry.

Comment thread open-sse/handlers/imageGeneration.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
Comment thread open-sse/config/imageRegistry.ts Resolved
@backryun

Copy link
Copy Markdown
Contributor Author

This commit is intended to correct the primary image model list.
The next commit will update some platform API specifications, as they use outdated methods.

@diegosouzapw
diegosouzapw changed the base branch from main to release/v3.7.3 April 28, 2026 11:21
@backryun

backryun commented Apr 28, 2026 •

Copy link
Copy Markdown
Contributor Author

An unexpected test failure has been identified and is currently being fixed.
It will be completed within 10 minutes.

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Thanks @backryun for this great contribution! 🎉 The image model registry refresh with FLUX Kontext / FLUX.2 mappings has been integrated into the release/v3.7.3 branch via a local merge (the GitHub auto-merge had conflicts from other PRs merged earlier). It will be part of the next release. We appreciate your effort!

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Merged locally into release/v3.7.3 — closing as integrated. Thank you!

@backryun

Copy link
Copy Markdown
Contributor Author

Thank you!

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.

2 participants