@htekdev/actions-debugger 1.0.40 → 1.0.41
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/errors/caching-artifacts/setup-node-npm-cache-monorepo-lockfile-not-found.yml +118 -0
- package/errors/concurrency-timing/cancel-in-progress-queued-run-status-never-posts.yml +111 -0
- package/errors/concurrency-timing/step-timeout-not-supported-job-holds-runner.yml +101 -0
- package/errors/concurrency-timing/workflow-dispatch-push-shared-concurrency-silent-cancel.yml +104 -0
- package/package.json +1 -1
|
@@ -0,0 +1,118 @@
|
|
|
1
|
+
id: caching-artifacts-032
|
|
2
|
+
title: 'setup-node cache: npm silently skips caching in monorepos — lockfile not at workspace root'
|
|
3
|
+
category: caching-artifacts
|
|
4
|
+
severity: silent-failure
|
|
5
|
+
tags:
|
|
6
|
+
- setup-node
|
|
7
|
+
- npm
|
|
8
|
+
- cache
|
|
9
|
+
- monorepo
|
|
10
|
+
- lockfile
|
|
11
|
+
- cache-dependency-path
|
|
12
|
+
- silent-failure
|
|
13
|
+
patterns:
|
|
14
|
+
- regex: "Warning: No file found for: package-lock\\.json"
|
|
15
|
+
flags: 'i'
|
|
16
|
+
- regex: 'No file found for.*No cache will be (saved|restored)'
|
|
17
|
+
flags: 'i'
|
|
18
|
+
- regex: 'cache-dependency-path.*not found|No lockfile found'
|
|
19
|
+
flags: 'i'
|
|
20
|
+
error_messages:
|
|
21
|
+
- "Warning: No file found for: package-lock.json"
|
|
22
|
+
- "Warning: No file found for: yarn.lock"
|
|
23
|
+
- "Warning: No file found for: pnpm-lock.yaml"
|
|
24
|
+
- "No file found for: package-lock.json. No cache will be saved."
|
|
25
|
+
root_cause: |
|
|
26
|
+
actions/setup-node with cache: 'npm' (or 'yarn' / 'pnpm') searches for the
|
|
27
|
+
package manager lockfile at the workspace root ($GITHUB_WORKSPACE) by default.
|
|
28
|
+
|
|
29
|
+
In monorepos where the lockfile lives in a subdirectory (apps/frontend/,
|
|
30
|
+
packages/api/, etc.), or when the workflow uses working-directory: to change
|
|
31
|
+
the build context, setup-node logs a warning and continues WITHOUT configuring
|
|
32
|
+
a cache. The step exits 0, making this a silent failure.
|
|
33
|
+
|
|
34
|
+
The result: npm ci (or yarn install / pnpm install) downloads all dependencies
|
|
35
|
+
from the network on every workflow run, as though no cache were configured.
|
|
36
|
+
Build times are identical to runs with no cache setting at all.
|
|
37
|
+
|
|
38
|
+
The warning message ("No file found for: package-lock.json") is easily missed
|
|
39
|
+
in long step logs and does not fail the step, so developers often go weeks
|
|
40
|
+
without noticing the cache was never active.
|
|
41
|
+
|
|
42
|
+
The same behavior applies when:
|
|
43
|
+
- yarn.lock is not at workspace root
|
|
44
|
+
- pnpm-lock.yaml is not at workspace root
|
|
45
|
+
- Multiple lockfiles exist across packages (only the first match is used
|
|
46
|
+
unless cache-dependency-path is explicitly set to a glob)
|
|
47
|
+
fix: |
|
|
48
|
+
Use the cache-dependency-path input to specify the path to the lockfile
|
|
49
|
+
relative to the workspace root. Supports glob patterns for monorepos.
|
|
50
|
+
|
|
51
|
+
Single package in subdirectory:
|
|
52
|
+
cache-dependency-path: 'frontend/package-lock.json'
|
|
53
|
+
|
|
54
|
+
Multiple lockfiles across a monorepo:
|
|
55
|
+
cache-dependency-path: '**/package-lock.json'
|
|
56
|
+
|
|
57
|
+
Using a glob creates a combined cache key from all matched lockfiles. Any
|
|
58
|
+
change to any package lockfile invalidates the shared cache — this is
|
|
59
|
+
correct behavior for a monorepo where cross-package installs are common.
|
|
60
|
+
fix_code:
|
|
61
|
+
- language: yaml
|
|
62
|
+
label: 'Single package in subdirectory — point cache-dependency-path at lockfile'
|
|
63
|
+
code: |
|
|
64
|
+
jobs:
|
|
65
|
+
build:
|
|
66
|
+
runs-on: ubuntu-latest
|
|
67
|
+
steps:
|
|
68
|
+
- uses: actions/checkout@v4
|
|
69
|
+
|
|
70
|
+
- uses: actions/setup-node@v4
|
|
71
|
+
with:
|
|
72
|
+
node-version: '22'
|
|
73
|
+
cache: 'npm'
|
|
74
|
+
# WRONG (omitted): setup-node looks at $GITHUB_WORKSPACE/package-lock.json
|
|
75
|
+
# and silently skips caching when not found there
|
|
76
|
+
|
|
77
|
+
# CORRECT: path relative to $GITHUB_WORKSPACE
|
|
78
|
+
cache-dependency-path: 'frontend/package-lock.json'
|
|
79
|
+
|
|
80
|
+
- name: Install dependencies
|
|
81
|
+
working-directory: frontend
|
|
82
|
+
run: npm ci
|
|
83
|
+
- language: yaml
|
|
84
|
+
label: 'Monorepo — cache all packages using glob pattern'
|
|
85
|
+
code: |
|
|
86
|
+
jobs:
|
|
87
|
+
build:
|
|
88
|
+
runs-on: ubuntu-latest
|
|
89
|
+
steps:
|
|
90
|
+
- uses: actions/checkout@v4
|
|
91
|
+
|
|
92
|
+
- uses: actions/setup-node@v4
|
|
93
|
+
with:
|
|
94
|
+
node-version: '22'
|
|
95
|
+
cache: 'npm'
|
|
96
|
+
# Glob matches all package-lock.json files anywhere in the repo
|
|
97
|
+
# Combined hash from all matched lockfiles forms the cache key
|
|
98
|
+
cache-dependency-path: '**/package-lock.json'
|
|
99
|
+
|
|
100
|
+
- name: Install root dependencies
|
|
101
|
+
run: npm ci
|
|
102
|
+
|
|
103
|
+
- name: Install frontend dependencies
|
|
104
|
+
working-directory: packages/frontend
|
|
105
|
+
run: npm ci
|
|
106
|
+
prevention:
|
|
107
|
+
- 'Always set cache-dependency-path when the lockfile is not in the repository root'
|
|
108
|
+
- 'Use **/package-lock.json glob in monorepos to cover all packages with a single cache configuration'
|
|
109
|
+
- 'Verify caching is active by checking setup-node logs for "Cache restored successfully" or "Cache saved"'
|
|
110
|
+
- 'Check for "No file found for: package-lock.json" warnings as early signal that caching is silently disabled'
|
|
111
|
+
- 'When using working-directory: on install steps, ensure cache-dependency-path is also adjusted to match'
|
|
112
|
+
docs:
|
|
113
|
+
- url: 'https://github.com/actions/setup-node#caching-global-packages-data'
|
|
114
|
+
label: 'actions/setup-node: Caching global packages data — cache-dependency-path input'
|
|
115
|
+
- url: 'https://github.com/actions/setup-node/issues/530'
|
|
116
|
+
label: 'actions/setup-node#530: cache: npm silently skips in monorepos without cache-dependency-path'
|
|
117
|
+
- url: 'https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/caching-dependencies-to-speed-up-workflows'
|
|
118
|
+
label: 'GitHub Docs: Caching dependencies to speed up workflows — lockfile path configuration'
|
|
@@ -0,0 +1,111 @@
|
|
|
1
|
+
id: concurrency-timing-027
|
|
2
|
+
title: 'Queued run cancelled by cancel-in-progress before any job starts — required status check never posts, PR permanently blocked'
|
|
3
|
+
category: concurrency-timing
|
|
4
|
+
severity: silent-failure
|
|
5
|
+
tags:
|
|
6
|
+
- concurrency
|
|
7
|
+
- cancel-in-progress
|
|
8
|
+
- required-status-check
|
|
9
|
+
- branch-protection
|
|
10
|
+
- pr-blocked
|
|
11
|
+
- status-check
|
|
12
|
+
patterns:
|
|
13
|
+
- regex: 'This run was cancelled because another run in the same concurrency group'
|
|
14
|
+
flags: 'i'
|
|
15
|
+
- regex: 'Waiting for.*status.*reported|Expected.*Waiting'
|
|
16
|
+
flags: 'i'
|
|
17
|
+
error_messages:
|
|
18
|
+
- "This run was cancelled because another run in the same concurrency group is in progress."
|
|
19
|
+
- "Waiting for status to be reported"
|
|
20
|
+
root_cause: |
|
|
21
|
+
When cancel-in-progress: true cancels a workflow run that was queued but had
|
|
22
|
+
not yet started any job, GitHub does NOT post a status (pending, cancelled, or
|
|
23
|
+
failure) to the commit SHA. The run silently disappears from the run list.
|
|
24
|
+
|
|
25
|
+
Branch protection rules that require a specific status check (e.g., "CI / test")
|
|
26
|
+
only observe statuses that were posted. A run cancelled before its first job
|
|
27
|
+
starts never posts any status — not even "pending".
|
|
28
|
+
|
|
29
|
+
Impact on PRs with high push frequency:
|
|
30
|
+
1. Developer opens PR, pushes commit A — run starts, posts "pending"
|
|
31
|
+
2. Developer pushes fix commit B — run for A is cancelled, run for B is queued
|
|
32
|
+
3. Developer pushes commit C before B's run starts — B's run cancelled (never
|
|
33
|
+
started), run for C queued
|
|
34
|
+
4. Run for C finally starts and posts statuses
|
|
35
|
+
5. But if C is also cancelled before starting, the commit has NO status
|
|
36
|
+
6. Branch protection shows the required check as "Expected" forever
|
|
37
|
+
7. PR cannot be merged — the Merge button stays disabled indefinitely
|
|
38
|
+
|
|
39
|
+
This condition is self-healing if a new commit is pushed (starting a fresh
|
|
40
|
+
run that won't be cancelled), but in rapid-push scenarios the window persists
|
|
41
|
+
for many minutes and developers mistakenly believe the CI is broken.
|
|
42
|
+
fix: |
|
|
43
|
+
Option 1 — Queue instead of cancel (safest):
|
|
44
|
+
concurrency:
|
|
45
|
+
group: "${{ github.workflow }}-${{ github.ref }}"
|
|
46
|
+
cancel-in-progress: false
|
|
47
|
+
|
|
48
|
+
Runs queue behind one another; every commit eventually gets a status posted.
|
|
49
|
+
|
|
50
|
+
Option 2 — Status anchor job:
|
|
51
|
+
Add a minimal first job that completes instantly. Its "queued" + "in_progress"
|
|
52
|
+
status is posted to the commit SHA immediately, ensuring GitHub registers the
|
|
53
|
+
run before any cancellation can remove the status.
|
|
54
|
+
|
|
55
|
+
Option 3 — GitHub Actions recommended pattern:
|
|
56
|
+
Use cancel-in-progress: true for the expensive test jobs only, and have a
|
|
57
|
+
separate fast-posting job that always runs (not subject to concurrency group).
|
|
58
|
+
fix_code:
|
|
59
|
+
- language: yaml
|
|
60
|
+
label: 'Queue runs instead of cancelling — every commit gets a status'
|
|
61
|
+
code: |
|
|
62
|
+
on:
|
|
63
|
+
pull_request:
|
|
64
|
+
|
|
65
|
+
concurrency:
|
|
66
|
+
# Queue instead of cancel — latest run waits, but always posts status
|
|
67
|
+
group: '${{ github.workflow }}-${{ github.ref }}'
|
|
68
|
+
cancel-in-progress: false
|
|
69
|
+
|
|
70
|
+
jobs:
|
|
71
|
+
test:
|
|
72
|
+
runs-on: ubuntu-latest
|
|
73
|
+
steps:
|
|
74
|
+
- uses: actions/checkout@v4
|
|
75
|
+
- run: npm test
|
|
76
|
+
- language: yaml
|
|
77
|
+
label: 'Status anchor job — posts status immediately before cancel window'
|
|
78
|
+
code: |
|
|
79
|
+
on:
|
|
80
|
+
pull_request:
|
|
81
|
+
|
|
82
|
+
concurrency:
|
|
83
|
+
group: '${{ github.workflow }}-${{ github.ref }}'
|
|
84
|
+
cancel-in-progress: true # OK — anchor ensures status is posted first
|
|
85
|
+
|
|
86
|
+
jobs:
|
|
87
|
+
# Minimal job: completes in seconds, ensuring a status is recorded on the SHA
|
|
88
|
+
anchor:
|
|
89
|
+
runs-on: ubuntu-latest
|
|
90
|
+
steps:
|
|
91
|
+
- run: echo "CI registered for ${{ github.sha }}"
|
|
92
|
+
|
|
93
|
+
# Expensive test job that is safe to cancel
|
|
94
|
+
test:
|
|
95
|
+
needs: anchor
|
|
96
|
+
runs-on: ubuntu-latest
|
|
97
|
+
steps:
|
|
98
|
+
- uses: actions/checkout@v4
|
|
99
|
+
- run: npm test
|
|
100
|
+
prevention:
|
|
101
|
+
- 'Avoid cancel-in-progress: true on workflows whose jobs are required status checks for branch protection'
|
|
102
|
+
- 'Use cancel-in-progress: false with queuing semantics when PR mergeability must be preserved on every commit'
|
|
103
|
+
- 'Add a lightweight anchor job as the first required job to ensure a status posts before any cancel window closes'
|
|
104
|
+
- 'Monitor PRs stuck with Expected required checks — check whether recent commits had all their runs cancelled before starting'
|
|
105
|
+
docs:
|
|
106
|
+
- url: 'https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/using-concurrency'
|
|
107
|
+
label: 'GitHub Actions: Using concurrency — cancel-in-progress behavior'
|
|
108
|
+
- url: 'https://github.com/orgs/community/discussions/21280'
|
|
109
|
+
label: 'GitHub Community: Required status check never posts when run cancelled before start (120+ reactions)'
|
|
110
|
+
- url: 'https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/about-protected-branches#require-status-checks-before-merging'
|
|
111
|
+
label: 'GitHub Docs: Required status checks — how commit statuses are evaluated for branch protection'
|
|
@@ -0,0 +1,101 @@
|
|
|
1
|
+
id: concurrency-timing-025
|
|
2
|
+
title: 'No step-level timeout — a hung step holds the runner slot for up to 6 hours'
|
|
3
|
+
category: concurrency-timing
|
|
4
|
+
severity: limitation
|
|
5
|
+
tags:
|
|
6
|
+
- timeout
|
|
7
|
+
- hung-step
|
|
8
|
+
- runner-slot
|
|
9
|
+
- step
|
|
10
|
+
- limitation
|
|
11
|
+
- self-hosted-runner
|
|
12
|
+
patterns:
|
|
13
|
+
- regex: 'The runner has received a shutdown signal'
|
|
14
|
+
flags: 'i'
|
|
15
|
+
- regex: 'Error: The process.*timed out after \d+ minutes'
|
|
16
|
+
flags: 'i'
|
|
17
|
+
- regex: 'Canceling since the workflow was cancelled'
|
|
18
|
+
flags: 'i'
|
|
19
|
+
error_messages:
|
|
20
|
+
- "The runner has received a shutdown signal. This can happen when the runner service is stopped, or a manually started runner is canceled."
|
|
21
|
+
- "Error: The process '/usr/bin/bash' failed with exit code 124"
|
|
22
|
+
- "Canceling since the workflow was cancelled."
|
|
23
|
+
root_cause: |
|
|
24
|
+
GitHub Actions supports timeout-minutes at the job level only. There is no
|
|
25
|
+
per-step timeout. A single run: step that hangs (network call blocked, test
|
|
26
|
+
suite deadlocked, subprocess waiting on stdin) holds the entire job until
|
|
27
|
+
the job-level timeout fires.
|
|
28
|
+
|
|
29
|
+
The default job timeout is 6 hours (360 minutes) for GitHub-hosted runners
|
|
30
|
+
and 35 days (unlimited in practice) for self-hosted runners. A single hung
|
|
31
|
+
step therefore silently consumes 6 hours of runner minutes and one complete
|
|
32
|
+
runner slot before GitHub kills the job.
|
|
33
|
+
|
|
34
|
+
Runner slot starvation is the secondary effect: while the hung job occupies a
|
|
35
|
+
runner, queued jobs wait. On self-hosted runners with limited capacity one
|
|
36
|
+
hung step can block an entire team's CI queue indefinitely.
|
|
37
|
+
|
|
38
|
+
This is a documented platform limitation tracked in actions/runner#1120
|
|
39
|
+
(220+ reactions, open since 2020) with no scheduled fix date.
|
|
40
|
+
fix: |
|
|
41
|
+
Apply one of these workarounds depending on operating system:
|
|
42
|
+
|
|
43
|
+
Linux/macOS: Wrap the command with the system timeout utility (seconds):
|
|
44
|
+
run: timeout 300 ./integration-test.sh
|
|
45
|
+
|
|
46
|
+
Cross-platform: Use curl/Invoke-RestMethod built-in timeout options for
|
|
47
|
+
network calls, and test-runner native timeouts for test suites.
|
|
48
|
+
|
|
49
|
+
Always set timeout-minutes explicitly on every job to establish a hard upper
|
|
50
|
+
bound regardless of which step hangs.
|
|
51
|
+
fix_code:
|
|
52
|
+
- language: yaml
|
|
53
|
+
label: 'Set explicit job timeout and use OS timeout for individual steps'
|
|
54
|
+
code: |
|
|
55
|
+
jobs:
|
|
56
|
+
build:
|
|
57
|
+
runs-on: ubuntu-latest
|
|
58
|
+
timeout-minutes: 30 # explicit ceiling — never rely on 6h default
|
|
59
|
+
|
|
60
|
+
steps:
|
|
61
|
+
- uses: actions/checkout@v4
|
|
62
|
+
|
|
63
|
+
# Linux/macOS: system timeout (in seconds)
|
|
64
|
+
- name: Run integration tests
|
|
65
|
+
run: timeout 180 ./scripts/integration-test.sh
|
|
66
|
+
|
|
67
|
+
# Network call with built-in timeout
|
|
68
|
+
- name: Fetch external resource
|
|
69
|
+
run: |
|
|
70
|
+
curl --max-time 60 --retry 3 https://api.example.com/data -o data.json
|
|
71
|
+
- language: yaml
|
|
72
|
+
label: 'Self-hosted runner — conservative timeout prevents slot starvation'
|
|
73
|
+
code: |
|
|
74
|
+
jobs:
|
|
75
|
+
deploy:
|
|
76
|
+
runs-on: [self-hosted, production]
|
|
77
|
+
timeout-minutes: 45 # critical on self-hosted — hung jobs block all runners
|
|
78
|
+
|
|
79
|
+
steps:
|
|
80
|
+
- name: Deploy with timeout guard
|
|
81
|
+
shell: pwsh
|
|
82
|
+
run: |
|
|
83
|
+
$job = Start-Job { ./deploy.ps1 }
|
|
84
|
+
if (-not (Wait-Job $job -Timeout 120)) {
|
|
85
|
+
Stop-Job $job
|
|
86
|
+
throw "Deploy script timed out after 120s"
|
|
87
|
+
}
|
|
88
|
+
Receive-Job $job
|
|
89
|
+
prevention:
|
|
90
|
+
- 'Always set explicit timeout-minutes on every job — never rely on the 6-hour GitHub-hosted default'
|
|
91
|
+
- 'Use OS-level timeout utilities (timeout on Linux/macOS) for individual network calls and scripts'
|
|
92
|
+
- 'Configure test-runner-native timeouts for test suites — they are more granular than job timeouts'
|
|
93
|
+
- 'Monitor runner queue depth on self-hosted runners — sudden growth often signals a hung step holding a slot'
|
|
94
|
+
- 'Set timeout-minutes: 5 on jobs you know should complete quickly to catch regressions early'
|
|
95
|
+
docs:
|
|
96
|
+
- url: 'https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#jobsjob_idtimeout-minutes'
|
|
97
|
+
label: 'GitHub Actions: jobs.<id>.timeout-minutes — job-level timeout (no step-level equivalent)'
|
|
98
|
+
- url: 'https://github.com/actions/runner/issues/1120'
|
|
99
|
+
label: 'actions/runner#1120: Feature request — step-level timeout (220+ reactions, open since 2020)'
|
|
100
|
+
- url: 'https://docs.github.com/en/actions/administering-github-actions/usage-limits-billing-and-administration#usage-limits'
|
|
101
|
+
label: 'GitHub Actions usage limits — default job timeout values per runner type'
|
|
@@ -0,0 +1,104 @@
|
|
|
1
|
+
id: concurrency-timing-026
|
|
2
|
+
title: 'workflow_dispatch run silently cancelled when push to same branch shares the concurrency group'
|
|
3
|
+
category: concurrency-timing
|
|
4
|
+
severity: silent-failure
|
|
5
|
+
tags:
|
|
6
|
+
- concurrency
|
|
7
|
+
- workflow-dispatch
|
|
8
|
+
- cancel-in-progress
|
|
9
|
+
- push
|
|
10
|
+
- manual-trigger
|
|
11
|
+
- silent-cancel
|
|
12
|
+
patterns:
|
|
13
|
+
- regex: 'This run was cancelled because another run in the same concurrency group'
|
|
14
|
+
flags: 'i'
|
|
15
|
+
- regex: 'Canceling run due to.*concurrency group'
|
|
16
|
+
flags: 'i'
|
|
17
|
+
- regex: 'This run has been canceled'
|
|
18
|
+
flags: 'i'
|
|
19
|
+
error_messages:
|
|
20
|
+
- "This run was cancelled because another run in the same concurrency group is in progress."
|
|
21
|
+
- "Canceling run due to a newer request for the same concurrency group."
|
|
22
|
+
- "This run has been canceled."
|
|
23
|
+
root_cause: |
|
|
24
|
+
A common concurrency pattern groups runs by workflow name and ref:
|
|
25
|
+
|
|
26
|
+
concurrency:
|
|
27
|
+
group: "${{ github.workflow }}-${{ github.ref }}"
|
|
28
|
+
cancel-in-progress: true
|
|
29
|
+
|
|
30
|
+
workflow_dispatch and push events on the same branch share identical
|
|
31
|
+
github.workflow and github.ref values, so they are placed in the same
|
|
32
|
+
concurrency slot.
|
|
33
|
+
|
|
34
|
+
A developer manually triggers a workflow_dispatch run (e.g., to deploy to
|
|
35
|
+
staging, run a migration, or kick off a release). While it is running, any
|
|
36
|
+
push to that branch — including a small documentation fix or revert — creates
|
|
37
|
+
a new run in the same concurrency slot and immediately cancels the in-progress
|
|
38
|
+
manual dispatch with no notification.
|
|
39
|
+
|
|
40
|
+
The developer discovers this only when checking on the deployment minutes
|
|
41
|
+
later to find it was cancelled mid-run. The automatic push run that replaced
|
|
42
|
+
it may be completely irrelevant to the manual operation.
|
|
43
|
+
fix: |
|
|
44
|
+
Include github.event_name in the concurrency group key to give workflow_dispatch
|
|
45
|
+
and push events separate concurrency slots:
|
|
46
|
+
|
|
47
|
+
concurrency:
|
|
48
|
+
group: "${{ github.workflow }}-${{ github.event_name }}-${{ github.ref }}"
|
|
49
|
+
cancel-in-progress: true
|
|
50
|
+
|
|
51
|
+
workflow_dispatch runs now share a slot only with other workflow_dispatch runs
|
|
52
|
+
on the same branch, and push runs share a slot only with other push runs.
|
|
53
|
+
|
|
54
|
+
For deployment or migration workflows where even manual-vs-manual cancellation
|
|
55
|
+
is undesirable, use cancel-in-progress: false and let runs queue.
|
|
56
|
+
fix_code:
|
|
57
|
+
- language: yaml
|
|
58
|
+
label: 'Include event_name in concurrency group — isolates manual from automatic triggers'
|
|
59
|
+
code: |
|
|
60
|
+
on:
|
|
61
|
+
push:
|
|
62
|
+
branches: [main]
|
|
63
|
+
workflow_dispatch:
|
|
64
|
+
|
|
65
|
+
concurrency:
|
|
66
|
+
# event_name isolates push and workflow_dispatch into separate concurrency slots
|
|
67
|
+
group: '${{ github.workflow }}-${{ github.event_name }}-${{ github.ref_name }}'
|
|
68
|
+
cancel-in-progress: true
|
|
69
|
+
|
|
70
|
+
jobs:
|
|
71
|
+
deploy:
|
|
72
|
+
runs-on: ubuntu-latest
|
|
73
|
+
steps:
|
|
74
|
+
- uses: actions/checkout@v4
|
|
75
|
+
- run: ./deploy.sh
|
|
76
|
+
- language: yaml
|
|
77
|
+
label: 'Separate workflow file for manual operations — no cancel-in-progress'
|
|
78
|
+
code: |
|
|
79
|
+
# deploy-manual.yml — only triggered by workflow_dispatch
|
|
80
|
+
on:
|
|
81
|
+
workflow_dispatch:
|
|
82
|
+
inputs:
|
|
83
|
+
environment:
|
|
84
|
+
description: 'Target environment'
|
|
85
|
+
required: true
|
|
86
|
+
type: choice
|
|
87
|
+
options: [staging, production]
|
|
88
|
+
|
|
89
|
+
# No shared concurrency group with push workflows
|
|
90
|
+
concurrency:
|
|
91
|
+
group: 'manual-deploy-${{ github.event.inputs.environment }}'
|
|
92
|
+
# cancel-in-progress defaults to false — manual deploys queue, not cancel
|
|
93
|
+
prevention:
|
|
94
|
+
- 'Always include github.event_name in concurrency group keys for workflows triggered by both push and workflow_dispatch'
|
|
95
|
+
- 'Use separate workflow files for manual deployment operations to avoid shared concurrency with automated triggers'
|
|
96
|
+
- 'Add a summary step or Telegram/Slack notification in the cleanup phase to alert when a run is cancelled mid-execution'
|
|
97
|
+
- 'Audit all concurrency group patterns when adding workflow_dispatch to an existing automated workflow'
|
|
98
|
+
docs:
|
|
99
|
+
- url: 'https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/using-concurrency'
|
|
100
|
+
label: 'GitHub Actions: Using concurrency — group patterns and cancel-in-progress'
|
|
101
|
+
- url: 'https://github.com/orgs/community/discussions/5435'
|
|
102
|
+
label: 'GitHub Community: workflow_dispatch cancelled by push with same concurrency group'
|
|
103
|
+
- url: 'https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/accessing-contextual-information-about-workflow-runs#github-context'
|
|
104
|
+
label: 'GitHub context: github.event_name property'
|
package/package.json
CHANGED