Skip to content

Normalized the AI disclosure comment to one fixed line per file - #69

Merged
fdesbiens merged 1 commit into
eclipse-threadx:devfrom
fdesbiens:chore/normalize-ai-disclosure
Sep 15, 2026
Merged

fdesbiens merged 1 commit into
eclipse-threadx:devfrom
fdesbiens:chore/normalize-ai-disclosure

Conversation

@fdesbiens

Copy link
Copy Markdown
Contributor

The per-edit AI disclosure named the product and model, so every tool and every model version appended another line. Deduplication was by exact string match, which a version bump defeats, and the product spelling drifted too.

Every file now carries exactly one line, fixed text naming no product:

Portions of this file were generated with AI assistance.

written with the comment character that file already uses. Precise attribution moves to where it is already mandated and already accurate: the Assisted-by trailer, which is per-change, dated and attached to its diff. git log --format='%h %(trailers:key=Assisted-by,valueonly)' -- <path> answers "which tools touched this file" better than a header ever could, because the code a header names gets rewritten while the line stays.

32 files changed, all C sources and headers.

Comment-only. 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.

The six submodules -- libs/threadx, libs/netxduo, libs/usbx, libs/filex and the two under MXChip/AZ3166/deps/lib -- were excluded. No submodule content or gitlink is touched; each of those projects gets its own change.

The per-edit disclosure named the product and model, so every tool and every
model version appended another line rather than recognising the one already
there. Deduplication was by exact string, which a version bump defeats.

Every file now carries exactly one line, fixed text naming no product:

    Portions of this file were generated with AI assistance.

written with the comment character that file already uses. Precise attribution
stays on the commit, where the Assisted-by trailer is dated and attached to the
diff it describes. A header line cannot keep that record honest, because the
code it names gets rewritten and the line stays.

Comment-only, 32 files: 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. Nested
repositories were excluded, so no submodule content or gitlink is touched.

Assisted-by: Claude Code (Opus 5) <noreply@anthropic.com>
@fdesbiens
fdesbiens merged commit 962dc1e into eclipse-threadx:dev Sep 15, 2026
5 checks passed
fdesbiens added a commit that referenced this pull request Sep 17, 2026
The NXP target and #69 were in flight at the same time, so the target landed
carrying the older per-edit disclosure that #69 replaced everywhere else. Eleven
files named the product and model inside the header's Contributors block, and
nine carried no disclosure at all.

All twenty now carry the same fixed line every other file in the repository uses,
written with the comment character already in the file. The Contributors blocks
are untouched: only the AI disclosure moves, and precise attribution stays on the
Assisted-by trailer where it is dated and attached to its diff.

Six vendored files are deliberately excluded. startup_mimxrt1064.S, the two
KSZ8081 PHY sources and the three NetX driver sources carry only their upstream
NXP or Microsoft copyright, with no Eclipse ThreadX line added, so they hold no
AI-generated work to disclose. tx_initialize_low_level.S and tx_user.h do carry
that line and are included.

Comment-only, 20 files: every removed line was a disclosure line and every added
line is the fixed text, with no file left holding zero or more than one. The
target builds clean with Arm GCC 14.3.Rel1.


Assisted-by: Claude Code (Opus 5) <noreply@anthropic.com>
Signed-off-by: Frédéric Desbiens <frederic.desbiens@eclipse-foundation.org>
@fdesbiens
fdesbiens deleted the chore/normalize-ai-disclosure branch September 17, 2026 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant