Update PicoRemote.ts

Author: donavanbecker

src/PicoRemote.ts

@@ -403,7 +403,7 @@ export class PicoRemote {
     const buttonNumber = this.hrefToButtonNumber.get(buttonHref)
     const alias = this.hrefToAlias.get(buttonHref)
 
-    // GenericSwitch path (Homebridge PR #3924): update switch position on button press.
+    // Composed OnOffSwitch path: pulse onOff for the pressed part.
     if (
       this.matterApi
       && buttonNumber !== undefined
@@ -415,17 +415,35 @@ export class PicoRemote {
 
       if (hasComposedParts) {
         void this.matterApi
-          .updateAccessoryState(this.accessory.UUID, 'switch', { currentPosition: 1 }, partId)
+          .updateAccessoryState(this.accessory.UUID, 'onOff', { onOff: true }, partId)
           .catch((error: unknown) => {
-            this.platform.log.debug(`[Matter] Failed to update composed switch part ${partId} for ${this.accessory.displayName}: ${String(error)}`)
+            this.platform.log.debug(`[Matter] Failed to update composed onOff part ${partId} for ${this.accessory.displayName}: ${String(error)}`)
           })
-      } else if ((this.accessory as any).clusters?.switch) {
+      } else if ((this.accessory as any).clusters?.onOff) {
         void this.matterApi
-          .updateAccessoryState(this.accessory.UUID, 'switch', { currentPosition: alias.index })
+          .updateAccessoryState(this.accessory.UUID, 'onOff', { onOff: true })
           .catch((error: unknown) => {
-            this.platform.log.debug(`[Matter] Failed to update switch position for ${this.accessory.displayName}: ${String(error)}`)
+            this.platform.log.debug(`[Matter] Failed to update onOff state for ${this.accessory.displayName}: ${String(error)}`)
           })
       }
+
+      // Stateless behavior: clear the pressed state after a short pulse so the
+      // button can be triggered repeatedly.
+      setTimeout(() => {
+        if (hasComposedParts) {
+          void this.matterApi
+            .updateAccessoryState(this.accessory.UUID, 'onOff', { onOff: false }, partId)
+            .catch((error: unknown) => {
+              this.platform.log.debug(`[Matter] Failed to clear composed onOff part ${partId} for ${this.accessory.displayName}: ${String(error)}`)
+            })
+        } else if ((this.accessory as any).clusters?.onOff) {
+          void this.matterApi
+            .updateAccessoryState(this.accessory.UUID, 'onOff', { onOff: false })
+            .catch((error: unknown) => {
+              this.platform.log.debug(`[Matter] Failed to clear onOff state for ${this.accessory.displayName}: ${String(error)}`)
+            })
+        }
+      }, 250)
     }
     const fullName = this.accessory.context.device.FullyQualifiedName.join(' ')
     // Raw Press/Release event from the bridge — fires twice per physical
@@ -462,12 +480,12 @@ export class PicoRemote {
       return {}
     }
 
-    // Expose Pico remotes as composed Matter endpoints: one GenericSwitch-like
+    // Expose Pico remotes as composed Matter endpoints: one OnOffSwitch-like
     // part per physical button. This aligns better with Home's separate button
     // presentation than a single multi-position switch endpoint.
-    const partDeviceType = this.matterApi?.deviceTypes?.GenericSwitch
+    const partDeviceType = this.matterApi?.deviceTypes?.OnOffSwitch
     if (!partDeviceType) {
-      this.platform.log.warn(`[Matter] GenericSwitch deviceType is unavailable for '${type}', returning empty clusters.`)
+      this.platform.log.warn(`[Matter] OnOffSwitch deviceType is unavailable for '${type}', returning empty clusters.`)
       return {}
     }
 
@@ -479,9 +497,8 @@ export class PicoRemote {
         displayName: alias.label,
         deviceType: partDeviceType,
         clusters: {
-          switch: {
-            currentPosition: 0,
-            numberOfPositions: 1,
+          onOff: {
+            onOff: false,
           },
         },
       }