Skip to content

ci: extend Claude Code timeout to 30 minutes - #120

Closed
keito4 wants to merge 1 commit into
mainfrom
ci/claude-timeout-30-20260616
Closed

ci: extend Claude Code timeout to 30 minutes#120
keito4 wants to merge 1 commit into
mainfrom
ci/claude-timeout-30-20260616

Conversation

@keito4

@keito4 keito4 commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Why

Reflects keito4/config#831.

Claude Code jobs can exceed the previous 20 minute cap while investigating and preparing changes. The shared config template now uses 30 minutes, so this repository should match it.

What

  • Set the Claude Code workflow timeout to 30 minutes.

Verification

  • actionlint .github/workflows/claude.yml
  • git diff --check

Summary by CodeRabbit

  • Chores
    • Improved CI/CD workflow reliability by implementing runtime limits to prevent extended processing durations.

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bc17322d-b660-4956-87bf-d02838c46bc9

📥 Commits

Reviewing files that changed from the base of the PR and between 7b7257f and 1371c10.

📒 Files selected for processing (1)
  • .github/workflows/claude.yml

📝 Walkthrough

Walkthrough

A timeout-minutes: 30 field is added to the claude job in the GitHub Actions workflow file .github/workflows/claude.yml, capping the maximum execution time for that job at 30 minutes.

Changes

Claude Code Workflow Timeout

Layer / File(s) Summary
Claude job timeout setting
.github/workflows/claude.yml
Adds timeout-minutes: 30 to the claude job definition to limit maximum job runtime.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐇 Tick-tock, the clock is set,
Thirty minutes, no more fret!
The claude job hops along with glee,
But stops on time — how wise, you see!
No runaway jobs, just efficiency! 🕐

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description provides a clear explanation of why (references config update), what (sets timeout to 30 minutes), and verification steps, but does not follow the repository's template structure. While the description is substantive and helpful, consider restructuring it to match the repository template with Description, Screencast, and Checklist sections for consistency.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: extending the Claude Code workflow timeout to 30 minutes, matching the PR's primary objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/claude-timeout-30-20260616

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


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 and usage tips.

@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: 1371c1062a

ℹ️ 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".

(github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) ||
(github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
runs-on: ubuntu-latest
timeout-minutes: 30

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 Badge Leave headroom for Claude action cleanup

When a @claude run spends close to 30 minutes in the Claude step, this job-level timeout cancels the entire job because GitHub applies it to checkout/setup/action cleanup as well as the Claude execution. I checked the workflow and the anthropics/claude-code-action@beta action already exposes a timeout_minutes input for the Claude execution (defaulting to 30) and then runs follow-up always() steps such as updating comments and revoking the app token; with the job capped at the same 30 minutes, setup time means Claude gets less than 30 minutes and the cleanup/comment steps can be skipped. Please either set the action input or give the job timeout extra headroom.

Useful? React with 👍 / 👎.

@keito4

keito4 commented Jun 17, 2026

Copy link
Copy Markdown
Owner Author

Closing as requested.

@keito4 keito4 closed this Jun 17, 2026
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