dsh-unlag 1.9.57 → 1.9.59

package/index.js

@@ -64,7 +64,6 @@ const CODE_STAMP = (() => {
     return 'unknown'
   }
 })()
-let logWrites = 0
 /** Tests point this at a scratch file, so a test run leaves the live log readable. */
 const logTarget = () => process.env.DSH_UNLAG_LOG ?? LOG_PATH
 /** Survives a module replacement, so a reload cannot leave a second watcher behind. */
@@ -261,17 +260,17 @@ function ensureLogDir(target) {
 function log(line) {
   try {
     const target = logTarget()
-    // A file that only grows eventually costs the app its own memory budget: every 200 lines the
-    // size is checked once, and a log past the cap starts over instead of growing without end.
-    logWrites += 1
-    if (logWrites % 200 === 1) {
-      try {
-        if (statSync(target).size > LOG_MAX_BYTES) writeFileSync(target, '')
-      } catch {
-        /* a missing or unreadable log is created by the append below */
-      }
-    }
     const text = `${new Date().toISOString()} ${oneLine(line)}\n`
+    /**
+     * A file that only grows eventually costs the app its own memory budget, and a burst of lines
+     * can land between two periodic checks: the size is read before every write and the whole
+     * line has to fit, so the file never passes the cap even for one event.
+     */
+    try {
+      if (statSync(target).size + Buffer.byteLength(text) > LOG_MAX_BYTES) writeFileSync(target, '')
+    } catch {
+      /* a missing or unreadable log is created by the append below */
+    }
     try {
       appendFileSync(target, text)
     } catch {
@@ -1270,7 +1269,12 @@ function serveBundlesAtCurrentRevision(ctx) {
        * from. Only when it has nothing is the same request tried at the revision of the plugin it
        * names, because a chunk is served at its entry's revision and nowhere else.
        */
-      const url = String(request?.url ?? '')
+      // Reading the address is the first thing that runs, and a request object that cannot answer it
+      // would reject this async handler before any guard below: it is read as no address instead.
+      let url = ''
+      try {
+        url = String(request?.url ?? '')
+      } catch {}
       // Same reason as the handler shape below: a rejection here is unhandled and ends the process.
       let retargeted = null
       try {
@@ -1316,7 +1320,12 @@ function serveBundlesAtCurrentRevision(ctx) {
    * first, so the served document and the chunks it later asks for are one generation.
    */
   const wrapped = async (req, res) => {
-    const originalUrl = String(req?.url ?? '')
+    // Same as the resource shape: an address that cannot be read is no address. Throwing here would
+    // reject the handler the server is awaiting, and a rejection nobody catches ends the process.
+    let originalUrl = ''
+    try {
+      originalUrl = String(req?.url ?? '')
+    } catch {}
     // Reading the table can throw while the plugin set is being swapped. This runs inside an async
     // route handler, where a rejection is an unhandled one and takes the whole app with it, so the
     // failure has to end here: answer the request as the table left it and say why in the log.

package/package.json

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