dsh-dracula-theme 0.1.0 → 0.1.2

package/lib/client.js

@@ -456,13 +456,49 @@ window.__ModuleLoader__.load({
 			}
 		}
 
-		/** Saved skin id (may be unknown/absent). */
+		/**
+		 * The DSH Desktop client boots its web app on a fresh random port
+		 * every launch (`--port 0`), and localStorage is scoped per origin —
+		 * including the port. A choice saved under http://127.0.0.1:55337 is
+		 * invisible to the next launch at http://127.0.0.1:49562, so the
+		 * theme silently reverts to default on every desktop start.
+		 * Cookies, by contrast, are scoped per host and ignore the port, and
+		 * the desktop client keeps a persistent cookie jar. So the saved
+		 * skin is mirrored into a host-scoped cookie; reads prefer the
+		 * cookie and fall back to localStorage (the plain `dsh web` browser
+		 * session keeps its fixed-port origin, where both stores agree).
+		 */
+		const COOKIE_KEY = "dsh-dracula-skin";
+		const COOKIE_MAX_AGE = 60 * 60 * 24 * 400;
+
+		/** Read a cookie value (null on absence). */
+		function readCookie(name) {
+			try {
+				const match = document.cookie.match(new RegExp(`(?:^|;\\s*)${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}=([^;]*)`));
+				return match ? decodeURIComponent(match[1]) : null;
+			} catch {
+				return null;
+			}
+		}
+
+		/** Write (or remove with max-age=0) a cookie. */
+		function writeCookie(name, value) {
+			try {
+				if (value === null) document.cookie = `${name}=; path=/; max-age=0`;
+				else document.cookie = `${name}=${encodeURIComponent(value)}; path=/; max-age=${COOKIE_MAX_AGE}`;
+			} catch {
+				// storage unavailable — the preference stays process-local
+			}
+		}
+
+		/** Saved skin id (may be unknown/absent): cookie first, localStorage fallback. */
 		function readSavedSkin() {
-			return readStorage(STORAGE_KEY);
+			return readCookie(COOKIE_KEY) ?? readStorage(STORAGE_KEY);
 		}
 
 		/** Persist a skin choice; DEFAULT_SKIN clears the stored value. */
 		function writeSavedSkin(id) {
+			writeCookie(COOKIE_KEY, id === DEFAULT_SKIN ? null : id);
 			writeStorage(STORAGE_KEY, id === DEFAULT_SKIN ? null : id);
 		}
 		//#endregion
@@ -765,29 +801,51 @@ window.__ModuleLoader__.load({
 				style.remove();
 			}, "dsh-dracula-theme: surface tint lifecycle");
 
-			// Restore the saved skin. The ThemeService adopts its durable
-			// built-in preference ("light"/"dark"/"system") from the Host
-			// settings scope asynchronously after boot, which overwrites a
-			// third-party preference restored too early. Re-assert the saved
-			// skin for a short boot window (a handful of change events or a
-			// few seconds), then yield to subsequent user actions.
+			// Restore the saved skin against the boot adoption storm. The
+			// ThemeService adopts its durable built-in preference
+			// ("light"/"dark"/"system") from the Host settings scope
+			// asynchronously after boot; every adoption emits theme/change
+			// with a built-in preference that overwrites a third-party one.
+			// In slower boot paths — the desktop client's RPC bridge emits
+			// more, later sync events — a fixed retry budget (a handful of
+			// tries within a few seconds) can be exhausted and the theme
+			// silently reverts to default. So instead of a time-boxed guard
+			// we stay armed while the storm is active: re-assert the saved
+			// skin on every built-in revert, and only yield to user actions
+			// once the preference has rested on the saved skin for a quiet
+			// period with no further adoption events.
 			const saved = readSavedSkin();
 			const savedValid = typeof saved === "string" && saved !== DEFAULT_SKIN && SKINS.some((skinDefinition) => skinDefinition.id === saved);
-			let bootGuard = savedValid ? 3 : 0;
-			const reassertSaved = () => {
-				if (bootGuard <= 0) return;
-				const current = ctx.theme.getTheme().preference;
-				if (current === saved) return;
-				bootGuard -= 1;
-				ctx.theme.setTheme(saved);
+			let armed = savedValid;
+			let quietTimer;
+			const disarm = () => {
+				armed = false;
+			};
+			const scheduleDisarm = () => {
+				clearTimeout(quietTimer);
+				quietTimer = setTimeout(disarm, 4000);
+			};
+			const reassertAgainstAdoption = () => {
+				if (!armed) return;
+				const pref = ctx.theme.getTheme().preference;
+				if (pref === saved) {
+					scheduleDisarm();
+					return;
+				}
+				if (pref === "light" || pref === "dark" || pref === DEFAULT_SKIN) {
+					// A built-in won — during boot this can only be a scope
+					// adoption (nobody reaches the Appearance row that fast).
+					// Re-assert, then require another quiet period.
+					ctx.theme.setTheme(saved);
+					scheduleDisarm();
+				}
+				// Another third-party theme — the drop-foreign convention
+				// below clears our stored choice instead of fighting.
 			};
-			reassertSaved();
-			const bootWindow = setTimeout(() => {
-				bootGuard = 0;
-			}, 5000);
+			reassertAgainstAdoption();
 			ctx.effect(() => () => {
-				clearTimeout(bootWindow);
-			}, "dsh-dracula: boot restore window");
+				clearTimeout(quietTimer);
+			}, "dsh-dracula-theme: boot restore lifecycle");
 
 			const skinStore = createSkinStore();
 			let skinBound;
@@ -807,7 +865,7 @@ window.__ModuleLoader__.load({
 				// dispatch is missed by other subscribers (ui-layout's
 				// ThemePresenter), so the restored skin would never reach the DOM.
 				setTimeout(() => {
-					reassertSaved();
+					reassertAgainstAdoption();
 				}, 0);
 			});
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-dracula-theme",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "🧛 Dracula theme for DeepSeek Harness: the classic dark palette (plus the Soft variant) registered into the built-in theme runtime",
   "license": "MIT",
   "author": "ossFrankFrank",