steroids-cli 0.9.42 → 0.10.13

package/dist/commands/loop-phases.js

@@ -1,11 +1,13 @@
 "use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.checkSafetyViolation = checkSafetyViolation;
+exports.runCoderPhase = runCoderPhase;
+exports.runReviewerPhase = runReviewerPhase;
+const global_db_js_1 = require("../runners/global-db.js");
 /**
  * Loop phase functions for coder and reviewer invocation
  * ORCHESTRATOR-DRIVEN: The orchestrator makes ALL status decisions
  */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.runCoderPhase = runCoderPhase;
-exports.runReviewerPhase = runReviewerPhase;
 const node_child_process_1 = require("node:child_process");
 const queries_js_1 = require("../database/queries.js");
 const coder_js_1 = require("../orchestrator/coder.js");
@@ -18,13 +20,11 @@ const invoke_js_1 = require("../orchestrator/invoke.js");
 const fallback_handler_js_1 = require("../orchestrator/fallback-handler.js");
 const loader_js_1 = require("../config/loader.js");
 const registry_js_1 = require("../providers/registry.js");
-const global_db_js_1 = require("../runners/global-db.js");
 function refreshParallelWorkstreamLease(projectPath, leaseFence) {
     if (!leaseFence?.parallelSessionId) {
         return;
     }
-    const { db, close } = (0, global_db_js_1.openGlobalDatabase)();
-    try {
+    (0, global_db_js_1.withGlobalDatabase)((db) => {
         const row = db
             .prepare(`SELECT id, claim_generation, runner_id
          FROM workstreams
@@ -48,10 +48,7 @@ function refreshParallelWorkstreamLease(projectPath, leaseFence) {
         if (updateResult.changes !== 1) {
             throw new Error('Parallel workstream lease fence check failed');
         }
-    }
-    finally {
-        close();
-    }
+    });
 }
 /**
  * Check a coder/reviewer result for credit exhaustion or rate limits using the provider's classifier.
@@ -127,7 +124,35 @@ async function checkNonRetryableProviderFailure(result, role, projectPath, revie
     const resolvedClassification = classification === undefined ? await classifyProviderResult(result, role, projectPath, reviewerConfig) : classification;
     if (!resolvedClassification)
         return null;
-    if (resolvedClassification.type === 'model_not_found' || resolvedClassification.type === 'context_exceeded') {
+    if (resolvedClassification.type === 'model_not_found' ||
+        resolvedClassification.type === 'context_exceeded') {
+        return {
+            type: resolvedClassification.type,
+            provider: providerName,
+            model: modelName,
+            message: resolvedClassification.message,
+        };
+    }
+    return null;
+}
+/**
+ * Check for policy or safety violations
+ */
+async function checkSafetyViolation(result, role, projectPath, reviewerConfig, classification) {
+    if (result.success)
+        return null;
+    const config = (0, loader_js_1.loadConfig)(projectPath);
+    const roleConfig = reviewerConfig || config.ai?.[role];
+    const providerName = roleConfig?.provider;
+    const modelName = roleConfig?.model;
+    if (!providerName || !modelName)
+        return null;
+    const resolvedClassification = classification === undefined ? await classifyProviderResult(result, role, projectPath, reviewerConfig) : classification;
+    if (!resolvedClassification)
+        return null;
+    if (resolvedClassification.type === 'safety_violation' ||
+        resolvedClassification.type === 'policy_violation' ||
+        resolvedClassification.type === 'invalid_prompt') {
         return {
             type: resolvedClassification.type,
             provider: providerName,
@@ -196,11 +221,15 @@ function countConsecutiveOrchestratorFallbackEntries(db, taskId, marker) {
         const entry = audit[i];
         if (entry.actor !== 'orchestrator')
             break;
-        if ((entry.notes ?? '').includes(marker)) {
+        // Use category and error_code instead of marker string search
+        const isFallback = entry.category === 'fallback';
+        const matchesCode = entry.error_code === marker;
+        if (isFallback && matchesCode) {
             count += 1;
-            continue;
         }
-        break;
+        else {
+            break;
+        }
     }
     return count;
 }
@@ -380,13 +409,14 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
     }
     const initialSha = (0, status_js_1.getCurrentCommitSha)(projectPath) || '';
     const coderConfigSnapshot = (0, loader_js_1.loadConfig)(projectPath).ai?.coder;
-    const coderResult = await (0, coder_js_1.invokeCoder)(task, projectPath, action, coordinatorGuidance);
+    const coderResult = await (0, coder_js_1.invokeCoder)(task, projectPath, action, coordinatorGuidance, leaseFence?.runnerId);
     if (coderResult.timedOut) {
         console.warn('Coder timed out. Will retry next iteration.');
         return;
     }
     const classification = await classifyProviderResult(coderResult, 'coder', projectPath, coderConfigSnapshot);
     const hardFailureFromKnownTypes = await checkNonRetryableProviderFailure(coderResult, 'coder', projectPath, coderConfigSnapshot, classification);
+    const safetyViolation = await checkSafetyViolation(coderResult, 'coder', projectPath, coderConfigSnapshot, classification);
     // Check for credit/rate-limit classification (once).
     const creditCheck = await checkCreditExhaustion(coderResult, 'coder', projectPath, coderConfigSnapshot, classification);
     if (hardFailureFromKnownTypes) {
@@ -394,6 +424,25 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
         (0, queries_js_1.updateTaskStatus)(db, task.id, 'failed', 'orchestrator', reason);
         return;
     }
+    if (safetyViolation) {
+        // Check if we've exhausted our 2 attempts to resolve safety violations
+        const pastRejections = (0, queries_js_1.getTaskRejections)(db, task.id);
+        const safetyRejections = pastRejections.filter(r => r.notes?.includes('[safety_violation]')).length;
+        if (safetyRejections >= 2) {
+            const reason = `Task failed: Provider safety/policy violation could not be automatically resolved after 2 attempts. Error: ${safetyViolation.message}`;
+            (0, queries_js_1.updateTaskStatus)(db, task.id, 'disputed', 'orchestrator', reason);
+            if (!jsonMode) {
+                console.log(`\nTask disputed: Provider safety/policy violation could not be resolved.`);
+            }
+            return;
+        }
+        // Inject the safety violation as a rejection so the orchestrator attempts to resolve it on the next loop
+        (0, queries_js_1.rejectTask)(db, task.id, `[safety_violation] The provider blocked the request due to a safety/policy violation: ${safetyViolation.message}\n\nPlease analyze the task description and previous outputs, and adjust your approach or prompt to comply with safety policies. You have ${2 - safetyRejections} attempt(s) remaining.`, 'orchestrator', 'coder');
+        if (!jsonMode) {
+            console.log(`\n⟳ Coder hit safety violation. Rejected back to orchestrator for automatic resolution (${safetyRejections + 1}/2 attempts).`);
+        }
+        return;
+    }
     if (creditCheck) {
         return creditCheck;
     }
@@ -455,12 +504,10 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                 reasoning: `FALLBACK: Non-retryable orchestrator failure (${orchestratorFailure.type})`,
                 commits: [],
                 next_status: 'failed',
-                metadata: {
-                    files_changed: 0,
-                    confidence: 'low',
-                    exit_clean: false,
-                    has_commits: false,
-                }
+                files_changed: 0,
+                confidence: 'low',
+                exit_clean: false,
+                has_commits: false,
             });
         }
         else {
@@ -475,12 +522,10 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                     reasoning: 'FALLBACK: Orchestrator failed but coder signaled completion',
                     commits: commits.map(c => c.sha),
                     next_status: 'review',
-                    metadata: {
-                        files_changed: files_changed.length,
-                        confidence: 'low',
-                        exit_clean: true,
-                        has_commits: commits.length > 0,
-                    }
+                    files_changed: files_changed.length,
+                    confidence: 'low',
+                    exit_clean: true,
+                    has_commits: commits.length > 0,
                 });
             }
             else {
@@ -490,12 +535,10 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                     reasoning: 'FALLBACK: Orchestrator failed, defaulting to retry',
                     commits: [],
                     next_status: 'in_progress',
-                    metadata: {
-                        files_changed: 0,
-                        confidence: 'low',
-                        exit_clean: true,
-                        has_commits: false,
-                    }
+                    files_changed: 0,
+                    confidence: 'low',
+                    exit_clean: true,
+                    has_commits: false,
                 });
             }
         }
@@ -518,12 +561,10 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                 commits: commits.map(c => c.sha),
                 commit_message: has_uncommitted ? 'feat: implement task specification' : undefined,
                 next_status: 'review',
-                metadata: {
-                    files_changed: files_changed.length,
-                    confidence: 'low',
-                    exit_clean: true,
-                    has_commits: commits.length > 0,
-                }
+                files_changed: files_changed.length,
+                confidence: 'low',
+                exit_clean: true,
+                has_commits: commits.length > 0,
             };
         }
         else {
@@ -535,11 +576,8 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                     action: 'error',
                     reasoning: `Orchestrator parse failed ${consecutiveParseFallbackRetries} times; escalating to failed to stop retry loop`,
                     next_status: 'failed',
-                    metadata: {
-                        ...decision.metadata,
-                        confidence: 'low',
-                        exit_clean: false,
-                    },
+                    confidence: 'low',
+                    exit_clean: false,
                 };
             }
             else {
@@ -573,11 +611,8 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                 next_status: 'failed',
                 contract_violation: contractViolation,
                 reasoning: `${marker} ${cleanReason} (retry_limit ${consecutiveContractViolations}/${MAX_CONTRACT_VIOLATION_RETRIES})`,
-                metadata: {
-                    ...decision.metadata,
-                    confidence: 'low',
-                    exit_clean: false,
-                },
+                confidence: 'low',
+                exit_clean: false,
             };
         }
         else {
@@ -587,10 +622,7 @@ async function runCoderPhase(db, task, projectPath, action, jsonMode = false, co
                 next_status: 'in_progress',
                 contract_violation: contractViolation,
                 reasoning: `${marker} ${cleanReason} (retry ${consecutiveContractViolations}/${MAX_CONTRACT_VIOLATION_RETRIES})`,
-                metadata: {
-                    ...decision.metadata,
-                    confidence: 'medium',
-                },
+                confidence: 'medium',
             };
         }
     }
@@ -613,10 +645,12 @@ ${mandatoryLines.join('\n')}
 
 This is a mandatory orchestrator override. Implement these changes before resubmitting.
 Only use WONT_FIX if you provide exceptional technical evidence and the orchestrator explicitly accepts it.`;
-        const persistedNote = `${MUST_IMPLEMENT_MARKER}[rc=${task.rejection_count}] ${mandatoryGuidance}`;
+        const persistedNote = mandatoryGuidance;
         (0, queries_js_1.addAuditEntry)(db, task.id, task.status, task.status, 'coordinator', {
             actorType: 'orchestrator',
             notes: persistedNote,
+            category: 'must_implement',
+            metadata: { rejection_count: task.rejection_count }
         });
         coordinatorCache?.set(task.id, {
             success: true,
@@ -627,17 +661,14 @@ Only use WONT_FIX if you provide exceptional technical evidence and the orchestr
             ...decision,
             action: 'retry',
             next_status: 'in_progress',
-            reasoning: `${MUST_IMPLEMENT_MARKER} WONT_FIX override applied`,
-            metadata: {
-                ...decision.metadata,
-                confidence: 'medium',
-            },
+            reasoning: `WONT_FIX override applied (MUST_IMPLEMENT)`,
+            confidence: 'medium',
         };
     }
     // STEP 6: Log orchestrator decision for audit trail
     (0, queries_js_1.addAuditEntry)(db, task.id, task.status, decision.next_status, 'orchestrator', {
         actorType: 'orchestrator',
-        notes: `[${decision.action}] ${decision.reasoning} (confidence: ${decision.metadata.confidence})`,
+        notes: `[${decision.action}] ${decision.reasoning} (confidence: ${decision.confidence})`,
     });
     // STEP 7: Execute the decision
     switch (decision.action) {
@@ -676,7 +707,7 @@ Only use WONT_FIX if you provide exceptional technical evidence and the orchestr
                 (0, queries_js_1.updateTaskStatus)(db, task.id, 'review', 'orchestrator', decision.reasoning, submissionCommitSha);
             }
             if (!jsonMode) {
-                console.log(`\n✓ Coder complete, submitted to review (confidence: ${decision.metadata.confidence})`);
+                console.log(`\n✓ Coder complete, submitted to review (confidence: ${decision.confidence})`);
             }
             break;
         case 'stage_commit_submit':
@@ -746,7 +777,7 @@ Only use WONT_FIX if you provide exceptional technical evidence and the orchestr
                 }
                 (0, queries_js_1.updateTaskStatus)(db, task.id, 'review', 'orchestrator', `Auto-committed and submitted (${decision.reasoning})`, submissionCommitSha);
                 if (!jsonMode) {
-                    console.log(`\n✓ Auto-committed and submitted to review (confidence: ${decision.metadata.confidence})`);
+                    console.log(`\n✓ Auto-committed and submitted to review (confidence: ${decision.confidence})`);
                 }
             }
             catch (error) {
@@ -759,7 +790,7 @@ Only use WONT_FIX if you provide exceptional technical evidence and the orchestr
             break;
         case 'retry':
             if (!jsonMode) {
-                console.log(`\n⟳ Retrying coder (${decision.reasoning}, confidence: ${decision.metadata.confidence})`);
+                console.log(`\n⟳ Retrying coder (${decision.reasoning}, confidence: ${decision.confidence})`);
             }
             break;
         case 'error':
@@ -801,7 +832,7 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                 console.log(`Coordinator guidance included (decision: ${coordinatorResult.decision})`);
             }
         }
-        reviewerResults = await (0, reviewer_js_1.invokeReviewers)(task, projectPath, reviewerConfigs, coordinatorResult?.guidance, coordinatorResult?.decision);
+        reviewerResults = await (0, reviewer_js_1.invokeReviewers)(task, projectPath, reviewerConfigs, coordinatorResult?.guidance, coordinatorResult?.decision, leaseFence?.runnerId);
         // Check for hard failures and classify all results
         const classifications = await Promise.all(reviewerResults.map((res, i) => classifyProviderResult(res, 'reviewer', projectPath, reviewerConfigs[i])));
         let creditExhaustionSignal;
@@ -817,6 +848,24 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                 (0, queries_js_1.updateTaskStatus)(db, task.id, 'failed', 'orchestrator', reason);
                 return;
             }
+            const safetyViolation = await checkSafetyViolation(res, 'reviewer', projectPath, reviewerConfigs[i], classification);
+            if (safetyViolation) {
+                const pastRejections = (0, queries_js_1.getTaskRejections)(db, task.id);
+                const safetyRejections = pastRejections.filter(r => r.notes?.includes('[safety_violation]')).length;
+                if (safetyRejections >= 2) {
+                    const reason = `Task failed: Provider safety/policy violation could not be automatically resolved after 2 attempts. Error: ${safetyViolation.message}`;
+                    (0, queries_js_1.updateTaskStatus)(db, task.id, 'disputed', 'orchestrator', reason);
+                    if (!jsonMode) {
+                        console.log(`\nTask disputed: Provider safety/policy violation could not be resolved.`);
+                    }
+                    return;
+                }
+                (0, queries_js_1.rejectTask)(db, task.id, `[safety_violation] The provider blocked the request due to a safety/policy violation: ${safetyViolation.message}\n\nPlease analyze the task description and previous outputs, and adjust your approach or prompt to comply with safety policies. You have ${2 - safetyRejections} attempt(s) remaining.`, 'orchestrator', 'reviewer');
+                if (!jsonMode) {
+                    console.log(`\n⟳ Reviewer hit safety violation. Rejected back to orchestrator for automatic resolution (${safetyRejections + 1}/2 attempts).`);
+                }
+                return;
+            }
             const creditCheck = await checkCreditExhaustion(res, 'reviewer', projectPath, reviewerConfigs[i], classification);
             if (creditCheck) {
                 if (creditCheck.action === 'pause_credit_exhaustion' && !creditExhaustionSignal) {
@@ -887,7 +936,7 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                 console.log(`Coordinator guidance included (decision: ${coordinatorResult.decision})`);
             }
         }
-        reviewerResult = await (0, reviewer_js_1.invokeReviewer)(task, projectPath, coordinatorResult?.guidance, coordinatorResult?.decision);
+        reviewerResult = await (0, reviewer_js_1.invokeReviewer)(task, projectPath, coordinatorResult?.guidance, coordinatorResult?.decision, undefined, leaseFence?.runnerId);
         if (reviewerResult.timedOut) {
             if (!jsonMode) {
                 console.warn('Reviewer timed out. Will retry next iteration.');
@@ -901,6 +950,24 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
             (0, queries_js_1.updateTaskStatus)(db, task.id, 'failed', 'orchestrator', reason);
             return;
         }
+        const safetyViolation = await checkSafetyViolation(reviewerResult, 'reviewer', projectPath, undefined, classification);
+        if (safetyViolation) {
+            const pastRejections = (0, queries_js_1.getTaskRejections)(db, task.id);
+            const safetyRejections = pastRejections.filter(r => r.notes?.includes('[safety_violation]')).length;
+            if (safetyRejections >= 2) {
+                const reason = `Task failed: Provider safety/policy violation could not be automatically resolved after 2 attempts. Error: ${safetyViolation.message}`;
+                (0, queries_js_1.updateTaskStatus)(db, task.id, 'disputed', 'orchestrator', reason);
+                if (!jsonMode) {
+                    console.log(`\nTask disputed: Provider safety/policy violation could not be resolved.`);
+                }
+                return;
+            }
+            (0, queries_js_1.rejectTask)(db, task.id, `[safety_violation] The provider blocked the request due to a safety/policy violation: ${safetyViolation.message}\n\nPlease analyze the task description and previous outputs, and adjust your approach or prompt to comply with safety policies. You have ${2 - safetyRejections} attempt(s) remaining.`, 'orchestrator', 'reviewer');
+            if (!jsonMode) {
+                console.log(`\n⟳ Reviewer hit safety violation. Rejected back to orchestrator for automatic resolution (${safetyRejections + 1}/2 attempts).`);
+            }
+            return;
+        }
         // Check for credit exhaustion
         const creditCheck = await checkCreditExhaustion(reviewerResult, 'reviewer', projectPath, undefined, classification);
         if (creditCheck) {
@@ -958,12 +1025,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                     reasoning: 'FALLBACK: Multi-reviewer orchestrator failed',
                     notes: 'Review unclear, retrying',
                     next_status: 'review',
-                    metadata: {
-                        rejection_count: task.rejection_count,
-                        confidence: 'low',
-                        push_to_remote: false,
-                        repeated_issue: false,
-                    }
+                    rejection_count: task.rejection_count,
+                    confidence: 'low',
+                    push_to_remote: false,
+                    repeated_issue: false,
                 };
             }
         }
@@ -1009,12 +1074,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                             finalDecision === 'reject' ? 'in_progress' :
                                 finalDecision === 'dispute' ? 'disputed' :
                                     finalDecision === 'skip' ? 'skipped' : 'review',
-                        metadata: {
-                            rejection_count: task.rejection_count,
-                            confidence: 'low',
-                            push_to_remote: ['approve', 'dispute', 'skip'].includes(finalDecision),
-                            repeated_issue: false,
-                        }
+                        rejection_count: task.rejection_count,
+                        confidence: 'low',
+                        push_to_remote: ['approve', 'dispute', 'skip'].includes(finalDecision),
+                        repeated_issue: false,
                     };
                 }
                 else {
@@ -1023,12 +1086,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                         reasoning: 'FALLBACK: Multi-reviewer orchestrator failed and no unanimous consensus',
                         notes: 'Review unclear, retrying',
                         next_status: 'review',
-                        metadata: {
-                            rejection_count: task.rejection_count,
-                            confidence: 'low',
-                            push_to_remote: false,
-                            repeated_issue: false,
-                        }
+                        rejection_count: task.rejection_count,
+                        confidence: 'low',
+                        push_to_remote: false,
+                        repeated_issue: false,
                     };
                 }
             }
@@ -1068,12 +1129,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                             explicitDecision === 'reject' ? 'in_progress' :
                                 explicitDecision === 'dispute' ? 'disputed' :
                                     explicitDecision === 'skip' ? 'skipped' : 'review',
-                        metadata: {
-                            rejection_count: task.rejection_count,
-                            confidence: 'medium',
-                            push_to_remote: ['approve', 'dispute', 'skip'].includes(explicitDecision),
-                            repeated_issue: false,
-                        }
+                        rejection_count: task.rejection_count,
+                        confidence: 'medium',
+                        push_to_remote: ['approve', 'dispute', 'skip'].includes(explicitDecision),
+                        repeated_issue: false,
                     };
                 }
             }
@@ -1096,12 +1155,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                         explicitDecision === 'reject' ? 'in_progress' :
                             explicitDecision === 'dispute' ? 'disputed' :
                                 explicitDecision === 'skip' ? 'skipped' : 'review',
-                    metadata: {
-                        rejection_count: task.rejection_count,
-                        confidence: 'low',
-                        push_to_remote: ['approve', 'dispute', 'skip'].includes(explicitDecision),
-                        repeated_issue: false,
-                    }
+                    rejection_count: task.rejection_count,
+                    confidence: 'low',
+                    push_to_remote: ['approve', 'dispute', 'skip'].includes(explicitDecision),
+                    repeated_issue: false,
                 };
             }
             else {
@@ -1110,12 +1167,10 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                     reasoning: `FALLBACK: Orchestrator failed (${failureReason}), retrying review`,
                     notes: 'Review unclear, retrying',
                     next_status: 'review',
-                    metadata: {
-                        rejection_count: task.rejection_count,
-                        confidence: 'low',
-                        push_to_remote: false,
-                        repeated_issue: false,
-                    }
+                    rejection_count: task.rejection_count,
+                    confidence: 'low',
+                    push_to_remote: false,
+                    repeated_issue: false,
                 };
             }
         }
@@ -1130,11 +1185,8 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
                 reasoning: `Orchestrator parse failed ${consecutiveParseFallbackRetries} times; escalating to dispute`,
                 notes: 'Escalated to disputed to prevent endless unclear-review retries',
                 next_status: 'disputed',
-                metadata: {
-                    ...decision.metadata,
-                    confidence: 'low',
-                    push_to_remote: false,
-                },
+                confidence: 'low',
+                push_to_remote: false,
             };
         }
         else {
@@ -1147,7 +1199,7 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
     // STEP 5: Log orchestrator decision for audit trail
     (0, queries_js_1.addAuditEntry)(db, task.id, task.status, decision.next_status, 'orchestrator', {
         actorType: 'orchestrator',
-        notes: `[${decision.decision}] ${decision.reasoning} (confidence: ${decision.metadata.confidence})`,
+        notes: `[${decision.decision}] ${decision.reasoning} (confidence: ${decision.confidence})`,
     });
     // STEP 5.5: Create follow-up tasks if any (ONLY on approval)
     if (decision.decision === 'approve' && decision.follow_up_tasks && decision.follow_up_tasks.length > 0) {
@@ -1193,7 +1245,7 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
         case 'approve':
             (0, queries_js_1.approveTask)(db, task.id, 'orchestrator', decision.notes, commitSha);
             if (!jsonMode) {
-                console.log(`\n✓ Task APPROVED (confidence: ${decision.metadata.confidence})`);
+                console.log(`\n✓ Task APPROVED (confidence: ${decision.confidence})`);
                 console.log('Pushing to git...');
             }
             refreshParallelWorkstreamLease(projectPath, leaseFence);
@@ -1208,14 +1260,14 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
         case 'reject':
             (0, queries_js_1.rejectTask)(db, task.id, 'orchestrator', decision.notes, commitSha);
             if (!jsonMode) {
-                console.log(`\n✗ Task REJECTED (${task.rejection_count + 1}/15, confidence: ${decision.metadata.confidence})`);
+                console.log(`\n✗ Task REJECTED (${task.rejection_count + 1}/15, confidence: ${decision.confidence})`);
                 console.log('Returning to coder for fixes.');
             }
             break;
         case 'dispute':
             (0, queries_js_1.updateTaskStatus)(db, task.id, 'disputed', 'orchestrator', decision.notes, commitSha);
             if (!jsonMode) {
-                console.log(`\n! Task DISPUTED (confidence: ${decision.metadata.confidence})`);
+                console.log(`\n! Task DISPUTED (confidence: ${decision.confidence})`);
                 console.log('Pushing current work and moving to next task.');
             }
             refreshParallelWorkstreamLease(projectPath, leaseFence);
@@ -1227,7 +1279,7 @@ async function runReviewerPhase(db, task, projectPath, jsonMode = false, coordin
         case 'skip':
             (0, queries_js_1.updateTaskStatus)(db, task.id, 'skipped', 'orchestrator', decision.notes, commitSha);
             if (!jsonMode) {
-                console.log(`\n⏭ Task SKIPPED (confidence: ${decision.metadata.confidence})`);
+                console.log(`\n⏭ Task SKIPPED (confidence: ${decision.confidence})`);
             }
             break;
         case 'unclear':