Revert "Agentic unit test fixing (#325)"

This reverts commit a87a7d9.

Author: NejcS

codeplain_REST_api.py

@@ -253,6 +253,38 @@ def render_functional_requirement(
 
         return self.post_request(endpoint_url, headers, payload, run_state)
 
+    def fix_unittests_issue(
+        self,
+        frid,
+        plain_source_tree,
+        linked_resources,
+        existing_files_content,
+        module_name: str,
+        required_modules,
+        unittests_issue,
+        run_state: RunState,
+        conformance_tests_fixes: list[dict] | None = None,
+    ):
+        endpoint_url = f"{self.api_url}/fix_unittests_issue"
+        headers = {"X-API-Key": self.api_key, "Content-Type": "application/json"}
+
+        payload = {
+            "frid": frid,
+            "plain_source_tree": plain_source_tree,
+            "linked_resources": linked_resources,
+            "existing_files_content": existing_files_content,
+            "module_name": module_name,
+            "required_modules": required_modules,
+            "unittests_issue": unittests_issue,
+            "unittest_batch_id": run_state.unittest_batch_id,
+        }
+        # Implementation code changes made by the conformance tests fixer right before this unit tests run;
+        # only sent when unit tests are processed inside the conformance tests phase.
+        if conformance_tests_fixes is not None:
+            payload["conformance_tests_fixes"] = conformance_tests_fixes
+
+        return self.post_request(endpoint_url, headers, payload, run_state)
+
     def distill_conformance_test_memory(
         self,
         frid,
@@ -526,29 +558,3 @@ def summarize_finished_conformance_tests(
         }
 
         return self.post_request(endpoint_url, headers, payload, run_state)
-
-    def agent_start(self, task_type: str, task_params: dict, frid: str, module_name: str, run_state: RunState):
-        """Start a server-side agent session; returns the first turn (tool calls or completion)."""
-        endpoint_url = f"{self.api_url}/agent/start"
-        headers = {"X-API-Key": self.api_key, "Content-Type": "application/json"}
-        payload = {
-            "task_type": task_type,
-            "task_params": task_params,
-            "frid": frid,
-            "module_name": module_name,
-        }
-        return self.post_request(endpoint_url, headers, payload, run_state)
-
-    def agent_continue(
-        self, session_id: str, tool_results: list[dict], frid: str, module_name: str, run_state: RunState
-    ):
-        """Feed tool results into an agent session and run its next turn."""
-        endpoint_url = f"{self.api_url}/agent/continue"
-        headers = {"X-API-Key": self.api_key, "Content-Type": "application/json"}
-        payload = {
-            "session_id": session_id,
-            "tool_results": tool_results,
-            "frid": frid,
-            "module_name": module_name,
-        }
-        return self.post_request(endpoint_url, headers, payload, run_state)

render_machine/actions/fix_unit_tests.py

@@ -16,15 +16,6 @@ class RunUnitTests(BaseAction):
     UNRECOVERABLE_ERROR_OUTCOME = "unrecoverable_error_occurred"
 
     def execute(self, render_context: RenderContext, _previous_action_payload: Any | None):
-        context = render_context.unit_tests_running_context
-        if context.verified_passing:
-            # The fixing agent's own run passed and no file changed since; running again is redundant.
-            context.verified_passing = False
-            console.info("Unit tests already passed in the fixing agent's last run; not running them again.")
-            render_context.script_execution_history.latest_unit_test_output_path = context.verified_passing_log_path
-            render_context.script_execution_history.should_update_script_outputs = True
-            return self.SUCCESSFUL_OUTCOME, None
-
         unittests_script = os.path.normpath(render_context.unittests_script)
 
         console.info(

render_machine/actions/run_unit_tests.py

@@ -19,7 +19,6 @@
     FridContext,
     ScriptExecutionHistory,
     TestExecutionPhase,
-    UnitTestsAgentSession,
     UnitTestsRunningContext,
 )
 
@@ -189,14 +188,6 @@ def should_run_unit_tests(self) -> bool:
     def should_run_conformance_tests(self) -> bool:
         return self.conformance_tests_script is not None
 
-    @property
-    def unit_tests_agent_session(self) -> UnitTestsAgentSession:
-        """The agent session fixing the unit tests: per unit-test loop, except during the conformance
-        phase, where one session spans every loop (see UnitTestsAgentSession)."""
-        if self.conformance_tests_running_context is not None:
-            return self.conformance_tests_running_context.unit_tests_agent_session
-        return self.unit_tests_running_context.agent_session
-
     def start_unittests_processing(self):
         self.unit_tests_running_context = UnitTestsRunningContext(fix_attempts=0)
         self.run_state.increment_unittest_batch_id()