fix: preserve contract behavior ordering

Co-Authored-By: Codex <noreply@openai.com>

Author: Pigbibi

scripts/run_codex_pr_review.py

@@ -811,7 +811,7 @@ def _normalize_contract_text(value: Any) -> str:
         "must", "should", "will", "return",
     }
     return " ".join(
-        sorted(token for token in re.findall(r"[a-z0-9]+", text) if token not in stop_words)
+        token for token in re.findall(r"[a-z0-9]+", text) if token not in stop_words
     )
 
 
@@ -842,9 +842,8 @@ def _behavior_digest(finding: dict[str, Any]) -> str:
     payload = json.dumps(
         {
             "contract_key": _contract_key(finding),
-            "behavior": _normalize_contract_text(
-                finding.get("suggestion") or finding.get("description")
-            ),
+            "description": _normalize_contract_text(finding.get("description")),
+            "suggestion": _normalize_contract_text(finding.get("suggestion")),
         },
         sort_keys=True,
         separators=(",", ":"),

tests/test_run_codex_pr_review.py

@@ -121,7 +121,9 @@ def test_replay_fixtures_cover_pure_contract_identity(self) -> None:
         crypto = fixtures["crypto_pr_125"]["findings"]
         dispatch_identity = run_codex_pr_review._contract_finding(dispatch["finding"])
         reworded_identity = run_codex_pr_review._contract_finding(dispatch["reworded"])
-        self.assertEqual(dispatch_identity["fingerprint_v2"], reworded_identity["fingerprint_v2"])
+        self.assertEqual(dispatch_identity["contract_key"], reworded_identity["contract_key"])
+        self.assertNotEqual(dispatch_identity["behavior_digest"], reworded_identity["behavior_digest"])
+        self.assertNotEqual(dispatch_identity["fingerprint_v2"], reworded_identity["fingerprint_v2"])
         self.assertNotEqual(
             run_codex_pr_review._contract_key(crypto[0]),
             run_codex_pr_review._contract_key(crypto[1]),
@@ -148,6 +150,30 @@ def test_no_anchor_subject_combines_description_and_suggestion(self) -> None:
             run_codex_pr_review._contract_key(first),
             run_codex_pr_review._contract_key(second),
         )
+        self.assertNotEqual(
+            run_codex_pr_review._behavior_digest(first),
+            run_codex_pr_review._behavior_digest(second),
+        )
+        self.assertNotEqual(
+            run_codex_pr_review._fingerprint_v2(first),
+            run_codex_pr_review._fingerprint_v2(second),
+        )
+
+    def test_ordered_contract_text_distinguishes_reversed_behavior(self) -> None:
+        before = {
+            "category": "logic", "file": "service/review.py",
+            "description": "Validate A before B.",
+            "suggestion": "Use the required order.",
+        }
+        after = dict(before, description="Validate B before A.")
+        self.assertEqual(
+            run_codex_pr_review._normalize_contract_text(before["description"]),
+            "validate a before b",
+        )
+        self.assertNotEqual(
+            run_codex_pr_review._behavior_digest(before),
+            run_codex_pr_review._behavior_digest(after),
+        )
 
     def test_parse_arbitration_output_requires_supported_verdict(self) -> None:
         self.assertEqual(