Address comments.

Author: axiapubsub

apps/dns-analytics/CONTRIBUTING.md

@@ -0,0 +1,66 @@
+# Setup
+
+If you'd like to iterate and test your MCP server, you can do so in local development.
+
+## Local Development
+
+1. Create a `.dev.vars` file in your project root:
+
+   If you're a Cloudflare employee:
+
+   ```
+   CLOUDFLARE_CLIENT_ID=your_development_cloudflare_client_id
+   CLOUDFLARE_CLIENT_SECRET=your_development_cloudflare_client_secret
+   DEV_CLOUDFLARE_API_TOKEN=your_development_api_token
+   ```
+
+   If you're an external contributor, you can provide a development API token (See [Cloudflare API](https://developers.cloudflare.com/api/) for information on creating an API Token):
+
+   ```
+   DEV_DISABLE_OAUTH=true
+   DEV_CLOUDFLARE_EMAIL=your_cloudflare_email
+   # This is your api token with endpoint access.
+   DEV_CLOUDFLARE_API_TOKEN=your_development_api_token
+   ```
+
+2. Start the local development server:
+
+   ```bash
+   npx wrangler dev
+   ```
+
+3. To test locally, open Inspector, and connect to `http://localhost:8976/sse`.
+   Once you follow the prompts, you'll be able to "List Tools". You can also connect with any MCP client.
+
+## Deploying the Worker ( Cloudflare employees only )
+
+Set secrets via Wrangler:
+
+```bash
+npx wrangler secret put CLOUDFLARE_CLIENT_ID -e <ENVIRONMENT>
+npx wrangler secret put CLOUDFLARE_CLIENT_SECRET -e <ENVIRONMENT>
+```
+
+## Set up a KV namespace
+
+Create the KV namespace:
+
+```bash
+npx wrangler kv namespace create "OAUTH_KV"
+```
+
+Then, update the Wrangler file with the generated KV namespace ID.
+
+## Deploy & Test
+
+Deploy the MCP server to make it available on your workers.dev domain:
+
+```bash
+npx wrangler deploy -e <ENVIRONMENT>
+```
+
+Test the remote server using [Inspector](https://modelcontextprotocol.io/docs/tools/inspector):
+
+```bash
+npx @modelcontextprotocol/inspector@latest
+```

apps/dns-analytics/README.md

@@ -1,4 +1,4 @@
-# Cloudflare Radar MCP Server 📡
+# Cloudflare DNS Analytics MCP Server 📡
 
 This is a [Model Context Protocol (MCP)](https://modelcontextprotocol.io/introduction) server that supports remote MCP
 connections, with Cloudflare OAuth built-in.

apps/dns-analytics/src/tools/analytics.ts

@@ -18,12 +18,6 @@ function getStartDate(days: number) {
 	return start_date.toISOString()
 }
 
-/**
- * Registers the dns analytic tool with the MCP server
- * @param server The MCP server instance
- * @param accountId Cloudflare account ID
- * @param apiToken Cloudflare API token
- */
 export function registerAnalyticTools(agent: DNSAnalyticsMCP) {
 	// Register DNS Report tool
 	agent.server.tool(
@@ -35,10 +29,8 @@ export function registerAnalyticTools(agent: DNSAnalyticsMCP) {
 		},
 		async ({ zone, days }) => {
 			try {
-				console.log('fetching DNS record')
 				const client = getCloudflareClient(env.DEV_CLOUDFLARE_API_TOKEN)
 				const start_date = getStartDate(days)
-				console.log(start_date)
 				const params: ReportGetParams = {
 					zone_id: zone,
 					metrics: 'responseTimeAvg,queryCount,uncachedCount,staleCount',
@@ -74,7 +66,6 @@ export function registerAnalyticTools(agent: DNSAnalyticsMCP) {
 		'Show DNS settings for current account',
 		async () => {
 			try {
-				console.log('Show Account DNS settings')
 				const accountId = await agent.getActiveAccountId()
 				if (!accountId) {
 					return {
@@ -122,7 +113,6 @@ export function registerAnalyticTools(agent: DNSAnalyticsMCP) {
 		},
 		async ({ zone }) => {
 			try {
-				console.log('Show Zone DNS settings')
 				const client = getCloudflareClient(env.DEV_CLOUDFLARE_API_TOKEN)
 				const params: ZoneGetParams = {
 					zone_id: zone,
@@ -157,7 +147,6 @@ export function registerAnalyticTools(agent: DNSAnalyticsMCP) {
 		'List zones under the current active account',
 		async () => {
 			try {
-				console.log('List zones under the current active account')
 				const client = getCloudflareClient(env.DEV_CLOUDFLARE_API_TOKEN)
 				const accountId = await agent.getActiveAccountId()
 				if (!accountId) {

apps/dns-analytics/wrangler.jsonc

@@ -10,7 +10,7 @@
 	"name": "mcp-cloudflare-dns-analytics-dev",
 	"migrations": [
 		{
-			"new_sqlite_classes": ["UserDetails", "DNSAnalyticsMCP"],
+			"new_sqlite_classes": ["DNSAnalyticsMCP"],
 			"tag": "v1"
 		}
 	],
@@ -25,7 +25,7 @@
 			},
 			{
 				"class_name": "UserDetails",
-				"name": "USER_DETAILS"
+				"name": "USER_DETAILS",
 			}
 		]
 	},
@@ -66,7 +66,8 @@
 					},
 					{
 						"class_name": "UserDetails",
-						"name": "USER_DETAILS"
+						"name": "USER_DETAILS",
+						"script_name": "mcp-cloudflare-workers-observability-staging"
 					}
 				]
 			},
@@ -100,7 +101,8 @@
 					},
 					{
 						"class_name": "UserDetails",
-						"name": "USER_DETAILS"
+						"name": "USER_DETAILS",
+						"script_name": "mcp-cloudflare-workers-observability-production"
 					}
 				]
 			},