docs: make the Co-Authored-By trailer model-agnostic - #307
Merged
Conversation
The git practices mandated 'Co-Authored-By: Claude Sonnet 4.6', which asked every non-Claude contributor (Codex, Gemini CLI, opencode, ...) to either misattribute their work or silently deviate. Generalize the rule: AI-assisted commits must include a Co-Authored-By trailer naming the ACTUAL assistant + model used, with examples for several tools. The task-workflow commit examples now vary across tools to illustrate this, and the PR template's tool-specific line is neutralized. Co-Authored-By: opencode/glm-5.2 <noreply@opencode.ai>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The git practices mandated
Co-Authored-By: Claude Sonnet 4.6, which asked every non-Claude contributor (Codex, Gemini CLI, opencode, …) to either misattribute their work or silently deviate. The trailer still has value (it signals AI-assisted commits to reviewers/future readers), so this keeps it but makes it accurate:docs/engineering-practices.md— the mandate now reads "include a co-author trailer naming the actual assistant + model used", with examples for Claude, Codex, and opencode.docs/task-workflow.md— the commit-message template uses a<assistant/model>placeholder; the two concrete examples now vary across tools (Claude + Codex) to illustrate; the PR template's🤖 Generated with [Claude Code]→🤖 Generated with an AI assistant.CLAUDE.md and AGENTS.md were checked and contain no literal trailer text, so no sync edit was needed there.
This commit itself uses the new convention (
Co-Authored-By: opencode/glm-5.2) as a demonstration.Test plan
npm run typecheck && npm run lint && npm run format:check— clean (docs-only change)grepconfirms no unintended hardcodedClaude Sonnet 4.6/Generated with [Claude Code]references remain (the two kept matches are deliberate examples, now alongside other tools).