Marking a query field argument as no longer mandatory is considered a breaking change

[klippx]

Graphql Inspector says that:
✖ Type for argument transactions on field Query.transactions changed from [TransactionIdentifier!]! to [TransactionIdentifier!]

To Reproduce

Checked in schema:

const Schema = gql`
  type Query {
    "The transactions query is the general no-frills transactions search."
    transactions(
      "Use this parameter to limit the search to the specified TransactionIdentifier objects (limited to 50 per request)"
      transactions: [TransactionIdentifier!]!
    ): Transactions!
  }
`

New schema:

const Schema = gql`
  type Query {
    "The transactions query is the general no-frills transactions search."
    transactions(
      "Use this parameter to limit the search to the specified TransactionIdentifier objects (limited to 50 per request)"
      transactions: [TransactionIdentifier!]
    ): Transactions!
  }
`

Expected behavior

No breaking changes detected

Environment:

• OS: MacOS 12.1
• @graphql-inspector/...: 2.9.0
• graphql: 15.8.0
• NodeJS: v12.18.4 (monorepo, dont look at me 😁 )

Additional context

[filipncs]

This appears to now be fixed, could you try again?

Your schemas are not valid in isolation, but the following similar schemas now give the expected behaviour (on 3.3.0).

Old:

input TransactionIdentifier  {
  internal: String!
}

type Query {
  transactions(transactions: [TransactionIdentifier!]!): String
}

New:

input TransactionIdentifier  {
  internal: String!
}

type Query {
  transactions(transactions: [TransactionIdentifier!]): String
}

Result:

✔  Type for argument transactions on field Query.transactions changed from [TransactionIdentifier!]! to [TransactionIdentifier!]
success No breaking changes detected