Conversation
- Update @modelcontextprotocol/sdk to 1.27.1 - Update undici to 7.24.4, graphql to 16.13.1, pino to 10.3.1 - Update prisma and @prisma/client to 7.5.0 - Update test runner to 30.3.0, eslint to 10.0.3, dotenv to 17.3.1 - Update @types/node to 25.5.0, typescript-eslint to 8.57.1 - Update @semantic-release/github to 12.0.6, @semantic-release/npm to 13.1.5 - Update @cloudflare/workers-types to 4.20260317.1 - Remove unused http-proxy-agent, https-proxy-agent, socks-proxy-agent - Remove obsolete proxy-agent mocks from fetch tests Closes #354
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThis pull request updates npm package versions across runtime and development dependencies, removes three unused proxy-agent packages, and updates corresponding test mocks. Tests pass and builds complete successfully. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
📝 Coding Plan
Comment |
Test Coverage ReportOverall Coverage: 95.86%
|
There was a problem hiding this comment.
Pull request overview
Updates the GitLab MCP server’s dependency set to current versions, while removing unused direct proxy-agent dependencies and simplifying the related unit-test mocks.
Changes:
- Bump key runtime/dev dependencies (e.g.,
@modelcontextprotocol/sdk,undici,graphql,prisma,eslint,jest,@types/node). - Remove unused direct dependencies:
http-proxy-agent,https-proxy-agent,socks-proxy-agent. - Clean up fetch unit tests by dropping obsolete mocks for removed proxy-agent packages.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
package.json |
Updates dependency versions and removes unused proxy-agent direct deps. |
yarn.lock |
Lockfile refresh reflecting updated versions and dependency graph changes. |
tests/unit/utils/fetch.test.ts |
Removes mocks for proxy-agent packages that are no longer installed directly. |
You can also share your feedback on Copilot code review. Take the survey.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
Key Updates
Test Plan
yarn build— cleanyarn lint— 0 errorsyarn test— 4676/4676 passed (146 suites)Closes #354
Summary by CodeRabbit
Chores
Tests