@bytebase/dbhub 0.22.5 → 0.22.6

package/dist/{chunk-BRXZ5ZQB.js → chunk-BXXHWEQS.js}

@@ -1,6 +1,6 @@
 import {
   stripCommentsAndStrings
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/utils/sql-row-limiter.ts
 var SQLRowLimiter = class {

package/dist/{chunk-C7WEAPX4.js → chunk-JEZZN2YZ.js}

@@ -267,6 +267,20 @@ function scanSingleLineComment(sql, i) {
   }
   return { type: TokenType.Comment, end: j };
 }
+function scanSingleLineCommentMySQL(sql, i) {
+  if (sql[i] !== "-" || sql[i + 1] !== "-") {
+    return null;
+  }
+  const next = sql[i + 2];
+  if (next !== void 0 && next.charCodeAt(0) > 32 && next.charCodeAt(0) !== 127) {
+    return null;
+  }
+  let j = i;
+  while (j < sql.length && sql[j] !== "\n") {
+    j++;
+  }
+  return { type: TokenType.Comment, end: j };
+}
 function scanMultiLineComment(sql, i) {
   if (sql[i] !== "/" || sql[i + 1] !== "*") {
     return null;
@@ -405,7 +419,7 @@ function scanTokenPostgres(sql, i) {
   return scanSingleLineComment(sql, i) ?? scanNestedMultiLineComment(sql, i) ?? scanSingleQuotedString(sql, i) ?? scanDoubleQuotedString(sql, i) ?? scanDollarQuotedBlock(sql, i) ?? plainToken(i);
 }
 function scanTokenMySQL(sql, i) {
-  return scanSingleLineComment(sql, i) ?? scanMultiLineCommentMySQL(sql, i) ?? scanSingleQuotedString(sql, i) ?? scanDoubleQuotedString(sql, i) ?? scanBacktickQuotedIdentifier(sql, i) ?? plainToken(i);
+  return scanSingleLineCommentMySQL(sql, i) ?? scanMultiLineCommentMySQL(sql, i) ?? scanSingleQuotedString(sql, i) ?? scanDoubleQuotedString(sql, i) ?? scanBacktickQuotedIdentifier(sql, i) ?? plainToken(i);
 }
 function scanTokenSQLite(sql, i) {
   return scanSingleLineComment(sql, i) ?? scanMultiLineComment(sql, i) ?? scanSingleQuotedString(sql, i) ?? scanDoubleQuotedString(sql, i) ?? scanBacktickQuotedIdentifier(sql, i) ?? scanBracketQuotedIdentifier(sql, i) ?? plainToken(i);

package/dist/{chunk-IOKERPF2.js → chunk-WPXPKRDU.js}

@@ -8,7 +8,7 @@ import {
   getDefaultPortForType,
   parseConnectionInfoFromDSN,
   stripCommentsAndStrings
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/tools/builtin-tools.ts
 var BUILTIN_TOOL_EXECUTE_SQL = "execute_sql";
@@ -1802,9 +1802,6 @@ var ConnectorManager = class {
     if (source.query_timeout !== void 0 && connector.id !== "sqlite") {
       config.queryTimeoutSeconds = source.query_timeout;
     }
-    if (source.readonly !== void 0) {
-      config.readonly = source.readonly;
-    }
     if (source.search_path) {
       config.searchPath = source.search_path;
     }

package/dist/index.js

@@ -15,7 +15,7 @@ import {
   resolveSourceConfigs,
   resolveTomlConfigPath,
   resolveTransport
-} from "./chunk-IOKERPF2.js";
+} from "./chunk-WPXPKRDU.js";
 import {
   loadConnectors
 } from "./chunk-WVVMH6FJ.js";
@@ -29,7 +29,7 @@ import {
   parseConnectionInfoFromDSN,
   splitSQLStatements,
   stripCommentsAndStrings
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/server.ts
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -134,6 +134,13 @@ var mutatingPatterns = {
   sqlserver: mutatingPattern
 };
 var selectIntoPattern = /\bselect\b[\s\S]+\binto\b/i;
+var sqliteReadOnlyArgPragmas = /* @__PURE__ */ new Set([
+  "table_info",
+  "index_info",
+  "index_list",
+  "foreign_key_list"
+]);
+var sqlitePragmaParenPattern = /^pragma\s+(?:[a-z0-9_]+\.)?([a-z0-9_]+)\s*\(/;
 var explainAnalyzePattern = /^explain\s+(?:\([^)]*\banalyze\b(?!\s*(?:=\s*)?(?:false|off|0)\b)[^)]*\)|\banalyze\b(?!\s*(?:=\s*)?(?:false|off|0)\b)(?:\s+verbose\b)?)/i;
 function isReadOnlySQL(sql, connectorType) {
   return checkReadOnly(
@@ -156,6 +163,15 @@ function checkReadOnly(cleanedSQL, connectorType) {
       return false;
     }
   }
+  if (firstWord === "pragma" && connectorType === "sqlite") {
+    if (cleanedSQL.includes("=")) {
+      return false;
+    }
+    const parenMatch = cleanedSQL.match(sqlitePragmaParenPattern);
+    if (parenMatch && !sqliteReadOnlyArgPragmas.has(parenMatch[1])) {
+      return false;
+    }
+  }
   if ((firstWord === "select" || firstWord === "with") && selectIntoPattern.test(cleanedSQL)) {
     return false;
   }
@@ -1656,7 +1672,7 @@ See documentation for more details on configuring database connections.
     const sources = sourceConfigsData.sources;
     console.error(`Configuration source: ${sourceConfigsData.source}`);
     await connectorManager.connectWithSources(sources);
-    const { initializeToolRegistry: initializeToolRegistry2 } = await import("./registry-L3ZIOUF2.js");
+    const { initializeToolRegistry: initializeToolRegistry2 } = await import("./registry-IPRZ55YC.js");
     initializeToolRegistry2({
       sources: sourceConfigsData.sources,
       tools: sourceConfigsData.tools
@@ -1810,11 +1826,11 @@ See documentation for more details on configuring database connections.
 
 // src/index.ts
 var connectorModules = [
-  { load: () => import("./postgres-EKVOBAH6.js"), name: "PostgreSQL", driver: "pg" },
-  { load: () => import("./sqlserver-5TFVBCMA.js"), name: "SQL Server", driver: "mssql" },
-  { load: () => import("./sqlite-2GBIF3MQ.js"), name: "SQLite", driver: "node:sqlite" },
-  { load: () => import("./mysql-WG6UFALX.js"), name: "MySQL", driver: "mysql2" },
-  { load: () => import("./mariadb-ERB4D3RP.js"), name: "MariaDB", driver: "mariadb" }
+  { load: () => import("./postgres-BKBCGDZS.js"), name: "PostgreSQL", driver: "pg" },
+  { load: () => import("./sqlserver-47APXMSF.js"), name: "SQL Server", driver: "mssql" },
+  { load: () => import("./sqlite-CQ4LCBB6.js"), name: "SQLite", driver: "node:sqlite" },
+  { load: () => import("./mysql-EK6TCER5.js"), name: "MySQL", driver: "mysql2" },
+  { load: () => import("./mariadb-JPEKVFEW.js"), name: "MariaDB", driver: "mariadb" }
 ];
 loadConnectors(connectorModules).then(() => main()).catch((error) => {
   console.error("Fatal error:", error);

package/dist/{mariadb-ERB4D3RP.js → mariadb-JPEKVFEW.js}

@@ -7,13 +7,13 @@ import {
 } from "./chunk-JFWX35TB.js";
 import {
   SQLRowLimiter
-} from "./chunk-BRXZ5ZQB.js";
+} from "./chunk-BXXHWEQS.js";
 import {
   ConnectorRegistry,
   SafeURL,
   obfuscateDSNPassword,
   splitSQLStatements
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/connectors/mariadb/index.ts
 import * as mariadb from "mariadb";
@@ -466,6 +466,9 @@ var MariaDBConnector = class _MariaDBConnector {
     }
     const conn = await this.pool.getConnection();
     try {
+      if (options.readonly) {
+        await conn.query("START TRANSACTION READ ONLY");
+      }
       let processedSQL = sql;
       if (options.maxRows) {
         const statements = splitSQLStatements(sql, "mariadb");
@@ -492,8 +495,17 @@ var MariaDBConnector = class _MariaDBConnector {
       }
       const rows = parseQueryResults(results);
       const rowCount = extractAffectedRows(results);
+      if (options.readonly) {
+        await conn.query("COMMIT");
+      }
       return { rows, rowCount };
     } catch (error) {
+      if (options.readonly) {
+        try {
+          await conn.query("ROLLBACK");
+        } catch {
+        }
+      }
       console.error("Error executing query:", error);
       throw error;
     } finally {

package/dist/{mysql-WG6UFALX.js → mysql-EK6TCER5.js}

@@ -7,13 +7,13 @@ import {
 } from "./chunk-JFWX35TB.js";
 import {
   SQLRowLimiter
-} from "./chunk-BRXZ5ZQB.js";
+} from "./chunk-BXXHWEQS.js";
 import {
   ConnectorRegistry,
   SafeURL,
   obfuscateDSNPassword,
   splitSQLStatements
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/connectors/mysql/index.ts
 import mysql from "mysql2/promise";
@@ -472,6 +472,9 @@ var MySQLConnector = class _MySQLConnector {
     }
     const conn = await this.pool.getConnection();
     try {
+      if (options.readonly) {
+        await conn.query("START TRANSACTION READ ONLY");
+      }
       let processedSQL = sql;
       if (options.maxRows) {
         const statements = splitSQLStatements(sql, "mysql");
@@ -499,8 +502,17 @@ var MySQLConnector = class _MySQLConnector {
       const [firstResult] = results;
       const rows = parseQueryResults(firstResult);
       const rowCount = extractAffectedRows(firstResult);
+      if (options.readonly) {
+        await conn.query("COMMIT");
+      }
       return { rows, rowCount };
     } catch (error) {
+      if (options.readonly) {
+        try {
+          await conn.query("ROLLBACK");
+        } catch {
+        }
+      }
       console.error("Error executing query:", error);
       throw error;
     } finally {

package/dist/{postgres-EKVOBAH6.js → postgres-BKBCGDZS.js}

@@ -3,13 +3,13 @@ import {
 } from "./chunk-JFWX35TB.js";
 import {
   SQLRowLimiter
-} from "./chunk-BRXZ5ZQB.js";
+} from "./chunk-BXXHWEQS.js";
 import {
   ConnectorRegistry,
   SafeURL,
   obfuscateDSNPassword,
   splitSQLStatements
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/connectors/postgres/index.ts
 import fs from "fs";
@@ -491,6 +491,25 @@ var PostgresConnector = class _PostgresConnector {
       const statements = splitSQLStatements(sql, "postgres");
       if (statements.length === 1) {
         const processedStatement = SQLRowLimiter.applyMaxRows(statements[0], options.maxRows);
+        if (options.readonly) {
+          await client.query("BEGIN READ ONLY");
+          try {
+            const result2 = parameters && parameters.length > 0 ? await client.query(processedStatement, parameters) : await client.query(processedStatement);
+            await client.query("COMMIT");
+            return { rows: result2.rows, rowCount: result2.rowCount ?? result2.rows.length };
+          } catch (error) {
+            try {
+              await client.query("ROLLBACK");
+            } catch {
+            }
+            console.error(`[PostgreSQL executeSQL] ERROR: ${error.message}`);
+            console.error(`[PostgreSQL executeSQL] SQL: ${processedStatement}`);
+            if (parameters && parameters.length > 0) {
+              console.error(`[PostgreSQL executeSQL] Parameters: ${JSON.stringify(parameters)}`);
+            }
+            throw error;
+          }
+        }
         let result;
         if (parameters && parameters.length > 0) {
           try {
@@ -511,7 +530,7 @@ var PostgresConnector = class _PostgresConnector {
         }
         let allRows = [];
         let totalRowCount = 0;
-        await client.query("BEGIN");
+        await client.query(options.readonly ? "BEGIN READ ONLY" : "BEGIN");
         try {
           for (let statement of statements) {
             const processedStatement = SQLRowLimiter.applyMaxRows(statement, options.maxRows);
@@ -525,7 +544,10 @@ var PostgresConnector = class _PostgresConnector {
           }
           await client.query("COMMIT");
         } catch (error) {
-          await client.query("ROLLBACK");
+          try {
+            await client.query("ROLLBACK");
+          } catch {
+          }
           throw error;
         }
         return { rows: allRows, rowCount: totalRowCount };

package/dist/{registry-L3ZIOUF2.js → registry-IPRZ55YC.js}

@@ -2,9 +2,9 @@ import {
   ToolRegistry,
   getToolRegistry,
   initializeToolRegistry
-} from "./chunk-IOKERPF2.js";
+} from "./chunk-WPXPKRDU.js";
 import "./chunk-WVVMH6FJ.js";
-import "./chunk-C7WEAPX4.js";
+import "./chunk-JEZZN2YZ.js";
 export {
   ToolRegistry,
   getToolRegistry,

package/dist/{sqlite-2GBIF3MQ.js → sqlite-CQ4LCBB6.js}

@@ -3,13 +3,13 @@ import {
 } from "./chunk-JFWX35TB.js";
 import {
   SQLRowLimiter
-} from "./chunk-BRXZ5ZQB.js";
+} from "./chunk-BXXHWEQS.js";
 import {
   ConnectorRegistry,
   SafeURL,
   obfuscateDSNPassword,
   splitSQLStatements
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/connectors/sqlite/suppress-experimental-warning.ts
 var installed = false;
@@ -319,6 +319,9 @@ var SQLiteConnector = class _SQLiteConnector {
     if (!this.db) {
       throw new Error("Not connected to SQLite database");
     }
+    if (options.readonly) {
+      this.db.exec("PRAGMA query_only = ON");
+    }
     try {
       const statements = splitSQLStatements(sql, "sqlite");
       if (statements.length === 1) {
@@ -375,19 +378,30 @@ var SQLiteConnector = class _SQLiteConnector {
         }
         let totalChanges = 0;
         for (const statement of writeStatements) {
+          if (options.readonly) {
+            this.db.exec("PRAGMA query_only = ON");
+          }
           const result = this.prepare(statement).run();
           totalChanges += Number(result.changes);
         }
         let allRows = [];
         for (let statement of readStatements) {
+          if (options.readonly) {
+            this.db.exec("PRAGMA query_only = ON");
+          }
           statement = SQLRowLimiter.applyMaxRows(statement, options.maxRows);
           const result = this.prepare(statement).all();
           allRows.push(...result);
         }
         return { rows: allRows, rowCount: totalChanges + allRows.length };
       }
-    } catch (error) {
-      throw error;
+    } finally {
+      if (options.readonly) {
+        try {
+          this.db.exec("PRAGMA query_only = OFF");
+        } catch {
+        }
+      }
     }
   }
 };

package/dist/{sqlserver-5TFVBCMA.js → sqlserver-47APXMSF.js}

@@ -3,13 +3,13 @@ import {
 } from "./chunk-WVVMH6FJ.js";
 import {
   SQLRowLimiter
-} from "./chunk-BRXZ5ZQB.js";
+} from "./chunk-BXXHWEQS.js";
 import {
   ConnectorRegistry,
   SafeURL,
   obfuscateDSNPassword,
   stripCommentsAndStrings
-} from "./chunk-C7WEAPX4.js";
+} from "./chunk-JEZZN2YZ.js";
 
 // src/connectors/sqlserver/index.ts
 import sql from "mssql";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bytebase/dbhub",
-  "version": "0.22.5",
+  "version": "0.22.6",
   "mcpName": "io.github.bytebase/dbhub",
   "description": "Minimal, token-efficient Database MCP Server for PostgreSQL, MySQL, SQL Server, SQLite, MariaDB",
   "repository": {