The AST node won't get fixed unless a field mapper is specified (#174)

Co-authored-by: Neeraj Joseph <njoseph@poppulo.com>
n1ru4l · Dec 2, 2022 · 7b4485c · 7b4485c
1 parent 5169495
commit 7b4485c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.changeset/smart-brooms-think.md b/.changeset/smart-brooms-think.md
@@ -0,0 +1,5 @@
+---
+"@n1ru4l/graphql-public-schema-filter": patch
+---
+
+fixed bug where the ast node was left unchanged after filtering the schema
diff --git a/src/public-schema-filter.ts b/src/public-schema-filter.ts
@@ -11,6 +11,7 @@ import {
   DirectiveNode,
   GraphQLFieldConfigArgumentMap,
   isNonNullType,
+  GraphQLFieldConfig,
 } from "graphql";
 import { MapperKind, mapSchema } from "@graphql-tools/utils";
 import { getWrappedType } from "./get-wrapped-type";
@@ -266,6 +267,9 @@ export const buildPublicSchema = (
       config.fields = newFields;
       return new GraphQLObjectType(config);
     },
+    [MapperKind.OBJECT_FIELD]: (fieldConfig) => {
+      return fieldConfig;
+    },
     [MapperKind.INPUT_OBJECT_TYPE]: (objectType) => {
       const config = objectType.toConfig();
       if (!publicTypeNames.has(config.name)) {