Skip to content

fix(ci): cap stalled unit jobs at 45 minutes#10854

Merged
marius-kilocode merged 3 commits into
mainfrom
fix/ci-unit-timeout
Jun 3, 2026
Merged

fix(ci): cap stalled unit jobs at 45 minutes#10854
marius-kilocode merged 3 commits into
mainfrom
fix/ci-unit-timeout

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

CLI unit subprocess leaks can leave the unit matrix job running until GitHub's default six-hour timeout, consuming runner capacity and blocking the required test check.

Cap the unit job at 45 minutes. Healthy runs finish well below that ceiling, while a leaked subprocess now fails within a bounded window instead of retaining a runner for hours.

This is intentionally a containment measure, not a test-runtime optimization. Follow-up work should harden process-tree cancellation in the CLI test runner and profile the Windows TUI plugin lifecycle slowdown.

@marius-kilocode marius-kilocode enabled auto-merge June 3, 2026 11:23
@kilo-code-bot

kilo-code-bot Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • .github/workflows/test.yml

Reviewed by claude-4.6-sonnet-20260217 · 155,016 tokens

Review guidance: REVIEW.md from base branch main

@marius-kilocode marius-kilocode merged commit e0f9bae into main Jun 3, 2026
19 of 20 checks passed
@marius-kilocode marius-kilocode deleted the fix/ci-unit-timeout branch June 3, 2026 11:41
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.

3 participants