Fix for #46

Author: NorthernMan54

.gitignore

@@ -127,3 +127,4 @@ DEVICES.md.toc.*
 /test/hbConfig/accessories
 /test/hbConfig/backups
 /test/hbConfig/persist
+test/hbConfig/config.json

config.schema.json

@@ -39,8 +39,8 @@
         "title": "Disconnected Device Cleanup Timing",
         "type": "number",
         "required": false,
-        "placeholder": 24,
-        "description": "Cleanup of devices no longer connected too or responding on the network.  Time is in hours, defaults to 1 Day.  Maximum value is 596 hours ( 24 days )."
+        "default": 24,
+        "description": "Cleanup of devices no longer connected too or responding on the network.  Time is in hours, defaults to 1 Day.  Maximum value is 596 hours ( 24 days ).  IF set to 0, cleanup is disabled."
       },
       "debug": {
         "title": "Debug Logging",
@@ -147,14 +147,15 @@
       "type": "fieldset",
       "title": "Required Settings",
       "items": [
-          "mqttHost"
+        "mqttHost"
       ]
     },
     {
       "type": "fieldset",
       "expandable": true,
       "title": "Optional Settings",
-      "items": [{
+      "items": [
+        {
           "type": "flex",
           "flex-flow": "row wrap",
           "items": [
@@ -174,7 +175,8 @@
       "expandable": true,
       "title": "Device Discovery Filtering",
       "description": "Control which devices are discovered and included with Homebridge.",
-      "items": [{
+      "items": [
+        {
           "key": "filterAllow",
           "type": "array",
           "required": false,
@@ -202,28 +204,30 @@
       "expandable": true,
       "title": "MQTT Discovery Message Injection",
       "description": "Ability to inject/over write portions of a MQTT Discovery message.",
-      "items": [{
+      "items": [
+        {
           "key": "injections",
           "type": "array",
-
           "items": [
             "injections[].topic",
             {
               "key": "injections[].injection",
               "type": "array",
-              "items": [{
-                "type": "flex",
-                "flex-flow": "row wrap",
-                "items": [
-                  "injections[].injection[].key",
-                  "injections[].injection[].value"
-                ]
-              }]
+              "items": [
+                {
+                  "type": "flex",
+                  "flex-flow": "row wrap",
+                  "items": [
+                    "injections[].injection[].key",
+                    "injections[].injection[].value"
+                  ]
+                }
+              ]
             }
           ]
         },
         "filter"
       ]
     }
   ]
-}
+}

package.json

@@ -1,7 +1,7 @@
 {
   "displayName": "Tasmota",
   "name": "homebridge-tasmota",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "type": "module",
   "description": "Homebridge plugin for Tasmota devices leveraging home assistant auto discovery.",
   "license": "Apache-2.0",

src/tasmotaPlatform.ts

@@ -91,7 +91,7 @@ export class tasmotaPlatform implements DynamicPlatformPlugin {
     this.CustomCharacteristics = new EveHomeKitTypes(this.api).Characteristics;
 
     this.log.debug('Finished initializing platform:', this.config.name);
-    this.cleanup = this.config.cleanup || 24; // Default removal of defunct devices after 24 hours
+    this.cleanup = (this.config.cleanup !== undefined ? this.config.cleanup : 24); // #46
     this.debug = this.config.debug || false;
     this.teleperiod = this.config.teleperiod || 300;
 
@@ -178,7 +178,9 @@ export class tasmotaPlatform implements DynamicPlatformPlugin {
 
     if (accessory.services.length > 1) {
       // debug('context', accessory.context);
-      accessory.context.timeout = this.autoCleanup(accessory);
+      if (this.cleanup > 0) {
+        accessory.context.timeout = this.autoCleanup(accessory);
+      }
 
       // add the restored accessory to the accessories cache so we can track if it has already been registered
       this.accessories.push(accessory);