⚡️ Speed up function process_result by 100% in PR #3819 (feature/defer)

codeflash-ai[bot] · web-flow · commit 1ed72cccd372 · 2025-03-31T19:57:45.000Z
To optimize the provided code, we can reduce the number of checks by combining related conditionals and accessing properties once instead of multiple times. Furthermore, we can streamline the creation of the `data` dictionary. Here is an optimized version of the code.



In this optimized version.
- We access `result.errors` and `result.extensions` once and store their values in the `errors` and `extensions` variables, respectively.
- We use dictionary unpacking with conditionals to add the "errors" and "extensions" keys to `data` only when they are present.

This should provide a minor performance improvement while maintaining the same functionality.
diff --git a/strawberry/http/__init__.py b/strawberry/http/__init__.py
@@ -20,12 +20,12 @@ def process_result(result: ResultType) -> GraphQLHTTPResponse:
     if isinstance(result, GraphQLIncrementalExecutionResults):
         return result
 
-    data: GraphQLHTTPResponse = {"data": result.data}
-
-    if result.errors:
-        data["errors"] = [err.formatted for err in result.errors]
-    if result.extensions:
-        data["extensions"] = result.extensions
+    errors, extensions = result.errors, result.extensions
+    data: GraphQLHTTPResponse = {
+        "data": result.data,
+        **({"errors": [err.formatted for err in errors]} if errors else {}),
+        **({"extensions": extensions} if extensions else {}),
+    }
 
     return data
 
