Skip to content

Feat/gj26 beta#2152

Draft
Samjin wants to merge 12 commits intoExpediaGroup:masterfrom
Samjin:feat/gj26-beta
Draft

Feat/gj26 beta#2152
Samjin wants to merge 12 commits intoExpediaGroup:masterfrom
Samjin:feat/gj26-beta

Conversation

@Samjin
Copy link
Contributor

@Samjin Samjin commented Feb 23, 2026

📝 Description

🔗 Related Issues

sjin and others added 12 commits February 20, 2026 21:00
- Change version from 9.0.0-SNAPSHOT to 9.0.0-beta-gj26 in gradle.properties
- Branch feat/gj26-beta created from tag 9.0.0-alpha.8

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…0.0, federation to 6.0.0

- graphql-java: 23.1 -> 26.0.beta-2
- java-dataloader: 4.0.0 -> 6.0.0
- federation-graphql-java-support: 5.2.0 -> 6.0.0

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…nd dataloader modules

- SchemaGenerator.generateAdditionalTypes() returns Set<GraphQLNamedType> (gj26 type change)
- KotlinDataLoader K type parameter bounded to Any (java-dataloader 6.0 JSpecify)
- ExecutionStrategyState uses local val for GraphQLType null safety (JSpecify)
- SyncExecutionExhaustedState handles nullable ExecutionId from JSpecify annotations
- ExecutionInputState null-checks parentFieldGraphQLType before isList call
- FederatedSchemaGeneratorHooks handles nullable getObjectName() return
- LinkImport coercing handles nullable StringValue.getValue()

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…or modules

- Server module (FlowSubscriptionExecutionStrategy) compiled without errors as predicted
- Spring-server and ktor-server modules compiled without errors (transitive only)
- APQ: handle nullable persistedQueryError.message from JSpecify
- Client-generator: replace deprecated getType() with getTypeOrNull() (3 call sites)
- Client-generator: handle nullable rootType from map lookup
- All 20+ modules compile: ./gradlew assemble BUILD SUCCESSFUL

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- PR body saved to PR_BODY.md for manual creation (EMU auth prevents gh pr create)
- Documents all dependency version changes, API migrations, and breaking changes
- Branch pushed to origin/feat/gj26-beta

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- APQ: handle PersistedQuerySupport.PERSISTED_QUERY_MARKER in query check,
  fix nullable extensions access with safe calls
- DataLoader: add .build() to DataLoaderOptions in all test data loaders
  (java-dataloader 6.0.0 API change)
- FlowSubscriptionExecutionStrategy: handle gj26 returning raw values
  from fetchField instead of FetchedValue wrapper
- DirectiveTests: make deprecated argument nullable with default to comply
  with gj26 schema validation for @deprecated on required args
- SchemaGenerator: remove unused GraphQLType import
…t 6.0.0

federation-graphql-java-support 6.0.0 references graphql.schema.idl.DirectiveInfo
which was removed in graphql-java 26.0.beta-2. This shim class bridges the gap
by delegating to graphql.Directives (the gj26 replacement).

Provides GRAPHQL_SPECIFICATION_DIRECTIVE_MAP and isGraphqlSpecifiedDirective()
needed by ServiceSDLPrinter at runtime during graphqlGenerateSDL task.

TODO: Remove when federation-graphql-java-support releases a gj26-compatible version.
…e images

- Add GraphQLClientGeneratorTest with implicit schema definition test to cover
  findRootType else branch, bringing BRANCH coverage from 0.84 to >= 0.85
- Add test for multiple operations in file error path
- Replace deprecated openjdk:17 Docker image with eclipse-temurin:17-jre in
  federation-compatibility, products-subgraph, and reviews-subgraph Dockerfiles

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove unused import kotlin.test.assertEquals
- Reformat writeText multi-line string for argument-list-wrapping rule
- All 3 ktlint violations resolved (unused import, missing newline after '(', missing newline before ')')

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Previous run canceled at 30min timeout with cold Gradle cache.
Second run should benefit from Gradle cache warm-up.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant