Skip to content

ci: review-fallback caller (Devin Review fallback) - #74

Closed
Icaro0310 wants to merge 1 commit into
mainfrom
feat/review-fallback-caller
Closed

Icaro0310 wants to merge 1 commit into
mainfrom
feat/review-fallback-caller

Conversation

@Icaro0310

Copy link
Copy Markdown
Owner

Wires the reusable review-fallback pipeline from devin-review-fallback into this repo via a single caller workflow (.github/workflows/review-fallback.yml).

Pipeline: classify → Ollama reviewers A/B (serialized — the tailnet endpoint is a single-slot queue) → Review Coverage gate → judge-gated auto-merge, plus an optional SonarCloud SAST layer (skips cleanly when SONAR_TOKEN/SONAR_PROJECT_KEY are unset).

Prerequisites are already provisioned on this repo:

  • secret TS_AUTHKEY (Tailscale auth key, tagged tag:gh-ollama-review)
  • vars OLLAMA_TAILNET_URL + OLLAMA_MODEL (qwen2.5-coder:1.5b)
  • auto-merge enabled at repo level

Honest degradation: if the Ollama endpoint or a reviewer is unreachable, the review reports UNAVAILABLE/FAILED rather than faking a green approval — the Coverage gate only passes on real reviewer output.

Wires the reusable review-fallback pipeline (classify -> Ollama
reviewers A/B serialized -> Review Coverage gate -> judge-gated
auto-merge + optional SonarCloud). Secret TS_AUTHKEY and vars
OLLAMA_TAILNET_URL/OLLAMA_MODEL already provisioned; reviewers
degrade to UNAVAILABLE/FAILED rather than faking green.

Generated with [Devin](https://devin.ai) & QA Engineer Icaro Galvao

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@Icaro0310
Icaro0310 enabled auto-merge (squash) October 27, 2026 14:36
Icaro0310 added a commit that referenced this pull request Oct 27, 2026
The caller workflow can never review its own PR (pull_request_target
runs the base-branch version). Admin path, same documented paradox.

Generated with [Devin](https://devin.ai) & QA Engineer Icaro Galvao

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@Icaro0310

Copy link
Copy Markdown
Owner Author

Closing as superseded: the caller file in this PR is byte-identical to main (it landed via a direct squash commit on 2026-10-27). The PR is orphaned — nothing left to merge.

@Icaro0310 Icaro0310 closed this Oct 29, 2026
auto-merge was automatically disabled October 29, 2026 17:44

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant