@noormdev/sdk 1.4.1 → 1.4.2

package/dist/{engine-MZQ364EH.js → engine-6ALQ5YTN.js}

@@ -1,11 +1,12 @@
 import { createRequire } from 'module';
-export { eta, isTemplate, processFile, processFiles, renderTemplate } from './chunk-G4B7EHL7.js';
+export { eta, isTemplate, processFile, processFiles, renderTemplate } from './chunk-CDTFXLXU.js';
 import './chunk-UOFJT5PY.js';
-import './chunk-LY2F7ZFI.js';
+import './chunk-A2OYZAVT.js';
+import './chunk-TXDGFA44.js';
 import './chunk-TBRUDUBQ.js';
 import './chunk-YJBKNUTP.js';
 import './chunk-D2XW2UOP.js';
 
 createRequire(import.meta.url);
-//# sourceMappingURL=engine-MZQ364EH.js.map
-//# sourceMappingURL=engine-MZQ364EH.js.map
+//# sourceMappingURL=engine-6ALQ5YTN.js.map
+//# sourceMappingURL=engine-6ALQ5YTN.js.map

package/dist/index.d.ts

@@ -2650,6 +2650,8 @@ export interface ProcedureSummary {
 	name: string;
 	schema?: string;
 	parameterCount: number;
+	/** Argument list that tells PostgreSQL overloads apart, e.g. `v integer, w text`. */
+	signature?: string;
 }
 /**
  * Function summary for list display.
@@ -2659,6 +2661,8 @@ export interface FunctionSummary {
 	schema?: string;
 	parameterCount: number;
 	returnType: string;
+	/** Argument list that tells PostgreSQL overloads apart, e.g. `v integer, w text`. */
+	signature?: string;
 }
 /**
  * Type/Enum summary for list display.
@@ -4688,7 +4692,11 @@ export interface NoormEvents extends SettingsEvents, UpdateEvents, VaultEvents,
 	};
 	"connection:error": {
 		configName: string;
+		/** Worded for the user. */
 		error: string;
+		/** What the server or driver reported, for the log: its code(s) and message(s). */
+		serverCode?: string;
+		serverMessage?: string;
 	};
 	"app:starting": {
 		mode: AppMode;

package/dist/index.js

@@ -1,9 +1,11 @@
 import { createRequire } from 'module';
-import { processFile, es_default, DtReader, isTemplate, FORMAT_VERSION, DT_EXTENSIONS, encryptWithPassphrase, ENCODED_TYPES } from './chunk-G4B7EHL7.js';
+import { processFile, es_default, DtReader, isTemplate, FORMAT_VERSION, DT_EXTENSIONS, encryptWithPassphrase, ENCODED_TYPES } from './chunk-CDTFXLXU.js';
 import './chunk-UOFJT5PY.js';
-import { resolveLegacyAccess, checkConfigPolicy, initializeVault, getVaultStatus, setVaultSecret, getVaultSecret, getAllVaultSecrets, listVaultSecretKeys, deleteVaultSecret, vaultSecretExists, getVaultKey, noormDb, getNoormTables, encryptVaultKey, assertVaultPolicy, assertPolicy, detectAgentHarness, guarded } from './chunk-FSVO6NAJ.js';
-import { makeNestedConfig, gigabytes, attempt, observer, attemptSync, getEnvConfigName, clone, merge, equals, retry, runWithTimeout, isTimeoutError, ObserverRelay, isDebug, wait } from './chunk-LY2F7ZFI.js';
-export { observer as noormObserver } from './chunk-LY2F7ZFI.js';
+import { resolveLegacyAccess, checkConfigPolicy, initializeVault, getVaultStatus, setVaultSecret, getVaultSecret, getAllVaultSecrets, listVaultSecretKeys, deleteVaultSecret, vaultSecretExists, getVaultKey, noormDb, getNoormTables, encryptVaultKey, assertVaultPolicy, assertPolicy, detectAgentHarness, guarded } from './chunk-OMY6CKR4.js';
+import { observer, getEnvConfigName, ObserverRelay, isDebug } from './chunk-A2OYZAVT.js';
+export { observer as noormObserver } from './chunk-A2OYZAVT.js';
+import { explainConnectionError, DatabaseConnectionError } from './chunk-OXTXFKWJ.js';
+import { makeNestedConfig, gigabytes, attempt, attemptSync, clone, merge, equals, retry, runWithTimeout, isTimeoutError, wait } from './chunk-TXDGFA44.js';
 import { external_exports, PortSchema, connectTimeoutFor, DEFAULT_PORTS } from './chunk-WE2JLJMT.js';
 import { dist_default } from './chunk-TBRUDUBQ.js';
 import { require_dist } from './chunk-YJBKNUTP.js';
@@ -2926,7 +2928,7 @@ async function getDialectFactory(dialect) {
     case "mysql":
       return (await import('./mysql-32B4SXWA.js')).createMysqlConnection;
     case "mssql":
-      return (await import('./mssql-3BQB6ACD.js')).createMssqlConnection;
+      return (await import('./mssql-II62FO2B.js')).createMssqlConnection;
     default:
       throw new Error(`Unsupported dialect: ${dialect}`);
   }
@@ -2997,6 +2999,9 @@ async function openConnection(config, configName, retryOptions, signal) {
         backoff,
         jitterFactor: 0.1,
         signal,
+        // Otherwise running out of retries reads "Max retries
+        // reached", and the server's own error is lost.
+        throwLastError: true,
         shouldRetry: (err2) => {
           if (err2 instanceof OperationAbortedError) return false;
           const msg = err2.message.toLowerCase();
@@ -3014,8 +3019,10 @@ async function openConnection(config, configName, retryOptions, signal) {
     )
   );
   if (err) {
-    observer.emit("connection:error", { configName, error: err.message });
-    throw err;
+    const explained = explainConnectionError(err, config);
+    const server = explained instanceof DatabaseConnectionError ? { serverCode: explained.serverCode, serverMessage: explained.serverMessage } : {};
+    observer.emit("connection:error", { configName, error: explained.message, ...server });
+    throw explained;
   }
   const manager = getConnectionManager();
   const trackId = manager.track(conn, configName);
@@ -3055,15 +3062,21 @@ var SYSTEM_DATABASES = {
   // SQLite creates the file on connect
   mssql: "master"
 };
-async function testConnection(config, options = {}) {
-  let testConfig = config;
-  if (options.testServerOnly && config.dialect !== "sqlite") {
-    const systemDb = SYSTEM_DATABASES[config.dialect];
-    testConfig = {
-      ...config,
-      database: systemDb ?? config.database
-    };
+async function probeConnection(config, signal) {
+  const [conn, err] = await attempt(
+    () => createConnection(config, "__test__", {}, signal)
+  );
+  if (err) {
+    if (err instanceof OperationAbortedError) {
+      return { ok: false, error: err.message, aborted: true };
+    }
+    return { ok: false, error: err.message };
   }
+  await conn.destroy();
+  return { ok: true };
+}
+async function testConnection(config, options = {}) {
+  const systemDb = SYSTEM_DATABASES[config.dialect];
   if (options.testServerOnly && config.dialect === "sqlite") {
     const filename = config.filename ?? config.database;
     if (filename !== ":memory:" && !existsSync(filename)) {
@@ -3074,17 +3087,12 @@ async function testConnection(config, options = {}) {
       return { ok: true };
     }
   }
-  const [conn, err] = await attempt(
-    () => createConnection(testConfig, "__test__", {}, options.signal)
-  );
-  if (err) {
-    if (err instanceof OperationAbortedError) {
-      return { ok: false, error: err.message, aborted: true };
-    }
-    return { ok: false, error: err.message };
+  const result = await probeConnection(config, options.signal);
+  const targetMissing = !result.ok && !result.aborted && !!result.error?.includes("does not exist");
+  if (options.testServerOnly && systemDb && targetMissing) {
+    return probeConnection({ ...config, database: systemDb }, options.signal);
   }
-  await conn.destroy();
-  return { ok: true };
+  return result;
 }
 function addIdColumn(builder, dialect) {
   if (dialect === "postgres") {
@@ -7546,7 +7554,7 @@ async function resolveVaultKey(db, dialect) {
 async function buildSecretsContext(stateManager, configName, db, vaultKey, dialect) {
   const secrets = {};
   if (db && vaultKey) {
-    const { getAllVaultSecrets: getAllVaultSecrets2 } = await import('./storage-V5I67ANZ.js');
+    const { getAllVaultSecrets: getAllVaultSecrets2 } = await import('./storage-V6BHWNS3.js');
     const vaultSecrets = await getAllVaultSecrets2(db, vaultKey, dialect ?? "postgres");
     for (const [key, secret] of Object.entries(vaultSecrets)) {
       secrets[key] = secret.value;
@@ -9227,7 +9235,8 @@ var postgresExploreOperations = {
             SELECT
                 p.proname,
                 n.nspname,
-                p.pronargs::text as param_count
+                p.pronargs::text as param_count,
+                pg_get_function_identity_arguments(p.oid) as identity_args
             FROM pg_proc p
             JOIN pg_namespace n ON p.pronamespace = n.oid
             WHERE n.nspname ${schemaFilter(schema)}
@@ -9242,7 +9251,8 @@ var postgresExploreOperations = {
     return result.rows.map((row) => ({
       name: row.proname,
       schema: row.nspname,
-      parameterCount: parseInt(row.param_count, 10)
+      parameterCount: parseInt(row.param_count, 10),
+      signature: row.identity_args
     }));
   },
   async listFunctions(db, schema) {
@@ -9251,7 +9261,8 @@ var postgresExploreOperations = {
                 p.proname,
                 n.nspname,
                 p.pronargs::text as param_count,
-                pg_get_function_result(p.oid) as return_type
+                pg_get_function_result(p.oid) as return_type,
+                pg_get_function_identity_arguments(p.oid) as identity_args
             FROM pg_proc p
             JOIN pg_namespace n ON p.pronamespace = n.oid
             WHERE n.nspname ${schemaFilter(schema)}
@@ -9267,7 +9278,8 @@ var postgresExploreOperations = {
       name: row.proname,
       schema: row.nspname,
       parameterCount: parseInt(row.param_count, 10),
-      returnType: row.return_type
+      returnType: row.return_type,
+      signature: row.identity_args
     }));
   },
   async listTypes(db, schema) {

package/dist/{mssql-3BQB6ACD.js → mssql-II62FO2B.js}

@@ -1,4 +1,6 @@
 import { createRequire } from 'module';
+import { explainMssqlLoginFailure } from './chunk-OXTXFKWJ.js';
+import { attempt } from './chunk-TXDGFA44.js';
 import { DEFAULT_PORTS, connectTimeoutFor } from './chunk-WE2JLJMT.js';
 import './chunk-D2XW2UOP.js';
 import { isIP } from 'net';
@@ -73,7 +75,7 @@ function resolveTlsServerName(config) {
   }
   return UNVERIFIED_TLS_SERVER_NAME;
 }
-function buildTediousOptions(config, database) {
+function buildTediousOptions(config) {
   return {
     server: config.host ?? "localhost",
     authentication: {
@@ -85,7 +87,7 @@ function buildTediousOptions(config, database) {
     },
     options: {
       port: config.port ?? DEFAULT_PORTS.mssql,
-      database: database ?? config.database,
+      database: config.database,
       trustServerCertificate: !config.ssl,
       encrypt: true,
       serverName: resolveTlsServerName(config),
@@ -98,46 +100,19 @@ function buildTediousOptions(config, database) {
     }
   };
 }
-function buildTediousConfig(Tedious, config, database) {
-  return new Tedious.Connection(buildTediousOptions(config, database));
-}
-async function verifyDatabaseExists(Tedious, Tarn, config) {
-  const masterDb = new Kysely({
-    dialect: new MssqlDialect({
-      tarn: {
-        ...Tarn,
-        options: {
-          min: 0,
-          max: 1,
-          propagateCreateError: true
-        }
-      },
-      tedious: {
-        ...Tedious,
-        connectionFactory: () => buildTediousConfig(Tedious, config, "master")
-      }
-    }),
-    plugins: [new MssqlLimitPlugin()]
-  });
-  try {
-    const { rows } = await sql`
-            SELECT name FROM sys.databases WHERE name = ${config.database}
-        `.execute(masterDb);
-    if (rows.length === 0) {
-      throw new Error(
-        `Database '${config.database}' does not exist on ${config.host ?? "localhost"}:${config.port ?? DEFAULT_PORTS.mssql}`
-      );
-    }
-  } finally {
-    await masterDb.destroy();
-  }
+function buildTediousConnection(Tedious, config, loginErrors) {
+  const connection = new Tedious.Connection(buildTediousOptions(config));
+  const record = (token) => loginErrors.push({ number: token.number, message: token.message });
+  connection.on("errorMessage", record);
+  connection.once("connect", () => connection.removeListener("errorMessage", record));
+  return connection;
 }
 async function createMssqlConnection(config) {
   const TediousImport = await import('tedious');
   const TarnImport = await import('tarn');
   const Tedious = TediousImport.default ?? TediousImport;
   const Tarn = TarnImport.default ?? TarnImport;
-  await verifyDatabaseExists(Tedious, Tarn, config);
+  let loginErrors = [];
   const db = new Kysely({
     dialect: new MssqlDialect({
       tarn: {
@@ -150,11 +125,19 @@ async function createMssqlConnection(config) {
       },
       tedious: {
         ...Tedious,
-        connectionFactory: () => buildTediousConfig(Tedious, config)
+        connectionFactory: () => {
+          loginErrors = [];
+          return buildTediousConnection(Tedious, config, loginErrors);
+        }
       }
     }),
     plugins: [new MssqlLimitPlugin()]
   });
+  const [, connectErr] = await attempt(() => sql`SELECT 1`.execute(db));
+  if (connectErr) {
+    await db.destroy();
+    throw explainMssqlLoginFailure(loginErrors, connectErr, config);
+  }
   return {
     db,
     dialect: "mssql",
@@ -163,5 +146,5 @@ async function createMssqlConnection(config) {
 }
 
 export { MssqlTlsServerNameError, UNVERIFIED_TLS_SERVER_NAME, buildTediousOptions, createMssqlConnection, resolveTlsServerName };
-//# sourceMappingURL=mssql-3BQB6ACD.js.map
-//# sourceMappingURL=mssql-3BQB6ACD.js.map
+//# sourceMappingURL=mssql-II62FO2B.js.map
+//# sourceMappingURL=mssql-II62FO2B.js.map

package/dist/{storage-V5I67ANZ.js → storage-V6BHWNS3.js}

@@ -1,8 +1,9 @@
 import { createRequire } from 'module';
-export { deleteVaultSecret, deleteVaultSecretChecked, getAllVaultSecrets, getVaultKey, getVaultKeyChecked, getVaultSecret, getVaultStatus, initializeVault, initializeVaultChecked, listVaultSecretKeys, listVaultSecretKeysChecked, setVaultSecret, setVaultSecretChecked, vaultSecretExists } from './chunk-FSVO6NAJ.js';
-import './chunk-LY2F7ZFI.js';
+export { deleteVaultSecret, deleteVaultSecretChecked, getAllVaultSecrets, getVaultKey, getVaultKeyChecked, getVaultSecret, getVaultStatus, initializeVault, initializeVaultChecked, listVaultSecretKeys, listVaultSecretKeysChecked, setVaultSecret, setVaultSecretChecked, vaultSecretExists } from './chunk-OMY6CKR4.js';
+import './chunk-A2OYZAVT.js';
+import './chunk-TXDGFA44.js';
 import './chunk-D2XW2UOP.js';
 
 createRequire(import.meta.url);
-//# sourceMappingURL=storage-V5I67ANZ.js.map
-//# sourceMappingURL=storage-V5I67ANZ.js.map
+//# sourceMappingURL=storage-V6BHWNS3.js.map
+//# sourceMappingURL=storage-V6BHWNS3.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noormdev/sdk",
-  "version": "1.4.1",
+  "version": "1.4.2",
   "description": "Database schema & changeset manager SDK",
   "type": "module",
   "main": "./dist/index.js",