Skip to content

Give the macOS CI job 45 minutes instead of 30 - #315

Merged
BunsDev merged 4 commits into
mainfrom
ci-macos-timeout
Oct 19, 2026
Merged

BunsDev merged 4 commits into
mainfrom
ci-macos-timeout

Conversation

@BunsDev

@BunsDev BunsDev commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

The required unit + e2e (macOS) job had a 30-minute limit, and green runs now take most of it.

Over the last 30 completed CI runs Minutes
Green unit + e2e (macOS) jobs 20–29 (most 22–26)
Their e2e step alone 16–22
Setup, build and unit tests before e2e about 6

On 2026-10-18, #310's run 39176355426 hit the limit with its e2e step still running, 25 minutes in. GitHub reports that as cancelled, so the merge guard refused a docs-only PR as failing.

This raises the limit to 45 minutes. That's about 55% headroom over the slowest green run, and a hung run still ends well before GitHub's 6-hour default. A comment above the line records why. The iPhone job's 30 minutes and every other workflow are unchanged, and no test or doc refers to the value.

Verification

  • The YAML parses. The macOS job reads 45; the iPhone job still reads 30.
  • pnpm build && pnpm test passed locally:
    • root suites: 109/109 and 16/16;
    • desktop: 1,606 tests, 1,590 passed, 16 skipped, 0 failed;
    • remaining stages: 21/21, 10/10 and 16/16.
  • This PR's own CI runs under the new limit.

The required `unit + e2e (macOS)` job ran close to its 30-minute limit. Over
the last 30 completed CI runs, green jobs took 20–29 minutes and the e2e step
alone took 16–22. On 2026-10-18, run 39176355426 (#310) was cut off at 30
minutes with its e2e step still running. GitHub reports that as cancelled, so
the guard refused the PR as failing, though the PR only changed docs.

45 minutes covers the slowest green run with about 55% headroom, and a hung
run still ends well before GitHub's 6-hour default. The iPhone job's 30 is
unchanged.

Coven-Harness: Claude Code / claude-opus-5-5
@vercel

vercel Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
modex Ready Ready Preview Oct 19, 2026 4:30am UTC

Request Review

Brings in #314 (v0.2.0 publication record).

Coven-Harness: Claude Code / claude-opus-5-5
Brings in #310 (README and routines spec wording).

Coven-Harness: Claude Code / claude-opus-5-5
@github-code-quality

github-code-quality Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Code coverage overview

Languages: TypeScript

TypeScript / code-coverage-agent

main f85a5b3 ci-macos-timeout 36da240 +/-
63% 63% 0%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.


Updated October 19, 2026 05:08 UTC

Brings in #299 (CF-5 memory citations).

Coven-Harness: Claude Code / claude-opus-5-5
@BunsDev
BunsDev merged commit cb33db5 into main Oct 19, 2026
11 checks passed
@BunsDev
BunsDev deleted the ci-macos-timeout branch October 19, 2026 05:29
BunsDev added a commit that referenced this pull request Oct 19, 2026
Coven-Harness: Claude Code / claude-opus-5-5
BunsDev added a commit that referenced this pull request Oct 19, 2026
Brings in #299 (CF-5 memory citations) and #315 (macOS CI timeout).

Coven-Harness: Claude Code / claude-opus-5-5
BunsDev added a commit that referenced this pull request Oct 19, 2026
Both entries kept in docs/status.md.

Coven-Harness: Claude Code / claude-opus-5-5
BunsDev added a commit that referenced this pull request Oct 19, 2026
Both entries kept in docs/status.md.

Coven-Harness: Claude Code / claude-opus-5-5

This branch was successfully deployed

1 active deployment
Preview — 36da2405 Deployed Oct 19, 2026 by vercel[bot]
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