fix: added more information for PR and removed duplication for PR creation

Author: harris-ranque

src/api/recommendations.py

@@ -471,6 +471,35 @@ def generate_pr_body(
     return "\n".join(body_lines)
 
 
+def generate_pr_body_for_suggested_rules(
+    repo_full_name: str,
+    rules_yaml: str,
+    rules_translated: int = 0,
+    rules_ambiguous: int = 0,
+    installation_id: int | None = None,
+) -> str:
+    """
+    Generate PR body for push-triggered "suggested rules" PRs (AI rule files translated to YAML).
+
+    Used by the push processor when creating a PR from translated AI rule files. Keeps
+    PR body formatting in one place alongside generate_pr_body (repository-analysis flow).
+    """
+    extracted_total = rules_translated + rules_ambiguous
+    summary_lines = [
+        f"- {extracted_total} rule statement(s) extracted from AI rule files",
+        f"- {rules_translated} rule(s) successfully translated to Watchflow YAML",
+        f"- {rules_ambiguous} rule(s) could not be translated (low confidence or infeasible)",
+    ]
+    translation_summary = "\n".join(summary_lines)
+    return (
+        "This PR was auto-generated by Watchflow because AI rule files (e.g. `rules.md`, "
+        "`*guidelines*.md`) were updated. It proposes updating `.watchflow/rules.yaml` with "
+        "the translated rules so your team can review the auto-generated constraints before merging.\n\n"
+        "**Translation Summary:**\n"
+        f"{translation_summary}"
+    )
+
+
 def generate_pr_title(recommendations: list[Any]) -> str:
     """
     Generate a professional, concise PR title based on recommendations.

src/event_processors/push.py

@@ -3,7 +3,7 @@
 from typing import Any
 
 from src.agents import get_agent
-from src.api.recommendations import get_suggested_rules_from_repo
+from src.api.recommendations import generate_pr_body_for_suggested_rules, get_suggested_rules_from_repo
 from src.core.config import config
 from src.core.models import Severity, Violation
 from src.core.utils.event_filter import NULL_SHA
@@ -122,6 +122,8 @@ async def process(self, task: Task) -> ProcessingResult:
                             github_token=github_token,
                             rules_yaml=rules_yaml,
                             push_sha=payload.get("after") or payload.get("head_commit", {}).get("sha"),
+                            rules_translated=rules_count,
+                            rules_ambiguous=len(ambiguous),
                         )
                 except Exception as e:
                     latency_ms = int((time.time() - scan_start) * 1000)
@@ -234,6 +236,8 @@ async def _create_pr_with_suggested_rules(
         github_token: str,
         rules_yaml: str,
         push_sha: str | None,
+        rules_translated: int = 0,
+        rules_ambiguous: int = 0,
     ) -> None:
         """
         Self-improving loop: create a branch with proposed .watchflow/rules.yaml and open a PR
@@ -366,10 +370,12 @@ async def _create_pr_with_suggested_rules(
                 )
                 return
 
-            pr_body = (
-                "This PR was auto-generated by Watchflow because AI rule files (e.g. `rules.md`, "
-                "`*guidelines*.md`) were updated. It proposes updating `.watchflow/rules.yaml` with "
-                "the translated rules so your team can review the auto-generated constraints before merging."
+            pr_body = generate_pr_body_for_suggested_rules(
+                repo_full_name=repo_full_name,
+                rules_yaml=rules_yaml,
+                rules_translated=rules_translated,
+                rules_ambiguous=rules_ambiguous,
+                installation_id=installation_id,
             )
             pr_result = await self.github_client.create_pull_request(
                 repo_full_name,