@theguild/federation-composition 0.5.0 → 0.5.1-rc-20231214204251-35140eb

package/README.md

@@ -98,6 +98,21 @@ pnpm test
 - Look for `skipIf` or `skip` in the tests.
 - Refactor code (piece by piece) if you feel like it.
 
+### Compatibility
+
+The lack of a publicly available specification for Apollo Federation, coupled with the non
+open-source license of the Apollo Composition library, makes it difficult or even impossible to
+assure complete compatibility of our open-source composition library.
+
+Given that Apollo tools utilize their composition library, there is a potential for conflicting
+results between our composition library and Apollo's. This may lead to variations in the supergraph,
+differing composition errors, or, in some cases, conflicting composition outcomes.
+
+We are working to ensure that our composition library is as compatible as possible with Apollo's and
+will continue to do so as we learn more about the Federation specification.
+
+Your feedback and bug reports are welcome and appreciated.
+
 ## Supergraph SDL Composition
 
 ✅ Done

package/cjs/supergraph/composition/object-type.js

@@ -4,11 +4,20 @@ exports.objectTypeBuilder = exports.isRealExtension = void 0;
 const ast_js_1 = require("./ast.js");
 const common_js_1 = require("./common.js");
 function isRealExtension(meta, version) {
-    return meta.extension
-        ? meta.extensionType !== '@extends' && version === 'v1.0'
-            ? false
-            : true
-        : false;
+    const hasExtendsDirective = meta.extensionType === '@extends';
+    if (meta.extension) {
+        if (version === 'v1.0' && !hasExtendsDirective) {
+            return false;
+        }
+        if (hasExtendsDirective) {
+            return true;
+        }
+        if (meta.hasDefinition) {
+            return false;
+        }
+        return true;
+    }
+    return false;
 }
 exports.isRealExtension = isRealExtension;
 function objectTypeBuilder() {
@@ -42,6 +51,7 @@ function objectTypeBuilder() {
             }
             type.interfaces.forEach(interfaceName => objectTypeState.interfaces.add(interfaceName));
             objectTypeState.byGraph.set(graph.id, {
+                hasDefinition: isDefinition,
                 extension: type.extension,
                 extensionType: type.extensionType,
                 external: type.external,
@@ -50,6 +60,7 @@ function objectTypeBuilder() {
                 shareable: type.shareable,
                 interfaces: type.interfaces,
             });
+            const typeInGraph = objectTypeState.byGraph.get(graph.id);
             for (const field of type.fields.values()) {
                 const fieldState = getOrCreateField(objectTypeState, field.name, field.type);
                 field.tags.forEach(tag => fieldState.tags.add(tag));
@@ -58,7 +69,7 @@ function objectTypeBuilder() {
                     fieldState.usedAsKey = true;
                 }
                 const isExternal = graph.version === 'v1.0'
-                    ? field.external && isRealExtension(type, graph.version)
+                    ? field.external && isRealExtension(typeInGraph, graph.version)
                     : field.external;
                 const shouldForceType = !usedAsKey && !isExternal && !fieldState.internal.seenNonExternal;
                 const shouldChangeType = shouldForceType ||

package/esm/supergraph/composition/object-type.js

@@ -1,11 +1,20 @@
 import { createObjectTypeNode } from './ast.js';
 import { convertToConst } from './common.js';
 export function isRealExtension(meta, version) {
-    return meta.extension
-        ? meta.extensionType !== '@extends' && version === 'v1.0'
-            ? false
-            : true
-        : false;
+    const hasExtendsDirective = meta.extensionType === '@extends';
+    if (meta.extension) {
+        if (version === 'v1.0' && !hasExtendsDirective) {
+            return false;
+        }
+        if (hasExtendsDirective) {
+            return true;
+        }
+        if (meta.hasDefinition) {
+            return false;
+        }
+        return true;
+    }
+    return false;
 }
 export function objectTypeBuilder() {
     return {
@@ -38,6 +47,7 @@ export function objectTypeBuilder() {
             }
             type.interfaces.forEach(interfaceName => objectTypeState.interfaces.add(interfaceName));
             objectTypeState.byGraph.set(graph.id, {
+                hasDefinition: isDefinition,
                 extension: type.extension,
                 extensionType: type.extensionType,
                 external: type.external,
@@ -46,6 +56,7 @@ export function objectTypeBuilder() {
                 shareable: type.shareable,
                 interfaces: type.interfaces,
             });
+            const typeInGraph = objectTypeState.byGraph.get(graph.id);
             for (const field of type.fields.values()) {
                 const fieldState = getOrCreateField(objectTypeState, field.name, field.type);
                 field.tags.forEach(tag => fieldState.tags.add(tag));
@@ -54,7 +65,7 @@ export function objectTypeBuilder() {
                     fieldState.usedAsKey = true;
                 }
                 const isExternal = graph.version === 'v1.0'
-                    ? field.external && isRealExtension(type, graph.version)
+                    ? field.external && isRealExtension(typeInGraph, graph.version)
                     : field.external;
                 const shouldForceType = !usedAsKey && !isExternal && !fieldState.internal.seenNonExternal;
                 const shouldChangeType = shouldForceType ||

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theguild/federation-composition",
-  "version": "0.5.0",
+  "version": "0.5.1-rc-20231214204251-35140eb",
   "description": "Open Source Composition library for Apollo Federation",
   "peerDependencies": {
     "graphql": "^16.0.0"

package/typings/supergraph/composition/object-type.d.cts

@@ -55,6 +55,7 @@ export type ObjectTypeFieldArgState = {
     };
 };
 export type ObjectTypeStateInGraph = {
+    hasDefinition: boolean;
     extension: boolean;
     extensionType?: '@extends' | 'extend';
     external: boolean;

package/typings/supergraph/composition/object-type.d.ts

@@ -55,6 +55,7 @@ export type ObjectTypeFieldArgState = {
     };
 };
 export type ObjectTypeStateInGraph = {
+    hasDefinition: boolean;
     extension: boolean;
     extensionType?: '@extends' | 'extend';
     external: boolean;