gui: convert ProcManager to separate Window, sync to active session

- Convert from Dialog to Window (not modal, separate window) - Title shows current session name - Syncs to activeSessionId/activeAgentId changes in main window - Header shows which session is being displayed - Double-click row to view output - Sort processes by PID descending (newest first) - Clear selection when session changes
2026-10-08 14:57:25 +02:00 · 2026-10-08 14:57:25 +02:00 · 4774cb9fe9
parent 491f5ff2ae
commit 4774cb9fe9
5 changed files with 104 additions and 34 deletions
--- a/kde/CMakeLists.txt
+++ b/kde/CMakeLists.txt
@ -145,7 +145,7 @@ qt_add_resources(ollie-gui "gui"
        gui/NewSessionDialog.qml
        gui/NewAgentDialog.qml
        gui/GoalDialog.qml
-        gui/ProcManagerDialog.qml
+        gui/ProcManagerWindow.qml
        gui/ChatBlockDelegate.qml
        gui/SessionTree.qml
        gui/ChatPane.qml
--- a/kde/gui/ChatPane.qml
+++ b/kde/gui/ChatPane.qml
@ -361,7 +361,7 @@ Item {
                    text: "⚙️ Procs"
                    font.pixelSize: theme.fontSize > 0 ? theme.fontSize * 0.9 : 11
                    implicitHeight: 28
-                    onClicked: procManagerDialog.open()
+                    onClicked: procManagerWindow.show()
                    ToolTip.visible: hovered
                    ToolTip.text: "Background processes"
                }
--- a/kde/gui/ProcManagerWindow.qml
+++ b/kde/gui/ProcManagerWindow.qml
@ -1,39 +1,85 @@
 import QtQuick
 import QtQuick.Controls
 import QtQuick.Layouts
+import QtQuick.Window

-Dialog {
-    id: procDialog
-    title: "Background Processes"
-    width: 600
-    height: 400
-    modal: false
-    standardButtons: Dialog.Close
+Window {
+    id: procWindow
+    title: "Background Processes" + (currentSessionName ? " — " + currentSessionName : "")
+    width: 650
+    height: 450
+    minimumWidth: 400
+    minimumHeight: 300

    property var procs: []
    property string outputText: ""
    property int selectedPid: -1

+    // Track the session we're displaying (synced from main window)
+    property string currentSessionId: ollie.activeSessionId
+    property string currentAgentId: ollie.activeAgentId
+    property string currentSessionName: ""
+
    function refresh() {
-        procs = ollie.listProcs(ollie.activeSessionId, ollie.activeAgentId)
+        if (!currentSessionId || !currentAgentId) {
+            procs = []
+            return
+        }
+        var rawProcs = ollie.listProcs(currentSessionId, currentAgentId)
+        // Sort by PID descending (newest first)
+        rawProcs.sort(function(a, b) { return b.pid - a.pid })
+        procs = rawProcs
+    }
+
+    function updateSessionName() {
+        if (currentSessionId) {
+            var cfg = ollie.readSessionConfig(currentSessionId)
+            currentSessionName = cfg.name || currentSessionId
+        } else {
+            currentSessionName = ""
+        }
    }

    Timer {
        id: refreshTimer
        interval: 2000
-        running: procDialog.visible
+        running: procWindow.visible
        repeat: true
        onTriggered: refresh()
    }

    onVisibleChanged: {
-        if (visible) refresh()
+        if (visible) {
+            currentSessionId = ollie.activeSessionId
+            currentAgentId = ollie.activeAgentId
+            updateSessionName()
+            refresh()
+        }
    }

+    // Sync to active session/agent changes
    Connections {
        target: ollie
+        function onActiveSessionIdChanged() {
+            if (procWindow.visible) {
+                currentSessionId = ollie.activeSessionId
+                currentAgentId = ollie.activeAgentId
+                updateSessionName()
+                selectedPid = -1
+                outputText = ""
+                refresh()
+            }
+        }
+        function onActiveAgentIdChanged() {
+            if (procWindow.visible) {
+                currentAgentId = ollie.activeAgentId
+                selectedPid = -1
+                outputText = ""
+                refresh()
+            }
+        }
        function onProcStarted(sessionId, agentId, payload) {
-            if (sessionId === ollie.activeSessionId && agentId === ollie.activeAgentId) {
+            if (procWindow.visible && sessionId === currentSessionId && agentId === currentAgentId) {
                refresh()
            }
        }
@ -41,9 +87,25 @@ Dialog {

    ColumnLayout {
        anchors.fill: parent
-        anchors.margins: 8
+        anchors.margins: 12
        spacing: 8

+        // Header showing current session/agent
+        RowLayout {
+            Layout.fillWidth: true
+            spacing: 8
+
+            Label {
+                text: "Session:"
+                font.bold: true
+            }
+            Label {
+                text: currentSessionName || "(none)"
+                Layout.fillWidth: true
+                elide: Text.ElideMiddle
+            }
+        }
+
        // Process list
        ListView {
            id: procList
@ -52,14 +114,20 @@ Dialog {
            model: procs
            clip: true

+            ScrollBar.vertical: ScrollBar { policy: ScrollBar.AsNeeded }
+
            delegate: Rectangle {
                width: procList.width
-                height: 48
+                height: 44
                color: selectedPid === modelData.pid ? palette.highlight : (index % 2 ? palette.alternateBase : palette.base)

                MouseArea {
                    anchors.fill: parent
                    onClicked: selectedPid = modelData.pid
+                    onDoubleClicked: {
+                        selectedPid = modelData.pid
+                        outputText = ollie.procOutput(currentSessionId, currentAgentId, selectedPid)
+                    }
                }

                RowLayout {
@ -69,17 +137,17 @@ Dialog {

                    // Status indicator
                    Rectangle {
-                        width: 12
-                        height: 12
-                        radius: 6
-                        color: modelData.status === "running" ? "green" : (modelData.exitCode === 0 ? "gray" : "red")
+                        width: 10
+                        height: 10
+                        radius: 5
+                        color: modelData.status === "running" ? "#4CAF50" : (modelData.exitCode === 0 ? "#9E9E9E" : "#F44336")
                    }

                    // PID
                    Label {
                        text: modelData.pid
                        font.family: "monospace"
-                        Layout.preferredWidth: 60
+                        Layout.preferredWidth: 50
                    }

                    // Tool
@ -102,17 +170,17 @@ Dialog {
                    Label {
                        text: formatRuntime(modelData.runtime)
                        font.family: "monospace"
-                        Layout.preferredWidth: 60
+                        Layout.preferredWidth: 55
                        horizontalAlignment: Text.AlignRight
                    }

                    // Exit code (if exited)
                    Label {
                        visible: modelData.status !== "running"
-                        text: "exit " + modelData.exitCode
+                        text: modelData.exitCode === 0 ? "✓" : "exit " + modelData.exitCode
                        font.family: "monospace"
-                        color: modelData.exitCode === 0 ? "gray" : "red"
-                        Layout.preferredWidth: 50
+                        color: modelData.exitCode === 0 ? "#4CAF50" : "#F44336"
+                        Layout.preferredWidth: 45
                    }
                }
            }
@ -120,15 +188,15 @@ Dialog {
            Label {
                anchors.centerIn: parent
                visible: procs.length === 0
-                text: "No background processes"
+                text: currentSessionId ? "No background processes" : "No session selected"
                opacity: 0.5
            }
        }

-        // Output pane (collapsible)
+        // Output pane (expandable)
        GroupBox {
            Layout.fillWidth: true
-            Layout.preferredHeight: outputText ? 150 : 0
+            Layout.preferredHeight: outputText.length > 0 ? 150 : 0
            visible: outputText.length > 0
            title: "Output (PID " + selectedPid + ")"

@ -140,6 +208,7 @@ Dialog {
                    font.family: "monospace"
                    font.pointSize: 9
                    wrapMode: TextEdit.Wrap
+                    selectByMouse: true
                }
            }
        }
@ -160,7 +229,7 @@ Dialog {
                icon.name: "document-preview"
                enabled: selectedPid > 0
                onClicked: {
-                    outputText = ollie.procOutput(ollie.activeSessionId, ollie.activeAgentId, selectedPid)
+                    outputText = ollie.procOutput(currentSessionId, currentAgentId, selectedPid)
                }
            }

@ -171,7 +240,7 @@ Dialog {
                icon.name: "process-stop"
                enabled: selectedPid > 0 && isRunning(selectedPid)
                onClicked: {
-                    ollie.procTerm(ollie.activeSessionId, ollie.activeAgentId, selectedPid)
+                    ollie.procTerm(currentSessionId, currentAgentId, selectedPid)
                    Qt.callLater(refresh)
                }
            }
@ -181,7 +250,7 @@ Dialog {
                icon.name: "edit-delete"
                enabled: selectedPid > 0 && isRunning(selectedPid)
                onClicked: {
-                    ollie.procKill(ollie.activeSessionId, ollie.activeAgentId, selectedPid)
+                    ollie.procKill(currentSessionId, currentAgentId, selectedPid)
                    Qt.callLater(refresh)
                }
            }
--- a/kde/gui/main.qml
+++ b/kde/gui/main.qml
@ -189,9 +189,10 @@ ApplicationWindow {
        id: goalDialog
    }

-    // --- Process Manager Dialog ---
-    ProcManagerDialog {
-        id: procManagerDialog
+    // --- Process Manager Window ---
+    ProcManagerWindow {
+        id: procManagerWindow
+        visible: false
    }

    SplitView {
--- a/kde/gui/qml.qrc
+++ b/kde/gui/qml.qrc
@ -5,7 +5,7 @@
        <file>NewSessionDialog.qml</file>
        <file>NewAgentDialog.qml</file>
        <file>GoalDialog.qml</file>
-        <file>ProcManagerDialog.qml</file>
+        <file>ProcManagerWindow.qml</file>
        <file>ChatBlockDelegate.qml</file>
        <file>SessionTree.qml</file>
        <file>ChatPane.qml</file>