Merge pull request #1264 from Mihir7027/fix/batch-manifest-byte-limit-message

fix(cli): use human-readable unit in batch manifest size error

Author: harry0703

cli.py

@@ -815,7 +815,7 @@ def _load_batch_manifest(raw_path: str) -> tuple[str, list[dict[str, Any]]]:
         payload = manifest_file.read(_BATCH_FILE_MAX_BYTES + 1)
     if len(payload) > _BATCH_FILE_MAX_BYTES:
         raise ValueError(
-            f"batch manifest exceeds the {_BATCH_FILE_MAX_BYTES}-byte limit"
+            f"batch manifest exceeds the 1 MiB limit"
         )
 
     try:

test/services/test_cli.py

@@ -990,7 +990,7 @@ def test_batch_manifest_limits_size_and_task_count(self):
         with tempfile.TemporaryDirectory() as temp_dir:
             oversized = Path(temp_dir) / "oversized.jsonl"
             oversized.write_bytes(b"x" * (cli._BATCH_FILE_MAX_BYTES + 1))
-            with self.assertRaisesRegex(ValueError, "byte limit"):
+            with self.assertRaisesRegex(ValueError, "1 MiB limit"):
                 cli._load_batch_manifest(str(oversized))
 
             too_many = Path(temp_dir) / "too-many.json"