From 8da5ef72314c054d6bbe69e8ae108dd31fa08052 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sun, 11 Jan 2026 08:10:21 +0000
Subject: [PATCH] Fix: Remove talk.js from SPA to prevent null element errors

The talk.js script has global-scope DOM access (getElementById at top level)
that causes "recordButton is null" errors when loaded in the SPA context.
The SPA talk view has its own inline JavaScript that handles recording
functionality with properly namespaced element IDs (spa-recordButton, etc.).

Co-authored-by: mudler <2420543+mudler@users.noreply.github.com>
---
 core/http/views/spa.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/http/views/spa.html b/core/http/views/spa.html
index 58c0fd3ac..b78013213 100644
--- a/core/http/views/spa.html
+++ b/core/http/views/spa.html
@@ -359,7 +359,8 @@
 <script defer src="static/chat.js"></script>
 <script defer src="static/image.js"></script>
 <script defer src="static/tts.js"></script>
-<script defer src="static/talk.js"></script>
+<!-- Note: talk.js is NOT included here because it has global-scope DOM access that 
+     conflicts with the SPA architecture. The SPA talk view has its own inline JS. -->
 <script src="static/assets/pdf.min.js"></script>
 <script>
   // Initialize PDF.js worker