Fix for spaces in wifi SSIDs and passwords

and some build issues
Fixes #3

Author: Rocka84

AGENTS.md

@@ -39,8 +39,8 @@ You do **not** need to flash the ESP32 to test UI changes! The project includes
 - `src/network/` - WiFi setup, Spoolman API clients, and Webhook/U1 HTTP request handling.
 - `src/serial/` - CLI interface via the USB serial port for device configuration.
 - `src/power/` - Battery voltage monitoring (ADC) and sleep management heuristics.
-- `docs/` - User-facing documentation and hardware setup guides.
-- `scripts/` - Python pre/post scripts for PlatformIO (e.g., binary copying, version injection).
+- `docs/` - User-facing documentation and hardware setup guides. **This is the source of truth.** Do not edit `web/docs/` directly.
+- `scripts/` - Python pre/post scripts for PlatformIO, and bash scripts like `copy_docs.sh` to sync docs to the `web/` folder.
 - `simulator/` - Mock data and configs for the native UI simulator.
 
 ## 6. Global Rules & Code Conventions
@@ -57,6 +57,7 @@ You do **not** need to flash the ESP32 to test UI changes! The project includes
 ## 8. Agentic Workflow
 1. Read the user's prompt carefully.
 2. If UI changes are requested, modify `src/ui/`, build the `simulator` environment, and run it to verify.
-3. Keep `notes.md` updated if you complete tasks or find new known issues.
-4. Keep `AGENTS.md` updated if the project architecture, dependencies, hardware integrations, or core guidelines change.
-5. If a task involves complex logic, use your `brainstorming` skill before writing code.
+3. **Documentation:** The files in `docs/` and the root `README.md` are the source of truth. If you edit them, you MUST run `./scripts/copy_docs.sh` to update the `web/` directory. Do not edit `web/docs/` or `web/README.md` directly.
+4. Keep `notes.md` updated if you complete tasks or find new known issues.
+5. Keep `AGENTS.md` updated if the project architecture, dependencies, hardware integrations, or core guidelines change.
+6. If a task involves complex logic, use your `brainstorming` skill before writing code.

README.md

@@ -65,7 +65,8 @@ You can flash the firmware and set up your Wi-Fi directly from your PC browser (
 The device stores settings in non-volatile memory (NVS). Type `help` in the Serial Terminal to see all available commands.
 
 ### 1. Settings
-- `set wifi YourWiFiName YourWiFiPassword` (Set your WiFi credentials)
+- `set wifi_ssid YourWiFiName` (Set your WiFi network name)
+- `set wifi_pass YourWiFiPassword` (Set your WiFi password)
 - `set webhook http://your-hook-url/webhook?spool={spool_id}&tool={toolhead}` (Set your webhook URL)
 - `set spoolman http://your-spoolman-ip:8000` (Set your Spoolman URL)
 - `set u1_host your-u1-ip:7125` (Enable Snapmaker U1 loading)

VERSION

@@ -1 +1 @@
-1.9.0
+1.9.1

docs/CONFIGURATION.md

@@ -10,7 +10,8 @@ CheapSpoolDisplay can be configured via a Serial Terminal (115200 baud) when con
 | :--- | :--- |
 | `help` | Show available commands |
 | `get config` | Show current configuration values |
-| `set wifi <ssid> <password>` | Special command to set Wi-Fi credentials |
+| `set wifi_ssid <ssid>` | Set Wi-Fi network name |
+| `set wifi_pass <password>` | Set Wi-Fi network password |
 | `set spoolman http://<ip>:<port>` | Set the Spoolman server URL |
 | `set webhook http://...` | Set the webhook URL |
 | `set u1_host <ip>:<port>` | Set the Snapmaker U1 host URL |
@@ -37,7 +38,8 @@ This setting also applies a specific gamma correction table to fix the washed-ou
 
 ### Wi-Fi
 
-`set wifi SSID PASSWORD`
+`set wifi_ssid SSID`
+`set wifi_pass PASSWORD`
 
 **Note:** The device will only connect to Wi-Fi when needed (e.g., for Spoolman or Webhooks). You can optionally configure the connection timeout limit using:
 `set wifi_timeout <seconds>` (Default: 60)
@@ -106,7 +108,8 @@ Configures how many seconds the device must be idle before entering deep sleep (
 
 ```bash
 # Set Wi-Fi
-set wifi MySSID MySecretPassword
+set wifi_ssid MySSID
+set wifi_pass MySecretPassword
 
 # Configure Spoolman
 set spoolman http://192.168.1.50:8000

docs/QUICKSTART.md

@@ -31,7 +31,8 @@ See [CONFIGURATION.md](CONFIGURATION.md) for details.
     * the **Web Installer** (click **Logs & Console**)
     * any **Serial Terminal** (115200 baud)
 2. Configuration Commands:
-    *   `set wifi <SSID> <PASSWORD>` to configure your wireless network
+    *   `set wifi_ssid <SSID>` to set your network name
+    *   `set wifi_pass <PASSWORD>` to set your network password
     *   `set tag_format <openspool|openprinttag|opentag3d|ask>` to set your preferred NFC format
     *   `set spoolman <URL>` to enable Spoolman enrichment (e.g., `http://192.168.1.50:8000`)
     *   `set u1_host <IP_OR_HOSTNAME>:7125` to enable Snapmaker U1 loading

docs/SPOOLMAN_SETUP.md

@@ -21,7 +21,7 @@ To enable Spoolman enrichment, you must configure your Spoolman server URL via t
 
 ## Requirements
 
-- **Wi-Fi**: The device must be configured with a valid SSID and Password (`set wifi ...`).
+- **Wi-Fi**: The device must be configured with a valid SSID and Password (`set wifi_ssid ...` and `set wifi_pass ...`).
 - **NFC Tag**: The tag must include a `spool_id` (OpenSpool format).
 - **Spoolman API**: The device will attempt to reach `${SPOOLMAN_URL}/api/v1/spool/${SPOOL_ID}`.
 

scripts/convert_logo.py

@@ -1,7 +1,12 @@
 Import("env")
 
 import os
-from PIL import Image
+try:
+    from PIL import Image
+except ImportError:
+    print("Pre-Build: Pillow not found. Installing via uv...")
+    env.Execute('"$PYTHONEXE" -m uv pip install Pillow')
+    from PIL import Image
 
 def generate_logo():
     input_path = "web/logo.png"

simulator/program

@@ -5,7 +5,7 @@
 #include "ui/DisplayUI.h"
 
 #include "config/ConfigManager.h"
-#include "network/NetworkManager.h"
+#include "network/AppNetworkManager.h"
 
 #ifndef USE_SDL2
 #include "nfc/NFCReader.h"
@@ -81,7 +81,7 @@ void loop() {
   PowerManager::tick();
 
   // WiFi idle timeout
-  NetworkManager::tick();
+  AppNetworkManager::tick();
 
   // If display is off and screen gets touched, wake it up
   if (PowerManager::isDisplayOff() && ts.getTouch().zRaw > 0) {
@@ -98,7 +98,7 @@ void loop() {
 
   // Poll WiFi state; update UI only on change
   static bool lastWifiState = false;
-  bool wifiNow = NetworkManager::isWiFiConnected();
+  bool wifiNow = AppNetworkManager::isWiFiConnected();
   if (wifiNow != lastWifiState) {
     lastWifiState = wifiNow;
     DisplayUI::updateWiFiStatus(wifiNow);
@@ -298,7 +298,7 @@ void loop() {
         (!currentSpoolData.spool_id.empty() || !currentSpoolData.lot_nr.empty())) {
       DisplayUI::showFetchingOverlay();
       lv_timer_handler(); // Force overlay render before blocking
-      NetworkManager::fetchSpoolmanData(currentSpoolData);
+      AppNetworkManager::fetchSpoolmanData(currentSpoolData);
       DisplayUI::hideFetchingOverlay();
     }
 

src/main.cpp

@@ -1,4 +1,4 @@
-#include "NetworkManager.h"
+#include "AppNetworkManager.h"
 #include "../config/ConfigManager.h"
 #include "../data/OpenSpool.h"
 #include "WebhookFormatter.h"
@@ -21,7 +21,7 @@ static bool ensureWiFi() {
     lastNetworkActivity = millis();
     return true;
   }
-  NetworkManager::connectWiFi();
+  AppNetworkManager::connectWiFi();
   if (WiFi.status() == WL_CONNECTED)
     lastNetworkActivity = millis();
   return (WiFi.status() == WL_CONNECTED);
@@ -30,7 +30,7 @@ static bool ensureWiFi() {
 #endif
 }
 
-void NetworkManager::tick() {
+void AppNetworkManager::tick() {
 #ifndef USE_SDL2
   if (WiFi.status() == WL_CONNECTED &&
       millis() - lastNetworkActivity > WIFI_IDLE_TIMEOUT_MS) {
@@ -40,15 +40,15 @@ void NetworkManager::tick() {
 #endif
 }
 
-bool NetworkManager::isWiFiConnected() {
+bool AppNetworkManager::isWiFiConnected() {
 #ifndef USE_SDL2
   return WiFi.status() == WL_CONNECTED;
 #else
   return true;
 #endif
 }
 
-void NetworkManager::connectWiFi() {
+void AppNetworkManager::connectWiFi() {
 #ifndef USE_SDL2
   std::string ssid = ConfigManager::getWifiSSID();
   std::string pass = ConfigManager::getWifiPass();
@@ -100,7 +100,7 @@ static size_t HeaderCallback(void *contents, size_t size, size_t nmemb, void *us
 }
 #endif
 
-bool NetworkManager::sendWebhookPayload(const OpenSpoolData &data,
+bool AppNetworkManager::sendWebhookPayload(const OpenSpoolData &data,
                                         int toolhead_id) {
   std::string spool_id = data.spool_id;
   std::string url = ConfigManager::getWebhook();
@@ -262,7 +262,7 @@ bool NetworkManager::sendWebhookPayload(const OpenSpoolData &data,
 #endif
 }
 
-bool NetworkManager::fetchSpoolmanData(OpenSpoolData &data) {
+bool AppNetworkManager::fetchSpoolmanData(OpenSpoolData &data) {
   std::string baseUrl = ConfigManager::getSpoolmanUrl();
   if (baseUrl.empty()) {
     return false;
@@ -319,7 +319,7 @@ bool NetworkManager::fetchSpoolmanData(OpenSpoolData &data) {
   return false;
 }
 
-bool NetworkManager::fetchSpoolmanByExternalId(OpenSpoolData &data) {
+bool AppNetworkManager::fetchSpoolmanByExternalId(OpenSpoolData &data) {
   std::string baseUrl = ConfigManager::getSpoolmanUrl();
   if (baseUrl.empty() || data.hardware_uid.empty()) {
     return false;
@@ -378,7 +378,7 @@ bool NetworkManager::fetchSpoolmanByExternalId(OpenSpoolData &data) {
   return false;
 }
 
-bool NetworkManager::fetchSpoolmanList(int page, int limit, std::vector<SpoolmanItem>& items, int& total_count) {
+bool AppNetworkManager::fetchSpoolmanList(int page, int limit, std::vector<SpoolmanItem>& items, int& total_count) {
   std::string baseUrl = ConfigManager::getSpoolmanUrl();
   if (baseUrl.empty()) {
     return false;