Repository navigation
test: verify exact uv lock delta under repaired dependency review - #2499
Closed
groupthinking wants to merge 4 commits into
Closed
groupthinking wants to merge 4 commits into
groupthinking wants to merge 4 commits into
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
🔍 PR Validation |
Contributor
Dependency ReviewThe following issues were found:
License Issuesuv.lock
OpenSSF ScorecardScorecard details
Scanned Files
|
2 of 5 tasks
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Completed exact-lock dependency diagnostic — closed without merge
Refs #2486, #2492, #2498, and issue #2490. This temporary off-main PR compared only the exact
uv.lockbytes from #2486 under the proposed dependency-review and locked-Python-CI repairs. It was closed unmerged and both temporary branches were deleted after verification. It does not prove snapshot parity against currentmainor clear #2486's existing red check.The final temporary base
3db6bc716e3fe8e7c03e9cf19bfd039aef89e643combined #2492's full-tree and scoped uv snapshot workflow with #2498's locked Python CI, plus a temporary push trigger/test used only for this diagnostic. Itsuv.lockwas byte-identical tomainatba2777d1b008c2148d413e07cc497791966310e0. The final head9ac578ea17c47b6109f5cd7cc271d222fb3dc0fddiffered from that base only in rootuv.lock, byte-identical to #2486 headb9c38d461c2cd8bccbb60024ab427fa2965d3589. The delta comprised 31 version replacements and three newly added transitive packages.Result
dev-only packagesblack,notebook, andpyjwtwere among those mislabeled, so that green check was not a reliable verdict. The upstream uv detector recognizes virtual project roots; EventRelay's root usessource.editable, leaving its development reachability unclassified. The correction in draft fix: match dependency snapshots on update pull requests #2492 submits a scopeduv.locksnapshot and checks the full GitHub diff against the exact lockfiles.review_outcome: successartifact contains 65uv.lockrows: 34 added and 31 removed. All package names, versions, manifests, and scopes match the exact lockfiles, including 11 development-only changed package names. Independent GitHub API readback had an empty snapshot-warning header and byte-identical diff (SHA-2563b645b716661271d36003b1850827cb09510a7610830eeb1ddd346b58b51fa5b). No vulnerabilities were reported;ddtrace-internalhas unknown license metadata.review_outcome: failure. The unchanged exact commit passed on attempt 2 after both base snapshots were available.uv lock --check --offline, Ruff, and diff check passed. The earlier exact-lock local full unit selection passed 2,960 tests plus 118 subtests on each Python version with two pre-existing macOS-specific tests excluded; the offline MCP selection passed 38 tests plus 20 subtests on each.Scope and cleanup
This is an off-main confirmation of snapshot scope and artifact accuracy. The original dependency PRs still need their bases refreshed and their hosted checks rerun after the shared repairs are merged. No approval or merge of #2486, #2487, #2492, or #2498 is claimed by this diagnostic. #2499 is closed unmerged;
codex/uv-lock-parity-canary-baseandcodex/uv-lock-parity-canary-headwere deleted. The exact hosted result and remaining main-based gate are recorded on draft #2492.