Skip to content

Commit 2963f91

Browse files
ten9876claude
andcommitted
chore(automation): drop unused bridge accessors (aethersdr-agent review)
isAutomationBridgeRunning() and automationBridgeEndpoint() were defined but unused. The MCP server auto-discovers the socket via the discovery file, so a UI endpoint display isn't needed — remove the dead API rather than wire a nicety. The empty-except in aether_mcp.py flagged by Copilot / aethersdr-agent already carries an explanatory comment (647bfb2). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 647bfb2 commit 2963f91

2 files changed

Lines changed: 0 additions & 14 deletions

File tree

src/gui/MainWindow.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -222,17 +222,13 @@ class MainWindow : public QMainWindow {
222222
// bridge is listening afterwards.
223223
bool startAutomationBridge(const QString& sockName = QString());
224224
void stopAutomationBridge();
225-
bool isAutomationBridgeRunning() const;
226225
// Persist a new shared-secret token and push it to the running bridge
227226
// (the Radio Setup → Network rotate button). Old tokens stop working
228227
// immediately.
229228
void setAutomationBridgeToken(const QString& token);
230229
// Persist the TX-via-MCP opt-in and push it live (Radio Setup → Network).
231230
// Enabling arms the force-unkey watchdog; disabling force-unkeys the radio.
232231
void setAutomationTxAllowed(bool allowed);
233-
// Resolved socket path / pipe name, or empty when stopped — shown in
234-
// the Network toggle so the operator can point their MCP client at it.
235-
QString automationBridgeEndpoint() const;
236232

237233
protected:
238234
void showEvent(QShowEvent* event) override;

src/gui/MainWindow_Session.cpp

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1718,16 +1718,6 @@ void MainWindow::stopAutomationBridge()
17181718
}
17191719
}
17201720

1721-
bool MainWindow::isAutomationBridgeRunning() const
1722-
{
1723-
return m_automation && m_automation->isRunning();
1724-
}
1725-
1726-
QString MainWindow::automationBridgeEndpoint() const
1727-
{
1728-
return m_automation ? m_automation->fullServerName() : QString();
1729-
}
1730-
17311721
void MainWindow::setAutomationBridgeToken(const QString& token)
17321722
{
17331723
// Persist to the OS secret store (survives restart; read by

0 commit comments

Comments
 (0)