fix : fix submodel adapter

[seefs001]

Summary by CodeRabbit

• Bug Fixes

  • Corrected request routing to use the channel-specific base URL, improving stability and preventing misrouted or failing calls in certain configurations.

• Chores

  • Introduced explicit handling for Gemini chat requests, returning a clear “not supported” error to users until the endpoint is available, ensuring predictable behavior and clearer feedback.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Introduces a new Adaptor method ConvertGeminiRequest that returns a not-supported error and modifies GetRequestURL to reference info.ChannelBaseUrl instead of info.BaseUrl. Both changes are confined to relay/channel/submodel/adaptor.go.

Changes

Cohort / File(s)	Summary of Changes
Adaptor updates relay/channel/submodel/adaptor.go	- Added ConvertGeminiRequest(c *gin.Context, info *relaycommon.RelayInfo, request *dto.GeminiChatRequest) (any, error) returning a not-supported error - Updated GetRequestURL to use info.ChannelBaseUrl instead of info.BaseUrl

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Client
  participant Adaptor
  note over Client,Adaptor: Gemini request conversion flow (new method)
  Client->>Adaptor: ConvertGeminiRequest(ctx, info, geminiReq)
  Adaptor-->>Client: error "not supported"
  note over Adaptor: Endpoint not implemented

Loading

sequenceDiagram
  autonumber
  actor Caller
  participant Adaptor
  participant RelayInfo as RelayInfo (info)
  note over Caller,Adaptor: Request URL resolution (modified)
  Caller->>Adaptor: GetRequestURL(info)
  Adaptor->>RelayInfo: read ChannelBaseUrl
  Adaptor-->>Caller: URL built from info.ChannelBaseUrl

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: Convert gemini request #1484 — Also introduces/updates ConvertGeminiRequest for Gemini→OpenAI conversion, aligning with this adaptor method addition.

Poem

I thump my paws on cables bright,
A stub today, no Gemini flight—
New paths point where channels lead,
BaseUrl swapped for Channel’s seed.
Soon I’ll hop from star to star,
For now: “Not yet!” from this bazaar. 🐇✨

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

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

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 83b2b07 and 76180b1.

📒 Files selected for processing (1)

• relay/channel/submodel/adaptor.go (2 hunks)

---

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.}