@youtyan/code-viewer 0.5.0 → 0.5.1

package/dist/code-viewer.js

@@ -146,14 +146,22 @@ function createJsonFileStore(options) {
   async function load(root) {
     const pendingWrite = queues.get(options.filePath(root));
     if (pendingWrite)
-      await pendingWrite.catch(() => {});
+      await pendingWrite.catch(() => {
+        return;
+      });
     return loadUnqueued(root);
   }
   async function save(root, state) {
     const file = options.filePath(root);
     const previous = queues.get(file) ?? Promise.resolve();
-    const run = previous.catch(() => {}).then(() => saveUnqueued(root, state));
-    const queued = run.then(() => {}, () => {});
+    const run = previous.catch(() => {
+      return;
+    }).then(() => saveUnqueued(root, state));
+    const queued = run.then(() => {
+      return;
+    }, () => {
+      return;
+    });
     queues.set(file, queued);
     try {
       await run;
@@ -165,13 +173,19 @@ function createJsonFileStore(options) {
   async function update(root, updater) {
     const file = options.filePath(root);
     const previous = queues.get(file) ?? Promise.resolve();
-    const run = previous.catch(() => {}).then(async () => {
+    const run = previous.catch(() => {
+      return;
+    }).then(async () => {
       const current = await loadUnqueued(root);
       const updated = await updater(current);
       await saveUnqueued(root, updated.state);
       return updated.result;
     });
-    const queued = run.then(() => {}, () => {});
+    const queued = run.then(() => {
+      return;
+    }, () => {
+      return;
+    });
     queues.set(file, queued);
     try {
       return await run;
@@ -7047,7 +7061,9 @@ function createTableMetaCache(now = () => Date.now()) {
   };
 }
 function observeBackgroundRejection(promise) {
-  promise.catch(() => {});
+  promise.catch(() => {
+    return;
+  });
   return promise;
 }
 function createDockerAdapter(config) {
@@ -8021,7 +8037,9 @@ async function getConnection(resolvedPath) {
     adapter,
     path: resolvedPath,
     lastUsed: Date.now(),
-    timer: setTimeout(() => {}, 0)
+    timer: setTimeout(() => {
+      return;
+    }, 0)
   };
   pool.set(resolvedPath, entry);
   scheduleEviction(resolvedPath, entry);
@@ -8574,13 +8592,60 @@ function createDockerAdapterCache(maxEntries = DEFAULT_MAX_DOCKER_ADAPTER_CACHE,
     }
   };
 }
+async function extractErrorReason(res) {
+  if (res.status < 400)
+    return "";
+  const ctype = res.headers.get("content-type") ?? "";
+  if (!ctype.startsWith("text/") && !ctype.includes("json"))
+    return "";
+  let cloned;
+  try {
+    cloned = res.clone();
+  } catch {
+    return "";
+  }
+  try {
+    const body = await cloned.text();
+    if (!body)
+      return "";
+    const trimmed = body.replace(/\s+/g, " ").trim();
+    if (!trimmed)
+      return "";
+    return trimmed.length > MAX_LOGGED_ERROR_BODY ? `${trimmed.slice(0, MAX_LOGGED_ERROR_BODY)}...` : trimmed;
+  } catch {
+    return "";
+  }
+}
+function enqueueLogLine(work) {
+  logQueue = logQueue.then(work, work);
+}
+function logResponseWithReason(prefix, req, url, res, startMs, opts = {}) {
+  const ms = Date.now() - startMs;
+  const qsLen = opts.qsLen ?? 0;
+  const qs = qsLen > 0 && url.search ? url.search.slice(0, qsLen) : "";
+  const head = `${prefix} ${req.method} ${url.pathname}${qs} ${res.status} ${ms}ms`;
+  if (res.status < 400) {
+    enqueueLogLine(async () => {
+      console.log(head);
+    });
+    return;
+  }
+  let cloned = null;
+  try {
+    cloned = res.clone();
+  } catch {}
+  enqueueLogLine(async () => {
+    const reason = cloned ? await extractErrorReason(cloned) : "";
+    if (reason)
+      console.warn(`${head} :: ${reason}`);
+    else
+      console.warn(head);
+  });
+}
 function createQueryStrippedLogger(prefix, req, url) {
-  const path = url.pathname;
   const start = Date.now();
-  const method = req.method;
   return (res) => {
-    const ms = Date.now() - start;
-    console.log(`[code-viewer] ${prefix} ${method} ${path} ${res.status} ${ms}ms`);
+    logResponseWithReason(`[code-viewer] ${prefix}`, req, url, res, start);
     return res;
   };
 }
@@ -8734,11 +8799,12 @@ function handleError(prefix, action, err) {
   }
   return textError(`failed to ${action}: ${message}`, 500);
 }
-var DEFAULT_MAX_DOCKER_ADAPTER_CACHE = 8, DEFAULT_DOCKER_ADAPTER_IDLE_MS;
+var DEFAULT_MAX_DOCKER_ADAPTER_CACHE = 8, DEFAULT_DOCKER_ADAPTER_IDLE_MS, MAX_LOGGED_ERROR_BODY = 500, logQueue;
 var init_handle_shared = __esm(() => {
   init_docker_utils();
   init_discovery();
   DEFAULT_DOCKER_ADAPTER_IDLE_MS = 5 * 60 * 1000;
+  logQueue = Promise.resolve();
 });
 
 // web-src/server/database/handle-elasticsearch.ts
@@ -9943,7 +10009,9 @@ function guardedS3Transport(signal, operation, deadline) {
   let timedOut = false;
   let settled = false;
   let timer;
-  let cleanupParent = () => {};
+  let cleanupParent = () => {
+    return;
+  };
   const abort = (err, reject) => {
     if (settled)
       return;
@@ -9988,7 +10056,9 @@ function guardedS3Transport(signal, operation, deadline) {
 async function readStreamChunkWithTimeout(reader, signal, deadline) {
   return guardedS3Transport(signal, (transportSignal) => {
     const cancelRead = () => {
-      reader.cancel(transportSignal.reason).catch(() => {});
+      reader.cancel(transportSignal.reason).catch(() => {
+        return;
+      });
     };
     if (transportSignal.aborted) {
       cancelRead();
@@ -12643,16 +12713,12 @@ async function handleDatabaseRoute(req, url, cwd, omitDirNames, sideEffectAllowe
     const { handleS3Route: handleS3Route2 } = await Promise.resolve().then(() => (init_handle_s3(), exports_handle_s3));
     return handleS3Route2(req, url, cwd, sideEffectAllowed, omitDirNames);
   }
-  const path = url.pathname;
   const start = Date.now();
   const method = req.method;
-  const qs = url.search ? url.search.slice(0, 120) : "";
-  const log = (status) => {
-    const ms = Date.now() - start;
-    console.log(`[code-viewer] ${method} ${path}${qs} ${status} ${ms}ms`);
-  };
   const wrapResponse = (res) => {
-    log(res.status);
+    logResponseWithReason("[code-viewer]", req, url, res, start, {
+      qsLen: 120
+    });
     return res;
   };
   return dispatchRoutes(req, url, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@youtyan/code-viewer",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Local browser-based code and git diff viewer",
   "type": "module",
   "bin": {

package/web/app.js

@@ -871,7 +871,9 @@
         emit();
         const item = deps.items()[clamped];
         if (item)
-          deps.jump(item.entryId).catch(() => {});
+          deps.jump(item.entryId).catch(() => {
+            return;
+          });
       }
     }
     function jumpTo(at) {
@@ -10922,7 +10924,9 @@ ${frontmatter.yaml}
           setTimeout(() => {
             copyBtn.textContent = text2().er.copyMermaid;
           }, 1500);
-        }, () => {});
+        }, () => {
+          return;
+        });
       }
     });
     let dragState = null;
@@ -11774,7 +11778,9 @@ ${frontmatter.yaml}
           setTimeout(() => {
             copyBtn.textContent = text2().history.copySql;
           }, 1500);
-        }, () => {});
+        }, () => {
+          return;
+        });
       });
       const deleteBtn = document.createElement("button");
       deleteBtn.className = "db-btn db-query-history-danger";
@@ -12624,7 +12630,9 @@ ${frontmatter.yaml}
             keyList.innerHTML = "";
             keyRowsByName.clear();
             setPaneEmpty(mainPane, text2().redis.selectKey);
-            const valuePromise = initial.key ? selectKey(initial.key).catch(() => {}) : null;
+            const valuePromise = initial.key ? selectKey(initial.key).catch(() => {
+              return;
+            }) : null;
             await loadKeys(false);
             if (currentDbId !== dbId)
               return;
@@ -14430,7 +14438,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
             setTimeout(() => {
               copyBtn.textContent = text2().sessionLog.copy;
             }, 1200);
-          }, () => {});
+          }, () => {
+            return;
+          });
         });
         actions.append(useBtn, copyBtn);
         detailCol.appendChild(actions);
@@ -18576,7 +18586,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         if (!res.ok)
           return;
         applyDbUiState(await res.json());
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
       return dbUiLoadPromise;
     }
     function getColumnWidths(dbId, table2) {
@@ -18594,7 +18606,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         method: "PATCH",
         headers: actionHeaders(),
         body: JSON.stringify({ columnWidths: { [dbId]: { [table2]: widths } } })
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     function getExpandedTables(scopeKey) {
       return [...dbUiState.expandedTables?.[scopeKey] || []];
@@ -18626,7 +18640,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
             [scopeKey]: nextTables.length > 0 ? nextTables : null
           }
         })
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     const dbUiPrefListeners = new Set;
     function applyDbUiState(next) {
@@ -18645,7 +18661,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         method: "PATCH",
         headers: actionHeaders(),
         body: JSON.stringify({ prefs: { [key]: value } })
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     function onDbUiPrefChange(listener) {
       dbUiPrefListeners.add(listener);
@@ -18758,7 +18776,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         }
         return;
       }
-      saveChain = saveChain.catch(() => {}).then(async () => {
+      saveChain = saveChain.catch(() => {
+        return;
+      }).then(async () => {
         const controller = new AbortController;
         saveController = controller;
         try {
@@ -18833,7 +18853,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         syncActiveRoute();
       scheduleSave();
       if (mounted && !isRestoring()) {
-        ensureInitialEnter(id)?.catch(() => {});
+        ensureInitialEnter(id)?.catch(() => {
+          return;
+        });
       }
     }
     function syncActiveRoute() {
@@ -18896,7 +18918,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
         "Content-Type": "application/json",
         "X-Code-Viewer-Action": "1"
       };
-      fetch("/_db/close", { method: "POST", headers, body }).catch(() => {});
+      fetch("/_db/close", { method: "POST", headers, body }).catch(() => {
+        return;
+      });
     }
     function clearDropTarget() {
       if (!dropTargetId)
@@ -19406,7 +19430,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
     }
     async function enter(db, schema, table2, view, options = {}) {
       const seq = lifecycleSeq;
-      enterQueue = enterQueue.catch(() => {}).then(() => doEnter(seq, db, schema, table2, view, options));
+      enterQueue = enterQueue.catch(() => {
+        return;
+      }).then(() => doEnter(seq, db, schema, table2, view, options));
       await enterQueue;
     }
     function leave() {
@@ -21716,7 +21742,9 @@ ${entry.message}` : entry.detail ?? entry.message ?? "";
       const force = options.force === true;
       const mount = options.mount || (force ? activeMount : defaultMount);
       activateMount(mount);
-      entering = entering.then(() => doEnterHistory(force)).catch(() => {});
+      entering = entering.then(() => doEnterHistory(force)).catch(() => {
+        return;
+      });
       return entering;
     }
     async function doEnterHistory(force = false) {
@@ -23627,7 +23655,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
         if (!GdpExpandLogic.shouldAttachTrailingExpand(data?.lines?.length || 0))
           return;
         attachTrailingExpandControls(item, file, ref, refPath);
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     function insertContextRows(targetTr, lines, newStart, oldStart, dir, sideIndex) {
       const tbody = targetTr.parentElement;
@@ -23766,7 +23796,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
     function fetchRefs() {
       return fetch("/_refs").then((r2) => r2.json()).then((refs) => {
         Object.assign(REFS, refs);
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     fetchRefs();
     let popTab = "commits";
@@ -28822,7 +28854,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
         headers: actionHeaders(),
         body,
         keepalive: options.keepalive
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     let pendingViewPatch = null;
     let pendingViewTimer = null;
@@ -28895,7 +28929,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
           headers: actionHeaders(),
           body,
           keepalive
-        }).catch(() => {});
+        }).catch(() => {
+          return;
+        });
       };
       if (options.keepalive) {
         if (pendingViewTimer !== null)
@@ -28931,7 +28967,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
         headers: actionHeaders(),
         body,
         keepalive
-      }).catch(() => {});
+      }).catch(() => {
+        return;
+      });
     }
     function savedScopeOmitDirs() {
       return APP_SETTINGS.scopeOmitDirs ? normalizeScopeOmitDirs(APP_SETTINGS.scopeOmitDirs) : null;
@@ -30997,7 +31035,9 @@ code-viewer query diff rows --id diff-xyz789 --table users --type inserted --lim
         STATE.to = range.to;
         activeHistoryPathFilter = pathFilter || null;
         syncRefInputs();
-        return load().then(() => {});
+        return load().then(() => {
+          return;
+        });
       },
       showEmptyDiffPane: () => {
         if (activeFileHistoryDiffHost || activeFileHistoryEmptyHost) {