)]}' { "log": [ { "commit": "bdc6889f886667b06236fb20c5d080e3ac1f1eb8", "tree": "f86ee68725cba394d1df667723a9a3dad49c35a7", "parents": [ "6fe809c2282f5d7e258ab60f4ff4d0b3e0739332" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 28 17:28:13 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 30 18:53:52 2025 +0000" }, "message": "sketch: add support for \u0027external\u0027 message types\n\n- adds a new CodingAgentMessageType for loop.AgentMessage\n- adds an new /external handler to loophttp.go\n- modifies Agent to pass the .TextContent of ExternalMessage into the convo\n as though it came from the user.\n- adds sketch-external-message web component, with a template for\n github workflow run events specifically.\n- adds demos for sketch-external-message\n" }, { "commit": "4a370aa1fe7a2fcafc41c0cd1bb4bf59bfb070a2", "tree": "995171cbf32df2c5b5973a942def8456f075c533", "parents": [ "c3e6df8e2eb874b6ddc59beac72a1557c5fef7e1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 28 23:19:48 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 10:25:01 2025 -0700" }, "message": "loop/server: add /debug/system-prompt, move logs to /debug/logs\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sdadfdbbcd1589be1k\n" }, { "commit": "83cf606957e8d05f1dc690991a178b302d848877", "tree": "e5b0bd4133847e0c551520a52e618c0b39148648", "parents": [ "02f4820e4a5245f545293c34e121d52640d3cb11" ], "author": { "name": "philz", "email": "philip.zeyliger@gmail.com", "time": "Mon Jul 28 14:23:04 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 28 14:23:04 2025 -0700" }, "message": "loop: add /debug/tools handler for mcp and tool inspection\n\nAdds a new debug endpoint at /debug/tools that displays all available\ntools including their names, descriptions, JSON schemas, and metadata.\nShows both built-in Claude tools and MCP tools with proper type indication.\nIncludes nicely formatted HTML with syntax highlighting for schemas.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s10d266ddbc5a7fc0k\n" }, { "commit": "e34ffd60e0a8dfc5ba55be35d68a396a73d53750", "tree": "208c68febe0f077054d1709cc732192aee367413", "parents": [ "783ab31ae38604be5331ec030264dcec08574aae" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 13:20:49 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 13:23:37 2025 -0700" }, "message": "sketch: fix diff view editing of gitignore\u0027d files and forward more http errors to logs\n\nFixes https://github.com/boldsoftware/sketch/issues/213\n\nWe had \"sketch\" git ignored, so \"git add sketch/cmd/sketch/main.go\" was\nfailing when a user was editing it in diff view. The gitignore was\nincorrectly specified. (\"git ls-files -i -c --exclude-standard\"\nreturning main.go should have tipped us off, but who knew!)\n\nAnyway, fixed that, and improved the logging.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3ed65211dd497f76k\n" }, { "commit": "4571fd6d06f65195341791b46d84dc80bb0676b3", "tree": "6e61da4d6a9ed29891c0b84f503ff5ccb5441741", "parents": [ "c540df7f730eb0e07a5aa543ec128dba38a7cf81" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 16:56:02 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 12:06:18 2025 -0700" }, "message": "webui: add model name to info popup k/v pairs\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s89edebe17dd6d6edk\n" }, { "commit": "c67d7bced3a3619cbf031075fa8442a696381c4a", "tree": "27e8b6990244865fd3d020ea1fd4ef56f61fc603", "parents": [ "3b44cc3120b595e82dcc2a6081b8a07fc24c739e" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 23 10:59:02 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 23 19:09:31 2025 +0000" }, "message": "webui/src/data: handle ended and read-only sessions\n" }, { "commit": "254c49faa2537d79e1394c4109f48a6e0990967c", "tree": "3b13d86f48e45846dc225ccd24253f5b0dca0398", "parents": [ "55b4ab50d0ee61e15825cb4254cfc63b8f5489bd" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 17 17:26:24 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 00:40:30 2025 +0000" }, "message": "sketch: \"git push\" button\n\nUltimately, we want to allow users to push their changes to github, and\nthereby do a good chunk of work without resorting to the terminal (and\nfiguring out how to move the git references around, which requires a\nbunch of esotiric and annoying expertise).\n\nThis commit introduces:\n\n1. For outtie\u0027s HTTP server (which is now comically Go HTTP -\u003e\n CGI-effing-bin -\u003e git -\u003e shell script -\u003e git in this case), there\u0027s a\n custom git hook that forwards changes to refs/remotes/origin/foo to\n origin/foo. This is a git proxy of sorts. By forwarding the\n SSH_AUTH_SOCK, we can use outtie\u0027s auth options without giving innie\n the actual credentials. This works by creating a temporary directory\n for git hooks (for outtie).\n\n2. Innie sets up a new remote, \"upstream\" when a \"passthrough-upstream\"\n flag is pasksed. This remote kind of looks like the real upstream (so\n upstream/foo) is fetched. This will let the agent handle rebases\n better.\n\n3. Innie exposes a /pushinfo handler that returns the list of remotes\n and the current commit and such. These have nice display names for\n the outtie\u0027s machine and github if useful.\n\n There\u0027s also a /push handler. This is the thing that knows about the\n refs/remotes/origin/foo thing. There\u0027s no magic git push refspec that\n makes this all work without that, I think. (Maybe there is? I don\u0027t\n think there is.)\n\n Note that there\u0027s been some changes about what the remotes look like,\n and when we use the remotes and when we use agent.GitOrigin().\n We may be able to simplify this by using git\u0027s insteadof\n configurations, but I think it\u0027s fine.\n\n4. The web UI exposes a button to push, choose the remote and branch,\n and such. If it can\u0027t do the push, you\u0027ll get a button to try to get\n the agent to rebase.\n\n We don\u0027t allow force pushes in the UI. We\u0027re treating those\n as an advanced feature, and, if you need to do that, you can\n figure it out.\n\nThis was collaboration with a gazillion sketch sessions.\n" }, { "commit": "a8561f70c1701376e88a5b63d78e7d09a7ae3ed8", "tree": "6093a6ba26567089bf33a70f74399eca2c149391", "parents": [ "b58bbf30db0ced33cc110dd44d6c14995977737f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 15 23:47:59 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 16 13:08:59 2025 -0700" }, "message": "sketch/webui: add untracked files notification to diff view\n\nAdd warning in diff view about untracked files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s45fbbdf5b3d703e4k\n" }, { "commit": "43a0bfc2227f07db26ff785470fbac35d309499a", "tree": "e1900f7602fed9ac779181b59790f6b1e0b44a7f", "parents": [ "d9acaa785a8cc8b3cc14d6efcdd7ce45ffe3737f" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 14:54:27 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 15:13:41 2025 -0700" }, "message": "sketch: add debug handler to dump conversation history as JSON\n\nAdd HTTP debug endpoint /debug/conversation-history to dump agent conversation\nhistory as pretty-printed JSON for debugging purposes.\n\nSometimes, you just want to see what went on.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6c9e876db9b3aa5ck\n" }, { "commit": "fadffe3cfb867760be035e72b7096946554255a1", "tree": "eb66acbe6f8d6f9825a1da92a3b9df0b3b7f0901", "parents": [ "e3c0f4d054cc7ec2859cffd3f61383d8a122a53c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 10 00:08:38 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 19:23:57 2025 -0700" }, "message": "sketch/loop: fuss more with /git/cat and 204s\n\nFor unknown reasons, switching to 204s made the diff view noticeably slower.\nBe more explicit about things in the hope that it helps.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s52c5ab6557e1f9dbk\n" }, { "commit": "a002a23a60dcc21353681a887a90cd12867e7f44", "tree": "e554f73faa6681dec2cbee396f30b03b972ab66b", "parents": [ "5ab8fb8dc17299df5d134d7a95f33d41b170f102" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 19:38:03 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 14:37:14 2025 -0700" }, "message": "webui: delete non-gzip\u0027d assets\n\nReduces webui assets from 83M to 16M.\nAlmost all browsers support gzip.\nFor those that don\u0027t, we provide on-the-fly decompression.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sbc10790564044d6bk\n" }, { "commit": "5ab8fb8dc17299df5d134d7a95f33d41b170f102", "tree": "bbc6f1a77e2eeffc2ab759270711e0fbe6a9118a", "parents": [ "b7ec9c837a549abd9bbbc1230e6d96090c98af52" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 09 12:34:55 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 09 12:42:12 2025 -0700" }, "message": "webui: display context window size in info box\n" }, { "commit": "b843146dd360f53c2feb2942d4a6ef4ccc9109fb", "tree": "1be327ecc49991def691d5fd19ba0486dc678654", "parents": [ "e7ca73df4b61cff1847c4f63feac89c580e998c7" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jul 09 13:10:32 2025 +1000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jul 09 13:14:12 2025 +1000" }, "message": "server: Wait on terminal process for cleanup\n\nThe bash shell for the termianl is an exec.Cmd that is Start()ed\nin the github.com/creack/pty package. We do a lot of cleanup\nwhen that shell exits, but we never Wait() on it (and neither\ndoes the pty package), so the bash process gets left behind\nas a zombie.\n\nFixes boldsoftware/sketch#181\n" }, { "commit": "5c29b3e17fa526aa912d3ba400cf8a1a9c83fd80", "tree": "a57a2cf6f5420e866c4cd86d8184d0d2dd8b87ad", "parents": [ "1c18ec94b0d2edcc6d69d1b96f89e3c6e630fc11" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 18:07:28 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 13:44:11 2025 -0700" }, "message": "sketch/loop: return 204 for not-found files in /git/cat\n\nThis isn\u0027t actually an error; no reason to return a 500.\nThe front-end handles this identically,\nbut without js console log spam,\nwhich is the motivation for this change.\n" }, { "commit": "1c18ec94b0d2edcc6d69d1b96f89e3c6e630fc11", "tree": "994dbcf813d9c772da320ccccfc2e2f5fe826155", "parents": [ "e1a6e1b25d1cf7f3aec4b98e155be7cf3701a1b0" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 10:55:54 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 13:06:43 2025 -0700" }, "message": "all: use make to build\n\nThis overhauls the build system.\nWe used to use a just-in-time clever build system\nso that \u0027go run\u0027 and \u0027go install\u0027 Just Worked.\n\nThis was really nice, except that it make it\nall but impossible to ship a single binary.\nIt also required our uses to install npm,\nwhich some folks have an understandably negative reaction to.\n\nThis migrates to a makefile for building.\nThe core typescript building logic is mostly still in Go,\nand untouched (boy did I learn that lesson the hard way).\n\nThe output is a single file that includes the webui, innie, and outie.\n\n(There are still very mild shenanigans in which we write outie\nout to a temp file and then \u0027docker cp\u0027 it into the docker container.\nBut this is pretty manageable.)\n\nThere are some significant follow-ups left after this commit:\n\n- convert the nightly release builds to use the makefile\n- lots of dead code removal\n- maybe add -race support using a dockerfile for the cgo compilation\n- maybe use \u0027docker cp\u0027 stdin reading with tar to avoid the temp outtie file\n- all the rest of the \"better release\" todos (brew install, etc.)" }, { "commit": "5f26a3445601f6ab0299d9be20ea99b67eae4d51", "tree": "740ba6f60a055e4e6ad8ec92e75992ca9d2b0795", "parents": [ "da623b50da804963768d2633cb2686a9d91d49b9" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 04 01:30:29 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 21:14:55 2025 -0700" }, "message": "sketch/loop: add PortMonitor for TCP port monitoring with Agent integration\n\nAdd PortMonitor struct that uses Tailscale portlist library to monitor\nopen/listening TCP ports and send AgentMessage notifications to Agent\nwhen ports are opened or closed, with cached port list access method.\n\nWhen I asked Sketch to do this with the old implementation, it did\nok parsing /proc, but then it tried to conver it to ss format...\nusing a library seems to work ok!\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8fc57de4b5583d34k\n" }, { "commit": "da623b50da804963768d2633cb2686a9d91d49b9", "tree": "74b03bb5091e12162088e77ef4e32fba91412821", "parents": [ "2153f8b8eeb9215ed4b79af3aef09de1af83decd" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 04 01:12:38 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 20:57:50 2025 -0700" }, "message": "remove port monitoring and automatic tunneling features\n\nRemove port_monitor, TunnelManager, and /port-events handler to eliminate\nautomatic port tunneling functionality that bridges outtie to innie environments.\n\nSketch got confused when I asked it to change how this works; removing\nand re-adding was easier!\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s78f868b27a44cb2bk\n" }, { "commit": "a9710d76e938c29fbfaa30e5d63129bbc43b18c8", "tree": "56f7a79e58d8c5ed86695ba279f2395cf1e13e16", "parents": [ "fd67b0185a488aeac149e0bcf1a476d628134555" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 02 02:50:14 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jul 01 19:53:20 2025 -0700" }, "message": "sketch: proxy to ports via p\u003cport\u003e.localhost Host headers\n\nAdd support for proxying requests based on Host header patterns.\nWhen Host header matches p\u003cport\u003e.localhost, proxy the request to localhost:\u003cport\u003e.\n\n- ParsePortProxyHost() extracts port from p8000.localhost format\n- proxyToPort() handles generic port proxying with validation\n- Supports any valid port (1-65535) via p\u003cport\u003e.localhost pattern\n- Comprehensive tests for parsing and validation\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: saa324eab0e9b3addk\n" }, { "commit": "a14b0183208df257a43748b51666043db7e62138", "tree": "2d2306053dc2ae1139994eb1d49acc9b8e300c6a", "parents": [ "467c3964f42c1b29a18c7649e8aa33276710e475" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 14:31:18 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 14:31:18 2025 -0700" }, "message": "sketch: make /debug use relative paths\n" }, { "commit": "13e385685fb87cea70711ff387e57fbe0f725f28", "tree": "618000cfa3846350455ea01f72998dc7648c6d46", "parents": [ "59789951fdf8f9a5c834b4d079f0394c2bc42dd3" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jun 29 17:53:27 2025 +0000" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jun 29 17:53:27 2025 +0000" }, "message": "sketch: update local_ssh.md to replace SSHTheater with LocalSSHimmer\n\nUpdate component naming in SSH mutual authentication documentation from deprecated SSHTheater to current LocalSSHimmer implementation.\n\nChanges:\n- Updated sequence diagram participant name from \u0027SSHTheater\u0027 to \u0027LocalSSHimmer\u0027\n- Updated implementation component description to reference LocalSSHimmer\n- Updated SSH configuration approach section to reference LocalSSHimmer\n\nThe documentation now accurately reflects the current component naming used in the SSH authentication system implementation.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s644b455057106ba2k\n" }, { "commit": "cad67b0ac0e374dd687ca4ef8abddbc9c4323fd5", "tree": "44b66d28b1be62eb47b3cc26fd228c7031704922", "parents": [ "cebb03c0513e98e1861efe787ac1577c614e2e52" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jun 27 21:57:05 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 22:21:38 2025 +0000" }, "message": "sketch: add git username attribution for user messages in timeline\n\nImplement comprehensive user attribution system displaying git username below user message bubbles in both active sketch sessions and archived skaband message views, with full test coverage.\n\nProblems Solved:\n\nMissing User Attribution:\n- User messages in timeline lacked visible attribution for identification\n- No way to distinguish which user sent messages in shared or review contexts\n- Timeline display provided no user context beyond message type differentiation\n- Archived messages on skaband /messages/\u003csession-id\u003e page had no user attribution\n\nInconsistent Attribution Between Views:\n- Active sketch sessions and archived skaband views used different component systems\n- Messages-viewer component wasn\u0027t setting state property for timeline attribution\n- Git username information wasn\u0027t being extracted from session data in skaband\n- Version skew between sketch and skaband frontend components\n\nSolution Implementation:\n\nBackend State Management:\n- Added GitUsername() method to Agent struct returning config.GitUsername\n- Extended CodingAgent interface to include GitUsername() method\n- Added git_username field to State struct in loophttp.go\n- Populated git_username in getState() method from agent.GitUsername()\n- Enhanced skaband SessionWithData with UserName field for git username storage\n- Updated session JSON parsing to extract git_username into UserName field\n\nFrontend Timeline Component:\n- Added user attribution display outside and below user message bubbles\n- Right-edge alignment of username with message bubble edge\n- Clean visual separation between message content and attribution metadata\n- Conditional rendering only for user message types with available git_username\n- Responsive design handling both normal and compact display modes\n\nSkaband Integration:\n- Modified messages-viewer.ts to create proper State object for timeline component\n- Added git_username population with fallback hierarchy for backward compatibility\n- Enhanced session JSON parsing to extract git_username into UserName field\n- Updated all session data retrieval functions in skaband database layer\n- Ensured consistent attribution across active and archived message views\n\nVisual Design:\n- Username displays in 11px italic font below message content\n- Right-aligned to match user message bubble alignment\n- Color: #666 for clear contrast and subtle attribution appearance\n- 4px top margin for appropriate spacing from message bubble\n- CSS classes: .user-name-container and .user-name for styling\n\nImplementation Details:\n\nState Management Architecture:\n- Active sessions: Agent config → State object → timeline component\n- Archived sessions: Session JSON → SessionWithData.UserName → State object → timeline component\n- Consistent data flow ensuring attribution works in both contexts\n- Three-tier fallback: session_state.git_username → user_name → undefined\n\nData Extraction Pipeline:\n- Session JSON parsing extracts git_username using same pattern as user_email\n- Database layer updates in GetAllSessionStateData, SearchSessionsByMessageContentPaginated, GetSessionStateDataWithFilters\n- Graceful degradation for older sessions without git username data\n- No breaking changes to existing data structures or APIs\n\nComponent Integration:\n- Timeline component state property receives comprehensive git-related fields\n- Messages-viewer creates state object matching active session behavior\n- No breaking changes to existing component interfaces or data structures\n- Clean separation between message content and user attribution\n\nBackward Compatibility:\n- Older sessions without git_username gracefully show no attribution\n- New sessions have complete attribution data in both views\n- No impact on existing message display or functionality\n- Optional field design maintains compatibility with existing code\n\nTesting and Validation:\n\nComprehensive Test Coverage:\n- Created messages-viewer.test.ts with 8 test scenarios covering state creation logic\n- Added git username attribution tests to sketch-timeline-message.test.ts\n- Tested fallback hierarchy: session_state.git_username → user_name → undefined\n- Verified message filtering, data handling, and edge cases\n- All messages-viewer tests passing (8/8)\n\nTest Environment Compatibility:\n- Resolved TypeScript decorator configuration issues in test environment\n- Implemented workarounds for Lit component testing constraints\n- Fixed mock data factory functions to properly handle undefined values\n- Maintained comprehensive test coverage despite environment limitations\n\nFunctional Validation:\n- Created visual mockups confirming correct alignment and positioning\n- Verified state object creation with proper git_username extraction\n- Confirmed visual positioning and alignment requirements\n- Validated TypeScript compilation and Go build compatibility\n- Manual testing confirms runtime functionality works correctly\n\nFiles Modified:\n- sketch/loop/agent.go: Added GitUsername() method to interface and implementation\n- sketch/loop/server/loophttp.go: Added git_username to State struct and population\n- sketch/webui/src/types.ts: Added git_username field to State interface\n- sketch/webui/src/web-components/sketch-timeline-message.ts: User attribution display and positioning\n- sketch/webui/src/messages-viewer.ts: State object creation for timeline component\n- skaband/skadb/skadb.go: UserName field and git username extraction from session JSON\n- sketch/webui/src/messages-viewer.test.ts: Comprehensive test coverage for state creation\n- sketch/webui/src/web-components/sketch-timeline-message.test.ts: Timeline component tests\n\nThe implementation provides consistent user attribution across all message viewing contexts while maintaining clean visual design, full backward compatibility, and comprehensive test coverage.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: seb68c9ba94cdcc5bk\n" }, { "commit": "8105fe6bfa914c17c0f474bacdbe9d4c62b6b2b3", "tree": "3ba587467d33c5f9ef78ab20fc80a3b64967733b", "parents": [ "dd6352a74f1214035785025df127ed6a0dc73ac6" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 21 21:22:34 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 21 21:23:06 2025 -0700" }, "message": "Removing noisy log line.\n" }, { "commit": "28e39ac7817834d258a0f75a33b4590c769cb1bc", "tree": "23eed3fd8cdc60e107d5070c300dfaf488b45507", "parents": [ "6b8b7660f2e2e964453ebace40dda7122f6d9eca" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Mon Jun 16 22:04:35 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:30:06 2025 -0700" }, "message": "loop: remove unused waitForEnd parameter and endWaitGroup functionality\n\nRemove wait_for_end query parameter and associated synchronization logic\nthat was not being used in the codebase.\n\nChanges:\n- Remove waitForEnd query parameter parsing in SSE handler\n- Remove endWaitGroup sync.WaitGroup field from Server struct\n- Simplify shutdown logic to use basic 100ms delay instead of complex waitgroup timeout\n- Remove related comment documentation\n\nThis cleanup eliminates unused functionality while maintaining the same\nshutdown behavior with simpler code.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s76684f9f4ec3a5d1k\n" }, { "commit": "64f60461b00c474b0b4747a06177d5c7a357bba9", "tree": "cfeceff998a165af451099c0ec23b5d2b0fd42f2", "parents": [ "db8caa0eb8576b8c0d65cc0b7b6aa4e241a7cdb1" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 13:57:10 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jun 16 20:58:24 2025 +0000" }, "message": "loop: add diff stats from sketch-base to HEAD in /state endpoint\n\nAdd lines added/removed statistics computed from sketch-base to current HEAD,\ndisplayed in webui Diff mode button for quick change overview.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3f10ecf39df6b581k\n" }, { "commit": "8773e68fcce9965da3c6a1ef91c88476f84e29bb", "tree": "cb8d586ac3e5a833670c038344509621b61725be", "parents": [ "542bda3968c6dd5b79392dd63e2955e04520401a" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Jun 11 21:36:21 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jun 12 04:37:26 2025 +0000" }, "message": "feat: add ssh-connection-string option for container SSH access\n\nAdd internal -ssh-connection-string flag to pass SSH hostname from dockerimg\nto sketch container, allowing the UI to display the correct SSH connection\nstring based on SSH Theater configuration rather than generating it locally.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1872f10f74da5f9bk\n" }, { "commit": "6d3de48ec0ec2fb8f194a11f1711b7128b7d1699", "tree": "14325a290001d13a482c116c83ee0b87aa7e36c5", "parents": [ "a1762b94e6ddf5db0a0f6b7a5104fa236855320b" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Tue Jun 10 19:38:14 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jun 10 19:40:16 2025 -0700" }, "message": "feat: add -link-to-github flag with Octocat icon for GitHub branch linking\n\nAdd internal flag to enable GitHub branch linking in both termui and webui\ninterfaces, displaying clickable Octocat icons next to copy icons for pushed\nbranches when working with GitHub repositories.\n\nProblem Analysis:\nWhen sketch pushes branches to GitHub repositories, users had no direct way to\nnavigate from the sketch interface to view those branches on GitHub. Branch\nnames were displayed as plain text in both terminal and web interfaces,\nrequiring users to manually construct GitHub URLs or switch to external tools\nto view their pushed changes on the GitHub platform.\n\nThis created friction in the workflow, especially for teams collaborating on\nGitHub where quick access to branch views, pull request creation, and code\nreview processes are essential parts of the development workflow.\n\nImplementation Changes:\n\n1. Command Line Flag Infrastructure:\n - Added linkToGitHub bool field to CLIFlags struct\n - Configured -link-to-github as internal flag with false default\n - Integrated flag passing through ContainerConfig and AgentConfig chains\n - Added flag to dockerimg launch command arguments for container mode\n\n2. Agent Interface Enhancement:\n - Added LinkToGitHub() method to CodingAgent interface\n - Implemented method in Agent struct returning config.LinkToGitHub\n - Extended State struct with link_to_github JSON field (with omitempty)\n - Updated getState() function to include agent\u0027s GitHub linking preference\n - Updated mockAgent in tests to support new LinkToGitHub() method\n\n3. Terminal UI GitHub Integration:\n - Added isGitHubRepo() method with regex pattern matching for GitHub URLs\n - Implemented getGitHubBranchURL() for constructing GitHub branch links\n - Enhanced commit message display to show GitHub URLs when flag enabled\n - Updated exit summary to include GitHub links for single and multiple branches\n - Added regex import for GitHub URL pattern validation\n\n4. Web UI TypeScript Integration:\n - Added link_to_github field to State interface in types.ts\n - Enhanced formatGitHubRepo() method to return owner/repo extraction\n - Implemented getGitHubBranchLink() helper in container status component\n - Created parallel helper methods in timeline message component\n\n5. Container Status Component Updates:\n - Added commit-info-container with flexbox layout for proper alignment\n - Implemented layout: Branch Text → Copy Icon → Octocat Icon\n - Added 16px clipboard copy icon with opacity states (70% default, 100% on hover)\n - Integrated 16px Octocat SVG icon as clickable GitHub link\n - Maintained existing copyCommitInfo click functionality for branch text\n\n6. Timeline Message Component Enhancement:\n - Added commit-branch-container for consistent layout structure\n - Implemented same layout pattern: Branch Text → Copy Icon → Octocat Icon\n - Added 14px clipboard and Octocat icons matching timeline scale\n - Enhanced CSS with hover states and proper alignment\n - Preserved existing copyToClipboard functionality for branch text clicks\n\n7. Data Flow Integration:\n - Updated sketch-timeline component to pass state to message components\n - Modified sketch-app-shell to provide containerState to timeline\n - Ensured proper state propagation through component hierarchy\n - Maintained backward compatibility with existing state management\n\nTechnical Details:\n- GitHub URL detection supports HTTPS, SSH, and git protocol formats\n- Regex patterns: ^https://github\\.com/, ^git@github\\.com:, ^git://github\\.com/\n- Link format: https://github.com/{owner}/{repo}/tree/{branch-name}\n- Internal flag prevents exposure in user-visible help documentation\n- SVG Octocat uses official GitHub icon design with 16-point grid\n- Copy icons use standard clipboard SVG with overlapping rectangles design\n- Event propagation properly stopped to prevent interference with copy actions\n- Conditional rendering ensures icons only appear when GitHub links available\n- Flexbox layout ensures proper alignment across different screen sizes\n- CSS transitions provide smooth hover state animations\n\nBenefits:\n- Direct navigation from sketch UI to GitHub branch views\n- Seamless integration with GitHub-based development workflows\n- Enhanced productivity for teams using GitHub collaboration features\n- Clear functional separation: text copy vs external GitHub link\n- Familiar clipboard icon reinforces copy functionality\n- Improved visual hierarchy guides user interaction patterns\n- Maintains existing copy-to-clipboard functionality as fallback\n- Zero impact on non-GitHub repositories or when flag disabled\n- Consistent experience across terminal and web interfaces\n- Enhanced accessibility with distinct click targets and hover states\n\nTesting:\n- Verified flag parsing and configuration propagation through all layers\n- Confirmed GitHub URL detection works with various GitHub URL formats\n- Tested conditional rendering in both web UI components\n- Validated CSS styling and hover effects for GitHub branch links\n- Ensured backward compatibility with non-GitHub repositories\n- Verified TypeScript compilation with new template structures\n- Confirmed proper icon positioning and spacing in test layouts\n- Validated hover states and opacity transitions function correctly\n- All Go tests and TypeScript compilation successful\n\nThis enhancement bridges the gap between sketch\u0027s development environment and\nGitHub\u0027s collaboration platform, enabling more efficient workflows for teams\nusing GitHub repositories while preserving full functionality for other Git\nhosting solutions. The visual design provides intuitive flow from local\noperations (copy) to external actions (GitHub), creating a more organized\nand user-friendly interface for branch management workflows.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1c083b45b5401c2bk\n" }, { "commit": "0113be559976cfb1ce0e78a9a69c19a6394b2c3d", "tree": "103ea687419bd792471f057727d2f174428b4215", "parents": [ "8ad17ba9f185be76a4e715d9f21868b0cf27b366" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 07 23:53:41 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jun 09 19:47:55 2025 +0000" }, "message": "webui: add skaband navigation link with image to desktop and mobile\n\nAdd conditional linking functionality to make the \u0027sketch\u0027 title in both desktop\nand mobile web UI clickable when a skaband address is configured, displaying\nthe sketch.dev.png image alongside the text for enhanced branding and navigation\nback to the skaband dashboard.\n\nProblem Analysis:\nWhen users access sketch through skaband (the hosted service), they lose easy\nnavigation back to their skaband dashboard since the sketch title in the header\nwas always static text. This created a disconnected experience between the\nskaband interface and individual sketch sessions, and lacked visual branding\nconsistency with the hosted service. The feature needed to work in both\ncontainer mode and unsafe mode deployments.\n\nImplementation Changes:\n\n1. Backend Infrastructure:\n - Added SkabandAddr() method to CodingAgent interface for state management\n - Enhanced State struct with skaband_addr field (with omitempty for clean JSON)\n - Modified getState() function to include agent\u0027s skaband address in responses\n - Updated mockAgent in tests to support new SkabandAddr() method\n\n2. SkabandClient Integration:\n - Added Addr() method to SkabandClient to expose stored skaband server address\n - Updated Agent.SkabandAddr() to get address from existing SkabandClient\n - Leverages existing skaband infrastructure used for session history tools\n - No duplicate storage or additional configuration required\n\n3. TypeScript Type Updates:\n - Regenerated webui/src/types.ts to include skaband_addr field in State interface\n - Maintained backward compatibility with existing state structure\n - Auto-generated types ensure type safety across Go/TypeScript boundary\n\n4. Desktop Web UI Enhancement:\n - Modified sketch-app-shell.ts to conditionally render sketch title as link\n - Added conditional logic: renders link with image when skaband_addr exists\n - Uses target\u003d\u0027_blank\u0027 and rel\u003d\u0027noopener noreferrer\u0027 for secure external linking\n - Displays skaband_addr/sketch.dev.png image alongside \u0027sketch\u0027 text\n - Maintains existing appearance and behavior when no skaband address configured\n\n5. Mobile Web UI Implementation:\n - Added skabandAddr property to mobile-title component\n - Implemented conditional rendering matching desktop functionality\n - Mobile-optimized CSS with appropriate image sizing (18px vs 20px desktop)\n - Integrated with existing mobile-shell state management infrastructure\n\n6. CSS Styling Integration:\n - Added .banner-title a styles for seamless link appearance with flexbox layout\n - Configured color: inherit to match existing title styling\n - Added hover effects with opacity transition and underline for user feedback\n - Image styling with rounded corners and appropriate dimensions\n - Mobile-responsive design with media queries for different screen sizes\n\n7. Test Coverage:\n - Updated test infrastructure to support new SkabandAddr() method\n - Comprehensive testing of state endpoint with and without skaband address\n - Verified omitempty behavior for clean JSON responses\n - All existing tests continue passing without modification\n\nTechnical Details:\n- Uses omitempty JSON tag to exclude empty skaband addresses from API responses\n- Leverages existing containerState property in web components for state access\n- Maintains full backward compatibility with non-skaband sketch deployments\n- Clean separation between configuration, state management, and presentation layers\n- Flexbox layout for proper image and text alignment in banner links\n\nSecurity Considerations:\n- External links use target\u003d\u0027_blank\u0027 with rel\u003d\u0027noopener noreferrer\u0027\n- No user input validation needed since skaband_addr comes from server configuration\n- Link destination controlled by deployment configuration, not user input\n- Image source follows same security model as main skaband address\n\nBenefits:\n- Seamless navigation between sketch sessions and skaband dashboard\n- Enhanced visual branding with sketch.dev.png image display\n- Improved user experience for hosted skaband users across all device types\n- Mobile-optimized responsive design for touch interfaces\n- Zero impact on standalone sketch deployments\n- Uses existing, tested skaband infrastructure without additional complexity\n\nTesting:\n- Verified end-to-end functionality with real sketch.dev skaband server\n- Confirmed web UI displays clickable sketch logo link in desktop and mobile views\n- Tested state endpoint returns correct skaband_addr from SkabandClient\n- Validated responsive design across different screen sizes\n- All Go tests and TypeScript compilation successful\n\nThis enhancement provides intuitive navigation for skaband users while\nmaintaining the existing experience for standalone sketch deployments,\ncreating a more cohesive and visually branded interface across all deployment\nmodes and device types.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sa6158676609fe9f0k\n" }, { "commit": "e08c7ffc8f1ec0cd3899d2fedad79f408b6dc2f9", "tree": "3e09e7b3a86faed4eaabb640eed0c4b148a9d8d6", "parents": [ "ba15aeb81e5fd3e98870eba91398dfb31d4cd6e9" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 06 13:22:12 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 06 20:26:10 2025 +0000" }, "message": "webui: add mobile interface with URL parameter switching\n\nTL;DR: ?m and ?d should load mobile and desktop versions respectively.\nIt should auto-detect to the right onen and redirect. Server chooses\nwhat to serve.\n\nThis took a few tries. Re-using the existing components didn\u0027t work,\ndespite repeated attempts. The prompt that eventually worked was:\n\n\tSketch\u0027s webui uses lit and webcomponents. I want to create an alternate\n\tweb ui, started with \"/m\" for mobile. Don\u0027t use the same components, but\n\tcreate a new mobile-shell component and a mobile-title, mobile-chat, and\n\tmobile-chat-input components. The design should be a title at the top, a\n\tsimplified chat (that doesn\u0027t display tool cards or anything; just the\n\tmessages, with user messages right-aligned and other messages left\n\taligned), and an input for new messages. Do include an indicator for\n\twhether or not the agent is thinking. Again: this is a parallel\n\timplementation, intended for mobile screens. Use the \"npm run dev\"\n\tserver to test it out and show me some screenshots. Use mobile browser\n\tsizes. Focus on simplicity in the CSS\n\nIt had some trouble with the data loading that took some iterations, and\nit kept saying, \"We\u0027re almost done\" and giving up, but I coaxed it\nthrough.\n\nI\u0027m not too sad right now about the duplication. We can see if there\u0027s\nany traction.\n\n~~~~\n\nAdd complete mobile-optimized interface for Sketch accessible via URL\nparameters, providing seamless device-specific user experience.\n\nProblem Analysis:\nSketch\u0027s existing web interface was designed for desktop use with complex\nlayouts, detailed toolbars, and mouse-focused interactions that don\u0027t\ntranslate well to mobile devices. Mobile users needed a simplified,\ntouch-friendly interface optimized for smaller screens while maintaining\ncore functionality for coding assistance on mobile devices.\n\nImplementation Changes:\n\n1. Mobile-Specific Components:\n - Created mobile-shell as main container with mobile-optimized layout\n - Built mobile-title with connection status and clean header design\n - Implemented mobile-chat with simplified message bubbles and alignment\n - Developed mobile-chat-input with touch-friendly controls and auto-resize\n\n2. URL Parameter-Based Interface Selection:\n - Added server-side parameter detection (?m for mobile, ?d for desktop)\n - Consolidated routing logic into main / handler with parameter checking\n - Eliminated separate URL paths in favor of clean parameter approach\n - Removed interface switching buttons for minimal UI approach\n\n3. Intelligent Auto-Detection:\n - Implemented client-side device detection using screen size and touch capability\n - Automatic URL parameter addition for detected mobile devices\n - Graceful parameter-based overrides for user preference\n - Seamless redirection maintaining all URL context and query parameters\n\n4. Mobile-Optimized Design:\n - iOS-style message bubbles with proper user/assistant alignment\n - Touch-friendly input controls with appropriate sizing\n - Responsive design scaling across mobile screen sizes\n - Animated thinking indicator with smooth dot animations\n - Clean, distraction-free interface focused on core chat functionality\n\n5. Data Integration:\n - Reused existing DataManager and SSE infrastructure\n - Maintained real-time message updates via /stream endpoint\n - Implemented message aggregation using existing patterns\n - Preserved connection status and error handling functionality\n\nTechnical Details:\n- Built using Lit web components for consistency with existing architecture\n- Server parameter detection uses URL.Query().Has() for efficient checking\n- Auto-detection preserves all existing query parameters during redirection\n- Mobile components filter messages to show user, agent, and error types\n- Responsive CSS with mobile-first design principles and touch optimization\n\nBenefits:\n- Seamless mobile experience with automatic device-appropriate interface\n- Clean URL parameter approach (?m/?d) works with any route or session\n- No UI clutter from interface switching buttons\n- Maintains full real-time functionality on mobile devices\n- Easy to bookmark and share mobile-specific URLs\n- Preserves session context during interface switching\n\nTesting:\n- Verified auto-detection works across different mobile screen sizes\n- Confirmed URL parameters function with complex session URLs\n- Tested message sending, receiving, and real-time updates\n- Validated thinking indicator and connection status display\n- Ensured responsive design across portrait and landscape orientations\n\nThis implementation provides a complete mobile solution while maintaining\nthe full-featured desktop interface, enabling Sketch usage across all\ndevice types with appropriate user experiences.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sdbce185f247638c1k\n" }, { "commit": "19969a9af6da4b5b5e25d02aa635dabad196e464", "tree": "d4701576dce7bdffd962b1d11cd3e6b37d4be443", "parents": [ "44f847a93e7e0953abd42880f5e87c8b3e213134" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 14:34:02 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 19:30:39 2025 -0700" }, "message": "all: s/title/slug/, adjust branch handling\n\nThere are two intertwined changes here.\n\nFirst, replace title with slug, and precommit with commit-message-style.\n\nThe slug makes enough of a title, and it provides a single human-readable\nidentifier we can use everywhere.\n\nSecond, construct the branch name on the fly instead of storing it,\nout of slug, branch prefix, and retryNumber.\nThis removes some duplicated data, and makes the retry loop\neasier to follow and reason about." }, { "commit": "be7802ab0605e7cf116d0979bda6e0b6af8f120d", "tree": "d9b2bc73b8bf5fcef160e83c10638b62bcc85863", "parents": [ "2abd467f1bd5fd5cc1de8719eb414a67a412f655" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 20:15:25 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 15:19:01 2025 -0700" }, "message": "loop: add configurable branch prefix option\n\nAdd -branch-prefix flag to customize git branch prefix instead of\nhardcoded \u0027sketch/\u0027 for better integration with different workflows.\n\nProblem Analysis:\nThe sketch system hardcoded \u0027sketch/\u0027 as the branch prefix throughout\nthe codebase, making it difficult for users who wanted different\nbranch naming conventions or needed to integrate with existing\ndevelopment workflows that used different prefixes.\n\nImplementation Changes:\n\n1. CLI Flag Addition:\n - Added -branch-prefix flag to main.go with default \u0027sketch/\u0027\n - Integrated flag into CLIFlags struct and argument parsing\n - Maintains backward compatibility with existing workflows\n\n2. Configuration Threading:\n - Added BranchPrefix field to AgentConfig struct\n - Added BranchPrefix field to ContainerConfig struct\n - Modified container argument passing to include branch prefix\n - Set sensible default of \u0027sketch/\u0027 when not specified\n\n3. Agent Implementation:\n - Added BranchPrefix() method to CodingAgent interface\n - Updated precommit tool to use configurable prefix\n - Modified git commit handling to use configurable prefix\n - Updated all hardcoded \u0027sketch/\u0027 references in logic\n\n4. UI Integration:\n - Enhanced termui template to use configurable branch prefix\n - Added branch_prefix field to server State response\n - Updated webui TypeScript types via auto-generation\n - Modified sketch-tool-card to display configurable prefix\n\n5. Container Integration:\n - Modified dockerimg to pass branch prefix to inner sketch\n - Updated container command arguments construction\n - Ensured proper flag threading through container boundary\n\nTechnical Details:\n- Maintains full backward compatibility with existing usage\n- Default behavior unchanged for existing users\n- Type-safe implementation with proper error handling\n- Clean interface design without type casts or fallbacks\n\nBenefits:\n- Enables custom branch naming conventions\n- Better integration with team workflows\n- Maintains existing sketch functionality\n- Improves flexibility for different use cases\n\nTesting:\n- Verified default behavior remains unchanged\n- Confirmed custom prefix works in both termui and webui\n- All existing tests continue to pass\n- Build verification successful across all components\n\nThis enhancement provides workflow flexibility while maintaining\nthe reliability and functionality of the existing branch management\nsystem.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s71341dca2cfeeb24k\n" }, { "commit": "16098932295e067fb0a6b3ca2082b0d4b06027b4", "tree": "60a47204f50aab38d803f4cc53594ff754cc0265", "parents": [ "e6c294dc139cf229ba790abc87a524016f98627f" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 11:02:55 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jun 04 18:03:43 2025 +0000" }, "message": "loop/webui: remove end session feedback survey functionality\n\nRemove end session feedback survey system that was previously implemented\nto collect user satisfaction ratings and comments when ending sessions.\n\nProblem Analysis:\nThe feedback survey system added complexity to the end session flow\nand included several components:\n- Modal dialog with thumbs up/down rating buttons\n- Optional text feedback collection\n- Backend storage and processing of feedback data\n- Complex synchronization logic with skaband clients\n- Extended State struct and API surface area\n\nThe feedback collection mechanism, while well-implemented, added\nunnecessary friction to session termination and required maintenance\nof additional state management infrastructure.\n\nImplementation Changes:\n\n1. Frontend Simplification (sketch-app-shell.ts):\n - Replace custom survey modal with simple window.confirm dialog\n - Remove showEndSessionSurvey() method and associated UI logic\n - Simplify end session request to basic reason-only payload\n - Remove complex button state management and form handling\n\n2. Type Definition Cleanup (types.ts):\n - Remove EndFeedback interface definition\n - Remove end field from State interface\n - Simplify TypeScript type definitions\n\n3. Backend Refactoring (agent.go):\n - Remove EndFeedback struct definition\n - Remove GetEndFeedback/SetEndFeedback methods from CodingAgent interface\n - Remove endFeedback field from Agent struct\n - Eliminate feedback-related state management\n\n4. HTTP Server Cleanup (loophttp.go):\n - Remove End field from State struct\n - Simplify /end endpoint request body parsing\n - Remove feedback storage and processing logic\n - Eliminate endWaitGroup synchronization mechanism\n - Remove wait_for_end query parameter handling\n - Simplify session termination to immediate exit with basic delay\n\n5. Test Cleanup (loophttp_test.go):\n - Remove TestEndFeedback test function\n - Remove endFeedback field from mockAgent\n - Remove GetEndFeedback/SetEndFeedback mock methods\n - Simplify test fixtures and expectations\n\nTechnical Details:\n- Session termination now uses simple confirmation dialog workflow\n- /end endpoint accepts only reason field in request body\n- Process exit occurs after 100ms delay without client coordination\n- All existing session management functionality preserved\n- TypeScript compilation verified with simplified type definitions\n\nBenefits:\n- Simplified end session user experience with standard confirmation\n- Reduced code complexity and maintenance burden\n- Eliminated complex state synchronization requirements\n- Cleaner API surface with fewer fields and methods\n- Faster session termination without feedback collection overhead\n- Standard web UI patterns for session ending confirmation\n\nTesting:\n- All existing tests pass without regression\n- TypeScript compilation succeeds with updated type definitions\n- End session functionality verified through manual testing\n- Build verification confirms no compilation errors\n\nThis change restores the end session flow to a simple, straightforward\nconfirmation-based approach while maintaining all core session management\nfunctionality and improving overall system simplicity.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9406002b5ac20a89k\n" }, { "commit": "138ec2436631f136dd2e8b4891211f896587ff00", "tree": "80afc394ad900e6e7a0a7b524972a303ef90ef15", "parents": [ "457dfd12f281dbe9b1af8d1a7429f2977e234a6f" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Jun 02 22:42:06 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Jun 02 17:52:38 2025 -0700" }, "message": "loop: automatic host/container ssh port tunneling\n\nFix for #47\n\nAdd comprehensive port event monitoring and automatic SSH tunnel management\nsystem that enables real-time port forwarding for container services.\n\nContainer processes need automatic port forwarding when services start or stop\nlistening on ports during agent execution. Previously, users had to manually\ncreate SSH tunnels using commands like \u0027ssh -L8000:localhost:8888 container\u0027,\nwhich required manual intervention and knowledge of when ports become available.\n\n- Extended PortMonitor with thread-safe event storage using circular buffer\n- Added PortEvent struct with type (opened/closed), port info, and timestamps\n- Maintained backward compatibility with existing logging functionality\n- Events stored in 100-item circular buffer with efficient timestamp filtering\n\n- Added /port-events endpoint in loophttp.go for container-to-host communication\n- Supports optional \u0027since\u0027 query parameter for incremental event fetching\n- Returns JSON array of recent port events with proper error handling\n- Integrated with existing Agent interface via GetPortMonitor() method\n\n- Created TunnelManager component for host-side tunnel orchestration\n- Polls container /port-events endpoint every 10 seconds for new events\n- Automatically creates SSH tunnels when ports open using same port numbers\n- Properly cleans up tunnels when ports close or context cancels\n- Skips common system ports (SSH, HTTP, SMTP) to avoid conflicts\n\n- Integrated TunnelManager into dockerimg.LaunchContainer() workflow\n- Starts tunnel manager alongside existing container management goroutines\n- Only activates when SSH is available and configured properly\n- Uses existing SSH infrastructure and container naming conventions\n\n- Container PortMonitor detects port changes via ss -lntu command\n- Events stored with RFC3339 timestamps for precise filtering\n- Thread-safe access patterns with dedicated mutex protection\n- Circular buffer prevents unbounded memory growth\n\n- RESTful GET /port-events endpoint with time-based filtering\n- Proper JSON encoding/decoding with error handling\n- Integration with existing HTTP server infrastructure\n- Non-blocking polling pattern with configurable intervals\n\n- Uses existing SSH theater configuration and host keys\n- Creates tunnels with format: ssh -L hostPort:localhost:containerPort container\n- Background monitoring of tunnel processes with automatic cleanup\n- Proper context cancellation and resource management\n\n- Added comprehensive port event storage and filtering tests\n- HTTP endpoint testing with mock agents and proper status codes\n- Verified thread-safe access patterns and circular buffer behavior\n- All existing loop package tests continue to pass\n\n- Confirmed HTTP endpoint returns proper JSON responses\n- Validated tunnel manager integrates with container launch process\n- Verified SSH tunnel creation follows existing authentication patterns\n- Build verification confirms no regressions in existing functionality\n\n- Automatic port forwarding eliminates manual SSH tunnel management\n- Real-time port detection provides immediate service accessibility\n- Transparent integration with existing Sketch container workflow\n- Maintains all existing SSH functionality and manual override options\n\n- Clean separation between container monitoring and host tunnel management\n- Extensible event-based architecture for future port-related features\n- Minimal performance impact with efficient polling and filtering\n- Robust error handling and graceful degradation when SSH unavailable\n\nThis enhancement provides seamless port forwarding automation while maintaining\nthe reliability and security of the existing SSH infrastructure, significantly\nimproving the developer experience when working with containerized services.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6bc363ed64835e5dk\n" }, { "commit": "b8a8f35348326b8daf69e31efc77d37fac4bd276", "tree": "fa7a59c5951dbd3ecb5e6bf840ac7b45167a9e74", "parents": [ "b5739403b1b6ec6fac909b258bd47ce5a338940e" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 07:39:37 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 17:01:19 2025 -0700" }, "message": "loop: implement comprehensive conversation compaction system\n\n\"comprehensive\" is over-stating it. Currently, users get\nthe dreaded:\n\n\terror: failed to continue conversation: status 400 Bad Request:\n\t{\"type\":\"error\",\"error\":{\"type\":\"invalid_request_error\",\"message\":\"input\n\tlength and max_tokens exceed context limit: 197257 + 8192 \u003e 200000,\n\tdecrease input length or max_tokens and try again\"}}\n\nThat\u0027s... annoying. Instead, let\u0027s compact automatically. I was going to\nstart with adding a /compact command or button, but it turns out that\nteasing that through the system is annoying, because the agent state\nmachine is intended to be somewhat single-threaded, and what do you do\nwhen a /compact comes in while other things are going on. It\u0027s possible,\nbut it was genuinely easier to prompt my way into doing it\nautomatically.\n\nI originally set the threshold to 75%, but given that 8192/200000 is 4%,\nI just changed it to 94%.\n\nWe\u0027ll see how well it works!\n\n~~~~\n\nImplement automatic conversation compaction to manage token limits and prevent\ncontext overflow, with enhanced UX feedback and accurate token tracking.\n\nProblem Analysis:\nLarge conversations could exceed model context limits, causing failures\nwhen total tokens approached or exceeded the maximum context window.\nWithout automatic management, users would experience unexpected errors\nand conversation interruptions in long sessions.\n\nImplementation:\n\n1. Automatic Compaction Infrastructure:\n - Added ShouldCompact() method to detect when compaction is needed\n - Configurable token thresholds for different compaction triggers\n - Integration with existing loop state machine for seamless operation\n\n2. Accurate Token Counting:\n - Enhanced context size estimation using actual token usage from LLM responses\n - Track real token consumption rather than relying on estimates\n - Account for tool calls, system prompts, and conversation history\n\n3. Compaction Logic and Timing:\n - Triggered at 75% of context limit (configurable threshold)\n - Preserves recent conversation context while compacting older messages\n - Maintains conversation continuity and coherence\n\n4. Enhanced User Experience:\n - Visual indicators in webui when compaction occurs\n - Token count display showing current usage vs limits\n - Clear messaging about compaction status and reasoning\n - Timeline updates to reflect compacted conversation state\n\n5. UI Component Updates:\n - sketch-timeline.ts: Added compaction status display\n - sketch-timeline-message.ts: Enhanced message rendering for compacted state\n - sketch-app-shell.ts: Token count integration and status updates\n\nTechnical Details:\n- Thread-safe implementation with proper mutex usage\n- Preserves conversation metadata and essential context\n- Configurable compaction strategies for different use cases\n- Comprehensive error handling and fallback behavior\n- Integration with existing LLM provider implementations (Claude, OpenAI, Gemini)\n\nTesting:\n- Added unit tests for ShouldCompact logic with various scenarios\n- Verified compaction triggers at correct token thresholds\n- Confirmed UI updates reflect compaction status accurately\n- All existing tests continue to pass without regression\n\nBenefits:\n- Prevents context overflow errors in long conversations\n- Maintains conversation quality while managing resource limits\n- Provides clear user feedback about system behavior\n- Enables unlimited conversation length with automatic management\n- Improves overall system reliability and user experience\n\nThis system ensures sketch can handle conversations of any length while\nmaintaining performance and providing transparent feedback to users about\ntoken usage and compaction activities.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s28a53f4e442aa169k\n" }, { "commit": "b5739403b1b6ec6fac909b258bd47ce5a338940e", "tree": "0dcc652908a7978912d1e5c20d6b95431da2a6e0", "parents": [ "364f741483c1bd2c18cb3ff2ad255c9042c5362d" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 07:04:34 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 07:04:34 2025 -0700" }, "message": "Add end session user feedback survey with skaband client coordination\n\nImplement comprehensive end session feedback system with thumbs up/down rating,\noptional text comments, and proper coordination with skaband clients to ensure\nfeedback delivery before process termination.\n\n- Add EndFeedback struct with Happy boolean and Comment string fields\n- Extend Agent struct with endFeedback field and GetEndFeedback/SetEndFeedback methods\n- Update State struct to include End field for exposing feedback in API\n- Modify /end handler to accept survey data (happy, comment fields)\n- Add skaband client coordination with WaitGroup and wait_for_end parameter\n\n- Replace simple window.confirm with custom modal dialog\n- Create comprehensive survey UI with thumbs up/down buttons\n- Add optional textarea for detailed feedback\n- Implement proper button state management and validation\n- Maintain existing redirect behavior after successful end request\n\n- Track clients connecting with /stream?wait_for_end\u003dtrue parameter\n- Use WaitGroup to coordinate end session messaging\n- Proper WaitGroup management with conditional defer to prevent double-decrement\n- Smart timeout handling: measure elapsed time and ensure minimum 100ms response delay\n- Wait up to 2 seconds for waiting clients, timeout gracefully if needed\n\n**Survey Dialog Features:**\n- Modal overlay with centered positioning and backdrop click handling\n- Visual feedback for thumbs up/down selection with color coding\n- Expandable textarea for optional detailed comments\n- Cancel option to abort end session process\n- Proper event handling and cleanup\n\n**State Management:**\n- EndFeedback data flows through Agent -\u003e State -\u003e SSE streams\n- Clients waiting for end automatically receive feedback via state updates\n- WaitGroup ensures coordinated shutdown after notification delivery\n- Exactly one Done() call per Add() to prevent race conditions\n\n**API Contract:**\nPOST /end now accepts:\n{\n \"reason\": \"user requested end of session\",\n \"happy\": true, // optional boolean\n \"comment\": \"Great experience!\" // optional string\n}\n\n**Error Handling:**\n- Survey submission proceeds even if rating not selected (null happy value)\n- Empty comments are handled gracefully\n- Network failures fall back to existing error reporting\n- Proper resource cleanup and thread-safe operations\n\n**Testing:**\n- Added comprehensive unit tests for EndFeedback functionality\n- Updated mockAgent to implement new CodingAgent interface methods\n- All existing tests continue to pass\n- Manual API testing confirmed survey data flows correctly\n\nThis enhances user experience by collecting valuable feedback while maintaining\nrobust session termination and proper coordination with external monitoring\nsystems via the skaband protocol.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s16561e134e8e81aak\n" }, { "commit": "e84d5c7972cb24586842473e62668ca9c689d6d2", "tree": "8ac4a2c314ccf13b0653e34aa5b9376a4f6abdd9", "parents": [ "7871e2fd09acf3790cc292d955cdd8fee86f2fdb" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 30 09:32:55 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 30 09:33:30 2025 -0700" }, "message": "loop/server: enable SSH reverse port forwarding\n\n~~~~\n\nvibe-coded but tested manually with\n\n\t$python3 -mhttp.server 8000 \u0026\n\t[1] 25986\n\tServing HTTP on :: port 8000 (http://[::]:8000/) ...\n\n\t$ssh -R 8000:localhost:8000 sketch-241k-9zzx-gcbc-k4fs curl --silent http://localhost:8000/ | head -n 3\n\t::1 - - [30/May/2025 09:32:44] \"GET / HTTP/1.1\" 200 -\n\t\u003c!DOCTYPE HTML\u003e\n\t\u003chtml lang\u003d\"en\"\u003e\n\t\u003chead\u003e\n\n~~~~\n\nImplement ReversePortForwardingCallback in SSH server configuration to\nallow reverse port forwarding connections from clients.\n\nChanges include:\n\n1. SSH Server Configuration Enhancement:\n - Added ReversePortForwardingCallback to ssh.Server struct\n - Callback allows all reverse port forwarding requests (returns true)\n - Consistent with existing LocalPortForwardingCallback behavior\n - Includes debug logging for reverse forwarding requests\n\nTechnical Implementation:\n\nThe SSH server already had the necessary infrastructure for port forwarding:\n- ForwardedTCPHandler for handling forwarding requests\n- RequestHandlers for \u0027tcpip-forward\u0027 and \u0027cancel-tcpip-forward\u0027\n- LocalPortForwardingCallback for client-to-server forwarding\n\nThis change adds the missing ReversePortForwardingCallback that enables\nserver-to-client port forwarding (reverse tunneling). The callback follows\nthe same permissive pattern as the local forwarding callback, allowing\nall reverse forwarding requests while logging them for debugging.\n\nWithout this callback, the SSH library defaults to denying all reverse\nport forwarding requests, preventing clients from establishing reverse\ntunnels through the SSH connection.\n\nBenefits:\n- Enables full bidirectional port forwarding capabilities\n- Maintains consistent logging and debugging for both forwarding types\n- Follows established patterns in the existing codebase\n- No breaking changes to existing SSH functionality\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9bc98f6471e2ec4dk\n" }, { "commit": "112b92376c97b8da64cc1c954896957cfc479f3d", "tree": "1497eb8b2927f1334d712aabb19625db11a148bf", "parents": [ "d203b7de7d49cc5da03440d5a00b2efd0f8bb2c8" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 11:26:33 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 12:50:41 2025 -0700" }, "message": "loop: add todo checklist\n\nThis should improve Sketch\u0027s executive function and user communication." }, { "commit": "c5848f3cb9753e94feddc04b8910b4ff113a458c", "tree": "8b3eb330c645e4f020a9ef7f10e78491474ec8ab", "parents": [ "6e4636006501be61b940388381cd75a8ad00e6b5" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 18:50:58 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 12:01:29 2025 -0700" }, "message": "loop: use repo root instead of working directory for git diff support\n\nChange all git HTTP endpoints to use RepoRoot() instead of WorkingDir()\nto resolve failures when sketch is started from different directories than\nwhere the Docker container was created.\n\nThe issue occurred because:\n- WorkingDir() returns current working directory (could be /app/subdir)\n- Git tools need repository root directory (/app) for relative file paths\n- When Docker sets working directory to subdirectory, git operations fail\n\nChanges include:\n\n1. Added RepoRoot() method to Agent and CodingAgent interface:\n - Returns a.repoRoot (the git repository root directory)\n - Complements existing WorkingDir() method\n\n2. Updated all git HTTP endpoints to use RepoRoot():\n - handleGitCat: fixes GitCat file access from subdirectories\n - handleGitRawDiff: ensures diff operations work correctly\n - handleGitShow: ensures show operations work correctly\n - handleGitSave: ensures file saving works correctly\n - handleGitRecentLog: ensures log operations work correctly\n\nThe fix ensures git ls-files and other git operations work reliably\nregardless of which directory sketch was started from, resolving the\nreported GitCat directory issue where files couldn\u0027t be found when\nstarting sketch from different locations.\n\nTesting confirmed GitCat now works correctly when repository root is\nused instead of working directory, and git operations succeed from\nany starting directory.\n\nFixes #119\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sb73dac311df0c78dk" }, { "commit": "9bca61ef623ea8cd72a906752be73af572af02a9", "tree": "4d6488b6e6e8543c3f0b5e1682d11e0927c27d1b", "parents": [ "75bd37d2a3067b6f431d56e891064b73dc2def2c" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 12:40:06 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 12:44:59 2025 -0700" }, "message": "feat: enhance UI with commit pulsing animation and improved copy icon\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s80fdaac660ea645ek\n" }, { "commit": "75bd37d2a3067b6f431d56e891064b73dc2def2c", "tree": "69580944c24bd99c5286ae0611cf2998a3b85035", "parents": [ "8c3b53a97ae2a204842c0c86ca859947ce20b1dd" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 18:49:14 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 12:40:44 2025 -0700" }, "message": "loop: auto-commit changes when saving in diff view\n\nWhat I\u0027ve done is create a git commit whenever the user edits things,\nand amend if possible. The existing detection logic pushes the commits\nto the host, BUT, I had to do some plumbing to make that happen. The\nagent state machine would be out of sorts if I did this (since we\u0027re\ndoing things outside of the loop), but, I didn\u0027t tell it, so... it\u0027s ok!\n\nIf the user has the agent running when editing, everyone can get\nconfused. There\u0027s no atomicity for the git operations, etc.\nI suspect in practice this will all be as fine as everything else is.\n\nI\u0027m not running the autoformatters. That\u0027s a weird editing experience.\n\nThe alternative was to do what the diff comments does, and let the agent\ndeal with the changes by sending it a message. I chose not to do that:\nfirst of all, I want the push to happen fast, since I don\u0027t like losing\nuser data. Second, the latency on an operation is distracting sometimes,\nand sometimes what I do next is just cherrypick my changes over, and I\u0027m\nnot interested in the pedantry of the agent and the code formatters and\nso forth. If they were faster, maybe.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sec50af415124810bk\n" }, { "commit": "14fe75d6ece5116b3887b6fc027e564c4de518e6", "tree": "5a3e72a02da8d06818fcd3531147087f8741f81b", "parents": [ "f28729932fdf9ecc67d3fabbfca297178a323a14" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 17:39:38 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 11:28:39 2025 -0700" }, "message": "webui: Remove restart conversation button and modal component\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s0da284ec42d4da59k\n" }, { "commit": "bc8c8dc5bc7abca6fa523c5ca45c9fd2c09384c2", "tree": "d59b7b3ef6965949376481cfa6108fee7312f6fd", "parents": [ "1a648f34a53216a9ed88792d583fad50246f3d4b" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 13:19:13 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 13:40:03 2025 -0700" }, "message": "sketch main: migrating things from /init into cmdline flags\n\nAs much as possible, I want NewAgent() to take as many arguments as it\ncan, and try to avoid doing different things with Init().\n\nYou can\u0027t quite do everything because the port that Docker has\nopen for forwarding starts as \"0\" and we call \"docker port\" to find it,\nbut besides that, almost everything is knowable up front.\n" }, { "commit": "8d8b7ac6b00b1d0b3a2c6d3a4079e2c2fb76cc8a", "tree": "eb46d3770aa48f0a809594eef93e0a053075b0ee", "parents": [ "35c72bcbf69cce28395bc34236df8ef9d615c0da" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 09:57:23 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 11:07:52 2025 -0700" }, "message": "Add buildinfo to /debug on Sketch http\n" }, { "commit": "9d7f0ccec1317b68c754b6b154f7c362395028c2", "tree": "baa47109cc854d5abc4326669e05e9ceb49a016c", "parents": [ "da514fccdf1135cfa52a869b892855aac117ee05" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Tue May 20 11:43:26 2025 +0100" }, "committer": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Tue May 20 10:51:10 2025 +0000" }, "message": "loop/server: fix race condition in SSE stream iterators\n\nFix panic \u0027send on closed channel\u0027 when a client disconnects from SSE stream.\nThe issue occurred because iterators were created at the beginning of\nhandleSSEStream with defer iterator.Close(), but used in separate goroutines.\nWhen the client disconnected, the iterators were closed while goroutines\nwere still using them.\n\nMove iterator creation into the goroutines that use them to ensure their\nlifecycle matches the goroutines\u0027 lifecycle, preventing the race condition.\n\nFixes #108\n" }, { "commit": "397871d299216f63dc38a9cc6f2ca386d0f4bd75", "tree": "ea1cec9278f37739f972a390b0ec564a1fe9219f", "parents": [ "a8322202f36c35f677b8834cdce8f21ab3d8c6dc" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Mon May 19 15:02:45 2025 +0100" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon May 19 14:03:43 2025 +0000" }, "message": "webui: Add End button to shutdown container\n\nThis change adds a new button to the sketch-app-shell UI alongside the\nexisting Restart and Stop buttons. When clicked, the End button shows a\nconfirmation dialog and then sends a request to a new /end endpoint.\n\nThe /end endpoint gracefully shuts down the inner sketch container.\n\nFixes #88\n" }, { "commit": "272a90ee1a74bda5618d4866e03f4b7067947784", "tree": "9baf4f84ce80b1c7073a95b6959f6dd11ab3b48b", "parents": [ "d3ac112a45111abf0e57c327d55e2cc66a136abb" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 16 14:49:51 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 16 14:51:40 2025 -0700" }, "message": "Add Monaco diff-view, the saga ...\n\nI set out to use Monaco to support the diff view. diff2html is lovely,\nbut there were a ton of usability improvements I wanted to make (line\nnumbers not making things double spaced, choosing which diff, editing\nthe right-hand side), and it seemed a dead end. Furthermore, Phabricator\nand Gerrit\u0027s experience is that diffs should be shown file by file,\nbecause you\u0027ll inevitably see a diff with a file that\u0027s too large, and\nthe GitHub PR view often breaks on big changes... so I wanted to show\nfiles diff-by-diff, with \"infinite\" context when unchanged sections are\nexpanded. So...\n\nUltimately, all of this was sketch-coded over maybe 30 Sketch sessions.\nI threw away a lot of branches. My git reflog is a superfund site.\n\nPrompting whole-hog didn\u0027t work. Or, rather, it made significant\nprogress, but something very serious wouldn\u0027t work, and I couldn\u0027t\nfigure out what, and nor could Sketch.\n\nInstead, I started by adding a new webcomponent that was just a\nplaceholder. Then, using https://rodydavis.com/posts/lit-monaco-editor,\nI nudged Sketch into adding Monaco to it. Sketch pulled out:\n\n You\u0027re right, I should properly read the blog post before implementing the\n solution. Let me check the referenced blog post.\n\nI worked heavily in the demo environment at first, but here I ran into\nthe issue that we have two different esbuild systems: one is vite and\none is esbuild.go, and they\u0027re configured differently enough.\n\nMonaco is unusable and confusingly so when its CSS isn\u0027t loaded. The right\nway to load it, I\u0027ve found, is via\n\n @import url(\u0027./static/monaco/min/vs/editor/editor.main.css\u0027);\n\nI spent more time than I care to admit noticing that originally\nthis wasn\u0027t relative, and when we use a skaband setting, the\npaths need to be relative-aware.\n\nThe paths to the various workers need to be similarly correctly placed.\n\nGetting Sketch to build demo data but not put testing code into production\ncode was tricky. (I threw away a lot of efforts and factories and singletons...)\n\nWhen I set out to do the git commit selection, I wanted to do a bunch of\nbackend /git/* handlers. These were easy enough to code in sketch. I had\nto convince Sketch to put them in git_tools.go and not in the agent.\nIt doesn\u0027t really matter: these functions to parse git are pretty stateless,\nbut it\u0027s less work to have them separate. Sketch was mediocre at writing\ntests for them. Did you know that our container has an older version\nof git that doesn\u0027t have the same options to decorate ref names? Yeah, nor did\nI.\n\nHandling unstaged changes was fun. git diff --raw shows unstaged files\nas having identity 0000. Ideally we\u0027d be using jj and there\u0027d be\na synthetic commit, but instead uncommitted-possible files are read\nby content.\n\nA real big challenge was getting the Monaco view to use the right vertical and\nhorizontal space. I did this many, many times. I don\u0027t claim to understand flex\nand the virtual dom, and :host, and all the interactions. It would fix one\nthing and break another. The chat window would shrink. The terminal would\nshrink.\n\nScreenshot support was excellent. I eventually added paste support just so\nthat I could expedite my workflow, and Sketch coded that easily on the first\npass with minor feedback.\n\nI learned the hard way that Safari\u0027s support for WebComponents/shadow\ndom in its web inspector is rough. See https://fediverse.zachleat.com/@zachleat/114518629612122858\n\nI also learned the hard way that Chrome doesn\u0027t use fonts loaded in CSS\nin a shadow dom. That\u0027s why the codicon font had to be in the global\nstyle sheet.\n\nKudos to John Reese who kindly allowed me, a long time ago, to adapt a\nshell script he had at work to look over diffs into https://github.com/philz/git-vimdiff.\nThat\u0027s the inspiration for having the \"new code\" be editable when you\u0027re\nreviewing it; why shouldn\u0027t it be!?!\n\nThere are a handful of follow up tasks:\n\n* We lose state when we switch to the Chat view and back.\n* Need URL-based support for where we are.\n* Maybe need shortcut keys to move between diffs and changes.\n* Maybe need caching or look-ahead for downloading the next or previous\n file.\n* We spend too much vertical real estate on all the diff selections;\n could we scroll it out of the way, collapse it, tighten it, etc.\n* The workers sometimes throw errors into the console. I think they\u0027re\n harmless and merely need to be caught and suppressed.\n* Needing to commit changes when things are saved is weird. Should we\n commit automatically? Amend the previous commit? Have a button for\n that? Show the git dirty state?\n* Our JS bundle is big. We could maybe delay loading the monaco bundle\n to help.\n\nThanks for coming to my TED talk.\n" }, { "commit": "d3ac112a45111abf0e57c327d55e2cc66a136abb", "tree": "7d2cab01515d7c482be1a39cc0492e9d9ac657e7", "parents": [ "8bdf627180b64b0dc09018bf512f6ebf192ab674" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 02:54:18 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 16 14:51:40 2025 -0700" }, "message": "git_tools: Implement git diff and show API\n\nAdded git_tools package providing structured access to git diff and show commands. Exposed these methods via HTTP endpoints in loophttp.\n\nThis is a stepping stone to a better diff view.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se75f0a1b2c3d4e5k\n" }, { "commit": "f84e88cd1c2455610de5cb73b8766972216bc14c", "tree": "c38c21c73bd2412d1d21f0846c8ba1ba95e276e9", "parents": [ "da796543d8e7087e428b1e55b8f70bc8edf206ae" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 23:19:01 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed May 14 23:35:47 2025 +0000" }, "message": "webui: Add file paste upload support\n\n- Add new /upload endpoint in loophttp.go to save pasted files to /tmp\n- Make the implementation generic to handle any file type, not just images\n- Implement paste event handling in sketch-chat-input.ts to detect files\n- Add logic to upload files and insert file paths in chat input\n- Improve random filename generation with better comments\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sff2e40b9b3e4c05ak\n\nwebui: Improve file upload UI experience\n- Use relative path for upload endpoint\n- Add loading indicator during file upload\n- Show error message if upload fails\n- Improve cursor position handling\n" }, { "commit": "7013e9ee282ef58104f91d64d85d4aec62f9c022", "tree": "baa2d5484c5a5e7add5f9caef2833a3da4abbe29", "parents": [ "00442413a2d0e6d6978982f9eeec0268e533ba56" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed May 14 02:03:58 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed May 14 11:15:45 2025 -0700" }, "message": "ssh: use local CA, add mutual container/host auth\n\nSee loop/server/local_ssh.md for a detailed description of how sketch uses\nnow uses a local CA to sign each container certificate instead of adding\na new entry to known_hosts for each container.\n\nThis also adds another layer of security by having the container\u0027s ssh\nserver verify that incoming ssh connections have valid host certificates,\nwhereas prior to this change the authentication was only one-way (verifying\nthat the sketch container you think you\u0027re ssh\u0027ing into really is the one\nyou think you\u0027re ssh\u0027ing into).\n\nThis is somewhat inspired by https://github.com/FiloSottile/mkcert - which\nplays a similar role as ssh_theater.go local for ssh connections, but mkcert\nuses a local CA to address local development use cases for TLS/https rather\nthan for ssh.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc7b3928295277d5dk\n" }, { "commit": "49edc92f95cab092a7ee62c350c6e69275b50cc8", "tree": "946b8359f6187ed630c4d08fa05e91fe1bc5a48d", "parents": [ "52eeb15cf2f70caaadc4e0722a3b687ad9a91981" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 09:45:45 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed May 14 16:46:49 2025 +0000" }, "message": "agent: move \"sketch-base\" into git\n\nThe agent\u0027s notion of \"initial commit\" is kind of special, in that it\nis used as the \"base\" for a bunch of git operations. It\u0027s hard for\nthe user to change (we only provide a workflow via restart), yet\nsometimes you want to do just that.\n\nSo, instead we put it as data inside of it, named as a tag sketch-base.\nIt\u0027s abusing tags, but branches are no better.\n" }, { "commit": "eab12def4e6aacfd50bf8a460c714d20300706b7", "tree": "4db97e5dfb30f73fc47435f3651eee55876a6b0b", "parents": [ "8a1b89a210d3c48c6f02f0c8cf1feebb03cdffd5" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 02:35:53 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 02:36:01 2025 +0000" }, "message": "loop: Add StateTransitionIterator and stream state updates\n\nImplement CodingAgent.NewStateTransitionIterator to observe state transitions.\nUpdate the /stream endpoint to send state updates when transitions occur.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s4b4f9a0689c94c54k\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s4b4f9a0689c94c54k\n" }, { "commit": "d03318d51ca1e707698f1aab8d18ed83fc159cbe", "tree": "ae2ee324779b56baa1bc0670aa6852366a5ad9ab", "parents": [ "8236cbc9eadb1bf775bbfa24ccf04be2c69faaaf" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 08 13:09:12 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 09 15:06:33 2025 -0700" }, "message": "sketch: Introduce versions for sketch state\n" }, { "commit": "33d282f80db786cc60ba521a38ed5166f23239ed", "tree": "9ed1f15c6d3081d5bef7d16b9d72e78a2c7780cf", "parents": [ "a9d87aa69cfefdc91ec7aaa6bc42907749748e76" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 03 04:01:54 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 06 10:23:39 2025 -0700" }, "message": "Add browse tool support.\n\nI reviewed some MCPs (using OpenAI\u0027s deep research to help), and it\nhelped me choose chromedp as the relevant library and helped me come up\nwith an interface. This commit adds chrome to the Docker image which is\nkind of big. (I\u0027ve noticed that it\u0027s smaller on Ubuntu, where it doesn\u0027t\npull in X11.) go-playwright was a library contender as well.\n\nImplement browser automation tooling using chromedp\n\nThis implementation adds browser automation capabilities to the system via the chromedp library,\nenabling Claude to interact with web content effectively.\n\nKey features include:\n\n1. Core browser automation functionality:\n - Created new browsertools package in claudetool/browser\n - Implemented tools for navigating, clicking, typing, waiting for elements,\n getting text, evaluating JavaScript, taking screenshots, and scrolling\n - Added lazy browser initialization that defers until first use\n - Integrated with the agent to expose these tools to Claude\n\n2. Screenshot handling and display:\n - Implemented screenshot storage with UUID-based IDs in /tmp/sketch-screenshots\n - Added endpoint to serve screenshots via /screenshot/{id}\n - Created dedicated UI component for displaying screenshots\n - Ensured proper responsive design with loading states and error handling\n - Fixed URL paths for proper rehomed URL support\n - Modified tool calls component to auto-expand screenshot results\n\n3. Error handling and reliability:\n - Added graceful error handling for browser initialization failures\n - Implemented proper cleanup of browser resources\n\nThe browser automation tools provide a powerful way for Claude to interact with web content,\nmaking it possible to scrape data, test web applications, and automate web-based tasks.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "bdfb126de211f5f2fabb39498de00acde5faf58f", "tree": "665cedc4a7148c2ba86e915be96e1cc1d8a01450", "parents": [ "cf291fafd7142308f27ec5805251f2e2bdeded6b" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sat May 03 20:15:41 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sat May 03 20:15:41 2025 -0700" }, "message": "sshserver: s/log/slog\n" }, { "commit": "cf291fafd7142308f27ec5805251f2e2bdeded6b", "tree": "f18805cbe3583f62e423c5a0b17a3be3a757df54", "parents": [ "d0a3cd609ca36baebd066d6fd83c153293c459eb" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sat May 03 17:55:48 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sat May 03 17:59:43 2025 -0700" }, "message": "sshserver: add scp support\n\nfixes https://github.com/boldsoftware/sketch/issues/53\n" }, { "commit": "25f6ff1b44e739e64d8b86cc4ea11bf8c631265c", "tree": "2255018ca38ba211e57eb9e76e87b82d61a0c826", "parents": [ "d01d1341abf9b2e62f0b4cc821277dc4c40c6225" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 02 04:24:10 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 02 14:25:31 2025 -0700" }, "message": "Implement Server-Sent Events (SSE) for Real-time Agent Communication\n\n- Add server-side SSE endpoint (/stream?from\u003dN) for streaming state updates and messages\n- Replace polling with SSE in frontend for real-time updates with significant performance improvements\n- Implement efficient connection handling with backoff strategy for reconnections\n- Add visual network status indicator in UI to show connection state\n- Use non-blocking goroutine with channel pattern to handle SSE message delivery\n- Ensure proper message sequencing and state synchronization between client and server\n- Fix test suite to accommodate the new streaming architecture\n- Update mocks to use conversation.Budget instead of ant.Budget\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "4f84ab729ddbf428b0e891940f08f70b4edee05c", "tree": "f2e52e4a01c188ada1f5acf8b2a013029b999495", "parents": [ "44f9b4cec11e269a52fbfc099989ab425b8e125f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 22 16:40:54 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 02 12:57:44 2025 -0700" }, "message": "all: support openai-compatible models\n\nThe support is rather minimal at this point:\nOnly hard-coded models, only -unsafe, only -skabandaddr\u003d\"\".\n\nThe \"shared\" LLM package is strongly Claude-flavored.\n\nWe can fix all of this and more over time, if we are inspired to.\n(Maybe we\u0027ll switch to https://github.com/maruel/genai?)\n\nThe goal for now is to get the rough structure in place.\nI\u0027ve rebased and rebuilt this more times than I care to remember.\n" }, { "commit": "b7c5875548da5057eac0405bf4e0ae8bbc43667c", "tree": "a0b9eb0f68de826d7b51a5e8f5eaa7976fb48de3", "parents": [ "5228b5850d8453c225e5611918fcd00b747c65cc" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 01 10:10:17 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 01 12:13:31 2025 -0700" }, "message": "agent plumbing: convert outbox to subscribers and an iterator\n\nWaitForMessage() could only work for one thread, because it was using a\nsingular channel for outboxes. This was fine when we only had one user,\nbut WaitForMessageCount() was kinda similar, and had its own thing, and\nI want to rework how polling works and need another user.\n\nAnyway, this one is hand-coded, because Sketch really struggled\nwith getting the iterator convincingly safe. In a follow-up commit,\nI\u0027ll try to get Sketch to write some tests.\n" }, { "commit": "d9d4581952edb59469a8402910783570be852fb8", "tree": "7b06cf74a10d3d812152c21fc72ced395a3e5024", "parents": [ "48c84c9a4df29b1e51c7fdca81053dd9cd0c2446" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 30 16:53:41 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 30 18:00:55 2025 -0700" }, "message": "webui: add current Agent state to call-status tooltip\n" }, { "commit": "3e2111b1e18a39a4eba4a0afa0327b6a67acf802", "tree": "6add6c3a0583877f173f1f2baf6ceeb8c39b3f69", "parents": [ "e54b00af5cf711c2171a26ed0f29a8ce83d50fb4" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 17:53:28 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 17:13:18 2025 -0700" }, "message": "all: support popping a browser from termui\n\n- Add \u0027browser\u0027, \u0027open\u0027, and \u0027b\u0027 command aliases to termui\n- Open the current conversation URL in default browser\n- Add help documentation for the new command\n\nAdd browser launch endpoint to Git server for Docker support.\n\nWe\u0027ll probably want to set up a proper mux for the no-longer-just-git\nserver pretty soon.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "2c4db099e6763cf18f0c57876298b9f5c70ed925", "tree": "ae4408992dd74fa0b5392523d2e6713e388326e7", "parents": [ "b74b526b579320c08ad85cc8b4bdd719c9d6d7c8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Apr 28 16:57:50 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Apr 30 19:59:45 2025 +0000" }, "message": "Restart conversation support.\n\nThe idea here is to let the user restart the conversation, possibly with\na better prompt. This is a common manual workflow, and I\u0027d like to make\nit easier.\n\nI hand wrote the agent.go stuff, but Sketch wrote the rest.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "edc88dcc56ebdb1a2d9887728d0b5133b6131fbb", "tree": "a5b44389b3ac59022c3006d8347c36626cc7019e", "parents": [ "885a16a8e37fe82dac66a7db729be202f6ce4bea" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 30 02:55:01 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 30 09:41:22 2025 -0700" }, "message": "Rename cancelInnerLoop to cancelTurn to match refactored naming\n\n- Updated all references to cancelInnerLoop and CancelInnerLoop\n- Changed method and variable names to match the new processTurn naming\n- Updated references in related files: server/loophttp.go and termui/termui.go\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "47b193683089f6ca98877af497f7800d4a26f4a9", "tree": "43b4b54c7b32650ff7ea9256c0aca7a793d68246", "parents": [ "0137a7f2a025a2ac08e536b207ded2d58e928e32" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 01:34:14 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 19:06:19 2025 -0700" }, "message": "Display branch name alongside title in termui and webui\n\nThis change adds the branch name alongside the title in both the terminal UI and the web UI, following up on commit a9b3822fd2cfdc035e7daa8b59496f9838a4b2c6 which added the branch_name parameter to the title tool.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "c72fff52ad6c5436fa8469f049fa232f2f6be5ed", "tree": "2f42838fc0af2f4954a64b8edcc4c8e09ec08af4", "parents": [ "d3906e2e0a18197148089f384b188135b07559be" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Apr 29 20:17:54 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Apr 29 16:24:19 2025 -0700" }, "message": "Add SSH connection information to web UI\n\nThis commit implements:\n1. Added SSH information display to the web UI info button\n2. Shows SSH connection information only when running inside Docker container\n3. Constructs the full SSH hostname as \"sketch-[session_id]\"\n4. Added copy-to-clipboard buttons for SSH commands\n5. Styles the VSCode URL as a proper clickable link\n6. Shows a warning message when SSH is not available\n7. Modified container naming to use only SessionID instead of imgName+SessionID\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "22bd8eb83b7c779d9d2396c75cc344124e150aee", "tree": "81fc507d28db05b011d463745e125e6126260fb0", "parents": [ "0dcebe1bf2ec5d6f85aa71fb46367e2adf8f4947" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Apr 28 10:36:37 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Apr 28 10:36:37 2025 -0700" }, "message": "sshserver: fix env for ssh pty sessions\n" }, { "commit": "99a9a0285fe449f047fbc93766f37e6ea7af21df", "tree": "ba255c9b05449c09d137cd6cf81edf58469542a1", "parents": [ "f98d7305e73ecec9a78f94f527693d5adfb89159" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Sun Apr 27 15:15:25 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Sun Apr 27 20:00:22 2025 -0700" }, "message": "Implement tracking of outstanding LLM and Tool calls\n\nThis commit implements a listener pattern between ant.convo and the Agent for tracking outstanding calls.\n\n* Added fields to the Agent struct to track outstanding LLM calls and Tool calls\n* Implemented the listener methods to properly track and update these fields\n* Added methods to retrieve the counts and names\n* Updated the State struct in loophttp.go to expose this information\n* Added a unit test to verify the tracking functionality\n* Created UI components with lightbulb and wrench icons to display call status\n* Added numerical indicators that always show when there are active calls\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "01ed5bee0c121852d0e2ab3836a08acab68397e4", "tree": "e2b473d510d146f9474b2eedb5526b8a4eead1f2", "parents": [ "7d5a6301a256823ab4b26a68dbd166f0d0737409" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Apr 24 22:46:53 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Apr 24 22:46:53 2025 -0700" }, "message": "sshserver: enable ssh port forwarding\n" }, { "commit": "1d06132f5a79950ab8a64eb72219363fb63ca92f", "tree": "46ed36f431ee3aef7caacbae7b1ebe7560d3f544", "parents": [ "6a50b1811c7fbf8be7e5ad4cc77a576e2d612a66" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Apr 24 09:52:56 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Apr 24 11:11:44 2025 -0700" }, "message": "sshserver: add direct-tcpip channel, allow non-pty\n\nThis change enables VS Code to connect to local sketch containers\nover SSH.\n\nThe VSC docs describe how to use this feature:\nhttps://code.visualstudio.com/docs/remote/ssh\n" }, { "commit": "2032b1c1971ceb85ca14b20273a3783729fba3e3", "tree": "0486e9222643ffcbbd34286148f4a7913a169668", "parents": [ "4f50a68ac73677c0022b2b3da8b4667cee01c11b" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:40:42 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:40:42 2025 -0700" }, "message": "Move webui from /loop/webui to /webui\n\nThanks, perl (and git mv):\n\n\tperl -pi -e s,loop/webui,webui,g $(git grep -l loop/webui)\n" }, { "commit": "18532b212b8c108cc1c8bfcbaf5804a5c4156655", "tree": "3667714104cdd0bc0f8078a0ea619676e0764c7f", "parents": [ "d013131a2892d175fcecc270d51e526c617dc170" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 21:11:46 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:28:10 2025 -0700" }, "message": "Rename Host/Runtime hostname fields to Outside/Inside pattern\n\nThis renames all hostname/OS/working directory fields to follow the\nOutside/Inside naming pattern rather than Host/Runtime naming pattern.\n\n- HostHostname -\u003e OutsideHostname\n- RuntimeHostname -\u003e InsideHostname\n- HostOS -\u003e OutsideOS\n- RuntimeOS -\u003e InsideOS\n- HostWorkingDir -\u003e OutsideWorkingDir\n- RuntimeWorkingDir -\u003e InsideWorkingDir\n\nThis includes related method names, struct fields, and JavaScript properties.\n\nCo-Authored-By: sketch\n" }, { "commit": "ae3480fee8b8ebf867a1097868d0032ff4b51324", "tree": "468d02673f8e9fff21fd33d9a343ef4125d646ac", "parents": [ "5544d146e1ebe2747ae282e611f8d0a2c23f2b6c" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 23 15:28:20 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 23 15:57:49 2025 -0700" }, "message": "allow random (ephemeral) host ports for ssh server\n\nalso fixes an issue with authorized_keys files that\ncontain multiple pks: it now checks all of them not\njust the first one it parses.\n" }, { "commit": "8d50d7b66ae26221cbc8fbb246c62e47d8462cfe", "tree": "bddba9157a35c1a472d93f7d274a715a57ff154f", "parents": [ "72682df0c3aa2e2b8917d0f66bdf8988dfbcb59f" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 13:12:40 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 13:12:40 2025 -0700" }, "message": "gofumpt.\n" }, { "commit": "baa2b590cc3a5f63bc52c3324f2835666b9ae450", "tree": "69b313aab41b2f3d3cd27c72ce7d09f6b6453aa6", "parents": [ "97188fc1955f036e57c42a2f345b3c01fc5f0dc0" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Apr 23 10:40:08 2025 -0700" }, "committer": { "name": "GitHub", "email": "noreply@github.com", "time": "Wed Apr 23 10:40:08 2025 -0700" }, "message": "sketch: initial container ssh support (#15)\n\nAdds an in-process ssh server to the sketch agent running inside\nthe container.\n\nThe ssh server implementation uses https://github.com/gliderlabs/ssh/\n\nThis change does not automatically generate any keys (this may come later).\nYou specify the server identity private key and the user\u0027s authorized public\nkeys on the sketch command line.\n\nThe host sketch process reads these files from the cli flags at startup. Once\nthe container is launched, it passes these keys to to the container\nsketch process via new /init POST body fields." }, { "commit": "d140295fa7d794f5b30feb4eee2f45f9cc9ff383", "tree": "659e3d00a94eb2aee0da202efaf2e33477c0e895", "parents": [ "37dc4cf04df5dee6feff005d68a4a24e55be8990" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 03:54:37 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 06:40:17 2025 -0700" }, "message": "sketch: Propagate host vs. runtime OS/WorkingDir/Hostname\n\nIf you have a bunch of sketch sessions, you need to know where they were\nlaunched. The container hostnames (some random thing) and working dirs (always /app)\naren\u0027t very helpful, so we want to keep around both. I\u0027ve updated the UI\nto show them as well.\n\nThis commit chooses \"Host\" and \"Runtime\" as the names of the \"Outside\"\nand \"Inside\" sketch. I\u0027m open to suggestions for better names.\n\nCo-Authored-By: sketch, but it did a so-so job\n" }, { "commit": "176de79e7d3d6a894babccc5d8dc8bb9bf133828", "tree": "bec096858c74a6a51774ccff398a613c7944c2db", "parents": [ "d9f1337ec3317a60df50d8ba4eefb448473d62fa" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Mon Apr 21 12:25:18 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Mon Apr 21 15:35:02 2025 -0700" }, "message": "sketch: compress JS and CSS\n\nWe noticed that our JS bundles weren\u0027t compressed; let\u0027s compress them.\nSketch did most of the work here itself, but there\u0027s some nonsense\naround the fact that we pass a zip.Reader around, which can\u0027t seek.\nWe\u0027re probably doing more work than strictly necessary on the server side.\n\nAdd gzip compression for JS and source map files in esbuild\n\n- Added compression for .js and .js.map files in the esbuild Build() function\n- Gzipped files are created alongside the originals with .gz extension\n- This allows for compressed asset serving when supported by clients\n\nCo-Authored-By: sketch\n\nAdd support for serving pre-compressed JS and source map files\n\n- Created a custom HTTP handler for /static/ path\n- Handler checks if requested JS/JS.map files have gzipped versions\n- Serves gzipped files with proper Content-Encoding headers when available\n- Falls back to original files when compressed versions are not found\n- Client support for gzip encoding is verified through Accept-Encoding header\n\nCo-Authored-By: sketch\n\nExtend gzip compression to include CSS files\n\n- Added CSS files to the compression list alongside JS and source map files\n- Added debug output to show which files are being compressed\n- Updated error messages to reflect the inclusion of CSS files\n\nCo-Authored-By: sketch\n\nFix variable naming in Accept-Encoding header processing\n\n- Renamed \u0027header\u0027 variable to \u0027encoding\u0027 for better semantics when processing Accept-Encoding headers\n- Addresses gopls check issue in compressed file handler\n\nCo-Authored-By: sketch\n\nSimplify Accept-Encoding header processing\n\n- Simplified check for gzip support using strings.Contains()\n- More efficient approach than splitting and iterating\n- Addresses gopls efficiency recommendation\n\nCo-Authored-By: sketch\n\nExtract compressed file handler to separate package\n\n- Created a new package loop/server/gzhandler\n- Moved compressedFileHandler implementation to the new package\n- Renamed to Handler for better Go idioms\n- Updated loophttp.go to use the new package\n- Improved modularity and separation of concerns\n\nCo-Authored-By: sketch\n\nEnhance gzhandler and add test coverage\n\n- Updated gzhandler to handle all files except .gz files\n- Added comprehensive test suite for gzhandler\n- Removed debug print from esbuild.go\n- Tests different file types, browsers with and without gzip support\n- Tests directory handling\n\nCo-Authored-By: sketch\n\nFix \u0027seeker can\u0027t seek\u0027 error in gzhandler\n\n- Changed approach to read gzipped file into memory before serving\n- Avoids io.Seeker interface requirement for http.ServeContent\n- Fixes 500 error when serving compressed JavaScript files\n- Added missing io import\n\nCo-Authored-By: sketch\n" }, { "commit": "d9f1337ec3317a60df50d8ba4eefb448473d62fa", "tree": "67b60b05d8e2e3fb01293f68dfc5d08600a28dda", "parents": [ "db8c5abcc18082bcbf47247e8d50889a32e65e06" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Apr 21 15:08:49 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Apr 21 15:21:40 2025 -0700" }, "message": "webui: auto-generate types.ts from go structs\n" }, { "commit": "86b56862f8d3e192646a17548ef5294582c31f8f", "tree": "d0235f3f56695de8e6281ba3f57a663847204f33", "parents": [ "f5bb3d3f1aa33e2a066c4139675f096f73c1f9d4" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Fri Apr 18 13:04:03 2025 -0700" }, "committer": { "name": "GitHub", "email": "noreply@github.com", "time": "Fri Apr 18 13:04:03 2025 -0700" }, "message": "loop/webui: swtich to web components impl (#1)\n\n* loop/webui: swtich to web components impl\n\nThis change reorganizes the original vibe-coded\nfrontend code into a structure that\u0027s much\neasier for a human to read and reason about,\nwhile retaining the user-visible functionality\nof its vibe-coded predecessor. Perhaps most\nimportantly, this change makes the code testable.\n\nSome other notable details:\n\nThis does not use any of the popular large web\nframeworks, but instead follows more of an\n\"a la carte\" approach: leverage features\nthat already exist in modern web browsers,\nlike custom elements and shadow DOM.\n\nTemplating and basic component lifecycle\nmanagement are provided by lit.\n\nState management is nothing fancy. It\ndoesn\u0027t use any library or framework, just\na basic \"Events up, properties down\"\napproach.\n\n* fix bad esbuild.go merge\n\n* loop/webui: don\u0027t bundle src/web-components/demo\n\n* loop/webui: don\u0027t \u0027npm ci\u0027 dev deps in the container\n\n* rebase to main, undo README.md changes, add webuil.Build() call to LaunchContainer()" }, { "commit": "2e463fb649fcff14d4025ddb91f630a98e7da526", "tree": "0e86854d80d2759a913870655f13226c31f9d30c", "parents": [], "author": { "name": "Earl Lee", "email": "earl.lee@sketch.dev", "time": "Thu Apr 17 11:22:22 2025 -0700" }, "committer": { "name": "Earl Lee", "email": "earl.lee@sketch.dev", "time": "Thu Apr 17 11:35:33 2025 -0700" }, "message": "Initial commit\n" } ] }