Only minify source once when generating ID

Author: dsanders11

packages/graphql-mini-transforms/src/document.ts

@@ -16,26 +16,14 @@ import {DocumentNode, SimpleDocument} from 'graphql-typed';
 const IMPORT_REGEX = /^#import\s+['"]([^'"]*)['"];?[\s\n]*/gm;
 const DEFAULT_NAME = 'Operation';
 
-function defaultGenerateId(normalizedSource: string) {
-  // This ID is a hash of the full file contents that are part of the document,
-  // including other documents that are injected in, but excluding any unused
-  // fragments. This is useful for things like persisted queries.
-  return createHash('sha256')
-    .update(minifySource(normalizedSource))
-    .digest('hex');
-}
-
 export interface CleanDocumentOptions {
   removeUnused?: boolean;
   generateId?: (normalizedSource: string) => string;
 }
 
 export function cleanDocument(
   document: UntypedDocumentNode,
-  {
-    removeUnused = true,
-    generateId = defaultGenerateId,
-  }: CleanDocumentOptions = {},
+  {removeUnused = true, generateId}: CleanDocumentOptions = {},
 ): DocumentNode<any, any, any> {
   if (removeUnused) {
     removeUnusedDefinitions(document);
@@ -53,7 +41,18 @@ export function cleanDocument(
     stripLoc(definition);
   }
 
-  const id = generateId(documentSource);
+  let id: string;
+
+  if (generateId === undefined) {
+    // This ID is a hash of the full file contents that are part of the document,
+    // including other documents that are injected in, but excluding any unused
+    // fragments. This is useful for things like persisted queries.
+    id = createHash('sha256')
+      .update(minifySource(normalizedSource))
+      .digest('hex');
+  } else {
+    id = generateId(documentSource);
+  }
 
   Reflect.defineProperty(normalizedDocument, 'id', {
     value: id,