matterbridge 3.4.2-dev-20251204-a39ec9e → 3.4.2-dev-20251204-5a2b977

package/CHANGELOG.md

@@ -30,6 +30,16 @@ Advantages:
 
 ## [3.4.2] - 2025-12-??
 
+### Race condition
+
+We have a race condition when, after a blackout or with docker compose or with other systems that start more then one process, Matterbridge starts before other required system or network components.
+
+Race condition can cause missing configuration or missed devices on the controller side. All Matterbridge official plugins already wait for system and network components to be ready so there is no need of delay.
+
+To solve the race condition on blackout, use the --delay parameter. There is no delay on normal restart.
+
+To solve the race condition on docker compose, use the --fixed_delay parameter. The start will always be delayed.
+
 ### Added
 
 - [frontend]: Added throttle and timeout to autoScroll in the logs.
@@ -37,7 +47,8 @@ Advantages:
 - [frontend]: Added close on success to Install dialog.
 - [BroadcastServer]: Added check for port closed.
 - [platform]: Added isShuttingDown property to MatterbridgePlatform.
-- [delay]: Added --delay [seconds]. It will wait to start Matterbridge for the specified delay (default 2 minutes) if the system uptime is less then 5 minutes. It is a safe switch to avoid race conditions on start.
+- [delay]: Added --delay [seconds]. It will wait to start Matterbridge for the specified delay (default 2 minutes) if the system uptime is less then 5 minutes. It is a safe switch to avoid race conditions on start after a blackout.
+- [fixed_delay]: Added --fixed_delay [seconds]. It will wait to start Matterbridge for the specified delay (default 2 minutes). It is a safe switch to always avoid race conditions on start on docker compose. Use only if really needed cause it will always wait.
 
 ### Changed
 

package/dist/matterbridge.js

@@ -19,7 +19,6 @@ import { copyDirectory } from './utils/copyDirectory.js';
 import { createDirectory } from './utils/createDirectory.js';
 import { isValidString, parseVersionString, isValidNumber, isValidObject } from './utils/isvalid.js';
 import { formatBytes, formatPercent, formatUptime } from './utils/format.js';
-import { withTimeout, waiter, wait } from './utils/wait.js';
 import { dev, MATTER_LOGGER_FILE, MATTER_STORAGE_NAME, MATTERBRIDGE_LOGGER_FILE, NODE_STORAGE_DIR, plg, typ } from './matterbridgeTypes.js';
 import { PluginManager } from './pluginManager.js';
 import { DeviceManager } from './deviceManager.js';
@@ -121,7 +120,6 @@ export class Matterbridge extends EventEmitter {
     aggregatorUniqueId = getParameter('uniqueId');
     advertisingNodes = new Map();
     server;
-    debug = hasParameter('debug') || hasParameter('verbose');
     verbose = hasParameter('verbose');
     constructor() {
         super();
@@ -654,10 +652,17 @@ export class Matterbridge extends EventEmitter {
             await this.nodeContext?.set('bridgeMode', 'bridge');
         }
         if (hasParameter('delay') && os.uptime() <= 60 * 5) {
+            const { wait } = await import('./utils/wait.js');
             const delay = getIntParameter('delay') || 2000;
-            this.log.warn('Delay switch found with system uptime less than 5 minutes. Waiting for ' + delay + ' seconds before starting matterbridge...');
+            this.log.warn('Delay switch found with system uptime less then 5 minutes. Waiting for ' + delay + ' seconds before starting matterbridge...');
             await wait(delay * 1000, 'Race condition delay', true);
         }
+        if (hasParameter('fixed_delay')) {
+            const { wait } = await import('./utils/wait.js');
+            const delay = getIntParameter('fixed_delay') || 2000;
+            this.log.warn('Fixed delay switch found. Waiting for ' + delay + ' seconds before starting matterbridge...');
+            await wait(delay * 1000, 'Fixed race condition delay', true);
+        }
         if (hasParameter('bridge') || (!hasParameter('childbridge') && (await this.nodeContext?.get('bridgeMode', '')) === 'bridge')) {
             this.bridgeMode = 'bridge';
             this.log.debug(`Starting matterbridge in mode ${this.bridgeMode}`);
@@ -928,6 +933,7 @@ export class Matterbridge extends EventEmitter {
         await this.cleanup('updating...', false);
     }
     async unregisterAndShutdownProcess(timeout = 1000) {
+        const { wait } = await import('./utils/wait.js');
         this.log.info('Unregistering all devices and shutting down...');
         for (const plugin of this.plugins.array()) {
             if (plugin.error || !plugin.enabled)
@@ -990,6 +996,7 @@ export class Matterbridge extends EventEmitter {
             }
             this.log.notice(`Stopping matter server nodes in ${this.bridgeMode} mode...`);
             if (pause > 0) {
+                const { wait } = await import('./utils/wait.js');
                 this.log.debug(`Waiting ${pause}ms for the MessageExchange to finish...`);
                 await wait(pause, `Waiting ${pause}ms for the MessageExchange to finish...`, false);
             }
@@ -1206,6 +1213,7 @@ export class Matterbridge extends EventEmitter {
     async startChildbridge(delay = 1000) {
         if (!this.matterStorageManager)
             throw new Error('No storage manager initialized');
+        const { wait } = await import('./utils/wait.js');
         this.log.debug('Loading all plugins in childbridge mode...');
         await this.startPlugins(true, false);
         this.log.debug('Creating server nodes for DynamicPlatform plugins and starting all plugins in childbridge mode...');
@@ -1507,6 +1515,7 @@ export class Matterbridge extends EventEmitter {
         await matterServerNode.start();
     }
     async stopServerNode(matterServerNode, timeout = 30000) {
+        const { withTimeout } = await import('./utils/wait.js');
         if (!matterServerNode)
             return;
         this.log.notice(`Closing ${matterServerNode.id} server node`);
@@ -1558,6 +1567,7 @@ export class Matterbridge extends EventEmitter {
         }
     }
     async addBridgedEndpoint(pluginName, device) {
+        const { waiter } = await import('./utils/wait.js');
         const plugin = this.plugins.get(pluginName);
         if (!plugin) {
             this.log.error(`Error adding bridged endpoint ${dev}${device.deviceName}${er} (${zb}${device.id}${er}) plugin ${plg}${pluginName}${er} not found`);
@@ -1695,6 +1705,7 @@ export class Matterbridge extends EventEmitter {
         this.devices.remove(device);
     }
     async removeAllBridgedEndpoints(pluginName, delay = 0) {
+        const { wait } = await import('./utils/wait.js');
         this.log.debug(`Removing all bridged endpoints for plugin ${plg}${pluginName}${db}${delay > 0 ? ` with delay ${delay} ms` : ''}`);
         for (const device of this.devices.array().filter((device) => device.plugin === pluginName)) {
             await this.removeBridgedEndpoint(pluginName, device);

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "matterbridge",
-  "version": "3.4.2-dev-20251204-a39ec9e",
+  "version": "3.4.2-dev-20251204-5a2b977",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "matterbridge",
-      "version": "3.4.2-dev-20251204-a39ec9e",
+      "version": "3.4.2-dev-20251204-5a2b977",
       "license": "Apache-2.0",
       "dependencies": {
         "@matter/main": "0.15.6",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "matterbridge",
-  "version": "3.4.2-dev-20251204-a39ec9e",
+  "version": "3.4.2-dev-20251204-5a2b977",
   "description": "Matterbridge plugin manager for Matter",
   "author": "https://github.com/Luligu",
   "license": "Apache-2.0",