feat: fill in some custom fields for vllm-omini

[seefs001]

Summary by CodeRabbit

Release Notes

• New Features
  • Extended audio request capabilities with support for task type, language, reference audio/text inputs, vector-only processing mode, and codec configuration options.

[coderabbitai]

Walkthrough

Two modifications enhance audio request handling: the AudioRequest struct was expanded with seven optional json.RawMessage fields for enhanced audio processing parameters, and the audio request serialization logic was updated to use a custom marshalling function for audio speech relay mode.

Changes

Cohort / File(s)	Summary
Audio Request Data Model dto/audio.go	Added seven optional json.RawMessage fields to AudioRequest struct: TaskType, Language, RefAudio, RefText, XVectorOnlyMode, MaxNewTokens, and InitialCodecChunkFrames for extended audio processing capabilities.
Audio Request Serialization relay/channel/openai/adaptor.go	Switched JSON marshalling in ConvertAudioRequest from standard encoding/json.Marshal to common.Marshal when handling RelayModeAudioSpeech relay mode.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Hop, hop, the audio fields now dance,
Seven new RawMessages in AudioRequest's grand stance,
And when speech must relay with care,
A custom marshal awaits us there!
Fields and functions intertwine so neat,
Making audio handling oh-so-sweet! 🎵

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat: fill in some custom fields for vllm-omini' accurately describes the main changes: adding custom optional fields to AudioRequest and adjusting serialization in the audio adaptor for vllm-omini support.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

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

Inline comments:
In `@dto/audio.go`:
- Around line 26-28: The three optional scalar fields XVectorOnlyMode,
MaxNewTokens, and InitialCodecChunkFrames in the dto.audio struct currently use
json.RawMessage which loses type guarantees; change them to typed pointer scalar
fields (e.g., *bool for XVectorOnlyMode and *int for MaxNewTokens and
InitialCodecChunkFrames or the correct scalar types used elsewhere in the
codebase), keep the `json:"...,omitempty"` tags, and remove json.RawMessage so
the DTO follows the scalar-pointer rule and preserves explicit zero values.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0ddcd24b-3787-42f8-b914-8054e2528b57

📥 Commits

Reviewing files that changed from the base of the PR and between 78e4cb3 and a19a63b.

📒 Files selected for processing (2)

• dto/audio.go
• relay/channel/openai/adaptor.go

[coderabbitai]

⚠️ Potential issue | 🟠 Major

Use typed pointer scalars for scalar extension parameters.

Line 26-Line 28 are optional scalar-style request params but currently use json.RawMessage. This loses type guarantees and conflicts with the DTO scalar-field rule.

💡 Suggested change

-	XVectorOnlyMode         json.RawMessage `json:"x_vector_only_mode,omitempty"`
-	MaxNewTokens            json.RawMessage `json:"max_new_tokens,omitempty"`
-	InitialCodecChunkFrames json.RawMessage `json:"initial_codec_chunk_frames,omitempty"`
+	XVectorOnlyMode         *bool `json:"x_vector_only_mode,omitempty"`
+	MaxNewTokens            *int  `json:"max_new_tokens,omitempty"`
+	InitialCodecChunkFrames *int  `json:"initial_codec_chunk_frames,omitempty"`

As per coding guidelines, dto/**/*.go optional scalar fields MUST use pointer types with omitempty to preserve explicit zero values.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	XVectorOnlyMode json.RawMessage `json:"x_vector_only_mode,omitempty"`
	MaxNewTokens json.RawMessage `json:"max_new_tokens,omitempty"`
	InitialCodecChunkFrames json.RawMessage `json:"initial_codec_chunk_frames,omitempty"`
	XVectorOnlyMode *bool `json:"x_vector_only_mode,omitempty"`
	MaxNewTokens *int `json:"max_new_tokens,omitempty"`
	InitialCodecChunkFrames *int `json:"initial_codec_chunk_frames,omitempty"`

🤖 Prompt for AI Agents

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

In `@dto/audio.go` around lines 26 - 28, The three optional scalar fields
XVectorOnlyMode, MaxNewTokens, and InitialCodecChunkFrames in the dto.audio
struct currently use json.RawMessage which loses type guarantees; change them to
typed pointer scalar fields (e.g., *bool for XVectorOnlyMode and *int for
MaxNewTokens and InitialCodecChunkFrames or the correct scalar types used
elsewhere in the codebase), keep the `json:"...,omitempty"` tags, and remove
json.RawMessage so the DTO follows the scalar-pointer rule and preserves
explicit zero values.