react-native-ble-mesh 2.0.0 → 2.1.2

package/src/mesh/fragment/Fragmenter.js

@@ -66,7 +66,7 @@ function createFragmentHeader(index, total, payloadLength) {
 /**
  * Parses a fragment header
  * @param {Uint8Array} data - Fragment data
- * @returns {Object} Parsed header { index, total, payloadLength }
+ * @returns {any} Parsed header { index, total, payloadLength }
  */
 function parseFragmentHeader(data) {
   if (data.length < FRAGMENT_HEADER_SIZE) {
@@ -121,7 +121,7 @@ function fragment(payload, messageId, maxSize = DEFAULT_FRAGMENT_SIZE) {
   for (let i = 0; i < fragmentCount; i++) {
     const remainingLength = payload.length - offset;
     const chunkLength = Math.min(payloadCapacity, remainingLength);
-    const chunk = payload.slice(offset, offset + chunkLength);
+    const chunk = payload.subarray(offset, offset + chunkLength);
 
     // Create fragment with header
     const header = createFragmentHeader(i, fragmentCount, chunkLength);
@@ -139,7 +139,7 @@ function fragment(payload, messageId, maxSize = DEFAULT_FRAGMENT_SIZE) {
 /**
  * Gets fragment information without parsing full data
  * @param {Uint8Array} fragmentData - Fragment data
- * @returns {Object} Fragment info { index, total, payloadLength, payload }
+ * @returns {any} Fragment info { index, total, payloadLength, payload }
  */
 function getFragmentInfo(fragmentData) {
   const header = parseFragmentHeader(fragmentData);

package/src/mesh/monitor/ConnectionQuality.js

@@ -12,7 +12,7 @@ const EventEmitter = require('../../utils/EventEmitter');
 
 /**
  * Connection quality levels
- * @constant {Object}
+ * @constant {any}
  */
 const QUALITY_LEVEL = Object.freeze({
   EXCELLENT: 'excellent',
@@ -24,7 +24,7 @@ const QUALITY_LEVEL = Object.freeze({
 
 /**
  * Default quality configuration
- * @constant {Object}
+ * @constant {any}
  */
 const DEFAULT_CONFIG = Object.freeze({
   /** How often to recalculate quality (ms) */
@@ -50,14 +50,28 @@ const DEFAULT_CONFIG = Object.freeze({
  * @class PeerQualityTracker
  */
 class PeerQualityTracker {
+  /**
+   * @param {string} peerId
+   * @param {any} config
+   */
   constructor(peerId, config) {
     this.peerId = peerId;
+    /** @type {any} */
     this._config = config;
-    this._rssiSamples = [];
-    this._latencySamples = [];
+    // Circular buffer for RSSI samples
+    this._rssiSamples = new Float64Array(config.sampleSize);
+    this._rssiIndex = 0;
+    this._rssiCount = 0;
+    this._rssiSum = 0;
+    // Circular buffer for latency samples
+    this._latencySamples = new Float64Array(config.sampleSize);
+    this._latencyIndex = 0;
+    this._latencyCount = 0;
+    this._latencySum = 0;
     this._packetsSent = 0;
     this._packetsAcked = 0;
     this._bytesTransferred = 0;
+    /** @type {number|null} */
     this._transferStartTime = null;
     this._lastActivity = Date.now();
     this._transport = 'ble';
@@ -65,25 +79,37 @@ class PeerQualityTracker {
   }
 
   /**
-   * Records an RSSI sample
+   * Records an RSSI sample using circular buffer
    * @param {number} rssi - Signal strength in dBm
    */
   recordRssi(rssi) {
-    this._rssiSamples.push(rssi);
-    if (this._rssiSamples.length > this._config.sampleSize) {
-      this._rssiSamples.shift();
+    const capacity = this._rssiSamples.length;
+    if (this._rssiCount >= capacity) {
+      this._rssiSum -= this._rssiSamples[this._rssiIndex];
+    }
+    this._rssiSamples[this._rssiIndex] = rssi;
+    this._rssiSum += rssi;
+    this._rssiIndex = (this._rssiIndex + 1) % capacity;
+    if (this._rssiCount < capacity) {
+      this._rssiCount++;
     }
     this._lastActivity = Date.now();
   }
 
   /**
-   * Records a latency measurement
+   * Records a latency measurement using circular buffer
    * @param {number} latencyMs - Round-trip latency in ms
    */
   recordLatency(latencyMs) {
-    this._latencySamples.push(latencyMs);
-    if (this._latencySamples.length > this._config.sampleSize) {
-      this._latencySamples.shift();
+    const capacity = this._latencySamples.length;
+    if (this._latencyCount >= capacity) {
+      this._latencySum -= this._latencySamples[this._latencyIndex];
+    }
+    this._latencySamples[this._latencyIndex] = latencyMs;
+    this._latencySum += latencyMs;
+    this._latencyIndex = (this._latencyIndex + 1) % capacity;
+    if (this._latencyCount < capacity) {
+      this._latencyCount++;
     }
     this._lastActivity = Date.now();
   }
@@ -125,21 +151,21 @@ class PeerQualityTracker {
   }
 
   /**
-   * Gets the average RSSI
+   * Gets the average RSSI (O(1) using running sum)
    * @returns {number|null}
    */
   getAvgRssi() {
-    if (this._rssiSamples.length === 0) { return null; }
-    return this._rssiSamples.reduce((a, b) => a + b, 0) / this._rssiSamples.length;
+    if (this._rssiCount === 0) { return null; }
+    return this._rssiSum / this._rssiCount;
   }
 
   /**
-   * Gets the average latency
+   * Gets the average latency (O(1) using running sum)
    * @returns {number|null}
    */
   getAvgLatency() {
-    if (this._latencySamples.length === 0) { return null; }
-    return this._latencySamples.reduce((a, b) => a + b, 0) / this._latencySamples.length;
+    if (this._latencyCount === 0) { return null; }
+    return this._latencySum / this._latencyCount;
   }
 
   /**
@@ -173,7 +199,7 @@ class PeerQualityTracker {
   /**
    * Calculates quality score (0-1) for a metric
    * @param {number|null} value - Metric value
-   * @param {Object} thresholds - {excellent, good, fair}
+   * @param {any} thresholds - {excellent, good, fair}
    * @param {boolean} [higherIsBetter=false]
    * @returns {number}
    */
@@ -196,7 +222,7 @@ class PeerQualityTracker {
 
   /**
    * Calculates overall quality
-   * @returns {Object} Quality report
+   * @returns {any} Quality report
    */
   calculate() {
     if (this.isTimedOut()) {
@@ -231,6 +257,7 @@ class PeerQualityTracker {
     if (score >= 0.8) { level = QUALITY_LEVEL.EXCELLENT; } else if (score >= 0.6) { level = QUALITY_LEVEL.GOOD; } else if (score >= 0.4) { level = QUALITY_LEVEL.FAIR; } else { level = QUALITY_LEVEL.POOR; }
 
     const previousLevel = this._currentLevel;
+    // @ts-ignore
     this._currentLevel = level;
 
     return {
@@ -255,13 +282,15 @@ class PeerQualityTracker {
  */
 class ConnectionQuality extends EventEmitter {
   /**
-   * @param {Object} [config={}]
+   * @param {any} [config={}]
    */
   constructor(config = {}) {
     super();
+    /** @type {any} */
     this._config = { ...DEFAULT_CONFIG, ...config };
     /** @type {Map<string, PeerQualityTracker>} */
     this._peers = new Map();
+    /** @type {any} */
     this._updateTimer = null;
   }
 
@@ -301,7 +330,7 @@ class ConnectionQuality extends EventEmitter {
     if (!this._peers.has(peerId)) {
       this._peers.set(peerId, new PeerQualityTracker(peerId, this._config));
     }
-    return this._peers.get(peerId);
+    return /** @type {PeerQualityTracker} */ (this._peers.get(peerId));
   }
 
   /**
@@ -367,7 +396,7 @@ class ConnectionQuality extends EventEmitter {
   /**
    * Gets quality for a specific peer
    * @param {string} peerId
-   * @returns {Object|null}
+   * @returns {any}
    */
   getQuality(peerId) {
     const tracker = this._peers.get(peerId);
@@ -377,7 +406,7 @@ class ConnectionQuality extends EventEmitter {
 
   /**
    * Gets quality for all peers
-   * @returns {Object[]}
+   * @returns {any[]}
    */
   getAllQuality() {
     const results = [];
@@ -388,10 +417,15 @@ class ConnectionQuality extends EventEmitter {
   }
 
   /**
-   * Periodic update — recalculates all qualities and emits changes
+   * Periodic update — recalculates all qualities and emits changes.
+   * Stops the timer automatically when no peers are connected.
    * @private
    */
   _update() {
+    if (this._peers.size === 0) {
+      this.stop();
+      return;
+    }
     for (const tracker of this._peers.values()) {
       const quality = tracker.calculate();
       if (quality.changed) {

package/src/mesh/monitor/NetworkMonitor.js

@@ -12,7 +12,7 @@ const EventEmitter = require('../../utils/EventEmitter');
 
 /**
  * Health status levels
- * @constant {Object}
+ * @constant {any}
  */
 const HEALTH_STATUS = Object.freeze({
   GOOD: 'good',
@@ -23,7 +23,7 @@ const HEALTH_STATUS = Object.freeze({
 
 /**
  * Default monitoring configuration
- * @constant {Object}
+ * @constant {any}
  */
 const DEFAULT_CONFIG = Object.freeze({
   /** Sample window size for latency calculations */
@@ -90,21 +90,21 @@ const DEFAULT_CONFIG = Object.freeze({
 class NetworkMonitor extends EventEmitter {
   /**
      * Creates a new NetworkMonitor instance.
-     * @param {Object} [options={}] - Configuration options
+     * @param {any} [options={}] - Configuration options
      */
   constructor(options = {}) {
     super();
 
     /**
          * Configuration
-         * @type {Object}
+         * @type {any}
          * @private
          */
     this._config = { ...DEFAULT_CONFIG, ...options };
 
     /**
          * Node health tracking
-         * @type {Map<string, NodeHealth>}
+         * @type {Map<string, any>}
          * @private
          */
     this._nodes = new Map();
@@ -117,15 +117,36 @@ class NetworkMonitor extends EventEmitter {
     this._pendingMessages = new Map();
 
     /**
-         * Latency samples
-         * @type {number[]}
+         * Latency samples (circular buffer)
+         * @type {Float64Array}
          * @private
          */
-    this._latencies = [];
+    this._latencies = new Float64Array(this._config.latencySampleSize);
+
+    /**
+         * Circular buffer write index
+         * @type {number}
+         * @private
+         */
+    this._latencyIndex = 0;
+
+    /**
+         * Number of latency samples stored
+         * @type {number}
+         * @private
+         */
+    this._latencyCount = 0;
+
+    /**
+         * Running sum of latency samples for O(1) average
+         * @type {number}
+         * @private
+         */
+    this._latencySum = 0;
 
     /**
          * Global statistics
-         * @type {Object}
+         * @type {any}
          * @private
          */
     this._stats = {
@@ -137,7 +158,7 @@ class NetworkMonitor extends EventEmitter {
 
     /**
          * Health check timer
-         * @type {number|null}
+         * @type {any}
          * @private
          */
     this._healthCheckTimer = null;
@@ -149,6 +170,13 @@ class NetworkMonitor extends EventEmitter {
          */
     this._lastHealthReport = null;
 
+    /**
+         * Previous health status for change detection
+         * @type {string|null}
+         * @private
+         */
+    this._previousHealth = null;
+
     // Start health check timer
     this._startHealthCheck();
   }
@@ -330,7 +358,7 @@ class NetworkMonitor extends EventEmitter {
   /**
      * Gets detailed health for a specific node.
      * @param {string} peerId - Peer ID
-     * @returns {NodeHealth|null} Node health or null
+     * @returns {any} Node health or null
      */
   getNodeHealth(peerId) {
     const node = this._nodes.get(peerId);
@@ -347,13 +375,13 @@ class NetworkMonitor extends EventEmitter {
 
   /**
      * Gets health information for all nodes.
-     * @returns {NodeHealth[]} Array of node health
+     * @returns {any[]} Array of node health
      */
   getAllNodeHealth() {
     const now = Date.now();
     const timeout = this._config.nodeTimeoutMs;
 
-    return Array.from(this._nodes.values()).map(node => ({
+    return Array.from(this._nodes.values()).map((/** @type {any} */ node) => ({
       ...node,
       isActive: (now - node.lastSeen) < timeout
     }));
@@ -369,7 +397,7 @@ class NetworkMonitor extends EventEmitter {
 
   /**
      * Gets monitoring statistics.
-     * @returns {Object} Statistics
+     * @returns {any} Statistics
      */
   getStats() {
     return {
@@ -386,7 +414,10 @@ class NetworkMonitor extends EventEmitter {
   reset() {
     this._nodes.clear();
     this._pendingMessages.clear();
-    this._latencies = [];
+    this._latencies = new Float64Array(this._config.latencySampleSize);
+    this._latencyIndex = 0;
+    this._latencyCount = 0;
+    this._latencySum = 0;
     this._stats = {
       totalMessagesSent: 0,
       totalMessagesDelivered: 0,
@@ -409,7 +440,7 @@ class NetworkMonitor extends EventEmitter {
   /**
      * Gets or creates a node entry.
      * @param {string} peerId - Peer ID
-     * @returns {NodeHealth} Node entry
+     * @returns {any} Node entry
      * @private
      */
   _getOrCreateNode(peerId) {
@@ -430,20 +461,27 @@ class NetworkMonitor extends EventEmitter {
   }
 
   /**
-     * Adds a latency sample.
+     * Adds a latency sample using circular buffer.
      * @param {number} latency - Latency in ms
      * @private
      */
   _addLatencySample(latency) {
-    this._latencies.push(latency);
-    if (this._latencies.length > this._config.latencySampleSize) {
-      this._latencies.shift();
+    const capacity = this._latencies.length;
+    // Subtract the old value being overwritten if buffer is full
+    if (this._latencyCount >= capacity) {
+      this._latencySum -= this._latencies[this._latencyIndex];
+    }
+    this._latencies[this._latencyIndex] = latency;
+    this._latencySum += latency;
+    this._latencyIndex = (this._latencyIndex + 1) % capacity;
+    if (this._latencyCount < capacity) {
+      this._latencyCount++;
     }
   }
 
   /**
      * Updates node latency with exponential moving average.
-     * @param {NodeHealth} node - Node to update
+     * @param {any} node - Node to update
      * @param {number} latency - New latency sample
      * @private
      */
@@ -458,7 +496,7 @@ class NetworkMonitor extends EventEmitter {
 
   /**
      * Updates node packet loss rate.
-     * @param {NodeHealth} node - Node to update
+     * @param {any} node - Node to update
      * @private
      */
   _updateNodePacketLoss(node) {
@@ -469,17 +507,30 @@ class NetworkMonitor extends EventEmitter {
   }
 
   /**
-     * Calculates average latency from samples.
+     * Calculates average latency from samples (O(1) using running sum).
      * @returns {number} Average latency
      * @private
      */
   _calculateAverageLatency() {
-    if (this._latencies.length === 0) {
+    if (this._latencyCount === 0) {
       return 0;
     }
-    return Math.round(
-      this._latencies.reduce((a, b) => a + b, 0) / this._latencies.length
-    );
+    return Math.round(this._latencySum / this._latencyCount);
+  }
+
+  /**
+     * Cleans up stale pending messages older than nodeTimeoutMs.
+     * @private
+     */
+  _cleanupPendingMessages() {
+    const now = Date.now();
+    const timeout = this._config.nodeTimeoutMs;
+    for (const [messageId, pending] of this._pendingMessages) {
+      if (now - pending.timestamp > timeout) {
+        this._pendingMessages.delete(messageId);
+        this._stats.totalMessagesFailed++;
+      }
+    }
   }
 
   /**
@@ -520,9 +571,10 @@ class NetworkMonitor extends EventEmitter {
     this._healthCheckTimer = setInterval(
       () => {
         try {
+          this._cleanupPendingMessages();
           const report = this.generateHealthReport();
           this.emit('health-report', report);
-        } catch (error) {
+        } catch (_error) {
           // Don't let health check errors crash the monitor
         }
       },

package/src/mesh/peer/Peer.js

@@ -8,6 +8,9 @@
 const { CONNECTION_STATE } = require('../../constants');
 const { ValidationError } = require('../../errors');
 
+/** Pre-computed Set of valid connection states for O(1) lookup */
+const CONNECTION_STATE_SET = new Set(Object.values(CONNECTION_STATE));
+
 /**
  * Represents a peer in the mesh network
  * @class Peer
@@ -15,12 +18,7 @@ const { ValidationError } = require('../../errors');
 class Peer {
   /**
    * Creates a new Peer
-   * @param {Object} options - Peer options
-   * @param {string} options.id - Unique peer identifier
-   * @param {Uint8Array} [options.publicKey] - Peer's public key
-   * @param {string} [options.name] - Display name
-   * @param {number} [options.rssi] - Signal strength
-   * @param {number} [options.hopCount] - Distance in hops
+   * @param {any} options - Peer options   *
    */
   constructor(options) {
     if (!options || typeof options.id !== 'string' || options.id.length === 0) {
@@ -85,7 +83,7 @@ class Peer {
 
     /**
      * Additional metadata
-     * @type {Object}
+     * @type {any}
      */
     this.metadata = options.metadata || {};
   }
@@ -130,9 +128,9 @@ class Peer {
    * @param {string} state - New connection state
    */
   setConnectionState(state) {
-    if (!Object.values(CONNECTION_STATE).includes(state)) {
+    if (!CONNECTION_STATE_SET.has(state)) {
       throw ValidationError.invalidArgument('state', state, {
-        validValues: Object.values(CONNECTION_STATE)
+        validValues: Array.from(CONNECTION_STATE_SET)
       });
     }
     this.connectionState = state;
@@ -198,7 +196,7 @@ class Peer {
 
   /**
    * Converts peer to a JSON-serializable object
-   * @returns {Object} JSON representation
+   * @returns {any} JSON representation
    */
   toJSON() {
     return {
@@ -219,7 +217,7 @@ class Peer {
 
   /**
    * Creates a Peer from JSON data
-   * @param {Object} data - JSON data
+   * @param {any} data - JSON data
    * @returns {Peer} New Peer instance
    */
   static fromJSON(data) {

package/src/mesh/peer/PeerDiscovery.js

@@ -16,7 +16,7 @@ const { ValidationError } = require('../../errors');
  * @property {string} [name] - Display name
  * @property {number[]} [publicKey] - Public key bytes
  * @property {number} [hopCount] - Hop count
- * @property {Object} [capabilities] - Peer capabilities
+ * @property {any} [capabilities] - Peer capabilities
  */
 
 /**
@@ -27,10 +27,7 @@ const { ValidationError } = require('../../errors');
 class PeerDiscovery extends EventEmitter {
   /**
    * Creates a new PeerDiscovery handler
-   * @param {Object} options - Configuration options
-   * @param {string} options.localPeerId - Local peer's ID
-   * @param {Uint8Array} [options.publicKey] - Local peer's public key
-   * @param {string} [options.displayName] - Local display name
+   * @param {any} options - Configuration options   *
    */
   constructor(options = {}) {
     super();
@@ -59,13 +56,13 @@ class PeerDiscovery extends EventEmitter {
 
     /**
      * Capabilities advertised to peers
-     * @type {Object}
+     * @type {any}
      */
     this.capabilities = options.capabilities || {};
 
     /**
      * Discovery statistics
-     * @type {Object}
+     * @type {any}
      * @private
      */
     this._stats = {
@@ -80,9 +77,10 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Creates an announce message payload
-   * @returns {Object} Announce payload
+   * @returns {any} Announce payload
    */
   createAnnouncePayload() {
+    /** @type {any} */
     const payload = {
       peerId: this.localPeerId,
       name: this.displayName,
@@ -100,8 +98,9 @@ class PeerDiscovery extends EventEmitter {
   /**
    * Creates a peer request payload
    * @param {string} [targetPeerId] - Specific peer to request (null for broadcast)
-   * @returns {Object} Request payload
+   * @returns {any} Request payload
    */
+  // @ts-ignore
   createRequestPayload(targetPeerId = null) {
     return {
       requesterId: this.localPeerId,
@@ -113,7 +112,7 @@ class PeerDiscovery extends EventEmitter {
   /**
    * Creates a peer response payload
    * @param {string} requesterId - ID of requesting peer
-   * @returns {Object} Response payload
+   * @returns {any} Response payload
    */
   createResponsePayload(requesterId) {
     return {
@@ -129,9 +128,9 @@ class PeerDiscovery extends EventEmitter {
   /**
    * Processes an incoming discovery message
    * @param {number} type - Message type
-   * @param {Object} payload - Message payload
+   * @param {any} payload - Message payload
    * @param {string} sourcePeerId - Source peer ID
-   * @returns {Object|null} Response to send, or null
+   * @returns {any} Response to send, or null
    */
   processMessage(type, payload, sourcePeerId) {
     // Ignore our own messages
@@ -158,7 +157,7 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Handles a peer announce message
-   * @param {Object} payload - Announce payload
+   * @param {any} payload - Announce payload
    * @param {string} sourcePeerId - Source peer ID
    * @returns {null} No response needed
    * @private
@@ -181,9 +180,9 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Handles a peer request message
-   * @param {Object} payload - Request payload
-   * @param {string} sourcePeerId - Source peer ID
-   * @returns {Object|null} Response payload or null
+   * @param {any} payload - Request payload
+   * @param {string} _sourcePeerId - Source peer ID
+   * @returns {any} Response payload or null
    * @private
    */
   _handleRequest(payload, _sourcePeerId) {
@@ -203,7 +202,7 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Handles a peer response message
-   * @param {Object} payload - Response payload
+   * @param {any} payload - Response payload
    * @param {string} sourcePeerId - Source peer ID
    * @returns {null} No response needed
    * @private
@@ -230,7 +229,7 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Updates local peer information
-   * @param {Object} info - New info
+   * @param {any} info - New info
    */
   updateLocalInfo(info) {
     if (info.displayName !== undefined) { this.displayName = info.displayName; }
@@ -242,7 +241,7 @@ class PeerDiscovery extends EventEmitter {
 
   /**
    * Gets discovery statistics
-   * @returns {Object} Statistics
+   * @returns {any} Statistics
    */
   getStats() {
     return { ...this._stats };