Suspend TargetController upon GDB detach, if running under CLion

src/DebugServer/CMakeLists.txt

@@ -23,6 +23,7 @@ target_sources(
         ${CMAKE_CURRENT_SOURCE_DIR}/Gdb/CommandPackets/BloomVersion.cpp
         ${CMAKE_CURRENT_SOURCE_DIR}/Gdb/CommandPackets/BloomVersionMachine.cpp
         ${CMAKE_CURRENT_SOURCE_DIR}/Gdb/CommandPackets/GenerateSvd.cpp
+        ${CMAKE_CURRENT_SOURCE_DIR}/Gdb/CommandPackets/Detach.cpp
         ${CMAKE_CURRENT_SOURCE_DIR}/Gdb/ResponsePackets/SupportedFeaturesResponse.cpp
 
         # AVR GDB RSP Server

src/DebugServer/Gdb/CommandPackets/CommandPacket.cpp

@@ -38,12 +38,6 @@ namespace Bloom::DebugServer::Gdb::CommandPackets
             return;
         }
 
-        if (packetString[0] == 'D') {
-            // Detach packet - there's not really anything we need to do here, so just respond with an OK
-            debugSession.connection.writePacket(OkResponsePacket());
-            return;
-        }
-
         if (packetString.find("vMustReplyEmpty") == 0) {
             Logger::debug("Handling vMustReplyEmpty");
             debugSession.connection.writePacket(EmptyResponsePacket());

src/DebugServer/Gdb/CommandPackets/Detach.cpp

@@ -0,0 +1,38 @@
+#include "Detach.hpp"
+
+#include "src/DebugServer/Gdb/ResponsePackets/OkResponsePacket.hpp"
+#include "src/DebugServer/Gdb/ResponsePackets/ErrorResponsePacket.hpp"
+
+#include "src/Helpers/Process.hpp"
+
+#include "src/Logger/Logger.hpp"
+
+namespace Bloom::DebugServer::Gdb::CommandPackets
+{
+    using TargetController::TargetControllerConsole;
+
+    using ResponsePackets::OkResponsePacket;
+    using ResponsePackets::ErrorResponsePacket;
+
+    using Exceptions::Exception;
+
+    Detach::Detach(const RawPacketType& rawPacket)
+        : CommandPacket(rawPacket)
+    {}
+
+    void Detach::handle(DebugSession& debugSession, TargetControllerConsole& targetControllerConsole) {
+        Logger::debug("Handling Detach packet");
+
+        try {
+            if (Process::isProcessManagedByClion(Process::getParentProcessId())) {
+                targetControllerConsole.suspendTargetController();
+            }
+
+            debugSession.connection.writePacket(OkResponsePacket());
+
+        } catch (const Exception& exception) {
+            Logger::error("Failed to suspend TargetController - " + exception.getMessage());
+            debugSession.connection.writePacket(ErrorResponsePacket());
+        }
+    }
+}

src/DebugServer/Gdb/CommandPackets/Detach.hpp

@@ -0,0 +1,17 @@
+#pragma once
+
+#include "CommandPacket.hpp"
+
+namespace Bloom::DebugServer::Gdb::CommandPackets
+{
+    class Detach: public CommandPacket
+    {
+    public:
+        explicit Detach(const RawPacketType& rawPacket);
+
+        void handle(
+            DebugSession& debugSession,
+            TargetController::TargetControllerConsole& targetControllerConsole
+        ) override;
+    };
+}

src/DebugServer/Gdb/GdbRspDebugServer.cpp

@@ -31,9 +31,11 @@
 #include "CommandPackets/BloomVersion.hpp"
 #include "CommandPackets/BloomVersionMachine.hpp"
 #include "CommandPackets/GenerateSvd.hpp"
+#include "CommandPackets/Detach.hpp"
 
 // Response packets
 #include "ResponsePackets/TargetStopped.hpp"
+#include "src/Helpers/Process.hpp"
 
 namespace Bloom::DebugServer::Gdb
 {
@@ -122,6 +124,12 @@ namespace Bloom::DebugServer::Gdb
         this->eventListener.registerCallbackForEventType<Events::TargetExecutionStopped>(
             std::bind(&GdbRspDebugServer::onTargetExecutionStopped, this, std::placeholders::_1)
         );
+
+        if (Process::isProcessManagedByClion(Process::getParentProcessId())) {
+            Logger::warning(
+                "Bloom's process is being managed by CLion - Bloom will automatically shutdown upon detaching from GDB."
+            );
+        }
     }
 
     void GdbRspDebugServer::close() {
@@ -252,6 +260,10 @@ namespace Bloom::DebugServer::Gdb
             return std::make_unique<CommandPackets::InterruptExecution>(rawPacket);
         }
 
+        if (rawPacket[1] == 'D') {
+            return std::make_unique<CommandPackets::Detach>(rawPacket);
+        }
+
         const auto rawPacketString = std::string(rawPacket.begin(), rawPacket.end());
 
         if (rawPacketString.size() >= 2) {