chore: migrate groups.removeModerator endpoint to new OpenAPI pattern with AJV validations

.changeset/groups.removeModerator.md

@@ -0,0 +1,5 @@
+"@rocket.chat/meteor": minor
+"@rocket.chat/rest-typings": minor
+---
+
+Add OpenAPI support for the Rocket.Chat groups.removeModerator API endpoints by migrating to a modern chained route definition syntax and utilizing shared AJV schemas for validation

apps/meteor/app/api/server/v1/groups.ts

@@ -1,7 +1,7 @@
 import { Team, isMeteorError } from '@rocket.chat/core-services';
 import type { IIntegration, IUser, IRoom, RoomType, UserStatus } from '@rocket.chat/core-typings';
 import { Integrations, Messages, Rooms, Subscriptions, Uploads, Users } from '@rocket.chat/models';
-import { isGroupsOnlineProps, isGroupsMessagesProps, isGroupsFilesProps } from '@rocket.chat/rest-typings';
+import { isGroupsOnlineProps, isGroupsMessagesProps, isGroupsFilesProps, ajv } from '@rocket.chat/rest-typings';
 import { isTruthy } from '@rocket.chat/tools';
 import { check, Match } from 'meteor/check';
 import { Meteor } from 'meteor/meteor';
@@ -36,6 +36,11 @@ import { addUserToFileObj } from '../helpers/addUserToFileObj';
 import { composeRoomWithLastMessage } from '../helpers/composeRoomWithLastMessage';
 import { getPaginationItems } from '../helpers/getPaginationItems';
 import { getUserFromParams, getUserListFromParams } from '../helpers/getUserFromParams';
+import { ExtractRoutesFromAPI } from '../ApiClass';
+import { validateBadRequestErrorResponse } from '@rocket.chat/rest-typings';
+import { validateUnauthorizedErrorResponse } from '@rocket.chat/rest-typings';
+import { withUserIdProps } from '@rocket.chat/rest-typings/dist/v1/groups/BaseProps';
+
 
 async function getRoomFromParams(params: { roomId?: string } | { roomName?: string }): Promise<IRoom> {
 	if (
@@ -891,11 +896,31 @@ API.v1.addRoute(
 	},
 );
 
-API.v1.addRoute(
-	'groups.removeModerator',
-	{ authRequired: true },
+
+
+
+
+
+
+const groupsEndpoints = API.v1.post(
+	"groups.removeModerator",
 	{
-		async post() {
+		authRequired:true,
+		body: withUserIdProps,
+		response:{
+			400:validateBadRequestErrorResponse,
+			401:validateUnauthorizedErrorResponse,
+			200:ajv.compile<void>({
+				type:'object',
+					properties:{
+						success:{type:'boolean', enum:[true]},
+					},
+					required:['success'],
+					additionalProperties:false,
+			}),
+		},
+	},
+	async function action() {
 			const findResult = await findPrivateGroupByIdOrName({
 				params: this.bodyParams,
 				userId: this.userId,
@@ -906,8 +931,8 @@ API.v1.addRoute(
 			await removeRoomModerator(this.userId, findResult.rid, user._id);
 
 			return API.v1.success();
-		},
 	},
+
 );
 
 API.v1.addRoute(
@@ -1300,3 +1325,11 @@ API.v1.addRoute(
 		},
 	},
 );
+
+
+export type GroupEndpoints = ExtractRoutesFromAPI<typeof groupsEndpoints>;
+
+declare module '@rocket.chat/rest-typings' {
+    // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-empty-interface
+	interface Endpoints extends GroupEndpoints {}
+}

packages/rest-typings/src/v1/groups/groups.ts

@@ -24,7 +24,6 @@ import type { GroupsModeratorsProps } from './GroupsModeratorsProps';
 import type { GroupsOnlineProps } from './GroupsOnlineProps';
 import type { GroupsOpenProps } from './GroupsOpenProps';
 import type { GroupsRemoveLeaderProps } from './GroupsRemoveLeaderProps';
-import type { GroupsRemoveModeratorProps } from './GroupsRemoveModeratorProps';
 import type { GroupsRemoveOwnerProps } from './GroupsRemoveOwnerProps';
 import type { GroupsRenameProps } from './GroupsRenameProps';
 import type { GroupsRolesProps } from './GroupsRolesProps';
@@ -107,9 +106,7 @@ export type GroupsEndpoints = {
 	'/v1/groups.addModerator': {
 		POST: (params: GroupsAddModeratorProps) => void;
 	};
-	'/v1/groups.removeModerator': {
-		POST: (params: GroupsRemoveModeratorProps) => void;
-	};
+
 	'/v1/groups.addOwner': {
 		POST: (params: GroupsAddOwnerProps) => void;
 	};