fix: add max_clarification_rounds parameter passing from frontend to backend (#616)

Bug Fix
This PR fixes the issue where max_clarification_rounds parameter was not being passed from the frontend to the backend, causing a TypeError: '<' not supported between instances of 'int' and 'NoneType' error.

Technical Details
The issue was that the frontend was not passing the max_clarification_rounds parameter to the backend API, causing the backend to receive None values and fail during comparison operations. This fix ensures the parameter is properly typed and passed through the entire request chain.
This commit is contained in:
jimmyuconn1982
2025-10-14 02:56:20 -07:00
committed by GitHub
parent 2510cc61de
commit 24e2d86f7b
2 changed files with 2 additions and 0 deletions

View File

@@ -19,6 +19,7 @@ export async function* chatStream(
resources?: Array<Resource>;
auto_accepted_plan: boolean;
enable_clarification?: boolean;
max_clarification_rounds?: number;
max_plan_iterations: number;
max_step_num: number;
max_search_results?: number;

View File

@@ -105,6 +105,7 @@ export async function sendMessage(
resources,
auto_accepted_plan: settings.autoAcceptedPlan,
enable_clarification: settings.enableClarification ?? false,
max_clarification_rounds: settings.maxClarificationRounds ?? 3,
enable_deep_thinking: settings.enableDeepThinking ?? false,
enable_background_investigation:
settings.enableBackgroundInvestigation ?? true,