fix(types): add -> None return type to __init__ methods (#742)

* fix(types): add -> None return type to __init__ methods

Addresses item from code quality report.

Co-authored-by: openhands <openhands@all-hands.dev>
Closes #741

* fix: kick off CI for PR #742

Co-authored-by: openhands <openhands@all-hands.dev>

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: enyst <engel.nyst@gmail.com>

Author: 3 people

openhands_cli/acp_impl/events/event.py

@@ -47,7 +47,7 @@ def __init__(
         session_id: str,
         conn: "Client",
         conversation: BaseConversation | None = None,
-    ):
+    ) -> None:
         """Initialize the event subscriber.
 
         Args:

openhands_cli/acp_impl/events/token_streamer.py

@@ -73,7 +73,7 @@ def __init__(
         conn: Client,
         loop: asyncio.AbstractEventLoop,
         conversation: BaseConversation | None = None,
-    ):
+    ) -> None:
         self.session_id = session_id
         self.conn = conn
         self.loop = loop

openhands_cli/acp_impl/events/tool_state.py

@@ -18,7 +18,7 @@ class ToolCallState:
     returns True. Accessing them before raises ValueError.
     """
 
-    def __init__(self, tool_call_id: str, tool_name: str):
+    def __init__(self, tool_call_id: str, tool_name: str) -> None:
         self.tool_call_id = tool_call_id
         self.tool_name = tool_name
         self.is_think = tool_name == "think"

openhands_cli/acp_impl/test_utils.py

@@ -31,7 +31,7 @@ class UnbufferedJsonRpcReader:
     that can cause messages to get stuck in buffers.
     """
 
-    def __init__(self, stdout):
+    def __init__(self, stdout) -> None:
         self.stdout = stdout
         self.buffer = b""
         self.fd = stdout.fileno()

openhands_cli/tui/modals/confirmation_modal.py

@@ -38,7 +38,7 @@ def __init__(
         current_policy: ConfirmationPolicyBase,
         on_policy_selected: Callable[[ConfirmationPolicyBase], None],
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the confirmation settings modal.
 
         Args:

openhands_cli/tui/modals/exit_modal.py

@@ -24,7 +24,7 @@ def __init__(
         on_exit_confirmed: Callable[[], None] | None = None,
         on_exit_cancelled: Callable[[], None] | None = None,
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the exit confirmation modal.
 
         Args:

openhands_cli/tui/modals/settings/components/cli_settings_tab.py

@@ -23,7 +23,7 @@ def __init__(
         switch_id: str,
         value: bool = False,
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the settings switch.
 
         Args:
@@ -49,7 +49,7 @@ def compose(self) -> ComposeResult:
 class CliSettingsTab(Container):
     """CLI Settings tab component containing CLI-specific settings."""
 
-    def __init__(self, initial_settings: CliSettings | None = None, **kwargs):
+    def __init__(self, initial_settings: CliSettings | None = None, **kwargs) -> None:
         """Initialize the CLI settings tab.
 
         Args:

openhands_cli/tui/modals/settings/components/critic_settings_tab.py

@@ -44,7 +44,7 @@ def __init__(
         value: float,
         disabled: bool = False,
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the threshold input.
 
         Args:
@@ -128,7 +128,9 @@ class CriticSettingsTab(Container):
     }
     """
 
-    def __init__(self, initial_settings: CriticSettings | None = None, **kwargs):
+    def __init__(
+        self, initial_settings: CriticSettings | None = None, **kwargs
+    ) -> None:
         """Initialize the Critic settings tab.
 
         Args:

openhands_cli/tui/modals/settings/settings_screen.py

@@ -75,7 +75,7 @@ def __init__(
         on_first_time_settings_cancelled: Callable[[], None] | None = None,
         env_overrides_enabled: bool = False,
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the settings screen.
 
         Args:

openhands_cli/tui/panels/confirmation_panel.py

@@ -16,7 +16,7 @@
 class ConfirmationOption(Static):
     """A confirmation option that shows > when highlighted."""
 
-    def __init__(self, label: str, **kwargs):
+    def __init__(self, label: str, **kwargs) -> None:
         super().__init__(**kwargs)
         self.label = label
         self.is_highlighted = False
@@ -63,7 +63,7 @@ def __init__(
         self,
         num_actions: int,
         **kwargs,
-    ):
+    ) -> None:
         """Initialize the inline confirmation panel.
 
         Args: