crewforge 0.2.2__tar.gz → 0.3.0__tar.gz

{crewforge-0.2.2/src/crewforge.egg-info → crewforge-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: crewforge
-Version: 0.2.2
+Version: 0.3.0
 Summary: One crew of LLM agents, five recipes: code review, document Q&A, API scaffolding, content and honest academic-writing help — on any OpenAI-compatible provider.
 Author: Muhammad Aiman
 License-Expression: MIT
@@ -145,7 +145,7 @@ tables are trimmed to their totals.
 
 | Command | Agents | What you get |
 |---|---|---|
-| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI |
+| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI, `-f sarif` for GitHub code scanning |
 | `ingest FILES` | ingester, indexer (no LLM) | How each file was parsed and chunked |
 | `ask FILES -q Q [--verify] [--rerank]` | retriever → answerer → fact-checker | An answer with `[Source N]` citations and a claim-by-claim verdict |
 | `summarize FILES` | summarizer | Summary and key points per document |
@@ -177,6 +177,30 @@ $ crewforge review app.py -f markdown -o review.md      # app.py builds SQL with
 total   7 calls   22,366 tokens (provider-reported)
 ```
 
+`-f sarif` writes a [Static Analysis Results Interchange Format 2.1.0](https://docs.oasis-open.org/sarif/sarif/v2.1.0/sarif-v2.1.0.html)
+log (`sarif`), the format GitHub code scanning reads, so findings can appear in a repository's Security tab. Each finding becomes one
+result: `critical` and `error` map to level `error`, `warning` to `warning`, `info` to `note`; the rule is the
+finding's CWE when it has one, otherwise the reviewer's name. When you review a directory, the line a reviewer
+reports is mapped back to the file it came from. A finding whose line is missing or outside the code points at the
+reviewed path without a line. Reviewers that fail are listed in `invocations`, and the exit code is the same as for
+the other formats.
+
+```yaml
+# .github/workflows/review.yml (needs an API key secret, e.g. OPENAI_API_KEY)
+permissions:
+  contents: read
+  security-events: write
+steps:
+  - uses: actions/checkout@v4
+  - run: pipx run crewforge review src/ -f sarif -o crewforge.sarif
+    env:
+      OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+  - uses: github/codeql-action/upload-sarif@v3
+    if: always()
+    with:
+      sarif_file: crewforge.sarif
+```
+
 ### `ask` and `summarize`
 
 <p align="center">
@@ -473,9 +497,13 @@ pip install crewforge                 # Python 3.10 hingga 3.14
 pip install "crewforge[docs,api]"     # pilihan: PDF/DOCX dan semakan aplikasi FastAPI
 export OPENAI_API_KEY=sk-...          # atau penyedia lain, lihat `crewforge providers`
 crewforge review src/
+crewforge review src/ -f sarif -o crewforge.sarif   # untuk GitHub code scanning
 crewforge academic feedback draf.md --brief taklimat.md --language ms
 ```
 
+`review -f sarif` menulis laporan `SARIF` 2.1.0 yang dibaca GitHub code scanning: setiap dapatan menjadi satu
+keputusan dalam tab Security, dan baris dalam semakan folder dipetakan kembali ke fail asalnya.
+
 Senarai penyedia, contoh setiap resipi dan seni bina ada dalam bahagian bahasa Inggeris di atas
 ([Providers](https://github.com/aimanmalib/crewforge#providers), [Recipes](https://github.com/aimanmalib/crewforge#recipes)).
 

{crewforge-0.2.2 → crewforge-0.3.0}/README.md

@@ -87,7 +87,7 @@ tables are trimmed to their totals.
 
 | Command | Agents | What you get |
 |---|---|---|
-| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI |
+| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI, `-f sarif` for GitHub code scanning |
 | `ingest FILES` | ingester, indexer (no LLM) | How each file was parsed and chunked |
 | `ask FILES -q Q [--verify] [--rerank]` | retriever → answerer → fact-checker | An answer with `[Source N]` citations and a claim-by-claim verdict |
 | `summarize FILES` | summarizer | Summary and key points per document |
@@ -119,6 +119,30 @@ $ crewforge review app.py -f markdown -o review.md      # app.py builds SQL with
 total   7 calls   22,366 tokens (provider-reported)
 ```
 
+`-f sarif` writes a [Static Analysis Results Interchange Format 2.1.0](https://docs.oasis-open.org/sarif/sarif/v2.1.0/sarif-v2.1.0.html)
+log (`sarif`), the format GitHub code scanning reads, so findings can appear in a repository's Security tab. Each finding becomes one
+result: `critical` and `error` map to level `error`, `warning` to `warning`, `info` to `note`; the rule is the
+finding's CWE when it has one, otherwise the reviewer's name. When you review a directory, the line a reviewer
+reports is mapped back to the file it came from. A finding whose line is missing or outside the code points at the
+reviewed path without a line. Reviewers that fail are listed in `invocations`, and the exit code is the same as for
+the other formats.
+
+```yaml
+# .github/workflows/review.yml (needs an API key secret, e.g. OPENAI_API_KEY)
+permissions:
+  contents: read
+  security-events: write
+steps:
+  - uses: actions/checkout@v4
+  - run: pipx run crewforge review src/ -f sarif -o crewforge.sarif
+    env:
+      OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+  - uses: github/codeql-action/upload-sarif@v3
+    if: always()
+    with:
+      sarif_file: crewforge.sarif
+```
+
 ### `ask` and `summarize`
 
 <p align="center">
@@ -415,9 +439,13 @@ pip install crewforge                 # Python 3.10 hingga 3.14
 pip install "crewforge[docs,api]"     # pilihan: PDF/DOCX dan semakan aplikasi FastAPI
 export OPENAI_API_KEY=sk-...          # atau penyedia lain, lihat `crewforge providers`
 crewforge review src/
+crewforge review src/ -f sarif -o crewforge.sarif   # untuk GitHub code scanning
 crewforge academic feedback draf.md --brief taklimat.md --language ms
 ```
 
+`review -f sarif` menulis laporan `SARIF` 2.1.0 yang dibaca GitHub code scanning: setiap dapatan menjadi satu
+keputusan dalam tab Security, dan baris dalam semakan folder dipetakan kembali ke fail asalnya.
+
 Senarai penyedia, contoh setiap resipi dan seni bina ada dalam bahagian bahasa Inggeris di atas
 ([Providers](https://github.com/aimanmalib/crewforge#providers), [Recipes](https://github.com/aimanmalib/crewforge#recipes)).
 

{crewforge-0.2.2 → crewforge-0.3.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "crewforge"
-version = "0.2.2"
+version = "0.3.0"
 description = "One crew of LLM agents, five recipes: code review, document Q&A, API scaffolding, content and honest academic-writing help — on any OpenAI-compatible provider."
 readme = "README.md"
 license = "MIT"

{crewforge-0.2.2 → crewforge-0.3.0}/src/crewforge/__init__.py

@@ -1,3 +1,3 @@
 """crewforge — one crew of LLM agents, five recipes, any OpenAI-compatible provider."""
 
-__version__ = "0.2.2"
+__version__ = "0.3.0"

{crewforge-0.2.2 → crewforge-0.3.0}/src/crewforge/recipes/review/cli.py

@@ -22,8 +22,14 @@ from ...cli_support import (
 from ...core.config import LLMConfig
 from .agents import REVIEW_AGENTS
 from .pipeline import CodeReviewCrew, ReviewResult
-from .report import report_dict, report_markdown
-from .sources import DEFAULT_MAX_FILE_BYTES, SourceError, collect, detect_language
+from .report import report_dict, report_markdown, report_sarif
+from .sources import (
+    DEFAULT_MAX_FILE_BYTES,
+    SourceError,
+    Sources,
+    collect,
+    detect_language,
+)
 
 # Kept under the old private name: codeguard's tests and users imported it.
 _detect_language = detect_language
@@ -60,7 +66,8 @@ async def _run_review(
     "fmt",
     default="text",
     show_default=True,
-    type=click.Choice(["text", "json", "markdown"]),
+    type=click.Choice(["text", "json", "markdown", "sarif"]),
+    help="sarif = SARIF 2.1.0 for GitHub code scanning.",
 )
 @click.option("--output", "-o", default="", help="Write the report to this file.")
 @click.option(
@@ -77,6 +84,7 @@ def review(ctx, path, path_opt, language, fmt, output, allow_partial):
     Example:
       crewforge review app.py
       crewforge review src/ -f json -o review.json
+      crewforge review src/ -f sarif -o review.sarif
     """
     target = path or path_opt
     if not target:
@@ -116,11 +124,18 @@ def review(ctx, path, path_opt, language, fmt, output, allow_partial):
             allow_partial=allow_partial,
         )
     )
-    _display_results(result, fmt, output)
+    _display_results(result, fmt, output, sources, target)
 
 
-def _display_results(result: ReviewResult, fmt: str, output: str) -> None:
-    text = json.dumps(report_dict(result), indent=2) if fmt == "json" else report_markdown(result)
+def _display_results(
+    result: ReviewResult, fmt: str, output: str, sources: Sources, target: str
+) -> None:
+    if fmt == "json":
+        text = json.dumps(report_dict(result), indent=2)
+    elif fmt == "sarif":
+        text = json.dumps(report_sarif(result, sources, target), indent=2)
+    else:
+        text = report_markdown(result)
     if output:
         Path(output).write_text(text, encoding="utf-8")
         err_console.print(f"[green]Report written to {output}[/]")

crewforge-0.3.0/src/crewforge/recipes/review/report.py

@@ -0,0 +1,191 @@
+"""Render a :class:`ReviewResult` as JSON, SARIF or Markdown."""
+
+from __future__ import annotations
+
+from pathlib import Path
+from typing import Any
+
+from ... import __version__
+from .agents import SEVERITIES
+from .pipeline import ReviewResult
+from .sources import Sources
+
+SARIF_SCHEMA = "https://json.schemastore.org/sarif-2.1.0.json"
+#: Finding severity -> SARIF ``level`` (GitHub code scanning shows error/warning/note).
+SARIF_LEVELS = {"critical": "error", "error": "error", "warning": "warning", "info": "note"}
+
+
+def report_dict(result: ReviewResult) -> dict[str, Any]:
+    """Machine-readable report: decision, per-agent findings/status, usage."""
+    summary = result.summary_result
+    return {
+        "status": result.status,
+        "language": result.language or "auto",
+        "decision": (summary.data.get("decision") if summary else None),
+        "quality_score": (summary.data.get("quality_score") if summary else None),
+        "total_findings": result.total_findings,
+        "total_tokens": result.total_tokens,
+        "duration_s": round(result.duration_s, 2),
+        "agents": {
+            name: {
+                "status": r.status,
+                "summary": r.summary,
+                "findings": r.findings,
+                "tokens": r.tokens_used,
+                "error": r.error,
+            }
+            for name, r in result.agent_results.items()
+        },
+        "summary": (
+            {
+                "status": summary.status,
+                "report": summary.summary,
+                "top_issues": summary.data.get("top_issues", []),
+                "error": summary.error,
+            }
+            if summary
+            else None
+        ),
+        "usage": result.usage,
+    }
+
+
+def _finding_line(agent: str, f: dict[str, Any]) -> str:
+    where = f" (line {f['line']})" if f.get("line") not in (None, "") else ""
+    cwe = f" [{f['cwe']}]" if f.get("cwe") else ""
+    text = f"- **{f.get('title') or 'Untitled'}**{where}{cwe} — _{agent}_"
+    if f.get("description"):
+        text += f": {f['description']}"
+    if f.get("suggestion"):
+        text += f"\n  - Fix: {f['suggestion']}"
+    return text
+
+
+def report_markdown(result: ReviewResult) -> str:
+    """Human-readable Markdown report."""
+    lines = ["# crewforge review", ""]
+    summary = result.summary_result
+    if summary and summary.ok:
+        lines += [
+            f"**Decision:** {summary.data.get('decision')} · "
+            f"**Quality score:** {summary.data.get('quality_score')}/100",
+            "",
+        ]
+        if summary.summary.strip():
+            lines += ["## Summary", "", summary.summary.strip(), ""]
+        top = summary.data.get("top_issues") or []
+        if top:
+            lines += ["### Top issues", ""] + [f"{i}. {t}" for i, t in enumerate(top, 1)] + [""]
+    elif summary:
+        lines += ["## Summary", "", f"_Summary agent failed: {summary.error}_", ""]
+
+    lines += ["## Findings", ""]
+    any_finding = False
+    for severity in SEVERITIES:
+        rows = [
+            _finding_line(name, f)
+            for name, r in result.agent_results.items()
+            for f in r.findings
+            if f.get("severity") == severity
+        ]
+        if rows:
+            any_finding = True
+            lines += [f"### {severity.capitalize()} ({len(rows)})", ""] + rows + [""]
+    if not any_finding:
+        lines += ["_No findings reported._", ""]
+
+    lines += ["## Reviewers", ""]
+    for name, r in result.agent_results.items():
+        if r.ok:
+            note = r.summary.strip() or "no summary"
+            lines.append(f"- **{name}** ({r.finding_count} finding(s)): {note}")
+        else:
+            lines.append(f"- **{name}**: _Agent failed: {r.error}_")
+    lines.append("")
+    agents_run = len(result.agent_results) + (1 if summary else 0)
+    lines.append(
+        f"_Tokens: {result.total_tokens:,} (provider-reported) · "
+        f"Duration: {result.duration_s:.1f}s · Agents: {agents_run}_"
+    )
+    return "\n".join(lines) + "\n"
+
+
+def _uri(path: Path) -> str:
+    """Artifact URI: relative paths stay relative (GitHub resolves them from the repo root)."""
+    return path.as_uri() if path.is_absolute() else path.as_posix()
+
+
+def _int_line(value: Any) -> int | None:
+    if isinstance(value, bool):
+        return None
+    if isinstance(value, int):
+        return value if value > 0 else None
+    if isinstance(value, str) and value.strip().isdigit():
+        return int(value) or None
+    return None
+
+
+def report_sarif(result: ReviewResult, sources: Sources, target: str | Path) -> dict[str, Any]:
+    """SARIF 2.1.0 log for GitHub code scanning: one ``result`` per finding.
+
+    A finding's line refers to the text the reviewers saw. For a directory that
+    is every file joined under ``# File:`` headers, so the line is mapped back to
+    its file; a line that cannot be mapped keeps the reviewed path, without a region.
+    """
+    results: list[dict[str, Any]] = []
+    rules: dict[str, dict[str, Any]] = {}
+    for agent, r in result.agent_results.items():
+        for f in r.findings:
+            rule_id = str(f.get("cwe") or agent)
+            rules.setdefault(rule_id, {"id": rule_id})
+            title = str(f.get("title") or "Untitled")
+            message = f"{title}: {f['description']}" if f.get("description") else title
+            if f.get("suggestion"):
+                message += f"\nFix: {f['suggestion']}"
+            physical: dict[str, Any] = {"artifactLocation": {"uri": _uri(Path(target))}}
+            line = _int_line(f.get("line"))
+            spot = sources.locate(line) if line is not None else None
+            if spot is not None:
+                physical = {
+                    "artifactLocation": {"uri": _uri(spot[0])},
+                    "region": {"startLine": spot[1]},
+                }
+            results.append(
+                {
+                    "ruleId": rule_id,
+                    "level": SARIF_LEVELS.get(str(f.get("severity")), "note"),
+                    "message": {"text": message},
+                    "locations": [{"physicalLocation": physical}],
+                    "properties": {"agent": agent, "severity": f.get("severity", "info")},
+                }
+            )
+    return {
+        "$schema": SARIF_SCHEMA,
+        "version": "2.1.0",
+        "runs": [
+            {
+                "tool": {
+                    "driver": {
+                        "name": "crewforge",
+                        "version": __version__,
+                        "informationUri": "https://github.com/aimanmalib/crewforge",
+                        "rules": list(rules.values()),
+                    }
+                },
+                "invocations": [
+                    {
+                        "executionSuccessful": not result.failed_agents,
+                        "toolExecutionNotifications": [
+                            {
+                                "level": "error",
+                                "message": {"text": f"agent '{name}' failed: {r.error}"},
+                            }
+                            for name, r in result.agent_results.items()
+                            if not r.ok
+                        ],
+                    }
+                ],
+                "results": results,
+            }
+        ],
+    }

{crewforge-0.2.2 → crewforge-0.3.0}/src/crewforge/recipes/review/sources.py

@@ -52,6 +52,19 @@ class Sources:
     language: str
     files: list[Path] = field(default_factory=list)
     skipped: list[str] = field(default_factory=list)
+    #: ``(file, first, last)``: lines ``first..last`` of :attr:`code` hold line 1.. of ``file``.
+    spans: list[tuple[Path, int, int]] = field(default_factory=list)
+
+    def locate(self, line: int) -> tuple[Path, int] | None:
+        """Map a line of :attr:`code` to ``(file, line in that file)``.
+
+        ``None`` when the line is outside the code or falls on a ``# File:``
+        header or the blank lines between two files.
+        """
+        for path, first, last in self.spans:
+            if first <= line <= last:
+                return path, line - first + 1
+        return None
 
 
 def collect(path: str | Path, *, max_file_bytes: int = DEFAULT_MAX_FILE_BYTES) -> Sources:
@@ -60,13 +73,20 @@ def collect(path: str | Path, *, max_file_bytes: int = DEFAULT_MAX_FILE_BYTES) -
         code = target.read_text(encoding="utf-8", errors="replace")
         if not code.strip():
             raise SourceError(f"{target} is empty")
-        return Sources(code=code, language=detect_language(target.suffix), files=[target])
+        return Sources(
+            code=code,
+            language=detect_language(target.suffix),
+            files=[target],
+            spans=[(target, 1, _line_count(code))],
+        )
     if not target.is_dir():
         raise SourceError(f"path not found: {target}")
 
     parts: list[str] = []
     files: list[Path] = []
     skipped: list[str] = []
+    spans: list[tuple[Path, int, int]] = []
+    start = 1  # line of ``code`` where the next "# File:" header goes
     languages: set[str] = set()
     for f in sorted(target.rglob("*")):
         rel = f.relative_to(target)
@@ -80,10 +100,20 @@ def collect(path: str | Path, *, max_file_bytes: int = DEFAULT_MAX_FILE_BYTES) -
         text = f.read_text(encoding="utf-8", errors="replace")
         if not text.strip():
             continue
-        parts.append(f"# File: {rel}\n{text}")
+        part = f"# File: {rel}\n{text}"
+        parts.append(part)
         files.append(f)
+        spans.append((f, start + 1, start + _line_count(text)))
+        start += part.count("\n") + 2  # the part, then the blank line joining the next one
         languages.add(detect_language(f.suffix))
     if not parts:
         raise SourceError(f"no supported source files found under {target}")
     language = languages.pop() if len(languages) == 1 else ""
-    return Sources(code="\n\n".join(parts), language=language, files=files, skipped=skipped)
+    return Sources(
+        code="\n\n".join(parts), language=language, files=files, skipped=skipped, spans=spans
+    )
+
+
+def _line_count(text: str) -> int:
+    """Lines in ``text`` as an editor numbers them (a final newline adds none)."""
+    return text.count("\n") + (0 if text.endswith("\n") else 1)

{crewforge-0.2.2 → crewforge-0.3.0/src/crewforge.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: crewforge
-Version: 0.2.2
+Version: 0.3.0
 Summary: One crew of LLM agents, five recipes: code review, document Q&A, API scaffolding, content and honest academic-writing help — on any OpenAI-compatible provider.
 Author: Muhammad Aiman
 License-Expression: MIT
@@ -145,7 +145,7 @@ tables are trimmed to their totals.
 
 | Command | Agents | What you get |
 |---|---|---|
-| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI |
+| `review PATH` | security, style, complexity, logic, testing, documentation (parallel) → summary | Findings by severity, a decision and a score; `-f json` for CI, `-f sarif` for GitHub code scanning |
 | `ingest FILES` | ingester, indexer (no LLM) | How each file was parsed and chunked |
 | `ask FILES -q Q [--verify] [--rerank]` | retriever → answerer → fact-checker | An answer with `[Source N]` citations and a claim-by-claim verdict |
 | `summarize FILES` | summarizer | Summary and key points per document |
@@ -177,6 +177,30 @@ $ crewforge review app.py -f markdown -o review.md      # app.py builds SQL with
 total   7 calls   22,366 tokens (provider-reported)
 ```
 
+`-f sarif` writes a [Static Analysis Results Interchange Format 2.1.0](https://docs.oasis-open.org/sarif/sarif/v2.1.0/sarif-v2.1.0.html)
+log (`sarif`), the format GitHub code scanning reads, so findings can appear in a repository's Security tab. Each finding becomes one
+result: `critical` and `error` map to level `error`, `warning` to `warning`, `info` to `note`; the rule is the
+finding's CWE when it has one, otherwise the reviewer's name. When you review a directory, the line a reviewer
+reports is mapped back to the file it came from. A finding whose line is missing or outside the code points at the
+reviewed path without a line. Reviewers that fail are listed in `invocations`, and the exit code is the same as for
+the other formats.
+
+```yaml
+# .github/workflows/review.yml (needs an API key secret, e.g. OPENAI_API_KEY)
+permissions:
+  contents: read
+  security-events: write
+steps:
+  - uses: actions/checkout@v4
+  - run: pipx run crewforge review src/ -f sarif -o crewforge.sarif
+    env:
+      OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
+  - uses: github/codeql-action/upload-sarif@v3
+    if: always()
+    with:
+      sarif_file: crewforge.sarif
+```
+
 ### `ask` and `summarize`
 
 <p align="center">
@@ -473,9 +497,13 @@ pip install crewforge                 # Python 3.10 hingga 3.14
 pip install "crewforge[docs,api]"     # pilihan: PDF/DOCX dan semakan aplikasi FastAPI
 export OPENAI_API_KEY=sk-...          # atau penyedia lain, lihat `crewforge providers`
 crewforge review src/
+crewforge review src/ -f sarif -o crewforge.sarif   # untuk GitHub code scanning
 crewforge academic feedback draf.md --brief taklimat.md --language ms
 ```
 
+`review -f sarif` menulis laporan `SARIF` 2.1.0 yang dibaca GitHub code scanning: setiap dapatan menjadi satu
+keputusan dalam tab Security, dan baris dalam semakan folder dipetakan kembali ke fail asalnya.
+
 Senarai penyedia, contoh setiap resipi dan seni bina ada dalam bahagian bahasa Inggeris di atas
 ([Providers](https://github.com/aimanmalib/crewforge#providers), [Recipes](https://github.com/aimanmalib/crewforge#recipes)).
 

crewforge-0.2.2/src/crewforge/recipes/review/report.py

@@ -1,103 +0,0 @@
-"""Render a :class:`ReviewResult` as JSON or Markdown."""
-
-from __future__ import annotations
-
-from typing import Any
-
-from .agents import SEVERITIES
-from .pipeline import ReviewResult
-
-
-def report_dict(result: ReviewResult) -> dict[str, Any]:
-    """Machine-readable report: decision, per-agent findings/status, usage."""
-    summary = result.summary_result
-    return {
-        "status": result.status,
-        "language": result.language or "auto",
-        "decision": (summary.data.get("decision") if summary else None),
-        "quality_score": (summary.data.get("quality_score") if summary else None),
-        "total_findings": result.total_findings,
-        "total_tokens": result.total_tokens,
-        "duration_s": round(result.duration_s, 2),
-        "agents": {
-            name: {
-                "status": r.status,
-                "summary": r.summary,
-                "findings": r.findings,
-                "tokens": r.tokens_used,
-                "error": r.error,
-            }
-            for name, r in result.agent_results.items()
-        },
-        "summary": (
-            {
-                "status": summary.status,
-                "report": summary.summary,
-                "top_issues": summary.data.get("top_issues", []),
-                "error": summary.error,
-            }
-            if summary
-            else None
-        ),
-        "usage": result.usage,
-    }
-
-
-def _finding_line(agent: str, f: dict[str, Any]) -> str:
-    where = f" (line {f['line']})" if f.get("line") not in (None, "") else ""
-    cwe = f" [{f['cwe']}]" if f.get("cwe") else ""
-    text = f"- **{f.get('title') or 'Untitled'}**{where}{cwe} — _{agent}_"
-    if f.get("description"):
-        text += f": {f['description']}"
-    if f.get("suggestion"):
-        text += f"\n  - Fix: {f['suggestion']}"
-    return text
-
-
-def report_markdown(result: ReviewResult) -> str:
-    """Human-readable Markdown report."""
-    lines = ["# crewforge review", ""]
-    summary = result.summary_result
-    if summary and summary.ok:
-        lines += [
-            f"**Decision:** {summary.data.get('decision')} · "
-            f"**Quality score:** {summary.data.get('quality_score')}/100",
-            "",
-        ]
-        if summary.summary.strip():
-            lines += ["## Summary", "", summary.summary.strip(), ""]
-        top = summary.data.get("top_issues") or []
-        if top:
-            lines += ["### Top issues", ""] + [f"{i}. {t}" for i, t in enumerate(top, 1)] + [""]
-    elif summary:
-        lines += ["## Summary", "", f"_Summary agent failed: {summary.error}_", ""]
-
-    lines += ["## Findings", ""]
-    any_finding = False
-    for severity in SEVERITIES:
-        rows = [
-            _finding_line(name, f)
-            for name, r in result.agent_results.items()
-            for f in r.findings
-            if f.get("severity") == severity
-        ]
-        if rows:
-            any_finding = True
-            lines += [f"### {severity.capitalize()} ({len(rows)})", ""] + rows + [""]
-    if not any_finding:
-        lines += ["_No findings reported._", ""]
-
-    lines += ["## Reviewers", ""]
-    for name, r in result.agent_results.items():
-        if r.ok:
-            note = r.summary.strip() or "no summary"
-            lines.append(f"- **{name}** ({r.finding_count} finding(s)): {note}")
-        else:
-            lines.append(f"- **{name}**: _Agent failed: {r.error}_")
-    lines.append("")
-    agents_run = len(result.agent_results) + (1 if summary else 0)
-    lines.append(
-        f"_Tokens: {result.total_tokens:,} (provider-reported) · "
-        f"Duration: {result.duration_s:.1f}s · Agents: {agents_run}_"
-    )
-    return "\n".join(lines) + "\n"