@travetto/model-sql 7.0.0-rc.1 → 7.0.0-rc.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@travetto/model-sql",
-  "version": "7.0.0-rc.1",
+  "version": "7.0.0-rc.3",
   "description": "SQL backing for the travetto model module, with real-time modeling support for SQL schemas.",
   "keywords": [
     "sql",
@@ -27,14 +27,14 @@
     "directory": "module/model-sql"
   },
   "dependencies": {
-    "@travetto/cli": "^7.0.0-rc.1",
-    "@travetto/config": "^7.0.0-rc.1",
-    "@travetto/context": "^7.0.0-rc.1",
-    "@travetto/model": "^7.0.0-rc.1",
-    "@travetto/model-query": "^7.0.0-rc.1"
+    "@travetto/cli": "^7.0.0-rc.3",
+    "@travetto/config": "^7.0.0-rc.3",
+    "@travetto/context": "^7.0.0-rc.3",
+    "@travetto/model": "^7.0.0-rc.3",
+    "@travetto/model-query": "^7.0.0-rc.3"
   },
   "peerDependencies": {
-    "@travetto/test": "^7.0.0-rc.1"
+    "@travetto/test": "^7.0.0-rc.3"
   },
   "peerDependenciesMeta": {
     "@travetto/test": {

package/src/config.ts

@@ -1,5 +1,5 @@
 import { Config } from '@travetto/config';
-import { asFull } from '@travetto/runtime';
+import { asFull, Runtime } from '@travetto/runtime';
 
 /**
  * SQL Model Config
@@ -17,11 +17,11 @@ export class SQLModelConfig<T extends {} = {}> {
   /**
    * Username
    */
-  user = '';
+  user = Runtime.production ? '' : 'travetto';
   /**
    * Password
    */
-  password = '';
+  password = Runtime.production ? '' : 'travetto';
   /**
    * Table prefix
    */
@@ -31,9 +31,9 @@ export class SQLModelConfig<T extends {} = {}> {
    */
   database = 'app';
   /**
-   * Auto schema creation
+   * Allow storage modification at runtime
    */
-  autoCreate?: boolean;
+  modifyStorage?: boolean;
   /**
    * Db version
    */

package/src/connection/base.ts

@@ -71,23 +71,23 @@ export abstract class Connection<C = unknown> {
   /**
    * Run operation with active connection
    * @param context
-   * @param op
+   * @param operation
    * @param args
    */
-  async runWithActive<R>(op: () => Promise<R>): Promise<R> {
+  async runWithActive<R>(operation: () => Promise<R>): Promise<R> {
     if (this.active) {
-      return op();
+      return operation();
     }
 
     return this.context.run(async () => {
-      let conn;
+      let connection;
       try {
-        conn = await this.acquire();
-        this.#active.set(conn);
-        return await op();
+        connection = await this.acquire();
+        this.#active.set(connection);
+        return await operation();
       } finally {
-        if (conn) {
-          this.release(conn);
+        if (connection) {
+          this.release(connection);
         }
       }
     });
@@ -96,19 +96,19 @@ export abstract class Connection<C = unknown> {
   /**
    * Iterate with active connection
    * @param context
-   * @param op
+   * @param operation
    * @param args
    */
-  async * iterateWithActive<R>(op: () => AsyncIterable<R>): AsyncIterable<R> {
+  async * iterateWithActive<R>(operation: () => AsyncIterable<R>): AsyncIterable<R> {
     if (this.active) {
-      yield* op();
+      yield* operation();
     }
 
     const self = castTo<Connection>(this);
     yield* this.context.iterate(async function* () {
       try {
         self.#active.set(await self.acquire());
-        yield* op();
+        yield* operation();
       } finally {
         if (self.active) {
           self.release(self.active);
@@ -120,26 +120,26 @@ export abstract class Connection<C = unknown> {
   /**
    * Run a function within a valid sql transaction.  Relies on @travetto/context.
    */
-  async runWithTransaction<R>(mode: TransactionType, op: () => Promise<R>): Promise<R> {
+  async runWithTransaction<R>(mode: TransactionType, operation: () => Promise<R>): Promise<R> {
     if (this.activeTx) {
       if (mode === 'isolated' || mode === 'force') {
         const txId = mode === 'isolated' ? `tx${Util.uuid()}` : undefined;
         try {
           await this.startTx(this.active!, txId);
-          const res = await op();
+          const result = await operation();
           await this.commitTx(this.active!, txId);
-          return res;
-        } catch (err) {
+          return result;
+        } catch (error) {
           try { await this.rollbackTx(this.active!, txId); } catch { }
-          throw err;
+          throw error;
         }
       } else {
-        return await op();
+        return await operation();
       }
     } else {
       return this.runWithActive(() => {
         this.#activeTx.set(true);
-        return this.runWithTransaction('force', op);
+        return this.runWithTransaction('force', operation);
       });
     }
   }
@@ -147,42 +147,42 @@ export abstract class Connection<C = unknown> {
   /**
    * Start a transaction
    */
-  async startTx(conn: C, transactionId?: string): Promise<void> {
+  async startTx(connection: C, transactionId?: string): Promise<void> {
     if (transactionId) {
       if (this.nestedTransactions) {
-        await this.execute(conn, this.transactionDialect.beginNested, [transactionId]);
+        await this.execute(connection, this.transactionDialect.beginNested, [transactionId]);
       }
     } else {
       if (this.isolatedTransactions) {
-        await this.execute(conn, this.transactionDialect.isolate);
+        await this.execute(connection, this.transactionDialect.isolate);
       }
-      await this.execute(conn, this.transactionDialect.begin);
+      await this.execute(connection, this.transactionDialect.begin);
     }
   }
 
   /**
    * Commit active transaction
    */
-  async commitTx(conn: C, transactionId?: string): Promise<void> {
+  async commitTx(connection: C, transactionId?: string): Promise<void> {
     if (transactionId) {
       if (this.nestedTransactions) {
-        await this.execute(conn, this.transactionDialect.commitNested, [transactionId]);
+        await this.execute(connection, this.transactionDialect.commitNested, [transactionId]);
       }
     } else {
-      await this.execute(conn, this.transactionDialect.commit);
+      await this.execute(connection, this.transactionDialect.commit);
     }
   }
 
   /**
    * Rollback active transaction
    */
-  async rollbackTx(conn: C, transactionId?: string): Promise<void> {
+  async rollbackTx(connection: C, transactionId?: string): Promise<void> {
     if (transactionId) {
       if (this.isolatedTransactions) {
-        await this.execute(conn, this.transactionDialect.rollbackNested, [transactionId]);
+        await this.execute(connection, this.transactionDialect.rollbackNested, [transactionId]);
       }
     } else {
-      await this.execute(conn, this.transactionDialect.rollback);
+      await this.execute(connection, this.transactionDialect.rollback);
     }
   }
 }

package/src/connection/decorator.ts

@@ -1,11 +1,11 @@
-import { AsyncItrMethodDescriptor, AsyncMethodDescriptor } from '@travetto/runtime';
+import { AsyncIterableMethodDescriptor, AsyncMethodDescriptor } from '@travetto/runtime';
 import { Connection, TransactionType } from './base.ts';
 
 /**
  * Indicating something is aware of connections
  */
 export interface ConnectionAware<C = unknown> {
-  conn: Connection<C>;
+  connection: Connection<C>;
 }
 
 /**
@@ -13,12 +13,12 @@ export interface ConnectionAware<C = unknown> {
  * @kind decorator
  */
 export function Connected() {
-  return function <T extends { conn?: Connection }>(
-    target: T, prop: string | symbol, desc: AsyncMethodDescriptor<T>
+  return function <T extends { connection?: Connection }>(
+    target: T, property: string, descriptor: AsyncMethodDescriptor<T>
   ): void {
-    const og = desc.value!;
-    desc.value = function (...args: unknown[]): ReturnType<typeof og> {
-      return this.conn!.runWithActive(() => og.call(this, ...args));
+    const handle = descriptor.value!;
+    descriptor.value = function (...args: unknown[]): ReturnType<typeof handle> {
+      return this.connection!.runWithActive(() => handle.call(this, ...args));
     };
   };
 }
@@ -28,12 +28,12 @@ export function Connected() {
  * @kind decorator
  */
 export function ConnectedIterator() {
-  return function <T extends { conn?: Connection }>(
-    target: T, prop: string | symbol, desc: AsyncItrMethodDescriptor<T>
+  return function <T extends { connection?: Connection }>(
+    target: T, property: string, descriptor: AsyncIterableMethodDescriptor<T>
   ): void {
-    const og = desc.value!;
-    desc.value = async function* (...args: unknown[]): ReturnType<typeof og> {
-      yield* this.conn!.iterateWithActive(() => og.call(this, ...args));
+    const handle = descriptor.value!;
+    descriptor.value = async function* (...args: unknown[]): ReturnType<typeof handle> {
+      yield* this.connection!.iterateWithActive(() => handle.call(this, ...args));
     };
   };
 }
@@ -43,12 +43,12 @@ export function ConnectedIterator() {
  * @kind decorator
  */
 export function Transactional(mode: TransactionType = 'required') {
-  return function <T extends { conn?: Connection }>(
-    target: unknown, prop: string | symbol, desc: AsyncMethodDescriptor<T>
+  return function <T extends { connection?: Connection }>(
+    target: unknown, property: string, descriptor: AsyncMethodDescriptor<T>
   ): void {
-    const og = desc.value!;
-    desc.value = function (...args: unknown[]): ReturnType<typeof og> {
-      return this.conn!.runWithTransaction(mode, () => og.call(this, ...args));
+    const handle = descriptor.value!;
+    descriptor.value = function (...args: unknown[]): ReturnType<typeof handle> {
+      return this.connection!.runWithTransaction(mode, () => handle.call(this, ...args));
     };
   };
 }