Skip to content

Conversation

@cf-ml
Copy link
Contributor

@cf-ml cf-ml commented Apr 23, 2025

WIP: Would like a review by the team with feedback on any structural issues.

@@ -0,0 +1,47 @@
import { fetchCloudflareApi } from '@repo/mcp-common/src/cloudflare-api'

export const fetchDexTestAnalyzation = async ({
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we able to use the cloudflare-typescript sdk here for DEX endpoints? Example here

@Maximo-Guk
Copy link
Member

Confused why CI didn't run on this PR, wonder if it's because it came from a fork?

@cmsparks
Copy link
Contributor

cmsparks commented Apr 25, 2025

Confused why CI didn't run on this PR, wonder if it's because it came from a fork?

Yeah, I enabled that setting. We have to manually run CI on forks because the evals job has some secrets.

@Maximo-Guk
Copy link
Member

Maximo-Guk commented Apr 25, 2025

Confused why CI didn't run on this PR, wonder if it's because it came from a fork?

Yeah, I enabled that setting. We have to manually run CI on forks because the evals job has some secrets.

Makes sense, although I remember on wrangler-action we had the ability to selectively run workflows on certain PR's
image
but don't see that option on this PR
image

export type State = { activeAccountId: string | null }

export class MyMCP extends McpAgent<Env, State, Props> {
server = new McpServer({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In order to get metrics/error tracking on all of our MCP servers, I added a utility class in a recent PR here: #69

I’ll write up a quick guide on how to use this tomorrow

Doesn’t need to be this PR, but we should use that McpServer wrapper before deploying this

@cf-ml cf-ml closed this Apr 29, 2025
@cf-ml cf-ml mentioned this pull request Apr 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants