Delete shared type rename patch as new blueprint handles it properly

Author: gnawf

lib/src/main/java/graphql/nadel/NadelExecutionHints.kt

@@ -3,7 +3,6 @@ package graphql.nadel
 import graphql.nadel.hints.AllDocumentVariablesHint
 import graphql.nadel.hints.LegacyOperationNamesHint
 import graphql.nadel.hints.NadelDeferSupportHint
-import graphql.nadel.hints.NadelSharedTypeRenamesHint
 import graphql.nadel.hints.NadelShortCircuitEmptyQueryHint
 import graphql.nadel.hints.NadelVirtualTypeSupportHint
 import graphql.nadel.hints.NewResultMergerAndNamespacedTypename
@@ -13,7 +12,6 @@ data class NadelExecutionHints(
     val allDocumentVariablesHint: AllDocumentVariablesHint,
     val newResultMergerAndNamespacedTypename: NewResultMergerAndNamespacedTypename,
     val deferSupport: NadelDeferSupportHint,
-    val sharedTypeRenames: NadelSharedTypeRenamesHint,
     val shortCircuitEmptyQuery: NadelShortCircuitEmptyQueryHint,
     val virtualTypeSupport: NadelVirtualTypeSupportHint,
 ) {
@@ -33,7 +31,6 @@ data class NadelExecutionHints(
         private var newResultMergerAndNamespacedTypename = NewResultMergerAndNamespacedTypename { false }
         private var deferSupport = NadelDeferSupportHint { false }
         private var shortCircuitEmptyQuery = NadelShortCircuitEmptyQueryHint { false }
-        private var sharedTypeRenames = NadelSharedTypeRenamesHint { false }
         private var virtualTypeSupport = NadelVirtualTypeSupportHint { false }
 
         constructor()
@@ -42,7 +39,9 @@ data class NadelExecutionHints(
             legacyOperationNames = nadelExecutionHints.legacyOperationNames
             allDocumentVariablesHint = nadelExecutionHints.allDocumentVariablesHint
             newResultMergerAndNamespacedTypename = nadelExecutionHints.newResultMergerAndNamespacedTypename
+            deferSupport = nadelExecutionHints.deferSupport
             shortCircuitEmptyQuery = nadelExecutionHints.shortCircuitEmptyQuery
+            virtualTypeSupport = nadelExecutionHints.virtualTypeSupport
         }
 
         fun legacyOperationNames(flag: LegacyOperationNamesHint): Builder {
@@ -70,11 +69,6 @@ data class NadelExecutionHints(
             return this
         }
 
-        fun sharedTypeRenames(flag: NadelSharedTypeRenamesHint): Builder {
-            sharedTypeRenames = flag
-            return this
-        }
-
         fun virtualTypeSupport(flag: NadelVirtualTypeSupportHint): Builder {
             virtualTypeSupport = flag
             return this
@@ -86,7 +80,6 @@ data class NadelExecutionHints(
                 allDocumentVariablesHint,
                 newResultMergerAndNamespacedTypename,
                 deferSupport,
-                sharedTypeRenames,
                 shortCircuitEmptyQuery,
                 virtualTypeSupport,
             )

lib/src/main/java/graphql/nadel/engine/transform/NadelServiceTypeFilterTransform.kt

@@ -100,7 +100,6 @@ class NadelServiceTypeFilterTransform : NadelTransform<State> {
             .all { objectTypeName ->
                 objectTypeName in typeNamesOwnedByService
                     || objectTypeName in underlyingTypeNamesOwnedByService
-                    || (executionContext.hints.sharedTypeRenames(service) && executionBlueprint.getUnderlyingTypeName(objectTypeName) in underlyingTypeNamesOwnedByService)
             }
 
         if (noForeignTypes) {
@@ -110,7 +109,6 @@ class NadelServiceTypeFilterTransform : NadelTransform<State> {
         val fieldObjectTypeNamesOwnedByService = overallField.objectTypeNames.filter { objectTypeName ->
             objectTypeName in typeNamesOwnedByService
                 || objectTypeName in underlyingTypeNamesOwnedByService
-                || (executionContext.hints.sharedTypeRenames(service) && executionBlueprint.getUnderlyingTypeName(objectTypeName) in underlyingTypeNamesOwnedByService)
         }
 
         return State(

lib/src/main/java/graphql/nadel/engine/transform/NadelTypeRenameResultTransform.kt

@@ -92,22 +92,10 @@ internal class NadelTypeRenameResultTransform : NadelTransform<State> {
                     ?: overallTypeName
             }
 
-            val typeName: String = if (executionContext.hints.sharedTypeRenames(service)) {
-                if (overallField.objectTypeNames.contains(overallTypeName)) {
-                    overallTypeName
-                } else {
-                    overallField.objectTypeNames.singleOrNull {
-                        executionBlueprint.getRename(it)?.underlyingName == underlyingTypeName
-                    } ?: overallTypeName
-                }
-            } else {
-                overallTypeName
-            }
-
             NadelResultInstruction.Set(
                 subject = parentNode,
                 key = NadelResultKey(overallField.resultKey),
-                newValue = JsonNode(typeName),
+                newValue = JsonNode(overallTypeName),
             )
         }
     }

lib/src/main/java/graphql/nadel/engine/transform/query/NadelQueryTransformer.kt

@@ -188,14 +188,8 @@ class NadelQueryTransformer private constructor(
     }
 
     private fun getUnderlyingTypeNames(objectTypeNames: Collection<String>): List<String> {
-        return if (executionContext.hints.sharedTypeRenames(service)) {
-            objectTypeNames.map {
-                executionBlueprint.getUnderlyingTypeName(overallTypeName = it)
-            }
-        } else {
-            objectTypeNames.map {
-                executionBlueprint.getUnderlyingTypeName(service, overallTypeName = it)
-            }
+        return objectTypeNames.map {
+            executionBlueprint.getUnderlyingTypeName(service, overallTypeName = it)
         }
     }
 

test/src/test/kotlin/graphql/nadel/tests/hooks/shared-types-rename.kt

@@ -1,20 +1,9 @@
 package graphql.nadel.tests.hooks
 
-import graphql.nadel.NadelExecutionInput
 import graphql.nadel.tests.EngineTestHook
 import graphql.nadel.tests.UseHook
 
-abstract class `shared-types-rename` : EngineTestHook {
-    override fun makeExecutionInput(
-        builder: NadelExecutionInput.Builder,
-    ): NadelExecutionInput.Builder {
-        return builder.transformExecutionHints {
-            it.sharedTypeRenames {
-                true
-            }
-        }
-    }
-}
+abstract class `shared-types-rename` : EngineTestHook
 
 @UseHook
-class `renamed-type-in-union-declared-in-another-service` : `shared-types-rename`()
+class `renamed-type-in-union-declared-in-another-service` : `shared-types-rename`()