)]}' { "log": [ { "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": "c540df7f730eb0e07a5aa543ec128dba38a7cf81", "tree": "02c00aabebdb39b72cc272a33c3d787f1809a602", "parents": [ "e67e3b65003d21f68c3736a563a94e5ad1ba26b0" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 09:21:56 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 09:21:56 2025 -0700" }, "message": "sketch/mcp: increase connection and tool timeouts\n" }, { "commit": "e67e3b65003d21f68c3736a563a94e5ad1ba26b0", "tree": "07f9899eb261dfc31d045192ee7e8e58d03ac9c4", "parents": [ "9c74cb9f4865a28d468fdeab0e5755a882e55409" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 24 16:54:21 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 24 16:55:19 2025 -0700" }, "message": "loop: nudge to install platform tools\n" }, { "commit": "d64bc91c50a8cb172ce82388c786f9382de19d51", "tree": "120defb3e1d3a2c52b8300ae4d9958d10bc8afb6", "parents": [ "aeaf892c3a40e99c699c006dbccbe9c859e0cc69" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 11:42:33 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 11:42:33 2025 -0700" }, "message": "loop: put each tool on its own line\n\nCosts nothing, prevents merge conflicts for me.\n" }, { "commit": "beaa86a1fa821ce5874f3a2dd6a1609f7bd79698", "tree": "acab6693c4e740b8908ad16327418a3578eaee06", "parents": [ "141547d1d7169bf1c9a9d156eb6b585f1d9d398c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 23 03:37:21 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 11:30:35 2025 -0700" }, "message": "all: delete commit-message-style tool\n\nIt added significant latency.\nAll the caching options were unappealing.\n\nInstead, let users specify in dear_llm.md if they care,\nand default to simple, concise messages.\n\nAt some point (soon?), we will likely\nmake it easier in the UI to edit commit messages.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sbcd1f38735d091aek\n" }, { "commit": "3dd3e4144d2e0b9fec6b5e0039ed12e2c0170265", "tree": "11eb418a7980cb9f7faa6b1a20dee2e811a24b13", "parents": [ "a6b995b83277dd730e35a8f5be14c139d82a544e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 20:32:03 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 23 17:53:58 2025 -0700" }, "message": "webui: add diff display for patches\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2e9bdfb014ddec3ck\n" }, { "commit": "9dc2f62c6d44f53e057bd7319ffbcbc00ef5f2a0", "tree": "753ea15307dd1ee834ae6973f1a363858e09f4b5", "parents": [ "ddd22da04ed405520012089ffb2bc244b82e41e7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 23 01:04:00 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 23 12:58:52 2025 -0700" }, "message": "all: remove multiplechoice tool\n\nIt\u0027s rarely used; doesn\u0027t pull its weight.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s11d827ec20c6ddb8k\n" }, { "commit": "3b44cc3120b595e82dcc2a6081b8a07fc24c739e", "tree": "3d4f3c2911390428c0c89aa8dcd13dbfb035ce31", "parents": [ "f18aafd177a90b928b1c1babbe5e763f85eab56d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 02:28:14 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 20:10:27 2025 -0700" }, "message": "loop: do slug generation outside the agent loop\n\n[This commit message written entirely by a human; it is all useful.]\n\nWe can make a slug based on the first message.\nIt\u0027s good enough.\nThat keeps it--and the slug tool--out of the agent\u0027s context.\nIt\u0027s also one fewer step for extremely short Sketch runs,\nwhich is the straw that broke this particular camel\u0027s back.\n\nThis is a mild UI regression, in that there\u0027s a slight stall\nafter the user types their first message, during which\nthe slug is being generated. See (2) below.\n\nWhile we\u0027re here, add handling of compaction agent messages.\n\nThis leaves two big TODOs outstanding:\n\n1.\n\nUntangle the awful rats nest that is slug and branch management;\nwe have callbacks back and forth and layers and locking and it\u0027s all confusing.\nOne visible for that this ugliness takes is that every time the agent tries out a slug,\nthe top bar in the webui updates, even if we then reject that slug as a duplicate.\nthere are other forms of ugliness, just less visible.\n\n2.\n\nMake slug generation concurrent with the rest of the agent,\nto avoid a short stall right after the user\u0027s first request (ick).\n\nWhen we make slug setting concurrent, we\u0027ll likely need to resuscitate\nthe bashPermissionCheck, except it\u0027ll be \"silently block and wait for\nbackground slug generation to complete\", rather than \"reject the tool call\".\nDitto for about_sketch, and any other tool call that expects\nthe slug or branch name to be set.\n\nGenerally, before undertaking this effort, we should fix (1) above,\nmake convos generally concurrency safe (maybe COW?), and\nfigure out to get race-enabled innie builds.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8ac5f6a9faa611ebk\n" }, { "commit": "43b60b9b6906a2077175bcb1c3ea53b90d92a9f8", "tree": "14ecadd5c94db811087f97b0e7858f1482ff5f4e", "parents": [ "95354b15c6469f871ecabd778c04304bcce16195" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 14:57:10 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:46:45 2025 -0700" }, "message": "llm: make Tool.Run return ToolOut\n\nThis is preliminary work towards\nallowing tools to add additional information.\nNo functional changes (at least, that\u0027s the intent).\n" }, { "commit": "accf37cc10b9606702e8719157f60fb0488d9228", "tree": "b0ae324feea823b5ba669e1290d2f408bbe4c652", "parents": [ "93b53d8e55e1b0c5c85d387ab46a6ee3f96a8e8c" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 18 07:29:19 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 18 07:29:19 2025 -0700" }, "message": "Fix sketch -unsafe after push button change.\n\nI missed a case for -unsafe.\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": "246132065e858778359e65250bc7f2e19d477710", "tree": "a937884247db726e6c9d7e475334e77659b6a88f", "parents": [ "183d114ef7f3b806b77ff6691bcadbbf5bdd805c" ], "author": { "name": "philz", "email": "philip.zeyliger@gmail.com", "time": "Tue Jul 15 20:56:21 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jul 15 20:56:31 2025 -0700" }, "message": "sketch: create empty commit on empty repo, both in innie and outtie\n\nLife\u0027s a bit too short to fail when the git repo exists but is empty.\nFor innie (e.g., using -unsafe on an empty repo), we just create the\nempty commit. For outtie, we do the same thing, since initializing an\nempty repo would be a weird different code path.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se0a9606cc9f43388k\n" }, { "commit": "3ad8c8d9847a3a6d2c69f8ef76e1af4758ff845e", "tree": "71f39eb5c3c79c697ade4c930839b7aa122f866a", "parents": [ "9022ae08d3483b711bc9bf6673e280aef633e8a0" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 15 21:05:23 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 15 21:05:23 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "9022ae08d3483b711bc9bf6673e280aef633e8a0", "tree": "f0d208ef1dc5ba47fa045e0aa0f51267398cae6e", "parents": [ "0af5fbe505d9b3fb76e5d2cbd5a7ef964f9c06a8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 20:52:30 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jul 15 13:53:41 2025 -0700" }, "message": "sketch: dump message history to /tmp on conversation compaction and include filename in summary\n\nAdd dumpMessageHistoryToTmp method that dumps complete agent message history to timestamped JSON file during conversation compaction, returning filename for inclusion in compaction summary message.\n\nWhen conversation compaction occurs, the agent now:\n- Dumps entire a.history array to /tmp/sketch-messages-{sessionID}-{timestamp}.json as formatted JSON\n- Returns filename from dumpMessageHistoryToTmp for reference in summary message\n- Includes dump file location in compaction summary: \"The complete message history has been dumped to {filename} for your reference if needed\"\n- Continues compaction process even if dump fails to avoid blocking conversation flow\n\nImplementation:\n- Added dumpMessageHistoryToTmp method returning (filename, error)\n- Modified CompactConversation to call dump before generating summary\n- Enhanced summary message to conditionally include dump file reference\n- Uses 2-space JSON indentation for readability and 0644 file permissions\n- Creates timestamped filenames to avoid conflicts across multiple compactions\n\nThe implementation provides access to complete conversation history during epoch restarts while maintaining conversation compaction benefits.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3784d6168585f9c4k\n" }, { "commit": "369f2628fdc10ceb46a776d98c168d48ecd1f22b", "tree": "b2ff54d6b7ed6b194b417994f97d60e55f708bbb", "parents": [ "2e967e55c5619e08b45a5a1e162774a0de859dab" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 15 00:02:59 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 14 17:25:50 2025 -0700" }, "message": "sketch/loop: handle existing git repos in innie sketch\n\nCheck if /app/.git already exists before attempting to clone. If it exists\n(e.g., from skaband images or user images with existing git repos), configure\nthe origin remote and fetch instead of cloning.\n\nThis fixes compatibility with skaband dockerfiles that create images with\nexisting git repositories, and adapts to the object-only approach introduced\nin commit 9e8f5c78e8cef4c73e7b2629b2270ab572d530f8.\n\nThe implementation uses a helper function upsertRemoteOrigin that handles\nboth setting the URL for existing origin remotes and adding new ones.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9625bfa389b6b7dek\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": "784d5bd5d850f90359edc850484c01f15b6ce173", "tree": "fdc0467922309487f0d28988b2664c51fac99c68", "parents": [ "eb91caaa81362c5528bd0e3a8ca0f2b61de43852" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 11 00:09:30 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 11 16:24:01 2025 -0700" }, "message": "dockerimg: add only git objects to docker image\n\nInstead of copying the entire working directory\n(including uncommitted changes, hooks, and config files),\ncreate a bare git repository and use git clone --reference.\n\nThis approach:\n- Avoids copying uncommitted changes, hooks, and local config files\n- Works correctly with git worktrees and submodules\n- Reduces Docker image size substantially\n- Maintains all git history and functionality\n\nFixes boldsoftware/sketch#190\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6af36147e2c4df00k\n" }, { "commit": "eb91caaa81362c5528bd0e3a8ca0f2b61de43852", "tree": "79e392448584cfcc64184365936fe8914a974437", "parents": [ "5e97dfc62b2c26cd6f483b06c1f5b034307c9791" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 11 15:29:18 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 11 15:29:30 2025 -0700" }, "message": "sketch/loop: minor cleanup\n\nSuggested by Ye Olde IDEe.\n" }, { "commit": "17b2fd9b8ed388f341b380c88e7c4c6f88494573", "tree": "1c1679f6ead6c587e8bf5add8ffcb978449c6703", "parents": [ "39eb24f7dfb82c802dfe7b4caac500acde26a907" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 22:47:13 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 10 19:38:07 2025 -0700" }, "message": "claudetool: replace timeout parameter with slow_ok boolean\n\nEmpirically, the agent doesn\u0027t set timeouts long enough,\nand doesn\u0027t retry on failure.\n\nGive it only one decision to make: Is this maybe a slow command?\n\nIf, horror of horrors, your project can\u0027t accomplish tasks within the\ndefault timeouts, there\u0027s a new command line flag to adjust them.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc26e3516f28c22d4k\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": "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": "e1c8b7bedc96d182916d5e1b9a07913945ffbafb", "tree": "f41e7f56daac66b7315c7fba2d52e97e2381449a", "parents": [ "c37e066371997d221af074b0bc1da392c82c6a2b" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 14:50:26 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 14:51:40 2025 -0700" }, "message": "move git config from Dockerfile to agent.go Init() method\n\nMove git configuration (user.email, user.name, http.postBuffer) from\nDockerfile creation to runtime initialization in agent.go Init() method.\n\nI want to make the layering for Docker images as simple as possible.\nThe git configuration here can be harmlessly done once sketch starts,\nsince it\u0027s a vew simple git operations.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s04b2af644e2dbe4fk\n" }, { "commit": "6534c7a6c4a10f8a9660835e2f0d8c27101bcd34", "tree": "606cf93a234df729dcec223737b1e8f2d2ade8de", "parents": [ "238c18f42ca7d7c795b3ba2ffe0049635ef7dbae" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 01 01:48:52 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 02 15:48:16 2025 -0700" }, "message": "sketch: add patch callback hook to warm codereview cache\n\nWhen the agent patches a file, concurrently pre-compile test binaries\nin the background to speed up future codereview runs.\n\nThis helps make codereview runs faster without\npre-flighting everything in the whole repository.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2a01805b644342f9k\n" }, { "commit": "238c18f42ca7d7c795b3ba2ffe0049635ef7dbae", "tree": "c51dbe837f297ab2022f83fa454f3c861bdbc079", "parents": [ "26b6f9ba1a9647f7f0b0983f8d640b2d81789f5d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jun 30 22:26:54 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 02 15:48:16 2025 -0700" }, "message": "claudetool: add callback hook system to patch tool\n\nWe\u0027ll use this to improve non-root guidance file usage\nand to warm up the codereview Go build cache.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s01ff6f94febc4485k\n" }, { "commit": "f2814eabe9d341a7d5f8805c41f0628bda9d1056", "tree": "bb1a5ffac41f41f30b4b3d789c44c571ffc8ae54", "parents": [ "d5f0a3ece70bc4f09d1941e08e66af882e540866" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 10:16:50 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 10:16:50 2025 -0700" }, "message": "Let MCP servers use env variables; pass along model key for /attach as well\n" }, { "commit": "4201bde20034516105d17deca502c6e76e7db603", "tree": "7c39dbc55cfc88e3aa2f6866f5400c7790fd7fb6", "parents": [ "cad67b0ac0e374dd687ca4ef8abddbc9c4323fd5" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 27 17:22:43 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 27 17:22:43 2025 -0700" }, "message": "Replace skaband tools with a sketch.dev MCP client\n\nsketch.dev provides seeing recent sessions and reading a past\nsession as tool.\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": "194bfa8cc3970d28f2d072dc82142e24b7e56c9f", "tree": "8cf0872d8d1d6b1ab5eec4ef1f20ab7b201a7de9", "parents": [ "ba351be3a9e50c67baef59af5ee35e3b654727e1" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jun 24 06:03:06 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jun 24 06:03:06 2025 -0700" }, "message": "sketch: add MCP support\n\nLets you initialize extra tools via MCP. This is additive, so it\u0027s\nharmless enough.\n\nHere are some examples of the kind of things you can pass to the -mcp\nflag.\n\n {\"name\": \"context7\", \"type\": \"http\", \"url\": \"https://mcp.context7.com/mcp\"}\n {\"name\": \"context7-http\", \"type\": \"http\", \"url\": \"https://mcp.context7.com/mcp\"}\n {\"name\": \"context7-stdio\", \"type\": \"stdio\", \"command\": \"npx\", \"args\": [\"-y\", \"@upstash/context7-mcp\"]}\n {\"name\": \"context7-sse\", \"type\": \"sse\", \"url\": \"https://mcp.context7.com/sse\"}\n {\"name\": \"local-tool\", \"type\": \"stdio\", \"command\": \"my_tool\", \"args\": [\"--option\", \"value\"], \"env\": {\"TOKEN\": \"secret\"}}\n { \"name\": \"playwright\", \"command\": \"npx\", \"args\": [ \"@playwright/mcp@latest\" ]}\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s259a35d11e7bd660k\n" }, { "commit": "882e7ea7097129ff75e3595b049df585976e12a1", "tree": "bdf436800495b973f3b8d808dc4ec9fdc83d4b31", "parents": [ "1f8fe9c0531de311dc3847f766f748da28fc3368" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Fri Jun 20 14:31:16 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 20 08:21:42 2025 -0700" }, "message": "loop: preserve cumulative usage across conversation compaction\n\nconversation: add optional CumulativeUsage parameter to New function\n\nUpdate CompactConversation to preserve cumulative usage statistics when\nresetting the conversation, preventing usage numbers from being lost during\ncompaction.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s11dcb84790847494k\n" }, { "commit": "d5c8d71adfec8b8cb5aa15c7ce74da19c697191f", "tree": "3875d82cf3791db0c49c160455616eeaafe208ea", "parents": [ "2343f8a73cb0a5a846a4f44075d8e039daac7816" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jun 17 15:19:45 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jun 17 22:21:57 2025 +0000" }, "message": "loop: avoid deadlock when pushing to a checked out branch\n\nSketch did a good job with this:\n\nThe Problem 🐛\nIn AgentGitState.handleGitCommits(), there was a double-locking scenario:\n\nMethod acquires mutex lock\nDuring git push retry logic, calls IncrementRetryNumber()\nIncrementRetryNumber() tries to acquire the same mutex again\nDeadlock - thread hangs forever waiting for a lock it already holds\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sfe0bcab73addc32ak\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": "c886ac5478194e95f93b166572a7a081eb6cadb8", "tree": "fd4be26cd6cf20162ff22ef21a8c1dc6e51e59d5", "parents": [ "1bd636c83311a9195fca38515ee254bf07ae0d12" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Fri Jun 13 23:40:03 2025 +0000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sat Jun 14 01:03:45 2025 +0000" }, "message": "loop: add special instruction field\n\nIncludes a demo.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc7078aeb8c59e49ak\n" }, { "commit": "1417b69f8aaee33e9e96341e7e97ffbb9e7bb051", "tree": "1568f44bbb31dde125571a7b9a3aa1eabc2d6002", "parents": [ "8773e68fcce9965da3c6a1ef91c88476f84e29bb" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jun 12 11:07:04 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jun 12 11:07:04 2025 -0700" }, "message": "Fixing git checkout after retrying hooks.\n\nA place where we didn\u0027t update the wip branch thing.\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": "4c1cea85b222a1ec94890be8960c69f549870454", "tree": "fd605725458f4a3b010292b927368963aeb3fcdd", "parents": [ "0113be559976cfb1ce0e78a9a69c19a6394b2c3d" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 09 14:16:52 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 09 14:16:52 2025 -0700" }, "message": "Fix up sketch-wip handling when --commit is not specified.\n\nIf -git-remote-url and -commit are both specified, we check out\nsketch-wip. We should do so always when we\u0027re in a container, so\ndoing that now.\n\nThe new sketch-wip stuff is weird if we\u0027re not in a container (the\n\"unsafe\" mode without -outside-hostname). I\u0027ve changed the system\nprompt slightly to try to figure that out.\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": "6c5beffcea347f8b5a738f9563980e64a9f8ccc0", "tree": "15e46908d94afd6ac357ab44e8bd72f34f2ad705", "parents": [ "715b8d9e46757b1254624e8b1329a9581baa10d3" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 06 13:03:49 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 06 13:07:02 2025 -0700" }, "message": "Disable pager in post commit hook we install.\n\nMakes using the terminal to do git operations miserable.\n" }, { "commit": "715b8d9e46757b1254624e8b1329a9581baa10d3", "tree": "2e7d423e145ccd8746b1bf1e0c441224530e691a", "parents": [ "cecba06431da9aa134898d23a353e1897f3e3625" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jun 06 12:36:38 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 06 12:36:38 2025 -0700" }, "message": "loop: always use branch sketch-wip in innie\n\nModify innie sketch to always create and work on a dedicated \u0027sketch-wip\u0027\nbranch for all git operations, pushing this branch to outie instead of\npushing HEAD directly.\n\nHaving a dedicated branch name makes it clearer how to operate\ninside the container, for both humans and sketch.\nIt also prevents the container from pushing whatever transient\ncommits occur while sketch does (say) a bisection or other git work.\nIt should also prevent sketch from constantly spinning up new\nbranches as it starts new tasks in a long conversation.\n\nI\u0027d rather have called the branch \u0027sketch\u0027 instead of \u0027sketch-wip\u0027,\nbut that conflicts with all the branches called \u0027sketch/foo\u0027. Alas.\n\nThis was mostly written by Josh, but I made it work whether or not\nsketch-wip already exists as a branch.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s4ea6db2873a60129k\n" }, { "commit": "c17ffe37ff9ac2c5e0f19a67854b77955976c1bb", "tree": "2318aae7918bd74a4391b0faac4355f0e861b7d5", "parents": [ "2a32f01e45169200bfb252031ca880ef09d7c0c2" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Thu Jun 05 19:49:13 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Thu Jun 05 19:49:46 2025 -0700" }, "message": "sketch: add session history tools using markdown table API\n\nAdd session history tools to allow the LLM to access previous sketch sessions:\n- New list_recent_sketch_sessions and read_sketch_session tools\n- Integration with skaband client for session data retrieval\n- Session history tools automatically added when skaband client available\n- Updated agent configuration to include skaband client\n- Client handles plain text markdown table response from API\n- Display server-generated markdown table directly to LLM\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s693acdfdaaa392c8k\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": "068f4bbc12594cd48a4d2b19803849434d0bdb24", "tree": "e703926e4d61f3a3b0e8cc117bc4ea4f0327aa6e", "parents": [ "d27921f73b315dc1e01b92cc27370c29c3986f64" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 19:12:22 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 19:30:39 2025 -0700" }, "message": "loop: remove pointless fmt.Sprintf" }, { "commit": "664404e9e53fe0d28a35f9b6da2274e3bdea4c4b", "tree": "be94392672a64c991b962e55e7a297e9631fa0af", "parents": [ "a9fd88ff17fe6d59ed2295b46dd8f389d30fdf23" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 21:56:42 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 12:05:33 2025 -0700" }, "message": "cmd/sketch: add -upstream flag for git branch management\n\nTo be used in future work.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s40fdfb9a579bf6f2k\n" }, { "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": "2f0eb6995fc1041d23b8224a099013e89bb03f67", "tree": "9713c17be9deab35f9252836e0f48d0b12ee295d", "parents": [ "0df32aacd4c14c8db1ea2876378f4bc23a2744dc" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 09:53:42 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 09:53:42 2025 -0700" }, "message": "Capture git origin before replacing it.\n" }, { "commit": "222bf413a2ab02a430d92961cb3eef05dcf5ff89", "tree": "e5096166ffe7ed487def369ac9f2981e745d2edf", "parents": [ "c898abf9973147658cb90862c071759b0a82259e" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 16:42:58 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 04 16:46:40 2025 +0000" }, "message": "loop: replace sketch-host remote with origin remote in agent.Init()\n\nChange git remote handling in agent initialization to replace any existing\norigin remote with the provided git remote URL instead of creating a\nsketch-host remote.\n\nProblem Analysis:\nPreviously, when a git remote URL was provided to the agent, it would:\n1. Add a new remote called \u0027sketch-host\u0027 pointing to the provided URL\n2. Configure additional fetch refs for feature branches\n3. Use \u0027sketch-host\u0027 for all fetch operations\n\nThis created confusion and inconsistency since most git workflows expect\nto interact with \u0027origin\u0027 as the primary remote, not a custom \u0027sketch-host\u0027\nremote.\n\nImplementation Changes:\n\n1. Git Remote Setup Logic:\n - Remove existing \u0027origin\u0027 remote if it exists (ignoring errors if not present)\n - Add the provided remote URL as the new \u0027origin\u0027 remote\n - Update fetch operations to use \u0027origin\u0027 instead of \u0027sketch-host\u0027\n\n2. Branch Detection Logic:\n - Updated branchExists() function to remove \u0027refs/remotes/sketch-host/\u0027 reference\n - Now only checks \u0027refs/heads/\u0027 and \u0027refs/remotes/origin/\u0027 for branch existence\n\n3. Comment Updates:\n - Updated git debugging comment in dockerimg/githttp.go to reference \u0027origin\u0027 instead of \u0027sketch-host\u0027\n\nTechnical Details:\n- The \u0027git remote remove origin\u0027 command gracefully handles the case where\n origin doesn\u0027t exist by logging and continuing\n- All existing git operations (fetch, checkout, commit tracking) continue\n to work seamlessly with the origin remote\n- branchExists() function correctly identifies branches in origin remotes\n\nBenefits:\n- Consistent with standard git workflows that use \u0027origin\u0027 as primary remote\n- Eliminates confusion about which remote to use for git operations\n- Maintains all existing functionality while using standard git conventions\n- Simplifies git operations by using the expected \u0027origin\u0027 remote name\n- Better integration with external git tools and workflows\n\nTesting:\n- All existing loop package tests continue to pass\n- Verified git fetch, config, and branch detection work correctly\n- Integration testing confirms seamless operation with origin remote\n\nThis change provides a more intuitive and standard git remote configuration\nwhile preserving all agent functionality and improving compatibility with\nstandard git workflows.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9edcdfe5bae3ef4ek\n" }, { "commit": "59e1c1694c0ea45893f344b98d7efaca072e5e87", "tree": "ba24ce303cea4aac8e2a63ca0805c42e04f93efb", "parents": [ "138ec2436631f136dd2e8b4891211f896587ff00" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 12:54:34 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jun 03 17:27:32 2025 +0000" }, "message": "loop: fix branch naming increment and add user notification for conflicts\n\nFix the branch naming logic when git push encounters \u0027refusing to update\nchecked out branch\u0027 errors to properly increment numbers (foo1, foo2, foo3)\nrather than appending \u00271\u0027 characters, and add user notification explaining\nwhy the branch was renamed.\n\nImplementation improvements:\n\n1. Branch Naming Logic Verification:\n - Confirmed the existing logic already uses proper incremental numbering\n - Added comprehensive test case TestBranchNamingIncrement to verify behavior\n - Test validates that retries generate: foo, foo1, foo2, foo3, etc.\n - The reported issue of foo1, foo11, foo111 appears to be from an older version\n\n2. Enhanced User Communication:\n - Added automatic notification when branch is renamed due to checkout conflict\n - Uses AutoMessageType for consistent automated notification styling\n - Message format: \u0027Branch renamed from X to Y because the original branch is currently checked out on the remote\u0027\n - Provides clear explanation of why the system chose a different branch name\n\n3. Code Quality Improvements:\n - Added detailed comments explaining the incremental naming logic\n - Enhanced error handling context with user-friendly explanations\n - Maintained existing retry logic (up to 10 attempts) for robustness\n\nTechnical details:\n- The retry loop uses \u0027for retries :\u003d range 10\u0027 giving values 0, 1, 2, 3...\n- When retries \u003e 0, fmt.Sprintf(\u0027%s%d\u0027, originalBranch, retries) creates foo1, foo2, etc.\n- User notification is added to the messages array when branch !\u003d originalBranch\n- Notification timing ensures users understand automatic branch renaming\n\nThis resolves potential confusion when users see their branch published with\na different name than expected, providing transparency about the automatic\nconflict resolution process.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s91efcab1b86b45dak\n\nloop: fix branch naming increment to properly handle existing numbers\n\nImplement proper branch naming logic that correctly parses and increments\nexisting numerical suffixes when git push encounters \u0027refusing to update\nchecked out branch\u0027 errors, ensuring foo1-\u003efoo2-\u003efoo3 instead of foo1-\u003efoo11-\u003efoo111.\n\nRoot cause analysis:\nThe previous logic used fmt.Sprintf(\"%s%d\", originalBranch, retries) which\nwould append numbers to whatever originalBranch contained. If originalBranch\nwas already \"foo1\", this created \"foo11\", \"foo12\", etc. instead of the\nintended incremental sequence.\n\nImplementation improvements:\n\n1. Added parseBranchNameAndNumber() function:\n - Extracts base branch name and existing numerical suffix\n - Handles branches like \"sketch/test-branch1\" -\u003e (\"sketch/test-branch\", 1)\n - Handles branches without numbers like \"sketch/test-branch\" -\u003e (\"sketch/test-branch\", 0)\n - Robust parsing that only considers trailing digits as suffixes\n\n2. Enhanced retry logic:\n - Uses parsed base name and starting number for proper incrementing\n - Formula: fmt.Sprintf(\"%s%d\", baseBranch, startNum+retries)\n - Ensures consistent incremental naming regardless of starting branch name\n - Maintains 10-retry limit for robustness\n\n3. Comprehensive test coverage:\n - TestBranchNamingIncrement covers multiple scenarios with table-driven tests\n - TestParseBranchNameAndNumber validates parsing logic with edge cases\n - Tests verify proper behavior for branches with and without existing numbers\n - Edge cases include numbers in middle vs end of branch names\n\nTechnical details:\n- parseBranchNameAndNumber() scans from end of string backward for consecutive digits\n- Only trailing digit sequences are treated as numerical suffixes\n- Error handling ensures malformed numbers default to treating branch as having no suffix\n- Maintains existing user notification when branch gets renamed\n\nExamples of corrected behavior:\n- \"foo\" -\u003e \"foo\", \"foo1\", \"foo2\", \"foo3\"\n- \"foo1\" -\u003e \"foo1\", \"foo2\", \"foo3\", \"foo4\"\n- \"foo42\" -\u003e \"foo42\", \"foo43\", \"foo44\", \"foo45\"\n\nThis resolves the reported issue where branches were getting names like\nfoo1, foo11, foo111 instead of proper incremental numbering.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sfba5fdcdaf99fdcdk\n\nloop: simplify parseBranchNameAndNumber with regex\n\nReplace verbose manual parsing logic with a clean regular expression\napproach to extract trailing digits from branch names.\n\nChanges:\n- Replace 20+ lines of manual digit scanning with simple regex: ^(.+?)(\\d+)$\n- Add regexp import to support the regex functionality\n- Maintain exact same behavior and test coverage\n- Cleaner, more readable, and less error-prone implementation\n\nThe regex ^(.+?)(\\d+)$ captures:\n- Group 1: Everything up to the trailing digits (non-greedy match)\n- Group 2: The trailing digit sequence\n\nThis handles all the same cases but with much simpler code.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sea27e90c492b83d8k\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": "fea9e27ac3067027b0483b85de9869ce200d984f", "tree": "f80dd919b650a2ede32e9c4b66b7659a0938da87", "parents": [ "d1bd519dafdcb8039c88ca386b886e1a0456e17f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jun 02 21:21:59 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jun 02 17:17:39 2025 -0700" }, "message": "loop: restrict git hooks installation to inner instances only\n\nModify git hooks setup to only install hooks when running in container (inner\nsketch instance) to prevent hooks from being installed in outer instances.\n\nPreviously, setupGitHooks was called unconditionally during agent initialization,\ncausing git hooks to be installed in both outer (host) and inner (container)\nsketch instances. This created potential conflicts and unnecessary hook\ninstallation in the outer environment.\n\n**Problem Analysis:**\n- Git hooks were being installed in both outer and inner sketch instances\n- Only inner instances (running in Docker containers) should have git hooks\n- setupGitHooks call was unconditional within the !ini.NoGit block\n- Agent.IsInContainer() method already existed to distinguish instance types\n\n**Implementation:**\n- Added IsInContainer() check around setupGitHooks call in Agent.Init()\n- Maintained existing git repository setup logic for both instance types\n- Preserved all other git operations (tag creation, codebase analysis, etc.)\n- No changes to setupGitHooks function itself\n\n**Technical Details:**\n- setupGitHooks now only executes when a.IsInContainer() returns true\n- IsInContainer() returns a.config.InDocker boolean value\n- Outer instances (InDocker\u003dfalse) skip hook installation entirely\n- Inner instances (InDocker\u003dtrue) continue installing hooks as before\n- removeGitHooks remains unchanged for error recovery scenarios\n\n**Testing:**\n- Added comprehensive TestGitHooksOnlyInContainer test with subtests\n- OuterInstance subtest verifies hooks are NOT created when InDocker\u003dfalse\n- InnerInstance subtest verifies hooks ARE created when InDocker\u003dtrue\n- Test validates both hook file existence and content verification\n- All existing tests continue to pass without modification\n\n**Hook Installation Behavior:**\n- Outer instances: No git hooks installed, clean git repository\n- Inner instances: Full hook installation (post-commit, prepare-commit-msg)\n- Error recovery removeGitHooks remains available for both instance types\n- Git operations like tagging and analysis work correctly in both cases\n\nThis ensures git hooks are only installed where they\u0027re needed (inner instances)\nwhile maintaining full git functionality and preventing conflicts in outer\nenvironments where hooks should not be present.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se8f7e7e9ec63a55ek" }, { "commit": "f75ba2cf69069b8bc6be98fc7a837b3efe6837d6", "tree": "30e604298cceb1ee27914aba467d122e63c20349", "parents": [ "b8a8f35348326b8daf69e31efc77d37fac4bd276" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 17:02:51 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 02 17:02:51 2025 -0700" }, "message": "Stop installing hooks when running innie outside of a container.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s90b20da95c2e7dd2k\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": "364f741483c1bd2c18cb3ff2ad255c9042c5362d", "tree": "95ff4d67ab4707d41a0ddad1e801c55cf8bf33c7", "parents": [ "6cad861fbb3dbb646d190b7a5efc2fe982ea3aa8" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Mon Jun 02 00:55:44 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sun Jun 01 18:29:15 2025 -0700" }, "message": "loop: add periodic port monitoring to container processes\n\nPartial fix for issue #47\n\nImplement periodic port monitoring functionality that runs ss every 5 seconds\nto detect changes in container listening ports, providing visibility into port\nusage changes during sketch agent execution.\n\nProblem Analysis:\nContainer processes need visibility into port changes that occur during\nagent execution. Without monitoring, it\u0027s difficult to detect when services\nstart or stop listening on ports, which can be crucial for debugging\nand understanding application behavior.\n\nImplementation:\nAdded comprehensive port monitoring system to Agent struct:\n\n1. Port Monitoring Infrastructure:\n - Added portMonitorMu mutex and lastPorts field to Agent struct\n - Created startPortMonitoring() method that launches background goroutine\n - Uses time.Ticker with 5-second intervals for periodic checks\n - Only activates when running in container mode (IsInContainer() check)\n\n2. Port Detection Logic:\n - updatePortState() executes ss -lntu to get listening ports\n - Compares current port state with previous state for change detection\n - Thread-safe port state updates using dedicated mutex\n\n3. Port Parsing and Comparison:\n - isSSOutput() automatically detects command output format\n - Extracts protocol and local address from port listings\n - Returns map[string]bool for efficient port comparison\n\n4. Change Detection and Logging:\n - logPortDifferences() identifies newly opened and closed ports\n - Structured logging with slog for port changes\n - Separate log entries for new ports and closed ports\n - Non-critical operation - errors don\u0027t interrupt agent execution\n\nTechnical Details:\n- Background goroutine lifecycle tied to agent context cancellation\n- Handles IPv4/IPv6 address formats correctly\n- Only monitors LISTEN state ports, ignores other connection states\n- 5-second polling interval balances responsiveness with resource usage\n\nTesting:\n- Added comprehensive test coverage for port parsing functions\n- Verifies port difference detection logic\n- All existing loop package tests continue to pass\n- Integration test confirms no regressions in agent functionality\n\nIntegration:\n- Port monitoring starts automatically in Agent.Loop() method\n- Only enabled for container execution mode\n- Uses same context pattern as existing background tasks\n- Follows established logging and error handling patterns\n\nThis enhancement provides real-time visibility into container port\nchanges without affecting core agent functionality or performance.\n\nBenefits:\n- Real-time port change detection for debugging\n- Thread-safe implementation with proper resource cleanup\n- Comprehensive test coverage ensures reliability\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9bd1b1bd0b518b2bk\n" }, { "commit": "b421a24f862d91101b9e5b4738f93693e18672eb", "tree": "2ccccb673504f5d5d3ca689816e95c92fd3c3041", "parents": [ "00bcaef0355aaff1daea17ac0631fd17cabb0235" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 23:22:55 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:58:07 2025 -0700" }, "message": "all: remove anthropic edit tool support\n\nThe Anthropic str_replace_editor tool implementation has rotted;\nit would require a bit of work to upgrade it for Claude 4.\n\nRemove it for now.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6129e8703301600dk" }, { "commit": "495c1fa247565e21b36bcb847c6cd3f08e0e196f", "tree": "5c7031ad37f4f0181824664f15220f87c7314861", "parents": [ "b6bc113f9ebb00084285e8c3aeaf2fd4a648afc0" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 00:37:22 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 15:12:53 2025 -0700" }, "message": "claudetool: add just-in-time tool installation for bash commands\n\nImplements automatic tool installation when bash commands use missing tools,\nproviding a seamless experience for the LLM to Just Use tools it knows ought to exist.\n\nCore Features:\n\n1. Command Analysis Pipeline:\n - Parse bash commands to extract individual tools/programs\n - Use exec.LookPath to check if tools exist in PATH\n - Handle shell built-ins, absolute/relative paths correctly\n - Support complex command chaining with \u0026\u0026, ||, ;, and |\n\n2. Subagent Tool Installation:\n - Spawn dedicated subagents to install missing tools\n\nThe system preserves existing bash tool behavior while adding invisible\ntool installation. Original commands run regardless of installation\nsuccess/failure, avoiding agent confusion.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s226cd6260a6469e9k\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": "b509a5ddb86a4881eb19030f321857b305901022", "tree": "84c8d40e7c0545c63e7a8ea56a71fa109e91f028", "parents": [ "7fbc8e43ff52b6f5dc69fdd26ad7388bdac70435" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 15:49:42 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 12:50:41 2025 -0700" }, "message": "loop: ensure proper blank line before git commit trailers\n\nThe prepare-commit-msg hook was incorrectly adding only a single newline\nbefore trailers in some cases, resulting in missing blank lines between\nthe commit message body and trailers like Co-Authored-By and Change-ID.\n\nThis was particularly noticeable after rebasing where commit messages\nwould have:\n- Subject line\n- Single newline (no blank line)\n- Trailers\n\nThe fix improves the logic to:\n- Check if file ends with newline using tail -c 1\n- If no trailing newline: add two newlines (complete line + blank line)\n- If trailing newline exists: check if last line is empty\n- If last line has content: add one newline for blank line\n- If last line is empty: add nothing (blank line already exists)\n\nThis ensures there\u0027s always exactly one blank line before trailers,\nfollowing proper git commit message formatting conventions.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s796b26b31c71b2e2k" }, { "commit": "90993a0d42e84be835ebf2096687a299d461cfeb", "tree": "97578882ad4cb95eec131fea74c4f37eeac36420", "parents": [ "7027307b381e2b8421c95ebb2f872b8e3c567f3e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 18:15:15 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 18:58:28 2025 -0700" }, "message": "loop: install hooks relative to repo root, not working dir\n\nAn evergreen git bug.\n\nFixes missing hooks when started from a subdirectory.\n" }, { "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": "ed17fdf0b3ea6c0b8a8f96125ccc785c5587bb9c", "tree": "d6463e625085f73143798b50c9fb2eb30b9c04ab", "parents": [ "160fb06bb7c8df956daecd4d3bdb3f94d8a93406" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 17:26:07 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 11:46:11 2025 -0700" }, "message": "loop: fix data race in Agent.CurrentStateName()\n\nThe CurrentStateName() method was directly accessing the stateMachine\u0027s\ncurrentState field without proper synchronization, causing data races\nwith TransitionWithEvent() which writes to the same field while holding\na write lock.\n\nThe StateMachine struct already provides a thread-safe CurrentState()\nmethod that properly acquires a read lock before accessing currentState.\n\nThis fix changes CurrentStateName() from:\n return a.stateMachine.currentState.String()\nto:\n return a.stateMachine.CurrentState().String()\n\nThis eliminates the race condition between HTTP handler goroutines\nreading state for SSE streams and the agent loop goroutine writing\nstate during transitions.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sadf9961b9f084fa7\n" }, { "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": "456d5f9d4213deab28a1f528cb8edc3a1f8d5262", "tree": "5529284acc56079240e1f2780c210f4431d9ea32", "parents": [ "14fe75d6ece5116b3887b6fc027e564c4de518e6" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 11:29:50 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 11:29:50 2025 -0700" }, "message": "Remove \"restart conversation\" backend functionality\n" }, { "commit": "f28729932fdf9ecc67d3fabbfca297178a323a14", "tree": "5ae2eca28c1c139a95dde0648a17eeaba3e60bce", "parents": [ "716bfee93847e19465723af2d7c553d23c9984df" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 10:35:28 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu May 22 11:28:35 2025 -0700" }, "message": "Refactor agent git state into its own struct to tease apart its locking a bit.\n\nI want to invoke calling the git state when editing files, and that\nrequires separating it somewhat from the agent\u0027s messy and coarse\nlocking.\n" }, { "commit": "716bfee93847e19465723af2d7c553d23c9984df", "tree": "6a14fd9aa287ec8fe7a1b50655a3a864151da43d", "parents": [ "bc8c8dc5bc7abca6fa523c5ca45c9fd2c09384c2" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 18:32:31 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 21 18:32:31 2025 -0700" }, "message": "Fix agent init of git repo from recent change.\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": "35c72bcbf69cce28395bc34236df8ef9d615c0da", "tree": "c56e26f5ce42cacf9630f3187a98fd356a469a7b", "parents": [ "7ce7b0294ba1efe6c7b3d462e905c1b2a7057a37" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Tue May 20 11:17:10 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 20 20:58:51 2025 -0700" }, "message": "loop: send modified over-budget message\n" }, { "commit": "9daa5183df5270beaf5611c20e4d77cd7cfe21f7", "tree": "84bd4bf7d7bfbc723925bfae2af828cd9e7ccd0b", "parents": [ "55c87e3ababb0800b5710cbb11bec4dea8c13515" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 16 18:34:00 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 20 14:27:29 2025 -0700" }, "message": "experiment: remove llm_review experiment and associated code\n\nIt creates confusion for the agent and doesn\u0027t provide much value.\nPerhaps with better prompt engineering, but I can\u0027t invest\nin that much right now.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9f2a8c418876c498k" }, { "commit": "a8322202f36c35f677b8834cdce8f21ab3d8c6dc", "tree": "449d31ca4023b97fa87b5831c5a071864406e516", "parents": [ "87d29ef5b027120ea0857a9aa5661a3fab7b25b9" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sat May 17 06:54:34 2025 -0700" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sat May 17 06:54:37 2025 -0700" }, "message": "loop: only do in-docker git init if connected to host git\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": "a4092d26857aa5aeba70813e8942f321232e987c", "tree": "3d8caf262c8f0b103010a2450bfea896bb07417e", "parents": [ "74d690e7b4794f2c164a9da30dd106a002b935d9" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 18:32:53 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 13:17:39 2025 -0700" }, "message": "loop: make about_sketch not experimental\n\nseems to work ok enough to unleash it on everyone else..." }, { "commit": "74d690e7b4794f2c164a9da30dd106a002b935d9", "tree": "412f1ad777e599ed61904c7f80c53859315babab", "parents": [ "039fc34bd9c8f7cf3e9f3c9aeee2e9677ce28e00" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 18:16:03 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 12:40:29 2025 -0700" }, "message": "claudetool: remove knowledge base, focus on about sketch\n\nWe should have a more general kb.\nMeanwhile, this is important and standalone.\nMake it all clearer and sharper.\n" }, { "commit": "039fc34bd9c8f7cf3e9f3c9aeee2e9677ce28e00", "tree": "d04e1be0103e486d0e1905e4aa431b7cc1820640", "parents": [ "6ada85d6c4401ca82e3809fbd3af0178de1b7b11" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 21:24:12 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 12:40:08 2025 -0700" }, "message": "loop/agent: add git commit hooks when running inside container\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8d03af4abcafd1dbk" }, { "commit": "f7bebdd0a51bfe6f4148d7717e833d8d3e4fe5a5", "tree": "2e280111e0e016971322952860bc76e7402ab051", "parents": [ "124c9421ab7650020428f5b26348a963b49f107f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 15:22:24 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 12:24:07 2025 -0700" }, "message": "loop: attempt to stop sketch from bouncing around between branches" }, { "commit": "0e5b8c69d3d4436a8b591f48fd1a031a3912585a", "tree": "186b38b488d1e4b0d8e4b2468c672b6fb7f79672", "parents": [ "64f2aa8db137ee801120fe38b19f60103a2326dd" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 20:58:20 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 14:05:46 2025 -0700" }, "message": "experiment: make memory (dear_llm.md) always enabled\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2bbca9bb0adeb9eek" }, { "commit": "64f2aa8db137ee801120fe38b19f60103a2326dd", "tree": "b322e422ac17af0e9ed0e59ea9f6d91d40291f5c", "parents": [ "b81d7d476ff2d104d34d3a637f2bd826a2a89eaf" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 18:31:05 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 13:53:29 2025 -0700" }, "message": "loop: make multiplechoice tool calls end the turn\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8d507faf9c095824sk\n" }, { "commit": "a5c971e132650339c92535106ffee77a95c051f8", "tree": "b3dab67621b2f79c5b4800dc66c857c0d44ca12d", "parents": [ "7013e9ee282ef58104f91d64d85d4aec62f9c022" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 10:49:08 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 13:44:23 2025 -0700" }, "message": "loop: make multiplechoice tool just a var\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": "44d1f1ad69d7db4df1249b9e37a7f747e610ba59", "tree": "10032ac0eaa586036381e8889dae533d5802d447", "parents": [ "385da541a187a3499bf8d6368a66c2ba1109c398" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 12 19:18:32 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 12 19:18:32 2025 -0700" }, "message": "loop: instruction the agent not to make new branches\n\nThings always get muddled when it starts making lots of branches.\n" }, { "commit": "31785aed38e0f63fa736430c6240c8a76b8285dc", "tree": "35916d3dbedaeac296ece3dd57610efd82c99c8d", "parents": [ "5c7f95714f34ea327b8f300238da6491dedd6adb" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 06 01:50:58 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 12 19:17:50 2025 -0700" }, "message": "loop: add knowledge_base tool for on-demand information\n\nThe knowledge_base tool provides a way for agents to access specialized information\nwhen needed. Initial topics include:\n\n- sketch: how to use Sketch, including SSH, secrets, and file management\n- go_iterators: information about Go\u0027s iterator feature added in Go 1.22\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "e577ef7f883e2b8d54cd6862a23b2f42510d33a0", "tree": "30b11d3880b5425ec2e64780858628be54159f23", "parents": [ "7a1136201c3444009e2eeda2fb13cb76c943443f" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon May 12 10:29:00 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon May 12 10:29:00 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "7a1136201c3444009e2eeda2fb13cb76c943443f", "tree": "479b870608115a330cd4da100e988bfd2d198887", "parents": [ "a9a786b458e989a1ea2d6e8e2b35fb24b79de2cf" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Mon May 12 10:58:45 2025 +0100" }, "committer": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Mon May 12 11:23:12 2025 +0100" }, "message": "loop: on git fetch or checkout failure, retry without hooks\n\nFix #10 by removing git hooks on git fetch or checkout failure to solve problem where hooks might require dependencies not available in the container (e.g., git-branchless). This prevents errors during container initialization when git operations trigger hooks.\n\nWe can revisit this approach if we find users who need their hooks to run in the container environment.\n" }, { "commit": "80b488d853e2766b638e65cfe44a5904d7cb24ee", "tree": "abd280ec3924efb4dbbd72622129bc4af56e28ca", "parents": [ "6458e7c751856a9f7f4004367cb6274e564a1489" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 18:21:54 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sun May 11 01:22:37 2025 +0000" }, "message": "Browser tools: initialize lazily and add timeouts.\n\nAlso rename browser_screenshot to browser_take_screenshot for clarity\\n- Update both Go and UI code to maintain consistency\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8a5cabff914f88dfk\n" }, { "commit": "72252cbcb97840d724133be67c4c69cc69ebb2d3", "tree": "a361499dc3fa6b9af2be3e74cfd59fd8ba34690e", "parents": [ "7ce5fb76d8748ebf73c5adf9d6cd8eb67716fba8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 17:00:08 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 17:00:08 2025 -0700" }, "message": "llm and everything: Update ToolResult to use []Content instead of string for multimodal support\n\nThis was a journey. The sketch-generated summary below is acceptable,\nbut I want to tell you about it in my voice too. The goal was to send\nscreenshots to Claude, so that it could... look at them. Currently\nthe take screenshot and read screenshot tools are different, and they\u0027ll\nneed to be renamed/prompt-engineered a bit, but that\u0027s all fine.\n\nThe miserable part was that we had to change the return value\nof tool from string to Content[], and this crosses several layers:\n - llm.Tool\n - llm.Content\n - ant.Content \u0026 openai and gemini friends\n - AgentMessage [we left this alone]\n\nExtra fun is that Claude\u0027s API for sending images has nested Content\nfields, and empty string and missing needs to be distinguished for the\nText field (because lots of shell commands return the empty string!).\n\nFor the UI, I made us transform the results into a string, dropping\nimages. This would have been yet more churn for not much obvious\nbenefit. Plus, it was going to break skaband\u0027s compatibility, and ...\nyet more work.\n\nOpenAI and Gemini don\u0027t obviously support images in this same way,\nso they just don\u0027t get the tools.\n\n~~~~~~~~~~ Sketch said:\n\nThis architectural change transforms tool results from plain strings to []Content arrays, enabling multimodal interaction in the system. Key changes include:\n\n- Core structural changes:\n - Modified ToolResult type from string to []Content across all packages\n - Added MediaType field to Content struct for MIME type support\n - Created TextContent and ImageContent helper functions\n - Updated all tool.Run implementations to return []Content\n\n- Image handling:\n - Implemented base64 image support in Anthropic adapter\n - Added proper media type detection and content formatting\n - Created browser_read_image tool for displaying screenshots\n - Updated browser_screenshot to provide usable image paths\n\n- Adapter improvements:\n - Updated all LLM adapters (ANT, OAI, GEM) to handle content arrays\n - Added specialized image content handling in the Anthropic adapter\n - Ensured proper JSON serialization/deserialization for all content types\n - Improved test coverage for content arrays\n\n- UI enhancements:\n - Added omitempty tags to reduce JSON response size\n - Updated TypeScript types to handle array content\n - Made field naming consistent (tool_error vs is_error)\n - Preserved backward compatibility for existing consumers\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1a2b3c4d5e6f7g8h\n" }, { "commit": "a997be617bd5548a307c708cdba325ea6562acec", "tree": "1890bd46f87cf7261ba5a6aaa3fd2d110e7d67cc", "parents": [ "dce8d84fdcda62e03b13c95d7614ee7ab983ddca" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 22:52:46 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 15:32:45 2025 -0700" }, "message": "claudetool/onstart: add codebase analysis tool and basic memory support\n\nThis is a preliminary approach. Big high level questions include:\n\n* should we keep the multichoice tool prompting?\n* should we push the list of quidance files or respond with them during codereview?\n* should we use the list of docs and build files at all?\n* are there other files we should hint (e.g. editor settings, something from aider, etc.)?\n\nWe should probably also blog about dear_llm.md to stop the endless proliferation of new files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "113e2053753d5b3e08be0f3509a204fe24ba5e87", "tree": "1d86ecce8062bd94d0daeb6052623cde019f5e99", "parents": [ "e97a8e54e0c7e5c5cabf29a00bce6a81094a6ca9" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 09 21:59:40 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 09 15:06:33 2025 -0700" }, "message": "git: add retry logic for checked out branch conflicts\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se1c139889d8c3e3fk\n" }, { "commit": "e97a8e54e0c7e5c5cabf29a00bce6a81094a6ca9", "tree": "166a672959cbee9c81e56e720eed3572dc375505", "parents": [ "51e8e2b0c970ef488094a33b3c259583fe142dc5" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 09 14:53:33 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri May 09 15:06:33 2025 -0700" }, "message": "git: Make \"git fetch sketch-host\" update origin refs as well.\n" }, { "commit": "9320265a8e0ae7fb3091985c1999ed43bdae9525", "tree": "25a424999bee4597b971764640985585ce46f792", "parents": [ "4d54493fe3808ecd0c6a9a4d0bbcc7786e97b094" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 08 02:05:57 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 08 12:34:07 2025 -0700" }, "message": "loop: update git commit instruction to use --trailer, add Change-ID\n\nWe have run \u0027git config\u0027 for the user, so we can trim those instructions and simplify.\n(And in unsafe mode, the user has \u0027git config\u0027 set up anyway.)\n\nInstead of prompting the model to manually add \u0027Co-Authored-By\u0027 as a line in the\ncommit message, now instruct it to use the --trailer option.\nThis streamlines adding a Change-ID trailer with a random string (s\u003crandom_hex\u003ek format).\n\nI\u0027d actually like to use precommmit hooks to automatically do all the trailer,\nbut that doesn\u0027t play nicely with -unsafe, so for now, do it this way.\nAt least we\u0027ll have Change-IDs that we can start using.\n(Pity the official change-id support in git hasn\u0027t landed yet.)\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9b68cbfa4c7eeb82k\n" }, { "commit": "4d54493fe3808ecd0c6a9a4d0bbcc7786e97b094", "tree": "9877409fc95aaa32aed882426ea15464b8036e34", "parents": [ "a4ad8af8b08a54326bbbd99d57110a42c459c54e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 13:33:53 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu May 08 19:25:30 2025 +0000" }, "message": "all: support hiding subconvo output\n\nSome of it is systematically noisy.\n" }, { "commit": "b529e733365034174f0e5c99e3f0fce347b34c5d", "tree": "69491005dfacfdbabfbb244ac72468d85a5f7a99", "parents": [ "d2f54c2e43ad3ad2c9b389068c1555c3e7f231f2" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 22:06:46 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 19:17:50 2025 -0700" }, "message": "loop: disable multiple choice tool in oneshot mode\n\nThis change disables the multiple choice tool when Sketch is running in one-shot mode.\nIt adds an OneShot flag to the AgentConfig struct and conditionally includes\nthe multiple choice tool based on this flag.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "6aaf6afdee5d097c23d5e80314ea13cf37bea491", "tree": "add2c3678c614dd45b718ef1ff636bd2b012e8b2", "parents": [ "6ab6fce0d5309447156935c39c3d29dc4977c49d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 20:47:13 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 14:04:30 2025 -0700" }, "message": "experiment: make precommit always on\n\nRemove the precommit experiment flag and make commit message style guidance\nalways available when using the precommit tool.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "a2a31508e1b7348a1337e2038c5560d4b1bc19d3", "tree": "7c3234c35772260a8cbee23d78ec37d7775a1fc7", "parents": [ "c3c202317359dae2647aee2bcdbe61382fa3b99f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 12:37:18 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 14:03:38 2025 -0700" }, "message": "loop: split title tool into title and precommit tools\n\ntitle wants to be called early, as soon as the topic is clear.\nprecommit wants to be called late, just before first git commit.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "4962f153fa0d6812543addd690be8dba8c04a406", "tree": "3a90c5736422c77f8ef3dc8d552640e65f1e07a0", "parents": [ "8b43ffbf41d06aee5ad5c734f1661a3b62cfe1a9" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue May 06 17:24:20 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue May 06 17:24:20 2025 +0000" }, "message": "all: fix formatting\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": "5cca56ff3aec7785494c6a9fcd4e846900968ed1", "tree": "25b99a05ae7450fc210110414252210d189ca993", "parents": [ "4fcde4a88794aec937fd116d6059871297d974e7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 06 01:10:16 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:32:34 2025 -0700" }, "message": "loop: migrate system prompt to Go templates\n\nIt was bound to happen eventually.\nBite the bullet now.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "4fcde4a88794aec937fd116d6059871297d974e7", "tree": "c64431e4d72508f414c4dea441e47b079e5476fe", "parents": [ "36a5cc15faa43d028f87706ca286adb7f5109120" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:28:13 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:28:13 2025 -0700" }, "message": "loop: clarify tool-ends-turn logic\n\nIt mis-handled ends-turn-tools in subconvos.\nThe multi-if was confusing.\nIt missed an early return.\nGenerally unindent.\n\nMost of this doesn\u0027t matter, but when I noticed the subconvo\nand the weird if structure, I just kept going...\n" }, { "commit": "36a5cc15faa43d028f87706ca286adb7f5109120", "tree": "a88ca83039990dd73f9582b6c1428af72da69e91", "parents": [ "d42577f657516eba00e41bd266d581b88e28ae05" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 17:59:53 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:20:24 2025 -0700" }, "message": "loop: don\u0027t rename title tool, even with precommit experiment\n\nDoing so breaks various UIs.\n" } ], "next": "c72ceb21966c5e7685db6c9bf28074042265d683" }