* feat(gitlab-ci): align review posting with the GitHub Action behaviors
Port the production-hardening of scripts/github-actions/post-review-comments.js
into examples/gitlab_ci/post_review.py so the GitLab example matches the
GitHub Action's publication semantics.
Refactor the transport from a single post() callable into a Poster object
(GitLabPoster / DryRunPoster) exposing post_note / post_discussion / list_notes
/ update_note / list_discussions / get_mr_diffs, so reads (sticky upsert,
incremental history, idempotent reconciliation) and the 400-fallback diff
inventory can flow through it.
Pure-logic ports (Group A):
- category/severity badge [category · severity] on every inline discussion
and fallback entry (byte-matching the CLI's buildBadge)
- publication policy routing findings to the summary by severity
(OCR_ROUTE_SEVERITY_BELOW) and category (OCR_ROUTE_CATEGORIES), fail-open
on unknown metadata; routed findings never enter the inline write path
- deterministic sort (path -> start_line -> end_line -> index) before posting
- detailed warning rendering (file (type): message bullets)
- backtick-safe fenced code blocks in the fallback Before/After; inline
suggestion keeps the fixed suggestion:-0+0 fence so the Apply-suggestion
button keeps working
Read-API features (Group B):
- sticky summary (default ON): the summary note is updated in place across
runs via a <!-- ocr-summary --> marker (find-then-PUT), instead of
accumulating one note per run; a pre-review anchor pins it above the
discussions on the first run
- incremental mode (OCR_INCREMENTAL, default off): skip comments whose line
range overlaps a prior bot discussion on the same path, using IoU above
OCR_INCREMENTAL_OVERLAP_THRESHOLD (default 0.6); single-line vs multi-line
never match; bot discussions detected by the <!-- ocr- marker
- idempotent retry: each inline discussion carries an invisible
<!-- ocr-<pipeline>-<job>-<hex> --> id tag; on a 5xx/408/network failure
(maybe-reached-server) the script GETs /discussions and skips the retry
when the id is found (no duplicate); when the read API is unavailable it
skips the retry and surfaces the reason rather than risk a duplicate
C group (minus batched createReview):
- 400 line-resolution fallback: when a discussion POST returns 400 with a
position error, fetch GET /merge_requests/:iid/diffs, classify the comment
(valid/invalid/unknown), and drop provably-out-of-diff findings to the
summary instead of blindly retrying; unknown keeps the existing fallback
- wait-until-reset: when RateLimit-Remaining == 0, sleep until
RateLimit-Reset (defensively handling both epoch-seconds and
seconds-until-reset formats), ahead of Retry-After / backoff
D group (pipeline alignment):
- dotenv stats output (/tmp/ocr-stats.env) with OCR_COMMENTS_{TOTAL,INLINE,
SUMMARY,ROUTED,SKIPPED,FAILED} and OCR_SUMMARY_URL, exposed via a
reports: dotenv artifact
- artifacts (when: always, 1 week): /tmp/ocr-result.json and
/tmp/ocr-stderr.log so a failed review is still inspectable
- severity fail gating (OCR_FAIL_ON_SEVERITY): non-zero exit when a comment
severity is at/above the threshold; the summary note is still posted first
Tests: 57 -> 117 (stdlib unittest, no network, no wall-clock). New coverage
for badge/policy/sort/warnings/safe-fence, sticky upsert (cold start + reuse),
incremental IoU (single/multi/cross-type/read-failure), idempotent
reconciliation (posted/unavailable/not-posted/400-no-reconcile), 400
classification (valid/invalid/unknown/truncated patch), wait-until-reset
(seconds/epoch/non-zero-remaining), stats file, severity gating.
The only behavior change for existing users: the summary note now updates in
place by default (sticky_summary default true, matching the GitHub Action);
incremental/routing/gating default to off / no-op.
* fix(gitlab-ci): harden OCR_INCREMENTAL_OVERLAP_THRESHOLD parsing, drop dead regex
- build_config used a bare float() on OCR_INCREMENTAL_OVERLAP_THRESHOLD,
which raised an unhandled ValueError and crashed the script when a user
set a non-numeric value. Reuse resolve_threshold, which already handles
non-numeric / out-of-range values and falls back to the default.
- Remove the unused _COMMENT_ID_RE module-level regex; the idempotency
check in _is_comment_posted uses a plain substring match, so the regex
was dead code.
Tests: +2 covering non-numeric and out-of-range env values (117 -> 119).
* fix(gitlab-ci): strip control chars in route_comment, drop dead badge init
- route_comment read category/severity via str(...).strip().lower() without
stripping C0/C1 control characters, while build_badge uses sanitize_metadata
for the same fields. If the LLM emits embedded control chars (e.g. "bug\r"),
the badge rendered [bug · high] but routing classified it as "unknown" and
skipped policy routing. Reuse sanitize_metadata to stay consistent.
- format_comment_fallback had a duplicate md/badge initialization block
immediately overwritten by an identical block, a merge artifact. Removed
the redundant first block.
* fix(gitlab-ci): classify line_code 400s, match MR version to OCR-reviewed SHAs
- LINE_RESOLUTION_PATTERNS missed the `line_code` substring, so GitLab's
`:line_code=>["can't be blank", "must be a valid line code"]` 400 was never
classified as a line-resolution failure. The per-comment diff-inventory
fallback never fired, and the raw error JSON leaked into the summary note
as `⚠️ Could not be posted inline: {"message":"400 Bad request - Note ..."}`
instead of a friendly "out of diff" / "line resolution failure" reason. Add
`line_code` to the pattern list.
- fetch_diff_refs took `versions[0]` blindly. After a force-push or follow-up
commit, that version's head_commit_sha no longer matches the diff OCR
actually reviewed, so the position's base_sha/head_sha describe the wrong
diff and every inline position fails to resolve on GitLab's side — the root
cause of the line_code 400 above. Now read resolved_head/resolved_base from
the OCR result manifest and pick the matching MR version, falling back to
newest-by-created_at with a warning when no version matches.
- Migrate artifact paths /tmp -> .ocr (GitLab Runner refuses to upload
artifacts outside the build directory) and drop the now-redundant temporary
sticky-summary diagnostic block.
- Add 7 tests: line_code pattern positive, version matching by head /
head+base / no-match-fallback / no-expected-shas-sorts-newest, and
_extract_expected_shas with/without manifest.
* feat(gitlab-ci): close GitHub Action parity gaps (config, flags, exit gate, pacing, per-run summary)
Brings the GitLab CI example to feature parity with the GitHub Action
(action.yml + scripts/github-actions/post-review-comments.js).
- .gitlab-ci.yml: expose action.yml inputs as CI/CD variables.
- OCR_VERSION pins the npm spec (default latest) for reproducible reviews.
- Optional LLM config written via `ocr config set` only when the CI var is
non-empty (no clobbering): OCR_LANGUAGE, OCR_LLM_AUTH_HEADER,
OCR_LLM_EXTRA_HEADERS. OCR_LLM_TIMEOUT is read natively by OCR from env.
- Review flags threaded env→flag: OCR_REVIEW_CONCURRENCY, OCR_BACKGROUND,
OCR_RULE.
- OCR exit-code gate: drop `|| true`, capture OCR_EXIT_CODE, run
post_review.py unconditionally (so stderr still posts to MR), then exit
non-zero on OCR failure — mirrors the Action's "Fail job on OCR error".
- Document `llm.use_anthropic` as a hardcoded (not CI-configurable) value
under a separate "Not a CI/CD variable" header; the false listing as a
required CI var was misleading.
- post_review.py: posting-logic parity.
- Read-API pacing: `_read_with_pacing` wraps list_notes/list_discussions/
get_mr_diffs with read_success_delay (0.5s default) and
read_low_remaining_spacing (5s default) when RateLimit-Remaining is
at/below rate_limit_threshold — mirrors readWithPacing so a large MR
does not hammer the read API.
- Non-sticky per-run summary matching: `summary_tag_for(run_tag)` /
`wrap_summary_body` embed both SUMMARY_MARKER and a per-run
`<!-- ocr-summary-run:TAG -->`; find_summary_note/upsert_summary/
ensure_summary_anchor/finalize_summary take a `tag` param. The pre-review
anchor now runs in non-sticky mode too (was sticky-only), so non-sticky
shows the "⏳ Posting…" live state and finalize updates the same note
instead of creating a duplicate.
- build_config exposes read_success_delay / read_low_remaining_spacing.
- tests (+12, 142 total, 0.01s):
- ReadPacingTest: success-delay pacing, low-quota long spacing, no pacing
sleep on failed read.
- NonStickySummaryTest: cold-start anchor→update same note; same-run reuse;
different-run not reused.
- SummaryTagPureTest: tag format, wrap embeds both markers, newest-first
find by tag/marker, None on no match.
- BuildConfigTest: read-pacing defaults & overrides.
- Recorder.final_summary_body / summary_call_count helpers adapt existing
assertions to the two-phase anchor→finalize flow.
- MainAuthHeaderTest: mock urlopen with canned responses — was making real
DNS lookups to gitlab.example, amplified 3x by the new always-on anchor
(13.7s → 0.01s).
- README.md: document the 9 new CI vars in the variables table, add
read-pacing vars to the retry/delay table, convert the version/rules/
concurrency/background recipes from "edit the YAML" to "set this CI var".
* fix(gitlab-ci): if-guard optional vars, capture post_review exit code, split diff-refs reason
- Replace [ -n "$VAR" ] && cmd with if-guards so FF_ENABLE_BASH_EXIT_CODE_CHECK
cannot abort on an empty optional variable (|| true would mask real failures)
- Capture post_review.py exit code (POST_EXIT_CODE) so the OCR_EXIT_CODE gate
runs regardless of severity-gate or posting failures
- Split 'not path or not end_line or not diff_refs' into two checks so
diff-refs-unavailable comments get an accurate reason in the summary
* fix(gitlab-ci): default OCR/POST exit codes to guard against unset gate vars
Without a default, an unset OCR_EXIT_CODE/POST_EXIT_CODE makes the gate
`[ "" != "0" ]` evaluate true and `exit ""` fail with
'numeric argument required'. Default both to 0 so the gate is always safe.
|
||
|---|---|---|
| .. | ||
| .gitlab-ci.yml | ||
| post_review.py | ||
| post_review_test.py | ||
| README.md | ||
OpenCodeReview - GitLab CI Demo
This demo shows how to integrate OpenCodeReview into your GitLab CI/CD pipeline to automatically review Merge Requests and post review comments as inline discussions.
How It Works
MR Created/Updated → GitLab Pipeline Triggered → OCR Reviews Diff → Discussions Posted on MR
- When a Merge Request is opened or updated, the pipeline triggers
- It installs OCR via npm in a
node:20Docker image - Runs
ocr review --from origin/<target> --to <commit_sha> --format json --audience agentto analyze the diff (uses commit SHA to support fork MRs) - Parses the JSON output and posts inline discussions on the MR using GitLab's Discussions API
Setup
1. Copy the pipeline and script files
Copy both .gitlab-ci.yml and post_review.py to your repository root (or a subdirectory — adjust the python3 post_review.py path in the YAML if you place the script elsewhere):
cp .gitlab-ci.yml post_review.py /path/to/your/repo/
Or use GitLab's include feature in your existing .gitlab-ci.yml (the script path is relative to the pipeline file's location):
include:
- local: 'ci_demo/gitlab_ci/.gitlab-ci.yml'
2. Configure CI/CD Variables
Go to your project's Settings → CI/CD → Variables and add:
| Variable | Required | Masked | Description |
|---|---|---|---|
OCR_LLM_URL |
Yes | No | LLM API endpoint URL (e.g., https://api.openai.com/v1/chat/completions) |
OCR_LLM_AUTH_TOKEN |
Yes | Yes | API authentication token |
OCR_LLM_MODEL |
Yes | No | Model name (e.g., gpt-4o) — OCR has no built-in default model and fails when this is unset |
GITLAB_API_TOKEN |
No | Yes | GitLab access token with api scope (falls back to CI_JOB_TOKEN if not set) |
OCR_VERSION |
No | No | npm version spec for @alibaba-group/open-code-review (default: latest). Pin it (e.g. 1.8.8 or ~1.8) for reproducible reviews. |
OCR_LANGUAGE |
No | No | Review output language, written via ocr config set language (e.g. English, 中文). Defaults to OCR's built-in default. |
OCR_LLM_AUTH_HEADER |
No | No | Custom auth header name, written via ocr config set llm.auth_header (e.g. x-api-key for some providers). |
OCR_LLM_EXTRA_HEADERS |
No | No | Extra headers K=V,K=V, written via ocr config set llm.extra_headers. |
OCR_LLM_TIMEOUT |
No | No | LLM request timeout in seconds (read natively by OCR from the env). Unset/empty = OCR's default. |
OCR_REVIEW_CONCURRENCY |
No | No | Max concurrent file reviews, passed via ocr review --concurrency (default: 8). |
OCR_BACKGROUND |
No | No | Business/requirement context, passed via ocr review --background (e.g. the MR title). |
OCR_RULE |
No | No | Path to a custom rules JSON file, passed via ocr review --rule. |
Note: GitLab CI/CD does not support variables with values shorter than 8 characters, so
use_anthropiccannot be set as a CI variable. The pipeline sets it tofalseby default. If you need to use Anthropic Claude models, you'll need to modify the.gitlab-ci.ymlscript directly.The pipeline also configures
llm.extra_bodyto disable thinking mode for compatibility with various LLM providers.
3. Create a GitLab Access Token
You need a token with api scope to post discussions on MRs. Options:
- Project Access Token (recommended): Settings → Access Tokens → Create with
apiscope - Personal Access Token: User Settings → Access Tokens → Create with
apiscope - Group Access Token: For organization-wide usage
Note: The built-in
CI_JOB_TOKENhas limited API scope and may not support all discussion features (e.g., creating new threads on older GitLab versions). IfGITLAB_API_TOKENis not set, the pipeline falls back toCI_JOB_TOKENautomatically — but for best results, a dedicated token withapiscope is recommended.Tip: For Project Access Tokens and Group Access Tokens, the token name determines the bot name shown in MR discussions. For example, naming your token
OpenCodeReview Botwill make review comments appear as posted byOpenCodeReview Bot.
Example Output
When an MR is reviewed, comments appear as:
- Inline discussions: Directly on the changed lines in the MR diff view, each prefixed with a
[category · severity]badge when the LLM provided that metadata - Summary note: A single note (updated in place across runs by default) summarizing the total number of issues found, with mutually-exclusive counts (inline / summary / routed / skipped / failed) and a detailed warnings list
- Fallback notes: Findings that could not be posted inline (no line info, routed by policy, or whose position could not be resolved against the diff) are collected into the summary note, each tagged with the reason it ended up there
Inline Discussion Example
Comments are posted using GitLab's Discussion API with position data, so they appear directly next to the relevant code in the "Changes" tab. Each comment carries an HTML-comment id tag (invisible when rendered) that the script uses to avoid duplicate posts on retry.
Supported LLM Providers
OCR supports both OpenAI and Anthropic API formats:
- OpenAI-compatible APIs (default):
- OpenAI (GPT-4o, GPT-4, etc.)
- Azure OpenAI
- Self-hosted models (vLLM, Ollama, etc.)
- Anthropic APIs (modify
.gitlab-ci.ymlto setuse_anthropic: true):- Anthropic Claude models
Customization
Most review behavior is configurable via CI/CD Variables (see the table above) — no YAML edits required. The recipes below cover the remaining cases.
Use a specific OCR version
Set the OCR_VERSION CI/CD variable (e.g. 1.8.8 or ~1.8). Unset defaults to latest.
Add custom review rules
Set the OCR_RULE CI/CD variable to the path of a custom rules JSON file (passed to ocr review --rule). For rules that live inside the repo, commit the file and point OCR_RULE at its path.
Adjust retry and delay settings
When posting review discussions, the script includes rate-limit handling with exponential backoff (with jitter), Retry-After header support, and proactive throttling based on GitLab's RateLimit-Remaining response header. All API requests — including summary notes and MR version fetches — use the same retry logic. See GitLab Rate Limits for details on GitLab's rate-limiting policies and recommended handling. You can configure the retry and delay behavior via CI/CD Variables (Settings → CI/CD → Variables):
| Variable | Default | Description |
|---|---|---|
OCR_RETRY_BASE_DELAY |
2000 |
Base delay (ms) for exponential backoff when a rate-limit error is hit |
OCR_MAX_RETRIES |
3 |
Maximum retry attempts per discussion when rate-limited |
OCR_MAX_RETRY_DELAY |
60000 |
Maximum delay (ms) per single retry, caps both Retry-After and backoff |
OCR_SUCCESS_DELAY |
2000 |
Delay (ms) after a successful discussion post to pace subsequent requests |
OCR_FAILURE_DELAY |
1000 |
Delay (ms) after a non-rate-limit failure to pace subsequent requests |
OCR_RATE_LIMIT_THRESHOLD |
10 |
Proactively slow down when GitLab RateLimit-Remaining is at/below this value (set 0 to disable). Applies to both writes (doubles the pacing delay) and reads (switches to the longer read spacing). |
OCR_READ_SUCCESS_DELAY |
500 |
Delay (ms) after a successful read (list_notes/list_discussions/get_mr_diffs) to pace read API calls, mirroring the GitHub Action's readWithPacing. |
OCR_READ_LOW_REMAINING_SPACING |
5000 |
Longer delay (ms) used after a read when the remaining quota is at/below OCR_RATE_LIMIT_THRESHOLD. |
OCR_ROUTE_SEVERITY_BELOW |
(empty) | Optional severity threshold (critical, high, medium, low) that routes findings at-or-below it from inline comments to summary notes (fail-open: never drops a finding). Empty or unknown values disable severity routing. |
OCR_ROUTE_CATEGORIES |
(empty) | Optional comma-separated categories (bug, security, performance, maintainability, test, style, documentation, other) routed from inline to summary notes. Unknown tokens are ignored. Combine with OCR_ROUTE_SEVERITY_BELOW to route on either condition. |
These variables are optional — if not configured, sensible defaults are used. Consider increasing delays for self-hosted GitLab instances with aggressive rate-limit configurations or for large MRs that generate numerous review comments. The OCR_RATE_LIMIT_THRESHOLD variable enables proactive throttling: when GitLab reports low remaining quota in the RateLimit-Remaining response header, the script automatically doubles the pacing delay to avoid hitting 429 errors.
When the primary rate limit is exhausted (RateLimit-Remaining: 0), the script additionally honors GitLab's RateLimit-Reset header, sleeping until the limit resets (defensively handling both epoch-seconds and seconds-until-reset formats). This mirrors the GitHub Action's x-ratelimit-reset handling.
Publication behavior (aligned with the GitHub Action)
The posting script implements the same publication behaviors as the GitHub Action (scripts/github-actions/post-review-comments.js), exposed as optional CI/CD variables. All default to no-op / current behavior except OCR_STICKY_SUMMARY, which defaults to true (the summary note is updated in place across runs instead of accumulating one note per run).
| Variable | Default | Description |
|---|---|---|
OCR_STICKY_SUMMARY |
true |
Update the summary note in place across runs (find-by-marker then PUT /notes/:id). Set false to post a fresh note each run. |
OCR_INCREMENTAL |
false |
Skip comments whose line range overlaps a prior bot discussion on the same path (so re-runs only append new findings). Uses an IoU threshold for multi-line spans. |
OCR_INCREMENTAL_OVERLAP_THRESHOLD |
0.6 |
IoU (0–1) above which two multi-line comments are considered the same. Single-line comments match only on the exact same line; a single-line and multi-line comment never match. |
OCR_ROUTE_SEVERITY_BELOW |
(empty) | Route findings whose severity is at-or-below this level to the summary note instead of posting them inline (e.g. low keeps only critical/high/medium inline). Unknown severities never match (fail-open). |
OCR_ROUTE_CATEGORIES |
(empty) | Comma-separated categories to route to the summary (e.g. style,documentation). Routed findings never enter the inline write path, so they cannot be double-posted on retry. |
OCR_FAIL_ON_SEVERITY |
(empty) | Fail the CI job (non-zero exit) when any comment's severity is at-or-above this level (e.g. critical). The summary note is still posted before the job fails, so visibility is preserved. |
Additional behaviors ported from the GitHub Action (always on, no variable):
- Category/severity badge: each inline discussion and fallback entry is prefixed with
[category · severity](using a middot, U+00B7) when the LLM provided that metadata, byte-matching the CLI's badge rendering. - Deterministic ordering: comments are sorted by
path → start_line → end_line → original indexbefore posting, so identical reruns produce identical post sequences (which is what makes idempotency reconciliation reproducible). - Detailed warnings: warnings are rendered as a bulleted list (
file (type): message) rather than a bare count. - Backtick-safe fences: fallback Before/After code blocks use a fence long enough to enclose any backticks in the code (the inline suggestion keeps the fixed triple-backtick
suggestion:-0+0form so GitLab's "Apply suggestion" button keeps working). - Idempotent retry: each inline discussion carries an invisible HTML-comment id tag (
<!-- ocr-<pipeline>-<job>-<hex> -->). When aPOST /discussionsfails with a 5xx/408/network error (the request may still have landed), the script queries existing discussions for the id before retrying; if found it is treated as success (no duplicate), and if the read API is unavailable it skips the retry rather than risk a duplicate. - 400 line-resolution fallback: when a discussion
POSTreturns 400 indicating a position problem, the script fetches the MR diff (GET /merge_requests/:iid/diffs), classifies the comment as valid/invalid/unknown, and drops provably-out-of-diff findings to the summary rather than blindly retrying. Unknown cases keep the existing fallback behavior.
CI outputs and artifacts
The pipeline exposes the following to downstream jobs:
- dotenv report (
.ocr/ocr-stats.env):OCR_COMMENTS_TOTAL,OCR_COMMENTS_INLINE,OCR_COMMENTS_SUMMARY,OCR_COMMENTS_ROUTED,OCR_COMMENTS_SKIPPED,OCR_COMMENTS_FAILED, andOCR_SUMMARY_URL. Consume them in later stages via thedotenvartifact. - Artifacts (
when: always, 1 week retention):.ocr/ocr-result.json(raw review JSON) and.ocr/ocr-stderr.log(OCR stderr), so you can inspect a failed review even when the job fails. Paths are project-relative (under.ocr/) because GitLab Runner refuses to upload artifacts outside the build directory.
Limit concurrency
Set the OCR_REVIEW_CONCURRENCY CI/CD variable (passed to ocr review --concurrency). For large MRs, lowering this caps concurrent LLM requests:
variables:
OCR_REVIEW_CONCURRENCY: "5"
Provide background context
Set the OCR_BACKGROUND CI/CD variable (passed to ocr review --background). A common choice is the MR title:
variables:
OCR_BACKGROUND: "$CI_MERGE_REQUEST_TITLE"
This is particularly useful when your MR titles follow semantic conventions (e.g., feat(auth): add OAuth2 support) that clearly summarize what the MR implements. The background information helps OCR provide more relevant and context-aware review comments.
Change the trigger events
By default, the pipeline uses only: [merge_requests], which triggers on all MR events (creation, updates, reopen). GitLab CI does not natively support fine-grained control to trigger only on MR creation.
To avoid re-reviewing on every push to an existing MR (and wasting LLM API tokens), you can check for existing OCR reviews before running ocr review. Use a wrapper script that skips the review step if OCR comments already exist:
script:
# Install OpenCodeReview
- npm install -g @alibaba-group/open-code-review
# Configure OCR
- |
: "${OCR_LLM_URL:?set OCR_LLM_URL in Settings -> CI/CD -> Variables}"
: "${OCR_LLM_AUTH_TOKEN:?set OCR_LLM_AUTH_TOKEN in Settings -> CI/CD -> Variables}"
: "${OCR_LLM_MODEL:?set OCR_LLM_MODEL in Settings -> CI/CD -> Variables}"
ocr config set llm.url "$OCR_LLM_URL"
ocr config set llm.auth_token "$OCR_LLM_AUTH_TOKEN"
ocr config set llm.model "$OCR_LLM_MODEL"
ocr config set llm.use_anthropic false
ocr config set llm.extra_body '{"thinking": {"type": "disabled"}}'
# Check for existing OCR reviews and run review only if not found
- |
python3 << 'WRAPPER_SCRIPT'
import json
import os
import subprocess
import sys
import urllib.request
GITLAB_URL = os.environ.get("CI_SERVER_URL", "https://gitlab.com")
PROJECT_ID = os.environ["CI_PROJECT_ID"]
MR_IID = os.environ["CI_MERGE_REQUEST_IID"]
API_TOKEN = os.environ["GITLAB_API_TOKEN"]
SOURCE_BRANCH = os.environ["CI_MERGE_REQUEST_SOURCE_BRANCH_NAME"]
TARGET_BRANCH = os.environ["CI_MERGE_REQUEST_TARGET_BRANCH_NAME"]
# Check for existing OCR reviews
url = f"{GITLAB_URL}/api/v4/projects/{PROJECT_ID}/merge_requests/{MR_IID}/notes?per_page=100"
req = urllib.request.Request(url, headers={"PRIVATE-TOKEN": API_TOKEN})
with urllib.request.urlopen(req) as resp:
notes = json.loads(resp.read().decode("utf-8"))
for note in notes:
if "OpenCodeReview" in note.get("body", ""):
print("⏭️ OCR has already reviewed this MR. Skipping to save tokens.")
print("Delete previous OCR comments to re-trigger review.")
sys.exit(0)
# No existing review found - run OCR
print("🔍 No existing OCR review found. Running review...")
COMMIT_SHA = os.environ["CI_COMMIT_SHA"]
result = subprocess.run([
"ocr", "review",
"--from", f"origin/{TARGET_BRANCH}",
"--to", COMMIT_SHA,
"--format", "json",
"--audience", "agent"
], capture_output=True, text=True)
# Save output for the posting script
os.makedirs(".ocr", exist_ok=True)
with open(".ocr/ocr-result.json", "w") as f:
f.write(result.stdout)
with open(".ocr/ocr-stderr.log", "w") as f:
f.write(result.stderr)
print("OCR review completed.")
WRAPPER_SCRIPT
# Post review comments to MR
- python3 post_review.py .ocr/ocr-result.json
The key logic: the Python wrapper checks for existing OCR comments before running ocr review. If found, it exits early with sys.exit(0) before consuming any LLM tokens. To re-trigger a review, users can manually delete the previous OCR comments.
Self-hosted GitLab
The script automatically uses CI_SERVER_URL to determine the GitLab API base URL, so it works with self-hosted GitLab instances out of the box.
Use a Service Account as Review Bot
By default, review comments are posted using the user who owns the access token configured in GITLAB_API_TOKEN. You can create a dedicated service account bot to post reviews with a custom identity, making it easier to distinguish automated reviews from human comments.
For more details about GitLab service accounts, see the GitLab Service Accounts documentation.
Step 1: Create a Service Account
Create a service account in your project:
- Go to your Project → Settings → Service Accounts
- Click New service account
- Fill in the following:
- Name: e.g.,
OpenCodeReview Bot(this will be the bot name shown in MR discussions) - Username: Will be auto-generated based on the name
- Name: e.g.,
- Click Create service account
Step 2: Invite the Service Account to Your Project
After the service account is created, invite it to your project with appropriate permissions:
- Go to your Project → Settings → Members
- Click Invite member
- Search for the service account by name (e.g.,
OpenCodeReview Bot) - Select the service account and assign a role (
DeveloperorMaintainerrequired for posting discussions) - Click Invite
Step 3: Create an Access Token
Generate an access token for the service account:
- Go to your Project → Settings → Service Accounts
- Click on the service account to view its details
- Click Add new token
- Configure the token:
- Name: e.g.,
ocr-review-token - Expiration: As needed
- Scope: Select
api(required for Discussions API)
- Name: e.g.,
- Click Create token and copy the token value
Step 4: Update CI/CD Variables
Update the GITLAB_API_TOKEN variable in your project's CI/CD settings:
Go to Settings → CI/CD → Variables and update GITLAB_API_TOKEN with the service account's token.
Now review comments will be posted with your service account identity (e.g., OpenCodeReview Bot), providing a clear and professional appearance for automated code reviews.
Troubleshooting
Common Issues
- "API error 403": The
GITLAB_API_TOKENlacksapiscope or doesn't have access to the project - "Failed to parse OCR output": Check that
OCR_LLM_URLandOCR_LLM_AUTH_TOKENvariables are correctly set - "Cannot find merge-base": Ensure
GIT_DEPTH: 0is set (full clone) - Inline comments on wrong lines: GitLab requires exact SHA matching; the script fetches MR version metadata to get correct diff refs
Debugging
Add verbose output to the review step:
script:
- cat .ocr/ocr-result.json
- cat .ocr/ocr-stderr.log
Testing
The posting logic is unit-tested with no network access and no wall-clock sleep cost. Tests use only the standard-library unittest and cover: comment formatting (badge, suggestion, fallback), the publication policy (severity/category routing), deterministic sort, warning rendering, safe-fence code blocks, the full publish() flow (partition → sticky summary → fallback), incremental overlap (IoU, single-vs-multi, bot detection), the GitLab transport (retry/backoff/jitter/Retry-After/rate-limit/auth/network errors), idempotent post_discussion reconciliation, the 400 line-resolution fallback classification, wait-until-reset, and the dotenv stats / severity-gating helpers.
# From the example directory
cd examples/gitlab_ci
python3 post_review_test.py
# From the repo root
python3 -m unittest discover -s examples/gitlab_ci -p '*_test.py'