From 029a19ca0531f3338fecdbdd55364114ed55e6a2 Mon Sep 17 00:00:00 2001 From: aeft Date: Thu, 7 Aug 2025 08:37:51 -0700 Subject: [PATCH] fix: remove duplicate error message in provider validator (#10088) --- openhands/cli/settings.py | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/openhands/cli/settings.py b/openhands/cli/settings.py index 21cb1f5d89..afa611b1af 100644 --- a/openhands/cli/settings.py +++ b/openhands/cli/settings.py @@ -205,20 +205,11 @@ async def modify_llm_settings_basic( provider = verified_providers[choice_index] else: # User selected "Select another provider" - use manual selection - # Define a validator function that prints an error message - def provider_validator(x): - is_valid = x in organized_models - if not is_valid: - print_formatted_text( - HTML('Invalid provider selected: {}'.format(x)) - ) - return is_valid - provider = await get_validated_input( session, '(Step 1/3) Select LLM Provider (TAB for options, CTRL-c to cancel): ', completer=provider_completer, - validator=provider_validator, + validator=lambda x: x in organized_models, error_message='Invalid provider selected', )