Fix parse_uncertain unconditional, stale previous-review, gpt-5.4 pricing

P1: parse_uncertain now unconditionally skips all write_review_state() calls
regardless of whether prior findings exist. Prevents advancing delta baseline
after any parse failure, not just when structured_findings is truthy.

P1: Branch/base mismatch block now also deletes local-review-previous.md to
prevent stale findings from leaking via --previous-review after branch switch.

P2: Update gpt-5.4 pricing to $2.50/$15.00 per 1M tokens (was $2.00/$8.00).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: igerber

.claude/commands/ai-review-local.md

@@ -228,6 +228,7 @@ if [ -f .claude/reviews/review-state.json ]; then
     if [ "$stored_branch" != "$branch_name" ] || [ "$stored_base" != "$comparison_ref" ]; then
         echo "Warning: review-state.json is from branch '$stored_branch' (base: '$stored_base'), but current is '$branch_name' (base: '$comparison_ref'). Discarding stale state."
         rm -f .claude/reviews/review-state.json
+        rm -f .claude/reviews/local-review-previous.md
         last_reviewed_commit=""
     fi
 

.claude/scripts/openai_review.py

@@ -750,7 +750,7 @@ def apply_token_budget(
 # Source: https://platform.openai.com/docs/pricing
 # MAINTENANCE: Update when OpenAI changes pricing.
 PRICING = {
-    "gpt-5.4": (2.00, 8.00),
+    "gpt-5.4": (2.50, 15.00),
     "gpt-4.1": (2.00, 8.00),
     "gpt-4.1-mini": (0.40, 1.60),
     "o3": (2.00, 8.00),
@@ -1440,14 +1440,16 @@ def main() -> None:
         current_findings, parse_uncertain = parse_review_findings(
             review_content, current_round
         )
-        if parse_uncertain and structured_findings:
+        if parse_uncertain:
             print(
                 "Warning: Could not parse findings from review output. "
-                "Preserving prior findings and review state baseline.",
+                "Preserving prior review state baseline (not advancing "
+                "last_reviewed_commit).",
                 file=sys.stderr,
             )
-            # Do NOT write review state — keep prior baseline intact so the
-            # next delta review doesn't skip unparsed code
+            # Do NOT write review state at all — keep prior baseline intact
+            # regardless of whether prior findings exist, so the next delta
+            # review doesn't skip unparsed code
         elif structured_findings:
             final_findings = merge_findings(structured_findings, current_findings)
             write_review_state(