fix: skip empty non-list queries during fetchAll pagination (#1395)

## Summary
- extend the regression test covering a token query whose only field
uses `@fetchAll` so it also verifies no extra non-list request is issued
- ensure paginated results create parent containers before setting list
data
- skip executing non-list GraphQL requests when no fields remain after
removing `@fetchAll`

## Testing
- bun test sdk/thegraph/src/utils/pagination.test.ts *(fails: missing
dependency `gql.tada` because packages cannot be downloaded in the
execution environment)*

------
https://chatgpt.com/codex/tasks/task_b_6904799dd93883208eb61e4f90b5c5ab

## Summary by Sourcery

Prevent empty non-list queries during @fetchall pagination by checking
for executable fields and extend tests to cover queries with only
fetchAll lists

Bug Fixes:
- Skip executing non-list GraphQL queries when no executable fields
remain after removing @fetchall

Enhancements:
- Add countExecutableFields helper to detect and skip empty queries
- Refactor createNonListQuery to prune fields without selections on
leave

Tests:
- Add test to verify queries with only @fetchall list fields skip
non-list requests and return paged data

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Skips executing non-list GraphQL requests when only @fetchall list
fields remain, with AST pruning and a regression test to verify
behavior.
> 
> - **Pagination utils**:
> - `createNonListQuery(...)`: removes list fields and prunes empty
selection sets; always returns a filtered document.
> - `countExecutableFields(...)`: new helper to detect empty documents;
used to skip non-list requests when no executable fields remain.
>   - Minor: import `FieldNode` for leave-visitor pruning.
> - **Tests**:
> - Add regression test ensuring a query with only `@fetchAll` list
fields returns data and does not issue an extra non-list request.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
95573b2. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: janb87

sdk/thegraph/src/utils/pagination.test.ts

@@ -353,6 +353,29 @@ describe("createTheGraphClientWithPagination", () => {
     });
   });
 
+  it("should return token if only @fetchAll field is requested", async () => {
+    const result = await client.query(
+      theGraphGraphql(`
+        query($name: String) {
+          token(name: $name) {
+            holders @fetchAll {
+              name
+            }
+          }
+        }`),
+      {
+        name: "Token 100",
+      },
+    );
+
+    expect(result).toEqual({
+      token: {
+        holders: TEST_HOLDERS,
+      },
+    });
+    expect(requestMock).toHaveBeenCalledTimes(3);
+  });
+
   it("should allow multiple queries in a single request", async () => {
     const result = await client.query(
       theGraphGraphql(`

sdk/thegraph/src/utils/pagination.ts

@@ -1,7 +1,7 @@
 import { sortBy } from "es-toolkit";
 import { get, isArray, isEmpty, set } from "es-toolkit/compat";
 import type { TadaDocumentNode } from "gql.tada";
-import { type ArgumentNode, type DocumentNode, Kind, parse, visit } from "graphql";
+import { type ArgumentNode, type DocumentNode, type FieldNode, Kind, parse, visit } from "graphql";
 import type { GraphQLClient, RequestDocument, RequestOptions, Variables } from "graphql-request";
 
 // Constants for TheGraph limits
@@ -244,7 +244,6 @@ function createSingleFieldQuery(
 
 // Create query without list fields
 function createNonListQuery(document: DocumentNode, listFields: ListFieldWithFetchAllDirective[]): DocumentNode | null {
-  let hasFields = false;
   const pathStack: string[] = [];
 
   const filtered = visit(document, {
@@ -263,17 +262,35 @@ function createNonListQuery(document: DocumentNode, listFields: ListFieldWithFet
           pathStack.pop();
           return null;
         }
-
-        hasFields = true;
         return undefined;
       },
-      leave: () => {
+      leave: (node: FieldNode) => {
         pathStack.pop();
+        if (node.selectionSet && node.selectionSet.selections.length === 0) {
+          return null;
+        }
+        return undefined;
       },
     },
   });
 
-  return hasFields ? filtered : null;
+  return filtered;
+}
+
+function countExecutableFields(document: DocumentNode): number {
+  let fieldCount = 0;
+
+  visit(document, {
+    Field: (node) => {
+      if (!node.name.value.startsWith("__")) {
+        if (!node.selectionSet || node.selectionSet.selections.length > 0) {
+          fieldCount += 1;
+        }
+      }
+    },
+  });
+
+  return fieldCount;
 }
 
 // Filter variables to only include used ones
@@ -416,6 +433,9 @@ export function createTheGraphClientWithPagination(theGraphClient: Pick<GraphQLC
       const nonListQuery = createNonListQuery(processedDocument, listFields);
 
       if (nonListQuery) {
+        if (countExecutableFields(nonListQuery) === 0) {
+          return result as TResult;
+        }
         const nonListResult = await theGraphClient.request(
           nonListQuery,
           filterVariables(variables, nonListQuery) ?? {},