claude-dev-env 1.59.0 → 1.61.0

package/skills/autoconverge/workflow/convergence_summary.py

@@ -0,0 +1,110 @@
+"""Build the convergence-summary agent prompt over a PR's aggregated findings."""
+
+from autoconverge_report_constants.render_report_constants import (
+    GITHUB_PR_URL_TEMPLATE,
+    SUMMARY_COPILOT_NOTE_TEMPLATE,
+    SUMMARY_DETAIL_MAX_CHARS,
+    SUMMARY_FINDING_LINE_TEMPLATE,
+    SUMMARY_FINDINGS_EMPTY_TEXT,
+    SUMMARY_FIX_EMPTY_TEXT,
+    SUMMARY_FIX_LINE_TEMPLATE,
+    SUMMARY_PR_COORDINATES_TEMPLATE,
+    SUMMARY_PROMPT_TEMPLATE,
+    SUMMARY_STANDARDS_NOTE_TEMPLATE,
+)
+
+
+def _format_findings_block(findings: list[dict]) -> str:
+    """Return the numbered findings block, or a clean-run sentence when empty.
+
+    Args:
+        findings: Aggregated distinct findings, each carrying severity, category,
+            file, line, title, and detail keys.
+
+    Returns:
+        A newline-joined numbered list, or a sentence stating every lens was clean.
+    """
+    if not findings:
+        return SUMMARY_FINDINGS_EMPTY_TEXT
+    numbered_lines: list[str] = []
+    for position, each_finding in enumerate(findings):
+        detail = str(each_finding.get("detail", ""))[:SUMMARY_DETAIL_MAX_CHARS]
+        numbered_lines.append(
+            SUMMARY_FINDING_LINE_TEMPLATE.format(
+                number=position + 1,
+                severity=each_finding.get("severity", ""),
+                category=each_finding.get("category", ""),
+                file=each_finding.get("file", ""),
+                line=each_finding.get("line", 0),
+                title=each_finding.get("title", ""),
+                detail=detail,
+            )
+        )
+    return "\n".join(numbered_lines)
+
+
+def _format_fix_block(fix_summaries: list[str]) -> str:
+    """Return the numbered per-round fix-summary block, or 'none' when empty.
+
+    Args:
+        fix_summaries: One-line fix summaries collected across every round.
+
+    Returns:
+        A newline-joined numbered list, or the empty-state literal.
+    """
+    if not fix_summaries:
+        return SUMMARY_FIX_EMPTY_TEXT
+    return "\n".join(
+        SUMMARY_FIX_LINE_TEMPLATE.format(number=position + 1, summary=each_summary)
+        for position, each_summary in enumerate(fix_summaries)
+    )
+
+
+def build_summary_prompt(
+    owner: str,
+    repo: str,
+    pr_number: int,
+    round_count: int,
+    findings: list[dict],
+    fix_summaries: list[str],
+    standards_note: str | None,
+    copilot_note: str | None,
+) -> str:
+    """Return the convergence-summary agent prompt for a PR's aggregated findings.
+
+    Args:
+        owner: The PR's repository owner.
+        repo: The PR's repository name.
+        pr_number: The PR number.
+        round_count: Total converge rounds across every run aggregated.
+        findings: Aggregated distinct findings across every run.
+        fix_summaries: One-line fix summaries collected across every run.
+        standards_note: Deferral note when a round was code-standard-only, else None.
+        copilot_note: Outage note when the Copilot gate was bypassed, else None.
+
+    Returns:
+        The full agent prompt instructing a StructuredOutput convergence summary.
+    """
+    pr_url = GITHUB_PR_URL_TEMPLATE.format(owner=owner, repo=repo, number=pr_number)
+    pr_coordinates = SUMMARY_PR_COORDINATES_TEMPLATE.format(
+        owner=owner, repo=repo, pr_number=pr_number, url=pr_url
+    )
+    standards_block = (
+        SUMMARY_STANDARDS_NOTE_TEMPLATE.format(note=standards_note)
+        if standards_note
+        else ""
+    )
+    copilot_block = (
+        SUMMARY_COPILOT_NOTE_TEMPLATE.format(note=copilot_note) if copilot_note else ""
+    )
+    return SUMMARY_PROMPT_TEMPLATE.format(
+        pr_coordinates=pr_coordinates,
+        owner=owner,
+        repo=repo,
+        pr_number=pr_number,
+        round_count=round_count,
+        findings_block=_format_findings_block(findings),
+        fix_block=_format_fix_block(fix_summaries),
+        standards_block=standards_block,
+        copilot_block=copilot_block,
+    )

package/skills/autoconverge/workflow/fixtures/wf_run/subagents/workflows/wf_881252e6-700/agent-ab1c2d3e4f5a6b7c8.jsonl

@@ -0,0 +1,2 @@
+{"type": "user", "uuid": "ab1c2d3e4f5a6b7c8-u", "message": {"role": "user", "content": "Return the convergence summary."}}
+{"type": "assistant", "uuid": "ab1c2d3e4f5a6b7c8-a", "message": {"role": "assistant", "content": [{"type": "tool_use", "id": "toolu_ab1c2d3e4f5a6b7c8", "name": "StructuredOutput", "input": {"prProblem": "DataBridge, the service that exports your records in batches, restarted an interrupted export from the beginning instead of continuing.", "prFix": "An interrupted export now resumes from the last finished batch.", "problemScenes": [{"trigger": "export stops at batch 90 of 100", "condition": "you restart it", "result": "starts again at batch 1", "caption": "A halted export threw away the 90 batches it had already finished and began again."}], "fixScenes": [{"trigger": "export stops at batch 90 of 100", "condition": "you restart it", "result": "continues at batch 91", "caption": "A restarted export now picks up at the next unfinished batch."}], "verdictLine": "Converged in 4 rounds; 3 distinct issue classes were caught and fixed.", "issueClasses": [{"plainName": "Tests did not declare their return type", "count": 7, "severity": "P2", "category": "code-standard", "status": "fixed", "cause": "Several new test functions did not state that they return nothing, which the project's type checker wants.", "medium": "code", "beforeLines": ["def test_resume_skip():", "    ..."], "afterLines": ["def test_resume_skip() -> None:", "    ..."]}, {"plainName": "A vague banned variable name", "count": 2, "severity": "P2", "category": "code-standard", "status": "fixed", "cause": "Two variables used a generic placeholder name the project bans.", "medium": "code", "beforeLines": ["result = fetch()"], "afterLines": ["exported_rows = fetch()"]}, {"plainName": "Hardcoded message strings", "count": 6, "severity": "P2", "category": "code-standard", "status": "fixed", "cause": "Warning text was written inline in code instead of shared configuration.", "medium": "text", "beforeLines": ["inline warning text"], "afterLines": ["shared config message"]}]}}]}}

package/skills/autoconverge/workflow/fixtures/wf_run/workflows/wf_881252e6-700.json

@@ -254,6 +254,13 @@
       "phaseTitle": "Finalize",
       "agentId": "add93d458ce4da043",
       "state": "done"
+    },
+    {
+      "index": 34,
+      "label": "convergence-summary",
+      "phaseTitle": "Finalize",
+      "agentId": "ab1c2d3e4f5a6b7c8",
+      "state": "done"
     }
   ]
 }