Merge branch 'develop' into feat/openapi-oauth-apps-update

Author: ahmed-n-abdeltwab

.changeset/strange-worms-smoke.md

@@ -0,0 +1,6 @@
+---
+"@rocket.chat/meteor": patch
+"@rocket.chat/rest-typings": patch
+---
+
+Add OpenAPI support for the Rocket.Chat oauth-apps.list API endpoints by migrating to a modern chained route definition syntax and utilizing shared AJV schemas for validation to enhance API documentation and ensure type safety through response validation.

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

@@ -17,16 +17,49 @@ import { updateOAuthApp } from '../../../oauth2-server-config/server/admin/metho
 import type { ExtractRoutesFromAPI } from '../ApiClass';
 import { API } from '../api';
 
-API.v1.addRoute(
+const oauthAppsListEndpoints = API.v1.get(
 	'oauth-apps.list',
-	{ authRequired: true, permissionsRequired: ['manage-oauth-apps'] },
 	{
-		async get() {
-			return API.v1.success({
-				oauthApps: await OAuthApps.find().toArray(),
-			});
+		authRequired: true,
+		query: ajv.compile<{ uid?: string }>({
+			type: 'object',
+			properties: {
+				uid: {
+					type: 'string',
+				},
+			},
+			additionalProperties: false,
+		}),
+		permissionsRequired: ['manage-oauth-apps'],
+		response: {
+			400: validateBadRequestErrorResponse,
+			401: validateUnauthorizedErrorResponse,
+			403: validateForbiddenErrorResponse,
+			200: ajv.compile<{ oauthApps: IOAuthApps[] }>({
+				type: 'object',
+				properties: {
+					oauthApps: {
+						type: 'array',
+						items: {
+							$ref: '#/components/schemas/IOAuthApps',
+						},
+					},
+					success: {
+						type: 'boolean',
+						enum: [true],
+					},
+				},
+				required: ['oauthApps', 'success'],
+				additionalProperties: false,
+			}),
 		},
 	},
+
+	async function action() {
+		return API.v1.success({
+			oauthApps: await OAuthApps.find().toArray(),
+		});
+	},
 );
 
 API.v1.addRoute(
@@ -199,11 +232,16 @@ type OauthAppsCreateEndpoints = ExtractRoutesFromAPI<typeof oauthAppsCreateEndpo
 
 type OauthAppsuUpdateEndpoints = ExtractRoutesFromAPI<typeof oauthAppsuUpdateEndpoints>;
 
-export type OAuthAppsEndpoints = OauthAppsCreateEndpoints | OauthAppsuUpdateEndpoints;
+type OauthAppsListEndpoints = ExtractRoutesFromAPI<typeof oauthAppsListEndpoints>;
+
+export type OAuthAppsEndpoints = OauthAppsCreateEndpoints | OauthAppsListEndpoints | OauthAppsuUpdateEndpoints;
+
 
 declare module '@rocket.chat/rest-typings' {
 	// eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-empty-interface
 	interface Endpoints extends OauthAppsCreateEndpoints {}
 	// eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-empty-interface
 	interface Endpoints extends OauthAppsuUpdateEndpoints {}
+  // eslint-disable-next-line @typescript-eslint/naming-convention, @typescript-eslint/no-empty-interface
+	interface Endpoints extends OauthAppsListEndpoints {}
 }

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

@@ -1,15 +1,9 @@
-import type { IOAuthApps, IUser } from '@rocket.chat/core-typings';
+import type { IOAuthApps } from '@rocket.chat/core-typings';
 
 import type { DeleteOAuthAppParams } from './oauthapps/DeleteOAuthAppParamsDELETE';
 import type { OauthAppsGetParams } from './oauthapps/OAuthAppsGetParamsGET';
 
 export type OAuthAppsEndpoint = {
-	'/v1/oauth-apps.list': {
-		GET: (params: { uid: IUser['_id'] }) => {
-			oauthApps: IOAuthApps[];
-		};
-	};
-
 	'/v1/oauth-apps.get': {
 		GET: (params: OauthAppsGetParams) => {
 			oauthApp: IOAuthApps;