postgres-schema-migrations 6.1.0 → 7.0.0

package/bin/validate.js

@@ -0,0 +1,12 @@
+#!/usr/bin/env node
+// tslint:disable no-console
+import { argv } from "process";
+import { loadMigrationFiles } from "../files-loader";
+async function main(args) {
+    const directory = args[0];
+    await loadMigrationFiles(directory, (x) => console.error(x));
+}
+main(argv.slice(2)).catch((e) => {
+    console.error(`ERROR: ${e.message}`);
+    process.exit(1);
+});

package/create.d.ts

@@ -0,0 +1,2 @@
+import { BasicPgClient, Logger } from "./types";
+export declare function runCreateQuery(dbName: string, log: Logger): (client: BasicPgClient) => Promise<void>;

package/create.js

@@ -0,0 +1,19 @@
+const DUPLICATE_DATABASE = "42P04";
+export function runCreateQuery(dbName, log) {
+    return async (client) => {
+        await client
+            .query(`CREATE DATABASE "${dbName.replace(/"/g, '""')}"`)
+            .catch((e) => {
+            switch (e.code) {
+                case DUPLICATE_DATABASE: {
+                    log(`'${dbName}' database already exists`);
+                    return;
+                }
+                default: {
+                    log(e);
+                    throw new Error(`Error creating database. Caused by: '${e.name}: ${e.message}'`);
+                }
+            }
+        });
+    };
+}

package/{dist/file-name-parser.js → file-name-parser.js}

@@ -1,6 +1,3 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.parseFileName = void 0;
 const parseId = (id) => {
     const parsed = parseInt(id, 10);
     if (isNaN(parsed)) {
@@ -8,7 +5,7 @@ const parseId = (id) => {
     }
     return parsed;
 };
-const parseFileName = (fileName) => {
+export const parseFileName = (fileName) => {
     const result = /^(-?\d+)[-_]?(.*).(sql|js)$/gi.exec(fileName);
     if (!result) {
         throw new Error(`Invalid file name: '${fileName}'.`);
@@ -24,4 +21,3 @@ const parseFileName = (fileName) => {
         type: lowerType,
     };
 };
-exports.parseFileName = parseFileName;

package/{dist/files-loader.js → files-loader.js}

@@ -1,12 +1,11 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadMigrationFiles = void 0;
-const fs = require("fs");
-const path = require("path");
-const util_1 = require("util");
-const migration_file_1 = require("./migration-file");
-const validation_1 = require("./validation");
-const readDir = util_1.promisify(fs.readdir);
+import * as fs from "fs";
+import * as path from "path";
+import { promisify } from "util";
+import { fileURLToPath } from "url";
+import { loadMigrationFile } from "./migration-file";
+import { validateMigrationOrdering } from "./validation";
+const currentDir = path.dirname(fileURLToPath(import.meta.url));
+const readDir = promisify(fs.readdir);
 const isValidFile = (fileName) => /\.(sql|js)$/gi.test(fileName);
 /**
  * Load the migration files and assert they are reasonably valid.
@@ -16,7 +15,7 @@ const isValidFile = (fileName) => /\.(sql|js)$/gi.test(fileName);
  *
  * No assertions are made about the validity of the SQL.
  */
-const loadMigrationFiles = async (directory, 
+export const loadMigrationFiles = async (directory, 
 // tslint:disable-next-line no-empty
 log = () => { }, schemaName = "public") => {
     log(`Loading migrations from: ${directory}`);
@@ -26,14 +25,13 @@ log = () => { }, schemaName = "public") => {
         return [];
     }
     const migrationFiles = [
-        path.join(__dirname, "migrations/0_create-migrations-table.sql"),
+        path.join(currentDir, "migrations/0_create-migrations-table.sql"),
         ...fileNames.map((fileName) => path.resolve(directory, fileName)),
     ].filter(isValidFile);
-    const unorderedMigrations = await Promise.all(migrationFiles.map(migration_file_1.loadMigrationFile));
+    const unorderedMigrations = await Promise.all(migrationFiles.map(loadMigrationFile));
     // Arrange in ID order
     const orderedMigrations = unorderedMigrations.sort((a, b) => a.id - b.id);
-    validation_1.validateMigrationOrdering(orderedMigrations);
+    validateMigrationOrdering(orderedMigrations);
     orderedMigrations[0].sql = orderedMigrations[0].sql.replace("CREATE TABLE IF NOT EXISTS migrations", `CREATE SCHEMA IF NOT EXISTS ${schemaName}; CREATE TABLE IF NOT EXISTS ${schemaName}.migrations`);
     return orderedMigrations;
 };
-exports.loadMigrationFiles = loadMigrationFiles;

package/index.d.ts

@@ -0,0 +1,3 @@
+export { migrate } from "./migrate";
+export { loadMigrationFiles } from "./files-loader";
+export { ConnectionParams, MigrateDBConfig, Logger, Config, MigrationError, } from "./types";

package/index.js

@@ -0,0 +1,3 @@
+export { migrate } from "./migrate";
+export { loadMigrationFiles } from "./files-loader";
+export { MigrationError, } from "./types";

package/load-sql-from-js.d.ts

@@ -0,0 +1 @@
+export declare const loadSqlFromJs: (filePath: string) => Promise<string>;

package/{dist/load-sql-from-js.js → load-sql-from-js.js}

@@ -1,9 +1,8 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadSqlFromJs = void 0;
-const path = require("path");
-const loadSqlFromJs = (filePath) => {
-    const migrationModule = require(filePath);
+import * as path from "path";
+import { pathToFileURL } from "url";
+export const loadSqlFromJs = async (filePath) => {
+    const fileUrl = pathToFileURL(filePath).href;
+    const migrationModule = await import(fileUrl);
     if (!migrationModule.generateSql) {
         throw new Error(`Invalid javascript migration file: '${path.basename(filePath)}'.
 It must to export a 'generateSql' function.`);
@@ -15,4 +14,3 @@ It must to export a 'generateSql' function.`);
     }
     return generatedValue;
 };
-exports.loadSqlFromJs = loadSqlFromJs;

package/{dist/migrate.js → migrate.js}

@@ -1,14 +1,11 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.doesTableExist = exports.migrate = void 0;
-const pg = require("pg");
-const sql_template_strings_1 = require("sql-template-strings");
-const create_1 = require("./create");
-const files_loader_1 = require("./files-loader");
-const run_migration_1 = require("./run-migration");
-const validation_1 = require("./validation");
-const with_connection_1 = require("./with-connection");
-const with_lock_1 = require("./with-lock");
+import * as pg from "pg";
+import SQL from "sql-template-strings";
+import { runCreateQuery } from "./create";
+import { loadMigrationFiles } from "./files-loader";
+import { runMigration } from "./run-migration";
+import { validateMigrationHashes } from "./validation";
+import { withConnection } from "./with-connection";
+import { withAdvisoryLock } from "./with-lock";
 const splitTableName = (tableName) => {
     const parts = tableName.split(".");
     if (parts.length > 1) {
@@ -29,7 +26,7 @@ const splitTableName = (tableName) => {
  * @param config Extra configuration
  * @returns Details about the migrations which were run
  */
-async function migrate(dbConfig, migrationsDirectory, config = {}) {
+export async function migrate(dbConfig, migrationsDirectory, config = {}) {
     const log = config.logger != null
         ? config.logger
         : () => {
@@ -41,10 +38,10 @@ async function migrate(dbConfig, migrationsDirectory, config = {}) {
     if (typeof migrationsDirectory !== "string") {
         throw new Error("Must pass migrations directory as a string");
     }
-    const intendedMigrations = await files_loader_1.loadMigrationFiles(migrationsDirectory, log, config.schema);
+    const intendedMigrations = await loadMigrationFiles(migrationsDirectory, log, config.schema);
     if ("client" in dbConfig) {
         // we have been given a client to use, it should already be connected
-        return with_lock_1.withAdvisoryLock(log, runMigrations(intendedMigrations, log, config.schema))(dbConfig.client);
+        return withAdvisoryLock(log, runMigrations(intendedMigrations, log, config.schema))(dbConfig.client);
     }
     if (typeof dbConfig.database !== "string" ||
         typeof dbConfig.user !== "string" ||
@@ -65,13 +62,13 @@ async function migrate(dbConfig, migrationsDirectory, config = {}) {
             host,
             port,
         });
-        const runWith = with_connection_1.withConnection(log, async (connectedClient) => {
+        const runWith = withConnection(log, async (connectedClient) => {
             const result = await connectedClient.query({
                 text: "SELECT 1 FROM pg_database WHERE datname=$1",
                 values: [dbConfig.database],
             });
             if (result.rowCount !== 1) {
-                await create_1.runCreateQuery(dbConfig.database, log)(connectedClient);
+                await runCreateQuery(dbConfig.database, log)(connectedClient);
             }
         });
         await runWith(client);
@@ -81,23 +78,22 @@ async function migrate(dbConfig, migrationsDirectory, config = {}) {
         client.on("error", (err) => {
             log(`pg client emitted an error: ${err.message}`);
         });
-        const runWith = with_connection_1.withConnection(log, with_lock_1.withAdvisoryLock(log, runMigrations(intendedMigrations, log, config.schema)));
+        const runWith = withConnection(log, withAdvisoryLock(log, runMigrations(intendedMigrations, log, config.schema)));
         return runWith(client);
     }
 }
-exports.migrate = migrate;
 function runMigrations(intendedMigrations, log, schemaName = "public") {
     return async (client) => {
         try {
             const migrationTableName = `${schemaName}.migrations`;
             log("Starting migrations");
             const appliedMigrations = await fetchAppliedMigrationFromDB(migrationTableName, client, log);
-            validation_1.validateMigrationHashes(intendedMigrations, appliedMigrations);
+            validateMigrationHashes(intendedMigrations, appliedMigrations);
             const migrationsToRun = filterMigrations(intendedMigrations, appliedMigrations);
             const completedMigrations = [];
             for (const migration of migrationsToRun) {
                 log(`Starting migration: ${migration.id} ${migration.name}`);
-                const result = await run_migration_1.runMigration(migrationTableName, client, log)(migration);
+                const result = await runMigration(migrationTableName, client, log)(migration);
                 log(`Finished migration: ${migration.id} ${migration.name}`);
                 completedMigrations.push(result);
             }
@@ -106,9 +102,14 @@ function runMigrations(intendedMigrations, log, schemaName = "public") {
             return completedMigrations;
         }
         catch (e) {
-            const exception = e;
-            const error = new Error(`Migration failed. Reason: ${exception.message}`);
-            error.cause = exception.message;
+            const reason = (() => {
+                if (e instanceof Error) {
+                    return e.message;
+                }
+                return `${e}`;
+            })();
+            const error = new Error(`Migration failed. Reason: ${reason}`);
+            error.cause = reason;
             throw error;
         }
     };
@@ -142,13 +143,13 @@ function logResult(completedMigrations, log) {
     }
 }
 /** Check whether table exists in postgres - http://stackoverflow.com/a/24089729 */
-async function doesTableExist(client, tableName) {
+export async function doesTableExist(client, tableName) {
     const [schema, table] = splitTableName(tableName);
-    const result = await client.query(sql_template_strings_1.default `
+    const result = await client.query(SQL `
     SELECT EXISTS (
       SELECT 1
       FROM   pg_catalog.pg_class c
-      JOIN   pg_catalog.pg_namespace n 
+      JOIN   pg_catalog.pg_namespace n
         ON     n.oid = c.relnamespace
       WHERE  c.relname = ${table}
         AND    c.relkind = 'r'
@@ -157,4 +158,3 @@ async function doesTableExist(client, tableName) {
   `);
     return result.rows.length > 0 && result.rows[0].exists;
 }
-exports.doesTableExist = doesTableExist;

package/migration-file.js

@@ -0,0 +1,45 @@
+import { promisify } from "util";
+import * as fs from "fs";
+import * as path from "path";
+import * as crypto from "crypto";
+import { loadSqlFromJs } from "./load-sql-from-js";
+import { parseFileName } from "./file-name-parser";
+const readFile = promisify(fs.readFile);
+const getFileName = (filePath) => path.basename(filePath);
+const getFileContents = async (filePath) => readFile(filePath, "utf8");
+const hashString = (s) => crypto.createHash("sha1").update(s, "utf8").digest("hex");
+const getSqlStringLiteral = async (filePath, contents, type) => {
+    switch (type) {
+        case "sql":
+            return contents;
+        case "js":
+            return await loadSqlFromJs(filePath);
+        default: {
+            const exhaustiveCheck = type;
+            return exhaustiveCheck;
+        }
+    }
+};
+export const loadMigrationFile = async (filePath) => {
+    const fileName = getFileName(filePath);
+    try {
+        const { id, name, type } = parseFileName(fileName);
+        const contents = await getFileContents(filePath);
+        const sql = await getSqlStringLiteral(filePath, contents, type);
+        const hash = hashString(fileName + sql);
+        return {
+            id,
+            name,
+            contents,
+            fileName,
+            hash,
+            sql,
+        };
+    }
+    catch (err) {
+        if (err instanceof Error) {
+            throw new Error(`${err.message} - Offending file: '${fileName}'.`);
+        }
+        throw err;
+    }
+};

package/package.json

@@ -1,13 +1,17 @@
 {
   "name": "postgres-schema-migrations",
-  "version": "6.1.0",
-  "description": "Stack Overflow style database migrations for PostgreSQL",
+  "version": "7.0.0",
+  "description": "Database migrations for PostgreSQL",
+  "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "bin": {
     "pg-validate-migrations": "./dist/bin/validate.js"
   },
-  "authors": ["Thom Wright", "Zak Patterson"],
+  "authors": [
+    "Thom Wright",
+    "Zak Patterson"
+  ],
   "keywords": [
     "postgres",
     "postgresql",
@@ -29,43 +33,43 @@
   "engines": {
     "node": ">10.17.0"
   },
-  "scripts": {
-    "checkPushed": "[ \"$(git rev-list --count @{upstream}..HEAD)\" -eq 0 ] || (echo You have unpushed commits && exit 1)",
-    "prepublishOnly": "npm run checkPushed && npm run build",
-    "check-formatting": "./node_modules/.bin/prettier '**/*.ts' --list-different",
-    "fix-formatting": "./node_modules/.bin/prettier '**/*.ts' --write",
-    "lint": "npm run tslint && npm run check-formatting",
-    "tslint": "tslint 'src/**/*.ts' --type-check --project tsconfig.json --format verbose",
-    "test-integration": "ava --config ava.config.integration.cjs",
-    "test-unit": "ava --config ava.config.unit.cjs",
-    "test": "npm run test-unit && npm run lint && npm run test-integration",
-    "preversion": "npm test",
-    "build": "rm -rf ./dist && rsync -a --exclude='*.ts' --exclude='__tests__' --exclude='__unit__' --prune-empty-dirs src/ dist/ && tsc --project tsconfig-build.json && cp ./package.json ./dist/"
-  },
   "husky": {
     "hooks": {
       "commit-msg": "node ./node_modules/fit-commit-js/lib/hook.js .git/COMMIT_EDITMSG",
-      "pre-commit": "npm run lint",
-      "pre-push": "npm test"
+      "pre-commit": "pnpm run lint",
+      "pre-push": "pnpm test"
     }
   },
   "dependencies": {
-    "pg": "^8.6.0",
+    "pg": "^8.17.1",
     "sql-template-strings": "^2.2.2"
   },
   "devDependencies": {
-    "@types/node": "^10.17.60",
-    "@types/pg": "^8.6.0",
-    "@types/sinon": "^9.0.11",
-    "ava": "^3.15.0",
+    "@eslint/js": "^9.39.2",
+    "@types/node": "^25.0.9",
+    "@types/pg": "^8.16.0",
+    "@types/sinon": "^21.0.0",
+    "ava": "^6.4.1",
+    "eslint": "^9.39.2",
     "fit-commit-js": "^0.3.2",
-    "husky": "^3.1.0",
-    "prettier": "^2.3.1",
-    "sinon": "^9.2.4",
-    "ts-node": "^10.0.0",
-    "tslint": "^6.1.3",
-    "tslint-config-prettier": "^1.18.0",
-    "typescript": "^4.3.4",
-    "typescript-tslint-plugin": "^1.0.1"
+    "husky": "^9.1.7",
+    "prettier": "^3.8.0",
+    "sinon": "^21.0.1",
+    "ts-node": "^10.9.2",
+    "tsx": "^4.21.0",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.53.0"
+  },
+  "scripts": {
+    "checkPushed": "[ \"$(git rev-list --count @{upstream}..HEAD)\" -eq 0 ] || (echo You have unpushed commits && exit 1)",
+    "check-formatting": "./node_modules/.bin/prettier '**/*.ts' --list-different",
+    "fix-formatting": "./node_modules/.bin/prettier '**/*.ts' --write",
+    "lint": "pnpm run eslint && pnpm run check-formatting",
+    "eslint": "eslint 'src/**/*.ts'",
+    "test-integration": "ava --config ava.config.integration.cjs",
+    "test-unit": "ava --config ava.config.unit.cjs",
+    "test": "pnpm run test-unit && npm run lint && pnpm run test-integration",
+    "preversion": "pnpm test",
+    "build": "rm -rf ./dist && rsync -a --exclude='*.ts' --exclude='__tests__' --exclude='__unit__' --prune-empty-dirs src/ dist/ && tsc --project tsconfig-build.json && cp ./package.json ./dist/"
   }
-}
+}

package/{dist/run-migration.js → run-migration.js}

@@ -1,19 +1,16 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.runMigration = void 0;
-const sql_template_strings_1 = require("sql-template-strings");
+import SQL from "sql-template-strings";
 const noop = () => {
     //
 };
 const asyncNoop = () => Promise.resolve();
 const insertMigration = async (migrationTableName, client, migration, log) => {
     log(`Saving migration to '${migrationTableName}': ${migration.id} | ${migration.name} | ${migration.hash}`);
-    const sql = sql_template_strings_1.default `INSERT INTO `
+    const sql = SQL `INSERT INTO `
         .append(migrationTableName)
-        .append(sql_template_strings_1.default ` ("id", "name", "hash") VALUES (${migration.id},${migration.name},${migration.hash})`);
+        .append(SQL ` ("id", "name", "hash") VALUES (${migration.id},${migration.name},${migration.hash})`);
     return client.query(sql);
 };
-const runMigration = (migrationTableName, client, log = noop) => async (migration) => {
+export const runMigration = (migrationTableName, client, log = noop) => async (migration) => {
     const inTransaction = migration.sql.includes("-- postgres-migrations disable-transaction") ===
         false;
     log(`Running migration in transaction: ${inTransaction}`);
@@ -38,7 +35,9 @@ const runMigration = (migrationTableName, client, log = noop) => async (migratio
         catch (_a) {
             //
         }
-        throw new Error(`An error occurred running '${migration.name}'. Rolled back this migration. No further migrations were run. Reason: ${err.message}`);
+        if (err instanceof Error) {
+            throw new Error(`An error occurred running '${migration.name}'. Rolled back this migration. No further migrations were run. Reason: ${err.message}`);
+        }
+        throw err;
     }
 };
-exports.runMigration = runMigration;

package/{dist/types.d.ts → types.d.ts}

@@ -17,7 +17,7 @@ export interface ClientParams {
     /** A connected Client, or a Pool Client. The caller is responsible for connecting and cleaning up. */
     readonly client: pg.Client | pg.PoolClient | pg.Pool;
 }
-export declare type EnsureDatabase = {
+export type EnsureDatabase = {
     /**
      * Might default to `true` in future versions
      * @default false
@@ -31,18 +31,11 @@ export declare type EnsureDatabase = {
 } | {
     readonly ensureDatabaseExists?: false;
 };
-/**
- * @deprecated Use `migrate` instead with `ensureDatabaseExists: true`.
- */
-export declare type CreateDBConfig = (ConnectionParams & {
-    /** The database to connect to when creating the new database. */
-    readonly defaultDatabase?: string;
-}) | ClientParams;
-export declare type MigrateDBConfig = (ConnectionParams & {
+export type MigrateDBConfig = (ConnectionParams & {
     readonly database: string;
 } & EnsureDatabase) | ClientParams;
-export declare type Logger = (msg: string) => void;
-export declare type Config = Partial<FullConfig>;
+export type Logger = (msg: string) => void;
+export type Config = Partial<FullConfig>;
 export interface FullConfig {
     readonly logger: Logger;
     readonly schema: string;
@@ -50,7 +43,7 @@ export interface FullConfig {
 export declare class MigrationError extends Error {
     cause?: string;
 }
-export declare type FileType = "sql" | "js";
+export type FileType = "sql" | "js";
 export interface BasicPgClient {
     query(queryTextOrConfig: string | pg.QueryConfig): Promise<pg.QueryResult>;
 }

package/types.js

@@ -0,0 +1,2 @@
+export class MigrationError extends Error {
+}

package/{dist/validation.js → validation.js}

@@ -1,17 +1,13 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.validateMigrationHashes = exports.validateMigrationOrdering = void 0;
 const indexNotMatch = (migration, index) => migration.id !== index;
 /** Assert migration IDs are consecutive integers */
-function validateMigrationOrdering(migrations) {
+export function validateMigrationOrdering(migrations) {
     const notMatchingId = migrations.find(indexNotMatch);
     if (notMatchingId) {
         throw new Error(`Found a non-consecutive migration ID on file: '${notMatchingId.fileName}'`);
     }
 }
-exports.validateMigrationOrdering = validateMigrationOrdering;
 /** Assert hashes match */
-function validateMigrationHashes(migrations, appliedMigrations) {
+export function validateMigrationHashes(migrations, appliedMigrations) {
     const invalidHash = (migration) => {
         const appliedMigration = appliedMigrations[migration.id];
         return appliedMigration != null && appliedMigration.hash !== migration.hash;
@@ -25,4 +21,3 @@ function validateMigrationHashes(migrations, appliedMigrations) {
 This means that the scripts have changed since it was applied.`);
     }
 }
-exports.validateMigrationHashes = validateMigrationHashes;

package/{dist/with-connection.js → with-connection.js}

@@ -1,7 +1,4 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.withConnection = void 0;
-function withConnection(log, f) {
+export function withConnection(log, f) {
     return async (client) => {
         try {
             try {
@@ -10,14 +7,18 @@ function withConnection(log, f) {
                 log("... connected to database");
             }
             catch (e) {
-                log(`Error connecting to database: ${e.message}`);
+                if (e instanceof Error) {
+                    log(`Error connecting to database: ${e.message}`);
+                }
                 throw e;
             }
             const result = await f(client);
             return result;
         }
         catch (e) {
-            log(`Error using connection: ${e.message}`);
+            if (e instanceof Error) {
+                log(`Error using connection: ${e.message}`);
+            }
             throw e;
         }
         finally {
@@ -28,9 +29,10 @@ function withConnection(log, f) {
                 log("... connection closed");
             }
             catch (e) {
-                log(`Error closing the connection: ${e.message}`);
+                if (e instanceof Error) {
+                    log(`Error closing the connection: ${e.message}`);
+                }
             }
         }
     };
 }
-exports.withConnection = withConnection;

package/{dist/with-lock.js → with-lock.js}

@@ -1,7 +1,4 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.withAdvisoryLock = void 0;
-function withAdvisoryLock(log, f) {
+export function withAdvisoryLock(log, f) {
     return async (client) => {
         try {
             try {
@@ -19,14 +16,18 @@ function withAdvisoryLock(log, f) {
                 log("... acquired advisory lock");
             }
             catch (e) {
-                log(`Error acquiring advisory lock: ${e.message}`);
+                if (e instanceof Error) {
+                    log(`Error acquiring advisory lock: ${e.message}`);
+                }
                 throw e;
             }
             const result = await f(client);
             return result;
         }
         catch (e) {
-            log(`Error while using lock: ${e.message}`);
+            if (e instanceof Error) {
+                log(`Error while using lock: ${e.message}`);
+            }
             throw e;
         }
         finally {
@@ -36,9 +37,11 @@ function withAdvisoryLock(log, f) {
                 log("... released advisory lock");
             }
             catch (e) {
-                log(`Error releasing advisory lock: ${e.message}`);
+                if (e instanceof Error) {
+                    log(`Error releasing advisory lock: ${e.message}`);
+                }
+                // Don't re-throw - avoid masking the original error from the try block
             }
         }
     };
 }
-exports.withAdvisoryLock = withAdvisoryLock;

package/.editorconfig

@@ -1,9 +0,0 @@
-root = true
-
-[*]
-end_of_line = lf
-charset = utf-8
-trim_trailing_whitespace = true
-insert_final_newline = true
-indent_style = space
-indent_size = 2

package/.fitcommitjsrc.yml

@@ -1,22 +0,0 @@
----
-validators:
-  lineLength:
-    enabled: true
-    maxLineLength: 72
-    subjectMaxLength: 50
-  ticketCode:
-    enabled: false
-  emptyLines:
-    enabled: true
-    emptyLines: 1
-  tags:
-    enabled: false
-  subjectTense:
-    enabled: true
-  subjectPeriod:
-    enabled: false
-  capitalizedSubject:
-    # to allow npm version commits e.g. 1.0.0
-    enabled: false
-  wip:
-    enabled: true

package/.gitattributes

@@ -1 +0,0 @@
-package-lock.json -diff