reviw 0.10.1 → 0.10.3

package/README.md

@@ -6,7 +6,7 @@ A lightweight browser-based tool for reviewing and annotating tabular data, text
 
 ### File Format Support
 - **CSV/TSV**: View tabular data with sticky headers, column freezing, filtering, and column resizing
-- **Markdown**: Side-by-side preview with synchronized scrolling
+- **Markdown**: Side-by-side preview with synchronized scrolling, click-to-comment from preview
 - **Diff/Patch**: GitHub-style diff view with syntax highlighting, collapsible large files (500+ lines), and binary files sorted to end
 - **Text**: Line-by-line commenting for plain text files
 
@@ -15,9 +15,12 @@ A lightweight browser-based tool for reviewing and annotating tabular data, text
 - Click any diagram to open fullscreen viewer
 - Zoom with mouse wheel (centered on cursor position, up to 10x)
 - Pan with mouse drag
-- Minimap showing current viewport position
 - Syntax error display in toast notifications
 
+### Media Fullscreen
+- Click images in Markdown preview to open fullscreen viewer
+- Click videos to open fullscreen playback with native controls
+
 ### UI Features
 - **Theme toggle**: Switch between light and dark modes
 - **Multi-file support**: Open multiple files simultaneously on separate ports
@@ -62,6 +65,8 @@ reviw changes.diff
 - `--port <number>`: Specify starting port (default: 4989)
 - `--encoding <encoding>`: Force specific encoding (auto-detected by default)
 - `--no-open`: Prevent automatic browser opening
+- `--help, -h`: Show help message
+- `--version, -v`: Show version number
 
 ### Workflow
 1. Browser opens automatically (macOS: `open` / Linux: `xdg-open` / Windows: `start`)
@@ -72,16 +77,16 @@ reviw changes.diff
 ## Screenshots
 
 ### CSV View
-![CSV View](./assets/screenshot-csv.png?v=2)
+![CSV View](https://raw.githubusercontent.com/kazuph/reviw/main/assets/screenshot-csv.png)
 
 ### Markdown View
-![Markdown View](./assets/screenshot-md.png?v=2)
+![Markdown View](https://raw.githubusercontent.com/kazuph/reviw/main/assets/screenshot-md.png)
 
 ### Diff View
-![Diff View](./assets/screenshot-diff.png?v=2)
+![Diff View](https://raw.githubusercontent.com/kazuph/reviw/main/assets/screenshot-diff.png)
 
 ### Mermaid Fullscreen
-![Mermaid Fullscreen](./assets/screenshot-mermaid.png?v=2)
+![Mermaid Fullscreen](https://raw.githubusercontent.com/kazuph/reviw/main/assets/screenshot-mermaid.png)
 
 ## Output Example
 

package/cli.cjs

@@ -21,6 +21,7 @@ const marked = require("marked");
 const yaml = require("js-yaml");
 
 // --- CLI arguments ---------------------------------------------------------
+const VERSION = "0.10.3";
 const args = process.argv.slice(2);
 
 const filePaths = [];
@@ -31,6 +32,49 @@ let stdinMode = false;
 let diffMode = false;
 let stdinContent = null;
 
+function showHelp() {
+  console.log(`reviw v${VERSION} - Lightweight file reviewer with in-browser comments
+
+Usage:
+  reviw <file...> [options]      Review files (CSV, TSV, Markdown, Text)
+  reviw <file.diff>              Review diff/patch file
+  git diff | reviw [options]     Review diff from stdin
+  reviw                          Auto run git diff HEAD
+
+Supported Formats:
+  CSV/TSV      Tabular data with sticky headers, filtering, column resizing
+  Markdown     Side-by-side preview with synchronized scrolling, Mermaid diagrams
+  Diff/Patch   GitHub-style view with syntax highlighting
+  Text         Line-by-line commenting
+
+Options:
+  --port <number>       Server port (default: 4989)
+  --encoding <enc>, -e  Force encoding (utf8, shift_jis, euc-jp, etc.)
+  --no-open             Don't open browser automatically
+  --help, -h            Show this help message
+  --version, -v         Show version number
+
+Examples:
+  reviw data.csv                    # Review CSV file
+  reviw README.md                   # Review Markdown with preview
+  reviw file1.csv file2.md          # Multiple files on consecutive ports
+  git diff | reviw                  # Review uncommitted changes
+  git diff HEAD~3 | reviw           # Review last 3 commits
+  reviw changes.patch               # Review patch file
+
+Workflow:
+  1. Browser opens automatically (use --no-open to disable)
+  2. Click cells/lines to add comments, drag to select multiple
+  3. Press Cmd/Ctrl+Enter or click "Submit & Exit"
+  4. Comments are output as YAML to stdout
+
+More info: https://github.com/kazuph/reviw`);
+}
+
+function showVersion() {
+  console.log(VERSION);
+}
+
 for (let i = 0; i < args.length; i += 1) {
   const arg = args[i];
   if (arg === "--port" && args[i + 1]) {
@@ -42,22 +86,10 @@ for (let i = 0; i < args.length; i += 1) {
   } else if (arg === "--no-open") {
     noOpen = true;
   } else if (arg === "--help" || arg === "-h") {
-    console.log(`Usage: reviw <file...> [options]
-       git diff | reviw [options]
-       reviw [options]  (auto runs git diff HEAD)
-
-Options:
-  --port <number>     Server port (default: 4989)
-  --encoding <enc>    Force encoding (utf8, shift_jis, etc.)
-  --no-open           Don't open browser automatically
-  --help, -h          Show this help message
-
-Examples:
-  reviw data.csv                    # View CSV file
-  reviw README.md                   # View Markdown file
-  git diff | reviw                  # Review diff from stdin
-  git diff HEAD~3 | reviw           # Review diff from last 3 commits
-  reviw                             # Auto run git diff HEAD`);
+    showHelp();
+    process.exit(0);
+  } else if (arg === "--version" || arg === "-v") {
+    showVersion();
     process.exit(0);
   } else if (!arg.startsWith("-")) {
     filePaths.push(arg);
@@ -116,7 +148,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 +156,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 +516,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 +3879,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 +3915,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 +4360,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 +4528,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.`,
@@ -4543,6 +4625,7 @@ function createDiffServer(diffContent) {
     console.log("Close all browser tabs or Submit & Exit to finish.");
   } else {
     // No files and no stdin: try auto git diff
+    console.log(`reviw v${VERSION}`);
     console.log("No files specified. Running git diff HEAD...");
     try {
       const gitDiff = await runGitDiff();
@@ -4551,7 +4634,9 @@ function createDiffServer(diffContent) {
         console.log("");
         console.log("Usage: reviw <file...> [options]");
         console.log("       git diff | reviw [options]");
-        console.log("       reviw  (auto runs git diff HEAD)");
+        console.log("       reviw              (auto runs git diff HEAD)");
+        console.log("");
+        console.log("Run 'reviw --help' for more information.");
         process.exit(0);
       }
       diffMode = true;
@@ -4565,6 +4650,8 @@ function createDiffServer(diffContent) {
       console.log("");
       console.log("Usage: reviw <file...> [options]");
       console.log("       git diff | reviw [options]");
+      console.log("");
+      console.log("Run 'reviw --help' for more information.");
       process.exit(1);
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reviw",
-  "version": "0.10.1",
+  "version": "0.10.3",
   "description": "Lightweight file reviewer with in-browser comments for CSV, TSV, Markdown, and Git diffs.",
   "type": "module",
   "bin": {
@@ -9,6 +9,10 @@
   "files": [
     "cli.cjs"
   ],
+  "scripts": {
+    "test": "vitest run",
+    "test:watch": "vitest"
+  },
   "license": "MIT",
   "author": "kazuph",
   "publishConfig": {
@@ -27,9 +31,5 @@
     "@playwright/test": "^1.57.0",
     "playwright": "^1.57.0",
     "vitest": "^4.0.14"
-  },
-  "scripts": {
-    "test": "vitest run",
-    "test:watch": "vitest"
   }
-}
+}