dsh-theme-gallery 0.1.3 → 0.1.4

package/lib/client.js

@@ -1118,22 +1118,36 @@ window.__ModuleLoader__.load({
     }
 
     /**
-     * Whether the gallery shows its diagnostic line.
+     * Where the debug switch is remembered.
      *
-     * Opt-in through the URL hash — `#theme-gallery-debug` — so the page stays
-     * clean in normal use while the one tool that actually located the token and
-     * presenter faults stays one reload away. From outside the app, "selected but
-     * nothing changed" cannot be split into "not registered", "not selected",
-     * "registered with broken tokens", or "painted over by our own CSS"; this line
-     * splits it by showing what the theme SERVICE believes next to what the
-     * document actually resolves.
-     * @returns true when the debug line should render.
+     * A localStorage flag in addition to the URL fragment, because this plugin
+     * ships as a package inside the desktop app: there is no address bar to add
+     * `#theme-gallery-debug` to, so DevTools is the way in.
+     */
+    const DEBUG_KEY = 'theme-gallery:debug'
+
+    /**
+     * Whether the gallery shows its diagnostic detail lines.
+     *
+     * The raw readings — token counts, geometry, hit tests, the attempt log — are
+     * for whoever is debugging, not for whoever is picking a skin, and this is a
+     * shipped package. So they are opt-in, through either switch:
+     *
+     *   DevTools console:  localStorage.setItem('theme-gallery:debug', '1')
+     *   …or a URL with      #theme-gallery-debug
+     *   turn it off:       localStorage.removeItem('theme-gallery:debug')
+     *
+     * The switch gates DETAIL, never bad news: a line that reports a problem prints
+     * whether or not it is on (see `sceneryLineIsWarning`), because "没有报错" and
+     * "没有观测到报错" have been confused in this project before.
+     * @returns true when the detail lines should render.
      */
     function debugEnabled() {
       try {
-        return typeof window !== 'undefined'
+        if (typeof window !== 'undefined'
           && typeof window.location?.hash === 'string'
-          && window.location.hash.includes('theme-gallery-debug')
+          && window.location.hash.includes('theme-gallery-debug')) return true
+        return typeof window !== 'undefined' && window.localStorage?.getItem(DEBUG_KEY) === '1'
       } catch {
         return false
       }
@@ -1211,12 +1225,15 @@ window.__ModuleLoader__.load({
     /**
      * The one-line scenery status shown on the panel.
      *
-     * Deliberately unconditional during bring-up, and deliberately dumb: it prints
-     * the report whatever it says. The previous revision only spoke when a
-     * self-check considered something wrong, so a check that passed while the
-     * scenery was still invisible produced **silence** — the worst possible output
-     * for a diagnostic. Printing the numbers always means the reader sees the
-     * geometry even when the code's opinion of it is wrong.
+     * Deliberately dumb: it prints the report whatever it says. An earlier
+     * revision only spoke when a self-check considered something wrong, so a check
+     * that PASSED while the scenery was still invisible produced **silence** — the
+     * worst possible output for a diagnostic. Keeping the numbers means the reader
+     * sees the geometry even when the code's opinion of it is wrong.
+     *
+     * It was unconditional while the scenery was being brought up; now that the
+     * package is public the RENDER SITE decides: routine reports wait for
+     * `debugEnabled()`, while warnings and errors always print.
      * @param selected - the preference the page is showing as selected.
      * @returns the line, or null when the active theme asks for no scenery.
      */
@@ -1231,6 +1248,20 @@ window.__ModuleLoader__.load({
       }
     }
 
+    /**
+     * Whether the scenery line reports a problem rather than a routine pass.
+     *
+     * The line has three shapes: `装饰自检通过 · …` (routine), `⚠ <warning> · …`,
+     * and `装饰自检失败: …`. Only the first may hide behind the debug switch — a
+     * failure has to reach the person looking at the panel without them knowing
+     * that a switch exists.
+     * @param line - the scenery line.
+     * @returns true when the line carries a warning or an error.
+     */
+    function sceneryLineIsWarning(line) {
+      return typeof line === 'string' && (line.startsWith('⚠') || line.includes('失败'))
+    }
+
     /**
      * Render the scenery report as one line.
      * @returns the report text.
@@ -2879,6 +2910,9 @@ window.__ModuleLoader__.load({
           ],
         })
       }
+      // Detail lines are opt-in; a line that reports a problem is not detail.
+      const scenery = sceneryLine(selected)
+      const showScenery = scenery !== null && (debugEnabled() || sceneryLineIsWarning(scenery))
       return jsxs('div', {
         className: 'tg-page',
         children: [
@@ -2891,16 +2925,15 @@ window.__ModuleLoader__.load({
             ],
           }),
           debugEnabled() ? jsx('div', { className: 'tg-debug', children: themeDiagnostics(selected) }) : null,
-          // Unconditional while the scenery is being brought up: when the active
-          // skin's decorations are not on screen, the reason has to reach the person
-          // looking at it. Shows nothing once the report says the layer is drawn and
-          // inside the column.
-          sceneryLine(selected) === null
-            ? null
-            : jsx('div', {
+          // Routine reports wait for the debug switch; warnings and errors print
+          // regardless — when the active skin's decorations are missing, the reason
+          // has to reach the person looking at the panel.
+          showScenery
+            ? jsx('div', {
               className: `tg-debug${ambientWarning(selected) === null ? '' : ' tg-warn'}`,
-              children: sceneryLine(selected),
-            }),
+              children: scenery,
+            })
+            : null,
           jsx('div', {
             className: 'tg-grid',
             children: ids.map((id) => jsx(ThemeCard, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-theme-gallery",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "A community theme gallery for DeepSeek Harness (DSH): JSON-defined skins with full-screen coverage and ported sidebar scenery, switched from the plugin's own sidebar panel — adding skins needs no code change",
   "author": "renjie2026 (https://github.com/renjie2026)",
   "license": "MIT",