)]}' { "log": [ { "commit": "38707678bca1807a3350b01cae952293517750da", "tree": "e5813a760f2b7ed985a4e1060632be244b9ded77", "parents": [ "051c3cdbf801bc84d5e9a71c90acc1a985c9215c" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 30 20:18:39 2025 +0000" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 30 13:29:02 2025 -0700" }, "message": "webui: add dark mode support to sketch-tool-calls render\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sab350b872b2e9fc0k\n" }, { "commit": "051c3cdbf801bc84d5e9a71c90acc1a985c9215c", "tree": "79aafef6c4ab483ed0c6abb280492e63e0a17f81", "parents": [ "66f45a6e299d2d0b758584f3c61e248f7c1ddaa8" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 30 13:23:03 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 30 13:23:03 2025 -0700" }, "message": "webui fix external message demo\n" }, { "commit": "66f45a6e299d2d0b758584f3c61e248f7c1ddaa8", "tree": "5493a445d4797f419a0247171fdf6bade654af4f", "parents": [ "6b1ceb1b40de83ac2bdbb83dd9aa38feeb69a17e" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 30 13:11:25 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 30 20:12:11 2025 +0000" }, "message": "webui: fix rendering for gh worfkow events\n" }, { "commit": "6b1ceb1b40de83ac2bdbb83dd9aa38feeb69a17e", "tree": "0c4292301139d00979e28898df3d94a419946a0a", "parents": [ "136b66dbe453549cc411abc6e6b2144cfb368add" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 12:09:43 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 12:19:56 2025 -0700" }, "message": "webui: fix patch display whitespace (again)\n\nThanks, prettier, for breaking my code. Again.\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": "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": "9b1c003e7374003edd7f7316c08d494a8aa06b2b", "tree": "351d13bf0611697514fcc6e1616db81e36e5f8d4", "parents": [ "2d832196004efc3ba31a83fdb207550d997b5e7b" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Sat Jul 26 17:53:09 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 09:44:21 2025 -0700" }, "message": "webui: silence more build spam\n" }, { "commit": "2d832196004efc3ba31a83fdb207550d997b5e7b", "tree": "ebcabedc8a7371325b90d5e869a8e8ce3e6239b4", "parents": [ "a77889bc1a0710acfb3fab612fc941d7a1b2f087" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 28 23:03:59 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 30 09:44:21 2025 -0700" }, "message": "webui: filter out empty messages\n\nThis shouldn\u0027t happen, but they do sometimes.\nThey\u0027re ugly but are harmless (or so it appears).\nFilter them away.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc3177fd6d7337586k\n" }, { "commit": "3d1308e3468915212bea595ddef14952eb82f7fc", "tree": "de337e13601c4f73496f529bcd0ddfa6fc6d2cf5", "parents": [ "5c86165937d556a7a37f51f609661d9c4910087e" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Tue Jul 29 17:20:10 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 29 17:23:01 2025 +0000" }, "message": "webui: dark mode support to demo fmwk, tool cards\n\nDemo framework fixes:\n- sketch-push-button.demo.ts: add dark variants for bg, border, text colors\n- chat-input.ts: add dark variants for message containers and status sections\n- demo-runner.ts: replace inline error styles with Tailwind dark mode classes\n- sketch-call-status.demo.ts: fix hardcoded white backgrounds in status cards\n- sketch-diff-range-picker.demo.ts: add dark variants to picker and status displays\n- sketch-timeline-message.demo.ts: fix message container backgrounds\n- sketch-view-mode-select.demo.ts: comprehensive dark mode for all scenarios\n\nTool card fixes:\n- Update shared createPreElement function with dark:bg-gray-700/dark:text-gray-100\n- bash tool: fix command display and result areas\n- think tool: fix input content area with proper dark background\n- patch tool: comprehensive diff rendering with dark variants for added/removed/context lines\n- codereview tool: inherits dark mode through shared utilities\n\nAll components now use consistent dark mode patterns with proper contrast:\nbg-white dark:bg-gray-800, border-gray-200 dark:border-gray-700,\ntext-gray-600 dark:text-gray-300, matching existing components.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8ac5253d0cbaa3ack\n" }, { "commit": "02f4820e4a5245f545293c34e121d52640d3cb11", "tree": "53ca4cb172e9c84ec0238c2ccb7095c08100e12f", "parents": [ "a8ac1500f4313203d0b0d453a59aa016a2b93859" ], "author": { "name": "philz", "email": "philip.zeyliger@gmail.com", "time": "Mon Jul 28 21:02:07 2025 +0000" }, "committer": { "name": "philz", "email": "philip.zeyliger@gmail.com", "time": "Mon Jul 28 21:16:02 2025 +0000" }, "message": "webui: expand patch tool cards by default\n\nMade patch tool cards expand by default to show inline diffs\nimmediately, similar to screenshot tool cards.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s56bd70ba6302978dk\n" }, { "commit": "7cae9d5d4c629d3c1f85aaab974632e093a8959d", "tree": "d2dab86b28c19100b55a14d06f9c631860d77860", "parents": [ "e34ffd60e0a8dfc5ba55be35d68a396a73d53750" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 23:08:39 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 25 13:36:14 2025 -0700" }, "message": "webui: add JavaScript alert for file save failures\n\nNow that Philip fixed #213, this should be safe to unleash on everyone.\n\nJavaScript alerts are ugly, but:\n\n- this shouldn\u0027t happen\n- if it does, the user should really know that their edits are in peril\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s95d29aed61ef60f6k\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": "aeaf892c3a40e99c699c006dbccbe9c859e0cc69", "tree": "f0db4d9b4f9826ceb527976e1846c7e64bbcd44e", "parents": [ "01bf5ae0d7249d83735e65b916c776825983aa34" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 24 18:31:16 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 24 18:31:16 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "01bf5ae0d7249d83735e65b916c776825983aa34", "tree": "8f83cd0ac09452ab66aaf20169d5d7dfdabb0457", "parents": [ "beaa86a1fa821ce5874f3a2dd6a1609f7bd79698" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 17:50:00 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 11:30:37 2025 -0700" }, "message": "webui: restore command+enter keyboard shortcut for diff comments\n\nThis used to exist. I missed it. Bring it back.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6cd4f7fa81a2ee71k\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": "141547d1d7169bf1c9a9d156eb6b585f1d9d398c", "tree": "e21b530b625a3433653a08fa6ec581a0b2add8a6", "parents": [ "568bebf2f2a23917ac6ce7d2f2f30aba54a57eca" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 24 09:21:18 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 24 16:25:01 2025 +0000" }, "message": "sketch-app-shell: nav to /messages/ on sessionEnd\n\nFixes https://github.com/boldsoftware/bold/issues/457\n" }, { "commit": "568bebf2f2a23917ac6ce7d2f2f30aba54a57eca", "tree": "0fa126370bdc31193834bd00b2d0ed36b952b82b", "parents": [ "7fe76973f6cba7c7a51f4b896444c2ce3c1ce0d5" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 24 02:44:54 2025 +0000" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 24 02:51:34 2025 +0000" }, "message": "webui: fix IDLE/DISCONNECTED oscillation by replacing initialLoadComplete misuse\n\nFix DataManager session_ended handler that caused rapid status switching\nbetween IDLE and DISCONNECTED when sessions ended.\n\nIssue Description:\nWhen a session ended, the UI\u0027s sketch-call-status component would rapidly\noscillate between IDLE and DISCONNECTED states, suggesting continued\nconnection attempts despite the session being over.\n\nRoot Cause Analysis:\nThe primary issue was session_ended handler improperly firing initialLoadComplete\nevent, confusing timeline component state management. The initialLoadComplete event\nwas added for newsessions functionality but broke assumptions in regular sketch\nsessions. The timeline component expects initialLoadComplete to signal new page\nloads, not ended sessions.\n\nWhen fired for ended sessions, this caused UI re-render cycles and state confusion\nthat interfered with connection status display, creating the oscillation effect.\n\nImplementation Changes:\n- Added sessionDataReady event type for ended session data readiness\n- Modified session_ended handler to emit sessionDataReady instead of\n initialLoadComplete for ended sessions\n- Preserved initialLoadComplete semantics for active session data loading\n- Added clarifying comment to existing isSessionEnded guard in connect()\n\nSecurity \u0026 Compatibility:\n- No security implications, purely UI state management fix\n- Newsessions component unaffected (doesn\u0027t use initialLoadComplete)\n- Timeline component behavior restored to expected state\n- All existing event handling preserved\n\nThis fix eliminates the connection status oscillation while maintaining\nproper separation of concerns between live session loading and ended\nsession display states.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: scd1b437c8ae7c381k\n" }, { "commit": "7fe76973f6cba7c7a51f4b896444c2ce3c1ce0d5", "tree": "619c0b433cd856fd2b0a977a3453dd9b79105d54", "parents": [ "3dd3e4144d2e0b9fec6b5e0039ed12e2c0170265" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 23 19:19:01 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 23 19:19:01 2025 -0700" }, "message": "data.ts: add missing .isSessionEnded check in .connect()\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": "a6b995b83277dd730e35a8f5be14c139d82a544e", "tree": "2cb9d54d6c38916a6639061e2086f24055e68ecf", "parents": [ "57afbca4ac1dbd4351aae93302e34ee45b36a25f" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 24 00:45:05 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 23 17:53:58 2025 -0700" }, "message": "webui: kill eslint\u0027s no-explicit-any warning dead dead dead\n\nA sign that it maybe it\u0027s not a good fit for us\nis the fact that it was sprinkled around our codebase\nas prolifically as blinding snow in an Antarctic blizzard.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s79f40917a73a3320k\n" }, { "commit": "aa22eb7947ae6bb09ce558bcfa52a1d8b2b3286a", "tree": "89a95de4ba4f9e7889f6133dea31a7e5f73f54c6", "parents": [ "9dc2f62c6d44f53e057bd7319ffbcbc00ef5f2a0" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 23 19:59:36 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 23 19:59:36 2025 +0000" }, "message": "all: fix formatting\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": "1ee0bc6c8f6ee7a6d0b26189815879f7455069c6", "tree": "9779a6c73738efb9375887d2dcd6de84a3ddcbc6", "parents": [ "d37f7a73a0b77329d4701edee3d6d9a6b79d4e47" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Tue Jul 22 23:24:18 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 22 23:31:38 2025 +0000" }, "message": "webui: add dark mode support to demo server\n\nCore Component Dark Mode Support:\n- sketch-timeline.ts: Welcome box, loading indicators, thinking bubbles, navigation\n- sketch-tool-card-base.ts: Status icons, elapsed time, hover states, details panel\n- All 14 sketch-tool-card-* components: Consistent dark styling for tool results\n\nDemo System Infrastructure:\n- Enhanced demo runner (demo.html) with complete dark theme CSS variables\n- Added sketch-theme-toggle integration in sidebar for easy theme switching\n- Extended demo-fixtures utilities with semantic color system (8 new CSS variables)\n- Comprehensive color mappings: backgrounds, text, borders, controls, buttons\n\nDemo File Compatibility (13 files updated):\n- Fixed 60+ instances of hardcoded colors across all demo components\n- Replaced light-mode-only colors (#24292f, #f6f8fa, etc.) with CSS variables\n- Updated text colors, backgrounds, borders for proper contrast in both themes\n- Maintained visual hierarchy while ensuring accessibility\n\nTechnical Implementation:\n- CSS custom properties system with automatic :root/.dark theme switching\n- GitHub-inspired dark color palette for professional appearance\n- Smooth 0.2s transitions for seamless theme changes\n- Semantic variable naming for maintainability and consistency\n\nKey Features Added:\n- Theme toggle accessible from any demo (no need to navigate to Theme Toggle demo)\n- Complete visual consistency between light and dark modes\n- Proper contrast ratios throughout for accessibility\n- Tool card demos showcase dark mode styling with realistic content\n\nComponents Updated:\nTimeline: Welcome messages, loading states, thinking indicators, jump button\nTool Cards: Status icons, input/output display, hover states, detailed views\nDemos: Labels, backgrounds, instruction panels, control elements, text content\n\nThe demo system now provides a complete, professional dark mode experience\nthat matches modern development tool standards with excellent usability\nand visual consistency across all components and demonstrations.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s97589e2fe2fdeeb3k\n" }, { "commit": "d37f7a73a0b77329d4701edee3d6d9a6b79d4e47", "tree": "07e5bb969f7a1e48292fa8c4674f4f1cb9fce58c", "parents": [ "488e8a476fccf2514f3703d0d576c35989987b34" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 13:43:52 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 22 06:54:08 2025 -0700" }, "message": "webui: make diff UI update after data loads\n\nSometimes diff content would load quickly but the UI would stay stuck\nshowing \u0027Loading...\u0027 for many seconds until some external action\n(like clicking in console) triggered a repaint.\n\nRoot cause: Lit wasn\u0027t automatically detecting that it needed to re-render\nwhen the async loadDiffData() completed and this.loading changed from\ntrue to false.\n\nFix: Explicitly call requestUpdate() to force a re-render after loading\ncompletes, ensuring the UI immediately shows the loaded diff content.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6ba963f401c89febk\n" }, { "commit": "488e8a476fccf2514f3703d0d576c35989987b34", "tree": "6e4935a8e789057408d1c8037fbca5ab2cbd51b9", "parents": [ "d1b7dd616f47c873e241d2354f03feacce46e637" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 22 02:47:29 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 22 02:47:29 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "15a0ffa7a68f5915eaed6044260edc794c806283", "tree": "20f7b89d99ff320f89ebb7025cc71aa2bcef3597", "parents": [ "0f00427c39583c905df906ed16843dadce8e19bd" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 15:53:48 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:46:45 2025 -0700" }, "message": "webui: improve reconnection delays\n\n- make them a lot more aggressive,\n as the user is likely sitting and staring at the screen\n- make the numbers rounder (easier to grok)\n- add some jitter\n" }, { "commit": "0f00427c39583c905df906ed16843dadce8e19bd", "tree": "bedcfe4b085e781317c7f2a40998094423b168fe", "parents": [ "106f6f5472c233011b6abee307cb6cd0e962f718" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 22:40:03 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 21 19:46:45 2025 -0700" }, "message": "webui: refactor reconnection timeout logic to use lookup table\n\nNo functional changes.\nJust a lot clearer, and a lot easier to reason about and edit.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3faf2c10280df867k\n" }, { "commit": "cf724583660cf683efa7fd8eb9b32e120e88beb7", "tree": "ca8f4072f3e131ace3400552698caa04786a155c", "parents": [ "4432056dc2db2f4b9b1120a890255a540a4c8537" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 21 22:22:57 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 21 22:45:56 2025 +0000" }, "message": "skaband: move timeline viewer from sketch/webui to resolve bundle conflicts\n\nMove read-only timeline viewer functionality from sketch/webui to skaband\nto eliminate custom element registration conflicts when using newsessions\nwith newui\u003dtrue parameter.\n\nChanges made:\n- Created skaband/sketch-timeline-readonly.ts with timeline viewer functionality\n- Updated skaband/messages.gohtml to use local sketch-timeline-readonly.js bundle\n- Removed messages-viewer.ts from sketch/webui build configuration\n- Updated Makefile to remove messages-viewer bundle extraction steps\n- Added sketch-timeline-readonly.ts to skaband build.js entry points\n\nFiles created:\n- skaband/sketch-timeline-readonly.ts: Timeline viewer component for archived sessions\n- skaband/sketch-timeline-readonly.test.ts: Test coverage for timeline functionality\n\nFiles removed:\n- sketch/webui/src/messages-viewer.ts: Relocated to skaband\n- sketch/webui/src/messages-viewer.test.ts: Relocated to skaband\n\nBuild configuration updates:\n- sketch/webui/esbuild.go: Remove messages-viewer.ts from bundleTs arrays\n- sketch/webui/package.json: Remove messages-viewer.js export\n- skaband/build.js: Add sketch-timeline-readonly.ts entry point\n- Makefile: Remove messages-viewer bundle extraction from build and prepare-deploy\n\nThe timeline viewer is only used by skaband for archived session viewing,\nso co-locating it with skaband eliminates the architectural issue where\nboth messages-viewer.js and newsessions-components.js contained overlapping\ncustom elements (sketch-timeline, sketch-tool-card-*, etc.) causing\nregistration conflicts.\n\nThis change resolves bundle conflicts on /newsessions?newui\u003dtrue while\nmaintaining identical functionality for archived message viewing.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sd62ec3594672899ek\n" }, { "commit": "4432056dc2db2f4b9b1120a890255a540a4c8537", "tree": "8e9738b3335fb5112ddb2ebfe70fe3aa8fb23f44", "parents": [ "d8eb4649835351b57d0143242b96547e8e67a14a" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 21 11:09:38 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 21 18:10:38 2025 +0000" }, "message": "webui: update DEAR_LLM.md, rm .shadowRoot refs\n" }, { "commit": "d8eb4649835351b57d0143242b96547e8e67a14a", "tree": "14130199c24d7b38777288b0d9849d36dac8d4c2", "parents": [ "44f7616740655a904c6b0319d739bfce97f4951b" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 21 03:04:52 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 21 17:50:00 2025 +0000" }, "message": "webui: migrate sketch-monaco-view to tailind\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2b8f6e849aabb45fk\n" }, { "commit": "44f7616740655a904c6b0319d739bfce97f4951b", "tree": "6728e7919443b389c53009365efc881089e47671", "parents": [ "581bd795582ab99b1aa81a39466a8433c14d4485" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Mon Jul 21 03:04:52 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 21 03:09:20 2025 +0000" }, "message": "webui: remove sketch-network-status component\n\nRemove legacy sketch-network-status component that was emptied in\nprevious commits. The component has been superseded by sketch-call-status\nwhich handles all connection status functionality.\n\nFiles removed:\n- sketch-network-status.ts: Component with empty render (display: none)\n- sketch-network-status.test.ts: Test file for invisible component\n- sketch-network-status.demo.ts: Demo file for non-functional component\n\nUpdated references:\n- sketch-app-shell-base.ts: Remove import and component usage\n- status-indicators.demo.ts: Remove unused import\n- demo-framework/demo-runner.ts: Remove from component list\n- index-generated.html: Regenerated without network status demo\n\nConnection status functionality is now fully handled by sketch-call-status\ncomponent which properly displays IDLE/WORKING/DISCONNECTED states with\nvisual indicators and animations. This removal eliminates dead code and\nreduces the LitElement migration scope to just sketch-monaco-view.ts.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2b8f6e849aabb45fk\n" }, { "commit": "581bd795582ab99b1aa81a39466a8433c14d4485", "tree": "ce1e1b90c6800486ad94c8f1fa58cf2cbbce9e05", "parents": [ "d52d39d31ad33400b6bdb0ff678a6739de92b2eb" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jul 20 18:30:12 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jul 20 18:34:11 2025 -0700" }, "message": "webui: s/demo-runner/demo/g\n" }, { "commit": "d52d39d31ad33400b6bdb0ff678a6739de92b2eb", "tree": "a777c975562ee7ace6e3459a4701f6098c80fa8c", "parents": [ "23a35b8da9da135ad130bee89a1f96cac4d02bf6" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jul 20 14:57:38 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 21 01:16:10 2025 +0000" }, "message": "webui: clean up component demos\n" }, { "commit": "23a35b8da9da135ad130bee89a1f96cac4d02bf6", "tree": "5d76d0de2217cd78ba33ed0dd3fa00f725bac90d", "parents": [ "9ee3042aa7826d9ae662c86743a5f87468972d27" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jul 20 21:18:31 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sun Jul 20 21:49:21 2025 +0000" }, "message": "webui: migrate mobile components to SketchTailwindElement\n\nComplete migration of all mobile web components from LitElement to\nSketchTailwindElement base class with Tailwind CSS styling:\n\nComponents migrated:\n- mobile-chat-input.ts: Chat input with file upload, textarea auto-resize\n- mobile-chat.ts: Message display with markdown rendering and tool calls\n- mobile-diff.ts: Git diff viewer with Monaco editor integration\n- mobile-shell.ts: Main container coordinating mobile UI layout\n- mobile-title.ts: Header with connection status and view switching\n\nKey changes:\n- Replaced LitElement inheritance with SketchTailwindElement\n- Converted all CSS-in-JS styles to Tailwind utility classes\n- Removed static styles blocks and shadow DOM styling\n- Added custom animations via document.head for non-Tailwind effects\n- Preserved all existing functionality and component interactions\n\nTechnical improvements:\n- Consistent iOS safe area support with env() CSS functions\n- Proper flexbox layouts for mobile responsive design\n- Maintained accessibility with proper ARIA labels and focus states\n- Enhanced hover and active states using Tailwind modifiers\n- Optimized touch interactions with -webkit-overflow-scrolling\n\nThe mobile components now follow the established SketchTailwindElement\npattern while maintaining full feature parity with the original\nshadow DOM implementations.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s21f840091392b02ek\n" }, { "commit": "9ee3042aa7826d9ae662c86743a5f87468972d27", "tree": "a669fecd906f39e504508cf2c8c31c3a26726322", "parents": [ "3eaa43377ddaec484ea3564f178078a84dc6e02d" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sat Jul 19 12:57:45 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sat Jul 19 12:57:45 2025 -0700" }, "message": "default to light mode instead of system, for now\n" }, { "commit": "3eaa43377ddaec484ea3564f178078a84dc6e02d", "tree": "79c226b85a10f50cf554b87899828e318d63f46f", "parents": [ "d082fdceb0378ddbceb81f51ab77ac1af904d57c" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sat Jul 19 02:19:06 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sat Jul 19 18:44:56 2025 +0000" }, "message": "webui: implement comprehensive dark mode support\n\nAdd complete dark mode implementation across all web UI components with\ntheme initialization and consistent styling:\n\nCore infrastructure:\n- Update DARK_MODE.md documentation with current implementation details\n- Add theme initialization to sketch-app-shell-base component\n- Implement ThemeService integration with existing theme toggle system\n\nComponent updates with dark mode variants:\n- sketch-app-shell: Main container backgrounds and text colors\n- sketch-app-shell-base: Top banner, todo panel container with gradient backgrounds\n- sketch-chat-input: Input fields, buttons, overlay messages, and drop zones\n- sketch-container-status: Info panels, SSH connection displays, and expandable details\n- sketch-call-status: Status indicators, banners, and activity states\n- sketch-view-mode-select: Tab container, button states, and active tab styling\n- sketch-timeline-message: Message bubbles, markdown content, code blocks, and commit info\n- sketch-push-button: Overlay popup, form controls, and result containers\n- sketch-todo-panel: Todo items, headers, comment modal, and form elements\n- sketch-diff-range-picker: Dropdown interface, commit display, and git reference badges\n\nCSS and styling improvements:\n- Comprehensive markdown content styling for dark theme\n- Code block backgrounds and syntax highlighting adjustments\n- Mermaid diagram container styling for dark mode\n- Auto-generated link styling with proper contrast\n- Git reference badge colors (tags: amber, branches: green, sketch-base: blue)\n\nInteractive element enhancements:\n- Consistent hover states across light and dark themes\n- Proper focus indicators with accessible contrast ratios\n- Loading spinners and progress indicators adapted for dark backgrounds\n- Error and success message styling with semantic color preservation\n\nKey implementation details:\n- Consistent color mappings: whiteβ†’gray-900, gray-100β†’gray-800, gray-200β†’gray-700\n- Preserved brand colors (blue-500, red-600, green-600) that work in both themes\n- Maintained semantic color coding for success/error/warning states\n- Ensured accessibility with proper contrast ratios throughout\n- Theme system integration enables seamless switching without page reload\n\nThe web UI now provides a complete, professional dark mode experience\nwith excellent usability and visual consistency while preserving all\nexisting functionality and accessibility standards.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8219557c3ecba46dk\n" }, { "commit": "ae3724ef039833e1227714412e2c5289ad8875c8", "tree": "7242aaea3ef677a86ace50e7c805935f27890d94", "parents": [ "19a32eaa120562170595e42584f3fc4b40828b97" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 16:52:37 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 20:29:23 2025 +0000" }, "message": "webui: add dark mode implementation plan\n\nAlso implements phase 1 of the plan, which just lays the foundation\nfor implementing the user-visible changes. This does not include\nany dark-mode theme settings for the rest of the web UI, and\nwhile it does inlcude a \"sketch-theme-toggle\" element, this is\nonly included in the demo:runner vite server for interactive testing.\nIt\u0027s not included in the app shell base yet.\n\n-SM\n\n---\n\nDocuments comprehensive strategy for implementing dark mode in Sketch\u0027s\nweb UI using Tailwind CSS class-based approach.\n\nThe plan covers:\n- Foundation setup (Tailwind config, theme service, toggle component)\n- Systematic component updates with dark mode variants\n- Accessibility considerations and testing checklist\n- 4-week implementation timeline\n\nKey technical decisions:\n- Uses SketchTailwindElement base class following existing patterns\n- Singleton theme service with event system for component coordination\n- Respects system preferences while allowing user override\n- Persistent theme storage in localStorage\n\nThis provides a roadmap for adding dark mode support while maintaining\nconsistency with Sketch\u0027s existing web component architecture.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s6b69ad95a4394f98k\n" }, { "commit": "2be768e4f4d28dc56c7e0b10df342c607ef04833", "tree": "0ee35d42338c188b6d88fb60f00700a3c07e3264", "parents": [ "5681b7c857682006489780fca02384e1db26ceab" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 18 16:41:39 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 18:59:33 2025 +0000" }, "message": "webui: convert SketchDiffRangePicker to SketchTailwindElement\n\nChanges the sketch-diff-range-picker component to extend SketchTailwindElement\ninstead of LitElement. This removes the shadow DOM and allows Tailwind CSS\nclasses to work properly with the component.\n\nChanges:\n- Updated imports to include SketchTailwindElement\n- Removed LitElement import\n- Changed class inheritance from LitElement to SketchTailwindElement\n- Added test file to verify the conversion works correctly\n- Added demo HTML file for manual testing\n\nThe component functionality remains unchanged - this is purely a refactoring\nto use the project\u0027s standard base class for web components.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: se92a5fe960a4312dk\n" }, { "commit": "5681b7c857682006489780fca02384e1db26ceab", "tree": "63d70c33be17d0eea5efd78fa5012e5985090dd0", "parents": [ "08b073b79f4372e2f61c03ba17e5d86a5cf7a769" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 18 10:42:47 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jul 18 10:43:17 2025 -0700" }, "message": "Fix terminal CSS.\n" }, { "commit": "93b53d8e55e1b0c5c85d387ab46a6ee3f96a8e8c", "tree": "a5a38ef8d33dcbd38efd39c527354e676be7edd3", "parents": [ "9d9c812b496268f033d48df5e8f91c421730a2e2" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Fri Jul 18 00:11:07 2025 +0000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Fri Jul 18 13:50:16 2025 +1000" }, "message": "skaband: fix terminal under /newsessions\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sf39677ac8518e6dfk\n" }, { "commit": "9d9c812b496268f033d48df5e8f91c421730a2e2", "tree": "0bfede0eb60d890faf499ec4d839c3b4eac95d08", "parents": [ "725cfe088d37e1d27057b13598b0955cae02bffd" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 03:37:23 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 03:37:23 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "4594388b5a5297295b39cccc924dd4727e97035a", "tree": "deffbe3d30f84a37da0e58a7f6a286c3a9b6f93b", "parents": [ "9db8c2a4713e4ddc0c2e5e9f75767d770412ee80" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 18 02:13:31 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 17 20:33:34 2025 -0700" }, "message": "webui: add tooltips to πŸ₯· and 🐒 in bash tool cards\n\nFor when people are curious.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sc49577a78023541dk\n" }, { "commit": "9db8c2a4713e4ddc0c2e5e9f75767d770412ee80", "tree": "7699e20a768625dfae8335aa7dc4acdc82bd81a8", "parents": [ "596aa33bf96e9003ad645403fb1f7dd6fafa2b0f" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 02:15:29 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jul 18 02:15:29 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "596aa33bf96e9003ad645403fb1f7dd6fafa2b0f", "tree": "a4ef8a3fc77ca20e5ef0d304143f9f35d0ae8d39", "parents": [ "7de3bdd597e962ab2533f67018545c6fd2a33d16" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 17 19:00:39 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 17 19:13:58 2025 -0700" }, "message": "webui: silence build noise\n\n**again**\n\nlast one was f8954383da9fbe04125c35548afbc6bf9d5272eb,\nwhich was earlier today.\n\ndear sketch: successful builds are SILENT. kthxbye.\n" }, { "commit": "7de3bdd597e962ab2533f67018545c6fd2a33d16", "tree": "5c3116b9e2fc88f47f820014a1ce071d104b1773", "parents": [ "ebe74cdcc911d70afd804b6218bab25744513578" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 18 01:51:53 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 17 19:13:58 2025 -0700" }, "message": "webui: hide GitHub link in push dialog for non-GitHub remotes\n\nOnly show \"Open on GitHub\" button when pushing to GitHub remotes.\nThe dialog now checks the remote\u0027s is_github property before generating\na branch URL, returning empty string for non-GitHub remotes.\n\nChanges:\n- Modified _computeBranchURL() to check selectedRemote.is_github\n- Added test coverage for GitHub, GitLab, and self-hosted remotes\n- Updated demo mock to include non-GitHub remotes for testing\n\nThis prevents the confusing \"Open on GitHub\" button from appearing\nwhen pushing to GitLab, self-hosted Git servers, or other non-GitHub\nremote repositories.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sb441bfcebf8fea07k\n" }, { "commit": "3a41f1501568811b7a6a83e6d674f19188595848", "tree": "2545072c225a258ac2b2d0fe35eb862c237ecc76", "parents": [ "75b45f5665fad8d386d36f88bf7dc37808a8dcac" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 18 01:51:54 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 18 01:51:54 2025 +0000" }, "message": "webui: update Push button tooltip to indicate it opens a dialog\n\nChange tooltip from \u0027Push current HEAD to remote\u0027 to \u0027Open push dialog\nto push current HEAD to remote\u0027 to better communicate that clicking\nthe button opens a dialog interface rather than immediately pushing.\n\nUser changes from diff view.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sa22bbc9efe9d6ad2k\n" }, { "commit": "7208bb4b7fbdfec4a86843020eb8d384ac9889e7", "tree": "b5b577fdde5d46c00d624595ddba49209701e7a4", "parents": [ "254c49faa2537d79e1394c4109f48a6e0990967c" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Fri Jul 18 00:44:11 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Thu Jul 17 18:04:13 2025 -0700" }, "message": "webui: add tooltips for upper-right corner buttons\n\nFixes https://github.com/boldsoftware/sketch/issues/160\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s7884db8812a87864k\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": "0269340cb3925dd8c5f5ed8ad7ebcd743f66fe6c", "tree": "e57fb0d4f57ed29e3defc33e00350a941017ce33", "parents": [ "2cc7563b105e381982b0732279609fde49c0ce2d" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 17 17:10:10 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 17 18:08:33 2025 +0000" }, "message": "sketch: fix Timeline messages width overflow issue\n\nFix issue where Timeline messages don\u0027t fit within their designated width,\ncausing text to overflow beyond message bubble boundaries and overlap with\nother UI elements.\n\nRoot cause was that max-width constraints were not being applied properly:\n- max-w-[calc(100%-160px)] Tailwind class wasn\u0027t being computed correctly\n- markdown-content div was missing proper overflow-wrap and word-break CSS\n- message content div was missing max-width constraints\n\nChanges:\n- Add explicit style attributes for max-width constraints on bubble containers\n- Add proper word-break and overflow-wrap CSS properties to markdown content\n- Update global CSS to ensure consistent word breaking behavior\n- Add overflow: hidden to prevent content from overflowing containers\n- Create test page to verify fix works in normal and compact padding modes\n\nThe fix ensures messages wrap properly within their containers in both normal\nmode (with 160px margin) and compact padding mode (full width) while\nbreaking long URLs and unbreakable strings appropriately.\n\nFixes https://github.com/boldsoftware/sketch/issues/201\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s7eaa3905ab8c4595k\n" }, { "commit": "2cc7563b105e381982b0732279609fde49c0ce2d", "tree": "b648353d0e8027abcf9d8c0188f2ee621e9b7530", "parents": [ "c5c3bea08ccfcd2e785bb4b13e9d8618e0f46364" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 17 17:10:17 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 17 17:42:00 2025 +0000" }, "message": "webui: fix copy button in markdown code blocks\n\nThe copy button in markdown code blocks was not working because the click\nevent listener was attached to the .markdown-content div, but the copy\nbutton was rendered outside of that scope in the .code-block-header.\n\nMove the event listener to the message content container level so it can\nproperly handle clicks on copy buttons through event delegation.\n\nFixes https://github.com/boldsoftware/sketch/issues/203\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s888913f32dc41c3fk\n" }, { "commit": "c5c3bea08ccfcd2e785bb4b13e9d8618e0f46364", "tree": "f365ebcbb0f7802f3c7c60469a024309e5673591", "parents": [ "c314f0c95a5b10cb10904181a181c4ce0d8d1a4b" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 17 10:11:35 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 17 17:24:05 2025 +0000" }, "message": "update web components DEAR_LLM.md\n" }, { "commit": "09b86f40dd7b2fd26b92dc49add2d8555785749c", "tree": "67302c7233854cc9e1318a6a4bb27d6a193fe146", "parents": [ "fb56a5cb097451f5702372da7af8356e35806a73" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 16 16:23:03 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 16 16:23:03 2025 -0700" }, "message": "webui: optimize npm ci caching for faster builds\n\nAdd caching for npm ci in webui build process by tracking\npackage-lock.json changes separately from other source files.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sce58f3444bbcc029k\n" }, { "commit": "390f847edb6fe450fc547938c2e806db693d9e61", "tree": "25ee5ba6832307d13801139fe0dc2329c0085421", "parents": [ "a8561f70c1701376e88a5b63d78e7d09a7ae3ed8" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 16 20:09:45 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 16 20:09:45 2025 +0000" }, "message": "all: fix formatting\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": "d9acaa785a8cc8b3cc14d6efcdd7ce45ffe3737f", "tree": "c3c615b10db1e8f9bd9c3d88c13f81f61c1a83b6", "parents": [ "a50a3bf9e4b996498289df50744bdce09a59cc13" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 14:51:55 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Mon Jul 14 15:04:32 2025 -0700" }, "message": "sketch: optimize code copy button performance\n\nThere was a setTimeout(..., 100) for no good reason.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s1d721e710776c3eck\n" }, { "commit": "333aa67a4c1639f49c7ba03edab4ace3c6439e2b", "tree": "9bb97e108a92eaf51e4ad7f12b07042ec3b96e8b", "parents": [ "dc27c395443e6d08de8cba3c9c4a85242f11ade6" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jul 13 19:49:21 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Mon Jul 14 02:44:44 2025 +0000" }, "message": "sketch: remove shadowDOM dependency from tool card components\n\nReplace shadowDOM-based slot system with property-based composition in all\nsketch-tool-card-[TOOL_NAME] components to support shadowDOM-free architecture.\n\nProblem Analysis:\n- sketch-tool-card component relied on HTML5 template slots which require shadowDOM\n- 13 tool card components used sketch-tool-card as composition base via slots\n- shadowDOM dependency blocked broader effort to reduce shadowDOM usage\n- Need to preserve all existing functionality while removing slot dependency\n\nSolution Implementation:\n- Created sketch-tool-card-base component with property-based content injection\n- Replaced slot system with summaryContent, inputContent, resultContent properties\n- Maintained all existing styling, behavior, and expand/collapse functionality\n- Migrated all 13 existing tool card components to use new base component\n\nComponents Migrated:\n- sketch-tool-card-about-sketch\n- sketch-tool-card-browser-clear-console-logs\n- sketch-tool-card-browser-click\n- sketch-tool-card-browser-eval\n- sketch-tool-card-browser-get-text\n- sketch-tool-card-browser-navigate\n- sketch-tool-card-browser-recent-console-logs\n- sketch-tool-card-browser-resize\n- sketch-tool-card-browser-scroll-into-view\n- sketch-tool-card-browser-type\n- sketch-tool-card-browser-wait-for\n- sketch-tool-card-read-image\n- sketch-tool-card-take-screenshot\n\nMigration Pattern:\n- Changed from: \u003cslot name\u003d\"summary\"\u003econtent\u003c/slot\u003e\n- Changed to: .summaryContent\u003dhtml content\n- Preserved all component-specific styling and logic\n- Maintained existing API surface for parent components\n\nArchitecture Benefits:\n- Removes shadowDOM requirement from 13+ components\n- Enables future shadowDOM-free component development\n- Maintains backward compatibility during migration\n- Preserves all existing tool card functionality\n\nFiles Added:\n- sketch/webui/src/web-components/sketch-tool-card-base.ts (new shadowDOM-free base)\n\nFiles Modified:\n- All 13 sketch-tool-card-[TOOL_NAME].ts components migrated to use new base\n\nVerification:\n- TypeScript compilation passes without errors\n- Demo pages render correctly with consistent styling\n- Expand/collapse behavior preserved across all tool types\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sa3288c1d986356e5k\n" }, { "commit": "dc27c395443e6d08de8cba3c9c4a85242f11ade6", "tree": "0ed1fef06da72f23ff8d59b28ce3dee535b1ae30", "parents": [ "b80db9c74081f89810ee13789345add0d6210297" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 11 18:36:14 2025 -0700" }, "committer": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jul 11 18:36:26 2025 -0700" }, "message": "fix clipped right edge of messages\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": "b7ec9c837a549abd9bbbc1230e6d96090c98af52", "tree": "c4dc6e6b0d159b0ae8a21456c591cb2e12dc3e63", "parents": [ "973413b1c24f1d2ab5f56729ae236892c4221fdf" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 09 10:16:39 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 09 17:17:12 2025 +0000" }, "message": "add link to gh user page under chat msg\n" }, { "commit": "e7ca73df4b61cff1847c4f63feac89c580e998c7", "tree": "2e76b9f8b9751578c7daeeaa8d274e0f9d64d0e6", "parents": [ "5ae245bd47becaf2aaa862b5c7e9bcec71e17831" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 16:37:09 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 19:49:34 2025 -0700" }, "message": "all: cull dead code\n\nMostly from moving builds to use make.\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": "39ee05a289740544030b5a2f83cd673839efd08f", "tree": "ef2e09c6519775586475648d9279468cf662a9f4", "parents": [ "454f8864b1830ff13f19a49dd53d91b5a201baf7" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 11:20:01 2025 -0700" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 12:02:47 2025 -0700" }, "message": "webui: bump esbuild version" }, { "commit": "289525bcecca82247e0908c5ff1f213213d9c9a4", "tree": "ffdc00894b39af359cba7f481c05e9a5b7b405f7", "parents": [ "828161b106396bf87a4c481895781f1d1d732e0a" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 08 04:03:02 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Mon Jul 07 21:13:27 2025 -0700" }, "message": "sketch/termui: handle PortMessageType messages more\n\n- Added case for loop.PortMessageType in termui message handling switch\n- Port messages now display with πŸ”Œ emoji and clean formatting\n- Follows same pattern as other message types (ErrorMessageType, BudgetMessageType, etc.)\n\n- Added loop.PortMessageType to AddMultipleUnion call in go2ts.go\n- Regenerated TypeScript types to include \u0027port\u0027 in CodingAgentMessageType union\n- Webui components now have proper type safety for port messages\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s7e1f4b705a66a9d5k\n" }, { "commit": "c52a250e07c2d84173c90dc076d45f14a26bdce3", "tree": "236d1d92297fbac41b76921d8694c59e22e37060", "parents": [ "255dc4320aed97625a17d9d398c5d076c35e9700" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sun Jul 06 20:27:59 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Sun Jul 06 20:27:59 2025 -0700" }, "message": "fix infobox z-index\n" }, { "commit": "255dc4320aed97625a17d9d398c5d076c35e9700", "tree": "81fb28bdd4b3aad8093349bac6cd3458fadc247c", "parents": [ "e5b2fc0061e542fe2350c1675e7d18172b418882" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sun Jul 06 21:58:00 2025 +0000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sun Jul 06 15:13:51 2025 -0700" }, "message": "webui: fix diff scrolling (broken in 265839b0 / 5450584)\n\nSketchTailwindElement disabled shadow DOM, so the :host CSS selector\nno longer worked since there\u0027s no shadow DOM. Also some heights\nstopped being calculated correctly.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s2155f0d365ba51cak\n" }, { "commit": "e5b2fc0061e542fe2350c1675e7d18172b418882", "tree": "db4f467c9237395cfb8ef2df2648288a05036f80", "parents": [ "e48f2bb05d937c1bcc51bca81a010173d905c0bb" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sun Jul 06 16:33:46 2025 +0000" }, "committer": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Sun Jul 06 11:20:38 2025 -0700" }, "message": "sketch/webui: skip welcome text in compact mode\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8fafc4bbd5d95705k\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": "6dc90c03abff887c09ca0418d4d493d16cf1b0c8", "tree": "d6d66f1950c0b3689c9f55972e0c7e1a829d1f7f", "parents": [ "5f26a3445601f6ab0299d9be20ea99b67eae4d51" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 20:12:49 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jul 03 21:14:55 2025 -0700" }, "message": "sketch/webui: add port display with external link functionality to UI header\n\nAdd port monitoring display next to Last Commit area in container status\nheader, showing open ports as clickable links with external link emoji\nand overflow handling for additional ports beyond first two displayed.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s205c16eb68a4b157k\n" }, { "commit": "f6e1dfe8ea4069d9180abf71c64b9d5ff4b172e0", "tree": "eb57c7c5e3c9e1fee2fd3c781597c2f50730c25c", "parents": [ "e1c8b7bedc96d182916d5e1b9a07913945ffbafb" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Jul 03 14:59:40 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 22:00:37 2025 +0000" }, "message": "webui rm shadowRoot references\n" }, { "commit": "c37e066371997d221af074b0bc1da392c82c6a2b", "tree": "b352de3edf0227f002aaa0ab674aea4feaed76fa", "parents": [ "df23403824257c1201f64f865eaf169ceef45c3d" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Jul 03 08:46:21 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 16:25:09 2025 +0000" }, "message": "sketch-container-status:latest context window size\n" }, { "commit": "df23403824257c1201f64f865eaf169ceef45c3d", "tree": "43738cf871ad223fd872ef247b12c90be293f36c", "parents": [ "32969cde983b415a2ef95ccf8a8f798acfa70870" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Jul 02 20:45:29 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Jul 02 20:45:48 2025 -0700" }, "message": "fix https://github.com/boldsoftware/sketch/issues/186\n" }, { "commit": "32969cde983b415a2ef95ccf8a8f798acfa70870", "tree": "9ac42989bb2defa11f4fc639be64b78bc4af6b83", "parents": [ "4844be2a83915a9041c1c3c2a8882cee8c5f2be4" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 03:25:25 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 03:25:25 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "4844be2a83915a9041c1c3c2a8882cee8c5f2be4", "tree": "3cee7b1035b037f34140856ec69346764b2f3968", "parents": [ "882b1d12f097997164523af4947954781ccfce09" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 03 00:32:55 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Jul 02 20:23:28 2025 -0700" }, "message": "webui: convert sketch-terminal to SketchTailwindElement inheritance\n\nConvert sketch-terminal component from LitElement to SketchTailwindElement inheritance\nto disable shadow DOM and enable consistent Tailwind utility class usage across the\nwebui component library.\n\nInheritance Change:\n- Replace LitElement with SketchTailwindElement as base class\n- Remove CSS-in-JS static styles in favor of Tailwind utility classes\n- Switch from shadow DOM to light DOM rendering for better Tailwind integration\n\nCSS to Tailwind Conversion:\n- Terminal view container: w-full bg-gray-100 rounded-lg overflow-hidden mb-5 shadow-md p-4\n- Terminal container: w-full h-full overflow-hidden\n- Maintain 70vh height using inline style for specific viewport requirement\n- Remove static CSS definitions while preserving visual appearance\n\nDOM Query Updates:\n- Change renderRoot.querySelector to this.querySelector for light DOM compatibility\n- Update CSS loading from shadow DOM injection to global document head injection\n- Rename loadXtermlCSS to loadXtermCSS for consistency\n\nTechnical Implementation:\n- Preserve all terminal functionality: xterm.js integration, SSE connections, resize handling\n- Maintain event listeners for view mode changes and window resize events\n- Keep terminal input queue processing and WebSocket communication intact\n- Update xterm CSS loading to work with light DOM architecture\n\nThe conversion enables the terminal component to benefit from global Tailwind\nstyles while maintaining all existing terminal emulation functionality and\nserver communication features.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s853becde7eccb88ck\n" }, { "commit": "983b58aceb758e480fc3fede42d21d06ee1de521", "tree": "4d219db8d5a580f4df91bac61f90617376a8f6df", "parents": [ "2ca1f10615ed3dede0c5f0bfe9a67c7d9e179794" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 02 19:42:08 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Wed Jul 02 19:42:08 2025 -0700" }, "message": "Re-work Sketch\u0027s Docker setup.\n\nWe were being fancy and creating Dockerfiles for folks. This sometimes\nworked, but quite often didn\u0027t.\n\nInstead, we you have -base-image and -force-rebuild-container, and the\n\"cache key\" for images is just the base image and the working dir.\n\nThe layer cake is\n\n (base image)\n (customization) [optional]\n (repo) [/app]\n" }, { "commit": "9f5cb2e25bb67ca67e5b8f5133452069b2ea709d", "tree": "ee1f1b4e553a49303a5176ae6ab1fdec61213d6d", "parents": [ "5450584a3e83904406aef4f1f8cfaa6b7de66268" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 00:25:35 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jul 03 00:25:35 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "5450584a3e83904406aef4f1f8cfaa6b7de66268", "tree": "9eb466e6188a6b3279eadd1a924cad9069ee803c", "parents": [ "9556fcf116434d39a351e5bebe4e7f772ea440d8" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jul 03 00:18:44 2025 +0000" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Wed Jul 02 17:24:38 2025 -0700" }, "message": "webui: convert sketch diff components to inherit from SketchTailwindElement\n\nConvert SketchDiffEmptyView and SketchDiff2View components from LitElement\nto SketchTailwindElement inheritance pattern to enable proper Tailwind CSS\nintegration across the diff view system.\n\nComponents Converted:\n- sketch-diff-empty-view: Simple empty state component with help text\n- sketch-diff2-view: Complex Monaco editor-based diff viewer with file management\n\nSketchDiffEmptyView Changes:\n- Updated imports: removed css, LitElement; added SketchTailwindElement\n- Changed class inheritance from LitElement to SketchTailwindElement\n- Removed static styles CSS block entirely\n- Converted custom CSS to Tailwind classes:\n - Container: flex flex-col items-center justify-center h-full w-full box-border\n - Content box: m-8 mx-auto max-w-4xl w-11/12 p-8 border-2 border-gray-300 rounded-lg shadow-sm bg-white text-center\n - Typography: text-2xl font-semibold mb-6 text-center text-gray-800\n - Body text: text-gray-600 leading-relaxed text-base text-left mb-4\n - Strong emphasis: font-semibold text-gray-800\n\nSketchDiff2View Changes:\n- Updated imports: removed css, LitElement; added SketchTailwindElement\n- Changed class inheritance from LitElement to SketchTailwindElement\n- Removed extensive static styles CSS block (280+ lines)\n- Preserved Monaco editor integration, scrollbar hiding, and dynamic height handling\n- Maintained all complex diff view functionality and state management\n\nCSS-to-Tailwind Mapping for SketchDiff2View:\n- Host container: flex h-full flex-1 flex-col min-h-0 overflow-hidden relative\n- Controls section: px-4 py-2 border-b border-gray-300 bg-gray-100 flex-shrink-0\n- File selector: min-w-[200px] px-3 py-2 border border-gray-400 rounded bg-white\n- Diff container: flex-1 overflow-auto flex flex-col min-h-0 relative h-full\n- File sections: flex flex-col border-b-4 border-gray-300 mb-0 last:border-b-0\n- File headers: bg-gray-100 border-b border-gray-300 px-4 py-2 sticky top-0 z-10\n- Monaco editors: flex flex-col w-full min-h-[200px] flex-1\n\nFile Status Badge System:\n- Added: bg-green-100 text-green-800 (green status badge)\n- Modified: bg-yellow-100 text-yellow-800 (yellow status badge)\n- Deleted: bg-red-100 text-red-800 (red status badge)\n- Renamed: bg-cyan-100 text-cyan-800 (cyan status badge)\n- Copied: bg-indigo-100 text-indigo-800 (indigo status badge)\n- Status badges: inline-block px-1.5 py-0.5 rounded text-xs font-bold mr-2\n\nInteractive Elements:\n- Expand/collapse buttons: bg-transparent border border-gray-300 rounded px-2 py-1\n- Button hover states: hover:bg-gray-200 with transition-colors duration-200\n- File paths: font-mono font-normal text-gray-600 for monospace display\n- Loading states: flex items-center justify-center h-full for centered display\n- Error states: text-red-600 p-4 for consistent error styling\n\nMethods Updated in SketchDiff2View:\n- render(): Main component layout with Tailwind flexbox containers\n- renderFileSelector(): File dropdown with focus states and disabled styling\n- renderDiffContent(): Content routing with loading/error/empty state styling\n- renderFileDiff(): Individual file diff sections with proper Monaco integration\n- renderFileHeader(): File header with status badges and expand/collapse controls\n- renderSingleFileExpandButton(): Header expand button with consistent styling\n- renderSingleFileView(): Full-screen single file view with proper layout\n- getFileStatusTailwindClasses(): New method mapping file status to Tailwind classes\n\nPreserved Complex Functionality:\n- Monaco editor height change handling and dynamic container sizing\n- Comment forwarding from Monaco editor to chat input system\n- File save operations with proper success/error notification\n- File expansion state management for show/hide unchanged regions\n- Range picker integration with commit selection and diff reloading\n- Single/multi file view mode switching with proper layout adaptation\n- Custom Monaco scrollbar hiding through global style injection\n- All git service integration for diff data loading and file content retrieval\n- Sticky file headers with proper z-index stacking for navigation\n- Dynamic Monaco editor height adjustment with container synchronization\n- Responsive file selector and range picker layout with flexible spacing\n\nBoth components now integrate properly with the project\u0027s Tailwind CSS\nstyling system by disabling shadow DOM while maintaining complete visual\nand functional parity with their original LitElement implementations.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s92d5cf7de96afe2ck\n" }, { "commit": "9556fcf116434d39a351e5bebe4e7f772ea440d8", "tree": "ccd40cd404008c1c62547c1c6dcc6671cbbd380d", "parents": [ "6534c7a6c4a10f8a9660835e2f0d8c27101bcd34" ], "author": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 02 22:48:51 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 02 22:48:51 2025 +0000" }, "message": "all: fix formatting\n" }, { "commit": "b3aff885a06ca0bbdfb2f61c94387789355a14c1", "tree": "cfab200942eb9718f5e2a8269a9b40180bcda81e", "parents": [ "cdb08a546baf90b43f266b77dfa98bb35d978e5a" ], "author": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Tue Jul 01 02:17:27 2025 +0000" }, "committer": { "name": "Josh Bleecher Snyder", "email": "josharian@gmail.com", "time": "Wed Jul 02 15:48:16 2025 -0700" }, "message": "webui: display file copy status (vs modify/rename) in diff view\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8e76980f4dd17de3k\n" }, { "commit": "cdb08a546baf90b43f266b77dfa98bb35d978e5a", "tree": "3f43115d4aadca2b57d9a2e7662aef8efb70aace", "parents": [ "0470a8af7259913898bd54fe55135a10d06f306d" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 02 20:28:29 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 02 20:35:22 2025 +0000" }, "message": "webui: convert sketch-todo-panel to SketchTailwindElement with comprehensive test suite\n\nConvert sketch-todo-panel component from LitElement with CSS-in-JS to SketchTailwindElement\ninheritance using Tailwind utility classes, and add complete testing infrastructure with\nTypeScript demo module and comprehensive test coverage.\n\nComponent Conversion:\n- Replace LitElement with SketchTailwindElement inheritance to disable shadow DOM\n- Remove 200+ lines of CSS-in-JS styles in favor of Tailwind utility classes\n- Convert all styling to Tailwind class compositions while maintaining visual parity\n- Add inline fadeIn animation using \u003cstyle\u003e tag following established patterns\n- Preserve all existing functionality: todo rendering, comment system, loading states\n\nCSS-to-Tailwind Mapping:\n- Main container: flex flex-col h-full bg-transparent overflow-hidden\n- Header section: py-2 px-3 border-b border-gray-300 bg-gray-100 font-semibold text-xs\n- Content area: flex-1 overflow-y-auto p-2 pb-5 text-xs leading-relaxed min-h-0\n- Todo items: flex items-start p-2 mb-1.5 rounded bg-white border border-gray-300 gap-2\n- Loading state: animate-spin with proper Tailwind spinner classes\n- Comment modal: fixed inset-0 bg-black bg-opacity-30 z-[10000] with centered content\n- Status icons: βœ… completed, πŸ¦‰ in-progress, βšͺ queued with proper sizing\n- Interactive buttons: hover states and transitions using Tailwind utility classes\n\nTest Infrastructure:\n- Create sketch-todo-panel.test.ts with 14 comprehensive test cases\n- Test initialization, visibility, state management (loading/error/empty states)\n- Test todo rendering: status icons, task descriptions, progress counts\n- Test comment system: button visibility, modal interactions, event dispatch\n- Test error handling: invalid JSON parsing, empty content scenarios\n- Test Tailwind integration: proper class usage, shadow DOM disabled\n- Test scrollable interface: large todo lists render and scroll correctly\n- Use @sand4rt/experimental-ct-web framework following established patterns\n- Include helper functions for mock TodoItem creation and test utilities\n\nDemo Module Integration:\n- Create sketch-todo-panel.demo.ts following established TypeScript demo pattern\n- Add comprehensive demo scenarios: basic usage, loading/error/empty states\n- Include large scrollable list demonstration with multiple todo items\n- Add interactive comment functionality testing with event logging\n- Add sketch-todo-panel to demo-runner.ts knownComponents registry\n- Demonstrate all component states and user interactions comprehensively\n\nTypeScript Compatibility:\n- Fix property access for private @state() properties using component.evaluate()\n- Use type assertions for addEventListener/removeEventListener on SketchTailwindElement\n- Address interface compatibility issues with proper TypeScript patterns\n- Remove unused imports and helper functions to maintain ESLint compliance\n- Follow established patterns from other SketchTailwindElement components\n\nFiles Modified:\n- sketch/webui/src/web-components/sketch-todo-panel.ts: TailwindElement conversion with complete CSS removal\n- sketch/webui/src/web-components/demo/demo-framework/demo-runner.ts: Added component to registry\n\nFiles Added:\n- sketch/webui/src/web-components/sketch-todo-panel.test.ts: Comprehensive test suite with 14 test cases\n- sketch/webui/src/web-components/demo/sketch-todo-panel.demo.ts: Interactive TypeScript demo module\n\nThe conversion maintains complete functional parity while enabling consistent\nTailwind-based styling, comprehensive test coverage, and improved development\nexperience through integrated demo infrastructure.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: seada6841c7c375e5k\n" }, { "commit": "0470a8af7259913898bd54fe55135a10d06f306d", "tree": "04310d8cda7cb968654db37f6ab1a1ff16fc8c01", "parents": [ "a9710d76e938c29fbfaa30e5d63129bbc43b18c8" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Wed Jul 02 00:57:41 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Wed Jul 02 17:05:19 2025 +0000" }, "message": "webui: convert sketch-tool-calls to inherit from SketchTailwindElement\n\n- Replace LitElement with SketchTailwindElement inheritance\n- Remove custom CSS styles in favor of Tailwind classes\n- Update template to use Tailwind utility classes:\n - Container: mt-2 pt-1 max-w-full w-full box-border\n - Card: flex flex-col bg-white/60 rounded-md mb-1.5 overflow-hidden cursor-pointer border-l-2 border-black/10 shadow-sm max-w-full break-words\n- Add basic unit tests to verify component functionality\n- Maintain existing component API and behavior\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s9bc7a30829559ed9k\n" }, { "commit": "fd67b0185a488aeac149e0bcf1a476d628134555", "tree": "f08f486b3a1baedcec77ba2eded5fd0895608237", "parents": [ "0accea1d184b7ce20daadca509296e4ac7236273" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Tue Jul 01 14:58:23 2025 -0700" }, "committer": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Tue Jul 01 15:46:32 2025 -0700" }, "message": "sketch-timeline-message: fix commit message txt size\n" }, { "commit": "0accea1d184b7ce20daadca509296e4ac7236273", "tree": "d33cfc78c2acc3f6bcd286147152ff92c81471d8", "parents": [ "c514748e2bebc4c0d2955c3da15224f4a71aed19" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Tue Jul 01 09:59:18 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 01 17:00:15 2025 +0000" }, "message": "webui: improve diff view comment box width and height handling\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sfafa36307f1bf673k\n" }, { "commit": "c514748e2bebc4c0d2955c3da15224f4a71aed19", "tree": "b849f32531dd8fad42060eb38127d018bb7fb131", "parents": [ "26bc659f8f7e8864a6fb0a71dcbee7d3742d3763" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sun Jun 29 00:41:58 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 01 03:51:08 2025 +0000" }, "message": "webui: convert SketchTimelineMessage to TailwindElement with TypeScript demo module\n\nConvert SketchTimelineMessage component from Lit CSS-in-JS styles to TailwindElement\ninheritance with Tailwind utility classes, and replace standalone HTML test with\ncomprehensive TypeScript demo module integrated with the demo runner framework.\n\nProblems Solved:\n\nCSS Inconsistency and Shadow DOM Isolation:\n- SketchTimelineMessage used shadow DOM with extensive CSS-in-JS styles while other components use TailwindElement\n- Component styling was isolated from global design system and Tailwind utilities\n- Over 400 lines of CSS-in-JS code created maintenance overhead and styling inconsistencies\n- No access to global Tailwind utility classes within shadow DOM environment\n\nTest Infrastructure Brittleness:\n- Tests relied on CSS class selectors that were implementation details\n- Complex CSS class selectors made tests fragile to styling changes\n- No standardized approach for testing UI elements across component library\n- Test selectors tightly coupled to internal CSS implementation\n\nMissing Development Infrastructure:\n- timeline-message-test.html was standalone and not integrated with demo runner\n- Required manual HTML file maintenance and Tailwind CDN loading\n- Component not discoverable through standardized demo system\n- No interactive controls for testing different component states\n- No integration with demo framework utilities and mock data\n\nSolution Implementation:\n\nTailwindElement Conversion:\n- Changed inheritance from LitElement to SketchTailwindElement to disable shadow DOM\n- Replaced all CSS-in-JS styles with equivalent Tailwind utility classes\n- Converted over 400 lines of CSS to responsive Tailwind class compositions\n- Maintained complete visual and functional parity while using global design system\n\nCSS Class Mapping and Styling:\n- .message β†’ relative mb-1.5 flex flex-col w-full (base message layout)\n- .message-content β†’ relative px-2.5 py-1.5 rounded-xl shadow-sm max-w-full w-fit (message bubble)\n- .user .message-content β†’ bg-blue-500 text-white rounded-br-sm (user message styling)\n- .agent .message-content β†’ bg-gray-100 text-black rounded-bl-sm (agent message styling)\n- .message-actions β†’ absolute top-1 right-1 z-10 opacity-0 hover:opacity-100 (interaction buttons)\n- .commit-card β†’ bg-gray-100 rounded-lg overflow-hidden mb-1.5 shadow-sm (commit display)\n- .commit-hash β†’ text-blue-600 font-bold font-mono cursor-pointer bg-blue-600/10 (commit hash styling)\n- .commit-branch β†’ text-green-600 font-medium cursor-pointer font-mono bg-green-600/10 (branch styling)\n\nTest Infrastructure Modernization:\n- Replaced CSS class selectors with Tailwind class selectors for reliable element targeting\n- Updated all test selectors to use new Tailwind class patterns for better maintainability\n- Converted .message-text to .overflow-x-auto for text content targeting\n- Converted .message-info-panel to .mt-2.p-2 for info panel targeting\n- Converted .commit-notification to .bg-green-100 for commit notification targeting\n- Maintained all existing test functionality while improving test reliability\n\nTypeScript Demo Module Creation:\n- Created sketch-timeline-message.demo.ts following established demo module pattern\n- Comprehensive component demonstration with multiple message types and features\n- Interactive controls for testing component behavior and state changes\n- Proper integration with demo framework types, utilities, and mock data system\n- Added component to knownComponents registry in demo-runner.ts for discoverability\n\nDemo Content Organization and Features:\n- Message Types section: User, agent, and error message examples with proper styling\n- Interactive Features section: Live component with control buttons for state testing\n- Advanced Examples section: Tool calls, commits, and complex markdown demonstrations\n- Interactive controls: Toggle info panel, change message type, toggle compact padding, cycle content examples\n- Event listeners for commit diff interactions and proper error handling\n\nGlobal Styling Architecture:\n- Added global CSS using document.head.appendChild for complex styling not easily replicated with Tailwind\n- Implemented floating message animations and transitions for user feedback\n- Created comprehensive markdown content styling for both user and agent messages\n- Added print media query support using Tailwind print: variants for proper printing\n- Used Tailwind @apply directive in global styles for complex component styling\n\nImplementation Details:\n\nComponent Structure and Functionality:\n- Maintained all existing properties, methods, and component lifecycle hooks\n- Preserved scroll handling, markdown rendering, and interaction features completely\n- Added comprehensive Tailwind class composition for dynamic styling based on message type\n- Kept all existing functionality while changing only the styling implementation approach\n\nVisual Consistency and Behavior:\n- All colors, spacing, borders, and animations maintained complete visual parity\n- User message styling: blue background with white text and right alignment\n- Agent message styling: gray background with black text and left alignment\n- Commit cards: consistent styling with color-coded elements and proper interaction states\n- Info panels: conditional styling based on message type with proper contrast\n\nInteractive Features and Accessibility:\n- Copy buttons with proper hover states and transition animations\n- Info toggle functionality with slide-in panel animations and proper state management\n- Commit hash and branch click-to-copy functionality with user feedback\n- Floating success/error messages with proper positioning and accessibility\n- Keyboard navigation and screen reader compatibility maintained\n\nDemo Module Architecture:\n- Follows DemoModule interface with title, description, imports, and setup function\n- Includes Tailwind CSS styles for proper component rendering in demo environment\n- Cleanup function for demo-specific style removal to prevent memory leaks\n- Comprehensive error handling for malformed message data and edge cases\n- Uses existing demo fixture utilities and realistic mock state for consistency\n\nMock Data Integration and Examples:\n- Realistic message examples with proper timestamps, IDs, and conversation threading\n- Tool call examples with proper structure, formatting, and result display\n- Git commit examples with hash, branch, subject, and GitHub integration\n- Usage information examples with token counts, costs, and performance metrics\n- Error message examples with proper error state styling and user guidance\n\nFiles Modified:\n- sketch/webui/src/web-components/sketch-timeline-message.ts: TailwindElement inheritance and complete Tailwind class conversion\n- sketch/webui/src/web-components/sketch-timeline-message.test.ts: Updated test selectors to use new Tailwind class patterns\n\nFiles Added:\n- sketch/webui/src/web-components/demo/sketch-timeline-message.demo.ts: Comprehensive TypeScript demo module\n\nFiles Modified (Demo Integration):\n- sketch/webui/src/web-components/demo/demo-framework/demo-runner.ts: Added sketch-timeline-message to knownComponents\n\nFiles Removed:\n- sketch/webui/src/web-components/demo/timeline-message-test.html: Replaced with TypeScript demo module\n\nThe conversion maintains complete visual and functional parity while enabling\nconsistent styling across the component library, improving test reliability\nthrough semantic Tailwind class targeting, and providing superior development\ncapabilities through integrated TypeScript demo infrastructure.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s0efb435d3be1c182k\n" }, { "commit": "26bc659f8f7e8864a6fb0a71dcbee7d3742d3763", "tree": "5429b19391d26f2cf8a82253237a52d2d33543e6", "parents": [ "a14b0183208df257a43748b51666043db7e62138" ], "author": { "name": "philip.zeyliger", "email": "philip.zeyliger@gmail.com", "time": "Mon Jun 30 20:15:30 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Tue Jul 01 03:20:42 2025 +0000" }, "message": "webui: add ESLint and eslint-typescript\n\nI\u0027ve historically found this stuff worthwhile, so let\u0027s swallow the pill.\n\nFortunately, sketch was happy to oblige.\n\n- Install ESLint, @eslint/js, and typescript-eslint packages\n- Configure eslint.config.mjs with recommended TypeScript ESLint rules\n- Add browser globals support for DOM and web APIs\n- Integrate ESLint into npm test workflow via package.json scripts\n- Update Makefile to run lint checks as part of test suite\n- Fix 249+ linting issues including:\n * Remove unused imports and variables (with _ prefix convention)\n * Fix case declaration issues with eslint-disable blocks\n * Remove unnecessary escape characters\n * Address prefer-const violations\n * Handle unused function parameters appropriately\n- Configure ignore patterns to exclude dist/ and node_modules/\n- Set rules to allow explicit \u0027any\u0027 types temporarily\n- Convert warnings for ts-ignore, async promise executors, and unsafe optional chaining\n\nResults:\n- ESLint now runs successfully with 0 errors and only 6 warnings\n- TypeScript compilation continues to work correctly\n- Linting integrated into test workflow for continuous quality enforcement\n- Codebase follows consistent ESLint TypeScript recommended practices\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sd7b538be0a28d294k\n" }, { "commit": "e59a2e151da777fba13a0978c00c16a1ee3b6122", "tree": "015082d0323ad7bd97f6c7e2fc5cca48e9e10f1b", "parents": [ "4337aa70a436963ed0e1d563215dc9eb1f176085" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Sat Jun 28 01:38:19 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sat Jun 28 05:29:10 2025 +0000" }, "message": "webui: convert SketchTimeline to use TailwindElement and Tailwind CSS classes\n\nConvert SketchTimeline component from Lit CSS-in-JS styles to TailwindElement\ninheritance with Tailwind utility classes, replacing shadow DOM styling with\nglobal Tailwind CSS classes.\n\nProblems Solved:\n\nCSS Inconsistency:\n- SketchTimeline used shadow DOM with CSS-in-JS styles while other components use TailwindElement\n- Component styling was isolated from global design system\n- Difficult to maintain consistent visual appearance across components\n- No access to global Tailwind utility classes within shadow DOM\n\nTest Brittleness:\n- Tests relied on CSS class selectors that were implementation details\n- Complex CSS class selectors made tests fragile to styling changes\n- No standardized approach for testing UI elements across components\n\nMissing Demo Infrastructure:\n- SketchTimeline had no TypeScript demo module for component development\n- Component not included in demo runner system for iterative development\n- Only had static HTML demo without interactive controls\n\nSolution Implementation:\n\nTailwindElement Conversion:\n- Changed inheritance from LitElement to SketchTailwindElement to disable shadow DOM\n- Replaced all CSS-in-JS styles with equivalent Tailwind utility classes\n- Added custom CSS for complex animations (thinking dots, loading spinner) that can\u0027t be easily replicated with Tailwind\n- Maintained all existing visual styling and behavior while using Tailwind classes\n\nCSS Class Mapping:\n- .timeline-container β†’ w-full relative max-w-full mx-auto px-[15px] box-border overflow-x-hidden flex-1 min-h-[100px]\n- .welcome-box β†’ my-8 mx-auto max-w-[90%] w-[90%] p-8 border-2 border-gray-300 rounded-lg shadow-sm bg-white text-center\n- .thinking-indicator β†’ pl-[85px] mt-1.5 mb-4 flex\n- .loading-indicator β†’ flex items-center justify-center p-5 text-gray-600 text-sm gap-2.5 opacity-100\n- Added print: utility variants for print styling support\n\nTest Infrastructure Updates:\n- Replaced CSS class selectors with data-testid attributes for reliable element targeting\n- Updated all test selectors to use [data-testid\u003d\u0027element-name\u0027] pattern\n- Added test IDs to welcome-box, timeline-container, thinking-indicator, loading-indicator, thinking-bubble, thinking-dots, and thinking-dot elements\n- Maintained all existing test functionality while improving test reliability\n\nDemo Module Creation:\n- Created sketch-timeline.demo.ts with comprehensive interactive demo\n- Implemented basic timeline, loading states, thinking states, and interactive controls\n- Added mock message generation with various message types and tool calls\n- Included controls for adding messages, toggling thinking state, compact padding, and reset functionality\n- Added SketchTimeline to knownComponents list in demo-runner.ts\n\nCustom Styling Architecture:\n- Added addCustomStyles() method to inject necessary CSS that can\u0027t be replicated with Tailwind\n- Created thinking-pulse keyframe animation for thinking dots\n- Added loading-spin animation for spinner elements\n- Implemented compact-padding responsive styling\n- Used document.head.appendChild for global style injection with duplicate prevention\n\nImplementation Details:\n\nComponent Structure:\n- Maintained all existing properties, methods, and component lifecycle\n- Preserved scroll handling, viewport management, and loading operations\n- Added data-testid attributes without affecting visual presentation\n- Kept all existing functionality while changing only the styling approach\n\nStyling Consistency:\n- All colors, spacing, borders, and animations maintained visual parity\n- Print styles converted to Tailwind print: variants\n- Hover and active states preserved with Tailwind state variants\n- Responsive design maintained with existing breakpoint behavior\n\nTest Reliability:\n- Test selectors now target semantic element roles rather than implementation details\n- More robust element identification reduces test flakiness\n- Consistent testing pattern across all timeline-related components\n- Better separation between styling and testing concerns\n\nDemo Development:\n- Interactive demo supports real-time component behavior testing\n- Mock data factory functions for consistent test data generation\n- Multiple demo scenarios covering empty state, populated timeline, and various loading states\n- Control buttons for testing user interactions and state changes\n\nFiles Modified:\n- sketch/webui/src/web-components/sketch-timeline.ts: TailwindElement inheritance and Tailwind class conversion\n- sketch/webui/src/web-components/sketch-timeline.test.ts: Updated test selectors to use data-testid attributes\n- sketch/webui/src/web-components/demo/sketch-timeline.demo.ts: New interactive demo module\n- sketch/webui/src/web-components/demo/demo-framework/demo-runner.ts: Added sketch-timeline to knownComponents\n\nThe conversion maintains complete visual and functional parity while enabling\nconsistent styling across the component library and improving test reliability\nthrough semantic element targeting.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s0621383cac6304dek\n" }, { "commit": "4337aa70a436963ed0e1d563215dc9eb1f176085", "tree": "90c2d000e2d901af1116f27bbb9e3a4d7988b542", "parents": [ "6fc754f4473d250a1873497d34727f540f214e79" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Fri Jun 27 23:41:33 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Sat Jun 28 01:04:16 2025 +0000" }, "message": "add sketch-app-shell demo fixture\n\nCreates TypeScript demo module for sketch-app-shell component with:\n- Interactive controls for mode switching (chat, diff, terminal)\n- Follows established demo framework conventions\n- Added component to demo runner registry\n- Enables testing via npm run demo:runner or npx vite\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s25f711ddd94f1609k\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": "297327174d1d13cf185cf781c19813f2a05ca809", "tree": "17e91a7f31ea3bb23ebb9282e6f682813278ed46", "parents": [ "4ce9dca335692e6aeff59c57f9a91b6ee19756ed" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 27 09:57:12 2025 -0700" }, "committer": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Fri Jun 27 09:57:41 2025 -0700" }, "message": "Pin \u0027npx prettier\u0027 version\n\nIf you don\u0027t do this, you get cruft from various autofixers competing.\n" }, { "commit": "9aa141a4bdde68a1ad57d56ba3cb0023109c6b7b", "tree": "343ad7ce6404ceeee210a1f43d4b3c0f86e70756", "parents": [ "b01b150abe7bf3066a08dcaafe1e443aa101c5c1" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jun 27 05:02:24 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 05:07:21 2025 +0000" }, "message": "skaband: remove all manual input switching functionality from sessions page\n\nfix for #430\n\nCompletely remove auto-switch and manual switch functionality to simplify\nthe repository selection interface and eliminate user confusion.\n\nProblems Solved:\n\nComplex Dual-Mode Interface:\n- Auto-switch feature automatically changed UI state without user consent\n- Manual switch buttons created confusing dual-mode interface\n- Users unclear when to use dropdown vs manual input vs auto-switching\n- Multiple code paths for repository input handling created maintenance overhead\n\nInconsistent User Experience:\n- Repository loading failures triggered unexpected auto-switch behavior\n- Manual buttons overlaid on repository selection interface\n- Console error logging without clear user benefit\n- No clear guidance on which input method to use\n\nJavaScript Complexity:\n- switchToManualInput() and enhancedSwitchToManualInput() functions\n- Complex validation setup and event handlers for switching\n- setTimeout scripts for auto-switch timeouts and console logging\n- Event listeners and DOM manipulation for button interactions\n\nSolution Implementation:\n\nComplete Feature Removal:\n- Removed all auto-switch timeout scripts and console error logging\n- Eliminated switchToManualInput() and enhancedSwitchToManualInput() functions\n- Removed all manual switch buttons from repository selection interface\n- Simplified JavaScript by removing switching-related event handlers\n\nSimplified Repository Selection:\n- Repository selection now uses standard HTML select dropdown only\n- Hidden text input remains for loading state (standard form behavior)\n- No manual override buttons or automatic switching functionality\n- Clean, predictable interface focused on dropdown selection\n\nTemplate Structure Cleanup:\n- Removed button elements with switch-to-manual IDs from HTML\n- Eliminated complex validation setup code tied to manual switching\n- Simplified repository selection container structure\n- Preserved core form functionality and submission workflow\n\nImplementation Details:\n\nRemoved JavaScript Functions:\n- switchToManualInput(): DOM manipulation for switching input modes\n- enhancedSwitchToManualInput(): Validation setup and enhanced switching\n- Auto-switch timeout script: 10-second timeout with console logging\n- Manual button event handlers: onclick setup and event listeners\n\nRemoved HTML Elements:\n- Manual switch button in dropdown mode\n- Manual switch button in loading state\n- All switch-to-manual ID references\n- Button overlay positioning and styling\n\nPreserved Core Functionality:\n- Repository select dropdown when GitHubRepos available\n- Hidden text input for loading states\n- Form submission and validation workflow\n- Repository selection and session creation process\n\nTesting Updates:\n- Updated TestRepositoryUIFeatures to reflect simplified interface\n- Removed tests checking for presence of removed switching functionality\n- Tests now focus on core dropdown and loading state functionality\n- Verified repository selection continues to work with standard form behavior\n\nUser Experience Impact:\n- Single, predictable way to select repositories via dropdown\n- No unexpected UI changes or automatic state transitions\n- Standard HTML form behavior users expect\n- Reduced cognitive load with simpler interface\n\nFiles Modified:\n- skaband/sessions.gohtml: Removed switching functions, buttons, and timeout scripts\n- skaband/skaband_test.go: Updated tests for simplified interface\n\nThe simplified interface eliminates confusion by providing a single,\nstandard way to select repositories through HTML dropdown selection.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: sdc7975f0ace4b511k\n" }, { "commit": "b01b150abe7bf3066a08dcaafe1e443aa101c5c1", "tree": "3488103cc3b3f9a124d04f7cd6f0619406e74f7a", "parents": [ "b379592340a40c4c22ac1a52be4d2e612a9aa4f4" ], "author": { "name": "Philip Zeyliger", "email": "philip@bold.dev", "time": "Thu Jun 26 21:20:12 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 04:21:19 2025 +0000" }, "message": "Fix nil pointer dereference #432\n" }, { "commit": "b379592340a40c4c22ac1a52be4d2e612a9aa4f4", "tree": "37d9ce08602651cc11b0b1bc2f46ebdd6907fc6a", "parents": [ "659b98361783ad139412ea2a0bc62c8ed25c292e" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Fri Jun 27 01:59:41 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 02:04:42 2025 +0000" }, "message": "webui: convert sketch-call-status and sketch-chat-input to SketchTailwindElement with comprehensive demo support\n\nConvert both sketch-call-status and sketch-chat-input components from shadow DOM CSS to Tailwind classes while maintaining test compatibility and adding comprehensive demo infrastructure.\n\nProblems Solved:\n\nShadow DOM Styling Limitations:\n- Both components used CSS-in-JS with shadow DOM preventing Tailwind integration\n- Large static styles blocks with custom CSS duplicated Tailwind functionality\n- Components couldn\u0027t benefit from design system consistency\n- Difficult to maintain custom CSS alongside Tailwind-based components\n\nMissing Demo Infrastructure:\n- sketch-call-status had no demo fixtures for testing component states\n- sketch-chat-input needed dedicated demo fixture following naming convention\n- Components not properly integrated into demo runner system\n\nTest Compatibility Issues:\n- Conversion to Tailwind required updating shadow DOM selectors\n- renderRoot.querySelector calls needed conversion to direct querySelector\n- Component tests needed updating for non-shadow DOM structure\n\nSolution Implementation:\n\nTailwind CSS Conversion - sketch-call-status:\n- Changed sketch-call-status to inherit from SketchTailwindElement\n- Replaced CSS-in-JS styles with Tailwind utility classes and inline animations\n- Converted animations using @keyframes in inline \u003cstyle\u003e tag\n- Maintained exact visual appearance while using Tailwind classes\n\nTailwind CSS Conversion - sketch-chat-input:\n- Changed sketch-chat-input to inherit from SketchTailwindElement\n- Replaced extensive static styles CSS block with Tailwind utility classes\n- Converted complex chat container, input wrapper, and button styling\n- Added custom fade-in animation to tailwind.config.js with keyframes\n\nKey Tailwind Class Mappings:\n- Call status indicators: bg-yellow-100 text-amber-500 (active), text-gray-400 (idle)\n- Status banners: bg-green-50 text-green-700 (idle), bg-orange-50 text-orange-600 (working)\n- Chat container: w-full bg-gray-100 p-4 min-h-[40px] relative\n- Chat input: flex-1 p-3 border border-gray-300 rounded resize-y font-mono\n- Send button: bg-blue-500 hover:bg-blue-600 disabled:bg-gray-400\n\nShadow DOM to Light DOM Conversion:\n- Removed static styles properties completely\n- Updated all renderRoot.querySelector calls to direct querySelector calls\n- Changed shadow DOM event handler setup to work with light DOM\n- Maintained all drag-and-drop and event handling functionality\n\nTest Compatibility Maintenance:\n- Added semantic CSS classes back to elements for test selectors\n- Updated sketch-chat-input.test.ts to use querySelector instead of renderRoot.querySelector\n- Fixed drag event simulation to work with light DOM structure\n- All existing tests continue to pass with updated selectors\n\nDemo Infrastructure Implementation:\n- Created call-status.ts demo fixtures with CallStatusState interface\n- Added comprehensive sketch-call-status.demo.ts with interactive controls\n- Created chat-input.ts demo fixture with message display and controls\n- Added both components to demo-runner.ts knownComponents list\n\nInteractive Demo Features:\n- Call status: Status variations, interactive LLM/tool call controls, connection toggle\n- Chat input: Message display with user/bot styling, multiple preset buttons\n- Both demos include real-time state updates and comprehensive examples\n\nDependencies:\n- Added @tailwindcss/vite package for Tailwind integration\n- Updated package.json and package-lock.json with new dependency\n\nFiles Modified:\n- sketch/webui/src/web-components/sketch-call-status.ts: Converted to SketchTailwindElement\n- sketch/webui/src/web-components/sketch-chat-input.ts: Converted to SketchTailwindElement\n- sketch/webui/src/web-components/sketch-chat-input.test.ts: Updated selectors for light DOM\n- sketch/webui/src/web-components/demo/demo-fixtures/call-status.ts: Added call status fixtures\n- sketch/webui/src/web-components/demo/demo-fixtures/index.ts: Export call status fixtures\n- sketch/webui/src/web-components/demo/sketch-call-status.demo.ts: Complete interactive demo\n- sketch/webui/src/web-components/demo/chat-input.ts: New chat input demo fixture\n- sketch/webui/src/web-components/demo/demo-framework/demo-runner.ts: Added both components\n- sketch/webui/tailwind.config.js: Added custom fade-in animation\n- sketch/webui/package.json: Added @tailwindcss/vite dependency\n\nTesting and Validation:\n- All component tests pass with updated selectors\n- Components render correctly with Tailwind classes\n- All functionality preserved including animations and interactions\n- Interactive demos load and function properly\n- Components appear in demo runner list\n\nThe conversion maintains all functionality while enabling better integration\nwith the Tailwind-based design system and providing comprehensive demo\ninfrastructure for development and testing.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s97f4190763cfe17ak\n" }, { "commit": "659b98361783ad139412ea2a0bc62c8ed25c292e", "tree": "ee1ce34629a13101b0cb1a492f0457c1e83a9eaa", "parents": [ "49577498f65808da9faaa1745e66a590e0ad3583" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Fri Jun 27 00:50:41 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 00:53:48 2025 +0000" }, "message": "webui: convert SketchCallStatus to Tailwind CSS with comprehensive demo support\n\nConvert SketchCallStatus component from shadow DOM CSS to Tailwind classes\nwhile maintaining test compatibility and adding complete demo infrastructure.\n\nProblems Solved:\n\nShadow DOM Styling Limitations:\n- SketchCallStatus used CSS-in-JS with shadow DOM preventing Tailwind integration\n- Custom CSS animations and styling duplicated Tailwind functionality\n- Component couldn\u0027t benefit from design system consistency\n- Difficult to maintain custom CSS alongside Tailwind-based components\n\nMissing Demo Infrastructure:\n- No demo fixtures for testing SketchCallStatus component states\n- Component not included in demo runner for development testing\n- Manual testing required for visual verification of component behavior\n\nTest Compatibility Issues:\n- Conversion to Tailwind removed semantic class names expected by tests\n- Need to maintain backward compatibility with existing test suite\n\nSolution Implementation:\n\nTailwind CSS Conversion:\n- Changed SketchCallStatus to inherit from SketchTailwindElement\n- Replaced CSS-in-JS styles with Tailwind utility classes\n- Converted animations using @keyframes in inline \u003cstyle\u003e tag\n- Maintained exact visual appearance while using Tailwind classes\n\nComponent State Styling:\n- LLM indicator: bg-yellow-100 text-amber-500 when active, text-gray-400 when idle\n- Tool indicator: bg-blue-100 text-blue-500 when active, text-gray-400 when idle\n- Status banner: bg-green-50 text-green-700 (idle), bg-orange-50 text-orange-600 (working), bg-red-50 text-red-600 (disconnected)\n- Gentle pulse animation preserved with animate-gentle-pulse class\n\nTest Compatibility Maintenance:\n- Added semantic CSS classes back to elements (.llm-indicator, .tool-indicator, .status-banner)\n- Added .active class when indicators are in active state\n- Added status state classes (status-idle, status-working, status-disconnected)\n- Maintains backward compatibility with existing Playwright tests\n\nDemo Fixtures Implementation:\n- Added call-status.ts with CallStatusState interface and sample states\n- Created demo fixtures: idleCallStatus, workingCallStatus, heavyWorkingCallStatus, disconnectedCallStatus, workingDisconnectedCallStatus\n- Fixed TypeScript module export issues using \u0027export type\u0027 syntax\n- Comprehensive sketch-call-status.demo.ts with interactive controls\n- Added component to demo-runner.ts knownComponents list\n\nInteractive Demo Features:\n- Status variations section showing all possible states\n- Interactive demo with buttons to add/remove LLM calls and tool calls\n- Toggle connection state and change agent state functionality\n- Reset button to return to idle state\n- Real-time simulation of activity changes\n\nFiles Modified:\n- sketch/webui/src/web-components/sketch-call-status.ts: Converted to SketchTailwindElement with Tailwind classes and semantic class names\n- sketch/webui/src/web-components/demo/demo-fixtures/call-status.ts: Added call status demo data\n- sketch/webui/src/web-components/demo/demo-fixtures/index.ts: Export call status fixtures with proper TypeScript module exports\n- sketch/webui/src/web-components/demo/sketch-call-status.demo.ts: Complete demo implementation with interactive controls\n- sketch/webui/src/web-components/demo/demo-framework/demo-runner.ts: Added sketch-call-status to knownComponents\n\nTesting and Validation:\n- Verified component renders correctly with Tailwind classes\n- Confirmed all state variations display proper colors and animations\n- Tested interactive demo controls function correctly\n- Validated component appears in demo runner list\n- Ensured test compatibility with semantic class preservation\n\nThe conversion maintains visual fidelity and test compatibility while enabling\nbetter integration with the Tailwind-based design system and providing\ncomprehensive demo infrastructure for development and testing.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s3437e5020555164dk\n" }, { "commit": "49577498f65808da9faaa1745e66a590e0ad3583", "tree": "aed3194233b9f8ceec24cb930f066abace25b477", "parents": [ "d5c849d0923c6b254047ebb07589adf10bc18548" ], "author": { "name": "Sean McCullough", "email": "banksean@gmail.com", "time": "Thu Jun 26 17:13:28 2025 -0700" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Fri Jun 27 00:16:07 2025 +0000" }, "message": "sketch-container-status: fix long branch names\n" }, { "commit": "d5c849d0923c6b254047ebb07589adf10bc18548", "tree": "4c3daefe24c0c618b1499452ca1dc974311f880b", "parents": [ "4b64468e6dd5d4283c90d85a483351af11933c2f" ], "author": { "name": "banksean", "email": "banksean@gmail.com", "time": "Thu Jun 26 15:48:31 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jun 26 19:25:55 2025 +0000" }, "message": "webui: convert SketchViewModeSelect to use SketchTailwindElement with Tailwind CSS\n\nReplace LitElement shadow DOM component with SketchTailwindElement base class\nto use Tailwind CSS utility classes instead of component-scoped CSS styles.\n\nAlso adds tailwind support for CSS container queries in addition to media\nqueries.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s0eeb80dd54594375k\n" }, { "commit": "4b64468e6dd5d4283c90d85a483351af11933c2f", "tree": "3c3721c5714d88bba7c04f292e705da339dfdbfc", "parents": [ "07b7400182aefc4153eb6617ab89a5f05f577349" ], "author": { "name": "David Crawshaw", "email": "david@zentus.com", "time": "Thu Jun 26 17:15:10 2025 +0000" }, "committer": { "name": "Autoformatter", "email": "bot@sketch.dev", "time": "Thu Jun 26 18:32:41 2025 +0000" }, "message": "webui: add compactPadding parameter to reduce message padding\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s33c29800305d4dd2k\n" } ], "next": "261c9116af76218e9d0c4c7c426f7829b911421c" }