Skip to content

ci: add timeout-minutes to desktop-ci jobs - #1054

Draft
devsy-app[bot] wants to merge 1 commit into
mainfrom
ci-optimizer/desktop-ci-timeout
Draft

ci: add timeout-minutes to desktop-ci jobs#1054
devsy-app[bot] wants to merge 1 commit into
mainfrom
ci-optimizer/desktop-ci-timeout

Conversation

@devsy-app

@devsy-app devsy-app Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Runs inspected

Recent desktop-ci.yml runs on devsy-org/devsy (via gh run list):

  • build-desktop (ubuntu/macos matrix): ~10-12 min
  • build-flatpak: ~7 min
  • lint-and-test: ~1 min

None of these jobs had a timeout-minutes set, so each inherited GitHub Actions 6-hour default.

Problem found

The desktop-ci.yml jobs (lint-and-test, build-desktop, build-flatpak) had no timeout-minutes, leaving them exposed to the 6-hour default. A hung job (a Playwright browser stall, an electron-builder freeze, or a flatpak build stall) would burn the full 6h of CI minutes before being killed.

Minimal change

Added timeout-minutes to the three jobs in .github/workflows/desktop-ci.yml:

  • lint-and-test: 15
  • build-desktop: 45
  • build-flatpak: 30

Values are based on observed run durations with headroom and match the existing convention in pr-ci.yml (timeout-minutes: 45). Single file, 3 added lines, no behavioral change beyond the timeout guard.

Validation performed

  • YAML parse: python3 -c "import yaml; yaml.safe_load(...)" -> YAML OK
  • act: act --list -W .github/workflows/desktop-ci.yml -> enumerated all 4 jobs (changes, lint-and-test, build-desktop, build-flatpak) successfully
  • task cli:format: clean
  • task cli:lint:ci: 0 issues (new-from-patch scoped to this diff)
  • task cli:test: passes; only pre-existing pkg/git (TestRepoClone*) failure remains, which is unrelated to this YAML-only change (known stale assertion on origin/main)

This PR was created by an AI agent as part of an automated daily CI optimization job.

Add timeout-minutes to the lint-and-test (15), build-desktop (45), and build-flatpak (30) jobs in desktop-ci.yml. These long-running build jobs previously had no timeout, so a hung job (playwright browser stall, electron-builder hang, flatpak build stall) would burn the 6h default. Values match the existing pr-ci.yml 45-min convention.
@netlify

netlify Bot commented Aug 15, 2026

Copy link
Copy Markdown

Deploy Preview for devsydev canceled.

Name Link
🔨 Latest commit c40523a
🔍 Latest deploy log https://app.netlify.com/projects/devsydev/deploys/6a80308a8586eb00083eb131

@netlify

netlify Bot commented Aug 15, 2026

Copy link
Copy Markdown

Deploy Preview for images-devsy-sh canceled.

Name Link
🔨 Latest commit c40523a
🔍 Latest deploy log https://app.netlify.com/projects/images-devsy-sh/deploys/6a80308a9a64010008e2d483

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: run a review on demand. To trigger the first review automatically, go to your organization or repository integration settings. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants