From e2e63a8d0bd7592313474646e5ba98c77ce4a9fb Mon Sep 17 00:00:00 2001 From: pyob-bot Date: Sat, 9 May 2026 05:47:59 +0000 Subject: [PATCH 1/2] Refactor: Improve pending proposal outcome handling in AutoReviewer --- src/pyob/autoreviewer.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/pyob/autoreviewer.py b/src/pyob/autoreviewer.py index 8731c56..a854132 100644 --- a/src/pyob/autoreviewer.py +++ b/src/pyob/autoreviewer.py @@ -344,16 +344,18 @@ def run_pipeline(self, current_iteration: int): logger.info( f"Found pending {PR_FILE_NAME} and/or {FEATURE_FILE_NAME} from a previous run." ) - proposals_handled = self._handle_pending_proposals( + proposals_outcome = self._handle_pending_proposals( "Hit ENTER to PROCEED, type 'SKIP' to ignore", allow_delete=True, ) - if not proposals_handled: + if proposals_outcome == "SKIPPED": logger.info( "Pending proposals were not applied or deleted. Halting current pipeline iteration to await user action." ) return - changes_made = True + elif proposals_outcome == "APPLIED": + changes_made = True + # If proposals_outcome is "DELETED", changes_made remains False, allowing the scan to proceed. if not changes_made: logger.info("==================================================") From dd316dc23b3e2918e7e28e13b6dc973de05c99ae Mon Sep 17 00:00:00 2001 From: pyob-bot Date: Sat, 9 May 2026 05:53:00 +0000 Subject: [PATCH 2/2] Refactor: Unify patch review API endpoint and payload --- src/pyob/dashboard_html.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/pyob/dashboard_html.py b/src/pyob/dashboard_html.py index d8a85d8..1b5e587 100644 --- a/src/pyob/dashboard_html.py +++ b/src/pyob/dashboard_html.py @@ -292,10 +292,10 @@ async function reviewPatch(patchId, action) { try { if (action === 'approved') { - await fetch('/api/approve_patch', { + await fetch('/api/review_patch', { method: 'POST', headers: { 'Content-Type': 'application/json' }, - body: JSON.stringify({ patch_id: patchId }) + body: JSON.stringify({ patch_id: patchId, action: 'approved' }) }); } else { await fetch('/api/review_patch', {