Skip to content

Fix concurrency-limits audit to report at workflow level#1789

Closed
woodruffw wants to merge 2 commits intozizmorcore:mainfrom
woodruffw-bot:claude/create-upstream-pr-fwH3b
Closed

Fix concurrency-limits audit to report at workflow level#1789
woodruffw wants to merge 2 commits intozizmorcore:mainfrom
woodruffw-bot:claude/create-upstream-pr-fwH3b

Conversation

@woodruffw
Copy link
Member

Closes #1627.

Fixes #1626.

claude and others added 2 commits March 23, 2026 08:56
…orcore#1627)

Rebase PR zizmorcore#1627 onto current main and update all integration test
snapshots to reflect the consolidated concurrency-limits findings.

The audit now collects affected jobs and produces a single workflow-level
finding per issue type (missing concurrency / missing cancel-in-progress)
instead of emitting one finding per job.

https://claude.ai/code/session_019imdGGVzhxj9FwhK2nWhRa
Regenerate all 5 crater test snapshots to reflect the consolidated
workflow-level concurrency-limits findings instead of per-job findings.

https://claude.ai/code/session_01PXxMnbkNKhN6f4dUVZnGW9
@woodruffw woodruffw self-assigned this Mar 23, 2026
@woodruffw woodruffw added the bugfix Fixes a known bug label Mar 23, 2026
= note: audit confidence → High

3 findings (1 ignored, 1 suppressed): 0 informational, 1 low, 0 medium, 0 high
help[concurrency-limits]: insufficient job-level concurrency limits
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This indicates that the ignore spans have changed, which needs to be fixed.

@woodruffw woodruffw closed this Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a known bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: concurrency-limits audit creates duplicate findings for each job

3 participants