Warn on empty smart-approval guardian responses
Cluster 117428. Data as of
Nothing open on GitHub. Fix merged in #117582 on 20 Sep 2026. Check it covers your case.
The short answer
- Known?
- Yes. 3 related items grouped here, 1 issue already closed.
- Fix event
- A fix was merged to main in PR #117582. Check it covers your case. No release data is tracked.
- Tracker
- The maintainers' tracker marks this
resolved, deciding PR #117582. That is separate from the fix event. - Still open
- Nothing open on GitHub.
These items address a logging gap where empty or missing guardian verdict responses in smart-approval were only logged at DEBUG level or silently ignored, and change the behavior to emit a warning instead.
Open issues0
Open PRs0
Fixes on record
-
#117582 fix(approval): empty smart-approval guardian answers now warn instead of silently escalating (#117428, salvage #117430)
files:
tests/tools/test_smart_approval_injection.py,tools/approval_smart.py
1 PR closed without merging
From the close comments: merged via another change 1
-
#117430 fix(approval): warn on empty smart-approval guardian answers · merged via another change →
262825d,#117582
Closed issues · 1
- Closed#117428Silent smart-approval failure mode: empty-verdict guardian responses log at DEBUG only (or not at all), invisible at default log levels
How this cluster was grouped
-
Link evidence (
structural links throughout): every member is connected by structural evidence: shared changed files, or a closing/superseded reference. -
Pairwise link density
0.67: the share of item pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36.1.00means every pair is linked.
Pick up a Lab task
Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.
Open the LabAt a glance
- Merged PRs
- #117582
- Closed issues
- 1
- Attempts
- 1 PR closed without merging
- Fix link checked
- 20 Sep 2026