Fix bash executor session reuse by resetting closed sessions (#964)

Co-authored-by: openhands <[email protected]>

Author: softpudding

openhands-tools/openhands/tools/execute_bash/impl.py

@@ -129,7 +129,7 @@ def __call__(
         if action.reset and action.is_input:
             raise ValueError("Cannot use reset=True with is_input=True")
 
-        if action.reset:
+        if action.reset or self.session._closed:
             reset_result = self.reset()
 
             # Handle command execution after reset

tests/tools/execute_bash/test_secrets_masking.py

@@ -65,6 +65,7 @@ def test_bash_executor_with_conversation_secrets():
                 output="Token: secret-value-123, Key: another-secret-456",
             )
             mock_session.execute.return_value = mock_observation
+            mock_session._closed = False
             executor.session = mock_session
 
             # Execute command with conversation - secrets should be exported and masked