feat: add support for Seedream image generation

[feitianbubu]

支持火山豆包渠道测试 seedream生图模型

Summary by CodeRabbit

• New Features

  • Added support for the seedream- image generation model.

• Tests

  • Enhanced image generation test handling with improved validation for model-specific image size requirements.

[coderabbitai]

Walkthrough

The pull request adds support for the "seedream-" image generation model by registering the prefix in the recognized models list and updating the test channel controller to properly classify and handle these models with specific image dimension requirements.

Changes

Cohort / File(s)	Summary
Model Registration common/model.go	Added "seedream-" prefix to the ImageGenerationModels slice to register it as a recognized image generation model.
Test Endpoint Handling controller/channel-test.go	Updated normalizeChannelTestEndpoint to classify seedream models as EndpointTypeImageGeneration; enforces 2048x2048 image dimensions for seedream models in the image generation test path before request conversion.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: VolcEngine渠道-图片生成 API-渠道测试报错 #1814 — Directly modifies channel-test.go to handle Seedream models as image-generation models with endpoint/path adjustments.
• fix: test model #1993 #1995 — Also modifies testModel normalization and handling in channel-test.go with related conditional logic.

Suggested reviewers

• Calcium-Ion
• seefs001

Poem

🐰 A seedream model hops into view,
With dimensions of 2048 too,
Image generation now takes flight,
Our little rabbits test with delight! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: adding support for Seedream image generation model, which aligns with the file changes adding 'seedream-' to ImageGenerationModels and test handling.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

controller/channel-test.go (1)

303-305: Case sensitivity mismatch with model detection logic.

strings.Contains(testModel, "seedream-") is case-sensitive, but IsImageGenerationModel() in common/model.go normalizes to lowercase before matching. If testModel (which is the upstream model name from line 247) has different casing, this check could fail while the model is still classified as image generation.

♻️ Suggested fix for consistent case handling

 		if imageReq, ok := request.(*dto.ImageRequest); ok {
-			if strings.Contains(testModel, "seedream-") {
+			if strings.Contains(strings.ToLower(testModel), "seedream-") {
 				imageReq.Size = "2048x2048" // VolcEngine Seedream-4.5 和 5.0 模型需要至少 2048x2048 的图片尺寸 见:https://www.volcengine.com/docs/82379/1824121
 			}
 			convertedRequest, err = adaptor.ConvertImageRequest(c, info, *imageReq)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@controller/channel-test.go` around lines 303 - 305, The check for seedream
model casing is inconsistent: normalize testModel to lowercase (same as
IsImageGenerationModel() in common/model.go) before testing; replace the
case-sensitive strings.Contains(testModel, "seedream-") with a lowercase-aware
check (e.g., set a local var like modelLower := strings.ToLower(testModel) and
use strings.Contains(modelLower, "seedream-")) and then set imageReq.Size =
"2048x2048" when matched so model detection and sizing logic are consistent.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@controller/channel-test.go`:
- Around line 303-305: The check for seedream model casing is inconsistent:
normalize testModel to lowercase (same as IsImageGenerationModel() in
common/model.go) before testing; replace the case-sensitive
strings.Contains(testModel, "seedream-") with a lowercase-aware check (e.g., set
a local var like modelLower := strings.ToLower(testModel) and use
strings.Contains(modelLower, "seedream-")) and then set imageReq.Size =
"2048x2048" when matched so model detection and sizing logic are consistent.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e708ff09-f177-4c67-adca-9a3d85e26a9d

📥 Commits

Reviewing files that changed from the base of the PR and between f0e938a and fcb99d5.

📒 Files selected for processing (2)

• common/model.go
• controller/channel-test.go