Skip to content

Commit b5a17fc

Browse files
authored
chore: fix AJV strictTypes warnings in rest-typings oneOf branches (RocketChat#40282)
1 parent a8244d4 commit b5a17fc

11 files changed

Lines changed: 46 additions & 12 deletions

File tree

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ const invites = API.v1
7070
authRequired: true,
7171
response: {
7272
200: ajv.compile<IInvite[]>({
73-
additionalProperties: false,
7473
type: 'array',
7574
items: {
7675
$ref: '#/components/schemas/IInvite',

packages/rest-typings/src/v1/channels/ChannelsFilesListProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,10 @@ const channelsFilesListPropsSchema = {
4949
type: 'boolean',
5050
},
5151
},
52-
oneOf: [{ required: ['roomId'] }, { required: ['roomName'] }],
52+
oneOf: [
53+
{ type: 'object', required: ['roomId'] },
54+
{ type: 'object', required: ['roomName'] },
55+
],
5356
required: [],
5457
additionalProperties: false,
5558
};

packages/rest-typings/src/v1/dm/DmFileProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,10 @@ const dmFilesListPropsSchema = {
4949
type: 'boolean',
5050
},
5151
},
52-
oneOf: [{ required: ['roomId'] }, { required: ['username'] }],
52+
oneOf: [
53+
{ type: 'object', required: ['roomId'] },
54+
{ type: 'object', required: ['username'] },
55+
],
5356
required: [],
5457
additionalProperties: false,
5558
};

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,10 @@ const GroupsFilesPropsSchema = {
4747
type: 'boolean',
4848
},
4949
},
50-
oneOf: [{ required: ['roomId'] }, { required: ['roomName'] }],
50+
oneOf: [
51+
{ type: 'object', required: ['roomId'] },
52+
{ type: 'object', required: ['roomName'] },
53+
],
5154
required: [],
5255
additionalProperties: true, // keep additional properties for backwards compatibility, otherwise this would be a breaking change
5356
};

packages/rest-typings/src/v1/moderation/ArchiveReportProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,10 @@ const archiveReportPropsSchema = {
2727
nullable: true,
2828
},
2929
},
30-
oneOf: [{ required: ['msgId'] }, { required: ['userId'] }],
30+
oneOf: [
31+
{ type: 'object', required: ['msgId'] },
32+
{ type: 'object', required: ['userId'] },
33+
],
3134
additionalProperties: false,
3235
};
3336

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

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,10 @@ const RoomsIsMemberPropsSchema = {
499499
userId: { type: 'string', minLength: 1 },
500500
username: { type: 'string', minLength: 1 },
501501
},
502-
oneOf: [{ required: ['roomId', 'userId'] }, { required: ['roomId', 'username'] }],
502+
oneOf: [
503+
{ type: 'object', required: ['roomId', 'userId'] },
504+
{ type: 'object', required: ['roomId', 'username'] },
505+
],
503506
additionalProperties: false,
504507
};
505508

@@ -770,6 +773,7 @@ type MembersOrderedByRoleProps = {
770773
export type RoomsMembersOrderedByRoleProps = PaginatedRequest<MembersOrderedByRoleProps>;
771774

772775
const membersOrderedByRoleRolePropsSchema = {
776+
type: 'object',
773777
properties: {
774778
roomId: {
775779
type: 'string',
@@ -796,7 +800,10 @@ const membersOrderedByRoleRolePropsSchema = {
796800
type: 'string',
797801
},
798802
},
799-
oneOf: [{ required: ['roomId'] }, { required: ['roomName'] }],
803+
oneOf: [
804+
{ type: 'object', required: ['roomId'] },
805+
{ type: 'object', required: ['roomName'] },
806+
],
800807
additionalProperties: false,
801808
};
802809

packages/rest-typings/src/v1/teams/TeamsInfoProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,10 @@ const teamsInfoPropsSchema = {
88
teamId: { type: 'string' },
99
teamName: { type: 'string' },
1010
},
11-
oneOf: [{ required: ['teamId'] }, { required: ['teamName'] }],
11+
oneOf: [
12+
{ type: 'object', required: ['teamId'] },
13+
{ type: 'object', required: ['teamName'] },
14+
],
1215
additionalProperties: false,
1316
};
1417

packages/rest-typings/src/v1/teams/TeamsListChildren.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,11 @@ const TeamsListChildrenPropsSchema = {
3030
sort: { type: 'string' },
3131
},
3232
additionalProperties: false,
33-
oneOf: [{ required: ['teamId'] }, { required: ['teamName'] }, { required: ['roomId'] }],
33+
oneOf: [
34+
{ type: 'object', required: ['teamId'] },
35+
{ type: 'object', required: ['teamName'] },
36+
{ type: 'object', required: ['roomId'] },
37+
],
3438
};
3539

3640
export const isTeamsListChildrenProps = ajvQuery.compile<TeamsListChildrenProps>(TeamsListChildrenPropsSchema);

packages/rest-typings/src/v1/teams/TeamsListRoomsOfUserProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ const teamsListRoomsOfUserPropsSchema = {
1919
count: { type: 'number', nullable: true },
2020
sort: { type: 'string', nullable: true },
2121
},
22-
oneOf: [{ required: ['teamId'] }, { required: ['teamName'] }],
22+
oneOf: [
23+
{ type: 'object', required: ['teamId'] },
24+
{ type: 'object', required: ['teamName'] },
25+
],
2326
required: ['userId'],
2427
additionalProperties: false,
2528
};

packages/rest-typings/src/v1/teams/TeamsListRoomsProps.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ const teamsListRoomsPropsSchema = {
1919
count: { type: 'number', nullable: true },
2020
sort: { type: 'string', nullable: true },
2121
},
22-
oneOf: [{ required: ['teamId'] }, { required: ['teamName'] }],
22+
oneOf: [
23+
{ type: 'object', required: ['teamId'] },
24+
{ type: 'object', required: ['teamName'] },
25+
],
2326
additionalProperties: false,
2427
};
2528

0 commit comments

Comments
 (0)