Skip to content

fix(copyright): recover source header author credits - #1414

Merged
mstykow merged 1 commit into
fix/author-candidate-boundariesfrom
fix/source-header-credits
Sep 3, 2026
Merged

fix(copyright): recover source header author credits#1414
mstykow merged 1 commit into
fix/author-candidate-boundariesfrom
fix/source-header-credits

Conversation

@mstykow

@mstykow mstykow commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • recover contact-backed Original by and artifact-header credits such as hints by
  • join bounded ... by: and wrapped action-attribution headers without absorbing following prose
  • preserve multiple chained attributions on one source-header line

Issues

Scope and exclusions

  • Included: generic, contact-backed source-header attribution forms and their continuation boundaries
  • Explicit exclusions: unconstrained prose by phrases and ScanCode-only render differences

How to verify

  • Scan Perl 5.38.4 with --copyright: Kaveh Ghazi, Andy Dougherty, Neil Bowers, and Luther Huffman are newly recovered; David Romano is upgraded to include the declared contact.
  • Scan InfoZIP 3.0 with --copyright: author, copyright, and holder output is unchanged.

Intentional differences from Python

  • ScanCode output is evidence, not the target contract. This keeps Provenant's cleaner existing renderings and rejects ScanCode false positives while recovering source-backed identities ScanCode itself misses, including Luther Huffman.

Expected-output fixture changes

  • Files changed: testdata/copyright-golden/copyrights/misco4/linux-copyrights/net/dccp/ccids/lib/packet_history.c.yml
  • Why the new expected output is correct: the fixture explicitly says the DCCP stack changes were written by Arnaldo Carvalho de Melo; the bounded wrapped-attribution logic now recovers that author.

Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
@mstykow
mstykow force-pushed the fix/source-header-credits branch 2 times, most recently from fdfdea6 to 47348ac Compare September 3, 2026 10:24
@mstykow

mstykow commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

@greptileai review

@mstykow
mstykow merged commit 32d6c6b into main Sep 3, 2026
12 checks passed
@mstykow
mstykow deleted the fix/source-header-credits branch September 3, 2026 11:04
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