Fix revision loop: add critique_details to AgentState

Bug: critique_details was not defined in AgentState TypedDict, causing
LangGraph to drop it between nodes. Revisions never triggered because
analyzer couldn't see critic's REJECTED status.

- Add critique_details: Optional[dict] to AgentState
- Add analyzer_revision_skipped: Optional[bool] to AgentState
- Add debug logging to trace state flow

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

src/nodes/analyzer.py

@@ -1489,6 +1489,9 @@ def analyzer_node(state, workflow_id=None, progress_store=None):
     critique_details = state.get("critique_details", {})
     is_revision = bool(critique_details) and critique_details.get("status") == "REJECTED"
 
+    # Debug: Log critique details presence
+    print(f"[DEBUG] Analyzer: critique_details={bool(critique_details)}, status={critique_details.get('status')}, is_revision={is_revision}")
+
     if is_revision and critique_details:
         # REVISION MODE: Use enhanced revision prompt with Critic feedback
         current_revision = state.get("revision_count", 0) + 1

src/nodes/critic.py

@@ -560,6 +560,9 @@ def critic_node(state, workflow_id=None, progress_store=None):
         "actionable_feedback": result.get("actionable_feedback", []),
     }
 
+    # Debug: Log what's being set in critique_details
+    print(f"[DEBUG] Critic: Setting critique_details status={status}, score={weighted_score:.1f}")
+
     # Update progress
     if workflow_id and progress_store:
         progress_store[workflow_id].update({

src/state.py

@@ -7,6 +7,7 @@ class AgentState(TypedDict):
     raw_data: Optional[str]
     draft_report: Optional[str]
     critique: Optional[str]
+    critique_details: Optional[dict]  # Structured critique for revision loop
     revision_count: int
     score: int
     messages: List[str]
@@ -25,3 +26,5 @@ class AgentState(TypedDict):
     # Metric reference for hallucination prevention (Layer 1)
     metric_reference: Optional[dict]  # {M01: {key, raw_value, formatted, as_of_date}, ...}
     metric_reference_hash: Optional[str]  # SHA256 hash for integrity verification
+    # Revision tracking
+    analyzer_revision_skipped: Optional[bool]  # True if revision was skipped due to LLM failure