Skip to content

chore(review): pin revision-aware runtime on dev - #307

Merged
777genius merged 1 commit into
devfrom
chore/pin-reviewrouter-ceb4d25-dev
Jul 23, 2026
Merged

chore(review): pin revision-aware runtime on dev#307
777genius merged 1 commit into
devfrom
chore/pin-reviewrouter-ceb4d25-dev

Conversation

@777genius

@777genius 777genius commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Pins the managed ReviewRouter review and interaction workflows on dev to the same immutable revision-aware Action release already merged on main. This is required for draft PRs targeting dev, including #252, to use the durable workflow-dispatch path instead of the legacy pull_request workflow.

Greptile Summary

Pins ReviewRouter workflows to an immutable revision-aware runtime.

  • Replaces pull-request-triggered Codex reviews with durable, identity-bearing workflow dispatches through a reusable workflow.
  • Pins the interaction runtime to the same immutable revision and enables its memory and ledger configuration.
  • Updates the discussion-reply Codex CLI dependency.

Confidence Score: 5/5

The PR appears safe to merge with no concrete changed-code failure identified.

The review and interaction workflows consistently use the same immutable runtime revision, and the new dispatch conditions cleanly separate identity-bearing reviews from scheduled or input-free maintenance runs.

Important Files Changed

Filename Overview
.github/workflows/reviewrouter-codex.yml Moves reviews to revision-aware workflow dispatch while retaining scheduled and input-free maintenance refreshes.
.github/workflows/reviewrouter-interaction.yml Pins the interaction runtime, adds memory and ledger settings, and updates the discussion CLI version.

Reviews (1): Last reviewed commit: "chore(review): pin revision-aware runtim..." | Re-trigger Greptile

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@777genius, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 5 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 5f22a140-9bc7-4b13-a86f-ac55dfd732b5

📥 Commits

Reviewing files that changed from the base of the PR and between 949dc7b and 7b5c6b8.

📒 Files selected for processing (2)
  • .github/workflows/reviewrouter-codex.yml
  • .github/workflows/reviewrouter-interaction.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pin-reviewrouter-ceb4d25-dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7b5c6b86b8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +46 to +47
secrets:
CODEX_AUTH_JSON: ${{ secrets.REVIEWROUTER_CODEX_AUTH_JSON }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Pass the ledger key into the reusable review

With the new T0 reusable workflow, the pinned runtime loads ReviewLedger from secrets.REVIEW_ROUTER_LEDGER_KEY before building the lifecycle inventory and treats a missing key as review_action_v2_command_ledger_unavailable; this caller only forwards CODEX_AUTH_JSON, so every dispatched Codex review in this workflow will run with an empty ledger key and fail before publishing results. Please pass REVIEW_ROUTER_LEDGER_KEY: ${{ secrets.REVIEW_ROUTER_LEDGER_KEY }} here, matching the interaction workflow that writes the ledger.

Useful? React with 👍 / 👎.

@777genius
777genius merged commit 5050bdd into dev Jul 23, 2026
13 checks passed
@777genius
777genius deleted the chore/pin-reviewrouter-ceb4d25-dev branch July 23, 2026 10:09
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.

1 participant