open-code-review-toolkit 0.8.1__tar.gz → 0.8.2__tar.gz

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: open-code-review-toolkit
-Version: 0.8.1
+Version: 0.8.2
 Summary: Unofficial GitLab CI integration layer for Open Code Review
 Project-URL: Homepage, https://github.com/xeonvs/open-code-review-toolkit
 Project-URL: Repository, https://github.com/xeonvs/open-code-review-toolkit
@@ -249,6 +249,7 @@ ocr-ci --help
 The exact recommended OCR release and its verified asset checksums live in the [versioned compatibility manifest](compatibility/ocr-support.json). CI should pin that release and checksum before execution.
 The [versioned compatibility policy](docs/compatibility.md) records tested assets and evidence and describes the conservative Dependabot-like qualification workflow for later upstream releases.
 Review output defaults to English. `OCR_REVIEW_LANGUAGE` accepts another explicit language name when a project needs localized review output; for example, `OCR_REVIEW_LANGUAGE=Russian`.
+The current OCR 1.10.0 integration defaults `OCR_REVIEW_EFFORT` to `medium` for two review rounds. `low` and `high` are explicit one- and three-round alternatives; see the [configuration reference](docs/configuration.md#review-effort) for cost, budget, and precedence boundaries.
 
 Stable distributions are published to [PyPI](https://pypi.org/project/open-code-review-toolkit/) and mirrored as checksum-listed, provenance-attested assets in the corresponding [GitHub Release](https://github.com/xeonvs/open-code-review-toolkit/releases). Development snapshots are published only to TestPyPI.
 

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/README.md

@@ -22,6 +22,7 @@ ocr-ci --help
 The exact recommended OCR release and its verified asset checksums live in the [versioned compatibility manifest](compatibility/ocr-support.json). CI should pin that release and checksum before execution.
 The [versioned compatibility policy](docs/compatibility.md) records tested assets and evidence and describes the conservative Dependabot-like qualification workflow for later upstream releases.
 Review output defaults to English. `OCR_REVIEW_LANGUAGE` accepts another explicit language name when a project needs localized review output; for example, `OCR_REVIEW_LANGUAGE=Russian`.
+The current OCR 1.10.0 integration defaults `OCR_REVIEW_EFFORT` to `medium` for two review rounds. `low` and `high` are explicit one- and three-round alternatives; see the [configuration reference](docs/configuration.md#review-effort) for cost, budget, and precedence boundaries.
 
 Stable distributions are published to [PyPI](https://pypi.org/project/open-code-review-toolkit/) and mirrored as checksum-listed, provenance-attested assets in the corresponding [GitHub Release](https://github.com/xeonvs/open-code-review-toolkit/releases). Development snapshots are published only to TestPyPI.
 

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/src/ocr_toolkit/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.8.1'
-__version_tuple__ = version_tuple = (0, 8, 1)
+__version__ = version = '0.8.2'
+__version_tuple__ = version_tuple = (0, 8, 2)
 
 __commit_id__ = commit_id = None

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/src/ocr_toolkit/configure.py

@@ -19,6 +19,9 @@ class OCRRuntimeConfigError(Exception):
     """OCR runtime config from CI env is invalid."""
 
 
+REVIEW_EFFORTS = frozenset({"low", "medium", "high"})
+
+
 def _env(name: str, default: str = "") -> str:
     return os.environ.get(name, default).strip()
 
@@ -27,6 +30,15 @@ def _bool_env(name: str) -> bool:
     return _env(name).lower() == "true"
 
 
+def review_effort() -> str:
+    """Return the closed OCR review-effort preset, defaulting to medium."""
+
+    value = (_env("OCR_REVIEW_EFFORT") or "medium").lower()
+    if value not in REVIEW_EFFORTS:
+        raise OCRRuntimeConfigError("OCR_REVIEW_EFFORT must be one of: low, medium, high")
+    return value
+
+
 def build_config_updates() -> dict[str, Any]:
     """Build OCR config updates from already-normalized CI environment."""
 
@@ -41,6 +53,7 @@ def build_config_updates() -> dict[str, Any]:
     request_controls = provider.request_controls
 
     updates: dict[str, Any] = {
+        "effort": review_effort(),
         "language": review_language,
         "llm.url": llm_url,
         "llm.auth_token": llm_token,

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/src/ocr_toolkit/preflight.py

@@ -28,7 +28,7 @@ DEFAULT_REQUEST_HEADERS = {
     "Accept": "application/json",
     "User-Agent": "open-code-review-ci-preflight/1.0",
 }
-EXPECTED_OCR_VERSION = "1.9.10"
+EXPECTED_OCR_VERSION = "1.10.0"
 
 
 class PreflightError(Exception):

{open_code_review_toolkit-0.8.1 → open_code_review_toolkit-0.8.2}/src/ocr_toolkit/review_runner.py

@@ -1101,8 +1101,8 @@ def _immutable_review_refs(refs: ReviewRefs) -> ReviewRefs:
     return ReviewRefs(reader.resolve_commit(refs.base), reader.resolve_commit(refs.head))
 
 
-def _reject_owned_background(args: list[str]) -> None:
-    """Reject caller attempts to compete with the toolkit-owned bootstrap."""
+def _reject_owned_review_options(args: list[str]) -> None:
+    """Reject caller attempts to compete with toolkit-owned review artifacts."""
 
     background_files = _option_values(args, "--background-file")
     backgrounds = _option_values(args, "--background")
@@ -1118,6 +1118,16 @@ def _reject_owned_background(args: list[str]) -> None:
         raise ReviewRunnerError(
             "--preview is managed by ocr-ci review as a pre-model background qualification"
         )
+    if any(
+        argument in {"--output", "-o"}
+        or argument.startswith("--output=")
+        or (argument.startswith("-o") and len(argument) > 2)
+        for argument in args
+    ):
+        raise ReviewRunnerError(
+            "--output is managed by ocr-ci review; OCR stdout must use the toolkit-owned "
+            "private result artifact"
+        )
 
 
 def _read_bounded_artifact(path: Path, *, limit: int, label: str) -> bytes:
@@ -1794,7 +1804,7 @@ def run_evidence_review(
     except OSError as exc:
         raise ReviewRunnerError("OCR private pre-execution state is unsafe") from exc
     refs = _immutable_review_refs(_review_refs(ocr_args))
-    _reject_owned_background(ocr_args)
+    _reject_owned_review_options(ocr_args)
     _prepare_review_output_artifacts(result_path, stderr_path)
     print("OCR evidence preflight: collecting immutable review refs", file=sys.stderr)
     previous_home = os.environ.get("HOME")