diff --git a/tailscalectl/TailscaleWidget.qml b/tailscalectl/TailscaleWidget.qml index 5b648a8..63f9df2 100644 --- a/tailscalectl/TailscaleWidget.qml +++ b/tailscalectl/TailscaleWidget.qml @@ -12,7 +12,6 @@ PluginComponent { property bool isConnected: false property string tailscaleIP: "" - property string _pendingToggleAction: "" property string currentExitNode: "" property var peers: [] property string cachedClipboardTool: "" @@ -25,10 +24,15 @@ PluginComponent { popoutHeight: 400 Timer { + id: refreshTimer interval: 5000 running: true repeat: true - onTriggered: statusCheck.running = true + onTriggered: { + if (!statusCheck.running) { + statusCheck.running = true + } + } } Component.onCompleted: { @@ -42,11 +46,10 @@ PluginComponent { onExited: (code, status) => { if (code !== 0) { - var action = root._pendingToggleAction || (root.isConnected ? "disconnect" : "connect") + var action = root.isConnected ? "disconnect" : "connect" var detail = toggleProcess.stderr.text.trim().slice(0, 120) ToastService.showError("tailscalectl", TailscaleLib.errorMessage(action) + (detail ? " — " + detail : "")) } - root._pendingToggleAction = "" statusCheck.running = true } } @@ -92,18 +95,16 @@ PluginComponent { command: ["tailscale", "status", "--json"] - stdout: StdioCollector { - onStreamFinished: { - const state = TailscaleLib.parseStatusResult(this.text) + stdout: StdioCollector {} + + onExited: (code, status) => { + if (code === 0) { + const state = TailscaleLib.parseStatusResult(statusCheck.stdout.text) root.isConnected = state.isConnected root.tailscaleIP = state.tailscaleIP root.currentExitNode = state.currentExitNode root.peers = state.peers - } - } - - onExited: (code, status) => { - if (code !== 0) { + } else { root.isConnected = false root.tailscaleIP = "" root.currentExitNode = "" @@ -114,14 +115,11 @@ PluginComponent { } function toggleTailscale() { - if (toggleProcess.running) return - root._pendingToggleAction = root.isConnected ? "disconnect" : "connect" toggleProcess.command = TailscaleLib.buildToggleCommand(root.isConnected) toggleProcess.running = true } function setExitNode(hostname) { - if (exitNodeProcess.running) return exitNodeProcess.command = TailscaleLib.makeExitNodeCommand(hostname) exitNodeProcess.running = true } @@ -304,14 +302,9 @@ PluginComponent { } } - // propagateComposedEvents: true so that right-clicks both trigger our context menu - // *and* bubble to any parent MouseArea (e.g. for shell-level drag handling). - // Documented because the default (false) is far more common and this choice - // frequently surprises future maintainers. MouseArea { anchors.fill: parent acceptedButtons: Qt.RightButton - propagateComposedEvents: true onClicked: { root.toggleTailscale() } diff --git a/tailscalectl/lib.js b/tailscalectl/lib.js index e24d8ba..fa684e8 100644 --- a/tailscalectl/lib.js +++ b/tailscalectl/lib.js @@ -1,6 +1,7 @@ function parsePeers(peerMap) { if (!peerMap) return [] return Object.keys(peerMap).map(function (key) { + if (!Object.prototype.hasOwnProperty.call(peerMap, key)) return null var p = peerMap[key] return { hostname: p.HostName || key, @@ -8,16 +9,20 @@ function parsePeers(peerMap) { online: p.Online || false, exitNode: p.ExitNodeOption || false } - }) + }).filter(function (peer) { return peer !== null }) } function makeExitNodeCommand(hostname) { + if (hostname === "") { + return ["tailscale", "set", "--exit-node="] + } return ["tailscale", "set", "--exit-node=" + hostname] } function findActiveExitNode(peerMap) { if (!peerMap) return "" for (const key in peerMap) { + if (!Object.prototype.hasOwnProperty.call(peerMap, key)) continue const p = peerMap[key] if (p.ExitNode) { return p.HostName || key diff --git a/tailscalectl/plugin.json b/tailscalectl/plugin.json index f0885d1..6d9280b 100644 --- a/tailscalectl/plugin.json +++ b/tailscalectl/plugin.json @@ -3,7 +3,7 @@ "name": "Tailscale", "description": "Tailscale status and controls on the Dank Bar", "version": "0.1.0", - "author": "John Morris & Vybe (AI Slop... er... Coding Assistant)", + "author": "John Morris", "icon": "vpn_key", "type": "widget", "component": "./TailscaleWidget.qml",