feat: surface diff-report summary and risk flags in workflow logs

Continue the diff-report optimization by exposing actionable summary signals directly in runtime logs, so operators can review high-level deltas without opening the JSON artifact first.

What changed:

- Add log_diff_report_summary() to emit compact file/prop/apk/risk counters.

- Emit risk-flag codes as warning logs when present in diff-report highlights.

- Wire summary logging into execute_porting() right after diff report persistence.

- Extend workflow tests to assert summary and risk-flag logging behavior.

Verification:

- .venv/bin/python -m ruff check src/app/workflow.py tests/test_workflow.py

- .venv/bin/python -m mypy --config-file mypy-curated.ini

- .venv/bin/python -m pytest -q tests/test_workflow.py tests/test_diff_report.py

- .venv/bin/python -m pytest -q

Co-authored-by: OpenAI Codex <noreply@openai.com>

Author: toraidl

src/app/workflow.py

@@ -129,6 +129,40 @@ def run_repacking(
         packer.pack_ota_payload()
 
 
+def log_diff_report_summary(diff_report: dict[str, object], logger: logging.Logger) -> None:
+    """Log a compact summary for generated artifact diff reports."""
+    summary = diff_report.get("summary", {})
+    if not isinstance(summary, dict):
+        summary = {}
+
+    logger.info(
+        "Artifact diff summary: +%s -%s ~%s props=%s apks=%s risks=%s",
+        summary.get("files_added", 0),
+        summary.get("files_removed", 0),
+        summary.get("files_modified", 0),
+        summary.get("prop_changes", 0),
+        summary.get("apk_changes", 0),
+        summary.get("risk_flags", 0),
+    )
+
+    highlights = diff_report.get("highlights", {})
+    if not isinstance(highlights, dict):
+        return
+    risk_flags = highlights.get("risk_flags", [])
+    if not isinstance(risk_flags, list) or not risk_flags:
+        return
+
+    codes: list[str] = []
+    for flag in risk_flags:
+        if not isinstance(flag, dict):
+            continue
+        code = flag.get("code")
+        if isinstance(code, str):
+            codes.append(code)
+    if codes:
+        logger.warning("Artifact diff risk flags: %s", ", ".join(codes))
+
+
 def execute_porting(args, logger: logging.Logger) -> int:
     """Execute the end-to-end porting workflow and return a process exit code."""
     is_official_modify = args.port is None
@@ -243,6 +277,7 @@ def execute_porting(args, logger: logging.Logger) -> int:
         diff_report = generate_diff_report(baseline_artifact_state, final_artifact_state)
         report_path = save_diff_report(diff_report, args.diff_report)
         logger.info(f"Artifact diff report saved to: {report_path}")
+        log_diff_report_summary(diff_report, logger)
 
     logger.info("=" * 70)
     logger.info("Porting completed successfully!")

tests/test_workflow.py

@@ -272,7 +272,22 @@ def test_execute_porting_generates_diff_report_when_enabled():
         patch("src.app.workflow.run_modification_phases"),
         patch("src.app.workflow.run_repacking"),
         patch("src.app.workflow.collect_artifact_state") as collect_artifact_state_mock,
-        patch("src.app.workflow.generate_diff_report", return_value={"summary": {}}) as generate_mock,
+        patch(
+            "src.app.workflow.generate_diff_report",
+            return_value={
+                "summary": {
+                    "files_added": 1,
+                    "files_removed": 0,
+                    "files_modified": 2,
+                    "prop_changes": 3,
+                    "apk_changes": 4,
+                    "risk_flags": 1,
+                },
+                "highlights": {
+                    "risk_flags": [{"code": "HIGH_IMPACT_PATH_CHANGED"}],
+                },
+            },
+        ) as generate_mock,
         patch("src.app.workflow.save_diff_report") as save_diff_report_mock,
     ):
         bootstrap.return_value.exit_code = None
@@ -296,3 +311,15 @@ def test_execute_porting_generates_diff_report_when_enabled():
     assert collect_artifact_state_mock.call_count == 2
     generate_mock.assert_called_once()
     save_diff_report_mock.assert_called_once()
+    logger.info.assert_any_call(
+        "Artifact diff summary: +%s -%s ~%s props=%s apks=%s risks=%s",
+        1,
+        0,
+        2,
+        3,
+        4,
+        1,
+    )
+    logger.warning.assert_any_call(
+        "Artifact diff risk flags: %s", "HIGH_IMPACT_PATH_CHANGED"
+    )