reviw 0.15.2 → 0.16.1

package/README.md

@@ -51,6 +51,9 @@ A lightweight browser-based tool for reviewing and annotating tabular data, text
 - **Real-time updates**: Hot reload on file changes via SSE
 - **Comment persistence**: Auto-save comments to localStorage with recovery modal
 - **Keyboard shortcuts**: Cmd/Ctrl+Enter to open submit modal
+- **Multi-tab sync**: Submit from any tab closes all tabs for the same file
+- **Server detection**: Reuse existing server instead of starting a new one (via lock files)
+- **Tab activation (macOS)**: Automatically activates existing browser tab via AppleScript
 
 ### Output
 - YAML format with file, mode, row, col, value, and comment text

package/cli.cjs

@@ -15,7 +15,8 @@ const fs = require("fs");
 const http = require("http");
 const os = require("os");
 const path = require("path");
-const { spawn } = require("child_process");
+const crypto = require("crypto");
+const { spawn, execSync, spawnSync } = require("child_process");
 const chardet = require("chardet");
 const iconv = require("iconv-lite");
 const marked = require("marked");
@@ -1764,7 +1765,17 @@ function diffHtmlTemplate(diffData) {
       let es = null;
       const connect = () => {
         es = new EventSource('/sse');
-        es.onmessage = ev => { if (ev.data === 'reload') location.reload(); };
+        es.onmessage = ev => {
+          if (ev.data === 'reload') location.reload();
+          if (ev.data === 'submitted') {
+            // Another tab submitted - try to close this tab
+            window.close();
+            // If window.close() didn't work, show completion message
+            setTimeout(() => {
+              document.body.innerHTML = '<div style="display:flex;flex-direction:column;align-items:center;justify-content:center;height:100vh;background:var(--bg,#1a1a2e);color:var(--text,#e0e0e0);font-family:system-ui,sans-serif;"><h1 style="font-size:2rem;margin-bottom:1rem;">✅ Review Submitted</h1><p style="color:var(--muted,#888);">Submitted from another tab. You can close this tab now.</p></div>';
+            }, 100);
+          }
+        };
         es.onerror = () => { es.close(); setTimeout(connect, 1500); };
       };
       connect();
@@ -2252,10 +2263,10 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
     .md-preview video.video-preview { max-width: 100%; height: auto; border-radius: 8px; background: #000; }
     .md-preview table video.video-preview {
       display: block;
-      width: auto;
+      width: 100%;
       height: auto;
-      max-width: 200px;
-      max-height: 300px;
+      min-width: 120px;
+      max-width: 250px;
     }
     .md-preview code { background: rgba(255,255,255,0.08); padding: 2px 4px; border-radius: 4px; }
     .md-preview pre {
@@ -2383,9 +2394,9 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
     }
     /* Markdown tables in preview */
     .md-preview table:not(.frontmatter-table table) {
-      width: max-content;
+      width: 100%;
       border-collapse: collapse;
-      table-layout: auto;
+      table-layout: fixed;
       margin: 16px 0;
       border: 1px solid var(--border);
       border-radius: 8px;
@@ -2396,15 +2407,31 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       text-align: left;
       border-bottom: 1px solid var(--border);
       vertical-align: top;
-      white-space: nowrap;
+      word-break: break-word;
+      overflow-wrap: anywhere;
+    }
+    .md-preview table:not(.frontmatter-table table) th:first-child,
+    .md-preview table:not(.frontmatter-table table) td:first-child {
+      width: 30%;
+      min-width: 200px;
+    }
+    .md-preview table:not(.frontmatter-table table) th:last-child,
+    .md-preview table:not(.frontmatter-table table) td:last-child {
+      width: 180px;
+      min-width: 180px;
+      max-width: 180px;
     }
     .md-preview table:not(.frontmatter-table table) td:has(video),
     .md-preview table:not(.frontmatter-table table) td:has(img) {
-      padding: 0;
-      width: 1%;
-      white-space: nowrap;
+      padding: 4px;
       line-height: 0;
     }
+    .md-preview table:not(.frontmatter-table table) td video,
+    .md-preview table:not(.frontmatter-table table) td img {
+      width: 100%;
+      max-width: 100%;
+      height: auto;
+    }
     .md-preview table:not(.frontmatter-table table) th {
       background: var(--panel);
       font-weight: 600;
@@ -3363,6 +3390,14 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           if (ev.data === 'reload') {
             location.reload();
           }
+          if (ev.data === 'submitted') {
+            // Another tab submitted - try to close this tab
+            window.close();
+            // If window.close() didn't work, show completion message
+            setTimeout(() => {
+              document.body.innerHTML = '<div style="display:flex;flex-direction:column;align-items:center;justify-content:center;height:100vh;background:var(--bg,#1a1a2e);color:var(--text,#e0e0e0);font-family:system-ui,sans-serif;"><h1 style="font-size:2rem;margin-bottom:1rem;">✅ Review Submitted</h1><p style="color:var(--muted,#888);">Submitted from another tab. You can close this tab now.</p></div>';
+            }, 100);
+          }
         };
         es.onerror = () => {
           es.close();
@@ -5613,6 +5648,229 @@ function readBody(req) {
 const MAX_PORT_ATTEMPTS = 100;
 const activeServers = new Map();
 
+// --- Lock File Management (for detecting existing servers) ---
+const LOCK_DIR = path.join(os.homedir(), '.reviw', 'locks');
+
+function getLockFilePath(filePath) {
+  // Use SHA256 hash of absolute path to prevent path traversal attacks
+  const hash = crypto.createHash('sha256').update(path.resolve(filePath)).digest('hex').slice(0, 16);
+  return path.join(LOCK_DIR, hash + '.lock');
+}
+
+function ensureLockDir() {
+  try {
+    if (!fs.existsSync(LOCK_DIR)) {
+      fs.mkdirSync(LOCK_DIR, { recursive: true, mode: 0o700 });
+    }
+  } catch (err) {
+    // Ignore errors - locks are optional optimization
+  }
+}
+
+function writeLockFile(filePath, port) {
+  try {
+    ensureLockDir();
+    const lockPath = getLockFilePath(filePath);
+    const lockData = {
+      pid: process.pid,
+      port: port,
+      file: path.resolve(filePath),
+      created: Date.now()
+    };
+    fs.writeFileSync(lockPath, JSON.stringify(lockData), { mode: 0o600 });
+  } catch (err) {
+    // Ignore errors - locks are optional
+  }
+}
+
+function removeLockFile(filePath) {
+  try {
+    const lockPath = getLockFilePath(filePath);
+    if (fs.existsSync(lockPath)) {
+      fs.unlinkSync(lockPath);
+    }
+  } catch (err) {
+    // Ignore errors
+  }
+}
+
+function checkExistingServer(filePath) {
+  try {
+    const lockPath = getLockFilePath(filePath);
+    if (!fs.existsSync(lockPath)) {
+      return null;
+    }
+
+    const lockData = JSON.parse(fs.readFileSync(lockPath, 'utf8'));
+
+    // Verify the process is still alive
+    try {
+      process.kill(lockData.pid, 0); // Signal 0 just checks if process exists
+    } catch (err) {
+      // Process doesn't exist - stale lock
+      fs.unlinkSync(lockPath);
+      return null;
+    }
+
+    // Verify the server is actually responding
+    return new Promise((resolve) => {
+      const req = http.request({
+        hostname: 'localhost',
+        port: lockData.port,
+        path: '/healthz',
+        method: 'GET',
+        timeout: 1000
+      }, (res) => {
+        if (res.statusCode === 200) {
+          resolve(lockData);
+        } else {
+          // Server not healthy - remove stale lock
+          try { fs.unlinkSync(lockPath); } catch (e) {}
+          resolve(null);
+        }
+      });
+      req.on('error', () => {
+        // Server not responding - remove stale lock
+        try { fs.unlinkSync(lockPath); } catch (e) {}
+        resolve(null);
+      });
+      req.on('timeout', () => {
+        req.destroy();
+        try { fs.unlinkSync(lockPath); } catch (e) {}
+        resolve(null);
+      });
+      req.end();
+    });
+  } catch (err) {
+    return null;
+  }
+}
+
+// Try to activate an existing browser tab with the given URL (macOS only)
+// Returns true if a tab was activated, false otherwise
+function tryActivateExistingTab(url) {
+  if (process.platform !== "darwin") {
+    return false;
+  }
+
+  try {
+    // AppleScript to find and activate Chrome tab with URL
+    // Uses "found" variable instead of return (AppleScript doesn't allow return at top level)
+    const chromeScript = [
+      'set found to false',
+      'tell application "System Events"',
+      '  if exists process "Google Chrome" then',
+      '    tell application "Google Chrome"',
+      '      set targetUrl to "' + url + '"',
+      '      repeat with w in windows',
+      '        set tabIndex to 1',
+      '        repeat with t in tabs of w',
+      '          if URL of t starts with targetUrl then',
+      '            set active tab index of w to tabIndex',
+      '            set index of w to 1',
+      '            activate',
+      '            set found to true',
+      '            exit repeat',
+      '          end if',
+      '          set tabIndex to tabIndex + 1',
+      '        end repeat',
+      '        if found then exit repeat',
+      '      end repeat',
+      '    end tell',
+      '  end if',
+      'end tell',
+      'found'
+    ].join('\n');
+
+    const chromeResult = spawnSync('osascript', ['-e', chromeScript], {
+      encoding: "utf8",
+      timeout: 3000
+    });
+
+    if (chromeResult.stdout && chromeResult.stdout.trim() === "true") {
+      console.log("Activated existing Chrome tab: " + url);
+      return true;
+    }
+
+    // Try Safari as fallback
+    const safariScript = [
+      'set found to false',
+      'tell application "System Events"',
+      '  if exists process "Safari" then',
+      '    tell application "Safari"',
+      '      set targetUrl to "' + url + '"',
+      '      repeat with w in windows',
+      '        repeat with t in tabs of w',
+      '          if URL of t starts with targetUrl then',
+      '            set current tab of w to t',
+      '            set index of w to 1',
+      '            activate',
+      '            set found to true',
+      '            exit repeat',
+      '          end if',
+      '        end repeat',
+      '        if found then exit repeat',
+      '      end repeat',
+      '    end tell',
+      '  end if',
+      'end tell',
+      'found'
+    ].join('\n');
+
+    const safariResult = spawnSync('osascript', ['-e', safariScript], {
+      encoding: "utf8",
+      timeout: 3000
+    });
+
+    if (safariResult.stdout && safariResult.stdout.trim() === "true") {
+      console.log("Activated existing Safari tab: " + url);
+      return true;
+    }
+
+    return false;
+  } catch (err) {
+    // AppleScript failed (not macOS, Chrome/Safari not installed, etc.)
+    return false;
+  }
+}
+
+// Open browser with the given URL, trying to reuse existing tab first (macOS)
+function openBrowser(url, delay = 0) {
+  const opener =
+    process.platform === "darwin"
+      ? "open"
+      : process.platform === "win32"
+        ? "start"
+        : "xdg-open";
+
+  setTimeout(function() {
+    // On macOS, try to activate existing tab first
+    if (process.platform === "darwin") {
+      var activated = tryActivateExistingTab(url);
+      if (activated) {
+        return;  // Successfully activated existing tab
+      }
+      // If activation failed, fall through to open new tab
+    }
+
+    try {
+      var child = spawn(opener, [url], { stdio: "ignore", detached: true });
+      child.on('error', function(err) {
+        console.warn(
+          "Failed to open browser automatically. Please open this URL manually:",
+          url
+        );
+      });
+      child.unref();
+    } catch (err) {
+      console.warn(
+        "Failed to open browser automatically. Please open this URL manually:",
+        url
+      );
+    }
+  }, delay);
+}
+
 function outputAllResults() {
   console.log("=== All comments received ===");
   if (allResults.length === 1) {
@@ -5724,6 +5982,7 @@ function createFileServer(filePath, fileIndex = 0) {
         ctx.server = null;
       }
       activeServers.delete(filePath);
+      removeLockFile(filePath);  // Clean up lock file
       if (result) allResults.push(result);
       serversRunning--;
       console.log(`Server for ${baseName} closed. (${serversRunning} remaining)`);
@@ -5764,12 +6023,15 @@ function createFileServer(filePath, fileIndex = 0) {
           }
           res.writeHead(200, { "Content-Type": "text/plain" });
           res.end("bye");
-          shutdownServer(payload);
+          // Notify all tabs to close before shutting down
+          broadcast("submitted");
+          setTimeout(() => shutdownServer(payload), 300);
         } catch (err) {
           console.error("payload parse error", err);
           res.writeHead(400, { "Content-Type": "text/plain" });
           res.end("bad request");
-          shutdownServer(null);
+          broadcast("submitted");
+          setTimeout(() => shutdownServer(null), 300);
         }
         return;
       }
@@ -5931,33 +6193,12 @@ function createFileServer(filePath, fileIndex = 0) {
         nextPort = attemptPort + 1;
         activeServers.set(filePath, ctx);
         console.log(`Viewer started: http://localhost:${attemptPort}  (file: ${baseName})`);
+        writeLockFile(filePath, attemptPort);  // Write lock file for server detection
         if (!noOpen) {
           const url = `http://localhost:${attemptPort}`;
-          const opener =
-            process.platform === "darwin"
-              ? "open"
-              : process.platform === "win32"
-                ? "start"
-                : "xdg-open";
           // Add delay for multiple files to avoid browser ignoring rapid open commands
           const delay = fileIndex * 300;
-          setTimeout(() => {
-            try {
-              const child = spawn(opener, [url], { stdio: "ignore", detached: true });
-              child.on('error', (err) => {
-                console.warn(
-                  "Failed to open browser automatically. Please open this URL manually:",
-                  url,
-                );
-              });
-              child.unref();
-            } catch (err) {
-              console.warn(
-                "Failed to open browser automatically. Please open this URL manually:",
-                url,
-              );
-            }
-          }, delay);
+          openBrowser(url, delay);
         }
         startWatcher();
         resolve(ctx);
@@ -6044,12 +6285,15 @@ function createDiffServer(diffContent) {
           }
           res.writeHead(200, { "Content-Type": "text/plain" });
           res.end("bye");
-          shutdownServer(payload);
+          // Notify all tabs to close before shutting down
+          broadcast("submitted");
+          setTimeout(() => shutdownServer(payload), 300);
         } catch (err) {
           console.error("payload parse error", err);
           res.writeHead(400, { "Content-Type": "text/plain" });
           res.end("bad request");
-          shutdownServer(null);
+          broadcast("submitted");
+          setTimeout(() => shutdownServer(null), 300);
         }
         return;
       }
@@ -6111,27 +6355,7 @@ function createDiffServer(diffContent) {
         console.log(`Diff viewer started: http://localhost:${attemptPort}`);
         if (!noOpen) {
           const url = `http://localhost:${attemptPort}`;
-          const opener =
-            process.platform === "darwin"
-              ? "open"
-              : process.platform === "win32"
-                ? "start"
-                : "xdg-open";
-          try {
-            const child = spawn(opener, [url], { stdio: "ignore", detached: true });
-            child.on('error', (err) => {
-              console.warn(
-                "Failed to open browser automatically. Please open this URL manually:",
-                url,
-              );
-            });
-            child.unref();
-          } catch (err) {
-            console.warn(
-              "Failed to open browser automatically. Please open this URL manually:",
-              url,
-            );
-          }
+          openBrowser(url, 0);
         }
         resolve(ctx);
       });
@@ -6182,10 +6406,32 @@ if (require.main === module) {
     }
   } else if (resolvedPaths.length > 0) {
     // File mode: files specified
-    console.log(`Starting servers for ${resolvedPaths.length} file(s)...`);
-    serversRunning = resolvedPaths.length;
-    for (let i = 0; i < resolvedPaths.length; i++) {
-      await createFileServer(resolvedPaths[i], i);
+    // Check for existing servers first
+    let filesToStart = [];
+    for (const fp of resolvedPaths) {
+      const existing = await checkExistingServer(fp);
+      if (existing) {
+        console.log(`Server already running for ${path.basename(fp)} on port ${existing.port}`);
+        const url = `http://localhost:${existing.port}`;
+        if (!noOpen) {
+          openBrowser(url, 0);
+        }
+      } else {
+        filesToStart.push(fp);
+      }
+    }
+
+    if (filesToStart.length === 0) {
+      console.log("All files already have running servers. Activating existing tabs.");
+      // Wait a moment for browser activation to complete before exiting
+      setTimeout(() => process.exit(0), 500);
+      return;
+    }
+
+    console.log(`Starting servers for ${filesToStart.length} file(s)...`);
+    serversRunning = filesToStart.length;
+    for (let i = 0; i < filesToStart.length; i++) {
+      await createFileServer(filesToStart[i], i);
     }
     console.log("Close all browser tabs or Submit & Exit to finish.");
   } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reviw",
-  "version": "0.15.2",
+  "version": "0.16.1",
   "description": "Lightweight file reviewer with in-browser comments for CSV, TSV, Markdown, and Git diffs.",
   "type": "module",
   "bin": {