Development record
One early commit carries the tool name in the git author field (pre-dating the Assisted-by policy) and two commits are dependabot; any contributors rendering from git history needs an explicit rule labelling bot and tool author rows separately from the humans who are authors of record, or the pre-policy commit silently appears as a human contributor
issueresolvedminoriss-2608220150157501
One early commit carries the tool name in the git author field (pre-dating the Assisted-by policy) and two commits are dependabot; any contributors rendering from git history needs an explicit rule labelling bot and tool author rows separately from the humans who are authors of record, or the pre-policy commit silently appears as a human contributor
Grounds
- pursued: the contributors rendering separates tool and bot authors from the humans of record; shown wrong if a pre-policy tool-authored commit renders as a human contributor
Frontmatter
| id | iss-2608220150157501 |
|---|---|
| type | issue |
| lifecycle | resolved |
| severity | minor |
.abcd/work/issues/resolved/iss-2608220150157501-pre-policy-tool-author-and-dependabot.md
open on GitHub ↗ · commit history ↗
mentions1
- iss-2609081940550352LoadAuthorship puts every non-None Assisted-by value into ByModel and registers its colon-prefix as a vendor, then routes any shortlog author whose name equals a vendor key into the Bots and tools row. A conformant trailer whose vendor token is an existing single-token git author name therefore moves that human author whole shortlog count into Bots on /contributors/. The match is exact and case-sensitive. A second, free-form Assisted-by line also publishes as a bar label, because scripts/check-attribution.sh is a presence check (grep -Eq TRAILER_RE) rather than an every-line-conforms check, and underAttributionEscape skips banned tokens on empty data-src spans. Display only, and the trailer stays review-visible. The vendors[au.Name] clause must not simply be dropped: TestAuthorshipSeparatesToolsFromPeople pins the pre-policy tool-author case it implements (iss-2608220150157501). Fix: demote to Bots on a structural machine-identity signal (a [bot] suffix, or a vendor-name match AND a noreply/bot address) so a vendor prefix cannot reclassify a human while Claude at a noreply address and the fixture Assistant author stay in Bots; chart only trailer-shaped values (the attribution grammar including the optional 1m suffix) and clip or drop free-form extras rather than publishing them as labels. Detector: a fixture author named like a human of record, plus an Assisted-by trailer whose vendor is that same name, stays in Humans; TestAuthorshipSeparatesToolsFromPeople still puts Assistant and depbot[bot] in Bots; a free-form extra trailer does not appear as a ByModel label. Reported as GitHub issue 623 against ec7f40d6.