Enable transitive pinning and add daily driver dependency health workflow - #4712
cheenamalhotra wants to merge 4 commits into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Transitive pins violate the repository’s TFM version policy, and incomplete daily runs can create unwanted issues.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Enables transitive dependency pinning and adds daily dependency-health automation.
Changes:
- Enables central transitive pinning.
- Adds dependency graph and vulnerability auditing.
- Generates draft dependency-fix PRs with validation and deduplication.
File summaries
| File | Description |
|---|---|
Directory.Packages.props |
Enables transitive pinning. |
.github/workflows/driver-dependency-health.md |
Defines the scheduled audit workflow. |
.github/workflows/driver-dependency-health.lock.yml |
Contains the compiled executable workflow. |
Review details
- Files reviewed: 2/3 changed files
- Comments generated: 2
- Review effort level: Balanced (auto)
Note
Copilot is running an experiment and ran this review at Balanced.
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟡 Changes recommended
Incomplete audits can create unwanted issues, and Package-mode validation may not consume the locally packed siblings.
Get a fresh assessment by requesting another Copilot review.
Review details
Suppressed comments (1)
.github/workflows/driver-dependency-health.md:113
report-failure-as-issue: falsedoes not suppress the auto-enabledreport_incompletehandler. The compiled workflow confirmsGH_AW_REPORT_INCOMPLETE_CREATE_ISSUE: "true", so the restore/audit failures routed toreport_incompletebelow will create daily issues despite this comment. Configurereport-incomplete.create-issue: falseas well, then recompile the lock file.
report-incomplete:
- Files reviewed: 2/3 changed files
- Comments generated: 1
- Review effort level: Balanced (auto)
Note
Copilot is running an experiment and ran this review at Balanced.
There was a problem hiding this comment.
🔵 Needs a closer look
Transitive pinning changes published dependency floors, and the new workflow has not yet been validated in hosted execution.
Review details
- Files reviewed: 2/3 changed files
- Comments generated: 0 new
- Review effort level: Balanced (auto)
Note
Copilot is running an experiment and ran this review at Balanced.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
Transitive pinning changes published dependency floors, and the extensive agentic workflow has not yet been validated in hosted execution.
Review details
- Files reviewed: 2/3 changed files
- Comments generated: 0 new
- Review effort level: Balanced (auto)
Note
Copilot is running an experiment and ran this review at Balanced.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4712 +/- ##
==========================================
- Coverage 66.21% 64.18% -2.03%
==========================================
Files 290 285 -5
Lines 45063 68081 +23018
==========================================
+ Hits 29838 43699 +13861
- Misses 15225 24382 +9157
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Description
Enable NuGet transitive pinning and add a daily, manually dispatchable
driver-dependency-healthagentic workflow.CentralPackageTransitivePinningEnabledand replace the contradictory comment. Existing package versions are unchanged.Only
Directory.Packages.props, the workflow Markdown, and its compiled lock YAML are included. No scratch files, caches, logs, or unrelated generated changes.Transitive pinning can promote dependencies into published NuGet dependency groups and raise consumer minimum versions. This is an intentional packaging-policy change requiring review before release.
Issues
No linked public issue.
Testing
Guidelines
Operational requirements
Requires
COPILOT_GITHUB_TOKENand permission for GitHub Actions to create PRs. Maintainers must ensure required CI runs for workflow-created PRs. The daily schedule becomes active after merging to the default branch.