tree fed8d755bb3a8fffd61f6cd5da706c9267a7c5f3
parent c540e8ea6a8dd2a5297345b6f9b885988338f237
author Philip Zeyliger <philip@bold.dev> 1750472261 +0000
committer Philip Zeyliger <philip@bold.dev> 1750472503 -0700

cleanup: remove unused functions and fix tests

Remove unused Go code identified through systematic analysis:

Removed Functions:
- BashRun() in claudetool/bash.go - legacy testing function
- ContentToString() in claudetool/util.go - unused utility function
- GetActiveTunnels() in dockerimg/tunnel_manager.go - unused getter method

Removed Types:
- baseResponse struct in claudetool/browse/browse.go - unused response type

Test Updates:
- Replaced BashRun tests with direct Bash.Run calls in bash_test.go
- Removed ContentToString test from agent_test.go (testing unused function)
- Updated tunnel manager tests to access internal activeTunnels map directly
- Fixed all compilation errors caused by removed functions

All tests now pass and the codebase is cleaner with reduced maintenance burden.
The removed functions had no references in production code and were only
used in tests, which have been updated or removed as appropriate.

Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s2cac4b024f877682k