matterAllowNonCompliantSinglePress

Author: donavanbecker

config.schema.json

@@ -76,6 +76,12 @@
               { "title": "Debug — visible only with global debug", "enum": ["debug"] },
               { "title": "Silent — never logged", "enum": ["silent"] }
             ]
+          },
+          "matterAllowNonCompliantSinglePress": {
+            "type": "boolean",
+            "title": "Allow non-compliant Matter single-press-only mode (breaks spec, hides double/long press in Home app)",
+            "description": "If enabled, and double press is disabled, exposes Pico remotes as single-press only in Matter (multiPressMax: 1). This breaks the Matter spec but improves Home app UI. Requires re-pairing to take effect.",
+            "default": false
           }
 
         }
@@ -164,6 +170,12 @@
               "notitle": false,
               "type": "string",
               "flex": "1 1 150px"
+            },
+            {
+              "key": "options.matterAllowNonCompliantSinglePress",
+              "notitle": false,
+              "type": "boolean",
+              "flex": "1 1 300px"
             }
           ]
 

package-lock.json

@@ -22,7 +22,7 @@
   ],
   "main": "dist/index.js",
   "engines": {
-    "homebridge": "^1.11.4 || ^2.0.0-beta.111",
+    "homebridge": "^2.0.0",
     "node": "^22 || ^24"
   },
   "scripts": {
@@ -56,7 +56,7 @@
     "eslint-plugin-format": "^2.0.1",
     "eslint-plugin-html": "^8.1.4",
     "eslint-plugin-perfectionist": "^5.8.0",
-    "homebridge": "^2.0.0-beta.111",
+    "homebridge": "^2.0.0",
     "homebridge-config-ui-x": "^5.21.0",
     "nodemon": "^3.1.14",
     "shx": "^0.4.0",

package.json

@@ -145,30 +145,55 @@ describe('picoRemote.getMatterClusters', () => {
       },
     }
 
-    const { platform, accessory } = createPlatformAndAccessory()
-    const remote = new PicoRemote(
-      platform,
-      accessory,
+    // Test with both double and long press disabled
+    // If non-compliant option is OFF, multiPressMax should be 2 (spec-compliant)
+    const { platform: platform1, accessory: accessory1 } = createPlatformAndAccessory()
+    const remote1 = new PicoRemote(
+      platform1,
+      accessory1,
       {} as any,
       createOptions({
         clickSpeedDouble: 'disabled',
         clickSpeedLong: 'disabled',
+        matterAllowNonCompliantSinglePress: false,
       }),
       {
         deviceTypes: {
           GenericSwitch: genericSwitchDeviceType,
         },
       },
     )
-
-    const clusters = remote.getMatterClusters()
-    const parts = (clusters as any).parts
-
-    expect(parts).toHaveLength(2)
-    for (const part of parts) {
-      // Matter spec requires multiPressMax >= 2; always 2 regardless of double-press config
+    const clusters1 = remote1.getMatterClusters()
+    const parts1 = (clusters1 as any).parts
+    expect(parts1).toHaveLength(2)
+    for (const part of parts1) {
       expect(part.clusters.switch.multiPressMax).toBe(2)
       expect(part.clusters.switch.longPressTime).toBeUndefined()
     }
+
+    // If non-compliant option is ON, multiPressMax should be undefined
+    const { platform: platform2, accessory: accessory2 } = createPlatformAndAccessory()
+    const remote2 = new PicoRemote(
+      platform2,
+      accessory2,
+      {} as any,
+      createOptions({
+        clickSpeedDouble: 'disabled',
+        clickSpeedLong: 'disabled',
+        matterAllowNonCompliantSinglePress: true,
+      }),
+      {
+        deviceTypes: {
+          GenericSwitch: genericSwitchDeviceType,
+        },
+      },
+    )
+    const clusters2 = remote2.getMatterClusters()
+    const parts2 = (clusters2 as any).parts
+    expect(parts2).toHaveLength(2)
+    for (const part of parts2) {
+      expect(part.clusters.switch.multiPressMax).toBeUndefined()
+      expect(part.clusters.switch.longPressTime).toBeUndefined()
+    }
   })
 })

src/PicoRemote.test.ts

@@ -257,8 +257,9 @@ export class PicoRemote {
       }
       for (const button of buttons) {
         const pmHref = (button.ProgrammingModel as { href?: string } | undefined)?.href
-        if (!pmHref)
+        if (!pmHref) {
           continue
+        }
         let pm: { Preset?: { href?: string } } | undefined
         try {
           const resp = await this.bridge.getHref({ href: pmHref } as any) as any
@@ -268,8 +269,9 @@ export class PicoRemote {
           continue
         }
         const presetHref = pm?.Preset?.href
-        if (!presetHref)
+        if (!presetHref) {
           continue
+        }
         let preset: unknown
         try {
           const resp = await this.bridge.getHref({ href: presetHref } as any) as any
@@ -569,19 +571,21 @@ export class PicoRemote {
     const sortedAliases = Array.from(dentry.values()).sort((a, b) => a.index - b.index)
     this.platform.log.debug(`[Matter] Creating ${sortedAliases.length} button parts for '${type}'`)
     const isLongPressEnabled = this.options.clickSpeedLong !== 'disabled'
+    const isDoublePressEnabled = this.options.clickSpeedDouble !== 'disabled'
+    const allowNonCompliant = !!this.options.matterAllowNonCompliantSinglePress
 
     const parts = sortedAliases.map((alias) => {
       const switchCluster: Record<string, number> = {
         currentPosition: 0,
         numberOfPositions: 2, // Button has 2 positions: unpressed (0) and pressed (1)
       }
 
-      // Matter's Switch cluster uses multiPressMax to advertise multi-press support.
-      // The Matter spec requires multiPressMax >= 2; use 2 regardless of whether
-      // double-press is enabled (we simply won't emit multi-press events when disabled).
-      switchCluster.multiPressMax = 2
+      // Always set multiPressMax = 2 if non-compliant option is off (spec-compliant), regardless of double press config
+      if (!allowNonCompliant) {
+        switchCluster.multiPressMax = 2
+      }
 
-      // longPressTime indicates long-press capability. Omit it when disabled.
+      // Only set longPressTime if long press is enabled
       if (isLongPressEnabled) {
         switchCluster.longPressTime = 1000
       }
@@ -625,13 +629,16 @@ export class PicoRemote {
 // with a non-empty array counts, which makes the check forward-compatible
 // with future LEAP types while ignoring unrelated array fields LEAP may add.
 export function presetIsProgrammed(preset: unknown): boolean {
-  if (!preset || typeof preset !== 'object')
+  if (!preset || typeof preset !== 'object') {
     return false
+  }
   for (const [k, v] of Object.entries(preset as Record<string, unknown>)) {
-    if (!k.endsWith('Assignments'))
+    if (!k.endsWith('Assignments')) {
       continue
-    if (Array.isArray(v) && v.length > 0)
+    }
+    if (Array.isArray(v) && v.length > 0) {
       return true
+    }
   }
   return false
 }

src/PicoRemote.ts

@@ -35,6 +35,11 @@ interface PlatformEvents {
 
 // see config.schema.json
 export interface GlobalOptions {
+  /**
+   * If true, and double press is disabled, expose Pico remotes as single-press only in Matter (multiPressMax: 1).
+   * This breaks the Matter spec but may improve Home app UI. Requires re-pairing to take effect.
+   */
+  matterAllowNonCompliantSinglePress?: boolean
   filterPico: boolean
   excludedDeviceTypes: string[]
   clickSpeedLong: 'quick' | 'default' | 'relaxed' | 'disabled'
@@ -168,7 +173,9 @@ export class LutronCasetaLeap
         // second unhandledRejection event rather than a process exit, so we
         // use process.nextTick() to break out of the handler's call stack.
         this.log.warn('Unhandled promise rejection (not a known lutron-leap timeout):', reason)
-        process.nextTick(() => { throw reason })
+        process.nextTick(() => {
+          throw reason
+        })
       })
     }
 

src/Platform.HAP.ts

@@ -8,6 +8,13 @@ <h4 class="card-title mt-2">Discovered Bridges</h4>
 
         <h4 class="card-title">Global Options</h4>
         <form id='optionsForm' class="form-horizontal">
+                <div class="form-check row py-sm-3 mb-0">
+                    <input class="form-check-input col-sm-1" type="checkbox" value="" id="matterAllowNonCompliantSinglePressChk"/>
+                    <label class="form-check-label col-sm-11 text-left" for="matterAllowNonCompliantSinglePressChk">
+                        Allow non-compliant Matter single-press-only mode (breaks spec, hides double/long press in Home app)
+                    </label>
+                    <br/>
+                </div>
 
                 <div class="form-check row py-sm-3 mb-0">
                     <input class="form-check-input col-sm-1" type="checkbox" value="" id="filterPicoChk"/>
@@ -146,6 +153,7 @@ <h4 class="card-title">Global Options</h4>
         document.querySelector('#clickSpeedLongSelect').value = pluginConfig[0].options.clickSpeedLong;
         document.querySelector('#clickSpeedDoubleSelect').value = pluginConfig[0].options.clickSpeedDouble;
         setSelectedDeviceTypes(pluginConfig[0].options.excludedDeviceTypes);
+        document.querySelector('#matterAllowNonCompliantSinglePressChk').checked = !!pluginConfig[0].options.matterAllowNonCompliantSinglePress;
 
         document.getElementById('optionsForm').addEventListener('input', () => {
             console.log("options clicked");
@@ -154,6 +162,7 @@ <h4 class="card-title">Global Options</h4>
             pluginConfig[0].options.clickSpeedLong = document.querySelector('#clickSpeedLongSelect').value;
             pluginConfig[0].options.clickSpeedDouble = document.querySelector('#clickSpeedDoubleSelect').value;
             pluginConfig[0].options.excludedDeviceTypes = getSelectedDeviceTypes();
+            pluginConfig[0].options.matterAllowNonCompliantSinglePress = document.querySelector('#matterAllowNonCompliantSinglePressChk').checked;
             console.log(pluginConfig[0].options);
             homebridge.updatePluginConfig(pluginConfig);
         });