k3ldar · k3ldar · Jul 17, 2026 · Jul 17, 2026
diff --git a/PowerControlHub/ExternalSensorNetworkHandler.cpp b/PowerControlHub/ExternalSensorNetworkHandler.cpp
@@ -217,7 +217,10 @@ CommandResult ExternalSensorNetworkHandler::handleRequest(const char* method,
     }
 
     if (result == ConfigResult::Success)
+    {
+        formatJsonResponse(responseBuffer, bufferSize, true);
         return CommandResult::ok();
+    }
 
     return CommandResult::error(static_cast<uint8_t>(result));
 }

diff --git a/PowerControlHub/PowerControlHub.ino b/PowerControlHub/PowerControlHub.ino
@@ -49,6 +49,16 @@ SerialCommandManager commandMgrComputer(&COMPUTER_SERIAL, onComputerCommandRecei
 
 PowerControlHubApp app(&commandMgrComputer);
 
+void powerControlHubTask(void* pvParameters)
+{
+	PowerControlHubApp* app = (PowerControlHubApp*)pvParameters;
+	while (true)
+	{
+		app->loop();
+		vTaskDelay(1);
+	}
+}
+
 void setup()
 {
 	// Serial initialization is performed first to ensure that any logging or error messages
@@ -59,11 +69,22 @@ void setup()
 
 	// configure app
 	app.setup(nullptr, 0);
+
+	// Create a dedicated task with 16KB stack
+	xTaskCreatePinnedToCore(
+		powerControlHubTask,
+		"PowerControlHub",
+		16384,           // stack size
+		&app,            // parameters
+		1,               // priority
+		NULL,            // task handle
+		1                // core 1
+	);
 }
 
 void loop()
 {
-	app.loop();
+	vTaskDelay(portMAX_DELAY);
 }
 
 // Called when a command arrives on COMPUTER_SERIAL that no registered handler

diff --git a/PowerControlHub/PowerControlHub.vcxproj b/PowerControlHub/PowerControlHub.vcxproj
diff --git a/PowerControlHub/PowerControlHub.vcxproj.filters b/PowerControlHub/PowerControlHub.vcxproj.filters
@@ -323,7 +323,7 @@
     </ClCompile>
     <ClCompile Include="PowerControlHub.ino" />
     <ClCompile Include="SensorConfigNetworkHandler.cpp">
-      <Filter>Source Files</Filter>
+      <Filter>Source Files\NetworkCommandHandlers</Filter>
     </ClCompile>
   </ItemGroup>
   <ItemGroup>
@@ -672,9 +672,6 @@
     <ClInclude Include="NavigationController.h">
       <Filter>Header Files</Filter>
     </ClInclude>
-    <ClInclude Include="build_opt.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
     <ClInclude Include="ExternalSensorConfigCommandHandler.h">
       <Filter>Header Files</Filter>
     </ClInclude>
@@ -714,8 +711,5 @@
     <ClInclude Include="SensorConfigNetworkHandler.h">
       <Filter>Header Files</Filter>
     </ClInclude>
-    <ClInclude Include="SensorConfig.h">
-      <Filter>Header Files</Filter>
-    </ClInclude>
   </ItemGroup>
 </Project>
diff --git a/PowerControlHub/SensorConfigNetworkHandler.cpp b/PowerControlHub/SensorConfigNetworkHandler.cpp
@@ -252,7 +252,10 @@ CommandResult SensorConfigNetworkHandler::handleRequest(const char* method,
 	}
 
 	if (result == ConfigResult::Success)
+	{
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
+	}
 
 	return CommandResult::error(static_cast<uint8_t>(result));
 }

diff --git a/PowerControlHub/SensorNetworkHandler.cpp b/PowerControlHub/SensorNetworkHandler.cpp
@@ -68,8 +68,10 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 					e.options2[0],
 					e.options2[1],
 					e.enabled ? 1u : 0u);
+
 				if (n < 0 || written + n >= static_cast<int>(bufferSize))
 					break;
+
 				written += n;
 			}
 		}
@@ -83,11 +85,13 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 			int n = snprintf(responseBuffer + written, bufferSize - written,
 				",\"meta\":{\"count\":%u,\"descriptors\":[",
 				static_cast<unsigned>(SensorIdList::Count));
+
 			if (n < 0 || written + n >= static_cast<int>(bufferSize))
 			{
 				responseBuffer[bufferSize - 1] = '\0';
 				return CommandResult::ok();
 			}
+
 			written += n;
 
 			for (unsigned di = 0; di < static_cast<unsigned>(SensorIdList::Count); di++)
@@ -97,63 +101,91 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 					"%s{\"id\":%u,\"name\":\"%s\","
 					"\"pins\":[",
 					di > 0 ? "," : "", di, d.name);
-				if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
-				written += n;
 
+				if (n < 0 || written + n >= static_cast<int>(bufferSize))
+					break;
+
+				written += n;
 				bool firstPin = true;
+
 				for (unsigned p = 0; p < ConfigMaxSensorPins; p++)
 				{
-					if (strcmp(d.pins[p].type, "none") == 0) continue;
+					if (strcmp(d.pins[p].type, "none") == 0)
+						continue;
+
 					n = snprintf(responseBuffer + written, bufferSize - written,
 						"%s{\"label\":\"%s\",\"type\":\"%s\",\"min\":%d,\"max\":%d,\"default\":%d}",
 						firstPin ? "" : ",",
 						d.pins[p].label, d.pins[p].type,
 						d.pins[p].minVal, d.pins[p].maxVal, d.pins[p].defaultValue);
-					if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
+
+					if (n < 0 || written + n >= static_cast<int>(bufferSize))
+						break;
+
 					written += n;
 					firstPin = false;
 				}
 
 				n = snprintf(responseBuffer + written, bufferSize - written,
 					"],\"options1\":[");
-				if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
+
+				if (n < 0 || written + n >= static_cast<int>(bufferSize))
+					break;
+
 				written += n;
 
 				bool firstOpt1 = true;
+
 				for (unsigned o = 0; o < 2; o++)
 				{
-					if (strcmp(d.options1[o].type, "none") == 0) continue;
+					if (strcmp(d.options1[o].type, "none") == 0)
+						continue;
+
 					n = snprintf(responseBuffer + written, bufferSize - written,
 						"%s{\"label\":\"%s\",\"type\":\"%s\",\"min\":%d,\"max\":%d,\"default\":%d}",
 						firstOpt1 ? "" : ",",
 						d.options1[o].label, d.options1[o].type,
 						d.options1[o].minVal, d.options1[o].maxVal, d.options1[o].defaultValue);
-					if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
+
+					if (n < 0 || written + n >= static_cast<int>(bufferSize))
+						break;
+
 					written += n;
 					firstOpt1 = false;
 				}
 
 				n = snprintf(responseBuffer + written, bufferSize - written,
 					"],\"options2\":[");
-				if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
-				written += n;
+
+				if (n < 0 || written + n >= static_cast<int>(bufferSize))
+					break;
 
+				written += n;
 				bool firstOpt2 = true;
+
 				for (unsigned o = 0; o < 2; o++)
 				{
-					if (strcmp(d.options2[o].type, "none") == 0) continue;
+					if (strcmp(d.options2[o].type, "none") == 0)
+						continue;
+
 					n = snprintf(responseBuffer + written, bufferSize - written,
 						"%s{\"label\":\"%s\",\"type\":\"%s\",\"min\":%d,\"max\":%d,\"default\":%d}",
 						firstOpt2 ? "" : ",",
 						d.options2[o].label, d.options2[o].type,
 						d.options2[o].minVal, d.options2[o].maxVal, d.options2[o].defaultValue);
-					if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
+
+					if (n < 0 || written + n >= static_cast<int>(bufferSize))
+						break;
+
 					written += n;
 					firstOpt2 = false;
 				}
 
 				n = snprintf(responseBuffer + written, bufferSize - written, "]}");
-				if (n < 0 || written + n >= static_cast<int>(bufferSize)) break;
+
+				if (n < 0 || written + n >= static_cast<int>(bufferSize))
+					break;
+
 				written += n;
 			}
 
@@ -193,6 +225,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 		if (idx >= config->sensors.count)
 			config->sensors.count = idx + 1;
 
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 
@@ -228,6 +261,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 			memset(&config->sensors.sensors[config->sensors.count], 0, sizeof(SensorEntry));
 		}
 
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 
@@ -255,6 +289,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 		SensorEntry& entry = config->sensors.sensors[idx];
 		strncpy(entry.name, params[0].value, sizeof(entry.name) - 1);
 		entry.name[sizeof(entry.name) - 1] = '\0';
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 
@@ -279,6 +314,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 		}
 
 		config->sensors.sensors[idx].pins[slot] = pin;
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 
@@ -305,6 +341,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 		}
 
 		config->sensors.sensors[idx].enabled = enabled;
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 
@@ -358,6 +395,7 @@ CommandResult SensorNetworkHandler::handleRequest(const char* method,
 			config->sensors.sensors[idx].options2[slot] = val16;
 		}
 
+		formatJsonResponse(responseBuffer, bufferSize, true);
 		return CommandResult::ok();
 	}
 

diff --git a/PowerControlHub/SystemDefinitions.h b/PowerControlHub/SystemDefinitions.h
@@ -220,7 +220,7 @@ constexpr const char* compassDirections[16] = {
     "S", "SSW", "SW", "WSW", "W", "WNW", "NW", "NNW"
 };
 
-constexpr uint16_t MaximumJsonResponseBufferSize = 1024;
+constexpr uint16_t MaximumJsonResponseBufferSize = 4096;
 
 /*
 * Unique IDs for different sensor types in the system.

diff --git a/PowerControlHub/WifiServer.cpp b/PowerControlHub/WifiServer.cpp
@@ -1112,16 +1112,14 @@ bool WifiServer::dispatchToHandler(IWifiClient& client, INetworkCommandHandler*
 		// Log response buffer for debugging
 		if (responseBuffer[0] != '\0')
 		{
-			char dbg[1051];
-			snprintf(dbg, sizeof(dbg), "Handler success response: %s", responseBuffer);
-			sendDebug(dbg, F("WifiServer"));
+			sendDebug("Handler success response", F("WifiServer"));
 		}
 
 		// Log parameters passed to handler
-		char paramDbg[512];
+		char paramDbg[128];
 		for (uint8_t p = 0; p < paramCount; p++)
 		{
-			snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %s=%s", p, params[p].key, params[p].value);
+			snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %*s=%*s", p, (int)sizeof(params[p].key) - 1, params[p].key, (int)sizeof(params[p].value) - 1, params[p].value);
 			sendDebug(paramDbg, F("WifiServer"));
 		}
 
@@ -1135,15 +1133,13 @@ bool WifiServer::dispatchToHandler(IWifiClient& client, INetworkCommandHandler*
 		if (responseBuffer[0] != '\0')
 		{
 			// Log error response for debugging
-			char dbg[1050];
-			snprintf(dbg, sizeof(dbg), "Handler error response: %s", responseBuffer);
-			sendDebug(dbg, F("WifiServer"));
+			sendDebug("Handler error response", F("WifiServer"));
 
 			// Also log parameters to help reproduce the error
-			char paramDbg[1024];
+			char paramDbg[128];
 			for (uint8_t p = 0; p < paramCount; p++)
 			{
-				snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %s=%s", p, params[p].key, params[p].value);
+				snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %*s=%*s", p, (int)sizeof(params[p].key) - 1, params[p].key, (int)sizeof(params[p].value) - 1, params[p].value);
 				sendDebug(paramDbg, F("WifiServer"));
 			}
 
@@ -1160,8 +1156,8 @@ bool WifiServer::dispatchToHandler(IWifiClient& client, INetworkCommandHandler*
 
 		for (uint8_t p = 0; p < paramCount; p++)
 		{
-			char paramDbg[400];
-			snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %s=%s", p, params[p].key, params[p].value);
+			char paramDbg[128];
+			snprintf(paramDbg, sizeof(paramDbg), "Param[%d] %*s=%*s", p, (int)sizeof(params[p].key) - 1, params[p].key, (int)sizeof(params[p].value) - 1	, params[p].value);
 			sendDebug(paramDbg, F("WifiServer"));
 		}
 

diff --git a/PowerControlHubApp/AppShell.xaml b/PowerControlHubApp/AppShell.xaml
@@ -21,13 +21,13 @@
 
     <ShellContent
         x:Name="SensorConfigPage"
-        Title="Local Sens."
+        Title="Local Sensors"
         Route="SensorConfigPage"
         ContentTemplate="{DataTemplate views:LocalSensorConfigPage}" />
 
     <ShellContent
         x:Name="ExternalSensorConfigPage"
-        Title="Ext. Sens."
+        Title="External Sensors"
         Route="ExternalSensorConfigPage"
         ContentTemplate="{DataTemplate views:ExternalSensorConfigPage}" />