Skip to content

Release v3.7.3 - #1724

Merged
diegosouzapw merged 5 commits into
mainfrom
release/v3.7.3
Apr 28, 2026
Merged

diegosouzapw merged 5 commits into
mainfrom
release/v3.7.3

Conversation

@diegosouzapw

Copy link
Copy Markdown
Owner

[3.7.3] — 2026-04-28

🐛 Bug Fixes

🔄 Updates

  • chore(image-models): refresh image generation model registry — replace stale FLUX aliases with FLUX Kontext / FLUX.2 mappings, remove deprecated FLUX Redux/Depth/Canny variants (Update Image Generation Models list #1722)

Tests

  • All tests pass (3803 passing)

⚠️ After merging: run Phase 2 steps to tag, publish, and deploy.

@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 application to version 3.7.3 and performs a significant refresh of the image generation model registry, updating model aliases and provider configurations for OpenAI, Black Forest Labs, and Stability AI. It also includes several bug fixes related to combo fallback orchestration, Codex tool restoration, and UI modal styling. Review feedback identifies typos in comments, inconsistent casing for model names, and the use of confusing placeholder names for inactive providers. Additionally, a discrepancy was found in the test suite where an unsupported image size was used for the OpenAI provider.

supportedSizes: ["1024x1024"],
},

//Curruntly no models serving

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.

medium

Typo in comment: 'Curruntly' should be 'Currently'.

Suggested change
//Curruntly no models serving
//Currently no models serving

{ id: "black-forest-labs/flux-dev", name: "FLUX.1 Dev" },
],
supportedSizes: ["1024x1024", "512x512"],
models: [{ id: "black-forest-labs/flux-schnell", name: "No Model yet" }],

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.

medium

Using 'No Model yet' as a model name is confusing for users. If the provider is temporarily unavailable, it is better to use an empty array or a dedicated status flag to handle this in the UI, rather than providing a dummy model that might still be selectable.

models: [{ id: "SDXL1.0-base", name: "No Model yet" }],
supportedSizes: ["1024x1024"],
},
//Curruntly no models serving

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.

medium

Typo in comment: 'Curruntly' should be 'Currently'.

Suggested change
//Curruntly no models serving
//Currently no models serving

Comment on lines +406 to +407
{ id: "flux-2-klein-9b", name: "flux 2 Klein 9B" },
{ id: "flux-2-klein-4b", name: "flux 2 Klein 4B" },

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.

medium

Inconsistent casing for the model names. Other models in this list use 'FLUX.2' (e.g., line 401). Please ensure consistent naming across the registry.

Suggested change
{ id: "flux-2-klein-9b", name: "flux 2 Klein 9B" },
{ id: "flux-2-klein-4b", name: "flux 2 Klein 4B" },
{ id: "flux-2-klein-9b", name: "FLUX.2 Klein 9B" },
{ id: "flux-2-klein-4b", name: "FLUX.2 Klein 4B" },

prompt: "city skyline",
n: 2,
size: "1024x1792",
size: "1024x1536",

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.

medium

The size '1024x1536' is not in the supported sizes list for the 'openai' provider in imageRegistry.ts (which only lists '1024x1024', '1024x1792', '1792x1024'). This might cause validation failures in the test suite.

        size: "1024x1792",

prompt: "city skyline",
n: 2,
size: "1024x1792",
size: "1024x1536",

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.

medium

The size '1024x1536' is not supported by the 'openai' provider. Use a supported size like '1024x1792' to match the expected upstream body and the registry configuration.

      size: "1024x1792",

@github-actions

Copy link
Copy Markdown
Contributor

CI Coverage Report

  • Coverage job: success
  • PR test policy: success

Coverage artifact was not available for this run.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d2da66c339

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 88 to +90
"pollinations/kontext": {
provider: "pollinations",
model: "flux-kontext",
name: "FLUX Kontext",
provider: "black-forest-labs",
model: "flux-kontext-pro",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep pollinations/kontext routed to Pollinations

This alias now maps an explicitly Pollinations-scoped model ID to black-forest-labs, so parseImageModel("pollinations/kontext") resolves to a different provider than the caller requested. In the image route, credentials are loaded by parsed provider, so users who only configured Pollinations now get No credentials for image provider: black-forest-labs for a model string that previously used Pollinations credentials. This is a backward-compatibility break for existing pollinations/kontext clients.

Useful? React with 👍 / 👎.

inputModalities: ["text", "image"],
description: "Higher quality Kontext editing with optional source image",
},
{ id: "gpt-image-2", name: "GPT Image 2" },

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Avoid introducing ambiguous bare model ID gpt-image-2

Adding gpt-image-2 to Pollinations duplicates an existing OpenAI model ID, and the no-prefix fallback in parseImageModel picks the first provider match by registry order. That means requests using model: "gpt-image-2" will always resolve to OpenAI, which can misroute traffic and produce credential errors in setups that intend to use Pollinations without prefixing. Keep bare IDs provider-unique (or require explicit provider/model).

Useful? React with 👍 / 👎.

@diegosouzapw
diegosouzapw merged commit ae8a7d2 into main Apr 28, 2026
66 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5213d4415

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

model: "flux-2-max",
name: "FLUX.2 Max",
listInCatalog: true,
inputModalities: ["text", "image"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Drop image-required modality from FLUX.2 catalog aliases

Marking flux-2-max (and the sibling flux-2-pro / flux-2-flex aliases added in this change) as inputModalities: ["text", "image"] makes the route layer require both prompt and image input (requiresImageInput check in src/app/api/v1/images/generations/route.ts), so prompt-only requests to these new catalog aliases now fail with Image input is required... even though the Black Forest Labs handler accepts text-only generation for these models. This turns the primary text-to-image path for the new aliases into a 400 unless callers provide an image.

Useful? React with 👍 / 👎.

@sonarqubecloud

Copy link
Copy Markdown

HouMinXi pushed a commit to HouMinXi/OmniRoute that referenced this pull request Aug 2, 2026
* Update image-model list

* docs: update CHANGELOG.md with merged PR entries for v3.7.3

* chore(release): bump to v3.7.3 — changelog, docs, version sync

* chore(workflow): update generate-release with Phase 2 validation step

---------

Co-authored-by: backryun <bakryun0718@proton.me>
Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
Poid-ZA pushed a commit to Poid-ZA/OmniRoute that referenced this pull request Aug 5, 2026
* Update image-model list

* docs: update CHANGELOG.md with merged PR entries for v3.7.3

* chore(release): bump to v3.7.3 — changelog, docs, version sync

* chore(workflow): update generate-release with Phase 2 validation step

---------

Co-authored-by: backryun <bakryun0718@proton.me>
Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
muhamadgalihsaputra pushed a commit to niyatna/NiyatnaRoute that referenced this pull request Sep 27, 2026
* Update image-model list

* docs: update CHANGELOG.md with merged PR entries for v3.7.3

* chore(release): bump to v3.7.3 — changelog, docs, version sync

* chore(workflow): update generate-release with Phase 2 validation step

---------

Co-authored-by: backryun <bakryun0718@proton.me>
Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
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