@wundergraph/composition 0.45.0 → 0.46.0

package/dist/v1/federation/federation-factory.d.ts

@@ -13,6 +13,7 @@ import { ContractTagOptions, FederationResult, FederationResultWithContracts, Mu
 import { GraphFieldData } from '../../utils/types';
 import { FederateSubgraphsContractV1Params, FederateSubgraphsWithContractsV1Params, FederationParams } from './types';
 import { FieldCoords, TypeName } from '../../types/types';
+import { ValidateOneOfDirectiveParams } from './params';
 export declare class FederationFactory {
     authorizationDataByParentTypeName: Map<string, AuthorizationData>;
     coordsByNamedTypeName: Map<string, Set<string>>;
@@ -90,6 +91,7 @@ export declare class FederationFactory {
     getValidFieldArgumentNodes(fieldData: FieldData): MutableInputValueNode[];
     getNodeWithPersistedDirectivesByFieldData(fieldData: FieldData, argumentNodes: Array<MutableInputValueNode>): MutableFieldNode;
     validateSemanticNonNull(data: FieldData): void;
+    validateOneOfDirective({ data, inputValueNodes, requiredFieldNames }: ValidateOneOfDirectiveParams): boolean;
     pushParentDefinitionDataToDocumentDefinitions(interfaceImplementations: InterfaceImplementationData[]): void;
     pushNamedTypeAuthDataToFields(): void;
     federateSubgraphData(): void;

package/dist/v1/federation/federation-factory.js

@@ -23,6 +23,7 @@ const type_merging_1 = require("../schema-building/type-merging");
 const string_constants_2 = require("../../utils/string-constants");
 const integer_constants_1 = require("../../utils/integer-constants");
 const utils_6 = require("../../utils/utils");
+const warnings_1 = require("../warnings/warnings");
 class FederationFactory {
     authorizationDataByParentTypeName;
     coordsByNamedTypeName = new Map();
@@ -51,6 +52,7 @@ class FederationFactory {
         [string_constants_2.AUTHENTICATED, constants_1.AUTHENTICATED_DEFINITION],
         [string_constants_2.DEPRECATED, constants_1.DEPRECATED_DEFINITION],
         [string_constants_2.INACCESSIBLE, constants_1.INACCESSIBLE_DEFINITION],
+        [string_constants_2.ONE_OF, constants_1.ONE_OF_DEFINITION],
         [string_constants_2.REQUIRES_SCOPES, constants_1.REQUIRES_SCOPES_DEFINITION],
         [string_constants_2.SEMANTIC_NON_NULL, constants_1.SEMANTIC_NON_NULL_DEFINITION],
         [string_constants_2.TAG, constants_1.TAG_DEFINITION],
@@ -1022,7 +1024,7 @@ class FederationFactory {
                     }
                 }
             }
-            if (interfaceDataByTypeName.size < 0 && !unionTypeName) {
+            if (interfaceDataByTypeName.size < 1 && !unionTypeName) {
                 this.errors.push((0, errors_1.incompatibleFederatedFieldNamedTypeError)(fieldCoordinates, subgraphNamesByNamedTypeName));
                 continue;
             }
@@ -1360,13 +1362,32 @@ class FederationFactory {
             }
         }
     }
+    validateOneOfDirective({ data, inputValueNodes, requiredFieldNames }) {
+        if (!data.directivesByDirectiveName.has(string_constants_2.ONE_OF)) {
+            return true;
+        }
+        if (requiredFieldNames.size > 0) {
+            this.errors.push((0, errors_1.oneOfRequiredFieldsError)({
+                requiredFieldNames: Array.from(requiredFieldNames),
+                typeName: data.name,
+            }));
+            return false;
+        }
+        if (inputValueNodes.length === 1) {
+            this.warnings.push((0, warnings_1.singleFederatedInputFieldOneOfWarning)({
+                fieldName: inputValueNodes[0].name.value,
+                typeName: data.name,
+            }));
+        }
+        return true;
+    }
     pushParentDefinitionDataToDocumentDefinitions(interfaceImplementations) {
         for (const [parentTypeName, parentDefinitionData] of this.parentDefinitionDataByTypeName) {
             if (parentDefinitionData.extensionType !== types_1.ExtensionType.NONE) {
                 this.errors.push((0, errors_1.noBaseDefinitionForExtensionError)((0, utils_6.kindToNodeType)(parentDefinitionData.kind), parentTypeName));
             }
             switch (parentDefinitionData.kind) {
-                case graphql_1.Kind.ENUM_TYPE_DEFINITION:
+                case graphql_1.Kind.ENUM_TYPE_DEFINITION: {
                     const enumValueNodes = [];
                     const clientEnumValueNodes = [];
                     const mergeMethod = this.getEnumValueMergeMethod(parentTypeName);
@@ -1421,11 +1442,16 @@ class FederationFactory {
                         values: clientEnumValueNodes,
                     });
                     break;
-                case graphql_1.Kind.INPUT_OBJECT_TYPE_DEFINITION:
-                    const invalidRequiredInputs = [];
-                    const inputValueNodes = [];
-                    const clientInputValueNodes = [];
+                }
+                case graphql_1.Kind.INPUT_OBJECT_TYPE_DEFINITION: {
+                    const invalidRequiredInputs = new Array();
+                    const inputValueNodes = new Array();
+                    const clientInputValueNodes = new Array();
+                    const requiredFieldNames = new Set();
                     for (const [inputValueName, inputValueData] of parentDefinitionData.inputValueDataByName) {
+                        if ((0, utils_5.isTypeRequired)(inputValueData.type)) {
+                            requiredFieldNames.add(inputValueName);
+                        }
                         if (parentDefinitionData.subgraphNames.size === inputValueData.subgraphNames.size) {
                             inputValueNodes.push(this.getNodeWithPersistedDirectivesByInputValueData(inputValueData));
                             if ((0, utils_5.isNodeDataInaccessible)(inputValueData)) {
@@ -1448,6 +1474,13 @@ class FederationFactory {
                         this.errors.push((0, errors_1.invalidRequiredInputValueError)(string_constants_2.INPUT_OBJECT, parentTypeName, invalidRequiredInputs, false));
                         break;
                     }
+                    if (!this.validateOneOfDirective({
+                        data: parentDefinitionData,
+                        inputValueNodes,
+                        requiredFieldNames,
+                    })) {
+                        break;
+                    }
                     parentDefinitionData.node.fields = inputValueNodes;
                     this.routerDefinitions.push(this.getNodeForRouterSchemaByData(parentDefinitionData));
                     if ((0, utils_5.isNodeDataInaccessible)(parentDefinitionData)) {
@@ -1455,7 +1488,7 @@ class FederationFactory {
                         break;
                     }
                     if (clientInputValueNodes.length < 1) {
-                        this.errors.push((0, errors_1.allChildDefinitionsAreInaccessibleError)((0, utils_6.kindToNodeType)(parentDefinitionData.kind), parentTypeName, 'input field'));
+                        this.errors.push((0, errors_1.allChildDefinitionsAreInaccessibleError)((0, utils_6.kindToNodeType)(parentDefinitionData.kind), parentTypeName, 'Input field'));
                         break;
                     }
                     this.clientDefinitions.push({
@@ -1464,9 +1497,10 @@ class FederationFactory {
                         fields: clientInputValueNodes,
                     });
                     break;
+                }
                 case graphql_1.Kind.INTERFACE_TYPE_DEFINITION:
                 // intentional fallthrough
-                case graphql_1.Kind.OBJECT_TYPE_DEFINITION:
+                case graphql_1.Kind.OBJECT_TYPE_DEFINITION: {
                     const fieldNodes = [];
                     const clientSchemaFieldNodes = [];
                     const graphFieldDataByFieldName = new Map();
@@ -1527,7 +1561,8 @@ class FederationFactory {
                         fields: clientSchemaFieldNodes,
                     });
                     break;
-                case graphql_1.Kind.SCALAR_TYPE_DEFINITION:
+                }
+                case graphql_1.Kind.SCALAR_TYPE_DEFINITION: {
                     if (constants_1.BASE_SCALARS.has(parentTypeName)) {
                         break;
                     }
@@ -1543,7 +1578,8 @@ class FederationFactory {
                         directives: (0, utils_5.getClientPersistedDirectiveNodes)(parentDefinitionData),
                     });
                     break;
-                case graphql_1.Kind.UNION_TYPE_DEFINITION:
+                }
+                case graphql_1.Kind.UNION_TYPE_DEFINITION: {
                     parentDefinitionData.node.types = (0, utils_3.mapToArrayOfValues)(parentDefinitionData.memberByMemberTypeName);
                     this.routerDefinitions.push(this.getNodeForRouterSchemaByData(parentDefinitionData));
                     if ((0, utils_5.isNodeDataInaccessible)(parentDefinitionData)) {
@@ -1562,6 +1598,7 @@ class FederationFactory {
                         types: clientMembers,
                     });
                     break;
+                }
             }
         }
     }
@@ -2136,15 +2173,18 @@ class FederationFactory {
                 switch (parentDefinitionData.kind) {
                     case graphql_1.Kind.SCALAR_TYPE_DEFINITION:
                     // intentional fallthrough
-                    case graphql_1.Kind.UNION_TYPE_DEFINITION:
-                        continue;
-                    case graphql_1.Kind.ENUM_TYPE_DEFINITION:
+                    case graphql_1.Kind.UNION_TYPE_DEFINITION: {
+                        break;
+                    }
+                    case graphql_1.Kind.ENUM_TYPE_DEFINITION: {
                         this.handleChildTagExclusions(parentDefinitionData, parentDefinitionData.enumValueDataByValueName, parentTagData.childTagDataByChildName, contractTagOptions.tagNamesToExclude);
                         break;
-                    case graphql_1.Kind.INPUT_OBJECT_TYPE_DEFINITION:
+                    }
+                    case graphql_1.Kind.INPUT_OBJECT_TYPE_DEFINITION: {
                         this.handleChildTagExclusions(parentDefinitionData, parentDefinitionData.inputValueDataByName, parentTagData.childTagDataByChildName, contractTagOptions.tagNamesToExclude);
                         break;
-                    default:
+                    }
+                    default: {
                         let accessibleFields = parentDefinitionData.fieldDataByName.size;
                         for (const [fieldName, childTagData] of parentTagData.childTagDataByChildName) {
                             const fieldData = (0, utils_6.getOrThrowError)(parentDefinitionData.fieldDataByName, fieldName, `${parentTypeName}.fieldDataByFieldName`);
@@ -2160,12 +2200,12 @@ class FederationFactory {
                                 accessibleFields -= 1;
                                 continue;
                             }
-                            for (const [argumentName, tagNames] of childTagData.tagNamesByArgumentName) {
+                            for (const [argumentName, argTagNames] of childTagData.tagNamesByArgumentName) {
                                 const inputValueData = (0, utils_6.getOrThrowError)(fieldData.argumentDataByName, argumentName, `${fieldName}.argumentDataByArgumentName`);
                                 if ((0, utils_5.isNodeDataInaccessible)(inputValueData)) {
                                     continue;
                                 }
-                                if (!tagNames.isDisjointFrom(tagNames)) {
+                                if (!contractTagOptions.tagNamesToExclude.isDisjointFrom(argTagNames)) {
                                     (0, utils_6.getValueOrDefault)(inputValueData.persistedDirectivesData.directivesByDirectiveName, string_constants_2.INACCESSIBLE, () => [(0, utils_6.generateSimpleDirective)(string_constants_2.INACCESSIBLE)]);
                                     this.inaccessibleCoords.add(inputValueData.federatedCoords);
                                 }
@@ -2177,6 +2217,7 @@ class FederationFactory {
                             ]);
                             this.inaccessibleCoords.add(parentTypeName);
                         }
+                    }
                 }
             }
         }