Summary by CodeRabbit

• New Features
  • Added support for the oneOf directive as a built-in directive in schema imports.

Summary by CodeRabbit

• New Features
  • Added oneOf directive to the list of built-in GraphQL directives, enabling proper schema handling and validation for input types using this specification.

Walkthrough

Registers the oneOf directive in the builtinDirectives array and updates tests and fixtures: adds input D annotated with @oneOf, and changes C.id from ID! to D! in test fixtures and expected SDL.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Tests as Test Suite
  participant ImportPkg as `@graphql-tools/import`
  participant Fixtures as Schema Fixtures

  Note over ImportPkg: builtinDirectives updated\n(includes `oneOf`)
  Tests->>Fixtures: read fixtures (`c.graphql`, `d.graphql`)
  Tests->>ImportPkg: importSchema(fixtures)
  ImportPkg->>ImportPkg: include builtinDirectives (contains `oneOf`)
  ImportPkg-->>Tests: produce SDL (input `D `@oneOf``, `C.id` -> `D!`)
  Note over Tests,ImportPkg: verify expected SDL

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Single-line change in packages/import/src/index.ts plus additions/updates to test fixtures and expectations.
• Areas to check:
  • Correct insertion/order of 'oneOf' among builtin directives in packages/import/src/index.ts.
  • Test SDL expectations in packages/import/tests/schema/import-schema.spec.ts match the new @oneOf annotation and updated C type.
  • Ensure new fixture d.graphql complies with schema formatting/lint rules.

Possibly related PRs

• Handle federation v2 links #7229 — Modifies the same builtinDirectives list in packages/import/src/index.ts; related to directive registration changes.

Suggested reviewers

• ardatan

Poem

🐇 OneOf hops into the list tonight,
D springs fields under moonlight.
C trades an ID for D's gentle cheer,
Tests nod softly — all is clear. 🥕

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

• X
• Mastodon
• Reddit
• LinkedIn