feat: complete refactoring to ESM

and mocha/chai replaced by native node test runner

Author: rombat

.gitignore

@@ -9,3 +9,4 @@ TODO.md
 /MusicBee_Export.csv
 /navidrome.db
 /backups/
+index.cjs

README.md

@@ -77,7 +77,7 @@ All commands have these options available:
 * `--db <path>` : Navidrome SQLITE .db source file path. By default if not passed, will look for a file named `navidrome.db` in the same folder as `musicbee-navidrome-sync.exe`
 * `-u, --user <user_name>` : selects Navidrome username (by default if not used, the first found user will be used)
 * `--verbose` : verbose debugging
-* `--show-not-found` : output that were not found in Navidrome database (useful for troubleshooting missing tracks without verbose output noise)
+* `--show-not-found` : display tracks that were not found in Navidrome database (useful for troubleshooting missing tracks without verbose output noise)
 * `-h, --help` : displays help for command
 
 

build.config.js

@@ -0,0 +1,33 @@
+import { build } from 'esbuild';
+
+const buildConfig = {
+  entryPoints: ['index.js'],
+  bundle: true,
+  platform: 'node',
+  target: 'node22',
+  format: 'cjs', // for pkg compatibility
+  outfile: '.dist/index.cjs',
+  external: ['node:*'],
+  packages: 'bundle',
+  minify: true,
+  sourcemap: false,
+  banner: {
+    js: `// Generated by esbuild - all dependencies bundled
+// Suppress SQLite experimental warnings before any imports
+const originalProcessEmitWarning = process.emitWarning;
+process.emitWarning = function(warning, type, code) {
+  if (type === 'ExperimentalWarning' && warning.includes('SQLite')) {
+    return;
+  }
+  return originalProcessEmitWarning.call(this, warning, type, code);
+};`
+  }
+};
+
+try {
+  await build(buildConfig);
+  console.log('✅ ESM → CommonJS bundling completed successfully');
+} catch (error) {
+  console.error('❌ Build failed:', error);
+  process.exit(1);
+}

build.sh

@@ -2,11 +2,14 @@ set -e
 
 rm -rf .dist
 mkdir -p .dist
-cp -r index.js package*.json lib .dist
-cd .dist
 
-echo "Installing dependencies..."
-npm install --omit=dev --omit=optional --no-fund --loglevel=error
+echo "Installing dependencies for bundling..."
+npm install --no-fund --loglevel=error
+
+echo "Bundling ESM to CommonJS (with all dependencies)..."
+node build.config.js
+
+cd .dist
 
 echo "Building MBNDS exe..."
-pkg . --compress Brotli -o musicbee-navidrome-sync.exe
+npx @yao-pkg/pkg index.cjs --target node22-win-x64 --compress Brotli -o musicbee-navidrome-sync.exe

index.js

@@ -1,6 +1,6 @@
-const { program } = require('commander');
-const { MBNDSynchronizer } = require('./lib/MBNDSynchronizer.js');
-const packageJson = require('./package.json');
+import { program } from 'commander';
+import { MBNDSynchronizer } from './lib/MBNDSynchronizer.js';
+import packageJson from './package.json' with { type: 'json' };
 
 const runAction = async (options, command) => {
   const synchronizer = new MBNDSynchronizer(options);

lib/Database.js

@@ -1,16 +1,8 @@
-// Suppress SQLite experimental warnings by setting warning filter first
-const originalProcessEmitWarning = process.emitWarning;
-process.emitWarning = function (warning, type, code) {
-  if (type === 'ExperimentalWarning' && warning.includes('SQLite')) {
-    return;
-  }
-  return originalProcessEmitWarning.call(this, warning, type, code);
-};
+import { randomUUID } from 'node:crypto';
+import { DatabaseSync } from 'node:sqlite';
+import dayjs from 'dayjs';
+import utc from 'dayjs/plugin/utc.js';
 
-const { DatabaseSync } = require('node:sqlite');
-const { randomUUID } = require('node:crypto');
-const dayjs = require('dayjs');
-const utc = require('dayjs/plugin/utc');
 dayjs.extend(utc);
 
 /**
@@ -162,7 +154,7 @@ class Database {
   }
 }
 
-exports.init = dbFilePath => {
+export const init = async dbFilePath => {
   try {
     const database = new Database(dbFilePath);
     console.log('Connection has been established successfully.');

lib/MBNDSynchronizer.js

@@ -1,22 +1,23 @@
-const fs = require('node:fs');
-const path = require('node:path');
-const csv2json = require('csvtojson');
-const pLimit = require('p-limit');
-const cliProgress = require('cli-progress');
-const camelCase = require('camelcase');
-const dayjs = require('dayjs');
-const customParseFormat = require('dayjs/plugin/customParseFormat');
-const duration = require('dayjs/plugin/duration');
-const relativeTime = require('dayjs/plugin/relativeTime');
-const utc = require('dayjs/plugin/utc');
+import fs from 'node:fs';
+import path from 'node:path';
+import camelCase from 'camelcase';
+import cliProgress from 'cli-progress';
+import csv2json from 'csvtojson';
+import dayjs from 'dayjs';
+import customParseFormat from 'dayjs/plugin/customParseFormat.js';
+import duration from 'dayjs/plugin/duration.js';
+import relativeTime from 'dayjs/plugin/relativeTime.js';
+import utc from 'dayjs/plugin/utc.js';
+import pLimit from 'p-limit';
+
 dayjs.extend(utc);
 dayjs.extend(customParseFormat);
 dayjs.extend(duration);
 dayjs.extend(relativeTime);
 
-const dbManager = require('./Database');
-const { findBestMatch, isDateAfter } = require('./helpers');
-const packageJson = require('../package.json');
+import packageJson from '../package.json' with { type: 'json' };
+import * as dbManager from './Database.js';
+import { findBestMatch, isDateAfter } from './helpers.js';
 
 class MBNDSynchronizer {
   constructor(options) {
@@ -670,4 +671,4 @@ class MBNDSynchronizer {
   };
 }
 
-exports.MBNDSynchronizer = MBNDSynchronizer;
+export { MBNDSynchronizer };

lib/helpers.js

@@ -1,5 +1,6 @@
-const dayjs = require('dayjs');
-const utc = require('dayjs/plugin/utc');
+import dayjs from 'dayjs';
+import utc from 'dayjs/plugin/utc.js';
+
 dayjs.extend(utc);
 
 /**
@@ -75,4 +76,4 @@ const isDateAfter = (dateA, dateB) => {
   return dayjsA.isAfter(dayjsB);
 };
 
-module.exports = { findBestMatch, isDateAfter };
+export { findBestMatch, isDateAfter };