Skip to content

ci: auto code review on PRs via OpenCodeReview (free tier) - #109

Merged
vishalveerareddy123 merged 3 commits into
mainfrom
ci/open-code-review
Sep 14, 2026
Merged

ci: auto code review on PRs via OpenCodeReview (free tier)#109
vishalveerareddy123 merged 3 commits into
mainfrom
ci/open-code-review

Conversation

@veerareddyvishal144

@veerareddyvishal144 veerareddyvishal144 commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Adds .github/workflows/ocr-review.yml: every PR (opened/synchronize/reopened) triggers an Alibaba open-code-review run on GitHub-hosted runners (free for this public repo), plus on-demand re-review via /open-code-review comments.

Cost design mirrors local Lynkr tiers (SIMPLE/MEDIUM/COMPLEX -> free models):

  • LLM = OpenRouter :free model (repo Variable OCR_LLM_MODEL, currently nvidia/nemotron-3-nano-30b-a3b:free, $0)
  • effort: low + sticky summary to stay in budget and avoid comment spam
  • Secrets OCR_LLM_URL / OCR_LLM_AUTH_TOKEN already set; model switchable via Variables with no code change.

Note: pull_request_target runs the workflow from base, so this setup PR itself won't be reviewed until after merge - open any test PR after merging to see it fire.

Greptile Summary

This revision hardens the OpenCodeReview workflow by replacing mutable third-party action references with full commit SHAs and selecting an immutable OpenCodeReview CLI release.

  • Pins actions/github-script to the commit annotated as v7.0.1.
  • Pins alibaba/open-code-review to the commit annotated as v1.12.0.
  • Configures ocr_version as 1.12.0.
  • Leaves the previously reported unsupported llm_reasoning_effort setting unchanged.

Confidence Score: 5/5

The latest changes appear safe to merge, although the existing non-blocking warning that llm_reasoning_effort is ignored remains outstanding.

The mutable privileged action references identified previously are now pinned and that thread was resolved. The unresolved previous finding remains valid because llm_reasoning_effort is still supplied even though it is not a declared OpenCodeReview action input, so the intended low-reasoning cost and latency control is not applied.

Files Needing Attention: .github/workflows/ocr-review.yml

Important Files Changed

Filename Overview
.github/workflows/ocr-review.yml Pins both workflow actions and the OpenCodeReview CLI version; the unresolved low-reasoning input from the previous review remains unchanged.

Reviews (3): Last reviewed commit: "ci: pin ocr action and CLI to immutable ..." | Re-trigger Greptile

Comment thread .github/workflows/ocr-review.yml Outdated
# Keep reviews fast and cheap on the diffusion model: low effort,
# low reasoning, sticky summary so re-pushes update one comment.
effort: low
llm_reasoning_effort: 'low'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Reasoning Setting Is Ignored

llm_reasoning_effort is not a declared input of alibaba/open-code-review, so GitHub Actions warns and silently ignores it. Reviews therefore use the CLI's default reasoning behavior rather than the documented low-reasoning setting, defeating the intended cost and latency control.

@vishalveerareddy123
vishalveerareddy123 merged commit 92700b6 into main Sep 14, 2026
2 checks passed
@vishalveerareddy123
vishalveerareddy123 deleted the ci/open-code-review branch September 14, 2026 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants