Skip to content

fix(action): Honor PR trigger parallelism#325

Merged
dcramer merged 1 commit into
mainfrom
fix/action-trigger-parallelism
May 15, 2026
Merged

fix(action): Honor PR trigger parallelism#325
dcramer merged 1 commit into
mainfrom
fix/action-trigger-parallelism

Conversation

@dcramer
Copy link
Copy Markdown
Member

@dcramer dcramer commented May 15, 2026

Matched PR triggers now run through the configured concurrency pool instead of fanning out all matched triggers at once. This makes the Action parallel input and runner concurrency setting serialize trigger execution when set to 1, while preserving the existing file-level semaphore inside running triggers.

Concurrency Docs

Document runner.concurrency and clarify the split between Action and CLI behavior: GitHub Actions use the setting for matched trigger dispatch and file analysis, while CLI --parallel gates file analysis across running skills.

Adds deterministic regression coverage for multiple matched triggers.

Fixes #323

Run matched PR triggers through the configured concurrency pool instead of fanning out every matched trigger at once. This makes the action-level parallel input and runner concurrency apply to trigger dispatch as documented.

Add regression coverage that holds the first matched trigger open and verifies a second trigger is not dispatched when parallel is set to 1.

Fixes GH-323

Co-Authored-By: GPT-5 Codex <noreply@anthropic.com>
@dcramer dcramer force-pushed the fix/action-trigger-parallelism branch from f898e30 to b41caf9 Compare May 15, 2026 17:44
@dcramer dcramer marked this pull request as ready for review May 15, 2026 17:45
@dcramer dcramer merged commit 3c1d8d2 into main May 15, 2026
15 checks passed
@dcramer dcramer deleted the fix/action-trigger-parallelism branch May 15, 2026 17:50
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.

GitHub Action PR workflow ignores configured trigger parallelism

1 participant