Michael Hart

Results 319 comments of Michael Hart

I wonder if we should keep the existing vectorize one and just create a new ai search one (but still use it to replace the live docs MCP server). So...

Also: do the evals take into account latency? That's one aspect that we should be sure to verify. We don't want it to be orders of magnitude slower.

This should be moved to the new stateless architecture (docs server doesn't have auth or use state). At least for the `/mcp` endpoint anyway. Can keep the DO for the...

Also will need to update these: https://github.com/cloudflare/mcp-server-cloudflare/blob/0df3a673735cc1cf0b7ab26855c2c7ec6c5ae2e4/apps/workers-bindings/src/bindings.app.ts#L17 https://github.com/cloudflare/mcp-server-cloudflare/blob/0df3a673735cc1cf0b7ab26855c2c7ec6c5ae2e4/apps/workers-observability/src/workers-observability.app.ts#L17

A couple of minor things, but happy after that! Just want to make sure before you merge and things cut over that you've got it running on staging, etc. Would...

> Also will need to update these: > > https://github.com/cloudflare/mcp-server-cloudflare/blob/0df3a673735cc1cf0b7ab26855c2c7ec6c5ae2e4/apps/workers-bindings/src/bindings.app.ts#L17 > > https://github.com/cloudflare/mcp-server-cloudflare/blob/0df3a673735cc1cf0b7ab26855c2c7ec6c5ae2e4/apps/workers-observability/src/workers-observability.app.ts#L17 Oh wait, these haven't been done?

Sounds like you're proxying a request? Normally you'd be constructing a request from scratch to call an API. What's the use case here out of interest? (just to set expectations...

Would also be nice to bump the `jose` dep too because of this line in v4: https://github.com/panva/jose/blob/051a18e13c9df98d0cfb7d9a7cca7f3746f23526/src/runtime/node/flags.ts#L1 My reading of the `jose` changelog suggests that v5 still supports Node.js 18,...

Btw, you can define these variables if you're using Cloudflare Workers. Add this (or similar) to your `wrangler.jsonc`: ```json "define": { "process.version": "\"v22.14.0\"", "process.versions": "{ \"node\": \"22.14.0\" }" }, ```...