reviw 0.10.1 → 0.10.2

package/cli.cjs

@@ -116,7 +116,7 @@ function runGitDiff() {
   });
 }
 
-// Validate all files exist (if files specified)
+// Validate all files exist and are not directories (if files specified)
 const resolvedPaths = [];
 for (const fp of filePaths) {
   const resolved = path.resolve(fp);
@@ -124,6 +124,13 @@ for (const fp of filePaths) {
     console.error(`File not found: ${resolved}`);
     process.exit(1);
   }
+  const stat = fs.statSync(resolved);
+  if (stat.isDirectory()) {
+    console.error(`Cannot open directory: ${resolved}`);
+    console.error(`Usage: reviw <file> [file2...]`);
+    console.error(`Please specify a file, not a directory.`);
+    process.exit(1);
+  }
   resolvedPaths.push(resolved);
 }
 
@@ -477,6 +484,19 @@ function escapeHtmlChars(str) {
 }
 
 function loadData(filePath) {
+  // Check if path exists
+  if (!fs.existsSync(filePath)) {
+    throw new Error(`File not found: ${filePath}`);
+  }
+  // Check if path is a directory
+  const stat = fs.statSync(filePath);
+  if (stat.isDirectory()) {
+    throw new Error(
+      `Cannot open directory: ${filePath}\n` +
+      `Usage: reviw <file> [file2...]\n` +
+      `Please specify a file, not a directory.`
+    );
+  }
   const ext = path.extname(filePath).toLowerCase();
   if (ext === ".csv" || ext === ".tsv") {
     const data = loadCsv(filePath);
@@ -3827,6 +3847,20 @@ function htmlTemplate(dataRows, cols, title, mode, previewHtml) {
       \`;
       document.head.appendChild(style);
 
+      // Helper: strip markdown formatting from text
+      function stripMarkdown(text) {
+        return text
+          .replace(/^[-*+]\\s+/, '')           // List markers: - * +
+          .replace(/^\\d+\\.\\s+/, '')          // Numbered list: 1. 2.
+          .replace(/\\*\\*([^*]+)\\*\\*/g, '$1') // Bold: **text**
+          .replace(/\\*([^*]+)\\*/g, '$1')     // Italic: *text*
+          .replace(/__([^_]+)__/g, '$1')      // Bold: __text__
+          .replace(/_([^_]+)_/g, '$1')        // Italic: _text_
+          .replace(/\`([^\`]+)\`/g, '$1')     // Inline code: \`code\`
+          .replace(/\\[([^\\]]+)\\]\\([^)]+\\)/g, '$1') // Links: [text](url)
+          .trim();
+      }
+
       // Helper: find matching source line for text
       function findSourceLine(text) {
         if (!text) return -1;
@@ -3849,6 +3883,14 @@ function htmlTemplate(dataRows, cols, title, mode, previewHtml) {
               return i + 1;
             }
           }
+
+          // Check for markdown list items: strip list markers and formatting
+          if (lineText.match(/^[-*+]\\s|^\\d+\\.\\s/)) {
+            const strippedLine = stripMarkdown(lineText).replace(/\\s+/g, ' ').slice(0, 100);
+            if (strippedLine === normalized) return i + 1;
+            if (strippedLine.includes(normalized.slice(0, 30)) && normalized.length > 5) return i + 1;
+            if (normalized.includes(strippedLine.slice(0, 30)) && strippedLine.length > 5) return i + 1;
+          }
         }
         return -1;
       }
@@ -4286,6 +4328,10 @@ function createFileServer(filePath) {
 
     function tryListen(attemptPort, attempts = 0) {
       if (serverStarted) return; // Prevent double-start race condition
+      // Clear previous listeners from failed attempts to avoid duplicate
+      // "listening" callbacks firing after a later successful bind.
+      ctx.server.removeAllListeners("error");
+      ctx.server.removeAllListeners("listening");
       if (attempts >= MAX_PORT_ATTEMPTS) {
         console.error(
           `Could not find an available port for ${baseName} after ${MAX_PORT_ATTEMPTS} attempts.`,
@@ -4450,6 +4496,10 @@ function createDiffServer(diffContent) {
 
     function tryListen(attemptPort, attempts = 0) {
       if (serverStarted) return; // Prevent double-start race condition
+      // Clear listeners from previous failed attempts to prevent stale
+      // "listening" handlers from firing on the successful bind.
+      ctx.server.removeAllListeners("error");
+      ctx.server.removeAllListeners("listening");
       if (attempts >= MAX_PORT_ATTEMPTS) {
         console.error(
           `Could not find an available port for diff viewer after ${MAX_PORT_ATTEMPTS} attempts.`,

package/package.json

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