From 560513d435b2732aa17837bcadca572044c96b54 Mon Sep 17 00:00:00 2001
From: ChuckBuilds <charlesmynard@gmail.com>
Date: Thu, 16 Jul 2026 16:11:00 -0400
Subject: [PATCH] fix(web): remove redundant htmx:afterSwap script re-execution
 (was double-executing every partial's inline script)

Re-verified this CodeRabbit finding, previously deferred as "needs isolated
testing" - traced it to a confirmed, active bug rather than a style
concern:

htmx 1.9.10's own config defaults to allowScriptTags: true (confirmed in
the vendored htmx.min.js, which itself contains the same clone-and-reinsert
<script> mechanism internally). This means htmx ALREADY re-executes every
<script> tag in swapped content by default, exactly like a browser
navigating to a new page. The custom htmx:afterSwap listener in
htmx-config.js did the identical clone-and-reinsert a SECOND time on top of
htmx's own handling - so every inline <script> block in every HTMX-loaded
partial (overview, display, durations, plugin config, etc. - most partials
have one) executed twice per load.

Confirmed safe to delete outright, not just narrow: grepped every hand-written
JS file for a manual `dispatchEvent(... 'htmx:afterSwap' ...)` that might
have relied on this handler for a non-htmx code path (e.g. the direct-fetch
fallbacks like loadOverviewDirect) - none exists, so nothing depended on
this listener specifically; htmx's native handling covers every real
htmx-driven swap on its own.

Left in place, unchanged: the console.error/console.warn global override
a few lines up in the same file, which suppresses known-noisy
HTMX-timing-race messages. That one is a legitimate anti-pattern too
(broad substring matching can mask unrelated errors) but redesigning it
needs care to preserve real diagnostics while still hiding the specific
harmless races it targets - a scoped follow-up, not a same-day deletion
like this confirmed-duplicate handler.

Validation: all 27 fast web tests pass; JS brace/paren balance sanity
checked (no local Node/browser available in this sandbox to execute the
file directly - verify manually in-browser before merge).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KEZK1P1Q1fu5pcuVrkrCFZ
---
 web_interface/static/v3/js/htmx-config.js | 38 ++++++-----------------
 1 file changed, 9 insertions(+), 29 deletions(-)

diff --git a/web_interface/static/v3/js/htmx-config.js b/web_interface/static/v3/js/htmx-config.js
index 731ce69c..20761507 100644
--- a/web_interface/static/v3/js/htmx-config.js
+++ b/web_interface/static/v3/js/htmx-config.js
@@ -162,35 +162,15 @@
                         // Log but don't break the app
                         console.warn('HTMX swap error:', event.detail);
                     });
-                    
-                    document.body.addEventListener('htmx:afterSwap', function(event) {
-                        if (event.detail && event.detail.target) {
-                            try {
-                                const scripts = event.detail.target.querySelectorAll('script');
-                                scripts.forEach(function(oldScript) {
-                                    try {
-                                        if (oldScript.textContent.trim() || oldScript.src) {
-                                            const newScript = document.createElement('script');
-                                            if (oldScript.src) newScript.src = oldScript.src;
-                                            if (oldScript.type) newScript.type = oldScript.type;
-                                            if (oldScript.textContent) newScript.textContent = oldScript.textContent;
-                                            if (oldScript.parentNode) {
-                                                oldScript.parentNode.insertBefore(newScript, oldScript);
-                                                oldScript.parentNode.removeChild(oldScript);
-                                            } else {
-                                                // If no parent, append to head or body
-                                                (document.head || document.body).appendChild(newScript);
-                                            }
-                                        }
-                                    } catch {
-                                        // Silently ignore script execution errors
-                                    }
-                                });
-                            } catch {
-                                // Silently ignore errors in script processing
-                            }
-                        }
-                    });
+
+                    // Note: no custom htmx:afterSwap script re-execution here.
+                    // htmx's own default config (allowScriptTags: true, on by
+                    // default - confirmed in the vendored htmx.min.js) already
+                    // clones and re-inserts <script> tags in swapped content
+                    // to make the browser execute them, exactly like a real
+                    // page load. A duplicate handler here previously did the
+                    // same clone-and-reinsert a second time, executing every
+                    // inline <script> in every HTMX-loaded partial TWICE.
 
                     // Mark tab containers as loaded once their content settles, so switching
                     // away and back doesn't re-fetch. Scoped to the "loadtab" trigger (tab