From 09ca1b882f348ba3122fc648f568239929878437 Mon Sep 17 00:00:00 2001 From: Rohit Malhotra Date: Tue, 17 Mar 2026 14:48:46 -0400 Subject: [PATCH] (Hotfix): use direct attrib for file download result (#13448) --- .../app_conversation/app_conversation_service_base.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openhands/app_server/app_conversation/app_conversation_service_base.py b/openhands/app_server/app_conversation/app_conversation_service_base.py index 97cfa30087..60a2036ce6 100644 --- a/openhands/app_server/app_conversation/app_conversation_service_base.py +++ b/openhands/app_server/app_conversation/app_conversation_service_base.py @@ -405,7 +405,7 @@ class AppConversationServiceBase(AppConversationService, ABC): # Check if there's an existing pre-commit hook with tempfile.TemporaryFile(mode='w+t') as temp_file: result = await workspace.file_download(PRE_COMMIT_HOOK, str(temp_file)) - if result.get('success'): + if result.success: _logger.info('Preserving existing pre-commit hook') # an existing pre-commit hook exists if 'This hook was installed by OpenHands' not in temp_file.read():