)]}' { "log": [ { "commit": "49b7b0ae150e08ed37b1318d883e4d6edeaadf5f", "tree": "9b1733142c865d62b3b52c40c7838b2488197cb6", "parents": [ "f9d7fc55fe9c8ba2bd818fbc7577f67868fbeb4f" ], "author": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Mon Jul 07 13:52:35 2025 +0400" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:08:03 2025 +0400" }, "message": "nodeId\n\nChange-Id: I6d8baf1f3fd0ff26a9c2a01bd7a1c1c59643b6d6\n" }, { "commit": "f9d7fc55fe9c8ba2bd818fbc7577f67868fbeb4f", "tree": "3a23d9edd6aff1df96aee6496272dbd43cace8aa", "parents": [ "05569839cafa5805e3ead6d6280b8afa283dc4b6" ], "author": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Mon Jul 07 07:09:42 2025 +0400" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:08:03 2025 +0400" }, "message": "deploy draft\n\nChange-Id: Ied7209985584e121e8874c1562344d6cdd14ab49\n" }, { "commit": "05569839cafa5805e3ead6d6280b8afa283dc4b6", "tree": "eda598e52b4f9b501338781894e03902f69aaacd", "parents": [ "6a4ca20cfe260d9de71938b5f74528584d3b78c9" ], "author": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Sun Jul 06 09:16:29 2025 +0400" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:08:03 2025 +0400" }, "message": "dodo: save config tool\n\nChange-Id: Ia8c8639cda093a3776ff018c2b77e63449cbc78c\n" }, { "commit": "6a4ca20cfe260d9de71938b5f74528584d3b78c9", "tree": "c0d9618c39c52f0541061ccce9597d77404125df", "parents": [ "4a89b33b0fcd7d42d65d8854f7be3f5d63730465" ], "author": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Sat Jul 05 20:06:27 2025 +0400" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:08:02 2025 +0400" }, "message": "initial interaction\n\nChange-Id: I1ccaec03045390076191bd0d1b94ab0044398805\n" }, { "commit": "4a89b33b0fcd7d42d65d8854f7be3f5d63730465", "tree": "18e7ae76ced73ef0492ebac7d6ec723e6ad2abec", "parents": [ "3d0067f501e6c84622146b88b38b9da5c7b36cc1" ], "author": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Fri Jul 04 18:35:34 2025 +0400" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:05:59 2025 +0400" }, "message": "Update system prompt\n\nChange-Id: I88445397f487b8b8dc4aa765b9bbe7c265f8f20f\n" }, { "commit": "fe6e714d9c0d61ac802e5fab849857ce26ae5ab9", "tree": "954bf8795762d137d620966b4f9174a02197137d", "parents": [ "3050307b9ddf4cec0227fca34440b7796533ae1b" ], "author": { "name": "gio", "email": "gio@v1.dodo.cloud", "time": "Wed Jun 18 08:51:23 2025 +0000" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:05:57 2025 +0400" }, "message": "dodo: implement validate and deploy tools\n\nChange-Id: I026cff45159dd1e6fa6a9bcef7c843564bcf9c8b\n" }, { "commit": "3050307b9ddf4cec0227fca34440b7796533ae1b", "tree": "41526fa89741ef3c5f27376f829d2190563de7d0", "parents": [ "a271a21dec3abec9543f7fbacb5e0de1f329d5dc" ], "author": { "name": "gio", "email": "gio@v1.dodo.cloud", "time": "Tue Jun 17 10:50:15 2025 +0000" }, "committer": { "name": "Giorgi Lekveishvili", "email": "lekva@gl-mbp-m1-max-2.local", "time": "Thu Jul 31 09:04:19 2025 +0400" }, "message": "dodo: implement dodo_get_project_config\n\nChange-Id: I8167f83d776d43d97c5b2a744524f88e517c2107\n" }, { "commit": "a271a21dec3abec9543f7fbacb5e0de1f329d5dc", "tree": "0b65b9b97c1ecf0f1e0ab7d9c8034ffc51fc14d0", "parents": [ "b44a162af5ecd96bb015430344a15ae8496362f2" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 23:08:00 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 31 04:57:12 2025 +0000" }, "message": "claudetool: streamline browser tools\n\nbrowser_click, browser_type, browser_get_text, browser_scroll_into_view,\nbrowser_resize, and wait_for can all be easily implemented using browser_eval,\nif browser_eval is given an await parameter.\n\nA bit of testing suggests that they are more robust in practice\nthat way, and that multiple tool calls can be combined\ninto a single browser_eval call, which reduces latency.\nAnd Sketch does in fact use them as needed.\n\nAlso, bump up timeouts; empirically, 5 seconds is not enough.\n\nAs a bonus, fewer tools is better for context management.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8d8cd418f3e97f26k\n" }, { "commit": "b44a162af5ecd96bb015430344a15ae8496362f2", "tree": "9aea1f38f1dd5943b119608786e69e19666dcc4e", "parents": [ "994e9844862b85015c07907c17764d3556164090" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 17:52:14 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 21:48:05 2025 -0700" }, "message": "claudetool: remove unused browser tool arg\n" }, { "commit": "994e9844862b85015c07907c17764d3556164090", "tree": "7818b6f3494136e6c94733980801f9b7fe81f378", "parents": [ "7f18fb61ea479598bf9c2c89e92cec636f45070d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 20:26:47 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 21:37:38 2025 -0700" }, "message": "claudetool: add simplified patch support\n\nFor weaker models.\n\nAlso, improve fallback parsing introduced earlier.\n" }, { "commit": "7f18fb61ea479598bf9c2c89e92cec636f45070d", "tree": "0562edb5a1423b42e0d42f0bcdc9641adc1e22a0", "parents": [ "dba25eeb00ab76b97eac1cb4576229266ab7e1a1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 18:12:29 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 21:16:33 2025 -0700" }, "message": "claudetool: add clipboard support to patch tool\n" }, { "commit": "534783d6d14bb11b7949a4d750ad2809afc7c7a3", "tree": "099e0d8e6fdc40396c6270e24b1239e5e94fd713", "parents": [ "112c60d7c87f09806fce6056af9fefd6f5814d0a" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 29 08:17:37 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 18:39:17 2025 -0700" }, "message": "claudetool: minor patch fixes\n\n- docs\n- actually use the more lenient parser in aae47584ce87e0f3e6d9c59a26433f515002816a\n- tweak tool description\n" }, { "commit": "112c60d7c87f09806fce6056af9fefd6f5814d0a", "tree": "c86dea160b10daebce1717a9ee2bc418cd8e44ca", "parents": [ "64aaf90ca1ba1b867e240ccbce461f3015694958" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 31 01:35:43 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 18:38:57 2025 -0700" }, "message": "loop/server: use pre tags for json schemas in debug page\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3ef91294c48fdad4k\n" }, { "commit": "64aaf90ca1ba1b867e240ccbce461f3015694958", "tree": "9baaff4ebd91bcceaf22ccd8be1dbc5e24f47442", "parents": [ "2c3e02c12535b737b5e82c16b82247759826bd97" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 31 01:26:16 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 18:38:57 2025 -0700" }, "message": "loop/server: use pre tags for tool descriptions in debug page\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sf5897ffabc6f6045k\n" }, { "commit": "136b66dbe453549cc411abc6e6b2144cfb368add", "tree": "49565876ae80d0cc06564b4fd301946f1027d267", "parents": [ "04f16a5af290588dd05f9f71da1f320711f625ff" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 11:48:58 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 12:19:56 2025 -0700" }, "message": "claudetool: accept relative paths in patch tool\n" }, { "commit": "04f16a5af290588dd05f9f71da1f320711f625ff", "tree": "32ab42aecb00ef87a989de9b7287e500deeee38e", "parents": [ "bdc6889f886667b06236fb20c5d080e3ac1f1eb8" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 11:46:25 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 12:19:56 2025 -0700" }, "message": "claudetool: make it easier to parameterize patch tool\n" }, { "commit": "bdc6889f886667b06236fb20c5d080e3ac1f1eb8", "tree": "f86ee68725cba394d1df667723a9a3dad49c35a7", "parents": [ "6fe809c2282f5d7e258ab60f4ff4d0b3e0739332" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 28 17:28:13 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 30 18:53:52 2025 +0000" }, "message": "sketch: add support for \u0027external\u0027 message types\n\n- adds a new CodingAgentMessageType for loop.AgentMessage\n- adds an new /external handler to loophttp.go\n- modifies Agent to pass the .TextContent of ExternalMessage into the convo\n as though it came from the user.\n- adds sketch-external-message web component, with a template for\n github workflow run events specifically.\n- adds demos for sketch-external-message\n" }, { "commit": "6fe809c2282f5d7e258ab60f4ff4d0b3e0739332", "tree": "1f9a0bccf5ab23ff15eb38c26bda8c9d4e3dfb3d", "parents": [ "4a370aa1fe7a2fcafc41c0cd1bb4bf59bfb070a2" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 16:22:51 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 11:30:10 2025 -0700" }, "message": "claude: bash tool smorgasbord\n\nThis is a combination of a bunch of changes and bug fixes\naccumulated over a week. It decided not to spend a bunch of time\nteasing it apart into its components. I apologize.\n\nSignificant changes include:\n\n- clean up background bash execution\n- improve documentation\n- remove TODO we\u0027re not gonna do\n- add a \"process completed\" note to stderr\n- convert background result printing to xml-ish\n- combine stdout and stderr in background bash to match foreground use\n- hint to sketch to kill the process group\n- add missing timeouts propagation\n- tell agent that bash calls are stateless\n- thread pwd through explicitly\n- unify command creation\n- speed up missing command installation\n I tried a bunch of different ways to prompt engineer this to be faster.\n But Claude will be Claude. Solution: switch from agent to one-shot.\n\n This is marginally more brittle, and can only use a package manager,\n but that also prevents a bunch of possible curl|bash mess, etc.\n And this was always best-effort anyway.\n\n It\u0027s now MUCH faster to fail on non-existent commands, and about 2x\n faster on the real commands I tried (yamllint, htop)...now mostly down\n to the irreducible work involved in actually doing the installation.\n- remove SKETCH_ from bash env, except SKETCH_PROXY_ID\n- delay kill instructions until actually needed\n- add simple GIT_SEQUENCE_EDITOR\n- overhaul cancellation\n- explicitly disable EDITOR to prevent hangs\n I have big plans here, but this will do for now.\n- simplify and unify handling of long outputs\n- switch to center trunctation of long outputs\n- add zombie process cleanup using unix.Wait4\n Wow, I tried a bunch of things here.\n\n When running as PID 1, we are responsible for reaping zombies.\n Unfortunately, we can\u0027t do this in the simple/obvious way,\n because simply listening for SIGCHILD and reaping races\n with running cmd.Wait. We can\u0027t use a separate init process\n or double-init sketch, because then we lose our seccomp\n protection, and there\u0027s no particularly good way to extend it.\n\n Instead, (h/t to Philip asking a good question), observe\n that we are in a very controlled environment, and pretty much\n the only way to get zombies is via the bash tool.\n So we add reaping tied specifically to process groups started\n by the bash tool, with an explicit understanding of their lifecycle.\n\n Auto-installation of tools still creates zombies.\n We now know how to fix it, but it is rare, so who cares.\n" }, { "commit": "4a370aa1fe7a2fcafc41c0cd1bb4bf59bfb070a2", "tree": "995171cbf32df2c5b5973a942def8456f075c533", "parents": [ "c3e6df8e2eb874b6ddc59beac72a1557c5fef7e1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 28 23:19:48 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 10:25:01 2025 -0700" }, "message": "loop/server: add /debug/system-prompt, move logs to /debug/logs\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sdadfdbbcd1589be1k\n" }, { "commit": "aabca2e901349b88391b3bd5c255687012ae52bd", "tree": "17e84913c0a40336929834cc92179307387776ba", "parents": [ "72cf08c81fb2a2c64c654d9ca660b30a0cd4d284" ], "author": { "name": "Euan Kemp", "email": "euank@euank.com", "time": "Mon Jul 21 05:44:44 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 09:57:11 2025 -0700" }, "message": "all: update shebangs for more portability\n\nNixOS by default doesn\u0027t ship with /bin/bash, rather /usr/bin/env bash is the way to get the right bash.\n\nThis caused two issues on NixOS:\n\n1. The `push` button on the web interface would fail with ` ! [remote rejected] \u003ccommit\u003e -\u003e \u003cbranch\u003e (pre-receive hook declined)`\n2. `make` would result in an error\n\nOn the host side, for 1 you could see `fatal: cannot exec \u0027/tmp/sketch-git-hooks-3993344797/pre-receive\u0027: No such file or directory`, and for 2, you could similarly see `make: ./build/webui.sh: No such file or directory`\n\nThis fixes both of those errors by updating to more portable shebangs.\n```\n" }, { "commit": "1e5516701e0135557a5de6604a7bf9631a4b3a67", "tree": "255d53b03dc42a313b94c9565aeb6c1e334063e6", "parents": [ "9b1c003e7374003edd7f7316c08d494a8aa06b2b" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 03:16:54 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 09:46:27 2025 -0700" }, "message": "sketch: add fetch-on-launch flag for git fetch on startup\n\nDefaults to true, i.e. no changes.\n\nBut now I can turn it off so that running Sketch\ndoesn\u0027t pop 1password asking for ssh keys.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s494a69769c279ad8k\n" }, { "commit": "83cf606957e8d05f1dc690991a178b302d848877", "tree": "e5b0bd4133847e0c551520a52e618c0b39148648", "parents": [ "02f4820e4a5245f545293c34e121d52640d3cb11" ], "author": { "name": "philz", "email": "philip.zeyliger@gmail.com", "time": "Mon Jul 28 14:23:04 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 28 14:23:04 2025 -0700" }, "message": "loop: add /debug/tools handler for mcp and tool inspection\n\nAdds a new debug endpoint at /debug/tools that displays all available\ntools including their names, descriptions, JSON schemas, and metadata.\nShows both built-in Claude tools and MCP tools with proper type indication.\nIncludes nicely formatted HTML with syntax highlighting for schemas.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s10d266ddbc5a7fc0k\n" }, { "commit": "9224eb0571146da5fce2c1ec0f7f13a72f30cc3a", "tree": "e1923e690cb165fd65c438f25b62ffa0b700a0d3", "parents": [ "7cae9d5d4c629d3c1f85aaab974632e093a8959d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Sat Jul 26 04:45:05 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 21:47:02 2025 -0700" }, "message": "loop: change current_datetime to current_date\n\nSo that we don\u0027t always invalidate the k/v cache.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s330e5fc0212d37d4k\n" }, { "commit": "e34ffd60e0a8dfc5ba55be35d68a396a73d53750", "tree": "208c68febe0f077054d1709cc732192aee367413", "parents": [ "783ab31ae38604be5331ec030264dcec08574aae" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 13:20:49 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 25 13:23:37 2025 -0700" }, "message": "sketch: fix diff view editing of gitignore\u0027d files and forward more http errors to logs\n\nFixes https://github.com/boldsoftware/sketch/issues/213\n\nWe had \"sketch\" git ignored, so \"git add sketch/cmd/sketch/main.go\" was\nfailing when a user was editing it in diff view. The gitignore was\nincorrectly specified. (\"git ls-files -i -c --exclude-standard\"\nreturning main.go should have tipped us off, but who knew!)\n\nAnyway, fixed that, and improved the logging.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3ed65211dd497f76k\n" }, { "commit": "783ab31ae38604be5331ec030264dcec08574aae", "tree": "39b46c979117377be2676c995b38f8854eb0cd8b", "parents": [ "8a0de526a223c834cb7323b4e1717f40f0d514d4" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 07:22:38 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 13:02:34 2025 -0700" }, "message": "loop: tone down some date handling\n\nLest it get out of control.\n24 hours is a long time.\n" }, { "commit": "8a0de526a223c834cb7323b4e1717f40f0d514d4", "tree": "4a2cdef6e018d52b94d57ec6769f038bca70aea2", "parents": [ "40c9da832c33b2d807c06f54eeccabb38fbe01d1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 19:29:07 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 13:02:34 2025 -0700" }, "message": "loop: add current datetime to agent system prompt\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1fb7648c40f24f62k\n" }, { "commit": "4571fd6d06f65195341791b46d84dc80bb0676b3", "tree": "6e61da4d6a9ed29891c0b84f503ff5ccb5441741", "parents": [ "c540df7f730eb0e07a5aa543ec128dba38a7cf81" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 16:56:02 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 12:06:18 2025 -0700" }, "message": "webui: add model name to info popup k/v pairs\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s89edebe17dd6d6edk\n" }, { "commit": "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": "9c74cb9f4865a28d468fdeab0e5755a882e55409", "tree": "c53473ee57a0ccfdfdd7549bc43a6b6f33554da1", "parents": [ "b7bd8d861748ee16a13bd9537a34d13336bd0fc1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 12:52:04 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 15:16:46 2025 -0700" }, "message": "loop: rework done tool\n\nSmaller, simpler schema.\nEliminates $ref, which some LLMs don\u0027t like / don\u0027t understand.\nSeems to work just as well.\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": "c67d7bced3a3619cbf031075fa8442a696381c4a", "tree": "27e8b6990244865fd3d020ea1fd4ef56f61fc603", "parents": [ "3b44cc3120b595e82dcc2a6081b8a07fc24c739e" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 23 10:59:02 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 23 19:09:31 2025 +0000" }, "message": "webui/src/data: handle ended and read-only sessions\n" }, { "commit": "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": "d1b7dd616f47c873e241d2354f03feacce46e637", "tree": "ca542f2d24021707404ccea1083a8b9bff0b8a9a", "parents": [ "43b60b9b6906a2077175bcb1c3ea53b90d92a9f8" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:36:23 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:46:45 2025 -0700" }, "message": "loop: skip TestPortMonitor_ShouldIgnoreProcess on macOS\n\nIt doesn\u0027t work, and can\u0027t be made to work.\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": "a8561f70c1701376e88a5b63d78e7d09a7ae3ed8", "tree": "6093a6ba26567089bf33a70f74399eca2c149391", "parents": [ "b58bbf30db0ced33cc110dd44d6c14995977737f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 15 23:47:59 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 16 13:08:59 2025 -0700" }, "message": "sketch/webui: add untracked files notification to diff view\n\nAdd warning in diff view about untracked files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s45fbbdf5b3d703e4k\n" }, { "commit": "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": "9b39aa66590f68f9039a8efc0c4e1917551016e6", "tree": "6ac5e639d348bf2b2a3017abbc147411b79cf9bf", "parents": [ "333aa67a4c1639f49c7ba03edab4ace3c6439e2b" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 11:56:02 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 11:56:24 2025 -0700" }, "message": "sketch: exclude internal processes (headless-chrome) from port monitoring\n\nAdd SKETCH_IGNORE_PORTS environment variable to headless-shell browser processes\nand modify port monitoring to exclude processes with this variable.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sff3b145df27ee3bek\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": "fadffe3cfb867760be035e72b7096946554255a1", "tree": "eb66acbe6f8d6f9825a1da92a3b9df0b3b7f0901", "parents": [ "e3c0f4d054cc7ec2859cffd3f61383d8a122a53c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 10 00:08:38 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 19:23:57 2025 -0700" }, "message": "sketch/loop: fuss more with /git/cat and 204s\n\nFor unknown reasons, switching to 204s made the diff view noticeably slower.\nBe more explicit about things in the hope that it helps.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s52c5ab6557e1f9dbk\n" }, { "commit": "a002a23a60dcc21353681a887a90cd12867e7f44", "tree": "e554f73faa6681dec2cbee396f30b03b972ab66b", "parents": [ "5ab8fb8dc17299df5d134d7a95f33d41b170f102" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 19:38:03 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 09 14:37:14 2025 -0700" }, "message": "webui: delete non-gzip\u0027d assets\n\nReduces webui assets from 83M to 16M.\nAlmost all browsers support gzip.\nFor those that don\u0027t, we provide on-the-fly decompression.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sbc10790564044d6bk\n" }, { "commit": "5ab8fb8dc17299df5d134d7a95f33d41b170f102", "tree": "bbc6f1a77e2eeffc2ab759270711e0fbe6a9118a", "parents": [ "b7ec9c837a549abd9bbbc1230e6d96090c98af52" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 09 12:34:55 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 09 12:42:12 2025 -0700" }, "message": "webui: display context window size in info box\n" }, { "commit": "b843146dd360f53c2feb2942d4a6ef4ccc9109fb", "tree": "1be327ecc49991def691d5fd19ba0486dc678654", "parents": [ "e7ca73df4b61cff1847c4f63feac89c580e998c7" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jul 09 13:10:32 2025 +1000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jul 09 13:14:12 2025 +1000" }, "message": "server: Wait on terminal process for cleanup\n\nThe bash shell for the termianl is an exec.Cmd that is Start()ed\nin the github.com/creack/pty package. We do a lot of cleanup\nwhen that shell exits, but we never Wait() on it (and neither\ndoes the pty package), so the bash process gets left behind\nas a zombie.\n\nFixes boldsoftware/sketch#181\n" }, { "commit": "5c29b3e17fa526aa912d3ba400cf8a1a9c83fd80", "tree": "a57a2cf6f5420e866c4cd86d8184d0d2dd8b87ad", "parents": [ "1c18ec94b0d2edcc6d69d1b96f89e3c6e630fc11" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 18:07:28 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 13:44:11 2025 -0700" }, "message": "sketch/loop: return 204 for not-found files in /git/cat\n\nThis isn\u0027t actually an error; no reason to return a 500.\nThe front-end handles this identically,\nbut without js console log spam,\nwhich is the motivation for this change.\n" }, { "commit": "1c18ec94b0d2edcc6d69d1b96f89e3c6e630fc11", "tree": "994dbcf813d9c772da320ccccfc2e2f5fe826155", "parents": [ "e1a6e1b25d1cf7f3aec4b98e155be7cf3701a1b0" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 10:55:54 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 13:06:43 2025 -0700" }, "message": "all: use make to build\n\nThis overhauls the build system.\nWe used to use a just-in-time clever build system\nso that \u0027go run\u0027 and \u0027go install\u0027 Just Worked.\n\nThis was really nice, except that it make it\nall but impossible to ship a single binary.\nIt also required our uses to install npm,\nwhich some folks have an understandably negative reaction to.\n\nThis migrates to a makefile for building.\nThe core typescript building logic is mostly still in Go,\nand untouched (boy did I learn that lesson the hard way).\n\nThe output is a single file that includes the webui, innie, and outie.\n\n(There are still very mild shenanigans in which we write outie\nout to a temp file and then \u0027docker cp\u0027 it into the docker container.\nBut this is pretty manageable.)\n\nThere are some significant follow-ups left after this commit:\n\n- convert the nightly release builds to use the makefile\n- lots of dead code removal\n- maybe add -race support using a dockerfile for the cgo compilation\n- maybe use \u0027docker cp\u0027 stdin reading with tar to avoid the temp outtie file\n- all the rest of the \"better release\" todos (brew install, etc.)" }, { "commit": "454f8864b1830ff13f19a49dd53d91b5a201baf7", "tree": "d52cec3d8a26fe8f6f0883008aaef86b425e55cb", "parents": [ "289525bcecca82247e0908c5ff1f213213d9c9a4" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 04:12:28 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 21:14:34 2025 -0700" }, "message": "sketch/loop: remove flaky TestPortMonitor_IntegrationDemo test\n\nDelete the integration test that was still flaky despite previous fixes.\n\nRoot Cause:\n- Test suffered from inherent race condition with system-wide port resources\n- After closing test ports, other processes could immediately bind to same port numbers\n- Race condition window existed between port closure and PortMonitor detection\n- Cannot be made truly hermetic because port numbers are OS-managed system resources\n\nTest Coverage Analysis:\n- Integration test added minimal unique value over existing unit tests\n- Agent.pushToOutbox already tested in agent_test.go\n- Port detection logic thoroughly tested in port_monitor_test.go\n- TCP filtering, sorting, added/removed detection all covered by unit tests\n- Core functionality has excellent test coverage without the integration test\n\nThe Fix:\n- Remove the flaky integration test entirely\n- Preserve all existing unit tests which provide reliable coverage\n- Eliminate CI/CD reliability issues caused by environmental dependencies\n- Remove maintenance burden of debugging false test failures\n\nThis removes the flaky test while maintaining comprehensive test coverage\nthrough the existing unit test suite.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8bd05f43558fc5c4k\n" }, { "commit": "828161b106396bf87a4c481895781f1d1d732e0a", "tree": "9ced21ca57a202fc44456c7cbd53a8a113435c51", "parents": [ "c7a98d8360cf03f977625a65e8dd637d6b2ab18b" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 03:44:39 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 21:06:38 2025 -0700" }, "message": "sketch/loop: fix flaky TestPortMonitor_IntegrationDemo test\n\nRemove assumptions about system-wide port state that made the test flaky.\n\nRoot Cause:\n- Test assumed it was the only process creating/destroying ports\n- Made brittle assertions about total port counts before/after test servers\n- Failed when other processes (tests, services, containers) modified ports\n\nThe Fix:\n- Focus only on the specific test ports created by the test\n- Use polling with timeout to detect port creation/removal\n- Remove assertions about total port counts\n- Test only verifies its own test ports are detected and removed\n\nChanges:\n- Replace fixed sleeps with polling loops with timeouts\n- Remove port count comparisons that depended on system state\n- Keep track of test ports separately from system port scanning\n- Add proper timeout handling to prevent hanging tests\n\nTest Results:\n- Before: Flaky failures when other processes modified ports\n- After: 5 consecutive runs all passed, focusing only on test-specific behavior\n\nThis preserves the test\u0027s usefulness as an integration test while making it\ndeterministic and isolated from system-wide port changes.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s58312d8275959960k\n" }, { "commit": "c7a98d8360cf03f977625a65e8dd637d6b2ab18b", "tree": "043d571f69c08cc1211dbd9d85821ca488160a38", "parents": [ "553cc842a3fea3594158c8314a577c2b916bf45d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:12:53 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:13:43 2025 -0700" }, "message": "sketch/loop: fix concurrency handling in port monitor shutdown\n\nThe graceful shutdown code assumes it can take the pm.mu lock.\nMake the Stop code oblige.\n\nThis fixes the issue at hand, but the remaining code still gives\nme the heebie jeebies--for example, after calling Stop, there could\nstill be notifications that come in.\n\nSketch really shouldn\u0027t write concurrent code.\n\nThis will work well enough, I guess, until it doesn\u0027t,\nat which point we\u0027ll have to carefully fix up the rats nest.\n\n\nBefore:\n\n2m10s: 1157 runs so far, 36 failures (3.11%)\n\nAfter:\n\n16m45s: 12484 runs so far, 0 failures\n\n\nFixes boldsoftware/bold#446 enough for now\n" }, { "commit": "9f5a051b23af48263ec8fde4cc9b1c6e11532433", "tree": "4017efac74b922cc14e8e2aae4cadf879a6b0644", "parents": [ "c52a250e07c2d84173c90dc076d45f14a26bdce3" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:59:06 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:59:06 2025 -0700" }, "message": "sketch/loop: remove flaky TestPortMonitor_PortDetection test\n\nThe test was inherently flaky because it assumed no other processes would\nopen/close ports during test execution. This caused false failures in CI\nand development environments.\n\nThe test provided minimal additional value since:\n- Core port monitoring logic is already well-tested by other tests\n- It was testing system integration rather than code logic\n- The portlist.Poller is a third-party dependency\n- Making it robust would require significant refactoring\n\nThe remaining tests provide comprehensive coverage of port monitoring\nfunctionality including filtering, sorting, diffing, and lifecycle management.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sf2d9c51227a4419ek\n" }, { "commit": "e48f2bb05d937c1bcc51bca81a010173d905c0bb", "tree": "90d08fc8e4b0775ec25af38580b6b68182087f31", "parents": [ "6dc90c03abff887c09ca0418d4d493d16cf1b0c8" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 04 04:15:26 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 04 04:15:26 2025 +0000" }, "message": "all: fix formatting\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": "cb5572629b5f7ea9182e5c9f32a14dc8c5a563e4", "tree": "edeec332686af45a971fa543a7f90772cd3b9162", "parents": [ "b3aff885a06ca0bbdfb2f61c94387789355a14c1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jun 30 23:55:20 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 02 15:48:16 2025 -0700" }, "message": "browse: refactor tool responses to match claudetool patterns\n\nThis was vibe-coded, but the vibe was wrong.\nMake it look like the rest of the code,\nwhich will ease upcoming refactoring work.\nSwitch from JSON to XML-ish for textual tool outputs.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se50bf57009dfd97ak\n" }, { "commit": "a9710d76e938c29fbfaa30e5d63129bbc43b18c8", "tree": "56f7a79e58d8c5ed86695ba279f2395cf1e13e16", "parents": [ "fd67b0185a488aeac149e0bcf1a476d628134555" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 02 02:50:14 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue Jul 01 19:53:20 2025 -0700" }, "message": "sketch: proxy to ports via p\u003cport\u003e.localhost Host headers\n\nAdd support for proxying requests based on Host header patterns.\nWhen Host header matches p\u003cport\u003e.localhost, proxy the request to localhost:\u003cport\u003e.\n\n- ParsePortProxyHost() extracts port from p8000.localhost format\n- proxyToPort() handles generic port proxying with validation\n- Supports any valid port (1-65535) via p\u003cport\u003e.localhost pattern\n- Comprehensive tests for parsing and validation\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: saa324eab0e9b3addk\n" }, { "commit": "a14b0183208df257a43748b51666043db7e62138", "tree": "2d2306053dc2ae1139994eb1d49acc9b8e300c6a", "parents": [ "467c3964f42c1b29a18c7649e8aa33276710e475" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 14:31:18 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 30 14:31:18 2025 -0700" }, "message": "sketch: make /debug use relative paths\n" }, { "commit": "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": "13e385685fb87cea70711ff387e57fbe0f725f28", "tree": "618000cfa3846350455ea01f72998dc7648c6d46", "parents": [ "59789951fdf8f9a5c834b4d079f0394c2bc42dd3" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jun 29 17:53:27 2025 +0000" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jun 29 17:53:27 2025 +0000" }, "message": "sketch: update local_ssh.md to replace SSHTheater with LocalSSHimmer\n\nUpdate component naming in SSH mutual authentication documentation from deprecated SSHTheater to current LocalSSHimmer implementation.\n\nChanges:\n- Updated sequence diagram participant name from \u0027SSHTheater\u0027 to \u0027LocalSSHimmer\u0027\n- Updated implementation component description to reference LocalSSHimmer\n- Updated SSH configuration approach section to reference LocalSSHimmer\n\nThe documentation now accurately reflects the current component naming used in the SSH authentication system implementation.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s644b455057106ba2k\n" }, { "commit": "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": "cff0ff8f582a970fa8bd688448964bef8d2b84ab", "tree": "be78da943008f817a8af3ba40c34f633ce855344", "parents": [ "7e36a04e5b9eb206faad20f0a37fa111285ffdce" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jun 25 16:43:47 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jun 25 18:39:06 2025 +0000" }, "message": "sketch: add /proc filesystem fallback for port monitoring when ss command unavailable\n\nImplements /proc/net/tcp* parsing as fallback when ss command fails to eliminate\ndependency on ss being installed in container environments.\n\nProblems Solved:\n\nss Command Dependency:\n- PortMonitor.updatePortState() relied on \u0027ss -lntu\u0027 command for port detection\n- Failed on systems where ss (iproute2 package) is not installed\n- No fallback mechanism when ss command execution failed\n- Port monitoring became non-functional in minimal container environments\n\nLimited Container Support:\n- Many minimal container images don\u0027t include ss command\n- Port monitoring silently failed without providing any functionality\n- No way to detect listening ports without external command dependencies\n\nSolution Architecture:\n\n/proc Filesystem Parsing:\n- Added getListeningPortsFromProc() method to read /proc/net/tcp* files\n- Parses /proc/net/tcp, /proc/net/tcp6, /proc/net/udp, /proc/net/udp6\n- Hex address decoding for both IPv4 and IPv6 addresses\n- Socket state filtering to identify listening sockets (state 0x0A for TCP, 0x07 for UDP)\n\nFallback Implementation:\n- updatePortState() tries ss command first, falls back to /proc on failure\n- parseAddress() handles little-endian hex encoding from /proc files\n- Generated output format matches ss command output for compatibility\n- Maintains existing parseSSPorts() functionality for ss output\n\nImplementation Details:\n\nAddress Parsing:\n- IPv4: 8-character hex string representing little-endian 32-bit address\n- IPv6: 32-character hex string with little-endian 32-bit chunks\n- Port numbers stored as big-endian hex values\n- Special address handling: 0.0.0.0 and :: converted to \u0027*\u0027\n\nSocket State Detection:\n- TCP listening sockets: state 0x0A (TCP_LISTEN)\n- UDP bound sockets: state 0x07 (TCP_CLOSE for UDP)\n- Filters out non-listening connections and states\n\nError Handling:\n- Graceful fallback when ss command fails\n- Logs debug messages for command failures\n- Continues with /proc parsing if available\n- Handles missing /proc files gracefully\n\nTesting:\n\nComprehensive Test Coverage:\n- TestParseAddress() verifies hex address decoding for IPv4/IPv6\n- TestParseProcData() validates /proc file parsing with mock data\n- TestGetListeningPortsFromProcFallback() tests complete fallback functionality\n- TestUpdatePortStateWithFallback() validates end-to-end behavior\n\nAddress Parsing Validation:\n- IPv4 localhost (0100007F:0050 -\u003e 127.0.0.1:80)\n- IPv4 wildcard (00000000:0016 -\u003e *:22)\n- IPv6 wildcard and specific addresses\n- Error handling for invalid formats and hex values\n\nFiles Modified:\n- sketch/loop/port_monitor.go: Added /proc parsing methods and fallback logic\n- sketch/loop/port_monitor_test.go: Added comprehensive tests for new functionality\n\nThe implementation ensures port monitoring works reliably in any Linux environment\nregardless of whether ss command is available, using native /proc filesystem access.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s72dd58a0b3f4304bk\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": "8105fe6bfa914c17c0f474bacdbe9d4c62b6b2b3", "tree": "3ba587467d33c5f9ef78ab20fc80a3b64967733b", "parents": [ "dd6352a74f1214035785025df127ed6a0dc73ac6" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 21 21:22:34 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 21 21:23:06 2025 -0700" }, "message": "Removing noisy log line.\n" }, { "commit": "cfd0fe64e379f066b117effe84100a38c48e493f", "tree": "fed8d755bb3a8fffd61f6cd5da706c9267a7c5f3", "parents": [ "c540e8ea6a8dd2a5297345b6f9b885988338f237" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 21 02:17:41 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 20 19:21:43 2025 -0700" }, "message": "cleanup: remove unused functions and fix tests\n\nRemove unused Go code identified through systematic analysis:\n\nRemoved Functions:\n- BashRun() in claudetool/bash.go - legacy testing function\n- ContentToString() in claudetool/util.go - unused utility function\n- GetActiveTunnels() in dockerimg/tunnel_manager.go - unused getter method\n\nRemoved Types:\n- baseResponse struct in claudetool/browse/browse.go - unused response type\n\nTest Updates:\n- Replaced BashRun tests with direct Bash.Run calls in bash_test.go\n- Removed ContentToString test from agent_test.go (testing unused function)\n- Updated tunnel manager tests to access internal activeTunnels map directly\n- Fixed all compilation errors caused by removed functions\n\nAll tests now pass and the codebase is cleaner with reduced maintenance burden.\nThe removed functions had no references in production code and were only\nused in tests, which have been updated or removed as appropriate.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2cac4b024f877682k\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": "2e3337d7127d24e2d22fde945de32c9a35fbb3f0", "tree": "03feb48b8ee8d00d259420db70745ce9108f923e", "parents": [ "0258add1c3e5858635a4286f3e308e32132f828a" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jun 18 10:33:41 2025 -0700" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Wed Jun 18 10:38:16 2025 -0700" }, "message": "loop: add docker to the system prompt\n" }, { "commit": "57162e0bd03c33b5201822a7038c71aafe3a12c8", "tree": "161c11f9782e3a0ab64e3e2b74ff6de9a41eea7c", "parents": [ "92dcbf56f0de92f99391f560b67a75d4a654e6fe" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Jun 18 09:59:48 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 18 09:59:48 2025 -0700" }, "message": "codereview: add 1-minute timeout parameter with context propagation\n\nUsers and I have seen codereview hanging. I think giving it one\nminute is better than nothing.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s32c0166e97ef0effk\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": "28e39ac7817834d258a0f75a33b4590c769cb1bc", "tree": "23eed3fd8cdc60e107d5070c300dfaf488b45507", "parents": [ "6b8b7660f2e2e964453ebace40dda7122f6d9eca" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Mon Jun 16 22:04:35 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:30:06 2025 -0700" }, "message": "loop: remove unused waitForEnd parameter and endWaitGroup functionality\n\nRemove wait_for_end query parameter and associated synchronization logic\nthat was not being used in the codebase.\n\nChanges:\n- Remove waitForEnd query parameter parsing in SSE handler\n- Remove endWaitGroup sync.WaitGroup field from Server struct\n- Simplify shutdown logic to use basic 100ms delay instead of complex waitgroup timeout\n- Remove related comment documentation\n\nThis cleanup eliminates unused functionality while maintaining the same\nshutdown behavior with simpler code.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s76684f9f4ec3a5d1k\n" }, { "commit": "64f60461b00c474b0b4747a06177d5c7a357bba9", "tree": "cfeceff998a165af451099c0ec23b5d2b0fd42f2", "parents": [ "db8caa0eb8576b8c0d65cc0b7b6aa4e241a7cdb1" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 13:57:10 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jun 16 20:58:24 2025 +0000" }, "message": "loop: add diff stats from sketch-base to HEAD in /state endpoint\n\nAdd lines added/removed statistics computed from sketch-base to current HEAD,\ndisplayed in webui Diff mode button for quick change overview.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3f10ecf39df6b581k\n" }, { "commit": "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": "542bda3968c6dd5b79392dd63e2955e04520401a", "tree": "ea1a0743849495ca2489c6363d2dc689dd0a56a7", "parents": [ "225e9668aeebc0cae667872dd45222d69ac3cbd8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jun 11 18:31:03 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jun 12 01:31:34 2025 +0000" }, "message": "browser: rename browser_read_image to read_image and auto-send screenshots to LLM\n\nRename browser_read_image tool to read_image and modify browser_take_screenshot\nto automatically send image content to the LLM instead of requiring a separate\nread_image tool call, streamlining the screenshot workflow.\n\nProblem Analysis:\nThe current browser screenshot workflow required two separate tool calls:\n1. browser_take_screenshot - saves screenshot and returns file path\n2. browser_read_image - reads saved screenshot and sends to LLM\n\nThis two-step process was inefficient and created unnecessary round trips.\nAdditionally, browser_read_image was specific to browser automation but\nthe functionality of reading and encoding images is more general purpose.\n\nImplementation Changes:\n\n1. Screenshot Tool Behavior (claudetool/browse/browse.go):\n - Modified browser_take_screenshot to automatically return image content\n - Removed screenshotOutput struct as ID-only response no longer needed\n - Added base64 encoding of screenshot data directly in screenshotRun\n - Returns []llm.Content with both text description and image data\n - Still saves screenshot file for potential future reference\n - Uses same image encoding format as existing read_image tool\n\n2. Tool Rename (claudetool/browse/browse.go):\n - Renamed browser_read_image tool to read_image\n - Updated tool name in NewReadImageTool from \u0027browser_read_image\u0027 to \u0027read_image\u0027\n - Maintained all existing functionality and input/output format\n - Tool description and schema remain unchanged\n\n3. UI Updates (termui/termui.go):\n - Updated template condition from \u0027browser_read_image\u0027 to \u0027read_image\u0027\n - Maintains existing emoji and display format for read_image tool calls\n\n4. WebUI Updates (webui/src/web-components/):\n - Updated sketch-tool-calls.ts to reference \u0027read_image\u0027 instead of \u0027browser_read_image\u0027\n - Renamed sketch-tool-card-browser-read-image.ts to sketch-tool-card-read-image.ts\n - Updated component class name from SketchToolCardBrowserReadImage to SketchToolCardReadImage\n - Updated custom element name from \u0027sketch-tool-card-browser-read-image\u0027 to \u0027sketch-tool-card-read-image\u0027\n - Updated import statement to reference new component file name\n - Removed old component file and updated TypeScript declarations\n\n5. Test Updates (claudetool/browse/browse_test.go):\n - Modified TestGetTools to allow read_image tool without \u0027browser_\u0027 prefix\n - Added special case handling for read_image in tool naming convention check\n - All existing tests continue to pass with updated tool name\n\nTechnical Details:\n- Screenshot auto-send uses same base64 encoding as existing read_image tool\n- Content structure matches browser_read_image output format for consistency\n- File saving still occurs for potential debugging or future reference\n- Error handling preserves existing behavior with proper fallbacks\n- Tool count remains the same (12 tools with screenshots, 10 without)\n\nBenefits:\n- Eliminates need for two-step screenshot workflow\n- Reduces round trips and simplifies user experience\n- More intuitive tool naming (read_image is general purpose)\n- Maintains full backward compatibility for read_image functionality\n- Consistent image encoding across all browser tools\n- Automatic screenshot viewing improves debugging and validation workflows\n\nTesting:\n- All existing browser tool tests pass with updated expectations\n- TestReadImageTool verifies renamed tool functionality\n- Tool naming convention test updated to handle read_image exception\n- TypeScript compilation successful with no type errors\n- Web component functionality preserved across rename\n\nThis enhancement streamlines screenshot workflows while maintaining the\ngeneral-purpose read_image tool for reading arbitrary image files, creating\na more efficient and intuitive browser automation experience.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se3e81f997f30f01ek\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": "e08c7ffc8f1ec0cd3899d2fedad79f408b6dc2f9", "tree": "3e09e7b3a86faed4eaabb640eed0c4b148a9d8d6", "parents": [ "ba15aeb81e5fd3e98870eba91398dfb31d4cd6e9" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 06 13:22:12 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 06 20:26:10 2025 +0000" }, "message": "webui: add mobile interface with URL parameter switching\n\nTL;DR: ?m and ?d should load mobile and desktop versions respectively.\nIt should auto-detect to the right onen and redirect. Server chooses\nwhat to serve.\n\nThis took a few tries. Re-using the existing components didn\u0027t work,\ndespite repeated attempts. The prompt that eventually worked was:\n\n\tSketch\u0027s webui uses lit and webcomponents. I want to create an alternate\n\tweb ui, started with \"/m\" for mobile. Don\u0027t use the same components, but\n\tcreate a new mobile-shell component and a mobile-title, mobile-chat, and\n\tmobile-chat-input components. The design should be a title at the top, a\n\tsimplified chat (that doesn\u0027t display tool cards or anything; just the\n\tmessages, with user messages right-aligned and other messages left\n\taligned), and an input for new messages. Do include an indicator for\n\twhether or not the agent is thinking. Again: this is a parallel\n\timplementation, intended for mobile screens. Use the \"npm run dev\"\n\tserver to test it out and show me some screenshots. Use mobile browser\n\tsizes. Focus on simplicity in the CSS\n\nIt had some trouble with the data loading that took some iterations, and\nit kept saying, \"We\u0027re almost done\" and giving up, but I coaxed it\nthrough.\n\nI\u0027m not too sad right now about the duplication. We can see if there\u0027s\nany traction.\n\n~~~~\n\nAdd complete mobile-optimized interface for Sketch accessible via URL\nparameters, providing seamless device-specific user experience.\n\nProblem Analysis:\nSketch\u0027s existing web interface was designed for desktop use with complex\nlayouts, detailed toolbars, and mouse-focused interactions that don\u0027t\ntranslate well to mobile devices. Mobile users needed a simplified,\ntouch-friendly interface optimized for smaller screens while maintaining\ncore functionality for coding assistance on mobile devices.\n\nImplementation Changes:\n\n1. Mobile-Specific Components:\n - Created mobile-shell as main container with mobile-optimized layout\n - Built mobile-title with connection status and clean header design\n - Implemented mobile-chat with simplified message bubbles and alignment\n - Developed mobile-chat-input with touch-friendly controls and auto-resize\n\n2. URL Parameter-Based Interface Selection:\n - Added server-side parameter detection (?m for mobile, ?d for desktop)\n - Consolidated routing logic into main / handler with parameter checking\n - Eliminated separate URL paths in favor of clean parameter approach\n - Removed interface switching buttons for minimal UI approach\n\n3. Intelligent Auto-Detection:\n - Implemented client-side device detection using screen size and touch capability\n - Automatic URL parameter addition for detected mobile devices\n - Graceful parameter-based overrides for user preference\n - Seamless redirection maintaining all URL context and query parameters\n\n4. Mobile-Optimized Design:\n - iOS-style message bubbles with proper user/assistant alignment\n - Touch-friendly input controls with appropriate sizing\n - Responsive design scaling across mobile screen sizes\n - Animated thinking indicator with smooth dot animations\n - Clean, distraction-free interface focused on core chat functionality\n\n5. Data Integration:\n - Reused existing DataManager and SSE infrastructure\n - Maintained real-time message updates via /stream endpoint\n - Implemented message aggregation using existing patterns\n - Preserved connection status and error handling functionality\n\nTechnical Details:\n- Built using Lit web components for consistency with existing architecture\n- Server parameter detection uses URL.Query().Has() for efficient checking\n- Auto-detection preserves all existing query parameters during redirection\n- Mobile components filter messages to show user, agent, and error types\n- Responsive CSS with mobile-first design principles and touch optimization\n\nBenefits:\n- Seamless mobile experience with automatic device-appropriate interface\n- Clean URL parameter approach (?m/?d) works with any route or session\n- No UI clutter from interface switching buttons\n- Maintains full real-time functionality on mobile devices\n- Easy to bookmark and share mobile-specific URLs\n- Preserves session context during interface switching\n\nTesting:\n- Verified auto-detection works across different mobile screen sizes\n- Confirmed URL parameters function with complex session URLs\n- Tested message sending, receiving, and real-time updates\n- Validated thinking indicator and connection status display\n- Ensured responsive design across portrait and landscape orientations\n\nThis implementation provides a complete mobile solution while maintaining\nthe full-featured desktop interface, enabling Sketch usage across all\ndevice types with appropriate user experiences.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sdbce185f247638c1k\n" }, { "commit": "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": "272c4a9aab39b3b7eadb6c6ae7b275da4663e9d7", "tree": "fe8322570545314e03f8869ce3a9ae37e3576cb4", "parents": [ "008f9580cb1df44d70f1daa32b2122e10b7d8f79" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 16:08:42 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 16:55:23 2025 -0700" }, "message": "claudetool/codereview: nudge agent to call codereview after each round" }, { "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": "43448c674f4cf1f6de7950f6ec9e2d197d7f700a", "tree": "65369f28dd9e2acd13a6506923ccc165d2240f3a", "parents": [ "53ab24547cd684fc38254a6bd63759d7121ca7d6" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 11:51:04 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 14:00:10 2025 -0700" }, "message": "loop: nudge agent towards less bouncy emoji babble" } ], "next": "16098932295e067fb0a6b3ca2082b0d4b06027b4" }