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("==================================================") 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', {