fix: reference to incorrect node when computing schema name of persis…

…ted call (#333)

Co-authored-by: Jovi De Croock <decroockjovi@gmail.com>

.changeset/lovely-phones-yell.md

@@ -0,0 +1,5 @@
+---
+"@0no-co/graphqlsp": patch
+---
+
+Fix schema derivation when using `graphql.persisted`, we used the wrong expression in the ast

packages/graphqlsp/src/ast/checks.ts

@@ -93,10 +93,15 @@ export const isGraphQLTag = (
 /** Retrieves the `__name` branded tag from gql.tada `graphql()` or `graphql.persisted()` calls */
 export const getSchemaName = (
   node: ts.CallExpression,
-  typeChecker: ts.TypeChecker | undefined
+  typeChecker: ts.TypeChecker | undefined,
+  isTadaPersistedCall = false
 ): string | null => {
   if (!typeChecker) return null;
-  const type = typeChecker.getTypeAtLocation(node.expression);
+  const type = typeChecker.getTypeAtLocation(
+    // When calling `graphql.persisted`, we need to access the `graphql` part of
+    // the expression; `node.expression` is the `.persisted` part
+    isTadaPersistedCall ? node.getChildAt(0).getChildAt(0) : node.expression
+  );
   if (type) {
     const brandTypeSymbol = type.getProperty('__name');
     if (brandTypeSymbol) {

packages/graphqlsp/src/ast/index.ts

@@ -215,7 +215,7 @@ export function findAllPersistedCallExpressions(
     if (!checks.isTadaPersistedCall(node, typeChecker)) {
       return;
     } else if (info) {
-      const name = checks.getSchemaName(node, typeChecker);
+      const name = checks.getSchemaName(node, typeChecker, true);
       result.push({ node, schema: name });
     } else {
       result.push(node);