admin/accounted
3
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

ci: add missing timeout-minutes and concurrency guards #15

Merged
admin merged 1 commits from ci/timeout-and-concurrency-guards into main 2026-09-27 02:31:24 +00:00
Conversation 0 Commits 1 Files Changed 1
+5
admin commented 2026-09-27 02:31:00 +00:00
Owner
Copy Link
Copy Source

Part of a fleet-wide CI-speed pass (audit already scoped this repo).

This repo has one workflow, .gitea/workflows/masterplan-lock.yml, and it had both of the issues this pass targets:

  1. Missing timeout-minutes on the check job. It's a cheap, dependency-free POSIX-sh check (no toolchain), so a 10-minute job-level timeout is added as a safety net against a hung runner.
  2. Double-trigger waste: the workflow runs on both push (branches: ["**"], i.e. every branch including any a PR could target) and pull_request, so every PR commit ran the same check twice. Added a top-level concurrency: block (same pattern already used in admin/serv0, admin/s0cial, admin/ppl0, admin/pers0n) so the superseded duplicate run is cancelled. The on: triggers themselves are unchanged.

No other changes: caching, action pinning, job logic, and branch protection are all left untouched.

🤖 Generated with Claude Code

Part of a fleet-wide CI-speed pass (audit already scoped this repo). This repo has one workflow, `.gitea/workflows/masterplan-lock.yml`, and it had both of the issues this pass targets: 1. **Missing `timeout-minutes`** on the `check` job. It's a cheap, dependency-free POSIX-sh check (no toolchain), so a 10-minute job-level timeout is added as a safety net against a hung runner. 2. **Double-trigger waste**: the workflow runs on both `push` (`branches: ["**"]`, i.e. every branch including any a PR could target) and `pull_request`, so every PR commit ran the same check twice. Added a top-level `concurrency:` block (same pattern already used in admin/serv0, admin/s0cial, admin/ppl0, admin/pers0n) so the superseded duplicate run is cancelled. The `on:` triggers themselves are unchanged. No other changes: caching, action pinning, job logic, and branch protection are all left untouched. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
admin added 1 commit 2026-09-27 02:31:00 +00:00
fix(ci): add timeout-minutes and concurrency guard to masterplan-lock
masterplan-lock / check (push) Successful in 10s
Details
masterplan-lock / check (pull_request) Successful in 7s
Details
f687158138 
Fleet-wide CI-speed pass: masterplan-lock.yml had no job-level
timeout-minutes (a hung runner could block indefinitely) and triggers
on both push (branches: ["**"]) and pull_request, which runs the same
check twice per PR commit. Add a 10-minute timeout for this cheap
single-check job and a concurrency group (matching the pattern already
used in admin/serv0, admin/s0cial, admin/ppl0, admin/pers0n) to cancel
the superseded duplicate run without changing what triggers CI.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
siax-bot approved these changes 2026-09-27 02:31:04 +00:00
admin merged commit b3c8d88f0d into main 2026-09-27 02:31:24 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
siax-bot
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
siax-bot siax-ci siax-healer (siax-healer (CI self-healing agent)) admin (Simon Axelsson)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/accounted#15