Merge pull request #4 from frankmeszaros/fm/v0

Fix Metrics Server Init

Author: frankmeszaros

apps/cloudflare-one-casb/src/index.ts

@@ -28,12 +28,29 @@ export type Props = {
 
 export type State = { activeAccountId: string | null }
 export class CASBMCP extends McpAgent<Env, State, Props> {
-	server = new CloudflareMCPServer(undefined, env.MCP_METRICS, {
-		name: env.MCP_SERVER_NAME,
-		version: env.MCP_SERVER_VERSION,
-	})
+	_server: CloudflareMCPServer | undefined
+	set server(server: CloudflareMCPServer) {
+		this._server = server
+	}
+
+	get server(): CloudflareMCPServer {
+		if (!this._server) {
+			throw new Error('Tried to access server before it was initialized')
+		}
+
+		return this._server
+	}
+
+	constructor(ctx: DurableObjectState, env: Env) {
+		super(ctx, env)
+	}
 
 	async init() {
+		this.server = new CloudflareMCPServer(this.props.user.id, this.env.MCP_METRICS, {
+			name: this.env.MCP_SERVER_NAME,
+			version: this.env.MCP_SERVER_VERSION,
+		})
+
 		registerAccountTools(this as unknown as CloudflareMcpAgent)
 		registerIntegrationsTools(this)
 	}

packages/mcp-common/src/api/account.ts

@@ -1,6 +1,6 @@
-import type { McpAgent } from 'agents/mcp'
 import type { Cloudflare } from 'cloudflare'
 import type { Account } from 'cloudflare/resources/accounts/accounts.mjs'
+import type { CloudflareMcpAgent } from '../types/cloudflare-mcp-agent'
 import type { ToolHandler } from '../types/tools'
 
 export async function handleAccountsList({ client }: { client: Cloudflare }): Promise<Account[]> {
@@ -9,12 +9,8 @@ export async function handleAccountsList({ client }: { client: Cloudflare }): Pr
 	return response.result
 }
 
-export interface McpAgentWithAccount extends McpAgent {
-	getActiveAccountId: () => string
-}
-
 export const withAccountCheck = <T extends Record<string, any>>(
-	agent: McpAgentWithAccount,
+	agent: CloudflareMcpAgent,
 	handler: ToolHandler<T>
 ) => {
 	return async (params: T) => {