fix: SDET mocking + assembler diagnostics + report reading

src/main.py

@@ -226,14 +226,30 @@ def results_assembler_node(state: AgentState):
     print("[*] Agent: Assembler 📊 - Building Native GitLab UX...")
     
     an_data, fin_data, test_data, dev_data = {}, {}, {}, {}
+    
     if os.path.exists("analyze_results.json"):
         with open("analyze_results.json") as f: an_data = json.load(f)
+        print("    -> ✅ Loaded analyze_results.json")
+    else:
+        print("    -> ⚠️  analyze_results.json NOT FOUND - check artifact paths")
+        
     if os.path.exists("finops_results.json"):
         with open("finops_results.json") as f: fin_data = json.load(f)
+        print("    -> ✅ Loaded finops_results.json")
+    else:
+        print("    -> ⚠️  finops_results.json NOT FOUND - check artifact paths")
+        
     if os.path.exists("devops_results.json"):
         with open("devops_results.json") as f: dev_data = json.load(f)
+        print("    -> ✅ Loaded devops_results.json")
+    else:
+        print("    -> ⚠️  devops_results.json NOT FOUND - check artifact paths")
+        
     if os.path.exists("test_results.json"):
         with open("test_results.json") as f: test_data = json.load(f)
+        print("    -> ✅ Loaded test_results.json")
+    else:
+        print("    -> ⚠️  test_results.json NOT FOUND - check artifact paths")
         
     # 1. Update MR Description
     summary = an_data.get("summary", "Code analyzed successfully.")

src/providers.py

@@ -273,11 +273,19 @@ class GeminiBrain:
         Analyze the code diffs carefully. If the MR introduces a new feature or core business logic (specifically Python code), generate a complete suite of Unit Tests using `pytest`.
         
         REQUIREMENTS:
-        1. Write robust test cases handling edge cases and mocking dependencies.
-        2. IMPORTANT IMPORT RULE: Your generated tests will be saved and executed from the root of the repository. You MUST infer the correct Python import paths based on the exact file paths provided in the MR DATA diffs. For example, if the MR data shows a change in `gcp_storage_handler.py` at the root, use `import gcp_storage_handler`. If it shows `src/api.py`, use `from src.api import ...`.
-        3. PYTEST RULES: You MUST start every single test function name with `test_` (e.g. `def test_calculate_billing()`). You MUST `import pytest` at the top of the file.
+        1. MOCKING IS MANDATORY: The test environment is isolated and does NOT have cloud SDKs (google-cloud-storage, boto3, etc.) or database drivers installed. You MUST mock ALL external dependencies using `unittest.mock.patch` or `unittest.mock.MagicMock`. Never import the real module directly if it touches any cloud SDK, database, or network. Instead, use `@patch('module_name.dependency')` decorators on your test functions.
+        2. SAFE IMPORT PATTERN: At the top of the file, only import `pytest`, `unittest.mock`, and the specific functions/classes under test. Do NOT import the entire module if it has SDK-level imports at module scope that will crash. Instead, use `sys.modules` patching to stub the whole dependency before importing:
+           ```
+           import sys
+           from unittest.mock import MagicMock
+           sys.modules['google.cloud'] = MagicMock()
+           sys.modules['google.cloud.storage'] = MagicMock()
+           # Then safely import the module under test
+           from my_module import my_function
+           ```
+        3. PYTEST RULES: You MUST start every single test function name with `test_`. You MUST `import pytest` at the top of the file.
         4. RETURN ONLY THE RAW RUNNABLE PYTHON TEST CODE. Absolutely no markdown formatting, no tick marks, no explanations. Strictly syntactically correct Python.
-        5. If the MR only contains configuration changes, documentation, lockfiles, or does not warrant tests, return exactly the string: "NO_TESTS_NEEDED".
+        5. If the MR only contains configuration changes, documentation, lockfiles, Dockerfiles, CI/CD YAML, or does not have Python business logic, return exactly the string: "NO_TESTS_NEEDED".
         """
         response = self.model.generate_content(prompt)
         text = response.text.strip('`').strip()