minor fixes

Author: harshil1712

apps/account-management/package.json

@@ -1,5 +1,5 @@
 {
-	"name": "cloudflare-browser-mcp-server",
+	"name": "cloudflare-account-management-mcp-server",
 	"version": "0.0.1",
 	"private": true,
 	"scripts": {

apps/account-management/src/index.ts

@@ -61,7 +61,6 @@ export class AccountManagementMCP extends McpAgent<Env, State, Props> {
 
 		registerAccountTools(this)
 
-		// Register Cloudflare Log Push tools
 		registerAccountManagementTools(this)
 	}
 

apps/account-management/src/tools/accountManagement.ts

@@ -5,63 +5,79 @@ import { getCloudflareClient } from '@repo/mcp-common/src/cloudflare-api'
 import type { AccountManagementMCP } from '../index'
 
 export function registerAccountManagementTools(agent: AccountManagementMCP) {
-	agent.server.tool('get_members_in_account', 'Get members in account', {}, async () => {
-		const accountId = await agent.getActiveAccountId()
-		if (!accountId) {
-			return {
-				content: [
-					{
-						type: 'text',
-						text: 'No currently active accountId. Try listing your accounts (accounts_list) and then setting an active account (set_active_account)',
-					},
-				],
+	agent.server.tool(
+		'get_members_in_account',
+		'Get members in a Cloudflare account',
+		{
+			page: z.number().min(1).default(1),
+			per_page: z.number().min(1).max(100).default(20),
+		},
+		async (params) => {
+			const accountId = await agent.getActiveAccountId()
+			if (!accountId) {
+				return {
+					content: [
+						{
+							type: 'text',
+							text: 'No currently active accountId. Try listing your accounts (accounts_list) and then setting an active account (set_active_account)',
+						},
+					],
+				}
 			}
-		}
-		try {
-			const client = getCloudflareClient(agent.props.accessToken)
-			const r = await client.accounts.members.list({
-				account_id: accountId,
-			})
+			try {
+				const client = getCloudflareClient(agent.props.accessToken)
+				const r = await client.accounts.members.list({
+					account_id: accountId,
+					page: params.page,
+					per_page: params.per_page,
+				})
+
+				const { result } = r
 
-			const { result } = r
+				// From results array return the user id, name, and roles array
+				// We should not return their email as it is sensitive information
+				const members = result.map((member) => {
+					return {
+						id: member.user?.id,
+						name: member.user?.first_name + ' ' + member.user?.last_name,
+						roles: member.roles,
+						policies: member.policies,
+						status: member.status,
+					}
+				})
 
-			// From results array return the user id, name, and roles array
-			// We should not return their email as it is sensitive information
-			const members = result.map((member) => {
 				return {
-					id: member.user?.id,
-					name: member.user?.first_name + ' ' + member.user?.last_name,
-					roles: member.roles,
-					policies: member.policies,
-					status: member.status,
+					content: [
+						{
+							type: 'text',
+							text: JSON.stringify({
+								result: members,
+								result_info: r.result_info,
+								page_info: {
+									page: params.page,
+									per_page: params.per_page,
+									has_more: members.length === params.per_page,
+								},
+							}),
+						},
+					],
+				}
+			} catch (error) {
+				return {
+					content: [
+						{
+							type: 'text',
+							text: `Error getting members in account: ${error instanceof Error && error.message}`,
+						},
+					],
 				}
-			})
-
-			return {
-				content: [
-					{
-						type: 'text',
-						text: JSON.stringify({
-							result: members,
-						}),
-					},
-				],
-			}
-		} catch (error) {
-			return {
-				content: [
-					{
-						type: 'text',
-						text: `Error getting members in account: ${error instanceof Error && error.message}`,
-					},
-				],
 			}
 		}
-	})
+	)
 
 	agent.server.tool(
 		'get_roles',
-		'List all the available roles',
+		'List all the available roles in a Cloudflare account',
 		{
 			page: z.number().min(1).default(1),
 			per_page: z.number().min(1).max(100).default(20),
@@ -116,10 +132,13 @@ export function registerAccountManagementTools(agent: AccountManagementMCP) {
 
 	agent.server.tool(
 		'invite_to_account',
-		'Invite a user to an account with at least one role. The role is an array with the value set as the ID of that role.',
+		'Invite a user to a Cloudflare account with at least one role.',
 		{
 			email: z.string().email(),
-			roles: z.array(z.string()),
+			roles: z
+				.array(z.string())
+				.min(1)
+				.describe('Array of role IDs. You can get the roles by using the get_roles tool.'),
 		},
 		async (params) => {
 			const accountId = await agent.getActiveAccountId()
@@ -173,7 +192,7 @@ export function registerAccountManagementTools(agent: AccountManagementMCP) {
 
 	agent.server.tool(
 		'update_member',
-		'Update a member in an account',
+		'Update a member in a Cloudflare account',
 		{
 			member_id: z.string(),
 			roles: z.array(z.string()),
@@ -226,7 +245,7 @@ export function registerAccountManagementTools(agent: AccountManagementMCP) {
 
 	agent.server.tool(
 		'remove_member',
-		'Remove a member from an account',
+		'Remove a member from a Cloudflare account',
 		{
 			member_id: z.string(),
 		},