@smartive/graphql-magic 17.2.0 → 17.2.2

package/CHANGELOG.md

@@ -1,6 +1,6 @@
-# [17.2.0](https://github.com/smartive/graphql-magic/compare/v17.1.0...v17.2.0) (2025-03-05)
+## [17.2.2](https://github.com/smartive/graphql-magic/compare/v17.2.1...v17.2.2) (2025-04-09)
 
 
-### Features
+### Bug Fixes
 
-* Add indices to foreign keys ([#249](https://github.com/smartive/graphql-magic/issues/249)) ([652e612](https://github.com/smartive/graphql-magic/commit/652e612b2ecceec62830b8b6b3d0b4b15b09b6c7))
+* allow void as return type for mutation hook ([#264](https://github.com/smartive/graphql-magic/issues/264)) ([2b2dbef](https://github.com/smartive/graphql-magic/commit/2b2dbefbec79505a2889bb67b9c88361459b905d))

package/dist/bin/gqm.cjs

@@ -2040,6 +2040,14 @@ var VISITOR = {
         return Symbol;
       case "Models":
         return Models;
+      case "Object":
+        return Object;
+      case "Array":
+        return Array;
+      case "Boolean":
+        return Boolean;
+      case "Number":
+        return Number;
     }
     const definitionNodes = node.getDefinitionNodes();
     if (!definitionNodes.length) {
@@ -2081,6 +2089,18 @@ var VISITOR = {
           case "toLowerCase":
             return target[name2].bind(target);
         }
+      } else if (typeof target === "function") {
+        const name2 = node.getName();
+        if (target === Object) {
+          switch (name2) {
+            case "keys":
+            case "values":
+            case "entries":
+            case "assign":
+            case "hasOwnProperty":
+              return target[name2].bind(target);
+          }
+        }
       }
       throw new Error(`Cannot handle method ${node.getName()} on type ${typeof target}`);
     }

package/dist/esm/models/mutation-hook.d.ts

@@ -7,4 +7,4 @@ export type MutationHook<DateType extends AnyDateType = AnyDateType> = (model: E
     input: Entity;
     normalizedInput: Entity;
     next: Entity;
-}, ctx: Context<DateType>) => Promise<void>;
+}, ctx: Context<DateType>) => Promise<void> | void;

package/docs/docs/11-migration-guides.md

@@ -0,0 +1,56 @@
+# Migration Guides
+
+## Upgrading to v17.2.0
+
+From now on, foreign keys will be indexed by default. For existing projects, you'll need to add indices manually to your existing foreign keys.
+
+Here's a sample script to generate the migration:
+
+```ts
+import { execSync } from 'child_process';
+import { writeFileSync } from 'fs';
+import { models } from '../../../src/config/models';
+
+const start = async () => {
+  const instructions: string[] = [];
+
+  instructions.push(`import { Knex } from 'knex';
+
+export const up = async (knex: Knex) => {`);
+
+  for (const model of models.entities) {
+    if (model.fields.some((field) => field.kind === 'relation')) {
+      instructions.push(`await knex.schema.alterTable('${model.name}', (table) => {`);
+      for (const field of model.fields) {
+        if (field.kind === 'relation') {
+          instructions.push(`table.index('${field.foreignKey || field.name + 'Id'}');`);
+        }
+      }
+      instructions.push(`});`);
+    }
+  }
+
+  instructions.push(`};`);
+
+  instructions.push(`export const down = async (knex: Knex) => {`);
+
+  for (const model of models.entities) {
+    if (model.fields.some((field) => field.kind === 'relation')) {
+      instructions.push(`await knex.schema.alterTable('${model.name}', (table) => {`);
+      for (const field of model.fields) {
+        if (field.kind === 'relation') {
+          instructions.push(`table.dropIndex('${field.foreignKey || field.name + 'Id'}');`);
+        }
+      }
+      instructions.push(`});`);
+    }
+  }
+
+  instructions.push(`};`);
+
+  writeFileSync('migrations/20250305130109_create-foreign-key-indices.ts', instructions.join('\n'));
+  execSync('npx eslint --fix migrations/20250305130109_create-foreign-key-indices.ts');
+};
+
+void start();
+```

package/docs/package-lock.json

@@ -20,7 +20,7 @@
         "@docusaurus/module-type-aliases": "3.7.0",
         "@docusaurus/tsconfig": "3.7.0",
         "@docusaurus/types": "3.7.0",
-        "typescript": "5.8.2"
+        "typescript": "5.8.3"
       },
       "engines": {
         "node": ">=22.0"
@@ -16466,9 +16466,9 @@
       }
     },
     "node_modules/typescript": {
-      "version": "5.8.2",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.8.2.tgz",
-      "integrity": "sha512-aJn6wq13/afZp/jT9QZmwEjDqqvSGp1VT5GVg+f/t6/oVyrgXM6BY1h9BRh/O5p3PlUPAe+WuiEZOmb/49RqoQ==",
+      "version": "5.8.3",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.8.3.tgz",
+      "integrity": "sha512-p1diW6TqL9L07nNxvRMM7hMMw4c5XOo/1ibL4aAIGmSAt9slTE1Xgw5KWuof2uTOvCg9BY7ZRi+GaF+7sfgPeQ==",
       "license": "Apache-2.0",
       "bin": {
         "tsc": "bin/tsc",

package/docs/package.json

@@ -27,7 +27,7 @@
     "@docusaurus/module-type-aliases": "3.7.0",
     "@docusaurus/tsconfig": "3.7.0",
     "@docusaurus/types": "3.7.0",
-    "typescript": "5.8.2"
+    "typescript": "5.8.3"
   },
   "browserslist": {
     "production": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartive/graphql-magic",
-  "version": "17.2.0",
+  "version": "17.2.2",
   "description": "",
   "source": "src/index.ts",
   "type": "module",
@@ -69,18 +69,18 @@
     "@smartive/prettier-config": "3.1.2",
     "@types/jest": "29.5.14",
     "@types/lodash": "4.17.16",
-    "@types/luxon": "3.4.2",
+    "@types/luxon": "3.6.2",
     "@types/pg": "8.11.11",
     "@types/uuid": "10.0.0",
     "create-ts-index": "1.14.0",
     "del-cli": "6.0.0",
-    "esbuild": "0.25.0",
-    "eslint": "9.21.0",
+    "esbuild": "0.25.2",
+    "eslint": "9.24.0",
     "jest": "29.7.0",
     "mock-knex": "0.4.13",
     "prettier": "3.5.3",
-    "ts-jest": "29.2.6",
+    "ts-jest": "29.3.1",
     "ts-node": "10.9.2",
-    "typescript": "5.8.2"
+    "typescript": "5.8.3"
   }
 }

package/src/bin/gqm/static-eval.ts

@@ -57,6 +57,14 @@ const VISITOR: Visitor<unknown, Dictionary<unknown>> = {
         return Symbol;
       case 'Models':
         return Models;
+      case 'Object':
+        return Object;
+      case 'Array':
+        return Array;
+      case 'Boolean':
+        return Boolean;
+      case 'Number':
+        return Number;
     }
     const definitionNodes = node.getDefinitionNodes();
     if (!definitionNodes.length) {
@@ -101,6 +109,18 @@ const VISITOR: Visitor<unknown, Dictionary<unknown>> = {
           case 'toLowerCase':
             return target[name].bind(target);
         }
+      } else if (typeof target === 'function') {
+        const name = node.getName();
+        if (target === Object) {
+          switch (name) {
+            case 'keys':
+            case 'values':
+            case 'entries':
+            case 'assign':
+            case 'hasOwnProperty':
+              return target[name].bind(target);
+          }
+        }
       }
       throw new Error(`Cannot handle method ${node.getName()} on type ${typeof target}`);
     }

package/src/models/mutation-hook.ts

@@ -11,4 +11,4 @@ export type MutationHook<DateType extends AnyDateType = AnyDateType> = (
   when: 'before' | 'after',
   data: { prev: Entity; input: Entity; normalizedInput: Entity; next: Entity },
   ctx: Context<DateType>,
-) => Promise<void>;
+) => Promise<void> | void;