From 45dbf9961a33adda0fe762af92617c0768fc95ba Mon Sep 17 00:00:00 2001 From: Purushottam Baghel <47588729+puru2901is@users.noreply.github.com> Date: Sat, 27 Dec 2025 23:06:19 +0530 Subject: [PATCH] Refactor OpenCodeEngineer init to pop specific kwargs Moved extraction of 'opencode_provider' and 'opencode_model' from kwargs to before the parent class initialization, using pop to avoid passing them to BaseEngineer. This ensures only relevant arguments are sent to the parent and clarifies initialization logic. --- src/reverse_api/opencode_engineer.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/reverse_api/opencode_engineer.py b/src/reverse_api/opencode_engineer.py index baf4ebe..d5087e2 100644 --- a/src/reverse_api/opencode_engineer.py +++ b/src/reverse_api/opencode_engineer.py @@ -40,12 +40,15 @@ class OpenCodeEngineer(BaseEngineer): } def __init__(self, *args, **kwargs): + # Pop OpenCode-specific kwargs before passing to parent class + self.opencode_provider = kwargs.pop("opencode_provider", "anthropic") + self.opencode_model = kwargs.pop("opencode_model", "claude-sonnet-4-5") + super().__init__(*args, **kwargs) + # Override UI with OpenCode-specific version self.opencode_ui = OpenCodeUI(verbose=kwargs.get("verbose", True)) self.ui = self.opencode_ui # Ensure base class uses our specialized UI - self.opencode_provider = kwargs.get("opencode_provider", "anthropic") - self.opencode_model = kwargs.get("opencode_model", "claude-sonnet-4-5") self._last_error: Optional[str] = None self._session_id: Optional[str] = None self._last_event_time = 0.0