Skip to content

Added a check that keeps the AI disclosure comment in its accepted form - #87

Open
fdesbiens wants to merge 1 commit into
eclipse-threadx:devfrom
fdesbiens:chore/add-ai-disclosure-check
Open

fdesbiens wants to merge 1 commit into
eclipse-threadx:devfrom
fdesbiens:chore/add-ai-disclosure-check

Conversation

@fdesbiens

Copy link
Copy Markdown
Contributor

Nothing enforced the disclosure convention, so the drift it exists to prevent returned twice: once when a port merged after the normalisation pass carrying the older per-edit form, and once because that pass had covered source files only, leaving build files and scripts untouched for months.

Added scripts/check_ai_disclosure.sh, which rejects the superseded per-edit form, a doubled comment marker, more than one disclosure line in a file, and any spelling of the line that is not exact. It runs from repo_checks.yml, a workflow with no path filter, because a source-path filter is what hid the build files the first time.

The check passes on this repository as it stands. Each of its four rules was confirmed to fail on a tree with that defect reintroduced, and to pass once it was removed.

Assisted-by: Claude Code (Opus 5) noreply@anthropic.com

…d form

Nothing enforced the disclosure convention. In the sibling repositories the
drift it exists to prevent returned twice, once when a port merged after the
normalisation pass carrying the older per-edit form, and once because that pass
had covered source files only. This repository is currently clean, and this
keeps it so.

Added scripts/check_ai_disclosure.sh, which rejects the superseded per-edit
form, a doubled comment marker, more than one disclosure line in a file, and
any spelling of the line that is not exact. It runs from repo_checks.yml, a
workflow with no path filter, because a source-path filter is what hid the
build files elsewhere.

The check passes on this repository as it stands. Each of its four rules was
confirmed to fail on a tree with that defect reintroduced, and to pass once it
was removed.

Assisted-by: Claude Code (Opus 5) <noreply@anthropic.com>

This branch has not been deployed

No deployments
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