Fund: Misleading static type for "execution result" attribute (introduced in `0.240.0`)

I think #3554 introduce a static typing bug in the execution result attribute.

It's statically typed as graphql.ExecutionResult type from graphql-core:

strawberry/strawberry/types/execution.py

Line 25 in 2941146

from graphql import ExecutionResult as GraphQLExecutionResult

strawberry/strawberry/types/execution.py

Line 56 in 2941146

result: Optional[GraphQLExecutionResult] = None

But in reality it's this custom dataclass from Strawberry's codebase (at least in some situations):

strawberry/strawberry/types/execution.py

Lines 92 to 108 in 2941146

    
           @dataclasses.dataclass 
        
           class ExecutionResult: 
        
               data: Optional[Dict[str, Any]] 
        
               errors: Optional[List[GraphQLError]] 
        
               extensions: Optional[Dict[str, Any]] = None 
        
           @dataclasses.dataclass 
        
           class PreExecutionError(ExecutionResult): 
        
               """Differentiate between a normal execution result and an immediate error. 
        
               Immediate errors are errors that occur before the execution phase i.e validation errors, 
        
               or any other error that occur before we interact with resolvers. 
        
               These errors are required by `graphql-ws-transport` protocol in order to close the operation 
        
               right away once the error is encountered. 
        
               """

PS: The specific problem is the typechecker makes us think it's okay to access the formatted property, which is actually missing:

https://github.com/graphql-python/graphql-core/blob/26701397d84338a42c7acbce78368ae8f9d97271/src/graphql/execution/incremental_publisher.py#L97-L105

Strawberry GraphQL/strawberry

Misleading static type for "execution result" attribute (introduced in 0.240.0)

How does funding with Polar work?

Backer

Contributor

Maintainer

	@dataclasses.dataclass
	class ExecutionResult:
	data: Optional[Dict[str, Any]]
	errors: Optional[List[GraphQLError]]
	extensions: Optional[Dict[str, Any]] = None


	@dataclasses.dataclass
	class PreExecutionError(ExecutionResult):
	"""Differentiate between a normal execution result and an immediate error.

	Immediate errors are errors that occur before the execution phase i.e validation errors,
	or any other error that occur before we interact with resolvers.

	These errors are required by `graphql-ws-transport` protocol in order to close the operation
	right away once the error is encountered.
	"""

Strawberry GraphQL/strawberry

Misleading static type for "execution result" attribute (introduced in 0.240.0)

How does funding with Polar work?

Backer

Why does "Fund on completion" require GitHub login?

When is the invoice due for "Fund on completion"?

What happens if the issue is never completed?

Do I get any extra benefits by funding?

Do I get progress updates?

Contributor

Do I get a reward?

Is rewards guaranteed?

Maintainer

How can I get funding like this for my open source initiatives?