xai-review 0.10.0__py3-none-any.whl → 0.11.0__py3-none-any.whl

ai_review/libs/json.py

@@ -1,19 +1,16 @@
 import re
 
 CONTROL_CHARS_RE = re.compile(r"[\x00-\x1F]")
+REPLACEMENTS = {
+    "\n": "\\n",
+    "\r": "\\r",
+    "\t": "\\t",
+}
 
 
 def sanitize_json_string(raw: str) -> str:
     def replace(match: re.Match) -> str:
         char = match.group()
-        match char:
-            case "\n":
-                return "\\n"
-            case "\r":
-                return "\\r"
-            case "\t":
-                return "\\t"
-            case _:
-                return f"\\u{ord(char):04x}"
+        return REPLACEMENTS.get(char, f"\\u{ord(char):04x}")
 
     return CONTROL_CHARS_RE.sub(replace, raw)

ai_review/services/review/inline/service.py

@@ -18,12 +18,12 @@ class InlineCommentService:
         try:
             return InlineCommentListSchema.model_validate_json(raw)
         except ValidationError as error:
-            logger.debug(f"Parse failed, trying sanitized JSON: {raw[:200]=}, {error=}")
+            logger.warning(f"Parse failed, trying sanitized JSON: {raw[:200]=}, {error=}")
             try:
                 cleaned = sanitize_json_string(raw)
                 return InlineCommentListSchema.model_validate_json(cleaned)
             except ValidationError as error:
-                logger.debug(f"Sanitized JSON still invalid: {raw[:200]=}, {error=}")
+                logger.warning(f"Sanitized JSON still invalid: {raw[:200]=}, {error=}")
                 return None
 
     @classmethod

ai_review/tests/suites/libs/test_json.py

@@ -12,30 +12,16 @@ from ai_review.libs.json import sanitize_json_string
         ("a\tb", "a\\tb"),
         ("abc\0def", "abc\\u0000def"),
         ("x\n\ry\t\0z", "x\\n\\ry\\t\\u0000z"),
+        ("\n\r\t\0", "\\n\\r\\t\\u0000"),
+        ("", "")
     ],
 )
 def test_sanitize_basic_cases(actual: str, expected: str) -> None:
     assert sanitize_json_string(actual) == expected
 
 
-def test_sanitize_multiple_control_chars() -> None:
-    raw = "A\nB\rC\tD\0E"
-    result = sanitize_json_string(raw)
-    assert result == "A\\nB\\rC\\tD\\u0000E"
-
-
 def test_sanitize_idempotent() -> None:
     raw = "foo\nbar"
     once = sanitize_json_string(raw)
     twice = sanitize_json_string(once)
     assert once == twice
-
-
-def test_sanitize_empty_string() -> None:
-    assert sanitize_json_string("") == ""
-
-
-def test_sanitize_only_control_chars() -> None:
-    raw = "\n\r\t\0"
-    result = sanitize_json_string(raw)
-    assert result == "\\n\\r\\t\\u0000"

ai_review/tests/suites/services/review/inline/test_service.py

@@ -53,6 +53,8 @@ def test_json_with_raw_newline_sanitized():
     output = '[{"file": "e.py", "line": 3, "message": "line1\nline2"}]'
     result = InlineCommentService.parse_model_output(output)
     assert len(result.root) == 1
+    assert result.root[0].file == "e.py"
+    assert result.root[0].line == 3
     assert result.root[0].message == "line1\nline2"
 
 
@@ -85,6 +87,8 @@ def test_try_parse_valid_json():
     assert isinstance(result, InlineCommentListSchema)
     assert len(result.root) == 1
     assert result.root[0].file == "ok.py"
+    assert result.root[0].line == 1
+    assert result.root[0].message == "all good"
 
 
 def test_try_parse_needs_sanitization():
@@ -92,6 +96,8 @@ def test_try_parse_needs_sanitization():
     result = InlineCommentService.try_parse_model_output(raw)
     assert result is not None
     assert result.root[0].file == "bad.py"
+    assert result.root[0].line == 2
+    assert result.root[0].message == "line1\nline2"
     assert "line1" in result.root[0].message
 
 

{xai_review-0.10.0.dist-info → xai_review-0.11.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.10.0
+Version: 0.11.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>

{xai_review-0.10.0.dist-info → xai_review-0.11.0.dist-info}/RECORD

@@ -26,7 +26,7 @@ ai_review/clients/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZ
 ai_review/clients/openai/client.py,sha256=N07ZRnbptOtab8imMUZbGL-kRoOwIZmNYbHySumD3IU,1707
 ai_review/clients/openai/schema.py,sha256=glxwMtBrDA6W0BQgH-ruKe0bKH3Ps1P-Y1-2jGdqaUM,764
 ai_review/libs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/libs/json.py,sha256=koGnjcPgHBq3DHvzr090pM1ZCPtM9TjAoqWhnZJIo1I,460
+ai_review/libs/json.py,sha256=g-P5_pNUomQ-bGHCXASvPKj9Og0s9MaLFVEAkzqGp1A,350
 ai_review/libs/logger.py,sha256=LbXR2Zk1btJ-83I-vHee7cUETgT1mHToSsqEI_8uM0U,370
 ai_review/libs/resources.py,sha256=s9taAbL1Shl_GiGkPpkkivUcM1Yh6d_IQAG97gffsJU,748
 ai_review/libs/asynchronous/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -103,7 +103,7 @@ ai_review/services/review/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJW
 ai_review/services/review/service.py,sha256=8YhRFqhZAk2pAnkDaytKSCENlOeOti1brAJq3R9tVMY,8394
 ai_review/services/review/inline/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/review/inline/schema.py,sha256=ry8sJdTgusQvFW51neRiapzgzVGwswwJzdYhaV3hbT0,1545
-ai_review/services/review/inline/service.py,sha256=qJqtjLY1En07_ZiI_wnhJHZDFfUSUHDqkMXutd1ZuMc,2131
+ai_review/services/review/inline/service.py,sha256=34I3eK2Ra1l3aUvSYfO-Wx9aPfcgCWqTgjdoGHlNO08,2135
 ai_review/services/review/policy/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/review/policy/service.py,sha256=yGDePLxAEF3N1Pkh47jGVd-4dGEESyxDXIXxV7KQfuY,2027
 ai_review/services/review/summary/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -131,7 +131,7 @@ ai_review/tests/suites/clients/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCe
 ai_review/tests/suites/clients/openai/test_client.py,sha256=Ox5ifP1C_gSeDRacBa9DnXhe__Z8-WcbzR2JH_V3xKo,1050
 ai_review/tests/suites/clients/openai/test_schema.py,sha256=x1tamS4GC9pOTpjieKDbK2D73CVV4BkATppytwMevLo,1599
 ai_review/tests/suites/libs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/libs/test_json.py,sha256=vmbkzRRyPuP-0uyLLlj-Tf_0MAWI_V2wDev9sM8tAHw,1054
+ai_review/tests/suites/libs/test_json.py,sha256=q_tvFcL9EaN5jr48Ed_mxMEmW6xXS0xlZGqSooPZBa4,713
 ai_review/tests/suites/libs/config/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/libs/config/test_prompt.py,sha256=kDMTnykC54tTPfE6cqYRBbV8d5jzAKucXdJfwtqUybM,2242
 ai_review/tests/suites/libs/diff/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -151,15 +151,15 @@ ai_review/tests/suites/services/prompt/test_service.py,sha256=M8vvBhEbyHnXCSiIRu
 ai_review/tests/suites/services/review/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/inline/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/inline/test_schema.py,sha256=tIz-1UA_GgwcdsyUqgrodiiVVmd_jhoOVmtEwzRVWiY,2474
-ai_review/tests/suites/services/review/inline/test_service.py,sha256=ZcrEWKyD-Fsu3tqAOzT1Et1bxLbX6Hn0_Jc9xE_1_78,3566
+ai_review/tests/suites/services/review/inline/test_service.py,sha256=x8d-dhw_uEXtGVLDVv4xVodmscrXIfDkoAnG4UapdlQ,3815
 ai_review/tests/suites/services/review/policy/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/policy/test_service.py,sha256=kRWT550OjWYQ7ZfsihBRc-tx-NMkhlynEsqur55RK0M,3687
 ai_review/tests/suites/services/review/summary/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/summary/test_schema.py,sha256=xSoydvABZldHaVDa0OBFvYrj8wMuZqUDN3MO-XdvxOI,819
 ai_review/tests/suites/services/review/summary/test_service.py,sha256=8UMvi_NL9frm280vD6Q1NCDrdI7K8YbXzoViIus-I2g,541
-xai_review-0.10.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
-xai_review-0.10.0.dist-info/METADATA,sha256=eiLnNDKQiqcKlXpaAL2-rkF8mHENCWJtUAUliMzeVic,9618
-xai_review-0.10.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-xai_review-0.10.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
-xai_review-0.10.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
-xai_review-0.10.0.dist-info/RECORD,,
+xai_review-0.11.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
+xai_review-0.11.0.dist-info/METADATA,sha256=HSYx4FRillzjL5CfB97QZZTxnCOlLRoRfDMg_Kufa4w,9618
+xai_review-0.11.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+xai_review-0.11.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
+xai_review-0.11.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
+xai_review-0.11.0.dist-info/RECORD,,