Repository navigation
fix: extend Dependabot coverage to Python SDK and MCP server - #2447
groupthinking with Copilot wants to merge 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the
|
🔍 PR Validation✅ Current validation passed. |
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.
Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsRe-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice. OpenSSF Scorecard
Scanned Files
|
groupthinking
left a comment
There was a problem hiding this comment.
Review status at head 8c4b9362: the new Dependabot entries and regression test are a reasonable draft scope, but this PR is still marked draft and the closing-keyword check fails. Please verify both directories have supported dependency manifests, resolve the issue-link check, and mark it ready for review.
groupthinking
left a comment
There was a problem hiding this comment.
I reviewed the configuration and focused tests at head 5bf7eade1867a87dd2a5a932f11f10253e000725. The linked issue’s corrected requirement is met: the SDK has one weekly pip entry, the MCP YouTube server has one weekly uv entry for its pyproject.toml/uv.lock, both have a 10-PR limit, and the existing root/backend entries remain. All three target manifests are present. The six focused tests, Ruff, and current hosted checks pass; there are no open review threads. I approve this code change.
Keep the PR in draft for now. Its green dependency-review job reported a 3-base/1-head snapshot warning and 56 unrelated removals on this configuration-only diff, so that artifact is not reliable dependency evidence. It should not be merged until the shared comparison is repaired, an exact-head merge decision is made, and a subsequent Dependabot run confirms the new schedules produce the intended updates.
groupthinking
left a comment
There was a problem hiding this comment.
I reviewed the current head 1f89b0b69a5bd35a5151dddded153920facc3fec after the repaired main refresh. The diff adds one weekly pip entry for /sdk/python and one weekly uv entry for the MCP YouTube server, each with a 10-PR limit; existing root/backend entries remain. The focused configuration tests previously passed, and current hosted checks are 29 successful, 12 skipped, with no failures. The dependency-review job correctly used code-only mode because this PR changes no dependency inputs, and there are no open review threads. I approve this configuration change.
The PR remains open and unmerged. A future Dependabot run is still needed to observe actual update PR creation; this approval does not establish that production behavior.
Canonical issue
Closes #2488.
Outcome
Extend weekly Python dependency updates to the previously untracked SDK and MCP YouTube server, with a 10-PR limit per location. The SDK uses
pip; the MCP server usesuvfor itspyproject.tomlanduv.lock.Scope
pipentry and one MCP serveruventry; regression coverage for unique Python entries, weekly schedules, ecosystems, and PR limits.pyproject.toml.Risk
Verification
uv run --no-project --with pytest --with pyyaml python -m pytest -q -o addopts='' tests/unit/test_dependabot_automation_workflow.py: 6 passed at head5bf7eade1867a87dd2a5a932f11f10253e000725.git diff --checkpassed.1f89b0b69a5bd35a5151dddded153920facc3fecon repairedmain: 29 hosted checks passed, 12 skipped, 0 failed. The dependency-review job succeeded in code-only mode; this PR changes configuration and tests, no dependency inputs, so it submitted no snapshot and produced no dependency artifact.main. The current dependency-review job took the expected code-only path. This check validates the absence of changed dependency inputs; it does not prove that Dependabot has created future update PRs.Production evidence
Not applicable; dependency automation configuration only. A future Dependabot run is needed to verify update PR creation.
Agent handoff
mainrefresh and current hosted checks; it remains open and unmerged pending an exact-head merge decision and a future Dependabot run.Security dependency evidence
Not applicable; no advisory-driven dependency changes.