Add connect/load/flash + setcap management

Author: leducp

tools/eeprom_editor/App.cc

@@ -1,14 +1,25 @@
 #ifndef KICKCAT_EEPROM_EDITOR_APP_H
 #define KICKCAT_EEPROM_EDITOR_APP_H
 
+#include <atomic>
+#include <memory>
+#include <mutex>
 #include <string>
+#include <thread>
 #include <vector>
 
 #include "imgui.h"
 #include "vendor/imgui_memory_editor.h"
 
+#include "kickcat/AbstractSocket.h"
 #include "kickcat/SIIParser.h"
 
+namespace kickcat
+{
+    class Bus;
+    class Link;
+}
+
 namespace kickcat::eeprom_editor
 {
     enum class Category
@@ -22,10 +33,13 @@ namespace kickcat::eeprom_editor
         RxPDO
     };
 
+    enum class SlaveAction { Load, Flash };
+
     class App
     {
     public:
         App();
+        ~App();
 
         void render();
 
@@ -38,20 +52,61 @@ namespace kickcat::eeprom_editor
         void openFilePath(std::string const& path);
         void saveFilePath(std::string const& path);
 
+        // Device operations
+        bool isConnected() const;
+        bool isBusy() const;
+        void connectToInterface(std::string const& name);
+        void disconnect();
+        void loadFromSlave(int slave_index);
+        void flashToSlave(int slave_index);
+        void joinWorker();
+        void finalizeWorker();
+
+        // Rendering
         void renderMenuBar();
         void renderSidebar();
         void renderContentPanel();
         void renderHexPanel();
         void renderStatusBar();
+        void renderConnectDialog();
+        void renderSlaveDialog();
+        bool renderSlaveTable();  // returns true on double-click
+        void renderPrivilegeDialog();
 
+        // EEPROM state
         eeprom::SII sii_{};
         std::vector<uint8_t> serialized_;
-
         std::string file_path_;
         bool modified_{false};
-
         Category active_category_{Category::Info};
         MemoryEditor mem_edit_;
+
+        // EtherCAT connection
+        std::shared_ptr<Link> link_;
+        std::unique_ptr<Bus> bus_;
+        std::string connected_interface_;
+
+        // Dialog state
+        bool show_connect_dialog_{false};
+        bool show_slave_dialog_{false};
+        SlaveAction slave_action_{SlaveAction::Load};
+        int selected_slave_index_{-1};
+        std::string device_error_;
+        std::vector<NetworkInterface> cached_interfaces_;
+
+        // Privilege escalation
+        bool show_privilege_dialog_{false};
+        bool privilege_granted_{false};
+        std::string privilege_error_;
+        std::atomic<bool> needs_privilege_escalation_{false};
+
+        // Background worker state
+        std::thread worker_;
+        std::atomic<float> worker_progress_{0.0f};
+        std::atomic<bool> worker_done_{false};
+        std::mutex worker_status_mutex_;
+        std::string worker_status_;
+        std::string worker_error_;
     };
 }
 

tools/eeprom_editor/App.h

@@ -22,7 +22,7 @@ namespace kickcat::eeprom_editor
 
         if (out_of_range)
         {
-            ImGui::PushStyleColor(ImGuiCol_Border, ImVec4(0.88f, 0.33f, 0.33f, 1.0f));
+            ImGui::PushStyleColor(ImGuiCol_Border, COLOR_RED);
         }
 
         ImGui::SetNextItemWidth(60.0f);

tools/eeprom_editor/EditorUtils.cc

@@ -11,6 +11,14 @@
 
 namespace kickcat::eeprom_editor
 {
+    constexpr ImVec4 COLOR_RED    {0.88f, 0.33f, 0.33f, 1.0f};
+    constexpr ImVec4 COLOR_GREEN  {0.31f, 0.79f, 0.41f, 1.0f};
+    constexpr ImVec4 COLOR_YELLOW {0.88f, 0.75f, 0.31f, 1.0f};
+    constexpr ImVec4 COLOR_BLUE   {0.55f, 0.75f, 0.95f, 1.0f};
+    constexpr ImVec4 COLOR_GREY   {0.65f, 0.65f, 0.65f, 1.0f};
+    constexpr ImVec4 COLOR_DIM    {0.55f, 0.55f, 0.55f, 1.0f};
+    constexpr ImVec4 COLOR_TITLE  {0.42f, 0.55f, 0.84f, 1.0f};
+
     char const* resolveString(std::vector<std::string> const& strings, uint8_t index);
     bool stringIndexInput(char const* label, uint8_t* index,
                           std::vector<std::string> const& strings);

tools/eeprom_editor/Editors.h

@@ -1,5 +1,4 @@
 #include <array>
-#include <imgui.h>
 
 #include "Editors.h"
 
@@ -18,7 +17,7 @@ namespace kickcat::eeprom_editor::fmmu
         bool changed = false;
         auto& fmmus = sii.fmmus;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "FMMU -- Category 40");
+        ImGui::TextColored(COLOR_TITLE, "FMMU -- Category 40");
         ImGui::Separator();
 
         constexpr ImGuiTableFlags table_flags =

tools/eeprom_editor/FMMUEditor.cc

@@ -17,7 +17,7 @@ namespace kickcat::eeprom_editor::general
         bool changed = false;
         auto& g = sii.general;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "General -- Category 30");
+        ImGui::TextColored(COLOR_TITLE, "General -- Category 30");
         ImGui::Separator();
 
         if (ImGui::CollapsingHeader("String References", ImGuiTreeNodeFlags_DefaultOpen))

tools/eeprom_editor/GeneralEditor.cc

@@ -7,7 +7,7 @@ namespace kickcat::eeprom_editor::info
         bool changed = false;
         auto& info = sii.info;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "Info (Header) -- Words 0x00-0x3F");
+        ImGui::TextColored(COLOR_TITLE, "Info (Header) -- Words 0x00-0x3F");
         ImGui::Separator();
         ImGui::Spacing();
 

tools/eeprom_editor/InfoEditor.cc

@@ -10,17 +10,17 @@ namespace kickcat::eeprom_editor::pdo
 
         bool const is_tx = (direction == Direction::Tx);
 
+        char const* title = "RxPDO (Outputs) -- Category 51";
         auto* mappings_ptr = &sii.RxPDO;
-        char const* title  = "RxPDO (Outputs) -- Category 51";
         if (is_tx)
         {
-            mappings_ptr = &sii.TxPDO;
             title        = "TxPDO (Inputs) -- Category 50";
+            mappings_ptr = &sii.TxPDO;
         }
         auto& mappings = *mappings_ptr;
         auto& strings  = sii.strings;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "%s", title);
+        ImGui::TextColored(COLOR_TITLE, "%s", title);
         ImGui::Separator();
 
         int delete_mapping = -1;

tools/eeprom_editor/PDOEditor.cc

@@ -9,7 +9,7 @@ namespace kickcat::eeprom_editor::strings
         bool changed = false;
         auto& strings = sii.strings;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "Strings -- Category 10");
+        ImGui::TextColored(COLOR_TITLE, "Strings -- Category 10");
         ImGui::Separator();
 
         if (strings.empty())

tools/eeprom_editor/StringsEditor.cc

@@ -18,7 +18,7 @@ namespace kickcat::eeprom_editor::syncm
         bool changed = false;
         auto& sms = sii.syncManagers;
 
-        ImGui::TextColored(ImVec4(0.42f, 0.55f, 0.84f, 1.0f), "Sync Managers -- Category 41");
+        ImGui::TextColored(COLOR_TITLE, "Sync Managers -- Category 41");
         ImGui::Separator();
 
         constexpr ImGuiTableFlags table_flags =