Skip to content

fix(research): expose resolved model in API and logs (#4567)#5382

Open
Elshayib wants to merge 1 commit into
odysseus-dev:devfrom
Elshayib:fix/research-model-visibility-4567
Open

fix(research): expose resolved model in API and logs (#4567)#5382
Elshayib wants to merge 1 commit into
odysseus-dev:devfrom
Elshayib:fix/research-model-visibility-4567

Conversation

@Elshayib

@Elshayib Elshayib commented Jul 9, 2026

Copy link
Copy Markdown

Summary

Include model in /api/research/start and status payloads and log when admin research overrides differ from the session model.

Target branch

  • This PR targets dev, not main. All PRs land in dev; main is curated by the maintainer at each release. If your PR is on main by accident, click "Edit" on this PR and change the base.

Linked Issue

Fixes #4567

Type of Change

  • Bug fix (non-breaking — fixes a confirmed issue)

Checklist

  • I searched open issues and open PRs — this is not a duplicate.
  • This PR targets dev
  • My changes are limited to the scope described above — no unrelated refactors or whitespace changes mixed in.
  • Verified with targeted pytest / syntax checks.

How to Test

  1. Configure a research-specific model override. 2. Start deep research. 3. Start response and status JSON include model field; server log notes override.

@github-actions github-actions Bot added the ready for review Description complete — ready for maintainer review label Jul 9, 2026
@alteixeira20 alteixeira20 added the bug Something isn't working label Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ready for review Description complete — ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deep Research can silently use a different model than the session-selected one when admin overrides are configured

2 participants