Define commit standards: authorship, AI attribution, and trailers
@Ethan-Arrowood ya está trabajando en esto.
Desde el 10/7/2026.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Summary
Write a commit standards guideline doc. Initial scope is authorship and attribution — who a commit is credited to and how AI involvement is disclosed — plus an exploration of commit trailers and eventual enforcement. Keep everything in this one issue for now; enforcement work can be split into sub-tasks later once we've settled the guidelines themselves.
Doc outline
1. Authorship and AI attribution
- Every commit must have a human primary author. The
Authorfield is always a person, never an AI agent or bot identity. - AI-generated or AI-assisted commits should attribute the agent via a trailer —
Co-Authored-By:is the current convention (e.g. what Claude Code appends), but the doc should confirm that's the trailer we want, or propose a better-suited one if the ecosystem has converged elsewhere. - Define where the line is: does tab-completion count? Does an agent writing the whole diff? The doc should give practical guidance so people aren't guessing.
2. Trailers: Signed-off-by and the DCO
Explore whether we should adopt Signed-off-by (as used by the Node.js project and the broader Linux/DCO ecosystem):
- What it actually asserts (Developer Certificate of Origin) vs. what commit signing asserts — they solve different problems and the doc should make that distinction clear
- Whether it makes sense org-wide, only for public OSS repos accepting external contributions, or not at all
- If adopted anywhere, when it's most appropriate (e.g. repos with outside contributors where provenance/licensing assertions matter)
- Recommendation with rationale — it's fine for the answer to be "not now, revisit when X"
3. AI agent workflows
These standards should be embedded in agent workflows, not just written down for humans — agents both author commits and review PRs, and skill-level guidance gives us soft enforcement before any tooling exists:
- Producing commits: skills / agent instructions (
skillsrepo,CLAUDE.md/AGENTS.mdconventions) should encode the authorship rules directly — human as primary author, agent attributed via the agreed trailer, plus any trailer conventions we adopt (e.g.Signed-off-bywhere applicable). Agents following the standard by construction is cheaper than catching violations after the fact. - Reviewing PRs: PR-review skills should include a commit-hygiene check — verify authorship/attribution trailers are present and correctly formatted, and flag violations in review output. This is our interim enforcement layer until (and even after) automated linting exists.
The doc should describe this pattern; updating individual skills to implement it can be follow-up work alongside the enforcement sub-tasks below.
4. Enforcement (exploratory — likely future sub-tasks)
We do very little commit linting/enforcement today. The doc should explore options without committing us to rolling them out everywhere:
- Commit message linting (e.g. commitlint, gitlint) — what we'd check (trailers present, author format) and where it would run (CI, merge queue)
- GitHub-side options: rulesets with metadata patterns, required workflows
- DCO enforcement apps/actions if we adopt
Signed-off-by - Explicit note that guidelines can be adopted before enforcement exists — deciding the standard and enforcing it org-wide are separate efforts, and enforcement is a heavier lift we can stage later. Skill-embedded guidance (§3) is the lightweight interim layer.
Acceptance criteria
- Doc added to this repo covering authorship, AI attribution, and the trailer exploration
- Clear recommendation on
Signed-off-by/DCO adoption (including "not yet" as a valid outcome) - AI agent workflow guidance included (agents follow the standard when committing; review skills check commit hygiene)
- Enforcement options surveyed, with candidate follow-up sub-tasks identified but not yet filed
Related
- #12 (commit signing) — signing verifies who pushed the bits; this doc covers how commits are attributed and formatted. The two docs should cross-link.
- Lenguaje dominante
- JavaScript
- Estrellas
- 3
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de HarperFast/code-guidelines
-
HarperFast/code-guidelines#12 · 1 asignado ·
Todos los issues de HarperFast/code-guidelines
Issues similares
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
apache/cloudstack#14222 ·
-
Browser Waiting for: Product Owner
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
getsentry/sentry-javascript#24577 · 1 comentario ·
-
curation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
amponce/archive-movie-browser#186 ·
-
light
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
aemdemos/patients-stryker#253 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
clerk/javascript#9852 ·