@reventlessdev/reventless-graphql-server 1.0.0-alpha.31 → 1.0.0-alpha.32

package/CHANGELOG.md

@@ -3,6 +3,13 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# 1.0.0-alpha.32 (2026-07-27)
+
+### Bug Fixes
+
+* **graphql:** keep a repeat mutation registration from duplicating SDL ([b54e844](https://github.com/ReventlessDev/reventless-core/commit/b54e8446236e16196a40a1a7df9f7ba84b03b167))
+
+
 # 1.0.0-alpha.31 (2026-07-27)
 
 **Note:** Version bump only for package @reventlessdev/reventless-graphql-server

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reventlessdev/reventless-graphql-server",
-  "version": "1.0.0-alpha.31",
+  "version": "1.0.0-alpha.32",
   "description": "Transport-neutral GraphQL server runtime for Reventless: schema-from-fragments, resolver registration, an HTTP + graphql-ws listener, and a subscription fan-out bridge with a pluggable PubSub backend",
   "license": "Apache-2.0",
   "dependencies": {

package/src/GraphQL_ServerInstance.res

@@ -85,7 +85,16 @@ let make = (~label: string="GraphQL"): t => {
   let lastFullSdl: ref<option<string>> = ref(None)
 
   let registerMutations = (~sdlFields: array<string>, ~resolvers: dict<resolverFn>) => {
-    mutationFields.contents = mutationFields.contents->Array.concat(sdlFields)
+    // A mutation field has two legitimate writers: the admin base fragment
+    // carries the Plugin aggregate's lifecycle fields, and the aggregate
+    // auto-flow registers the same fields when the admin is constructed again
+    // for a later deploy onto the same server. buildASTSchema rejects a schema
+    // that defines a field twice, so keep the first SDL definition and drop
+    // repeats by field name — mirroring the resolver dict, where a repeat
+    // registration overwrites by key (last resolver wins).
+    let existing = Set.fromArray(mutationFields.contents->Array.map(extractFieldName))
+    let fresh = sdlFields->Array.filter(f => !(existing->Set.has(extractFieldName(f))))
+    mutationFields.contents = mutationFields.contents->Array.concat(fresh)
     resolvers->Dict.toArray->Array.forEach(((k, v)) => mutationResolvers.contents->Dict.set(k, v))
   }
 

package/src/GraphQL_ServerInstance.res.mjs

@@ -52,7 +52,9 @@ function make(labelOpt) {
     contents: undefined
   };
   let registerMutations = (sdlFields, resolvers) => {
-    mutationFields.contents = mutationFields.contents.concat(sdlFields);
+    let existing = new Set(mutationFields.contents.map(extractFieldName));
+    let fresh = sdlFields.filter(f => !existing.has(extractFieldName(f)));
+    mutationFields.contents = mutationFields.contents.concat(fresh);
     Object.entries(resolvers).forEach(param => {
       mutationResolvers.contents[param[0]] = param[1];
     });