kysely 0.28.6 → 0.28.8

package/dist/cjs/dialect/postgres/postgres-introspector.js

@@ -50,6 +50,8 @@ class PostgresIntrospector {
             .where('ns.nspname', '!=', 'information_schema')
             // Filter out internal cockroachdb schema
             .where('ns.nspname', '!=', 'crdb_internal')
+            // Only schemas where we are allowed access
+            .where((0, sql_js_1.sql) `has_schema_privilege(ns.nspname, 'USAGE')`)
             // No system columns
             .where('a.attnum', '>=', 0)
             .where('a.attisdropped', '!=', true)

package/dist/cjs/expression/expression-builder.d.ts

@@ -108,7 +108,7 @@ export interface ExpressionBuilder<DB, TB extends keyof DB> {
      * )
      * ```
      */
-    <RE extends ReferenceExpression<DB, TB>, OP extends BinaryOperatorExpression, VE extends OperandValueExpressionOrList<DB, TB, RE>>(lhs: RE, op: OP, rhs: VE): ExpressionWrapper<DB, TB, OP extends ComparisonOperator ? SqlBool : ExtractTypeFromReferenceExpression<DB, TB, RE>>;
+    <RE extends ReferenceExpression<DB, TB>, OP extends BinaryOperatorExpression, VE extends OperandValueExpressionOrList<DB, TB, RE>>(lhs: RE, op: OP, rhs: VE): ExpressionWrapper<DB, TB, OP extends ComparisonOperator ? SqlBool : OP extends Expression<infer T> ? unknown extends T ? SqlBool : T : ExtractTypeFromReferenceExpression<DB, TB, RE>>;
     /**
      * Returns a copy of `this` expression builder, for destructuring purposes.
      *

package/dist/cjs/index.d.ts

@@ -50,6 +50,8 @@ export * from './schema/refresh-materialized-view-builder.js';
 export * from './schema/drop-view-builder.js';
 export * from './schema/alter-column-builder.js';
 export * from './dynamic/dynamic.js';
+export * from './dynamic/dynamic-reference-builder.js';
+export * from './dynamic/dynamic-table-builder.js';
 export * from './driver/driver.js';
 export * from './driver/database-connection.js';
 export * from './driver/connection-provider.js';

package/dist/cjs/index.js

@@ -68,6 +68,8 @@ __exportStar(require("./schema/refresh-materialized-view-builder.js"), exports);
 __exportStar(require("./schema/drop-view-builder.js"), exports);
 __exportStar(require("./schema/alter-column-builder.js"), exports);
 __exportStar(require("./dynamic/dynamic.js"), exports);
+__exportStar(require("./dynamic/dynamic-reference-builder.js"), exports);
+__exportStar(require("./dynamic/dynamic-table-builder.js"), exports);
 __exportStar(require("./driver/driver.js"), exports);
 __exportStar(require("./driver/database-connection.js"), exports);
 __exportStar(require("./driver/connection-provider.js"), exports);

package/dist/cjs/kysely.js

@@ -570,14 +570,18 @@ class TransactionBuilder {
                 ...kyselyProps,
                 executor,
             });
+            let transactionBegun = false;
             try {
                 await this.#props.driver.beginTransaction(connection, settings);
+                transactionBegun = true;
                 const result = await callback(transaction);
                 await this.#props.driver.commitTransaction(connection);
                 return result;
             }
             catch (error) {
-                await this.#props.driver.rollbackTransaction(connection);
+                if (transactionBegun) {
+                    await this.#props.driver.rollbackTransaction(connection);
+                }
                 throw error;
             }
         });

package/dist/cjs/migration/migrator.d.ts

@@ -340,12 +340,12 @@ export interface MigrationResult {
      * The execution status.
      *
      *  - `Success` means the migration was successfully executed. Note that
-     *    if any of the later migrations in the {@link MigrationResult.results}
+     *    if any of the later migrations in the {@link MigrationResultSet.results}
      *    list failed (have status `Error`) AND the dialect supports transactional
      *    DDL, even the successfull migrations were rolled back.
      *
      *  - `Error` means the migration failed. In this case the
-     *    {@link MigrationResult.error} contains the error.
+     *    {@link MigrationResultSet.error} contains the error.
      *
      *  - `NotExecuted` means that the migration was supposed to be executed
      *    but wasn't because an earlier migration failed.

package/dist/cjs/query-compiler/default-query-compiler.js

@@ -6,11 +6,11 @@ const insert_query_node_js_1 = require("../operation-node/insert-query-node.js")
 const operation_node_visitor_js_1 = require("../operation-node/operation-node-visitor.js");
 const operator_node_js_1 = require("../operation-node/operator-node.js");
 const parens_node_js_1 = require("../operation-node/parens-node.js");
-const query_node_js_1 = require("../operation-node/query-node.js");
+const raw_node_js_1 = require("../operation-node/raw-node.js");
 const object_utils_js_1 = require("../util/object-utils.js");
 const create_view_node_js_1 = require("../operation-node/create-view-node.js");
 const set_operation_node_js_1 = require("../operation-node/set-operation-node.js");
-const merge_query_node_js_1 = require("../operation-node/merge-query-node.js");
+const when_node_js_1 = require("../operation-node/when-node.js");
 const log_once_js_1 = require("../util/log-once.js");
 const LIT_WRAP_REGEX = /'/g;
 class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisitor {
@@ -150,13 +150,15 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
         this.visitNode(node.having);
     }
     visitInsertQuery(node) {
-        const rootQueryNode = this.nodeStack.find(query_node_js_1.QueryNode.is);
-        const isSubQuery = rootQueryNode !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !parens_node_js_1.ParensNode.is(this.parentNode) &&
+            !raw_node_js_1.RawNode.is(this.parentNode) &&
+            !when_node_js_1.WhenNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery && !merge_query_node_js_1.MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -210,7 +212,7 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery && !merge_query_node_js_1.MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {
@@ -223,12 +225,14 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
         this.compileList(node.values);
     }
     visitDeleteQuery(node) {
-        const isSubQuery = this.nodeStack.find(query_node_js_1.QueryNode.is) !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !parens_node_js_1.ParensNode.is(this.parentNode) &&
+            !raw_node_js_1.RawNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -269,7 +273,7 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {
@@ -530,13 +534,15 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
         this.visitNode(node.groupBy);
     }
     visitUpdateQuery(node) {
-        const rootQueryNode = this.nodeStack.find(query_node_js_1.QueryNode.is);
-        const isSubQuery = rootQueryNode !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !parens_node_js_1.ParensNode.is(this.parentNode) &&
+            !raw_node_js_1.RawNode.is(this.parentNode) &&
+            !when_node_js_1.WhenNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery && !merge_query_node_js_1.MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -587,7 +593,7 @@ class DefaultQueryCompiler extends operation_node_visitor_js_1.OperationNodeVisi
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery && !merge_query_node_js_1.MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {

package/dist/esm/dialect/postgres/postgres-introspector.js

@@ -48,6 +48,8 @@ export class PostgresIntrospector {
             .where('ns.nspname', '!=', 'information_schema')
             // Filter out internal cockroachdb schema
             .where('ns.nspname', '!=', 'crdb_internal')
+            // Only schemas where we are allowed access
+            .where(sql `has_schema_privilege(ns.nspname, 'USAGE')`)
             // No system columns
             .where('a.attnum', '>=', 0)
             .where('a.attisdropped', '!=', true)

package/dist/esm/expression/expression-builder.d.ts

@@ -108,7 +108,7 @@ export interface ExpressionBuilder<DB, TB extends keyof DB> {
      * )
      * ```
      */
-    <RE extends ReferenceExpression<DB, TB>, OP extends BinaryOperatorExpression, VE extends OperandValueExpressionOrList<DB, TB, RE>>(lhs: RE, op: OP, rhs: VE): ExpressionWrapper<DB, TB, OP extends ComparisonOperator ? SqlBool : ExtractTypeFromReferenceExpression<DB, TB, RE>>;
+    <RE extends ReferenceExpression<DB, TB>, OP extends BinaryOperatorExpression, VE extends OperandValueExpressionOrList<DB, TB, RE>>(lhs: RE, op: OP, rhs: VE): ExpressionWrapper<DB, TB, OP extends ComparisonOperator ? SqlBool : OP extends Expression<infer T> ? unknown extends T ? SqlBool : T : ExtractTypeFromReferenceExpression<DB, TB, RE>>;
     /**
      * Returns a copy of `this` expression builder, for destructuring purposes.
      *

package/dist/esm/index.d.ts

@@ -50,6 +50,8 @@ export * from './schema/refresh-materialized-view-builder.js';
 export * from './schema/drop-view-builder.js';
 export * from './schema/alter-column-builder.js';
 export * from './dynamic/dynamic.js';
+export * from './dynamic/dynamic-reference-builder.js';
+export * from './dynamic/dynamic-table-builder.js';
 export * from './driver/driver.js';
 export * from './driver/database-connection.js';
 export * from './driver/connection-provider.js';

package/dist/esm/index.js

@@ -51,6 +51,8 @@ export * from './schema/refresh-materialized-view-builder.js';
 export * from './schema/drop-view-builder.js';
 export * from './schema/alter-column-builder.js';
 export * from './dynamic/dynamic.js';
+export * from './dynamic/dynamic-reference-builder.js';
+export * from './dynamic/dynamic-table-builder.js';
 export * from './driver/driver.js';
 export * from './driver/database-connection.js';
 export * from './driver/connection-provider.js';

package/dist/esm/kysely.js

@@ -564,14 +564,18 @@ export class TransactionBuilder {
                 ...kyselyProps,
                 executor,
             });
+            let transactionBegun = false;
             try {
                 await this.#props.driver.beginTransaction(connection, settings);
+                transactionBegun = true;
                 const result = await callback(transaction);
                 await this.#props.driver.commitTransaction(connection);
                 return result;
             }
             catch (error) {
-                await this.#props.driver.rollbackTransaction(connection);
+                if (transactionBegun) {
+                    await this.#props.driver.rollbackTransaction(connection);
+                }
                 throw error;
             }
         });

package/dist/esm/migration/migrator.d.ts

@@ -340,12 +340,12 @@ export interface MigrationResult {
      * The execution status.
      *
      *  - `Success` means the migration was successfully executed. Note that
-     *    if any of the later migrations in the {@link MigrationResult.results}
+     *    if any of the later migrations in the {@link MigrationResultSet.results}
      *    list failed (have status `Error`) AND the dialect supports transactional
      *    DDL, even the successfull migrations were rolled back.
      *
      *  - `Error` means the migration failed. In this case the
-     *    {@link MigrationResult.error} contains the error.
+     *    {@link MigrationResultSet.error} contains the error.
      *
      *  - `NotExecuted` means that the migration was supposed to be executed
      *    but wasn't because an earlier migration failed.

package/dist/esm/query-compiler/default-query-compiler.js

@@ -4,11 +4,11 @@ import { InsertQueryNode } from '../operation-node/insert-query-node.js';
 import { OperationNodeVisitor } from '../operation-node/operation-node-visitor.js';
 import { OperatorNode } from '../operation-node/operator-node.js';
 import { ParensNode } from '../operation-node/parens-node.js';
-import { QueryNode } from '../operation-node/query-node.js';
+import { RawNode } from '../operation-node/raw-node.js';
 import { freeze, isString, isNumber, isBoolean, isNull, isDate, isBigInt, } from '../util/object-utils.js';
 import { CreateViewNode } from '../operation-node/create-view-node.js';
 import { SetOperationNode } from '../operation-node/set-operation-node.js';
-import { MergeQueryNode } from '../operation-node/merge-query-node.js';
+import { WhenNode } from '../operation-node/when-node.js';
 import { logOnce } from '../util/log-once.js';
 const LIT_WRAP_REGEX = /'/g;
 export class DefaultQueryCompiler extends OperationNodeVisitor {
@@ -148,13 +148,15 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
         this.visitNode(node.having);
     }
     visitInsertQuery(node) {
-        const rootQueryNode = this.nodeStack.find(QueryNode.is);
-        const isSubQuery = rootQueryNode !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !ParensNode.is(this.parentNode) &&
+            !RawNode.is(this.parentNode) &&
+            !WhenNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery && !MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -208,7 +210,7 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery && !MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {
@@ -221,12 +223,14 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
         this.compileList(node.values);
     }
     visitDeleteQuery(node) {
-        const isSubQuery = this.nodeStack.find(QueryNode.is) !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !ParensNode.is(this.parentNode) &&
+            !RawNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -267,7 +271,7 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {
@@ -528,13 +532,15 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
         this.visitNode(node.groupBy);
     }
     visitUpdateQuery(node) {
-        const rootQueryNode = this.nodeStack.find(QueryNode.is);
-        const isSubQuery = rootQueryNode !== node;
-        if (!isSubQuery && node.explain) {
+        const wrapInParens = this.parentNode !== undefined &&
+            !ParensNode.is(this.parentNode) &&
+            !RawNode.is(this.parentNode) &&
+            !WhenNode.is(this.parentNode);
+        if (this.parentNode === undefined && node.explain) {
             this.visitNode(node.explain);
             this.append(' ');
         }
-        if (isSubQuery && !MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append('(');
         }
         if (node.with) {
@@ -585,7 +591,7 @@ export class DefaultQueryCompiler extends OperationNodeVisitor {
             this.append(' ');
             this.visitNode(node.returning);
         }
-        if (isSubQuery && !MergeQueryNode.is(rootQueryNode)) {
+        if (wrapInParens) {
             this.append(')');
         }
         if (node.endModifiers?.length) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kysely",
-  "version": "0.28.6",
+  "version": "0.28.8",
   "description": "Type safe SQL query builder",
   "repository": {
     "type": "git",
@@ -81,31 +81,31 @@
     "@types/chai": "^5.0.1",
     "@types/chai-as-promised": "^8.0.2",
     "@types/mocha": "^10.0.10",
-    "@types/node": "^24.3.3",
+    "@types/node": "^24.7.0",
     "@types/pg": "^8.15.5",
     "@types/pg-cursor": "^2.7.2",
     "@types/semver": "^7.7.1",
     "@types/sinon": "^17.0.4",
-    "better-sqlite3": "^12.2.0",
-    "chai": "^6.0.1",
+    "better-sqlite3": "^12.4.1",
+    "chai": "^6.2.0",
     "chai-as-promised": "^8.0.2",
-    "esbuild": "^0.25.9",
+    "esbuild": "^0.25.10",
     "lodash": "^4.17.21",
-    "mocha": "^11.7.2",
-    "mysql2": "^3.14.5",
+    "mocha": "^11.7.4",
+    "mysql2": "^3.15.1",
     "pathe": "^2.0.3",
     "pg": "^8.15.6",
     "pg-cursor": "^2.14.6",
     "pkg-types": "^2.3.0",
-    "playwright": "^1.55.0",
+    "playwright": "^1.56.0",
     "prettier": "^3.5.3",
-    "semver": "^7.7.1",
+    "semver": "^7.7.3",
     "sinon": "^21.0.0",
     "std-env": "^3.9.0",
     "tarn": "^3.0.2",
     "tedious": "^19.0.0",
     "tsd": "^0.33.0",
-    "typescript": "~5.9.2"
+    "typescript": "~5.9.3"
   },
   "scripts": {
     "clean": "rm -rf dist & rm -rf test/node/dist & rm -rf test/browser/bundle.js & rm -rf helpers",