claude-code-hub 1.3.3 → 1.3.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-hub",
-  "version": "1.3.3",
+  "version": "1.3.4",
   "description": "Unified hub for Claude Code tools — Marketplace + Kanban in one PWA",
   "main": "server.js",
   "bin": {

package/public/app.js

@@ -236,7 +236,6 @@ async function init() {
   bindPalette();
   registerSW();
   bindErrorToast();
-  bindSlowToast();
   showUpdateToast();
 }
 
@@ -293,42 +292,6 @@ function hideToast(toast) {
   }, 200);
 }
 
-// The stalls belong to the port, so a dismissal holds while the app stays on it.
-const SLOW_POLL_MS = 30_000;
-const slowDismissed = new Map();
-const slowToast = { id: null, port: 0, confirm: false, busy: false };
-
-function hideSlowToast() {
-  slowToast.id = null;
-  hideToast(document.getElementById('slow-toast'));
-}
-
-function renderSlowToast(app) {
-  const restart = document.getElementById('slow-restart');
-  document.getElementById('slow-title').textContent = `${app.name} answers slowly`;
-  document.getElementById('slow-detail').textContent =
-    `Its port delays requests by up to ${(app.worstMs / 1000).toFixed(1)} s. A restart moves it to a new port.`;
-  restart.textContent = 'Restart';
-  restart.disabled = false;
-  slowToast.id = app.id;
-  slowToast.port = app.port;
-  slowToast.confirm = false;
-  showToast(document.getElementById('slow-toast'));
-}
-
-async function pollSlowApps() {
-  if (document.hidden || slowToast.busy) return;
-  let slow;
-  try {
-    ({ slow } = await sendJson('GET', '/api/apps/health'));
-  } catch {
-    return;
-  }
-  if (slowToast.id && !slow.some((a) => a.id === slowToast.id)) hideSlowToast();
-  const next = slow.find((a) => slowDismissed.get(a.id) !== a.port);
-  if (next && next.id !== slowToast.id) renderSlowToast(next);
-}
-
 // Null on success. A 409 with live terminals asks for force: true on the next call.
 async function requestRestart(id, force) {
   try {
@@ -343,37 +306,6 @@ async function requestRestart(id, force) {
   }
 }
 
-async function restartSlowApp() {
-  const { id } = slowToast;
-  if (!id) return;
-  const restart = document.getElementById('slow-restart');
-  slowToast.busy = true;
-  restart.disabled = true;
-  restart.textContent = 'Restarting…';
-  const failed = await requestRestart(id, slowToast.confirm);
-  slowToast.busy = false;
-  if (!failed) {
-    hideSlowToast();
-    replaceIframe(id);
-    return;
-  }
-  slowToast.confirm = !!failed.terminals;
-  document.getElementById('slow-detail').textContent = failed.message;
-  restart.textContent = failed.terminals ? 'Restart anyway' : 'Restart';
-  restart.disabled = false;
-}
-
-function bindSlowToast() {
-  document.getElementById('slow-restart').addEventListener('click', restartSlowApp);
-  document.getElementById('slow-close').addEventListener('click', () => {
-    if (slowToast.id) slowDismissed.set(slowToast.id, slowToast.port);
-    hideSlowToast();
-  });
-  setInterval(pollSlowApps, SLOW_POLL_MS);
-  document.addEventListener('visibilitychange', pollSlowApps);
-  pollSlowApps();
-}
-
 const ERROR_TOAST_MS = 8_000;
 let errorToastTimer = 0;
 
@@ -1048,7 +980,12 @@ const PALETTE_MODES = {
     fillRow(li, row, q) {
       li.append(appIcon(row.app), markedSpan('name', row.app.name, q));
       const s = palette.appStats?.[row.id];
-      const stats = el('span', 'stats', s ? `${Math.round(s.rss / 1048576)} MB · ${s.cpu}%` : '');
+      if (s?.slowMs) {
+        const badge = el('span', 'slow', 'slow port');
+        badge.title = `Its port delays requests by up to ${(s.slowMs / 1000).toFixed(1)} s. Restart (Ctrl+R) moves it to a new port.`;
+        li.append(badge);
+      }
+      const stats = el('span', 'stats', s?.rss ? `${Math.round(s.rss / 1048576)} MB · ${s.cpu}%` : '');
       stats.title = 'Memory and CPU of the app process. Child processes are not counted. 100% is one core.';
       li.append(stats);
       const key = row.id === activeApp ? 'active' : row.index < 9 ? `Alt+${row.index + 1}` : '';

package/public/index.html

@@ -272,6 +272,15 @@
 
     .palette-row .stats + .age { margin-left: 0; }
 
+    .palette-row .slow {
+      margin-left: 10px;
+      padding: 0 6px;
+      border-radius: 4px;
+      color: #f5a524;
+      border: 1px solid #f5a524;
+      font-size: 11px;
+    }
+
     .palette-row.literal {
       border-top: 1px solid var(--border);
       color: var(--accent);
@@ -385,7 +394,6 @@
 
     .update-toast-close:hover { color: var(--text); }
     .error-toast { z-index: 25; border-left-color: #e5484d; }
-    .slow-toast { z-index: 20; border-left-color: #f5a524; }
   </style>
 </head>
 <body>
@@ -458,17 +466,6 @@
     <button class="update-toast-close" id="update-close" type="button" aria-label="Close">×</button>
   </div>
 
-  <div class="update-toast slow-toast" id="slow-toast" role="status" hidden>
-    <div class="update-toast-body">
-      <div class="update-toast-title" id="slow-title"></div>
-      <div class="update-toast-meta" id="slow-detail"></div>
-      <div class="update-toast-actions">
-        <button class="update-toast-skip" id="slow-restart" type="button">Restart</button>
-      </div>
-    </div>
-    <button class="update-toast-close" id="slow-close" type="button" aria-label="Close">×</button>
-  </div>
-
   <div class="update-toast error-toast" id="error-toast" role="alert" hidden>
     <div class="update-toast-body">
       <div class="update-toast-title" id="error-title"></div>

package/server.js

@@ -953,16 +953,6 @@ app.post('/api/config-dirs/activate', async (req, res) => {
   res.json({ apps: appsConfig() });
 });
 
-const STATS_TIMEOUT_MS = 1000;
-app.get('/api/apps/stats', async (_req, res) => {
-  res.setHeader('Cache-Control', 'no-store');
-  const pool = activePool();
-  const entries = await Promise.all(
-    ENABLED_APPS.map(async (a) => [a.id, await stats(pool?.children.get(a.id), STATS_TIMEOUT_MS)]),
-  );
-  res.json(Object.fromEntries(entries));
-});
-
 // The stalls hit a few percent of requests, so each round sends several probes per app.
 const HEALTH_INTERVAL_MS = 30_000;
 const HEALTH_PROBES = 5;
@@ -1000,15 +990,18 @@ async function checkPortHealth() {
   }
 }
 
-app.get('/api/apps/health', (_req, res) => {
+const STATS_TIMEOUT_MS = 1000;
+app.get('/api/apps/stats', async (_req, res) => {
   res.setHeader('Cache-Control', 'no-store');
   const pool = activePool();
-  const slow = [];
-  for (const a of ENABLED_APPS) {
-    const f = pool?.children.has(a.id) && portHealth.flagged(pool.children.get(a.id));
-    if (f) slow.push({ id: a.id, name: a.name, port: pool.ports[a.id], worstMs: f.worstMs });
-  }
-  res.json({ slow });
+  const entries = await Promise.all(
+    ENABLED_APPS.map(async (a) => {
+      const child = pool?.children.get(a.id);
+      const slowMs = (child && portHealth.flagged(child)?.worstMs) ?? null;
+      return [a.id, { ...(await stats(child, STATS_TIMEOUT_MS)), slowMs }];
+    }),
+  );
+  res.json(Object.fromEntries(entries));
 });
 
 // Restarting the terminal provider ends its live terminals, so that takes force: true.