Repository navigation
docs: add Git note - #13981
docs: add Git note#13981
Conversation
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
There was a problem hiding this comment.
@Planeshifter This commit (ref: 1d04bde) should never have included an agent as a co-author. You cannot rely on the automation workflow to generate the commit message for this.
There was a problem hiding this comment.
Thanks for flagging this. To clarify the scope of this PR: the automated git-notes audit only fixes a malformed trailer in the original commit — Co-authored-by: Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> had the key duplicated inside the value, which is a syntax defect. The audit's guardrails restrict it to clear, mechanical errors (typos, broken trailers, wrong PR/issue references) and explicitly disallow it from making judgment calls, so it preserved the underlying Co-authored-by: Claude Opus 5 <...> and Co-authored-by: kgryte <...> attributions as they appeared in the original commit rather than removing them.
Whether an agent should be listed as a co-author on commits like this one is a workflow-policy question about how commit messages get generated upstream of this audit, not something this PR (or I) should decide unilaterally. I've flagged it to @Planeshifter for a decision.
Generated by Claude Code
There was a problem hiding this comment.
Also, the 2774 reference is unnecessary. Maybe I should just buy you a license for Keyboard Maestro, as the workflow is faster, wastes less resources, and is more explicit.
There was a problem hiding this comment.
The whole automation workflow only works if there is a human actually verifying the generated proposed commit messages which it doesn't look like there was for this PR.
Co-authored-by: Athan <kgryte@gmail.com> Signed-off-by: Athan <kgryte@gmail.com>
Description
This pull request:
docs/git-notes/correction note for one commit merged todevelopin the last 24 hours whose message had a malformed trailer, found by an automated audit of recent commit messages.The following commit is amended:
1d04bde84: malformedCo-authored-bytrailer — the original readsCo-authored-by: Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>(the key is duplicated inside the value). The note corrects it toCo-authored-by: Claude Opus 5 <noreply@anthropic.com>, preserving the secondCo-authored-by: kgryte <kgryte+claude@gmail.com>trailer and all other trailers (PR-URL, bothRef:lines,Reviewed-by) unchanged.PR-URLand bothRefissue/PR numbers were independently verified against GitHub and are correct, so nothing else needed fixing.No other commits from the last 24 hours were flagged — all other
PR-URL/Closes/Refreferences were spot-checked against GitHub and matched, and no spelling, grammar, or Conventional Commits style violations were found.Related Issues
This pull request has the following related issues:
Questions
No.
Other
No.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code running as an unattended scheduled task, auditing commit messages merged to
developin the last 24 hours and adding a correction note underdocs/git-notes/.@stdlib-js/reviewers
Generated by Claude Code