chore(cloudflare): add /docs worker (poc) #545
Conversation
dcramer
commented
Sep 8, 2025
dcramer
commented
- Update Cloudflare docs: deployment and overview
- Add turbo config for mcp-cloudflare
- Update wrangler and vite configs
- Adjust Cloudflare server types and index
- Update MCP prompts and resources definitions
- Refresh deploy and smoke-test workflows
- Update package manifests and lockfile
- Add docs package scaffold
- Update smoke tests
…ions - Update Cloudflare docs: deployment and overview - Add turbo config for mcp-cloudflare - Update wrangler and vite configs - Adjust Cloudflare server types and index - Update MCP prompts and resources definitions - Refresh deploy and smoke-test workflows - Update package manifests and lockfile - Add docs package scaffold - Update smoke tests Co-Authored-By: Codex CLI Agent <[email protected]>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| }, | ||
| }; | ||
|
|
||
| export default handler; |
There was a problem hiding this comment.
Bug: Error Handling Flaw Masks Service Issues
The try-catch block for /docs requests catches errors from the docs service proxy but then falls through to the baseHandler. This causes requests to /docs to return the main application's response when the docs service is unavailable, masking service issues and leading to confusing user experiences.
|
im not convinced this is the simplest path forward. I took this prototype much farther and honestly it gets super complex. hyp[othetically it makes a lot of sense to abstract things into diff workers, but in this case, for example, we probably would want ALL UI (at minimum) to be in one worker. we could easily abstract the mcp service into its own worker, which actually would be of some value, but even that creates a bunch of packaging complexity. i will probably hold on this and reconsider later. |
