chore: update all mcp-worker dependencies to latest versions#558
chore: update all mcp-worker dependencies to latest versions#558jonathannorris wants to merge 4 commits intomainfrom
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
devcycle-mcp-server | 5bba4be | Mar 12 2026, 07:59 PM |
There was a problem hiding this comment.
Pull request overview
Updates the mcp-worker workspace to newer dependency versions and applies the necessary code changes to keep the worker building/running with the upgraded packages (notably Ably 2.x and workers-oauth-provider 0.3.x). It also adds a scoped Yarn resolution intended to deduplicate the MCP SDK version across the root package and the agents dependency chain.
Changes:
- Bumped
mcp-workerruntime/dev dependencies (e.g.@cloudflare/workers-oauth-provider,ably,agents,wrangler,@types/node) and updatedyarn.lockaccordingly. - Migrated Ably usage from
Rest.Promise+ webworker-specific import tonew Rest()+ top-levelablyimport, and updated unit tests/mocks. - Added a scoped Yarn
resolutionsoverride foragents/@modelcontextprotocol/sdkto prefer1.27.1.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
yarn.lock |
Lockfile updates reflecting dependency upgrades and new transitive dependency graph. |
package.json |
Adds scoped resolutions override for agents/@modelcontextprotocol/sdk. |
mcp-worker/package.json |
Updates mcp-worker dependency versions (runtime + dev). |
mcp-worker/src/index.ts |
Removes no-longer-needed TS suppression around OAuth provider wiring. |
mcp-worker/src/ably.ts |
Updates Ably import and Rest client instantiation for Ably 2.x. |
mcp-worker/src/ably.test.ts |
Updates Ably module mock/test expectations to match Ably 2.x Rest usage. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 11 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
|
should sanity test the mcp before merging, if not dont yet. code changes make sense as per the migration guide by ably |
|
yea @suthar26 ran the MCP worker locally, but will need to test it once its released. |
38070d2 to
5bba4be
Compare
Summary
Rest.Promiseremoved, webworker-specific import path dropped)@ts-expect-errorworkaround for Hono/OAuthProvider type mismatch (fixed upstream inworkers-oauth-provider0.2.4+)agents/@modelcontextprotocol/sdkresolution to deduplicate the MCP SDK (agents pins1.26.0, root uses1.27.1)aimodule via wrangler[alias]— agents 0.7.6 bundles unused features that dynamicallyimport("ai"), which breaks wrangler's esbuild bundlerworker-configuration.d.tswith wrangler 4.72.0, moves secret type declarations (AUTH0_CLIENT_ID,AUTH0_CLIENT_SECRET,ABLY_API_KEY) to a separateenv.d.tssowrangler typescan be re-run without losing themDependency changes
@cloudflare/workers-oauth-providerrevokeExistingGrantsdefault, improved TS typesablyRest.Promise->Rest, webworker import removedagentsjoseoauth4webapi@types/nodewranglerhonoandvitestwere already at latest.