Skip to content

fix: move routed experts helpers to response utils#1373

Merged
S1ro1 merged 1 commit into
mainfrom
fix/routed-experts-utils
May 14, 2026
Merged

fix: move routed experts helpers to response utils#1373
S1ro1 merged 1 commit into
mainfrom
fix/routed-experts-utils

Conversation

@S1ro1
Copy link
Copy Markdown
Contributor

@S1ro1 S1ro1 commented May 14, 2026

Summary

  • move routed expert parsing/truncation helpers out of verifiers.clients and into existing response utilities
  • update OpenAI clients to import routed expert parsing from verifiers.utils.response_utils

Verification

  • uv run ruff check verifiers/clients/openai_chat_completions_client.py verifiers/clients/openai_completions_client.py verifiers/utils/response_utils.py

Note

Low Risk
Primarily a namespace refactor moving parse_routed_experts/truncate_routed_experts without behavioral changes; risk is limited to potential import/path issues affecting token parsing/truncation.

Overview
Refactors routed-experts handling by moving parse_routed_experts and truncate_routed_experts out of verifiers.clients.routed_experts (module removed) into verifiers.utils.response_utils.

Updates OpenAI chat/completions clients and response token parsing to import parse_routed_experts from the utils module, consolidating routed-experts parsing/truncation alongside other response utilities.

Reviewed by Cursor Bugbot for commit 9e1844b. Bugbot is set up for automated code reviews on this repo. Configure here.

@S1ro1 S1ro1 marked this pull request as ready for review May 14, 2026 17:34
@S1ro1 S1ro1 force-pushed the fix/routed-experts-utils branch from 6ce4900 to 9e1844b Compare May 14, 2026 17:44
@S1ro1 S1ro1 changed the title fix: move routed experts helpers to utils fix: move routed experts helpers to response utils May 14, 2026
@S1ro1 S1ro1 merged commit 7fdf522 into main May 14, 2026
9 checks 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