cjoudrey/graphql-schema-linter

Non-null Int types for "first" argument are not accepted by rule "relay-connection-arguments-spec"

Open

#148 opened on 2018年9月7日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)JavaScript (669 stars) (62 forks)batch import
enhancementhelp wanted

説明

The following queries all fail lint checks:

  notablePeople(first: Int!, after: ID): NotablePersonConnection!

  users(first: Int!, after: ID, where: UsersQueryWhereInput): UserConnection!
    @requireOneOfRoles(roles: [MODERATOR])

  notablePeopleEvents(
    first: Int!
    after: ID
    where: NotablePersonEventsQueryWhereInput
  ): NotablePersonEventConnection!

If I remove the !, they all pass. Should not non-null integer types be accepted?

コントリビューターガイド

技術スタック
javascriptgraphql
領域
apitooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
basic GraphQL knowledgefamiliarity with graphql schema linter
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
55
調査方針
The issue is in the `relay connection arguments spec` rule which likely validates that connection fields have `first`/`last` arguments of type `Int`. The current implementation does not accept non null `Int!` types. Investigate the rule's source code in the repository (likely in `src/rules/relay connection arguments spec.js`) to understand how type checking is performed. The fix may involve allowing both `Int` and `Int!` by updating the type comparison logic. Check the GraphQL spec for Relay connections to confirm that non null `Int` is valid according to the spec.