Skip to content

passing proper title fix for openrouter. - #9

Merged
kiranharidas187 merged 1 commit into
ELEVATE-Project:release-1.0.0from
darshilbabel:release-1.0.0
Jun 26, 2026
Merged

passing proper title fix for openrouter.#9
kiranharidas187 merged 1 commit into
ELEVATE-Project:release-1.0.0from
darshilbabel:release-1.0.0

Conversation

@KUNALTEMPEST

@KUNALTEMPEST KUNALTEMPEST commented Jun 26, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Bug Fixes
    • Title generation now uses a custom AI model when one is configured, and falls back to the default model otherwise.
    • Improved consistency in session title results by ensuring the selected model is applied during generation.

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a00b4438-bc6f-4892-8abf-48297a132bad

📥 Commits

Reviewing files that changed from the base of the PR and between 3d55492 and c36e5d3.

📒 Files selected for processing (1)
  • chatbot/celery_tasks/title_tasks.py

📝 Walkthrough

Walkthrough

generate_session_title now selects an LLM model from company_bot.other_params.custom_model when that value is a non-empty string, otherwise it uses company_bot.llm_model, and passes the selected model into call_llm_gateway.

Changes

Session title model selection

Layer / File(s) Summary
Effective model resolution
chatbot/celery_tasks/title_tasks.py
generate_session_title computes an effective model from company_bot.other_params.custom_model with a fallback to company_bot.llm_model, then passes it to call_llm_gateway as model.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A bunny hopped through title mist,
and picked the model on the list.
If custom carrots were not there,
it chose the default with soft care.
Hop hop — the title sparkles bright 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is related to the OpenRouter title-generation fix, though it is a bit awkward and less specific than the code change.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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

Comment @coderabbitai help to get the list of available commands.

@KUNALTEMPEST

Copy link
Copy Markdown
Author

@CodeRabbit reivew

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

@KUNALTEMPEST Reviewing the PR now, with attention to the OpenRouter title fix.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@kiranharidas187
kiranharidas187 merged commit dcd6283 into ELEVATE-Project:release-1.0.0 Jun 26, 2026
1 check passed
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