fix(report): improve legacy status compatibility for release readiness

Author: mohamedxk9tb

bladerecon/modules/report.py

@@ -1844,9 +1844,12 @@ def run(target: str, output: Path = Path("results"), scan_duration: Optional[str
         parameters = _load_parameters(target_dir)
         discovered_parameters = _load_discovered_parameters(target_dir)
     screenshots_skipped_reason = ""
+    screenshots_incomplete_reason = ""
     screenshot_state_status = resolve_module_status(screenshot_state, has_artifact=bool(screenshots))
     if screenshot_state_status == "skipped":
         screenshots_skipped_reason = _normalize_skip_reason(str(screenshot_state.get("error") or "Missing Dependency"))
+    elif screenshot_state_status in {"failed", "timeout", "partial"}:
+        screenshots_incomplete_reason = _normalize_skip_reason(str(screenshot_state.get("error") or screenshot_state_status))
     else:
         chromium_ok, chromium_detail = check_playwright_chromium()
         if not screenshots and not chromium_ok:
@@ -1896,7 +1899,11 @@ def run(target: str, output: Path = Path("results"), scan_duration: Optional[str
     screenshots_status = (
         f"Skipped ({screenshots_skipped_reason})"
         if screenshots_skipped_reason
-        else ("Completed" if isinstance(screenshot_state, dict) and screenshot_state.get("status") == "completed" and not screenshots else (len(screenshots) if screenshots_available else "Not Run"))
+        else (
+            f"{_status_label(screenshot_state_status)} ({screenshots_incomplete_reason})"
+            if screenshots_incomplete_reason
+            else ("Completed" if screenshot_state_status == "completed" and not screenshots else (len(screenshots) if screenshots_available else "Not Run"))
+        )
     )
     nuclei_status = (
         f"{nuclei_status_label} ({nuclei_skipped_reason})"

tests/test_report.py

@@ -598,6 +598,20 @@ def test_report_shows_skipped_parameters_from_scan_state(tmp_path: Path) -> None
     assert "redirect_url" not in html
 
 
+def test_report_preserves_legacy_screenshot_timeout_status(tmp_path: Path) -> None:
+    target = tmp_path / "timeout.example"
+    target.mkdir(parents=True)
+    (target / "scan_state.json").write_text(
+        json.dumps({"modules": {"screenshots": {"status": "timed_out", "error": "browser timeout"}}}),
+        encoding="utf-8",
+    )
+
+    report.run("timeout.example", output=tmp_path)
+
+    md = (target / "reports" / "report.md").read_text(encoding="utf-8")
+    assert "- Screenshots: Timed Out (browser timeout)" in md
+
+
 def test_report_shows_template_unavailable_nuclei_as_skipped(tmp_path: Path) -> None:
     target = tmp_path / "example.com"
     target.mkdir(parents=True)