Skip to content

fix: verify dotnet/runtime benchmark target#821

Merged
mstykow merged 8 commits intomainfrom
verify/dotnet-runtime-benchmark
Apr 29, 2026
Merged

fix: verify dotnet/runtime benchmark target#821
mstykow merged 8 commits intomainfrom
verify/dotnet-runtime-benchmark

Conversation

@mstykow
Copy link
Copy Markdown
Owner

@mstykow mstykow commented Apr 29, 2026

Summary

  • Fix npm sibling assembly so package-lock.json dependencies survive when sibling npm manifests omit full identity but still clearly refer to the same package name.
  • Recognize zlib-style configure headers as autotools package surfaces and cover the parser change with focused unit and golden tests.
  • Add a follow-up copyright/holder/author cleanup pass for dotnet/runtime that drops XML/feed element-value noise, version-info and DTD metadata spans, and code-like member-access / unicode-escape junk while preserving legitimate markup author attributes.
  • Re-verify dotnet/runtime @ d1163e5 with compare-outputs --profile common, updating the final evidence path to .provenant/compare-runs/20260429T185606Z-runtime-45960 and reducing runtime junk higher-count deltas from authors 120 -> 96, holders 65 -> 47, and copyrights 56 -> 37 versus the original benchmark checkpoint run.

Scope and exclusions

  • Included: one compare-outputs --profile common verification lane for https://github.com/dotnet/runtime.git at d1163e5a8f3f3aaa374993e8b5805911689aba28, targeted assembly/parser/copyright-detector fixes, and the resulting benchmark documentation refresh already on this branch.
  • Explicit exclusions: no scorecard row edits because .NET / NuGet is already verified and this PR remains a benchmark checkpoint plus general scanner-quality fixes; no changes for the remaining reviewed compare tail where ScanCode times out on EncryptedXmlSample4.xml or reports extra clue/binary-license noise (google-cla on brotli/CONTRIBUTING.md, NGPL on the wasm .dat assets).

Expected-output fixture changes

  • Files changed: testdata/autotools/zlib-ng/configure.expected.json
  • Why the new expected output is correct: the autotools parser now intentionally recognizes generated configure scripts that advertise themselves with a # configure script for ... header, so the new golden locks in the corrected pkg:autotools/zlib-ng package identity for that parser surface.

@mstykow mstykow force-pushed the verify/dotnet-runtime-benchmark branch from f4f3284 to e9acd91 Compare April 29, 2026 19:14
@mstykow mstykow enabled auto-merge (rebase) April 29, 2026 19:15
mstykow and others added 8 commits April 29, 2026 22:29
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
@mstykow mstykow force-pushed the verify/dotnet-runtime-benchmark branch from e9acd91 to c518471 Compare April 29, 2026 20:32
@mstykow mstykow merged commit fa5d345 into main Apr 29, 2026
15 checks passed
@mstykow mstykow deleted the verify/dotnet-runtime-benchmark branch April 29, 2026 20:44
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