Skip to content

Commit 6ec93f4

Browse files
committed
fix(examples): add belt-and-braces redaction to DB-stored report_json
1 parent 4342da9 commit 6ec93f4

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

examples/skills_code_review_agent/review/pipeline.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@
1818
from .findings import Finding, dedupe, gate, severity_distribution
1919
from .governance import DEFAULT_ALLOWED_SCRIPTS, GovernanceEngine
2020
from .llm_review import run_llm_review
21-
from .report import build_report, write_reports
21+
from .redaction import redact_text
22+
from .report import build_report, render_markdown, write_reports
2223
from .sandbox import DIFF_WS_PATH, SandboxSession, create_runtime
2324

2425
DEFAULT_SKILL_ROOT = str(Path(__file__).resolve().parents[1] / "skills" / "code-review")
@@ -165,7 +166,10 @@ async def run_review(opts: ReviewOptions) -> ReviewResult:
165166
filter_events=filter_events, sandbox_outcomes=sandbox_outcomes,
166167
metrics=metrics, llm_summary=llm_summary, warnings=warnings)
167168
json_path, md_path = write_reports(report, opts.output_dir)
168-
await store.add_report(task_id, report, Path(md_path).read_text(encoding="utf-8"))
169+
report_json_text = redact_text(json.dumps(report, indent=2, ensure_ascii=False, default=str))
170+
report_for_db = json.loads(report_json_text)
171+
md_text = render_markdown(report)
172+
await store.add_report(task_id, report_for_db, md_text)
169173
await store.update_task(task_id, status="completed",
170174
diff_summary=diff_summary, finished=True)
171175
return ReviewResult(task_id=task_id, report=report,

0 commit comments

Comments
 (0)