Skip to content

fix: make authorship labels idempotent#14

Merged
haasonsaas merged 1 commit intomainfrom
codex/fix-agent-authorship-label-churn-20260428
Apr 28, 2026
Merged

fix: make authorship labels idempotent#14
haasonsaas merged 1 commit intomainfrom
codex/fix-agent-authorship-label-churn-20260428

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • stop removing and re-adding the selected authorship label on every synchronize event
  • replace misleading no-trailer human-authored classification with agent-assisted
  • keep clearing the legacy human-authored label and document the new semantics
  • accept one-character required Maestro trailer values

Verification

  • ruby -c .github/scripts/classify-agent-authorship.rb
  • classifier smoke checks for agent-assisted, agent-authored, and mixed-authorship outputs
  • ruby YAML/JSON parse check for workflow and template metadata
  • actionlint .github/workflows/agent-authorship-label.yml

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 28, 2026

PR Summary

Low Risk
Low risk: changes are limited to PR labeling/classification logic in GitHub Actions and a small Ruby classifier; main risk is mislabeling PRs due to updated heuristics/regex tweaks.

Overview
Updates authorship classification to treat PRs without explicit Maestro trailers as agent-assisted (replacing human-authored), while still labeling mixed-authorship when both trailered and untrailered commits exist.

Tweaks required-trailer regexes to accept one-character values, and makes the labeling workflow idempotent by only removing stale labels and only adding the desired label when it’s not already present; docs/template text are updated to match the new semantics.

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

@haasonsaas haasonsaas merged commit 6e2afc0 into main Apr 28, 2026
4 checks passed
@haasonsaas haasonsaas deleted the codex/fix-agent-authorship-label-churn-20260428 branch April 28, 2026 18:54
Copy link
Copy Markdown

@cursor cursor Bot left a comment

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 65837c7. 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