Cleanup RPC component

components/rpc/CMakeLists.txt

@@ -1,3 +1,3 @@
-idf_component_register(SRCS "rpc.cpp" "TCPServer.cpp" "WifiManager.cpp"
+idf_component_register(SRCS "rpc.cpp" "WifiManager.cpp"
                        PRIV_REQUIRES driver esp_event nvs_flash esp_netif esp_wifi
                        INCLUDE_DIRS "include")

components/rpc/RPCFactory.cpp

@@ -1,3 +0,0 @@
-//
-// Created by Johnathon Slightham on 2025-05-25.
-//

components/rpc/WifiManager.cpp

@@ -8,26 +8,17 @@
 #include "freertos/semphr.h"
 #include "esp_wifi.h"
 
-int WifiManager::init() {
+WifiManager::WifiManager() {
     esp_netif_init();
     esp_event_loop_create_default();
     esp_netif_create_default_wifi_sta();
 
-    printf("finished esp init");
-
     this->m_mutex = xSemaphoreCreateMutex();
-    this->m_state = wifi_state::connect;
+    this->m_state = wifi_state::disconnected;
     this->m_attempts = 0;
-
-    printf("finished variable init");
-
-    printf("lambda");
+    this->m_task = nullptr; // suppress warning
 
     xTaskCreate(reinterpret_cast<TaskFunction_t>(s_manage), "wifi_task", 4096, this, 5, &m_task);
-
-    printf("task create");
-
-    return 0;
 }
 
 int WifiManager::connect() {
@@ -51,25 +42,20 @@ int WifiManager::disconnect() {
 
         switch (state) {
             case wifi_state::connect:
-                printf("connect state\n");
                 init_connection();
                 update_state(wifi_state::connecting);
                 break;
             case wifi_state::connecting:
-                printf("connecting state\n");
                 handle_connecting();
                 break;
             case wifi_state::searching:
-                printf("searching state\n");
                 handle_search();
                 break;
             case wifi_state::disconnect:
-                printf("disconnect state\n");
                 handle_disconnect();
                 update_state(wifi_state::disconnected);
                 break;
             default:
-                printf("sleep state\n");
                 vTaskSuspend(nullptr);
                 break;
         }
@@ -128,6 +114,8 @@ int WifiManager::handle_disconnect() {
 }
 
 int WifiManager::handle_search() {
+    // todo: softap
+
     printf("Waiting for configuration");
     return 0;
 }

components/rpc/include/CommunicationRouter.h

@@ -7,7 +7,7 @@
 
 
 
-class CommunicationRouter {
+class CommunicationRouter : ICommunicationRouter {
 
 };
 

components/rpc/include/ICommunicationRouter.h

@@ -5,4 +5,8 @@
 #ifndef ICOMMUNICATIONROUTER_H
 #define ICOMMUNICATIONROUTER_H
 
+class ICommunicationRouter {
+
+}
+
 #endif //ICOMMUNICATIONROUTER_H

components/rpc/include/IDiscoveryService.h

@@ -0,0 +1,12 @@
+//
+// Created by Johnathon Slightham on 2025-05-26.
+//
+
+#ifndef IDISCOVERYSERVICE_H
+#define IDISCOVERYSERVICE_H
+
+class IDiscoveryService {
+
+}
+
+#endif //IDISCOVERYSERVICE_H

components/rpc/include/IRPCServer.h

@@ -5,4 +5,8 @@
 #ifndef IRPCSERVER_H
 #define IRPCSERVER_H
 
+class IRPCServer {
+
+}
+
 #endif //IRPCSERVER_H

components/rpc/include/IWifiManager.h

@@ -0,0 +1,15 @@
+//
+// Created by Johnathon Slightham on 2025-05-26.
+//
+
+#ifndef IWIFIMANAGER_H
+#define IWIFIMANAGER_H
+
+class IWifiManager {
+public:
+    virtual ~IWifiManager() {};
+    virtual int connect() = 0;
+    virtual int disconnect() = 0;
+};
+
+#endif //IWIFIMANAGER_H

components/rpc/include/RPCFactory.h

@@ -5,4 +5,10 @@
 #ifndef RPCFACTORY_H
 #define RPCFACTORY_H
 
+class RPCFactory {
+public:
+static std::shared_ptr<IRPCServer> createRPCServer() {
+    return std::make_shared<TCPServer>();
+}
+
 #endif //RPCFACTORY_H

components/rpc/include/TCPServer.h

@@ -5,4 +5,15 @@
 #ifndef TCPSERVER_H
 #define TCPSERVER_H
 
+#include "IRPCServer.h"
+
+class TCPServer : IRPCServer {
+public:
+TCPServer();
+~TCPServer();
+
+private:
+
+}
+
 #endif //TCPSERVER_H

components/rpc/include/WifiManager.h

@@ -6,13 +6,14 @@
 #include "freertos/FreeRTOS.h"
 #include "freertos/semphr.h"
 
-class WifiManager {
+#include "IWifiManager.h"
+
+class WifiManager final : IWifiManager {
 public:
-  WifiManager() = default;
-  ~WifiManager() = default;
-  int init();
-  int connect();
-  int disconnect();
+  WifiManager();
+  ~WifiManager() override = default;
+  int connect() override;
+  int disconnect() override;
 
   [[noreturn]] void manage();
 
@@ -28,7 +29,6 @@ private:
 
   static void wifi_event_handler(void *event_handler_arg, esp_event_base_t event_base, int32_t event_id, void *event_data);
 
-
   SemaphoreHandle_t m_mutex;
   wifi_state m_state;
   TaskHandle_t m_task;

components/rpc/include/mDNSDiscoveryService.h

@@ -5,4 +5,8 @@
 #ifndef DISCOVERYSERVICE_H
 #define DISCOVERYSERVICE_H
 
+class mDNSDiscoveryService : IDiscoveryService {
+
+}
+
 #endif //DISCOVERYSERVICE_H