dsh-unlag 1.9.77 → 1.9.79

package/client.js

@@ -72,6 +72,14 @@ window.__ModuleLoader__.load({
       }
     }
 
+    function stampClear() {
+      try {
+        window.sessionStorage.removeItem(STAMP_KEY);
+      } catch {
+        // Pacing is the only thing this stamp feeds: a store that refuses the clear costs nothing.
+      }
+    }
+
     function stampWrite() {
       try {
         window.sessionStorage.setItem(STAMP_KEY, String(Date.now()));
@@ -100,15 +108,23 @@ window.__ModuleLoader__.load({
      */
     const HOST_BASE = '/dsh-unlag'; // ui-copy-ignore
 
-    async function readSettings() {
+    async function readSettings(attempt = 0) {
+      const again = async () => {
+        if (attempt >= 1) return null;
+        await new Promise((resolve) => setTimeout(resolve, 250)); // ui-copy-ignore
+        return readSettings(attempt + 1);
+      };
       try {
+        // The answer can be missing for a moment while the plugin set is recomposed, which reads
+        // exactly like a press with nothing switched on. One more ask costs a fraction of a second
+        // and turns that into the press the person meant; a second miss is still reported as such.
         const response = await withTimeout(fetch(`${HOST_BASE}/state`, { cache: 'no-store', credentials: 'same-origin' }));
-        if (!response.ok) return null;
+        if (!response.ok) return again();
         const data = await withTimeout(response.json());
-        if (data === null || typeof data !== 'object') return null;
+        if (data === null || typeof data !== 'object') return again();
         return { plugins: data.plugins !== false, page: data.page !== false };
       } catch {
-        return null;
+        return again();
       }
     }
 
@@ -852,7 +868,6 @@ window.__ModuleLoader__.load({
             report('replaced');
             return stop(replaced ? 'replaced' : 'done');
           }
-          stampWrite();
           const reason = await prepareInterface();
           if (reason !== null) {
             // The log keeps the exact check that refused, so a held press can be explained later.
@@ -860,6 +875,9 @@ window.__ModuleLoader__.load({
             report('held', replaced ? `applied:${gate}` : gate);
             return stop(reason, replaced);
           }
+          // Written only now, when the document really is about to go: a press that was held changed
+          // nothing, so the next press must not be turned away as if this one had been answered.
+          stampWrite();
           report('rebuild');
           navigated = true;
           reloadFreshPage();
@@ -877,6 +895,9 @@ window.__ModuleLoader__.load({
               // Still here after asking for a rebuild: the page held the navigation. Say what
               // happened and take the press back, so the button is never left unresponsive.
               report('held', 'navigation');
+              // The document stayed, so no press was answered: clear the pacing stamp this press
+              // wrote, or the next press inside three seconds would be told it is already applying.
+              stampClear();
               setPending(false);
               // Only a press that really replaced the plugins may say so: a page-only press that
               // held the navigation changed no code.
@@ -918,6 +939,7 @@ window.__ModuleLoader__.load({
      */
     const PROBE = {
       prepareInterface,
+      readSettings,
       startupLooksServable,
       startupHoldReason: () => holdReason,
       noticeFor,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-unlag",
-  "version": "1.9.77",
+  "version": "1.9.79",
   "type": "module",
   "description": "A press applies plugin edits right away and brings a laggy interface back; automatic reloading stays off unless you turn it on, so the press is the only trigger, and nothing restarts.",
   "license": "MIT",