Skip to content

fix: tolerate authorship label delete races - #15

Merged
haasonsaas merged 1 commit into
mainfrom
codex/fix-agent-authorship-delete-race-20260428
Apr 28, 2026
Merged

fix: tolerate authorship label delete races#15
haasonsaas merged 1 commit into
mainfrom
codex/fix-agent-authorship-delete-race-20260428

Conversation

@haasonsaas

Copy link
Copy Markdown
Contributor

Summary

  • handle the missed Bugbot feedback from fix: make authorship labels idempotent #14
  • keep stale authorship-label deletes strict for real failures, but tolerate a 404 when a concurrent run already deleted the label
  • preserve idempotent label application without introducing noisy race failures

Verification

  • actionlint .github/workflows/agent-authorship-label.yml
  • ruby -c .github/scripts/classify-agent-authorship.rb

@cursor

cursor Bot commented Apr 28, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Low risk workflow-only change that makes label cleanup idempotent by tolerating expected 404/not-found responses from concurrent runs while still surfacing other API failures.

Overview
Makes the agent-authorship-label workflow resilient to concurrent executions by wrapping label deletion in delete_label_if_present, which treats 404/"not found" responses as non-fatal.

This keeps removal of stale authorship labels strict for real errors while avoiding flaky failures when another run already deleted the label.

Reviewed by Cursor Bugbot for commit 25ef152. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas
haasonsaas merged commit 6d8dd1b into main Apr 28, 2026
4 checks passed
@haasonsaas
haasonsaas deleted the codex/fix-agent-authorship-delete-race-20260428 branch April 28, 2026 19:15

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 25ef152. Configure here.

Comment thread .github/workflows/agent-authorship-label.yml
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.

1 participant