Skip to content

docs(benchmarks): record decimal.js-light comparison - #1432

Merged
mstykow merged 1 commit into
mainfrom
verify/decimal-js-light
Sep 9, 2026
Merged

docs(benchmarks): record decimal.js-light comparison#1432
mstykow merged 1 commit into
mainfrom
verify/decimal-js-light

Conversation

@mstykow

@mstykow mstykow commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Record a benchmark-grade common comparison for MikeMcl/decimal.js-light at immutable revision b2b0c28e04e4f96d21450d2075ea36064fc9e22d.
  • Document matched package, license, copyright, author, email, and file coverage, plus source-backed Provenant advantages in holder and URL extraction.
  • Regenerate the benchmark chart and aggregate headline for 255 recorded runs.

Compare artifacts: .provenant/compare-runs/20260909T081928Z-decimal.js-light-13251/

Issues

Scope and exclusions

  • Included: benchmark evidence and generated benchmark documentation.
  • Explicit exclusions: no scanner changes were needed because the comparison had zero ScanCode-favored signals.

How to verify

  • Run cargo run --manifest-path xtask/Cargo.toml --bin compare-outputs -- --repo-url https://github.com/MikeMcl/decimal.js-light.git --repo-ref b2b0c28e04e4f96d21450d2075ea36064fc9e22d --profile common; confirm 46 common file paths, zero ScanCode-favored signals, and the documented timing/advantages.

Intentional differences from Python

  • Provenant reports the two explicit Markdown image targets ScanCode omits, retains a valid repeated decimal.js documentation link, and carries the manifest author into the assembled npm package holder.

Follow-up work

  • Created or intentionally deferred: None; the target reached the verification exit condition without scanner changes.

Expected-output fixture changes

  • Files changed: None.
  • Why the new expected output is correct: No scanner output changed; this PR records the verified comparison and regenerates its chart.

🤖 Generated with Claude Code

Signed-off-by: Maxim Stykow <maxim.stykow@gmail.com>
@greptile-apps

greptile-apps Bot commented Sep 9, 2026

Copy link
Copy Markdown

Greptile Summary

Records the benchmark comparison against decimal.js-light at a pinned revision and regenerates the aggregate benchmark presentation.

  • Adds the 46-file comparison with machine context, timings, matched outputs, and Provenant-specific advantages.
  • Updates the headline statistics from 254 to 255 recorded runs.
  • Adds matching Provenant and ScanCode points to the generated SVG chart.

Confidence Score: 5/5

The documentation-only PR appears safe to merge with no actionable correctness or repository-rule issues identified.

The benchmark entry follows the established record format, and its timing data, calculated speedup, chart points, tooltips, ordering, and updated run count are internally consistent.

Important Files Changed

Filename Overview
docs/BENCHMARKS.md Adds a complete decimal.js-light benchmark record and updates aggregate statistics consistently with the new run.
docs/scan-duration-vs-files.svg Adds chart points and tooltips whose file count and timings match the documented benchmark.

Reviews (1): Last reviewed commit: "docs(benchmarks): record decimal.js-ligh..." | Re-trigger Greptile

@mstykow
mstykow merged commit fe9429f into main Sep 9, 2026
13 checks passed
@mstykow
mstykow deleted the verify/decimal-js-light branch September 9, 2026 08:37
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