feat: Add more validations

Author: EmanuelVasileGresnu

config/config.js.sample

@@ -104,7 +104,8 @@ let config = {
 				broadcastNewsUpdates: true
 			}
 		},
-		// Example Sense HAT module configuration:
+		// Example Sense HAT module configuration.
+		// Uncomment this block on a Raspberry Pi with a Sense HAT installed:
 		// {
 		// 	module: "sensehat",
 		// 	position: "top_right",

modules/default/sensehat/README.md

@@ -60,8 +60,72 @@ How it works
 
 Troubleshooting
 
-- If you see warnings about "sense_hat library not available", ensure the package python3-sense-hat is installed and you are on a Raspberry Pi.
-- If LED text rendering blocks updates, remember that show_message scrolls the text; it will finish and allow the next poll. Reduce updateInterval or avoid text mode if you prefer continuous updates.
+1. Check that the Sense HAT is detected by the kernel
+
+```
+ls -l /dev/i2c*
+dmesg | grep -i "sense"
+```
+
+You should see:
+
+- /dev/i2c-1 present
+- A line like: fb1: RPi-Sense FB frame buffer device
+- A joystick device entry for the Sense HAT
+
+2. Probe I²C bus 1
+
+```
+sudo i2cdetect -y 1
+```
+
+On a working Sense HAT, you should see several non-"--" addresses (e.g. 1c, 39, 5c, 5f, 6a). If everything shows "--", the HAT may not be seated correctly or could be faulty.
+
+3. Test using the official Python library
+
+```
+python3 - << 'PY'
+from sense_hat import SenseHat
+
+sh = SenseHat()
+
+print("Temperature:", sh.get_temperature())
+print("Humidity   :", sh.get_humidity())
+print("Pressure   :", sh.get_pressure())
+PY
+```
+
+- If you get numeric values, the sensors are working.
+- If you see errors like `OSError: Humidity Init Failed`, there may be a contact problem on the header or a sensor issue.
+
+4. Check the LED matrix
+
+```
+python3 - << 'PY'
+from sense_hat import SenseHat
+from time import sleep
+
+sh = SenseHat()
+sh.clear()
+sh.show_message("HI", text_colour=(0, 255, 0), scroll_speed=0.07)
+sleep(1)
+sh.clear()
+PY
+```
+
+If you don’t see LEDs:
+
+- Power off the Raspberry Pi.
+- Firmly press the Sense HAT onto the 40-pin header (common on new boards).
+- Boot again and re-run the test.
+
+5. What the MagicMirror module will show
+
+- "Loading Sense HAT data…" → Python helper hasn’t delivered any data yet.
+- "Sense HAT: no sensor data (check hardware or drivers)" → helper runs, but all sensor fields are null.
+- "Sense HAT error: …" → helper reported an explicit error (library missing, init failure, etc.).
+
+If the Python tests fail, the issue is with hardware/OS/driver rather than this MagicMirror module.
 
 License
 MIT (follow the MagicMirror² project license)

modules/default/sensehat/node_helper.js

@@ -70,6 +70,8 @@ module.exports = NodeHelper.create({
 				const data = JSON.parse(stdout.trim());
 				if (data && data.error) {
 					Log.warn(`[sensehat] Python reported error: ${data.error}`);
+					// Forward error payload to frontend so UI can display it
+					this.sendSocketNotification("SENSEHAT_DATA", data);
 					return;
 				}
 				this.sendSocketNotification("SENSEHAT_DATA", data);

modules/default/sensehat/sensehat.js

@@ -23,21 +23,41 @@ Module.register("sensehat", {
 	start () {
 		this.sensorData = null;
 		this.ready = false;
-		// Send configuration to node_helper
 		this.sendSocketNotification("SENSEHAT_CONFIG", this.config);
 	},
 
 	getDom () {
 		const wrapper = document.createElement("div");
 		wrapper.className = "sensehat";
 
-		if (!this.sensorData) {
+		// Loading state
+		if (this.sensorData === null) {
 			wrapper.innerHTML = "Loading Sense HAT data...";
 			return wrapper;
 		}
 
 		const data = this.sensorData;
 
+		// Error state from backend
+		if (data && typeof data.error === "string" && data.error.trim() !== "") {
+			wrapper.innerHTML = `Sense HAT error: ${data.error}`;
+			return wrapper;
+		}
+
+		const hasAnyValue
+			= typeof data.temperature === "number"
+			  || typeof data.humidity === "number"
+			  || typeof data.pressure === "number"
+			  || (data.orientation
+			    && (typeof data.orientation.pitch === "number"
+			      || typeof data.orientation.roll === "number"
+			      || typeof data.orientation.yaw === "number"));
+
+		if (!hasAnyValue) {
+			wrapper.innerHTML = "Sense HAT: no sensor data (check hardware or drivers)";
+			return wrapper;
+		}
+
 		// Temperature
 		if (this.config.showTemperature && typeof data.temperature === "number") {
 			const t = this.config.temperatureUnit === "F" ? (data.temperature * 9) / 5 + 32 : data.temperature;
@@ -91,13 +111,21 @@ Module.register("sensehat", {
 
 			// Optional LED status logic
 			if (this.config.ledMatrixEnabled && this.config.ledMode !== "text") {
+				// Only act when we have real numeric data and no error
+				const hasNumeric = typeof payload.temperature === "number" || typeof payload.humidity === "number";
+				if (!(hasNumeric) || (payload && typeof payload.error === "string" && payload.error.trim() !== "")) {
+					return;
+				}
 				const { temperatureHigh, temperatureLow, humidityHigh, humidityLow } = this.config.criticalThresholds || {};
 				let color = this.config.ledColor || [0, 255, 0];
 
-				// Danger if out of range
 				if (
-					(typeof payload.temperature === "number" && (((temperatureHigh !== null && temperatureHigh !== undefined) && payload.temperature > temperatureHigh) || ((temperatureLow !== null && temperatureLow !== undefined) && payload.temperature < temperatureLow)))
-					|| (typeof payload.humidity === "number" && (((humidityHigh !== null && humidityHigh !== undefined) && payload.humidity > humidityHigh) || ((humidityLow !== null && humidityLow !== undefined) && payload.humidity < humidityLow)))
+					(typeof payload.temperature === "number"
+					  && (((temperatureHigh !== null && temperatureHigh !== undefined) && payload.temperature > temperatureHigh)
+					    || ((temperatureLow !== null && temperatureLow !== undefined) && payload.temperature < temperatureLow)))
+			    || (typeof payload.humidity === "number"
+			      && (((humidityHigh !== null && humidityHigh !== undefined) && payload.humidity > humidityHigh)
+			        || ((humidityLow !== null && humidityLow !== undefined) && payload.humidity < humidityLow)))
 				) {
 					color = [255, 0, 0]; // red
 				} else {