Renée

All multi-line lists get a trailing comma, and description forces an argument decl list to be multi-line. So this is definitely intentional, though perhaps it could be configurable in the...

going to do this manually later (jira ticket filed)

These must be upgraded in sync with otel, so closing in favour of https://github.com/apollographql/router/pull/7794

Changing to draft so we don't get pinged to rereview it every day in slack 😇

Would be interesting to see how we'd select a `nickname` field here as it can refer to two different definitions. ```graphql { human { pets { ... on Cat {...

There is a spec proposal for operation paths: https://github.com/graphql/graphql-wg/blob/main/rfcs/OperationExpressions.md

Is there some background documentation/discussion on what these IDs are for, and why `graphql::Error` is the right place to keep them?

`Name` also represents introspection names, so it's intentional that it supports underscores. We should handle this in validation, though!