Update 26.01.25 #2

README.md

@@ -76,6 +76,13 @@ With the ioBroker Matter Adapter, it is possible to map the following use cases:
 -->
 
 ## Changelog
+
+### __WORK IN PROGRESS__
+* (@Apollon77) Enhanced error and invalid devices display for UI
+* (@Apollon77) Fixed Button Press Controller support
+* (@Apollon77) Added support to also select folders when adding devices
+* (@Apollon77) Fixed Illuminance State type min/max
+
 ### 0.4.8 (2025-01-26)
 * (@Apollon77) Acknowledges Power states also on SET states
 * (@Apollon77) Fixed Color Temperature handling for devices

src-admin/src/Tabs/Bridges.tsx

@@ -884,7 +884,7 @@ export class Bridges extends BridgesAndDevices<BridgesProps, BridgesState> {
                 <SelectID
                     imagePrefix="../.."
                     dialogName="matter"
-                    types={addDeviceDialog.detectionType === 'device' ? ['device', 'channel'] : ['state']}
+                    types={addDeviceDialog.detectionType === 'device' ? ['device', 'channel', 'folder'] : ['state']}
                     themeType={this.props.themeType}
                     socket={this.props.socket}
                     theme={this.props.theme}

src-admin/src/Tabs/Devices.tsx

@@ -592,7 +592,7 @@ class Devices extends BridgesAndDevices<DevicesProps, DevicesState> {
             return (
                 <SelectID
                     imagePrefix="../.."
-                    types={addDeviceDialog.detectionType === 'device' ? ['device', 'channel'] : ['state']}
+                    types={addDeviceDialog.detectionType === 'device' ? ['device', 'channel', 'folder'] : ['state']}
                     dialogName="matter"
                     themeType={this.props.themeType}
                     socket={this.props.socket}

src/lib/devices/Illuminance.ts

@@ -11,7 +11,7 @@ export class Illuminance extends GenericDevice {
             this.addDeviceStates([
                 {
                     name: 'ACTUAL',
-                    valueType: ValueType.NumberPercent,
+                    valueType: ValueType.Number,
                     accessType: StateAccessType.Read,
                     type: PropertyType.Brightness,
                     callback: state => (this.#getBrightnessState = state),

src/main.ts

@@ -624,7 +624,7 @@ export class MatterAdapter extends utils.Adapter {
                 (objParts[0] === 'bridges' && objPartsLength === 2)) &&
             !obj
         ) {
-            this.log.warn(`Object ${id} deleted ... trying to also remove it from matter`);
+            this.log.debug(`Object ${id} deleted ... trying to also remove it from matter`);
             // We try to restore a minimum object that we can handle the deletion
             obj = {
                 _id: id,

src/matter/BridgedDevicesNode.ts

@@ -402,19 +402,22 @@ class BridgedDevices extends BaseServerNode {
 
         if (bridgedDeviceUuid === undefined) {
             const error = this.error;
-            if (Array.isArray(error)) {
+            if (error) {
                 details.error = {
                     __header__error: 'Error information',
-                    __text__info: `${error.length} Bridged Device(s) are in an error state. Fix the errors before enabling it again.`,
+                    __text__info: Array.isArray(error)
+                        ? `${error.length} Bridged Device(s) are in an error state. Fix the errors before enabling it again.`
+                        : `The Bridge could not be initialized. Please check the logfile for more information.`,
                     __text__info2: `Please refer to the error details at the bridged device level.`,
                     uuid: this.uuid,
                 };
             } else {
                 // The error boolean state should never end here because then this object should have not been created
             }
         } else {
-            const { error } = this.#devices.get(bridgedDeviceUuid) ?? {};
-            if (error) {
+            const { error, device } = this.#devices.get(bridgedDeviceUuid) ?? {};
+            const isValid = device?.isValid;
+            if (error || !isValid) {
                 details.error = {
                     __header__error: 'Error information',
                     __text__info: `Bridged Device is in an error state. Fix the error before enabling it again.`,

src/matter/to-iobroker/GenericDeviceToIoBroker.ts

@@ -1,4 +1,11 @@
-import { type AttributeId, type ClusterId, Diagnostic, EndpointNumber, type EventId, MaybePromise } from '@matter/main';
+import {
+    type MaybePromise,
+    type AttributeId,
+    type ClusterId,
+    type EventId,
+    Diagnostic,
+    EndpointNumber,
+} from '@matter/main';
 import { BasicInformation, BridgedDeviceBasicInformation, Identify, PowerSource } from '@matter/main/clusters';
 import type { DecodedEventData } from '@matter/main/protocol';
 import type { Endpoint, PairedNode, DeviceBasicInformation } from '@project-chip/matter.js/device';
@@ -355,10 +362,7 @@ export abstract class GenericDeviceToIoBroker {
         }
         const { convertValue } = properties;
         if (convertValue !== undefined) {
-            value = convertValue(value);
-            if (MaybePromise.is(value)) {
-                value = await value;
-            }
+            value = await convertValue(value);
         }
         if (value !== undefined) {
             try {
@@ -386,13 +390,10 @@ export abstract class GenericDeviceToIoBroker {
             return;
         }
         if (typeof pathProperty === 'function') {
-            const result = pathProperty(data.value);
-            if (MaybePromise.is(result)) {
-                await result;
-            }
+            await pathProperty(data.value);
             return;
         }
-        return this.updateIoBrokerState(pathProperty, data.value);
+        await this.updateIoBrokerState(pathProperty, data.value);
     }
 
     async handleTriggeredEvent(data: {
@@ -404,15 +405,15 @@ export abstract class GenericDeviceToIoBroker {
     }): Promise<void> {
         const pathId = eventPathToString(data);
         const pathProperty = this.#matterMappings.get(pathId);
+        this.#adapter.log.debug(
+            `Handle event ${pathId} with property type ${typeof pathProperty === 'function' ? 'function' : pathProperty}`,
+        );
         if (pathProperty === undefined) {
             return;
         }
         if (typeof pathProperty === 'function') {
             for (const event of data.events) {
-                const result = pathProperty(event);
-                if (MaybePromise.is(result)) {
-                    await result;
-                }
+                await pathProperty(event);
             }
             return;
         }
@@ -422,12 +423,14 @@ export abstract class GenericDeviceToIoBroker {
         }
         for (const { convertValue } of propertyHandlers) {
             for (const event of data.events) {
-                let value = convertValue(pathProperty, event);
+                const value = await convertValue(pathProperty, event);
                 if (value !== undefined) {
-                    if (MaybePromise.is(value)) {
-                        value = await value;
+                    this.#adapter.log.debug(`Update state for ${pathId} and type ${pathProperty} with value ${value}`);
+                    try {
+                        await this.ioBrokerDevice.updatePropertyValue(pathProperty, value);
+                    } catch (e) {
+                        this.#adapter.log.error(`Error updating property ${pathProperty} with value ${value}: ${e}`);
                     }
-                    await this.updateIoBrokerState(pathProperty, value);
                 }
             }
         }

src/matter/to-matter/GenericDeviceToMatter.ts

@@ -49,6 +49,9 @@ export abstract class GenericDeviceToMatter {
         await this.registerHandlersAndInitialize();
         this.ioBrokerDevice.on('validChanged', () => {
             const valid = this.ioBrokerDevice.isValid;
+            this.ioBrokerDevice.adapter.log.info(
+                `Device ${this.name} is now ${valid ? 'valid' : 'invalid'} (before: ${this.#valid ? 'valid' : 'invalid'})`,
+            );
             if (valid === this.#valid) {
                 return;
             }