-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathrenderer.py
More file actions
168 lines (137 loc) · 6.3 KB
/
Copy pathrenderer.py
File metadata and controls
168 lines (137 loc) · 6.3 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
"""Deterministic local renderer for PR Review Memory handoffs."""
from __future__ import annotations
import re
from collections.abc import Iterable, Mapping, Set
from typing import Any
REQUIRED_FIELDS = ("repository", "pr_number", "branch", "head_sha", "validation_summary", "next_action")
DIFF_MARKER_PREFIXES = ("diff --git", "index ", "@@", "+++", "---")
SECRET_PATTERN = re.compile(
r"(?i)\b(api[_-]?key|secret|token|password)\b\s*[:=]\s*(?:\"[^\"]*\"|'[^']*'|[^\s,;]+)"
)
def render_pr_review_memory_handoff(data: dict[str, Any]) -> str:
"""Render structured PR review memory as compact Token Saver handoff markdown."""
if not isinstance(data, dict):
raise TypeError("PR review memory data must be a dictionary.")
missing = [field for field in REQUIRED_FIELDS if _is_missing_required(field, data.get(field))]
if missing:
raise ValueError(f"Missing required PR review memory field(s): {', '.join(missing)}")
lines = [
"# PR Review Memory",
"",
f"Repository: {_clean_text(data['repository'])}",
f"PR: {_format_pr(data['pr_number'], data.get('pr_url'))}",
f"Branch: {_clean_text(data['branch'])}",
f"Head SHA: {_clean_text(data['head_sha'])}",
]
if not _is_empty(data.get("status")):
lines.append(f"Review status: {_clean_text(data['status'])}")
_append_item_section(lines, "Actionable comments", data.get("actionable_items"))
_append_item_section(lines, "Fixes applied", data.get("completed_fixes"))
_append_validation(lines, data.get("validation_summary"))
_append_item_section(lines, "Resolved threads", data.get("resolved_items"))
_append_item_section(lines, "Unresolved threads", data.get("unresolved_items"))
_append_merge_readiness(lines, data.get("merge_readiness"))
lines.append(f"Next action: {_clean_text(data['next_action'])}")
return "\n".join(lines).rstrip() + "\n"
def _is_empty(value: Any) -> bool:
return value is None or value == "" or value == [] or value == {}
def _is_missing_required(field: str, value: Any) -> bool:
if _is_empty(value):
return True
if field == "validation_summary":
return not _validation_parts(value)
return _clean_text(value) == ""
def _clean_text(value: Any) -> str:
text = str(value)
kept_lines = []
for line in text.splitlines():
stripped = line.strip()
if any(stripped.startswith(prefix) for prefix in DIFF_MARKER_PREFIXES):
continue
kept_lines.append(stripped)
cleaned = " ".join(part for part in kept_lines if part)
cleaned = SECRET_PATTERN.sub(lambda match: f"{match.group(1)}=<redacted>", cleaned)
return cleaned
def _format_pr(pr_number: Any, pr_url: Any) -> str:
number = _clean_text(pr_number)
if not number.startswith("#"):
number = f"#{number}"
rendered_url = _clean_text(pr_url)
if not rendered_url:
return number
return f"{number} ({rendered_url})"
def _append_item_section(lines: list[str], label: str, items: Any) -> None:
normalized = _normalize_items(items)
if not normalized:
return
rendered_items = [_render_item(item) for item in normalized]
rendered_items = [item for item in rendered_items if item]
if not rendered_items:
return
lines.append(f"{label}:")
for rendered in rendered_items:
lines.append(f"- {rendered}")
def _normalize_items(items: Any) -> list[Any]:
if _is_empty(items):
return []
if isinstance(items, Set) and not isinstance(items, (str, bytes, Mapping)):
return sorted(items, key=_clean_text)
if isinstance(items, Iterable) and not isinstance(items, (str, bytes, Mapping)):
return list(items)
return [items]
def _render_item(item: Any) -> str:
if isinstance(item, Mapping):
parts: list[str] = []
thread_id = item.get("thread_id") or item.get("id")
if not _is_empty(thread_id):
parts.append(str(thread_id))
file_path = item.get("file")
if not _is_empty(file_path):
parts.append(_clean_text(file_path))
files = item.get("files")
if isinstance(files, Iterable) and not isinstance(files, (str, bytes, Mapping)):
file_values = sorted(files, key=_clean_text) if isinstance(files, Set) else files
rendered_files = ", ".join(_clean_text(file) for file in file_values if not _is_empty(file))
if rendered_files:
parts.append(rendered_files)
summary = item.get("summary") or item.get("reason") or item.get("status")
if not _is_empty(summary):
parts.append(_clean_text(summary))
return " - ".join(part for part in parts if part)
return _clean_text(item)
def _append_validation(lines: list[str], validation: Any) -> None:
parts = _validation_parts(validation)
if parts:
lines.append(f"Validation: {'; '.join(parts)}")
def _validation_parts(validation: Any) -> list[str]:
if isinstance(validation, Mapping):
parts = []
for command, status in sorted(validation.items()):
rendered_command = _clean_text(command)
rendered_status = _clean_text(status)
if rendered_command and rendered_status:
parts.append(f"{rendered_command}: {rendered_status}")
return parts
if isinstance(validation, Iterable) and not isinstance(validation, (str, bytes)):
return [_clean_text(item) for item in validation if not _is_empty(item) and _clean_text(item)]
rendered = _render_item(validation)
return [rendered] if rendered else []
def _append_merge_readiness(lines: list[str], readiness: Any) -> None:
if _is_empty(readiness):
return
if isinstance(readiness, Mapping):
status = readiness.get("status")
reason = readiness.get("reason")
parts = []
rendered_status = _clean_text(status) if not _is_empty(status) else ""
rendered_reason = _clean_text(reason) if not _is_empty(reason) else ""
if rendered_status:
parts.append(rendered_status)
if rendered_reason:
parts.append(rendered_reason)
if parts:
lines.append(f"Merge readiness: {' - '.join(parts)}")
return
rendered_readiness = _clean_text(readiness)
if rendered_readiness:
lines.append(f"Merge readiness: {rendered_readiness}")