fix: guard togglePopup against missing event argument

AgentGUI · claude · AgentGUI · commit 8fdb279b9155 · 2026-04-18T04:54:49.000Z
Co-Authored-By: Claude Opus 4.7 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/static/js/tools-manager.js b/static/js/tools-manager.js
@@ -142,7 +142,7 @@
     applyDiff(scroll, vnodes);
   }
 
-  function togglePopup(e) { e.stopPropagation(); if (!popup.classList.contains('open')) { isRefreshing = false; refresh(); } popup.classList.toggle('open'); }
+  function togglePopup(e) { if (e) e.stopPropagation(); if (!popup.classList.contains('open')) { isRefreshing = false; refresh(); } popup.classList.toggle('open'); }
   function closePopup() { popup.classList.remove('open'); }
   function refresh() { fetchTools(); }
 

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@`
`142`	`142`	`applyDiff(scroll, vnodes);`
`143`	`143`	`}`
`144`	`144`
`145`		`- function togglePopup(e) { e.stopPropagation(); if (!popup.classList.contains('open')) { isRefreshing = false; refresh(); } popup.classList.toggle('open'); }`
	`145`	`+ function togglePopup(e) { if (e) e.stopPropagation(); if (!popup.classList.contains('open')) { isRefreshing = false; refresh(); } popup.classList.toggle('open'); }`
`146`	`146`	`function closePopup() { popup.classList.remove('open'); }`
`147`	`147`	`function refresh() { fetchTools(); }`
`148`	`148`