let `require-id-when-available` validate when fragment is present

consider

fragment a on A {
  id
  aField
}

# import "./a.fragment.graphql"
query AQuery {
  as { ...a }
}

Field “id” must be selected when it’s available on a type. Please make sure to include it in your selection set!eslint@graphql-eslint/require-id-when-available

require-id-when-available should allow fields which contain (only?) fragment spreads. Bonus points if the fragment themselves validate as having id

[dotansimha]

Added another round of fixes in https://github.com/dotansimha/graphql-eslint/commit/08007cbf53731b29fba1cf3c01cb7db7eec7222c , now it should work better with inline fragments. Available in @graphql-eslint/eslint-plugin@0.5.0. Let me know if there are other issues.

[dotansimha]

@tvvignesh this issue is currently blocked because I didn’t have the time to implementing the siblings operations support. Since ESLint runs only on a single file every time, we can’t really know that’s in that fragment, so we don’t know if there is id field there. I started a PR with parser performance improvements required for this, hope to have progress on that next week 😃 This feature should allow us to do so much more with graphql-eslint 😃