JSpecify gigantic wave 4 by dondonz · Pull Request #4301 · graphql-java/graphql-java

claude

and others added 30 commits

February 28, 2026 23:21

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

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

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

# Conflicts:
#	src/test/groovy/graphql/archunit/JSpecifyAnnotationsCheck.groovy

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

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

dondonz changed the base branch from claude/agent-team-jspecify-Frd74 to master

March 7, 2026 20:35

TreeParallelTransformer only set List.class as a context var but never
NodeAdapter.class. After JSpecify annotations wrapped getVar() calls with
assertNotNull(), this caused test failures in TreeParallelTransformerTest
and AstTransformerTest.

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

# Conflicts:
#	.claude/commands/jspecify-annotate.md
#	src/main/java/graphql/execution/ExecutionContext.java
#	src/main/java/graphql/introspection/GoodFaithIntrospection.java
#	src/main/java/graphql/normalized/ExecutableNormalizedOperation.java
#	src/test/groovy/graphql/archunit/JSpecifyAnnotationsCheck.groovy

# Conflicts:
#	src/main/java/graphql/execution/directives/QueryAppliedDirectiveArgument.java
#	src/main/java/graphql/execution/preparsed/PreparsedDocumentEntry.java
#	src/main/java/graphql/util/Anonymizer.java

Removes 31 classes from JSPECIFY_EXEMPTION_LIST in JSpecifyAnnotationsCheck.groovy
that have already been annotated with @NullMarked/@NullUnmarked across the wave 4
work. The "exempted classes should not be annotated" archunit test was failing
because these classes were both exempt and annotated.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

# Conflicts:
#	src/main/java/graphql/language/AstSignature.java
#	src/main/java/graphql/language/VariableDefinition.java

dondonz marked this pull request as ready for review

May 24, 2026 00:31

…ql-java into claude/jspecify-wave4