Re: [PATCH v2] scripts/checkpatch: add Assisted-by: tag validation

[Jonathan Corbet <corbet@lwn.net>]

Harry Wentland <harry.wentland@amd.com> writes:

> The coding-assistants.rst documentation defines the Assisted-by: tag
> format for AI-assisted contributions as:
>
>   Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]
>
> This format does not use an email address, so checkpatch currently
> reports a false positive about an invalid email when encountering this
> tag.
>
> Add Assisted-by: to the recognized signature tags and standard signature
> list. When an Assisted-by: tag is found, validate it instead of checking
> for an email address.
>
> Examples of passing tags:
> - Claude:claude-3-opus coccinelle sparse
> - FOO:BAR.baz
> - Copilot Github:claude-3-opus
> - GitHub Copilot:Claude Opus 4.6
> - My Cool Agent:v1.2.3 coccinelle sparse
>
> Examples of tags triggering the new warning:
> - Claude coccinelle sparse
> - JustAName
> - :missing-agent
>
> Cc: Jonathan Corbet <corbet@lwn.net>
> Cc: Jani Nikula <jani.nikula@linux.intel.com>
> Assisted-by: Claude:claude-opus-4.6
> Co-developed-by: Alex Hung <alex.hung@amd.com>
> Signed-off-by: Alex Hung <alex.hung@amd.com>
> Signed-off-by: Harry Wentland <harry.wentland@amd.com>
> ---
>  scripts/checkpatch.pl | 12 +++++++++++-
>  1 file changed, 11 insertions(+), 1 deletion(-)

Given that people are running into this problem, and that we *do* want
people using that tag, it would make sense to go ahead and get this in,
perhaps even with a CC: stable.  Would you like me to route it
Linusward, or did you have another path in mind?

Joe, does this look good to you?

Thanks,

jon