fix: address adversarial review iteration 2 findings

- simplify-ignore: re-append missing blocks instead of full backup restore
- simplify-ignore: clear cache after fallback so next Read re-establishes state
- cross-model-reviewer: align severity schema with flowctl parsers (critical/warning/info)

Author: z23cc

agents/cross-model-reviewer.md

@@ -90,18 +90,22 @@ Each model's review MUST produce findings in this structure:
   "confidence": 0.0-1.0,
   "findings": [
     {
-      "severity": "Critical | Important | Suggestion",
-      "dimension": "Correctness | Readability | Architecture | Security | Performance",
+      "severity": "critical | warning | info",
+      "category": "Correctness | Readability | Architecture | Security | Performance",
       "file": "path/to/file.rs",
       "line": 42,
       "description": "What is wrong",
-      "suggestion": "How to fix it"
+      "suggestion": "How to fix it",
+      "confidence": 0.0-1.0,
+      "evidence": ["grep output", "test failure"]
     }
   ],
   "positives": ["At least one positive observation"]
 }
 ```
 
+**Parser compatibility:** severity uses `critical/warning/info` (matching flowctl's ReviewFinding parser). The dimension maps to the `category` field. Findings with `confidence < 0.6` are suppressed unless severity is `critical`.
+
 The consensus algorithm uses severity to weight disagreements: a `Critical` finding from any model blocks SHIP regardless of the other model's verdict. `Suggestion`-only findings do not block.
 
 ### ReviewVerdict

hooks/simplify-ignore.sh

@@ -193,9 +193,21 @@ if [ "$TOOL_NAME" = "Edit" ] || [ "$TOOL_NAME" = "Write" ]; then
     fi
   done
   if [ "$MISSING_BLOCKS" -gt 0 ]; then
-    # Restore original backup (contains unfiltered code), then re-filter
-    cat "$CACHE/${ID}.bak" > "$FILE_PATH"
-    printf 'Restored %s from backup (%d missing placeholder(s))\n' "$FILE_PATH" "$MISSING_BLOCKS" >&2
+    # Re-insert missing blocks into the edited file (not a full backup restore,
+    # which would discard legitimate edits outside protected regions).
+    # Strategy: for each missing block, append it at the end with a warning comment.
+    for bf in "$CACHE/${ID}".block.*; do
+      [ -f "$bf" ] || continue
+      h="${bf##*.}"
+      if ! grep -q "BLOCK_${h}" "$FILE_PATH" 2>/dev/null; then
+        block_content=$(cat "$bf")
+        printf '\n%s\n' "$block_content" >> "$FILE_PATH"
+        printf 'Warning: BLOCK_%s was dropped — re-appended protected block at end of file\n' "$h" >&2
+      fi
+    done
+    # Clear cache so next Read re-establishes placeholders cleanly
+    rm -f "$CACHE/${ID}.bak" "$CACHE/${ID}.path" "$CACHE/${ID}".block.* \
+      "$CACHE/${ID}".reason.* "$CACHE/${ID}".prefix.* "$CACHE/${ID}".suffix.*
     exit 0
   fi