dsh-unlag 1.9.46 → 1.9.48

package/index.js

@@ -1,6 +1,6 @@
 import { AsyncLocalStorage } from 'node:async_hooks'
 import { createHash } from 'node:crypto'
-import { appendFileSync, existsSync, readFileSync, readdirSync, statSync, watch, writeFileSync } from 'node:fs'
+import { appendFileSync, existsSync, mkdirSync, readFileSync, readdirSync, statSync, watch, writeFileSync } from 'node:fs'
 import { homedir } from 'node:os'
 import { join, resolve } from 'node:path'
 import { Script } from 'node:vm'
@@ -222,6 +222,23 @@ export function oneLine(value, limit = 2000) {
   return text.length > limit ? `${text.slice(0, limit)}\u2026` : text
 }
 
+/** Whether the directory holding the log has already been made, so one attempt per process. */
+let logDirTried = false
+/**
+ * The log is the only evidence a failed update leaves behind, and a home directory that never ran
+ * this app yet has no directory for it: one attempt per process to make that directory, then the
+ * write is retried. Every failure stays silent, because logging must never take a press down.
+ */
+function ensureLogDir(target) {
+  if (logDirTried) return
+  logDirTried = true
+  const cut = target.lastIndexOf('/')
+  if (cut <= 0) return
+  try {
+    mkdirSync(target.slice(0, cut), { recursive: true })
+  } catch {}
+}
+
 function log(line) {
   try {
     const target = logTarget()
@@ -235,7 +252,15 @@ function log(line) {
         /* a missing or unreadable log is created by the append below */
       }
     }
-    appendFileSync(target, `${new Date().toISOString()} ${oneLine(line)}\n`)
+    const text = `${new Date().toISOString()} ${oneLine(line)}\n`
+    try {
+      appendFileSync(target, text)
+    } catch {
+      ensureLogDir(target)
+      try {
+        appendFileSync(target, text)
+      } catch {}
+    }
   } catch {}
 }
 
@@ -1760,8 +1785,14 @@ function serveButton(ctx, controller, config) {
           if (!settings.reloadPlugins) return sendJson(res, 200, { status: 'off' })
           const limit = Number(process.env.DSH_UNLAG_ANSWER_MS ?? '') || RELOAD_ANSWER_MS
           let timer = null
+          let timedOut = false
           const stillApplying = new Promise((resolve) => {
-            timer = setTimeout(() => resolve({ status: 'applying', warnings: [], pending: [], restartReason: null, failedEntries: [] }), limit)
+            timer = setTimeout(() => {
+              // Only the timer's own answer means the update outlasted the page: an update that
+              // says "applying" itself is a press that arrived during another one, not a slow one.
+              timedOut = true
+              resolve({ status: 'applying', warnings: [], pending: [], restartReason: null, failedEntries: [] })
+            }, limit)
           })
           let answeredEarly = false
           try {
@@ -1782,7 +1813,8 @@ function serveButton(ctx, controller, config) {
             const outcome = await Promise.race([work, stillApplying])
             if (outcome?.status === 'applying') {
               answeredEarly = true
-              log('the update outlasted the answer the page waits for: the press is told it is still applying')
+              if (timedOut) log('the update outlasted the answer the page waits for: the press is told it is still applying')
+              else log('a press arrived while an update is already applying: the press is told it is still applying')
             }
             sendJson(res, 200, outcome)
           } finally {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-unlag",
-  "version": "1.9.46",
+  "version": "1.9.48",
   "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",