perfectionist-plugins 1.0.1 → 1.0.3

package/README.md

@@ -48,31 +48,19 @@ Built for [Pengu Loader](https://pengu.lol)
 2. Move the `Perfectionism` folder into your Pengu Loader `plugins` folder.
 3. Restart the League Client.
 
-That's it. The folder holds one tiny loader file — the plugins themselves come from the
+That's it. The folder holds one tiny loader file. The plugins themselves come from the
 [`perfectionist-plugins`](https://www.npmjs.com/package/perfectionist-plugins) npm package the
 first time the client starts. Install, remove and toggle plugins in **Settings → Perfectionism**.
 
-<br>
-
 ## Updates
 
-Every time the client starts, the loader checks npm for a newer version and runs it — publishing
+Every time the client starts, the loader checks npm for a newer version and runs it. Publishing
 a new version of the npm package **is** the update. If a new version appears while the client is
 already open, the suite downloads it with a progress window and restarts the client at a safe
 moment (never in the middle of a game). You never replace files by hand.
 
 <br>
 
-## Publishing an update (maintainers)
-
-1. Bump `version` in `package.json`.
-2. Run `npm publish` from the repo root (first time only: `npm login` first).
-
-That's the whole release process — every install picks the new version up on its own. Never edit
-anything inside the `Perfectionism` folder: that one file stays the same for everyone.
-
-<br>
-
 ---
 
 <div align="center">

package/hide-unowned-skins.js

@@ -1,4 +1,4 @@
-export const version = "2.1.0";
+export const version = "2.2.0";
 
 const PLUGIN_ID = "hide-unowned-skins";
 const STYLE_ID = "perfectionist-hide-unowned-skins-style";
@@ -11,7 +11,7 @@ const CAROUSEL_URI = "/lol-champ-select/v1/skin-carousel-skins";
 const CURRENT_SUMMONER_URI = "/lol-summoner/v1/current-summoner";
 const SUMMONER_TTL = 10 * 60 * 1000;
 const INVENTORY_TTL = 60 * 1000;
-const SESSION_POLL_MS = 1000;
+const REFRESH_DEBOUNCE_MS = 350;
 const RETRY_MS = 5000;
 const STATE_KEY = "perfectionist-plugins:state";
 
@@ -34,7 +34,7 @@ let clientSkinOrder = [];
 function readDefaultEnabled() {
   try {
     const saved = JSON.parse(localStorage.getItem(STATE_KEY) || "{}");
-    return saved[PLUGIN_ID] !== false;
+    return saved.enabled?.[PLUGIN_ID] !== false;
   } catch {
     return true;
   }
@@ -57,6 +57,7 @@ function filterSkinList(list) {
 }
 
 function recordClientOrder(list) {
+  scheduleRefresh();
   if (Array.isArray(list)) {
     clientSkinOrder = list.map((skin) => (skin && typeof skin.id === "number" ? skin.id : null)).filter((id) => id !== null);
   }
@@ -262,7 +263,8 @@ let carouselOrder = [];
 let selectedSkinId = null;
 let failure = null;
 let requestToken = 0;
-let pollTimer = null;
+let refreshTimer = null;
+let unsubscribe = null;
 let pollInFlight = false;
 let selectInFlight = false;
 let sessionMisses = 0;
@@ -452,7 +454,7 @@ function resetState() {
   failure = null;
   sessionMisses = 0;
   championMisses = 0;
-  applyAll();
+  applyAllGuarded();
 }
 
 function loadChampion(championId) {
@@ -467,7 +469,7 @@ function loadChampion(championId) {
       if (token !== requestToken) return;
       ownedSkinIds = owned;
       carouselOrder = carouselIds.length > 0 ? carouselIds : inventoryOrder;
-      applyAll();
+      applyAllGuarded();
     },
     (error) => {
       if (token !== requestToken) return;
@@ -507,7 +509,23 @@ function updateForSession(session, selection) {
     }
   }
 
-  applyAll();
+  applyAllGuarded();
+}
+
+function scheduleRefresh() {
+  clearTimeout(refreshTimer);
+  refreshTimer = setTimeout(pollSession, REFRESH_DEBOUNCE_MS);
+}
+
+function onSessionPush(raw) {
+  const msg = raw ?? {};
+  if (msg.eventType === "Delete") {
+    resetState();
+    return;
+  }
+  const s = msg.data ?? msg;
+  if (!s?.myTeam) return;
+  updateForSession(s, null);
 }
 
 async function pollSession() {
@@ -515,11 +533,9 @@ async function pollSession() {
   ensureSourceFilter();
   pollInFlight = true;
   try {
-    const [session, selection] = await Promise.all([
-      api.lcu.get(SESSION_URI),
-      api.lcu.get(SELECTION_URI).catch(() => null),
-    ]);
+    const session = await api.lcu.get(SESSION_URI);
     sessionMisses = 0;
+    const selection = await api.lcu.get(SELECTION_URI).catch(() => null);
     updateForSession(session, selection);
   } catch {
     sessionMisses++;
@@ -534,16 +550,21 @@ function onEnable(utils) {
   filterActive = true;
   removeStyle = api.addStyle(STYLE_ID, CSS);
 
-  renderObserver = new MutationObserver(() => applyAllGuarded());
+  renderObserver = new MutationObserver(() => {
+    applyAllGuarded();
+    if (document.querySelector(SKIN_ITEM_SELECTOR)) scheduleRefresh();
+  });
   renderObserver.observe(document.documentElement ?? document, OBSERVER_OPTIONS);
 
-  pollTimer = setInterval(pollSession, SESSION_POLL_MS);
+  unsubscribe = api.subscribe(SESSION_URI, onSessionPush);
   pollSession();
 }
 
 function onDisable() {
-  clearInterval(pollTimer);
-  pollTimer = null;
+  unsubscribe?.();
+  unsubscribe = null;
+  clearTimeout(refreshTimer);
+  refreshTimer = null;
   renderObserver?.disconnect();
   removeStyle?.();
   renderObserver = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "perfectionist-plugins",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "description": "Small plugins that fix the annoying parts of League of Legends champ select. Built for Pengu Loader.",
   "type": "module",
   "main": "perfectionist-utils.js",

package/perfectionist-utils.js

@@ -33,7 +33,7 @@ const BUNDLED_MODULES = [
     id: "hide-unowned-skins",
     file: "hide-unowned-skins.js",
     name: "Hide Unowned Skins",
-    version: "2.1.0",
+    version: "2.2.0",
     description: "In champion select, only shows the skins you own for the picked champion.",
   },
 ];
@@ -113,11 +113,6 @@ async function loadState() {
     state = { installed: saved.installed, enabled: saved.enabled ?? {} };
     return;
   }
-  for (const mod of BUNDLED_MODULES) {
-    if (mod.core) continue;
-    state.installed[mod.id] = true;
-    state.enabled[mod.id] = mod.defaultEnabled ?? true;
-  }
   saveState();
 }
 
@@ -315,6 +310,7 @@ async function runUpdate(target) {
     win.setProgress(i / files.length);
     try {
       const response = await fetch(`${CDN}/${PKG}@${target}/${files[i].file}`);
+      if (response.status === 404 && files[i].id) continue;
       if (!response.ok) throw new Error(`download failed (${response.status})`);
       const text = await response.text();
       if (!text || !/export/.test(text)) throw new Error("the downloaded file looks invalid");