[Schema Inaccuracy] exclude property of Migration model is inaccurate

[baywet]

Schema Inaccuracy

The exclude property for the migration model doesn't have a schema, which will be problematic for any client code generator.
This model is used by the /orgs/{org}/migrations endpoint.

Expected

exclude property for the Migration model to have a schema or to be removed

Reproduction Steps

[nickfloyd]

Hey @baywet would you mind including the snip of schema (the migrations schema) that you're looking at and the operationId? I found a couple of references to that property that appear to have some form of schema (see below), but I want to make sure we're looking at the same thing.

Also, if you could provide a link to the descriptions that you are using that would help me make sure were working in the same place - we have a few different versions for our different products of our OpenAPI descriptions

"/orgs/{org}/migrations":
    get:
      summary: List organization migrations
      description: |-
        Lists the most recent migrations, including both exports (which can be started through the REST API) and imports (which cannot be started using the REST API).

        A list of `repositories` is only returned for export migrations.
      tags:
      - migrations
      operationId: migrations/list-for-org
      externalDocs:
        description: API method documentation
        url: https://docs.github.com/rest/migrations/orgs#list-organization-migrations
      parameters:
      - "$ref": "#/components/parameters/org"
      - "$ref": "#/components/parameters/per-page"
      - "$ref": "#/components/parameters/page"
      - name: exclude
        description: Exclude attributes from the API response to improve performance
        in: query
        schema:
          type: array
          items:
            description: Allowed values that can be passed to the exclude param.
            enum:
            - repositories
            example: repositories
            type: string

and

"/orgs/{org}/migrations/{migration_id}":
    get:
      summary: Get an organization migration status
      description: |-
        Fetches the status of a migration.

        The `state` of a migration can be one of the following values:

        *   `pending`, which means the migration hasn't started yet.
        *   `exporting`, which means the migration is in progress.
        *   `exported`, which means the migration finished successfully.
        *   `failed`, which means the migration failed.
      tags:
      - migrations
      operationId: migrations/get-status-for-org
      externalDocs:
        description: API method documentation
        url: https://docs.github.com/rest/migrations/orgs#get-an-organization-migration-status
      parameters:
      - "$ref": "#/components/parameters/org"
      - "$ref": "#/components/parameters/migration-id"
      - name: exclude
        description: Exclude attributes from the API response to improve performance
        in: query
        schema:
          type: array
          items:
            description: Allowed values that can be passed to the exclude param.
            enum:
            - repositories
            example: repositories
            type: string

[baywet]

The failing model/component is #/components/schemas/migration
More precisely the exclude property.

        exclude:
          type: array
          items: {}

This model is being referenced by the following endpoints:

• /orgs/{org}/migrations
• /orgs/{org}/migrations/{migration_id}
• /user/migrations
• /user/migrations/{migration_id}

I hope that helps, let me know if you need anything else.

[nickfloyd]

Got it. Thank you for the clarity.