test(lockfile): add property roundtrip coverage#376
Conversation
Greptile SummaryAdds Confidence Score: 5/5Safe to merge — changes are limited to new test coverage and dev-dependency additions with no production logic modifications. No P0 or P1 issues found. The single P2 observation (declared_dependencies not verified in normalization) is a test coverage gap, not a correctness defect. Cargo.lock additions are all reputable crates.io packages used only for testing. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "test(lockfile): add property roundtrip c..." | Re-trigger Greptile |
962bd46 to
499374f
Compare
Summary
Adds property-based lockfile roundtrip coverage for generated registry-only graphs across pnpm, npm, Bun, Yarn classic, and Yarn Berry lockfile writers/parsers. The tests normalize dependency edges before comparing so they focus on structural graph preservation across each format.
Validation
This PR was generated by Codex.
Note
Low Risk
Test-only changes plus a new dev-dependency; no production code paths are modified.
Overview
Adds property-based roundtrip tests in
aube-lockfilethat generate small registry-only dependency graphs, write them as each supported lockfile format (pnpm/npm/bun/yarn classic/yarn berry), re-parse them, and compare a normalized representation to ensure structural preservation.Introduces
proptestas a dev-dependency (and updatesCargo.lock) to support this new randomized/parametric test coverage.Reviewed by Cursor Bugbot for commit 499374f. Bugbot is set up for automated code reviews on this repo. Configure here.