remove port monitoring and automatic tunneling features
Remove port_monitor, TunnelManager, and /port-events handler to eliminate
automatic port tunneling functionality that bridges outtie to innie environments.
Sketch got confused when I asked it to change how this works; removing
and re-adding was easier!
Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s78f868b27a44cb2bk
diff --git a/loop/agent.go b/loop/agent.go
index f01e601..8f96924 100644
--- a/loop/agent.go
+++ b/loop/agent.go
@@ -152,8 +152,7 @@
// CompactConversation compacts the current conversation by generating a summary
// and restarting the conversation with that summary as the initial context
CompactConversation(ctx context.Context) error
- // GetPortMonitor returns the port monitor instance for accessing port events
- GetPortMonitor() *PortMonitor
+
// SkabandAddr returns the skaband address if configured
SkabandAddr() string
}
@@ -460,9 +459,6 @@
// Track outstanding tool calls by ID with their names
outstandingToolCalls map[string]string
-
- // Port monitoring
- portMonitor *PortMonitor
}
// NewIterator implements CodingAgent.
@@ -1071,8 +1067,8 @@
stateMachine: NewStateMachine(),
workingDir: config.WorkingDir,
outsideHTTP: config.OutsideHTTP,
- portMonitor: NewPortMonitor(),
- mcpManager: mcp.NewMCPManager(),
+
+ mcpManager: mcp.NewMCPManager(),
}
return agent
}
@@ -1526,12 +1522,6 @@
}
func (a *Agent) Loop(ctxOuter context.Context) {
- // Start port monitoring when the agent loop begins
- // Only monitor ports when running in a container
- if a.IsInContainer() {
- a.portMonitor.Start(ctxOuter)
- }
-
// Set up cleanup when context is done
defer func() {
if a.mcpManager != nil {
@@ -2540,11 +2530,6 @@
return nil
}
-// GetPortMonitor returns the port monitor instance for accessing port events
-func (a *Agent) GetPortMonitor() *PortMonitor {
- return a.portMonitor
-}
-
// SkabandAddr returns the skaband address if configured
func (a *Agent) SkabandAddr() string {
if a.config.SkabandClient != nil {