Normalized the AI disclosure comment to one fixed line per file - #740
Merged
fdesbiens merged 1 commit intoSep 15, 2026
Merged
Conversation
The per-edit disclosure named the product and model, so every agent and every
model version appended another line. 74 files carried two to four of them, and
the same five products had accumulated 13 spellings -- Copilot against GitHub
Copilot, Claude Sonnet 4.6 against claude-sonnet-4.6, four spellings of Codex.
Twenty assembly lines carried a doubled comment marker, `; //` or `@ //`.
Every file now carries exactly one line, fixed text naming no product:
Portions of this file were generated with AI assistance.
It is written with the comment character that file already uses, so the `;`
and `@` assembly files keep theirs and the doubled markers are gone. Precise
attribution stays on the commit, where the Assisted-by trailer is per-change,
dated and attached to the diff it describes. A header line cannot hold that
record honestly, because the code it names gets rewritten and the line stays.
A file-level flag answers whether; the history answers who.
Comment-only. 455 files, 455 insertions and 574 deletions: every removed line
was a disclosure line, every added line is the fixed text, and no file is left
with zero or with more than one. `scripts/check_ports.sh` passes, including the
reproducibility check that would catch a ports_arch master and its generated
copies drifting apart. Recompiled against dev, every file that builds without a
vendor toolchain gives a byte-identical object: 19 of 19 C files under common,
100 of 100 GNU assembly files, and all 16 assemblable files whose comment
marker changed. The 10 remaining marker changes are ac5 and IAR sources where
`;` already started the comment and only the redundant `//` was removed.
Assisted-by: Claude Code (Opus 5) <noreply@anthropic.com>
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.
The per-edit AI disclosure named the product and model, so every agent and every
model version appended another line. Agents were deduplicating on exact string
match, which a version bump defeats:
Codex (gpt 5.4),Codex (gpt 5.5),Codex (GPT-5)andCodex (gpt-5.6-sol)are four lines for one product.Worst case,
ports/cortex_m4/gnu/inc/tx_port.hcarried four lines naming Gemini,Copilot twice and Claude Code.
Every file now carries exactly one, fixed text naming no product:
written with the comment character that file already uses, so
;and@assembly files keep theirs and the doubled
; //and@ //forms are gone.Precise attribution moves to where it is already mandated and already accurate:
the
Assisted-bytrailer, which is per-change, dated and attached to its diff.git log --format='%h %(trailers:key=Assisted-by,valueonly)' -- <path>answers"which agents touched this file" better than the header ever did. The header line
could not stay honest, because the code it named gets rewritten and the line
stays -- files credited Gemini 58 times against one Gemini commit in history.
Comment-only. 455 files, 455 insertions, 574 deletions; every removed line was a
disclosure line, every added line is the fixed text, and no file is left with
zero or more than one.
scripts/check_ports.shpasses, including the check thatwould catch a
ports_archmaster and its generated copies drifting apart.Recompiled against
dev, every file that builds without a vendor toolchain isbyte-identical: 19/19 C under
common, 100/100 GNU assembly, and all 16assemblable files whose marker changed. The 10 remaining marker changes are ac5
and IAR sources where
;already started the comment and only a redundant//came off.
Not covered here, and left alone deliberately: five new-file
AI Disclosure:headers are malformed -- four say "Claude Code" with no model, one says "Claude
Opus 5" with no product. Fixing those means guessing which version wrote them,
so they need a human decision rather than a script.