@platformatic/sql-graphql 3.52.0 → 3.52.2

package/lib/entity-to-type.js

@@ -107,20 +107,23 @@ export function constructGraph (app, entity, opts, ignore) {
   resolvers.Mutation = resolvers.Mutation || {}
   loaders.Query = loaders.Query || {}
 
-  const getBy = camelcase(
-    [
-      'get',
-      singular,
-      ...primaryKeys
-        .map((key, i) => {
-          if (i === 0) {
-            return ['by', key]
-          }
-          return ['and', key]
-        })
-        .flat()
-    ].join('_')
-  )
+  let getBy
+  if (primaryKeys.length > 0) {
+    getBy = camelcase(
+      [
+        'get',
+        singular,
+        ...primaryKeys
+          .map((key, i) => {
+            if (i === 0) {
+              return ['by', key]
+            }
+            return ['and', key]
+          })
+          .flat()
+      ].join('_')
+    )
+  }
 
   const whereFields = Object.keys(fields).reduce((acc, field) => {
     let graphqlFields
@@ -172,29 +175,31 @@ export function constructGraph (app, entity, opts, ignore) {
     }
   })
 
-  queryTopFields[getBy] = {
-    type,
-    args: {
-      ...primaryKeys.reduce((acc, primaryKey) => {
-        acc[primaryKey] = { type: new GraphQLNonNull(fields[primaryKey].type) }
-        return acc
-      }, {})
+  if (getBy) {
+    queryTopFields[getBy] = {
+      type,
+      args: {
+        ...primaryKeys.reduce((acc, primaryKey) => {
+          acc[primaryKey] = { type: new GraphQLNonNull(fields[primaryKey].type) }
+          return acc
+        }, {})
+      }
     }
-  }
-  loaders.Query[getBy] = {
-    loader (queries, ctx) {
-      const keys = []
-      for (const query of queries) {
-        const pairs = []
-        for (const key of primaryKeys) {
-          pairs.push({ key, value: query.params[key] })
+    loaders.Query[getBy] = {
+      loader (queries, ctx) {
+        const keys = []
+        for (const query of queries) {
+          const pairs = []
+          for (const key of primaryKeys) {
+            pairs.push({ key, value: query.params[key] })
+          }
+          keys.push(pairs)
         }
-        keys.push(pairs)
+        return loadMany(keys, queries, ctx)
+      },
+      opts: {
+        cache: false
       }
-      return loadMany(keys, queries, ctx)
-    },
-    opts: {
-      cache: false
     }
   }
 
@@ -275,68 +280,73 @@ export function constructGraph (app, entity, opts, ignore) {
     return { total }
   }
 
-  const save = camelcase(['save', singular])
+  // Views are read-only: skip all mutations
+  if (!entity.isView) {
+    const save = camelcase(['save', singular])
 
-  mutationTopFields[save] = {
-    type,
-    args: {
-      input: { type: new GraphQLNonNull(inputType) }
+    mutationTopFields[save] = {
+      type,
+      args: {
+        input: { type: new GraphQLNonNull(inputType) }
+      }
     }
-  }
 
-  resolvers.Mutation[save] = async (_, { input }, ctx, info) => {
-    const fields = fromSelectionSet(info.fieldNodes[0].selectionSet)
-    return entity.save({ input, ctx, fields: [...fields, ...relationalFields] })
-  }
+    resolvers.Mutation[save] = async (_, { input }, ctx, info) => {
+      const fields = fromSelectionSet(info.fieldNodes[0].selectionSet)
+      return entity.save({ input, ctx, fields: [...fields, ...relationalFields] })
+    }
 
-  const insert = camelcase(['insert', plural])
+    const insert = camelcase(['insert', plural])
 
-  mutationTopFields[insert] = {
-    type: new GraphQLList(type),
-    args: {
-      inputs: { type: new GraphQLNonNull(new GraphQLList(inputType)) }
+    mutationTopFields[insert] = {
+      type: new GraphQLList(type),
+      args: {
+        inputs: { type: new GraphQLNonNull(new GraphQLList(inputType)) }
+      }
     }
-  }
 
-  resolvers.Mutation[insert] = (_, { inputs }, ctx, info) => {
-    const fields = fromSelectionSet(info.fieldNodes[0].selectionSet)
-    return entity.insert({ inputs, ctx, fields: [...fields, ...relationalFields] })
-  }
+    resolvers.Mutation[insert] = (_, { inputs }, ctx, info) => {
+      const fields = fromSelectionSet(info.fieldNodes[0].selectionSet)
+      return entity.insert({ inputs, ctx, fields: [...fields, ...relationalFields] })
+    }
 
-  const deleteKey = camelcase(['delete', plural])
-  mutationTopFields[deleteKey] = {
-    type: new GraphQLList(type),
-    args: {
-      where: { type: whereArgType }
+    const deleteKey = camelcase(['delete', plural])
+    mutationTopFields[deleteKey] = {
+      type: new GraphQLList(type),
+      args: {
+        where: { type: whereArgType }
+      }
     }
-  }
 
-  resolvers.Mutation[deleteKey] = (_, args, ctx, info) => {
-    const fields = info.fieldNodes[0].selectionSet.selections.map(s => s.name.value)
-    return entity.delete({ ...args, fields: [...fields, ...relationalFields], ctx })
+    resolvers.Mutation[deleteKey] = (_, args, ctx, info) => {
+      const fields = info.fieldNodes[0].selectionSet.selections.map(s => s.name.value)
+      return entity.delete({ ...args, fields: [...fields, ...relationalFields], ctx })
+    }
   }
 
-  federationReplacements.push({
-    find: new RegExp(`type ${entityName}`),
-    replace: `type ${entityName} @key(fields: "${primaryKeys}")`
-  })
+  if (primaryKeys.length > 0) {
+    federationReplacements.push({
+      find: new RegExp(`type ${entityName}`),
+      replace: `type ${entityName} @key(fields: "${primaryKeys}")`
+    })
 
-  if (federationMetadata) {
-    loaders[entityName] = loaders[entityName] || {}
-    loaders[entityName].__resolveReference = {
-      loader (queries, ctx) {
-        const keys = []
-        for (const { obj } of queries) {
-          const pairs = []
-          for (const key of primaryKeys) {
-            pairs.push({ key, value: obj[key] })
+    if (federationMetadata) {
+      loaders[entityName] = loaders[entityName] || {}
+      loaders[entityName].__resolveReference = {
+        loader (queries, ctx) {
+          const keys = []
+          for (const { obj } of queries) {
+            const pairs = []
+            for (const key of primaryKeys) {
+              pairs.push({ key, value: obj[key] })
+            }
+            keys.push(pairs)
           }
-          keys.push(pairs)
+          return loadMany(keys, queries, ctx)
+        },
+        opts: {
+          cache: false
         }
-        return loadMany(keys, queries, ctx)
-      },
-      opts: {
-        cache: false
       }
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/sql-graphql",
-  "version": "3.52.0",
+  "version": "3.52.2",
   "description": "Map SQL dbs to GraphQL",
   "main": "index.js",
   "type": "module",
@@ -27,9 +27,9 @@
     "typescript": "^5.5.4",
     "why-is-node-running": "^2.2.2",
     "ws": "^8.16.0",
-    "@platformatic/sql-events": "3.52.0",
-    "@platformatic/sql-mapper": "3.52.0",
-    "@platformatic/telemetry": "3.52.0"
+    "@platformatic/telemetry": "3.52.2",
+    "@platformatic/sql-events": "3.52.2",
+    "@platformatic/sql-mapper": "3.52.2"
   },
   "dependencies": {
     "@fastify/error": "^4.0.0",
@@ -41,7 +41,7 @@
     "graphql-type-json": "^0.3.2",
     "inflected": "^2.1.0",
     "mercurius": "^16.0.0",
-    "@platformatic/foundation": "3.52.0"
+    "@platformatic/foundation": "3.52.2"
   },
   "engines": {
     "node": ">=22.19.0"