Fix new redocly lints (#2182)

Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>

changelogs/application_service/newsfragments/2182.clarification

@@ -0,0 +1 @@
+Minor fixes to JSON schemas.

changelogs/client_server/newsfragments/2182.clarification

@@ -0,0 +1 @@
+Minor fixes to JSON schemas.

data/api/application-service/definitions/protocol.yaml

@@ -23,3 +23,4 @@ allOf:
       type: array
       items:
         $ref: protocol_instance.yaml
+  required: ['instances']

data/api/application-service/definitions/protocol_base.yaml

@@ -77,4 +77,4 @@ properties:
         "placeholder": "#foobar"
       }
     }
-required: ['user_fields', 'location_fields', 'icon', 'field_types', 'instances']
+required: ['user_fields', 'location_fields', 'icon', 'field_types']

data/api/client-server/definitions/protocol.yaml

@@ -37,7 +37,8 @@ allOf:
                   A unique identifier for this instance on the homeserver. This field is added
                   to the response of [`GET /_matrix/app/v1/thirdparty/protocol/{protocol}`](/application-service-api/#get_matrixappv1thirdpartyprotocolprotocol)
                   by the homeserver.
-                  
+
                   This is the identifier to use as the `third_party_instance_id` in a request to
                   [`POST /_matrix/client/v3/publicRooms`](/client-server-api/#post_matrixclientv3publicrooms).
                 example: "irc-freenode"
+  required: ['instances']

data/api/client-server/definitions/room_summary.yaml

@@ -42,9 +42,3 @@ allOf:
       room_version:
         description: The version of the room.
         type: string
-
-required:
-  - room_id
-  - num_joined_members
-  - world_readable
-  - guest_can_join

data/api/client-server/room_summary.yaml

@@ -89,11 +89,6 @@ paths:
                           - leave
                           - ban
                         type: string
-                required:
-                  - guest_can_join
-                  - num_joined_members
-                  - room_id
-                  - world_readable
               examples:
                 response:
                   value: {

data/event-schemas/schema/core-event-schema/room_event.yaml

@@ -1,13 +1,13 @@
+title: RoomEvent
+description: Room Events have the following fields.
 allOf:
 - $ref: sync_room_event.yaml
-description: Room Events have the following fields.
+- type: object
-properties:
+  properties:
-  room_id:
+    room_id:
-    description: |-
+      description: |-
-      The ID of the room associated with this event. Will not be present on events
+        The ID of the room associated with this event. Will not be present on events
-      that arrive through `/sync`, despite being required everywhere else.
+        that arrive through `/sync`, despite being required everywhere else.
-    type: string
+      type: string
-required:
+  required:
-- room_id
+  - room_id
-title: RoomEvent
-type: object

data/event-schemas/schema/core-event-schema/sync_room_event.yaml

@@ -17,27 +17,27 @@
 # be on events, so we give it a whole event structure as a base for room_event.
 # This base doesn't declare a room_id, which instead appears in the room_event.
 
-allOf:
+title: SyncRoomEvent
-- $ref: event.yaml
 description: In addition to the Event fields, Room Events have the following additional
   fields.
-properties:
+allOf:
-  event_id:
+- $ref: event.yaml
-    description: The globally unique event identifier.
+- type: object
-    type: string
+  properties:
-  sender:
+    event_id:
-    description: Contains the fully-qualified ID of the user who sent this event.
+      description: The globally unique event identifier.
-    type: string
+      type: string
-  origin_server_ts:
+    sender:
-    description: Timestamp in milliseconds on originating homeserver
+      description: Contains the fully-qualified ID of the user who sent this event.
-      when this event was sent.
+      type: string
-    type: integer
+    origin_server_ts:
-    format: int64
+      description: Timestamp in milliseconds on originating homeserver
-  unsigned:
+        when this event was sent.
-    $ref: unsigned_prop.yaml
+      type: integer
-required:
+      format: int64
-- event_id
+    unsigned:
-- sender
+      $ref: unsigned_prop.yaml
-- origin_server_ts
+  required:
-title: SyncRoomEvent
+  - event_id
-type: object
+  - sender
+  - origin_server_ts

data/event-schemas/schema/core-event-schema/sync_state_event.yaml

@@ -14,21 +14,21 @@
 
 # See sync_room_event.yaml for why this file is here.
 
-allOf:
+title: SyncStateEvent
-- $ref: sync_room_event.yaml
 description: In addition to the Room Event fields, State Events have the following
   additional fields.
-properties:
+allOf:
-  state_key:
+- $ref: sync_room_event.yaml
-    description: A unique key which defines the overwriting semantics for this piece
+- type: object
-      of room state. This value is often a zero-length string. The presence of this
+  properties:
-      key makes this event a State Event.
+    state_key:
+      description: A unique key which defines the overwriting semantics for this piece
+        of room state. This value is often a zero-length string. The presence of this
+        key makes this event a State Event.
 
-      State keys starting with an `@` are reserved for referencing user IDs, such
+        State keys starting with an `@` are reserved for referencing user IDs, such
-      as room members. With the exception of a few events, state events set with a
+        as room members. With the exception of a few events, state events set with a
-      given user's ID as the state key MUST only be set by that user.
+        given user's ID as the state key MUST only be set by that user.
-    type: string
+      type: string
-required:
+  required:
-- state_key
+  - state_key
-title: SyncStateEvent
-type: object