Skip to content

ci: extend codegenie review timeout - #2357

Merged
pratikspatil024 merged 1 commit into
developfrom
ppatil-codegenie-timeout
Aug 13, 2026
Merged

pratikspatil024 merged 1 commit into
developfrom
ppatil-codegenie-timeout

Conversation

@pratikspatil024

Copy link
Copy Markdown
Member

Summary

Increase CodeGini's repository-level review budget from its hidden 30-minute default to 120 minutes, and raise the GitHub Actions job timeout to 135 minutes so final status publishing and artifact upload have a cleanup margin. This addresses the incomplete review on #2319, where the 30-minute budget expired before review work was dispatched across the 1,023-hunk upstream merge.

Executed tests

  • Parsed .github/workflows/codegenie-review.yml with Ruby YAML
  • Parsed codegenie.toml with Python tomllib
  • Ran git diff --check
  • Verified the CodeGini v0.5.5 schema accepts review.maxTime in minutes

Rollout notes

CI-only, backwards-compatible change. No consensus, runtime, or operator impact. Reviews remain explicitly comment-triggered and retain CodeGini's separate 8M-token safety cap.

Copilot AI lite review requested due to automatic review settings August 13, 2026 04:36

@claude claude 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.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Extends the Codegenie review time budget in CI to better handle very large PRs (e.g., upstream-merge batches) by increasing both the repository-level review budget and the GitHub Actions job timeout to allow for review completion plus post-run cleanup.

Changes:

  • Increase codegenie.toml [review].maxTime to 120 minutes (from the implicit 30-minute default).
  • Increase the GitHub Actions timeout-minutes for the Codegenie review job to 135 minutes to allow time for final status posting and artifact upload.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
codegenie.toml Sets review.maxTime = 120 to raise the repository-level Codegenie review budget.
.github/workflows/codegenie-review.yml Raises the workflow job timeout to 135 minutes to exceed the review budget and leave cleanup margin.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@rukimedo
rukimedo self-requested a review August 13, 2026 05:00
@pratikspatil024
pratikspatil024 merged commit 02dcc32 into develop Aug 13, 2026
16 checks passed
@pratikspatil024
pratikspatil024 deleted the ppatil-codegenie-timeout branch August 13, 2026 05:01
@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.21%. Comparing base (a48c2e6) to head (2893162).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2357      +/-   ##
===========================================
- Coverage    54.22%   54.21%   -0.01%     
===========================================
  Files          908      908              
  Lines       162252   162252              
===========================================
- Hits         87981    87966      -15     
- Misses       68836    68854      +18     
+ Partials      5435     5432       -3     

see 23 files with indirect coverage changes
see 23 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

4 participants