)]}' { "log": [ { "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": "106f6f5472c233011b6abee307cb6cd0e962f718", "tree": "06caf9e7ab2a1ecafebdc6d9a3a9846d8cfe161c", "parents": [ "cf724583660cf683efa7fd8eb9b32e120e88beb7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 15:22:00 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:46:45 2025 -0700" }, "message": "claudetool: tweak bashkit error message when changing branches\n\n- remove reference to outie (normal sketch sessions\n don\u0027t have the concept introduced in the prompt anywhere)\n- tweak wording for precision\n- add some extra details\n- point out the Push button\n- let claude rewrite a bit for concision\n" }, { "commit": "d082fdceb0378ddbceb81f51ab77ac1af904d57c", "tree": "85ebda02944618e18dfc4612fe8954f58ff05c29", "parents": [ "ae3724ef039833e1227714412e2c5289ad8875c8" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 13:31:52 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 13:33:10 2025 -0700" }, "message": "claudetool/bashkit: don\u0027t mention set-slug when blocking sketch-wip renames\n" }, { "commit": "19a32eaa120562170595e42584f3fc4b40828b97", "tree": "eaa6cac6a052a02a97a94a62a2fc13adadd50a6e", "parents": [ "2be768e4f4d28dc56c7e0b10df342c607ef04833" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 18:29:31 2025 +0000" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 12:04:31 2025 -0700" }, "message": "claudetool/bashkit: add sketch-wip branch protection with process-level tracking\n\nAdd a best-effort check to reject git commands that would change the\ncontainer-side \u0027sketch-wip\u0027 git branch. The protection prevents:\n\n1. Branch renaming: git branch -m sketch-wip newname\n2. Branch switching: git checkout otherbranch, git switch otherbranch\n3. Force branch renaming: git branch -M sketch-wip newname\n\nThe check allows legitimate operations like:\n- File checkout: git checkout -- file.txt\n- Path operations: git checkout src/main.go\n- Branch creation: git switch -c newbranch\n- Standard git operations: git commit, git status, etc.\n\nKey features:\n- Process-level tracking: Shows warning only once per process\n- Informative error message explaining why it\u0027s blocked\n- Suggests using \u0027set-slug\u0027 tool for external branch naming\n- Tells user they can repeat the command if really needed\n\nImplementation:\n- Added process-aware check alongside existing static checks\n- Process-level tracking via mutex-protected boolean\n- Comprehensive test coverage including edge cases\n- Maintains backward compatibility with existing Check() function\n\nThis prevents agents from inadvertently breaking the outie\u0027s ability\nto detect and push changes to GitHub by changing the expected branch name.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3bb00ecac8a4badek\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": "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": "553cc842a3fea3594158c8314a577c2b916bf45d", "tree": "ddfc127b4dd7db561f184e6a9ab85a5103a27aaf", "parents": [ "9f5a051b23af48263ec8fde4cc9b1c6e11532433" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:05:22 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 19:13:06 2025 -0700" }, "message": "sketch/claudetool/browse: fix broken test\n\nApparently this wasn\u0027t actually running anywhere in CI. Sigh.\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": "26b6f9ba1a9647f7f0b0983f8d640b2d81789f5d", "tree": "0aca1555e9f221a43b5190f144d6b10e487569e3", "parents": [ "cb5572629b5f7ea9182e5c9f32a14dc8c5a563e4" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 01 01:41:11 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 02 15:48:16 2025 -0700" }, "message": "claudetool/codereview: add caching in findRelatedFiles\n\nPrimary goal is latency reduction.\nAlso slightly reduces context usage.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sa1007d82a5165ab4k\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": "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": "07b7400182aefc4153eb6617ab89a5f05f577349", "tree": "c94fccc6dea3ca946c2c9039f062d3814d61477f", "parents": [ "9aa7828cbff8e8d66e63909db5a222d1b90fa986" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jun 26 16:05:25 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Jun 26 09:07:17 2025 -0700" }, "message": "fix: prevent nil pointer panic in codereview compareTestResults\n\nFix for https://github.com/boldsoftware/sketch/issues/175\n\nFix nil pointer dereference when comparing test results between commits\nwhere a package exists in the after state but not the before state.\n\nThe panic occurred at line 867 when accessing beforeResult.TestStatus[test]\nwithout checking if beforeResult was nil. This happens when testing new\npackages that didn\u0027t exist in the base commit.\n\nNow properly check for nil beforeResult and default to testStatusUnknown\nfor tests in packages that didn\u0027t exist in the before state.\n\nAdd comprehensive unit tests covering:\n- New package with passing tests (the panic scenario)\n- New package with failing tests (regression detection)\n- Existing package regressions (ensure normal flow still works)\n- End-to-end integration test for new package scenario\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sab802d91eff08039k\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": "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": "d4eea221b455e06fe7f897d1e04a8833015e893c", "tree": "c5dd56fd56611f5e1c930e9930e1449d8ade7cc6", "parents": [ "fe51d1d1b1f205ce2a417d312991e71d31ce6dc3" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:13:21 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:22:20 2025 -0700" }, "message": "Fix vet_error baseline?\n\nNo, I don\u0027t know how this crawled up.\n" }, { "commit": "fe51d1d1b1f205ce2a417d312991e71d31ce6dc3", "tree": "e45895b2546f9b0a54d6bbd7a635e47f821f36bb", "parents": [ "64f60461b00c474b0b4747a06177d5c7a357bba9" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:19:44 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 16 21:22:20 2025 -0700" }, "message": "Further loosen browser timeouts.\n" }, { "commit": "29c481c8d3bbe6124d3b273d2a154081d8472f08", "tree": "616a5f8097fa56da5e315ef2e2948fc049851a01", "parents": [ "c0a4459e3f5c8e68649bb45f2c309f6f01928ab7" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sun Jun 15 21:25:45 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sun Jun 15 21:25:45 2025 -0700" }, "message": "browser tests: further increasing timeouts\n" }, { "commit": "a35de5f73d31756c8ac972a54bdb370bb26eed5a", "tree": "e86111f53c0e701a38510ed8543403f25665a891", "parents": [ "46be0967c78b0e60b18b4b9ff3c642ebc96c7e4d" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 14 12:00:48 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat Jun 14 12:00:48 2025 -0700" }, "message": "Using GH actions provided Chrome for test speedup (#149)\n\nOMG. This had wild behavior. Sometimes you\u0027d log into tmate, and it\nwould work. Something about running Chrome manually would tickle some\ndbus thing into working. The error would sometimes show up, but most\noften it would be swallowed.\n\nThe punchline? We had a context timeout of 5 or 10 seconds in the tests\nthat would trigger silently, and we\u0027d see nothing. Everything else\n(including maybe the dbus stuff) was a red herring: the test executor\nmachine is swamped, and shit\u0027s slow, and the various timeouts needed\ntweaking. I bet there are a few more timeouts to loosen eventually, but\nit seems to be passing now.\n\nError message:\n\tfailed to start browser (please apt get chromium or equivalent): chrome failed to start:\n" }, { "commit": "c013134814facb7da15b59c79399ec1939eb6ef9", "tree": "6072025f9c2e5a87af79a61070fab1f0500c32a5", "parents": [ "c886ac5478194e95f93b166572a7a081eb6cadb8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 13 21:07:08 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 13 21:07:08 2025 -0700" }, "message": "Fail to speed up CI by not installing Chrome.\n\nI have no idea why the same \"vintage\" machine I logged into via tmate\nhas this working, but this doesn\u0027t, but the --no-sandbox thing is\ncorrect regardless.\n\nVariants of the following is what SHOULD work but doesn\u0027t yet.\n\n\tPATH\u003d/usr/local/share/chromium/chrome-linux:$PATH GOEXPERIMENT\u003dsynctest stdbuf -oL -eL go test -v -count\u003d1 -run TestBrowserInitialization ./claudetool/browse/...\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": "209ea91bc220b945936250f8aecd9d69974e0ada", "tree": "e0b83da2543678561379e523c95c79f221361c3c", "parents": [ "4c1cea85b222a1ec94890be8960c69f549870454" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jun 09 21:54:12 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jun 09 21:58:59 2025 +0000" }, "message": "test: fix unicode filename testdata causing Go module packaging issues\n\nReplace static unicode testdata files with dynamic file creation to avoid Go\nmodule zip creation errors while preserving comprehensive unicode filename\ntesting functionality.\n\nProblem Analysis:\nUnicode characters in testdata filenames (🚀rocket.md, café.js, 测试文件.go,\nрусский.py, Übung.html, Makefile-日本語, readme-español.md, claude.한국어.md)\nwere causing go get failures when creating module zip files. The error\n\u0027malformed file path \"claudetool/onstart/testdata/🚀rocket.md\": invalid char \u0027🚀\u0027\u0027\nprevented users from downloading the sketch.dev module using go get.\n\nThe existing tests verified that AnalyzeCodebase could handle unicode filenames\ncorrectly, which is important functionality for international users. However,\nthe static testdata approach was incompatible with Go module packaging.\n\nImplementation Changes:\n\n1. Test File Modification:\n - Modified TestAnalyzeCodebase Non-ASCII Filenames test to use t.TempDir()\n - Added dynamic creation of unicode test files at runtime instead of static testdata\n - Created temporary git repository with proper unicode configuration\n - Added comprehensive git setup with core.quotepath\u003dfalse and core.precomposeunicode\u003dtrue\n\n2. Dynamic File Creation:\n - Implemented map[string]string for test files with unicode filenames as keys\n - Files include same content as original testdata but created dynamically\n - Added proper subdirectory creation for subdir/claude.한국어.md test case\n - Git repository initialization and file addition handled programmatically\n\n3. Git Configuration:\n - Added proper git config setup for unicode filename handling\n - Set user.name and user.email for temporary test repositories\n - Configured core.quotepath\u003dfalse to handle unicode paths correctly\n - Used \u0027git add .\u0027 to add all files at once avoiding individual unicode filename issues\n\n4. Static File Removal:\n - Removed all 8 problematic unicode testdata files\n - Deleted empty testdata directory structure\n - Cleaned up repository to eliminate unicode filenames from git history\n\n5. Test Preservation:\n - Maintained identical test coverage for unicode filename functionality\n - Preserved all categorization tests (build files, documentation, guidance)\n - Kept extension counting verification for unicode files\n - Added proper imports (os/exec, path/filepath) for dynamic file creation\n\nTechnical Details:\n- Uses Go\u0027s testing.T.TempDir() for isolated test environments\n- Temporary git repositories prevent unicode files from entering main repository\n- Same unicode characters tested: Chinese (测试), Korean (한국어), Russian (русский),\n French (café), German (Übung), Japanese (日本語), and emoji (🚀)\n- File categorization still validates Makefile detection, README recognition, and claude.md guidance\n- Error handling for git commands ensures test failures provide clear diagnostics\n\nBenefits:\n- Resolves Go module packaging issues allowing successful go get operations\n- Maintains comprehensive unicode filename testing without repository pollution\n- Dynamic approach is more robust and doesn\u0027t require static test file maintenance\n- Tests run in isolation with proper cleanup via t.TempDir()\n- Preserves international user support validation while fixing distribution issues\n\nTesting:\n- All tests pass with dynamic file creation approach\n- Unicode filename categorization works identically to static file approach\n- Extension counting and file analysis functionality preserved\n- Git operations handle unicode filenames correctly in test environment\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s31257070090e907dk\n" }, { "commit": "f1e517d64fe4a726552f5d240a1ecb3d418f16b6", "tree": "ace5b25920201d79054a8b3f7cc17cf884fae8c5", "parents": [ "de19aca257ab21956f2fba828d9265ef218687da" ], "author": { "name": "Marc-Antoine Ruel", "email": "maruel@gmail.com", "time": "Sun Jun 08 17:30:37 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Sun Jun 08 13:08:59 2025 -0700" }, "message": "claudetool/onstart: add comprehensive tests for non-ASCII filename handling\n\nAdd test cases validating AnalyzeCodebase() correctly processes files with\nUnicode characters in filenames, ensuring proper categorization and analysis.\n\nProblem Analysis:\nThe AnalyzeCodebase function uses git ls-files to enumerate repository files\nand categorize them by type. While the implementation should theoretically\nhandle Unicode filenames, there were no existing tests to verify this\nbehavior with international characters, emojis, combining characters, or\nright-to-left scripts.\n\nImplementation Changes:\n\n1. Test Data Creation:\n - Created testdata directory with files containing non-ASCII characters\n - Included Chinese (\\u6d4b\\u8bd5\\u6587\\u4ef6.go), French (café.js), Russian (\\u0440\\u0443\\u0441\\u0441\\u043a\\u0438\\u0439.py)\n - Added emoji (\\ud83d\\ude80rocket.md), German umlauts (\\u00dcbung.html)\n - Included Japanese (Makefile-\\u65e5\\u672c\\u8a9e), Spanish (readme-español.md)\n - Added Korean guidance file (subdir/claude.\\ud55c\\uad6d\\uc5b4.md) in subdirectory\n\n2. Comprehensive Test Cases:\n - TestAnalyzeCodebase validates file counting and extension tracking\n - Verifies proper categorization of build, documentation, and guidance files\n - Tests git ls-files integration with Unicode filenames\n - Confirms extension counting works with non-ASCII characters\n\n3. Edge Case Testing:\n - Added combining characters test (file\\u0301\\u0302.go)\n - Arabic right-to-left script test (\\u0645\\u0631\\u062d\\u0628\\u0627.py)\n - Mixed Unicode with emoji test (test\\u4e2d\\u6587\\ud83d\\ude80.txt)\n - Validates categorizeFile function handles Unicode paths correctly\n\n4. File Categorization Validation:\n - Japanese Makefile correctly identified as build file\n - Spanish README properly categorized as documentation\n - Korean Claude file in subdirectory marked as guidance file\n - Extension counting accurate across all Unicode filenames\n\nTechnical Details:\n- Uses git ls-files -z for null-separated output handling Unicode safely\n- Test files represent major Unicode blocks: CJK, Latin Extended, Cyrillic\n- Proper handling of combining characters and emoji sequences\n- Validates both filename parsing and categorization logic paths\n\nBenefits:\n- Ensures international users can use non-ASCII filenames\n- Validates Unicode safety in codebase analysis pipeline\n- Prevents regressions in Unicode filename handling\n- Comprehensive coverage of real-world filename scenarios\n\nTesting:\n- All tests pass with current implementation\n- Verified git ls-files correctly enumerates Unicode filenames\n- Confirmed extension extraction works with international characters\n- Validated categorization logic handles Unicode paths properly\n\nThis test suite ensures AnalyzeCodebase robustly handles international\ncodebases with diverse filename conventions and character encodings.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2431e70f6f23ec83k\n" }, { "commit": "de19aca257ab21956f2fba828d9265ef218687da", "tree": "dc32644d65f2fa9b7feb3581da953e0467eb0e45", "parents": [ "c139982287731dfab910389deeba766d73cf1bd4" ], "author": { "name": "Marc-Antoine Ruel", "email": "maruel@gmail.com", "time": "Sun Jun 08 13:20:43 2025 -0400" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Sun Jun 08 13:08:59 2025 -0700" }, "message": "analyze: use git ls-files -z\n\nThis unblock support for non-ascii characters.\n\nCoded with my bare hands.\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": "95632c8a1e5953eddcb3073764b3db2c7f7bddb5", "tree": "30d2fd9f686e10dd26506c3da1b4fb5438532d4e", "parents": [ "a727f7073c024573017c2f46517fc2e90545f9c1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jun 05 01:23:05 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 19:03:43 2025 -0700" }, "message": "claudetool/codereview: fix runGenerate to filter pre-existing modified files\n\nThe runGenerate function was returning all files that appear modified in git\nstatus after running \u0027go generate\u0027, including files that were already modified\nbefore the generate command ran. This led to misleading output suggesting that\n\u0027go generate\u0027 had changed files like .DS_Store, .projections.json, and other\nunrelated dirty files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sfdde2d72136bfb64k\n" }, { "commit": "a727f7073c024573017c2f46517fc2e90545f9c1", "tree": "71b2925c4d3cbf2b4d3563157ff95a0c5c6378c4", "parents": [ "cf19190dd743f3f3c906a7b6f4fd89b87913663e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 18:47:33 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 19:03:43 2025 -0700" }, "message": "claudetool/codereview: stabilize related files list\n" }, { "commit": "cf19190dd743f3f3c906a7b6f4fd89b87913663e", "tree": "94c715b3c77e417940dd043388847860031ec358", "parents": [ "272c4a9aab39b3b7eadb6c6ae7b275da4663e9d7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 18:18:40 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jun 04 17:00:43 2025 -0700" }, "message": "claudetool/codereview: add go generate support\n\nAutomatically run \u0027go generate\u0027 as part of codereview,\nsince sketch sometimes forgets to do it.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: safe9348e7c24beffk" }, { "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": "855afffc57824d79d25c432bb850474fcfffd4d3", "tree": "ed66218490ec071539b65872c3b45c2d6bba83d2", "parents": [ "67b4c77c37021647363ed36d4a2b87e5d0925baf" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:47:47 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:58:10 2025 -0700" }, "message": "claudetool: don\u0027t attempt to install zero tools\n\nFollow-up to 495c1fa247565e21b36bcb847c6cd3f08e0e196f\n\nFixes #127\n" }, { "commit": "67b4c77c37021647363ed36d4a2b87e5d0925baf", "tree": "d1cf08a4fefaf0a8c060c90356f53aee434eab01", "parents": [ "bf381a7b6c6df65e7b74796181ab0489566e3795" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:46:27 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:58:10 2025 -0700" }, "message": "claudetool: minor auto-install cleanup\n\nFollow-up to 495c1fa247565e21b36bcb847c6cd3f08e0e196f" }, { "commit": "bf381a7b6c6df65e7b74796181ab0489566e3795", "tree": "070e419f99682b82892ec85594902d07222dd21a", "parents": [ "b421a24f862d91101b9e5b4738f93693e18672eb" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 23:45:02 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:58:10 2025 -0700" }, "message": "claudetool/browse: block port 80 access in browser tools\n\nPrevents agents from attempting to browse to port 80, which is reserved\nfor the Sketch server itself. This is a common source of confusion.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sf481df655a16903ek" }, { "commit": "b421a24f862d91101b9e5b4738f93693e18672eb", "tree": "2ccccb673504f5d5d3ca689816e95c92fd3c3041", "parents": [ "00bcaef0355aaff1daea17ac0631fd17cabb0235" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 23:22:55 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 30 08:58:07 2025 -0700" }, "message": "all: remove anthropic edit tool support\n\nThe Anthropic str_replace_editor tool implementation has rotted;\nit would require a bit of work to upgrade it for Claude 4.\n\nRemove it for now.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6129e8703301600dk" }, { "commit": "495c1fa247565e21b36bcb847c6cd3f08e0e196f", "tree": "5c7031ad37f4f0181824664f15220f87c7314861", "parents": [ "b6bc113f9ebb00084285e8c3aeaf2fd4a648afc0" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 00:37:22 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 15:12:53 2025 -0700" }, "message": "claudetool: add just-in-time tool installation for bash commands\n\nImplements automatic tool installation when bash commands use missing tools,\nproviding a seamless experience for the LLM to Just Use tools it knows ought to exist.\n\nCore Features:\n\n1. Command Analysis Pipeline:\n - Parse bash commands to extract individual tools/programs\n - Use exec.LookPath to check if tools exist in PATH\n - Handle shell built-ins, absolute/relative paths correctly\n - Support complex command chaining with \u0026\u0026, ||, ;, and |\n\n2. Subagent Tool Installation:\n - Spawn dedicated subagents to install missing tools\n\nThe system preserves existing bash tool behavior while adding invisible\ntool installation. Original commands run regardless of installation\nsuccess/failure, avoiding agent confusion.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s226cd6260a6469e9k\n" }, { "commit": "112b92376c97b8da64cc1c954896957cfc479f3d", "tree": "1497eb8b2927f1334d712aabb19625db11a148bf", "parents": [ "d203b7de7d49cc5da03440d5a00b2efd0f8bb2c8" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 11:26:33 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 12:50:41 2025 -0700" }, "message": "loop: add todo checklist\n\nThis should improve Sketch\u0027s executive function and user communication." }, { "commit": "7fbc8e43ff52b6f5dc69fdd26ad7388bdac70435", "tree": "abc653d8bf0563f00d232ad91edbed38eb5d4abd", "parents": [ "57893c274ca6d7936580b8bd97af9e5a4b4dcd51" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 19:42:25 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 29 12:50:41 2025 -0700" }, "message": "browse: set default viewport to 1280x720 (16:9 widescreen)\n\nImprove browser screenshot quality by setting a modern default viewport\nsize during browser initialization. This replaces the previous system-\ndependent default with a consistent 1280x720 resolution that provides:\n\n- Better readability and content coverage in screenshots\n- Modern 16:9 aspect ratio matching typical displays\n- Consistent testing environment across all users\n\nImplementation:\n- Add EmulateViewport call during browser startup in browse.go\n- Set viewport immediately after browser initialization\n- Add comprehensive test (TestDefaultViewportSize) to verify functionality\n\nThe test confirms the viewport is correctly set to 1280x720 on browser\nstartup and validates the JavaScript window dimensions match expectations.\n\nThis addresses screenshot quality issues with small default viewport sizes\nand provides a better default experience for browser automation.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s98815cfd1eaca88dk\n" }, { "commit": "9500617d20b037b9ad1f88eb7550068679832a4d", "tree": "15593d668443d3df4f6d223bd753c216f5f0274a", "parents": [ "e89b3080f934a4bc70a0cfa85ffff49ac78d6f2b" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 28 20:05:46 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed May 28 20:26:29 2025 -0700" }, "message": "dockerimg: fix Chromium support with multi-stage Docker build\n\nOMG, people, OMG. So, an earlier commit moved us to Ubuntu, and it turns\nout that \"apt-get install chromium-browser\" or whatever just does\nnothing, and tells you to use the snap. Snap requires systemd, and if\nyou\u0027re using containers, you don\u0027t usually have systemd, and ... yeah,\nno screenshots. There are no great stories for where to get Chromium.\nThere\u0027s a dude who publishes the Mint Linux packages in a compatible way\nfor Ubuntu. I chose instead the headless-chrome from a Docker build\nrecommended by the Chromedp library that we use to control Chromium.\n\nI\u0027m a bit snappy about all of this.\n\n...\n\nReplace Ubuntu 24 snap-based Chromium installation with chromedp/headless-shell\nto resolve container compatibility issues where snaps don\u0027t work properly.\n\nChanges include:\n\n1. Multi-stage Dockerfile.base build:\n - Stage 1: Extract headless-shell from docker.io/chromedp/headless-shell:stable\n - Stage 2: Main Ubuntu 24.04 application image with required Chrome dependencies\n - Remove chromium package from apt-get install (replaced with headless-shell)\n - Add required libraries: libglib2.0-0, libnss3, libx11-6, libxcomposite1,\n libxdamage1, libxext6, libxi6, libxrandr2, libgbm1, libgtk-3-0\n - Add headless-shell to PATH so chromedp can find it automatically\n\n2. Updated documentation in browse/README.md:\n - Document Docker multi-stage build approach\n - Clarify requirements for Docker vs local development\n\nBenefits:\n- Resolves Ubuntu 24 snap incompatibility issues in containers\n- Provides self-contained Chrome installation without system dependencies\n- Maintains backward compatibility for local development\n- Uses proven chromedp/headless-shell for reliable browser automation\n- Eliminates need for manual Chrome/Chromium installation in containers\n- No code changes needed in browse.go - chromedp finds headless-shell via PATH\n\nThe headless-shell binary is automatically discovered by chromedp\u0027s default\nexecutable search since it\u0027s added to PATH in the Docker environment.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se4808dca7afba802k\n" }, { "commit": "efa8f436182a310f7c5eaaf0f9f4787ba4fd3693", "tree": "9fb7306edcfb68ebe9af2519784040ce25e16665", "parents": [ "90993a0d42e84be835ebf2096687a299d461cfeb" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 18:04:41 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 28 18:58:46 2025 -0700" }, "message": "claudetool: strengthen the \"no trailers\" commit message guidance\n\nShould prevent duplicate trailers.\n" }, { "commit": "83b2d35b61eb68c0d633a10c2f5b7c0905e584df", "tree": "b29314fe23190123cc4e24d2327a237de7ebf096", "parents": [ "3b733a5e21229c1ff29389895bb587c0243d6ba7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 11:39:50 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Sat May 24 12:07:43 2025 -0700" }, "message": "claudetool/codereview: expand on when to commit vs revert" }, { "commit": "586ecb184b4466435507d9935e4240630e80f482", "tree": "4e49af79c8fe7df45567507c642dd31fe015ae8b", "parents": [ "0efb29db25ff141e6463b40675defc423a42379c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 22 21:04:33 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Sat May 24 12:06:12 2025 -0700" }, "message": "claudetool: fix sha command formatting" }, { "commit": "dbfd36a4149c03a501be12aacfecdbe237c7b7bf", "tree": "c8928ebe5568b7c5330f9211538ab8139832652f", "parents": [ "afeafea91f20db4e5b99620df98baa1628bfeaf1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 20:57:50 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 14:25:55 2025 -0700" }, "message": "claudetool/bashkit: add validation to reject blind git add commands\n\nImplements new validation function to prevent bash commands that blindly\nadd all files to git commits, requiring agents to specify files explicitly.\n\nChanges include:\n\n1. New noBlindGitAdd validation function that rejects:\n - git add -A (add all)\n - git add --all (add all)\n - git add . (add current directory)\n - git add * (add all files via glob)\n\n2. Enhanced hasBlindGitAdd helper function:\n - Parses git command structure using existing AST approach\n - Finds \u0027add\u0027 subcommand and checks subsequent arguments\n - Handles git flags before \u0027add\u0027 subcommand correctly\n - Allows legitimate specific file additions like \u0027git add main.go\u0027\n\n3. Comprehensive test coverage for all blind add patterns:\n - Tests rejection of all blind patterns including flags combinations\n - Tests acceptance of specific file additions and valid use cases\n - Tests multiline scripts and edge cases like filenames that look\n like flags (e.g., \u0027file.A\u0027)\n\n4. Integration with existing validation framework:\n - Added noBlindGitAdd to the checks slice alongside existing\n noGitConfigUsernameEmailChanges validation\n - Uses same error reporting pattern and AST parsing infrastructure\n\nThis prevents agents from accidentally committing unintended files or\nbuild artifacts by forcing explicit file specification in git add\ncommands. The validation maintains the existing pattern of being\nmistake-prevention rather than security-focused.\n\nError message provides clear guidance: \u0027blind git add commands (git add -A,\ngit add ., git add --all, git add *) are not allowed, specify files explicitly\u0027\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s4d8fe71ef5816b36k\n" }, { "commit": "160fb06bb7c8df956daecd4d3bdb3f94d8a93406", "tree": "9456f5a0df9eb51180d189d87f8d52016ccb5f40", "parents": [ "0e8073a19a88ef5e7a03acc8b5f924bfda0afdd5" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 22 21:04:22 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 23 11:30:35 2025 -0700" }, "message": "claudetool: improve error message when all search terms match too much" }, { "commit": "e10f0e6c540ac0f6c038a4a6b729e9401dd39e52", "tree": "c3704ca3c01f833195fee4f16f5764b80e268544", "parents": [ "564043144868f401515c8ed250d989f60ea2e1de" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 10:57:09 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 11:37:49 2025 -0700" }, "message": "claudetool: ignore trailers in git commit style analysis\n\n" }, { "commit": "6f041f5ac59a17987efdfe63d1d05e487e23a89e", "tree": "fe277e0b16bff201ef26fb586b85eb639859a223", "parents": [ "6e35332ceac45166d6dd549d1e3d94937f683320" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 17:31:08 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 11:36:32 2025 -0700" }, "message": "claudetool/onstart: support .github/copilot-instructions.md as inject file\n\nUpdate categorizeFile function to recognize .github/copilot-instructions.md\nas an injectable file for the agent prompt, allowing GitHub Copilot\nguidance to be included in the system prompt.\n\nFixes #105\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s211c42575972fdbek\n" }, { "commit": "6e35332ceac45166d6dd549d1e3d94937f683320", "tree": "87b4eeea74b9f6d9786d99983a976c1cf0f7ff05", "parents": [ "8d8b7ac6b00b1d0b3a2c6d3a4079e2c2fb76cc8a" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 18:21:24 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 21 11:36:32 2025 -0700" }, "message": "all: remove testify\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s51244611964f2413k" }, { "commit": "7ce7b0294ba1efe6c7b3d462e905c1b2a7057a37", "tree": "9b0b92fbcf2b27a2b971481716e4d1dc99d3ec6f", "parents": [ "9daa5183df5270beaf5611c20e4d77cd7cfe21f7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 16 11:35:51 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 20 20:49:05 2025 -0700" }, "message": "claudetool: draw attention to commit message capitalization and punctuation" }, { "commit": "9daa5183df5270beaf5611c20e4d77cd7cfe21f7", "tree": "84bd4bf7d7bfbc723925bfae2af828cd9e7ccd0b", "parents": [ "55c87e3ababb0800b5710cbb11bec4dea8c13515" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 16 18:34:00 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 20 14:27:29 2025 -0700" }, "message": "experiment: remove llm_review experiment and associated code\n\nIt creates confusion for the agent and doesn\u0027t provide much value.\nPerhaps with better prompt engineering, but I can\u0027t invest\nin that much right now.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9f2a8c418876c498k" }, { "commit": "276f460cc7136e50a07cf544ee5ef6bf2a1c2630", "tree": "0e00d887c0bac515e49775041be83eadbc174074", "parents": [ "a4500c98a8f7c2e5500d330d7f4e82037fb38239" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 17:57:51 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 22:00:19 2025 -0700" }, "message": "all: read agent.md files too\n\ntoo many agent rules files!\n" }, { "commit": "f163e9e785845bed2d7fba31ddf06a22a2e1e853", "tree": "c728e8da4f5c9a7a3d885f3130bc9bde13f28c3e", "parents": [ "de5f74466b6db86c310762d627ea5f05b24cd08e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 12:38:30 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 13:17:42 2025 -0700" }, "message": "claudetool/codereview: stop checking string iteration patterns\n\nEven frontier LLMs get confused by these recent language improvements.\nI\u0027m sick of fighting them (i.e. watching them fight themselves).\nWe\u0027ll just have to wait for them to either get more capable,\nor to learn the new idioms. Sigh.\n" }, { "commit": "de5f74466b6db86c310762d627ea5f05b24cd08e", "tree": "12196afbbb7362c93883cf63fe8d94d5e87b5387", "parents": [ "fbbf83bba97e2e2fd5e5ea1693c96efce1fdc3b1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 18:32:32 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 13:17:42 2025 -0700" }, "message": "claudetool: ignore some gopls (/vet) checks\n\nFixes #95\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s934f225a42c1d576k\n" }, { "commit": "74d690e7b4794f2c164a9da30dd106a002b935d9", "tree": "412f1ad777e599ed61904c7f80c53859315babab", "parents": [ "039fc34bd9c8f7cf3e9f3c9aeee2e9677ce28e00" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 18:16:03 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 15 12:40:29 2025 -0700" }, "message": "claudetool: remove knowledge base, focus on about sketch\n\nWe should have a more general kb.\nMeanwhile, this is important and standalone.\nMake it all clearer and sharper.\n" }, { "commit": "75f93a1e5f32366c138c1584fdeb4919e257719d", "tree": "977a7932cbfd1c1f95b0b56cbe0930a5aeb5e7b8", "parents": [ "a10f151746c0dafd073d4998755bedbec8f82db6" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Thu May 15 13:51:55 2025 +0000" }, "committer": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Thu May 15 14:24:48 2025 +0000" }, "message": "codereview: Filter out non-existent files from related files suggestions\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8c2a925e22191308k\n" }, { "commit": "6525d826f22d12a57b9acb474a5e7b22b48ba7ea", "tree": "6cb917c5c860c976c264a58d2753a5b6df44b57e", "parents": [ "9007dd98202f8b41db1663b308fb4641476f8fa5" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Thu May 15 14:55:28 2025 +0100" }, "committer": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Thu May 15 14:56:29 2025 +0100" }, "message": "claudetool/kb: Prompt engineering to get sketch to more readily use kb\n" }, { "commit": "7bb1bf0dfe872ae2209308f6524edb206143b37e", "tree": "28667b522c7af3180f5dde5042434c7f1d2a458f", "parents": [ "0e5b8c69d3d4436a8b591f48fd1a031a3912585a" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 14:58:20 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 14:58:20 2025 -0700" }, "message": "claudetool/kb: add kb info about sharing sketches" }, { "commit": "c0a1e53bae35673dc77d69ba936f064439e25e1c", "tree": "697a470cbf1a0cabd9d153af1832804b49cf6551", "parents": [ "436655e80bfeec05db1b3fce62990ba4d1374002" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 17:29:39 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 14 10:33:06 2025 -0700" }, "message": "claudetool: remove telemetry endpoint and related functions\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s88833bbb3878c2a9k\n" }, { "commit": "49edc92f95cab092a7ee62c350c6e69275b50cc8", "tree": "946b8359f6187ed630c4d08fa05e91fe1bc5a48d", "parents": [ "52eeb15cf2f70caaadc4e0722a3b687ad9a91981" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 09:45:45 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed May 14 16:46:49 2025 +0000" }, "message": "agent: move \"sketch-base\" into git\n\nThe agent\u0027s notion of \"initial commit\" is kind of special, in that it\nis used as the \"base\" for a bunch of git operations. It\u0027s hard for\nthe user to change (we only provide a workflow via restart), yet\nsometimes you want to do just that.\n\nSo, instead we put it as data inside of it, named as a tag sketch-base.\nIt\u0027s abusing tags, but branches are no better.\n" }, { "commit": "8a1b89a210d3c48c6f02f0c8cf1feebb03cdffd5", "tree": "7d89b37aa585c23e8d6f640aea65e4f7c979263f", "parents": [ "ec398cb9e4650d1e6043966166a99404879719fa" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 17:58:41 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 17:58:41 2025 -0700" }, "message": "claudetool/bash: include partial output in error message for timeouts\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se1cb20e743a878d4k\n" }, { "commit": "064f8ae0b93b8f0ea92bf9555416b1c51b2969e0", "tree": "3e9ef9a84330abddcb38ed1488290e7f4e046a7d", "parents": [ "208938f612a498ec6b594564a643fe9172da061f" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed May 14 00:47:41 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 17:54:39 2025 -0700" }, "message": "claudetool/bash: include partial output when command times out\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s829d2dbdfda28988k\n" }, { "commit": "208938f612a498ec6b594564a643fe9172da061f", "tree": "80a8c313fbd548fc488ffeab3f00357eb7286726", "parents": [ "3e9d80c6b6315ccf1754996e9b691555915d9839" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 17:17:18 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 17:45:17 2025 -0700" }, "message": "bash: Shorten bash tool default timeout to 10s.\n" }, { "commit": "18e336804bb6f052b29d891f7db54bd45e1de503", "tree": "48a979aa7f5e89975cf8f579722ff9f363768985", "parents": [ "34bb09adf0cfc2f7fa4a2615e72d634aa88a6016" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 16:34:21 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 13 16:34:21 2025 -0700" }, "message": "browser: Add console log capture tools\n\nThis was entirely Sketch-coded.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc6bbdde6b1a71d8fk\n" }, { "commit": "31785aed38e0f63fa736430c6240c8a76b8285dc", "tree": "35916d3dbedaeac296ece3dd57610efd82c99c8d", "parents": [ "5c7f95714f34ea327b8f300238da6491dedd6adb" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 06 01:50:58 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 12 19:17:50 2025 -0700" }, "message": "loop: add knowledge_base tool for on-demand information\n\nThe knowledge_base tool provides a way for agents to access specialized information\nwhen needed. Initial topics include:\n\n- sketch: how to use Sketch, including SSH, secrets, and file management\n- go_iterators: information about Go\u0027s iterator feature added in Go 1.22\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "05224846741b500d8103bd6930a7717e1211ef01", "tree": "54eaad583c19b5c00719384a268d8dccab0dd13e", "parents": [ "80b488d853e2766b638e65cfe44a5904d7cb24ee" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 18:26:08 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 18:26:08 2025 -0700" }, "message": "browser: add window resize tool with chromedp\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s68bdc7fb84309a7ck\n" }, { "commit": "80b488d853e2766b638e65cfe44a5904d7cb24ee", "tree": "abd280ec3924efb4dbbd72622129bc4af56e28ca", "parents": [ "6458e7c751856a9f7f4004367cb6274e564a1489" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 18:21:54 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sun May 11 01:22:37 2025 +0000" }, "message": "Browser tools: initialize lazily and add timeouts.\n\nAlso rename browser_screenshot to browser_take_screenshot for clarity\\n- Update both Go and UI code to maintain consistency\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8a5cabff914f88dfk\n" }, { "commit": "72252cbcb97840d724133be67c4c69cc69ebb2d3", "tree": "a361499dc3fa6b9af2be3e74cfd59fd8ba34690e", "parents": [ "7ce5fb76d8748ebf73c5adf9d6cd8eb67716fba8" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 17:00:08 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 10 17:00:08 2025 -0700" }, "message": "llm and everything: Update ToolResult to use []Content instead of string for multimodal support\n\nThis was a journey. The sketch-generated summary below is acceptable,\nbut I want to tell you about it in my voice too. The goal was to send\nscreenshots to Claude, so that it could... look at them. Currently\nthe take screenshot and read screenshot tools are different, and they\u0027ll\nneed to be renamed/prompt-engineered a bit, but that\u0027s all fine.\n\nThe miserable part was that we had to change the return value\nof tool from string to Content[], and this crosses several layers:\n - llm.Tool\n - llm.Content\n - ant.Content \u0026 openai and gemini friends\n - AgentMessage [we left this alone]\n\nExtra fun is that Claude\u0027s API for sending images has nested Content\nfields, and empty string and missing needs to be distinguished for the\nText field (because lots of shell commands return the empty string!).\n\nFor the UI, I made us transform the results into a string, dropping\nimages. This would have been yet more churn for not much obvious\nbenefit. Plus, it was going to break skaband\u0027s compatibility, and ...\nyet more work.\n\nOpenAI and Gemini don\u0027t obviously support images in this same way,\nso they just don\u0027t get the tools.\n\n~~~~~~~~~~ Sketch said:\n\nThis architectural change transforms tool results from plain strings to []Content arrays, enabling multimodal interaction in the system. Key changes include:\n\n- Core structural changes:\n - Modified ToolResult type from string to []Content across all packages\n - Added MediaType field to Content struct for MIME type support\n - Created TextContent and ImageContent helper functions\n - Updated all tool.Run implementations to return []Content\n\n- Image handling:\n - Implemented base64 image support in Anthropic adapter\n - Added proper media type detection and content formatting\n - Created browser_read_image tool for displaying screenshots\n - Updated browser_screenshot to provide usable image paths\n\n- Adapter improvements:\n - Updated all LLM adapters (ANT, OAI, GEM) to handle content arrays\n - Added specialized image content handling in the Anthropic adapter\n - Ensured proper JSON serialization/deserialization for all content types\n - Improved test coverage for content arrays\n\n- UI enhancements:\n - Added omitempty tags to reduce JSON response size\n - Updated TypeScript types to handle array content\n - Made field naming consistent (tool_error vs is_error)\n - Preserved backward compatibility for existing consumers\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1a2b3c4d5e6f7g8h\n" }, { "commit": "a997be617bd5548a307c708cdba325ea6562acec", "tree": "1890bd46f87cf7261ba5a6aaa3fd2d110e7d67cc", "parents": [ "dce8d84fdcda62e03b13c95d7614ee7ab983ddca" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 22:52:46 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 15:32:45 2025 -0700" }, "message": "claudetool/onstart: add codebase analysis tool and basic memory support\n\nThis is a preliminary approach. Big high level questions include:\n\n* should we keep the multichoice tool prompting?\n* should we push the list of quidance files or respond with them during codereview?\n* should we use the list of docs and build files at all?\n* are there other files we should hint (e.g. editor settings, something from aider, etc.)?\n\nWe should probably also blog about dear_llm.md to stop the endless proliferation of new files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "dce8d84fdcda62e03b13c95d7614ee7ab983ddca", "tree": "bf7f0dad1b07863b80ae16ea0a183a9c2b270b34", "parents": [ "f59df72da9277859885d4dd65afe76d6de70fd0d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 12:27:19 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 15:32:17 2025 -0700" }, "message": "claudetool: remove mention of go doc" }, { "commit": "f59df72da9277859885d4dd65afe76d6de70fd0d", "tree": "c5972e45b3666b5e66277bc864b923bb6339771b", "parents": [ "113e2053753d5b3e08be0f3509a204fe24ba5e87" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 12:24:03 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 09 15:31:54 2025 -0700" }, "message": "claudetool: don\u0027t list out available binaries in bash tool\n\nIt does what it wants to do; I\u0027ve never seen it use this information.\nAnd there\u0027s no impact on Go eval, and it biases the model\ntowards assuming it is working in Go.\n\n" }, { "commit": "4d54493fe3808ecd0c6a9a4d0bbcc7786e97b094", "tree": "9877409fc95aaa32aed882426ea15464b8036e34", "parents": [ "a4ad8af8b08a54326bbbd99d57110a42c459c54e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 13:33:53 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu May 08 19:25:30 2025 +0000" }, "message": "all: support hiding subconvo output\n\nSome of it is systematically noisy.\n" }, { "commit": "1ed1cc459db26690eddb05da525e1580ae688b59", "tree": "d94c5ad81bdc6aa15af399081cac3d915592eadc", "parents": [ "85de37e3be602ca67ef5e2f2185ee819323ec31c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 20:21:40 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 14:03:38 2025 -0700" }, "message": "codereview: only run go mod tidy when go.mod has changed\n\nThis change modifies the codereview tool to only run \u0027go mod tidy\u0027 when\ngo.mod or go.sum files have been modified.\n\nThis makes sketch work better with non-Go projects.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "924a77070bc1edc1a861de6462a4ef7bb863e648", "tree": "c29b5c835e41c7df9ead819659b8ec9283ade2fc", "parents": [ "4936de3834a2c70db563d19975c072bb42d953cc" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue May 06 02:14:40 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 14:02:13 2025 -0700" }, "message": "claudetool: improve commit message guidance in precommit tool\n\nThe precommit tool now:\n1. Directly executes git commands to extract example commit messages\n2. Displays actual commit message examples from the repository\n3. Includes stronger, mandatory language about following the commit style\n\nThis improves compliance with the repository\u0027s commit message style by\nproviding clear examples rather than just instructions on how to view them.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "593ca6455ea14bf48f6ae4ff6f5d3296c6befa7b", "tree": "195d854dcbad19949840d997701315c6c2496965", "parents": [ "a23587b969adfacca40bd443b74fa37d2d91dc79" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 05:19:32 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed May 07 14:00:16 2025 -0700" }, "message": "claudetool: disable prompt caching for one-shot subconvos" }, { "commit": "b6d6d384a2cdbabd2d6abc94bd6b2c6c937d5e22", "tree": "64194d0e29739fc0dd686c23590f140d3ae883b7", "parents": [ "31f152493929b3b171c140c2e0e22dbcf9059cac" ], "author": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Wed May 07 10:29:03 2025 +0100" }, "committer": { "name": "Pokey Rule", "email": "755842+pokey@users.noreply.github.com", "time": "Wed May 07 13:34:11 2025 +0100" }, "message": "claudetool: Add SKETCH\u003d1 to bash tool environment\n\nThis adds the SKETCH\u003d1 environment variable to all commands executed\nby the bash tool. This makes the SKETCH environment variable available\nto scripts and programs run through the bash tool.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "4962f153fa0d6812543addd690be8dba8c04a406", "tree": "3a90c5736422c77f8ef3dc8d552640e65f1e07a0", "parents": [ "8b43ffbf41d06aee5ad5c734f1661a3b62cfe1a9" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue May 06 17:24:20 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue May 06 17:24:20 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "33d282f80db786cc60ba521a38ed5166f23239ed", "tree": "9ed1f15c6d3081d5bef7d16b9d72e78a2c7780cf", "parents": [ "a9d87aa69cfefdc91ec7aaa6bc42907749748e76" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Sat May 03 04:01:54 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Tue May 06 10:23:39 2025 -0700" }, "message": "Add browse tool support.\n\nI reviewed some MCPs (using OpenAI\u0027s deep research to help), and it\nhelped me choose chromedp as the relevant library and helped me come up\nwith an interface. This commit adds chrome to the Docker image which is\nkind of big. (I\u0027ve noticed that it\u0027s smaller on Ubuntu, where it doesn\u0027t\npull in X11.) go-playwright was a library contender as well.\n\nImplement browser automation tooling using chromedp\n\nThis implementation adds browser automation capabilities to the system via the chromedp library,\nenabling Claude to interact with web content effectively.\n\nKey features include:\n\n1. Core browser automation functionality:\n - Created new browsertools package in claudetool/browser\n - Implemented tools for navigating, clicking, typing, waiting for elements,\n getting text, evaluating JavaScript, taking screenshots, and scrolling\n - Added lazy browser initialization that defers until first use\n - Integrated with the agent to expose these tools to Claude\n\n2. Screenshot handling and display:\n - Implemented screenshot storage with UUID-based IDs in /tmp/sketch-screenshots\n - Added endpoint to serve screenshots via /screenshot/{id}\n - Created dedicated UI component for displaying screenshots\n - Ensured proper responsive design with loading states and error handling\n - Fixed URL paths for proper rehomed URL support\n - Modified tool calls component to auto-expand screenshot results\n\n3. Error handling and reliability:\n - Added graceful error handling for browser initialization failures\n - Implemented proper cleanup of browser resources\n\nThe browser automation tools provide a powerful way for Claude to interact with web content,\nmaking it possible to scrape data, test web applications, and automate web-based tasks.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "c72ceb21966c5e7685db6c9bf28074042265d683", "tree": "8113c9aa9fe4fe689b8b1e3b7ff01c1aa5ff27bb", "parents": [ "bfebfd141e023d12ae44ddcbcfc78c4d83b13344" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 23:30:15 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:20:24 2025 -0700" }, "message": "claudetool: add go mod tidy check to codereview\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "a70246a5b4b528205491bf3052ac0cf98124e9c7", "tree": "441f2c42a750b69cd5ff3229be7bf2d420a2b17f", "parents": [ "f4047bbd0440a5cec265ef6805f97d18ca3b255e" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 16:26:18 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:20:24 2025 -0700" }, "message": "claudetool/codereview: add readme.md\n" }, { "commit": "f4047bbd0440a5cec265ef6805f97d18ca3b255e", "tree": "e8d00718fe1363d3826156bc5ce61fad1c25c582", "parents": [ "e750ec9fb4821e241d6f0fbdfa1aeb031d025168" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 23:02:56 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 18:20:24 2025 -0700" }, "message": "claudetool/codereview: new package extracted from claudetool\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "d7970e62c34243aaa9c7a6405ceedfb9a073a783", "tree": "aaf5359fe136652624580e48d240d732790de3c2", "parents": [ "e2518e5c7b80d3f0d8f3017e34b19e59d6deeac6" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 01 01:56:28 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 13:48:06 2025 -0700" }, "message": "claudetool: change title tool to precommit, add commit style guidance\n" }, { "commit": "e2518e5c7b80d3f0d8f3017e34b19e59d6deeac6", "tree": "4d23f4326e76a8f587b4a148aa4347cdc9d243f6", "parents": [ "503b5e3543f69cc56c05bd02ce48666259d01f90" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 11:13:40 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon May 05 13:48:06 2025 -0700" }, "message": "claudetool: add experimental LLM reviewer stage" }, { "commit": "4f84ab729ddbf428b0e891940f08f70b4edee05c", "tree": "f2e52e4a01c188ada1f5acf8b2a013029b999495", "parents": [ "44f9b4cec11e269a52fbfc099989ab425b8e125f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 22 16:40:54 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri May 02 12:57:44 2025 -0700" }, "message": "all: support openai-compatible models\n\nThe support is rather minimal at this point:\nOnly hard-coded models, only -unsafe, only -skabandaddr\u003d\"\".\n\nThe \"shared\" LLM package is strongly Claude-flavored.\n\nWe can fix all of this and more over time, if we are inspired to.\n(Maybe we\u0027ll switch to https://github.com/maruel/genai?)\n\nThe goal for now is to get the rough structure in place.\nI\u0027ve rebased and rebuilt this more times than I care to remember.\n" }, { "commit": "2fde465a8a7099e3e121ecc552d6420d8c4636cb", "tree": "35ef77eef0818029ec53abd91b5f6bbcc2547d43", "parents": [ "2bdc95355f3ace09daab7c5a3413ee64e4fd979d" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 01 17:50:34 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu May 01 17:51:19 2025 -0700" }, "message": "claudetool: redact initial commit hash in test fixtures\n\nThis was causing failures when switching between Go versions,\nbecause the go version ends up in the go.mod, which changes\nthe git hash.\n" }, { "commit": "250348ec3c8bcba21caa62fd4ae53e4d9476fafa", "tree": "4be2e3660c84343af23cb81fa14b3c7a5709ebfd", "parents": [ "eb5166a5043946ad6782cb64af987d23a32e3203" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 10:31:28 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 11:13:16 2025 -0700" }, "message": "all: mild prompt engineering\n\n" }, { "commit": "d499fd6e12374193df2d769d12753bbd1ddd1493", "tree": "e8c2a8a134773164910b78b4606bd196c55d6d5f", "parents": [ "dae19073cfdc420af8da311c4f3974ac6db77b46" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 01:31:29 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 19:06:19 2025 -0700" }, "message": "Add permission callback to Bash tool to enforce title setting before git commits\n\n- Convert Bash tool from a plain variable to a struct with a permission callback function\n- Add a constructor for creating Bash tools with custom callback functions\n- Implement a permission check in agent.go that verifies a branch name has been set before allowing git commits\n- If no branch name is set and a git commit command is attempted, return an error instructing to use the title tool first\n- Maintain backward compatibility for tests with BashRun function\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "dae19073cfdc420af8da311c4f3974ac6db77b46", "tree": "d2744d5ff7a9477e34537d7be9b772a3d3de522b", "parents": [ "a9b3822fd2cfdc035e7daa8b59496f9838a4b2c6" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Apr 30 01:08:57 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 19:06:19 2025 -0700" }, "message": "Add WillRunGitCommit function to bashkit\n\nThis function inspects a bash script to determine if it will run \u0027git commit\u0027 commands.\nThe implementation is similar to the existing Check function but is kept separate as requested.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "2b2090e7fa8fd53eda5dca132980c3e2fa4cb3b8", "tree": "269e400e420a30079bfbaa7dea562e4b6d80d9d7", "parents": [ "c72fff52ad6c5436fa8469f049fa232f2f6be5ed" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 16:41:34 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 16:42:35 2025 -0700" }, "message": "claudetool: add missing json schema field\n\nApparently this is required, and Anthropic happens to not care." }, { "commit": "2dc86b9bda41cf30926833b044d8669d92273b20", "tree": "e8ebd9c7710b6cfa311d7fcdb1c26b2a2b482162", "parents": [ "50a1d62e0dc285c69b7c495e4a01a47fa3561692" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 14:11:58 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 10:48:31 2025 -0700" }, "message": "Update codereview tool UI to show different icons based on result content\n\nAdded cross-linking comments between the UI component and differential.go\nto ensure they stay in sync going forward.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e" }, { "commit": "d4b1cc76714b5e139e539259673afe7d74afa1cb", "tree": "69e30a90339000e8a42fba993acff1c58a7d2e98", "parents": [ "15272c1ddf6eba4e8d9b2697a3e6af09752a6cf1" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 13:49:18 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Apr 29 06:53:26 2025 -0700" }, "message": "Sort git rev-list by time and limit results for related files\n\nSort the git rev-list by time (descending) and limit the result to max 100\ncommits in the related files calculation. This limits runtime and favors\nrecent commits, which are more likely to be relevant.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\n" }, { "commit": "15272c1ddf6eba4e8d9b2697a3e6af09752a6cf1", "tree": "c8022595159550121c43087cdfab1620d9ae6539", "parents": [ "9c07e1d30f7f576bd7679a155873f6419743e6a6" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Apr 28 19:36:15 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Apr 28 19:36:15 2025 -0700" }, "message": "claudetool: add two more \"related files\" todos" }, { "commit": "ffecb1e9fbaca84d7f142aa294fa795c324587af", "tree": "00bcb908d9a6b2d09b0ebf7bc3d1024f5cd9a16d", "parents": [ "ffb58a30bd18ff8e21d228ac072dc636cb0510c8" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Apr 28 18:59:14 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Apr 28 16:15:34 2025 -0700" }, "message": "claudetool: add \"related files\" detection to codereview tool\n\n" }, { "commit": "833a0f83af52be33eed3baf0f67ffa3ddab55cb4", "tree": "0f1c78df648d499ed6ac3e975276ecc977ab4e70", "parents": [ "0cbd976940708c94eb8b644b14bfda3f19087997" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Apr 24 18:39:36 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Apr 25 19:31:00 2025 -0700" }, "message": "claudetool: improve codereview\n\nDo a bunch of un-vibecoding and bug fixing.\nUnfortunately, lots left.\nGet rid of vet; gopls check covers it.\n\nAdd testing infrastructure and a bunch of fixtures.\n" }, { "commit": "56ac60591b50b638ee292e45489d2f0301e4f638", "tree": "5de7ee9bea124df72536da07d996921c66b6c00f", "parents": [ "7660e4e48978ed32452ed444514238c41c27c0c9" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Apr 24 10:40:55 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Apr 24 10:40:55 2025 -0700" }, "message": "claudetool: add some cleanup TODOs to bash background execution\n" }, { "commit": "b4c7e1bdd860836d46c64065f268007e721e5fb8", "tree": "fb3cd12df47cd05c4f549e39f5ee29fcd360040f", "parents": [ "d43e57238d1b471938d10faa5b9762394b70aaac" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:30:58 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:30:58 2025 -0700" }, "message": "Removing LLM-generated skipped test\n" }, { "commit": "b60f0f2c4f6baf635c7eeb2972d6ed25d9485891", "tree": "93a6b85c34010b1ef41072aad309e56e759694a6", "parents": [ "cc3ba220373b62dd5a098180ea6797af76407525" ], "author": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 18:19:32 2025 +0000" }, "committer": { "name": "Philip Zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Wed Apr 23 19:28:10 2025 -0700" }, "message": "Add background execution capability to bash tool\n\nThis adds a \u0027background\u0027 flag to the bash tool, allowing commands to be\nexecuted without waiting for them to complete. When running in background\nmode, the command\u0027s stdout and stderr are redirected to temporary files,\nand the tool returns the PID and paths to these files as a JSON structure.\n\nFeatures:\n- Different default timeouts: 1m for foreground, 10m for background\n- Goroutine to properly reap background processes when they complete\n- Uses \u0027sketch-bg-\u0027 prefix for temporary directories\n- Added robust test helpers for waiting on files and processes\n- Skip agent test that uses recorded HTTP interactions incompatible with\n the updated schema\n\nSketch also updated the various UIs\n\nCo-Authored-By: sketch\n" }, { "commit": "2e463fb649fcff14d4025ddb91f630a98e7da526", "tree": "0e86854d80d2759a913870655f13226c31f9d30c", "parents": [], "author": { "name": "Earl Lee", "email": "earl.lee@sketch.dev", "time": "Thu Apr 17 11:22:22 2025 -0700" }, "committer": { "name": "Earl Lee", "email": "earl.lee@sketch.dev", "time": "Thu Apr 17 11:35:33 2025 -0700" }, "message": "Initial commit\n" } ] }