Skip to content

fix(copyright): recognize mixed POD author headings - #1420

Merged
mstykow merged 1 commit into
fix/source-comment-action-creditsfrom
fix/mixed-pod-author-headings
Sep 3, 2026
Merged

fix(copyright): recognize mixed POD author headings#1420
mstykow merged 1 commit into
fix/source-comment-action-creditsfrom
fix/mixed-pod-author-headings

Conversation

@mstykow

@mstykow mstykow commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Treat bounded POD headings that combine AUTHOR(S) with copyright, license, maintenance, or modification-history topics as author sections.
  • Accept common separator and casing variants without treating copyright-only or unrelated mixed headings as author evidence.
  • Keep the existing bounded section termination rules intact.

Issues

Scope and exclusions

  • Included: POD author-section heading classification.
  • Explicit exclusions: maintainer credits outside bounded author sections, handled separately.

How to verify

  • Scan Perl 5.38.4 with --copyright; pod/perlopentut.pod, pod/perlretut.pod, and cpan/libnet/lib/Net/libnetFAQ.pod should recover their maintainer credits.
  • Before/after corpus scans add exactly those three valid findings; Info-ZIP, musl, Dancer2, and Valgrind author output is unchanged, and copyright/holder output is unchanged across all five targets.

Intentional differences from Python

  • Provenant recognizes the section structurally and keeps richer contact data where present instead of relying on ScanCode rendering.

Follow-up work

  • Created or intentionally deferred: explicit collective maintainer credits outside author sections remain for the next stacked PR.

Expected-output fixture changes

  • Files changed: none.
  • Why the new expected output is correct: focused unit coverage verifies accepted and rejected heading shapes plus bounded extraction behavior.

Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
@mstykow
mstykow force-pushed the fix/mixed-pod-author-headings branch from 01dd200 to 481b3e4 Compare September 3, 2026 11:06
@mstykow

mstykow commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

@greptileai review

@mstykow
mstykow merged commit 87c752d into main Sep 3, 2026
12 checks passed
@mstykow
mstykow deleted the fix/mixed-pod-author-headings branch September 3, 2026 11:30
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