samples: matter: Add "list" command to bridge shell

The new command lists all bridged devices that are registered
in the device.

- Disabled LTO and minimal SHELL for nRF54LM20 to improve user
experience.

Signed-off-by: Arkadiusz Balys <[email protected]>

Author: ArekBalysNordic

applications/matter_bridge/boards/nrf5340dk_nrf5340_cpuapp.conf

@@ -22,3 +22,7 @@ CONFIG_BT_MAX_PAIRED=9
 CONFIG_BRIDGE_MAX_DYNAMIC_ENDPOINTS_NUMBER=18
 # 15.4 RX buffers were decreased to save RAM
 CONFIG_NRF_802154_RX_BUFFERS=14
+
+# Enable LTO to decrease the flash usage.
+CONFIG_LTO=y
+CONFIG_ISR_TABLES_LOCAL_DECLARATION=y

applications/matter_bridge/boards/nrf54lm20dk_nrf54lm20a_cpuapp.conf

@@ -23,3 +23,6 @@ CONFIG_BT_MAX_PAIRED=9
 
 # Assume that every bridged device uses two endpoints, however it can be increased if specific use case requires it.
 CONFIG_BRIDGE_MAX_DYNAMIC_ENDPOINTS_NUMBER=18
+
+# Enable full SHELL support to improve user experience
+CONFIG_SHELL_MINIMAL=n

applications/matter_bridge/boards/nrf7002dk_nrf5340_cpuapp.conf

@@ -0,0 +1,9 @@
+#
+# Copyright (c) 2025 Nordic Semiconductor ASA
+#
+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+# Enable LTO to decrease the flash usage.
+CONFIG_LTO=y
+CONFIG_ISR_TABLES_LOCAL_DECLARATION=y

applications/matter_bridge/prj.conf

@@ -53,9 +53,5 @@ CONFIG_USE_SEGGER_RTT=n
 CONFIG_CHIP_FACTORY_DATA=y
 CONFIG_CHIP_FACTORY_DATA_BUILD=y
 
-# Enable LTO to decrease the flash usage.
-CONFIG_LTO=y
-CONFIG_ISR_TABLES_LOCAL_DECLARATION=y
-
 # Increase heap size to accommodate the memory required for additional bridged devices.
 CONFIG_CHIP_MALLOC_SYS_HEAP_SIZE=16384

applications/matter_bridge/src/bridge_shell.cpp

@@ -64,7 +64,9 @@ static int AddBridgedDeviceHandler(const struct shell *shell, size_t argc, char
 	    CHIP_NO_ERROR) {
 		shell_fprintf(shell, SHELL_ERROR, "Invalid Bluetooth LE device index.\n");
 	} else {
-		shell_fprintf(shell, SHELL_ERROR, "Found device address\n");
+		shell_fprintf(shell, SHELL_INFO, "Found device address: %02x:%02x:%02x:%02x:%02x:%02x\n",
+			      address.a.val[5], address.a.val[4], address.a.val[3], address.a.val[2], address.a.val[1],
+			      address.a.val[0]);
 	}
 
 	uint16_t uuid;
@@ -125,6 +127,63 @@ static int RemoveBridgedDeviceHandler(const struct shell *shell, size_t argc, ch
 	return 0;
 }
 
+static const char *GetDeviceTypeString(uint16_t deviceType)
+{
+	using DeviceType = Nrf::MatterBridgedDevice::DeviceType;
+	switch (deviceType) {
+	case DeviceType::BridgedNode:
+		return "BridgedNode (0x0013)";
+	case DeviceType::OnOffLight:
+		return "OnOffLight (0x0100)";
+	case DeviceType::OnOffLightSwitch:
+		return "OnOffLightSwitch (0x0103)";
+	case DeviceType::TemperatureSensor:
+		return "TemperatureSensor (0x0302)";
+	case DeviceType::HumiditySensor:
+		return "HumiditySensor (0x0307)";
+	case DeviceType::GenericSwitch:
+		return "GenericSwitch (0x000F)";
+	default:
+		return "Unknown";
+	}
+}
+
+static int ListBridgedDevicesHandler(const struct shell *shell, size_t argc, char **argv)
+{
+	shell_fprintf(shell, SHELL_INFO, "Bridged devices list:\n");
+	shell_fprintf(shell, SHELL_INFO, "---------------------------------------------------------------------\n");
+	shell_fprintf(shell, SHELL_INFO, "| Endpoint ID |        Name        |           Type           \n");
+	shell_fprintf(shell, SHELL_INFO, "---------------------------------------------------------------------\n");
+
+	uint8_t count = 0;
+	constexpr uint16_t kMaxEndpointId = CONFIG_BRIDGE_MAX_DYNAMIC_ENDPOINTS_NUMBER;
+
+	for (chip::EndpointId endpointId = 1; endpointId <= kMaxEndpointId; endpointId++) {
+		uint16_t deviceType = 0;
+		auto *provider = Nrf::BridgeManager().Instance().GetProvider(endpointId, deviceType);
+
+		if (provider != nullptr) {
+			const char *nodeLabel = Nrf::BridgeManager().Instance().GetNodeLabel(endpointId);
+			if (!nodeLabel || strnlen(nodeLabel, Nrf::MatterBridgedDevice::kNodeLabelSize) == 0) {
+				nodeLabel = "(no label)";
+			}
+
+			shell_fprintf(shell, SHELL_INFO, "| %-11d | %-18s | %-24s\n", endpointId, nodeLabel,
+				      GetDeviceTypeString(deviceType));
+			count++;
+		}
+	}
+
+	if (count == 0) {
+		shell_fprintf(shell, SHELL_INFO, "| No bridged devices found                                      |\n");
+	}
+
+	shell_fprintf(shell, SHELL_INFO, "---------------------------------------------------------------------\n");
+	shell_fprintf(shell, SHELL_INFO, "Total: %d device(s)\n", count);
+
+	return 0;
+}
+
 #ifdef CONFIG_BRIDGED_DEVICE_SIMULATED_ONOFF_SHELL
 static int SimulatedBridgedDeviceOnOffWriteHandler(const struct shell *shell, size_t argc, char **argv)
 {
@@ -265,6 +324,11 @@ SHELL_STATIC_SUBCMD_SET_CREATE(
 		"Usage: remove <bridged_device_endpoint_id>\n"
 		"* bridged_device_endpoint_id - the bridged device's endpoint on which it was previously created\n",
 		RemoveBridgedDeviceHandler, 2, 0),
+	SHELL_CMD_ARG(list, NULL,
+		      "Lists all currently connected bridged devices. \n"
+		      "Usage: list\n"
+		      "Displays endpoint ID, node label (name), and device type for all bridged devices.\n",
+		      ListBridgedDevicesHandler, 1, 0),
 #ifdef CONFIG_BRIDGED_DEVICE_SIMULATED_ONOFF_SHELL
 	SHELL_CMD_ARG(
 		onoff, NULL,

applications/matter_bridge/src/core/bridge_manager.cpp

@@ -509,6 +509,18 @@ BridgedDeviceDataProvider *BridgeManager::GetProvider(EndpointId endpoint, uint1
 	return nullptr;
 }
 
+const char *BridgeManager::GetNodeLabel(EndpointId endpoint)
+{
+	uint16_t endpointIndex = emberAfGetDynamicIndexFromEndpoint(endpoint);
+	if (Instance().mDevicesMap.Contains(endpointIndex)) {
+		BridgedDevicePair &bridgedDevices = Instance().mDevicesMap[endpointIndex];
+		if (bridgedDevices.mDevice) {
+			return bridgedDevices.mDevice->GetNodeLabel();
+		}
+	}
+	return nullptr;
+}
+
 } /* namespace Nrf */
 
 Protocols::InteractionModel::Status

applications/matter_bridge/src/core/bridge_manager.h

@@ -109,6 +109,14 @@ class BridgeManager {
 	 */
 	BridgedDeviceDataProvider *GetProvider(chip::EndpointId endpoint, uint16_t &deviceType);
 
+	/**
+	 * @brief Get the node label of the bridged device on the specified endpoint.
+	 *
+	 * @param endpoint endpoint on which the bridged device is stored
+	 * @return pointer to the node label string, or nullptr if endpoint not found
+	 */
+	const char *GetNodeLabel(chip::EndpointId endpoint);
+
 	static CHIP_ERROR HandleRead(uint16_t index, chip::ClusterId clusterId,
 				     const EmberAfAttributeMetadata *attributeMetadata, uint8_t *buffer,
 				     uint16_t maxReadLength);