From 9e3755f9caeb5112454fd8360d41bf53aaa5fd3f Mon Sep 17 00:00:00 2001
From: Gmer4Lfe <gmer4lfe@gmail.com>
Date: Fri, 7 Aug 2026 10:29:13 -0400
Subject: [PATCH] Take the literal script tag out of the comment that broke
 every tab

Written with real angle brackets inside the inline block, it ended that element early, so the
file rendered as visible text and no page JS ran at all.
---
 Plugin/unraid/Varaverk.page | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Plugin/unraid/Varaverk.page b/Plugin/unraid/Varaverk.page
index 5859901..c75725c 100644
--- a/Plugin/unraid/Varaverk.page
+++ b/Plugin/unraid/Varaverk.page
@@ -65,10 +65,16 @@ require_once "$pluginDir/include/config.php";
 // available to it at all.
 //
 // Not in js/varaverk.js, which would otherwise be the obvious home: that file is loaded by a
-// <script src> *below* the tab include, so it is not defined yet while a page's inline script is
+// src-tag *below* the tab include, so it is not defined yet while a page's inline script is
 // running. The shared formatters there survive only because every caller is inside a fetch
 // callback. An escaping helper must be callable from the first synchronous line of a page.
 //
+// Never write a literal script tag inside this block, not even in a comment. The tab body is
+// injected into the page by the webGUI rather than parsed as part of the document, and the
+// opening sequence ends this element early wherever it appears — the whole file then renders as
+// visible text and every tab loads dead. That is not theoretical: the comment above said it with
+// real angle brackets and took the monitor tab down on 2026-08-07.
+//
 // String() rather than assuming a string: these are fed payload fields that are frequently
 // numbers, and sometimes null or undefined. A helper that throws on a number is a helper call
 // sites will skip.