@travetto/model-mysql 5.0.0-rc.0 → 5.0.0-rc.10

package/README.md

@@ -42,11 +42,12 @@ export class Init {
 }
 ```
 
-where the [SQLModelConfig](https://github.com/travetto/travetto/tree/main/module/model-sql/src/config.ts#L7) is defined by:
+where the [SQLModelConfig](https://github.com/travetto/travetto/tree/main/module/model-sql/src/config.ts#L8) is defined by:
 
 **Code: Structure of SQLModelConfig**
 ```typescript
 import { Config } from '@travetto/config';
+import { asFull } from '@travetto/runtime';
 
 /**
  * SQL Model Config
@@ -88,8 +89,7 @@ export class SQLModelConfig<T extends {} = {}> {
   /**
    * Raw client options
    */
-
-  options: T = {} as T;
+  options: T = asFull({});
 }
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/model-mysql",
-  "version": "5.0.0-rc.0",
+  "version": "5.0.0-rc.10",
   "description": "MySQL backing for the travetto model module, with real-time modeling support for SQL schemas.",
   "keywords": [
     "sql",
@@ -27,15 +27,15 @@
     "directory": "module/model-mysql"
   },
   "dependencies": {
-    "@travetto/config": "^5.0.0-rc.0",
-    "@travetto/context": "^5.0.0-rc.0",
-    "@travetto/model": "^5.0.0-rc.0",
-    "@travetto/model-query": "^5.0.0-rc.0",
-    "@travetto/model-sql": "^5.0.0-rc.0",
+    "@travetto/config": "^5.0.0-rc.10",
+    "@travetto/context": "^5.0.0-rc.9",
+    "@travetto/model": "^5.0.0-rc.10",
+    "@travetto/model-query": "^5.0.0-rc.10",
+    "@travetto/model-sql": "^5.0.0-rc.10",
     "mysql2": "^3.10.2"
   },
   "peerDependencies": {
-    "@travetto/command": "^5.0.0-rc.0"
+    "@travetto/command": "^5.0.0-rc.9"
   },
   "peerDependenciesMeta": {
     "@travetto/command": {

package/src/connection.ts

@@ -1,9 +1,10 @@
-import mysql, { OkPacket, ResultSetHeader } from 'mysql2';
+import { createPool } from 'mysql2';
 
-import { ShutdownManager } from '@travetto/base';
+import { castTo, ShutdownManager } from '@travetto/runtime';
 import { AsyncContext } from '@travetto/context';
 import { ExistsError } from '@travetto/model';
 import { Connection, SQLModelConfig } from '@travetto/model-sql';
+import { PoolConnection, Pool, OkPacket, ResultSetHeader } from 'mysql2/promise';
 
 function isSimplePacket(o: unknown): o is OkPacket | ResultSetHeader {
   return o !== null && o !== undefined && typeof o === 'object' && 'constructor' in o && (
@@ -14,9 +15,9 @@ function isSimplePacket(o: unknown): o is OkPacket | ResultSetHeader {
 /**
  * Connection support for mysql
  */
-export class MySQLConnection extends Connection<mysql.PoolConnection> {
+export class MySQLConnection extends Connection<PoolConnection> {
 
-  #pool: mysql.Pool;
+  #pool: Pool;
   #config: SQLModelConfig;
 
   constructor(
@@ -28,7 +29,7 @@ export class MySQLConnection extends Connection<mysql.PoolConnection> {
   }
 
   async init(): Promise<void> {
-    this.#pool = mysql.createPool({
+    this.#pool = createPool({
       user: this.#config.user,
       password: this.#config.password,
       database: this.#config.database,
@@ -37,10 +38,10 @@ export class MySQLConnection extends Connection<mysql.PoolConnection> {
       timezone: '+00:00',
       typeCast: this.typeCast.bind(this),
       ...(this.#config.options || {})
-    });
+    }).promise();
 
     // Close mysql
-    ShutdownManager.onGracefulShutdown(() => new Promise<void>(r => this.#pool.end(() => r())), this);
+    ShutdownManager.onGracefulShutdown(() => this.#pool.end(), this);
   }
 
   /**
@@ -58,39 +59,34 @@ export class MySQLConnection extends Connection<mysql.PoolConnection> {
     return res;
   }
 
-  async execute<T = unknown>(conn: mysql.Connection, query: string): Promise<{ count: number, records: T[] }> {
-    return new Promise<{ count: number, records: T[] }>((res, rej) => {
-      console.debug('Executing Query', { query });
-      conn.query(query, (err, results, fields) => {
-        if (err) {
-          console.debug('Failed query', { error: err, query });
-          if (err.message.startsWith('Duplicate entry')) {
-            rej(new ExistsError('query', query));
-          } else {
-            rej(err);
-          }
-        } else {
-          if (isSimplePacket(results)) {
-            return res({ records: [], count: results.affectedRows });
-          } else {
-            if (isSimplePacket(results[0])) {
-              return res({ records: [], count: results[0].affectedRows });
-            }
-            // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-            const records: T[] = [...results].map(v => ({ ...v }) as T);
-            return res({ records, count: records.length });
-          }
+  async execute<T = unknown>(conn: PoolConnection, query: string): Promise<{ count: number, records: T[] }> {
+    console.debug('Executing Query', { query });
+    try {
+      const [results,] = await conn.query(query);
+      if (isSimplePacket(results)) {
+        return { records: [], count: results.affectedRows };
+      } else {
+        if (isSimplePacket(results[0])) {
+          return { records: [], count: results[0].affectedRows };
         }
-      });
-    });
+        const records: T[] = [...results].map(v => castTo({ ...v }));
+        return { records, count: records.length };
+      }
+    } catch (err) {
+      console.debug('Failed query', { error: err, query });
+      if (err instanceof Error && err.message.startsWith('Duplicate entry')) {
+        throw new ExistsError('query', query);
+      } else {
+        throw err;
+      }
+    }
   }
 
-  acquire(): Promise<mysql.PoolConnection> {
-    return new Promise<mysql.PoolConnection>((res, rej) =>
-      this.#pool.getConnection((err, conn) => err ? rej(err) : res(conn)));
+  acquire(): Promise<PoolConnection> {
+    return this.#pool.getConnection();
   }
 
-  release(conn: mysql.PoolConnection): void {
+  release(conn: PoolConnection): void {
     conn.release();
   }
 }

package/src/dialect.ts

@@ -2,7 +2,7 @@ import { FieldConfig } from '@travetto/schema';
 import { Injectable } from '@travetto/di';
 import { AsyncContext } from '@travetto/context';
 import { WhereClause } from '@travetto/model-query';
-import { Class } from '@travetto/base';
+import { castTo, Class } from '@travetto/runtime';
 import { ModelType } from '@travetto/model';
 import { SQLModelConfig, SQLDialect } from '@travetto/model-sql';
 import { VisitStack } from '@travetto/model-sql/src/internal/util';
@@ -80,8 +80,7 @@ export class MySQLDialect extends SQLDialect {
    * Define column modification
    */
   getModifyColumnSQL(stack: VisitStack[]): string {
-    // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
-    const field = stack[stack.length - 1] as FieldConfig;
+    const field: FieldConfig = castTo(stack[stack.length - 1]);
     return `ALTER TABLE ${this.parentTable(stack)} MODIFY COLUMN ${this.getColumnDefinition(field)};`;
   }