reviw 0.18.0 → 0.19.0

package/README.ja.md

@@ -177,7 +177,7 @@ plugin/
 
 | 種類 | 名前 | 説明 |
 |------|------|------|
-| **コマンド** | `/reviw:do` | タスク開始 - gwqでworktree作成、計画、todo登録 |
+| **コマンド** | `/reviw:do` | タスク開始 - git wtでworktree作成、計画、todo登録 |
 | **コマンド** | `/reviw:done` | 完了チェックリスト - 7レビューエージェント実行、エビデンス収集、レビュー開始 |
 | **エージェント** | `report-builder` | ユーザーレビュー用レポート準備 |
 | **エージェント** | `review-code-quality` | コード品質: 可読性、DRY、型安全性、エラーハンドリング |
@@ -201,14 +201,14 @@ plugin/
 適切な環境セットアップで新しいタスクを開始します。
 
 **処理内容:**
-1. gwqを使用して分離開発用のgit worktreeを作成（`feature/<name>`、`fix/<name>`など）
+1. git wtを使用して分離開発用のgit worktreeを作成（`feature/<name>`、`fix/<name>`など）
 2. エビデンス用の`.artifacts/<feature>/`ディレクトリをセットアップ
 3. 計画とTODOチェックリスト付きの`REPORT.md`を作成
 4. 進捗追跡用にTodoWriteにtodoを登録
 
 **作成されるディレクトリ構成:**
 ```
-<worktree>/                   # 例: ~/src/github.com/owner/myrepo-feature-auth/
+<worktree>/                   # 例: .worktree/feature-auth/
 └── .artifacts/
     └── <feature>/            # 例: auth（feature/authから）
         ├── REPORT.md         # 計画、進捗、エビデンスリンク
@@ -216,7 +216,7 @@ plugin/
         └── videos/           # 動画録画
 ```
 
-**タスク再開:** セッション開始時またはコンテキスト圧縮後、コマンドは既存のworktreeを確認（`gwq list`）し、`REPORT.md`から再開します。
+**タスク再開:** セッション開始時またはコンテキスト圧縮後、コマンドは既存のworktreeを確認（`git wt`）し、`REPORT.md`から再開します。
 
 #### `/reviw:done`
 

package/README.md

@@ -180,7 +180,7 @@ plugin/
 
 | Type | Name | Description |
 |------|------|-------------|
-| **Command** | `/reviw:do` | Start a task - create worktree with gwq, plan, register todos |
+| **Command** | `/reviw:do` | Start a task - create worktree with git wt, plan, register todos |
 | **Command** | `/reviw:done` | Complete checklist - run 7 review agents, collect evidence, start review |
 | **Agent** | `report-builder` | Prepare reports and evidence for user review |
 | **Agent** | `review-code-quality` | Code quality: readability, DRY, type safety, error handling |
@@ -204,14 +204,14 @@ plugin/
 Starts a new task with proper environment setup.
 
 **What it does:**
-1. Creates a git worktree using gwq for isolated development (`feature/<name>`, `fix/<name>`, etc.)
+1. Creates a git worktree using git wt for isolated development (`feature/<name>`, `fix/<name>`, etc.)
 2. Sets up `.artifacts/<feature>/` directory for evidence
 3. Creates `REPORT.md` with plan and TODO checklist
 4. Registers todos in TodoWrite for progress tracking
 
 **Directory structure created:**
 ```
-<worktree>/                   # e.g., ~/src/github.com/owner/myrepo-feature-auth/
+<worktree>/                   # e.g., .worktree/feature-auth/
 └── .artifacts/
     └── <feature>/            # e.g., auth (from feature/auth)
         ├── REPORT.md         # Plan, progress, evidence links
@@ -219,7 +219,7 @@ Starts a new task with proper environment setup.
         └── videos/           # Video recordings
 ```
 
-**Task resumption:** When a session starts or after context compaction, the command checks for existing worktrees (via `gwq list`) and resumes from `REPORT.md`.
+**Task resumption:** When a session starts or after context compaction, the command checks for existing worktrees (via `git wt`) and resumes from `REPORT.md`.
 
 #### `/reviw:done`
 

package/cli.cjs

@@ -37,9 +37,10 @@ const timelineTempDirs = new Set();
 
 // Extract video timeline thumbnails using ffmpeg scene detection
 // Uses "stabilization filter" - groups consecutive similar frames and takes the last frame of each group
-function extractVideoTimeline(videoPath, tmpDir, res, onComplete) {
-  const STABILIZATION_THRESHOLD = 0.5; // seconds - consecutive changes within this are grouped
-  const SCENE_THRESHOLD = 0.01; // Very low threshold to catch subtle color changes
+function extractVideoTimeline(videoPath, tmpDir, res, onComplete, options = {}) {
+  // Use provided thresholds or defaults
+  const STABILIZATION_THRESHOLD = options.stabilizationThreshold || 0.5; // seconds - consecutive changes within this are grouped
+  const SCENE_THRESHOLD = options.sceneThreshold || 0.005; // Lower default threshold (was 0.01) to catch more subtle changes
   const MIN_INTERVAL = 1.0; // Minimum interval for additional keyframes (seconds)
 
   // First, get video duration using ffprobe
@@ -373,6 +374,13 @@ function sanitizeHtml(html) {
   });
 }
 
+// [TDD Fix] 重複見出しキーを防ぐためのカウンター
+// 同じテキストの見出しが複数ある場合でも、ユニークなキーを生成する
+var headingKeyCounter = {};
+function resetHeadingKeyCounter() {
+  headingKeyCounter = {};
+}
+
 marked.use({
   hooks: {
     // テーブルをスクロールラッパーで囲む（後処理）
@@ -382,6 +390,21 @@ marked.use({
     }
   },
   renderer: {
+    // 見出しをトグル可能なセクションとしてレンダリング
+    // markedのrenderer.headingは (text, level, raw) を引数に取る
+    heading: function(text, level, raw) {
+      // トグル用のdata属性を追加（見出しテキスト + 出現順でユニークキーを生成）
+      var baseKey = (text || '').replace(/[^a-zA-Z0-9\u3040-\u309F\u30A0-\u30FF\u4E00-\u9FAF\u3400-\u4DBF]/g, '-').toLowerCase();
+      // 同じbaseKeyが出現した回数をカウントしてユニーク化
+      if (headingKeyCounter[baseKey] === undefined) {
+        headingKeyCounter[baseKey] = 0;
+      }
+      var headingKey = baseKey + '-' + headingKeyCounter[baseKey]++;
+      return '<h' + level + ' class="md-heading-toggle" data-heading-key="' + escapeHtmlForXss(headingKey) + '" data-heading-level="' + level + '">' +
+             '<span class="heading-toggle-icon">▼</span>' +
+             text +
+             '</h' + level + '>';
+    },
     // 生HTMLブロックをサニタイズ
     html: function(token) {
       var text = token.raw || token.text || token;
@@ -862,6 +885,9 @@ function loadText(filePath) {
 }
 
 function loadMarkdown(filePath) {
+  // [TDD Fix] カウンターをリセットして新しいMarkdownファイルの見出しキーを初期化
+  resetHeadingKeyCounter();
+
   const raw = fs.readFileSync(filePath);
   const text = decodeBuffer(raw);
   const lines = text.split(/\r?\n/);
@@ -3020,6 +3046,35 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
     .md-preview h1, .md-preview h2, .md-preview h3, .md-preview h4 {
       margin: 0.4em 0 0.2em;
     }
+    /* Heading toggle feature */
+    .md-preview .md-heading-toggle {
+      display: flex;
+      align-items: center;
+      gap: 6px;
+    }
+    .md-preview .heading-toggle-icon {
+      font-size: 0.6em;
+      transition: transform 150ms ease;
+      color: var(--muted);
+      flex-shrink: 0;
+      cursor: pointer;
+      padding: 4px 8px;
+      margin: -4px 0 -4px -8px;
+      border-radius: 4px;
+    }
+    .md-preview .heading-toggle-icon:hover {
+      background: var(--hover-bg);
+      color: var(--accent);
+    }
+    .md-preview .md-heading-toggle.collapsed .heading-toggle-icon {
+      transform: rotate(-90deg);
+    }
+    .md-preview .heading-section-content {
+      /* Content wrapper for collapsible sections */
+    }
+    .md-preview .heading-section-content.hidden {
+      display: none;
+    }
     .md-preview p { margin: 0.3em 0; line-height: 1.5; }
     .md-preview img { max-width: 100%; height: auto; border-radius: 8px; }
     .md-preview video.video-preview { max-width: 100%; height: auto; border-radius: 8px; background: #000; }
@@ -3425,6 +3480,111 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       position: relative;
       flex-shrink: 0;
     }
+    /* Video threshold settings panel */
+    .video-settings-btn {
+      position: absolute;
+      top: 14px;
+      right: 64px;
+      width: 40px;
+      height: 40px;
+      display: flex;
+      align-items: center;
+      justify-content: center;
+      background: rgba(0, 0, 0, 0.55);
+      border: 1px solid rgba(255, 255, 255, 0.25);
+      border-radius: 50%;
+      cursor: pointer;
+      color: #fff;
+      font-size: 18px;
+      z-index: 10;
+      backdrop-filter: blur(4px);
+      transition: background 120ms ease, transform 120ms ease;
+    }
+    .video-settings-btn:hover {
+      background: rgba(0, 0, 0, 0.75);
+      transform: scale(1.04);
+    }
+    .video-settings-panel {
+      position: absolute;
+      top: 60px;
+      right: 14px;
+      background: rgba(0, 0, 0, 0.9);
+      border: 1px solid rgba(255, 255, 255, 0.2);
+      border-radius: 12px;
+      padding: 16px;
+      z-index: 15;
+      min-width: 280px;
+      display: none;
+      backdrop-filter: blur(8px);
+    }
+    .video-settings-panel.visible {
+      display: block;
+    }
+    .video-settings-panel h4 {
+      margin: 0 0 8px;
+      color: #fff;
+      font-size: 14px;
+      font-weight: 500;
+    }
+    .video-settings-desc {
+      margin: 0 0 12px;
+      color: rgba(255, 255, 255, 0.6);
+      font-size: 11px;
+      line-height: 1.4;
+    }
+    .video-settings-buttons {
+      display: flex;
+      gap: 6px;
+      margin-bottom: 12px;
+    }
+    .video-settings-buttons button {
+      flex: 1;
+      padding: 8px 4px;
+      border: 1px solid rgba(255, 255, 255, 0.2);
+      border-radius: 6px;
+      background: rgba(255, 255, 255, 0.1);
+      color: rgba(255, 255, 255, 0.8);
+      font-size: 11px;
+      cursor: pointer;
+      transition: all 120ms ease;
+    }
+    .video-settings-buttons button:hover {
+      background: rgba(255, 255, 255, 0.2);
+      border-color: rgba(255, 255, 255, 0.3);
+    }
+    .video-settings-buttons button.selected {
+      background: #3b82f6;
+      border-color: #3b82f6;
+      color: #fff;
+    }
+    .video-settings-actions {
+      display: flex;
+      gap: 8px;
+      margin-top: 8px;
+    }
+    .video-settings-actions button {
+      flex: 1;
+      padding: 8px 12px;
+      border: none;
+      border-radius: 6px;
+      font-size: 12px;
+      cursor: pointer;
+      transition: background 120ms ease;
+    }
+    .video-settings-actions .regenerate-btn {
+      background: #3b82f6;
+      color: #fff;
+    }
+    .video-settings-actions .regenerate-btn:hover {
+      background: #2563eb;
+    }
+    .video-settings-actions .reset-btn {
+      background: rgba(255, 255, 255, 0.15);
+      color: #fff;
+    }
+    .video-settings-actions .reset-btn:hover {
+      background: rgba(255, 255, 255, 0.25);
+    }
     /* Video Shortcuts Help */
     .video-shortcuts-help {
       opacity: 0.85;
@@ -4358,6 +4518,21 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
   </div>
   <div class="video-fullscreen-overlay" id="video-fullscreen">
     <button class="video-close-btn" id="video-close" aria-label="Close video" title="Close (ESC)">✕</button>
+    <button class="video-settings-btn" id="video-settings-btn" aria-label="Timeline settings" title="Timeline settings">⚙</button>
+    <div class="video-settings-panel" id="video-settings-panel">
+      <h4>サムネイル数の調整</h4>
+      <p class="video-settings-desc">重要シーンの見逃しを防ぐため、サムネイル数を調整できます</p>
+      <div class="video-settings-buttons" id="scene-buttons">
+        <button data-scene="0.05" data-stab="1.0">少なめ</button>
+        <button data-scene="0.02" data-stab="0.8">やや少</button>
+        <button data-scene="0.005" data-stab="0.5" class="selected">標準</button>
+        <button data-scene="0.002" data-stab="0.3">やや多</button>
+        <button data-scene="0.001" data-stab="0.2">多め</button>
+      </div>
+      <div class="video-settings-actions">
+        <button class="regenerate-btn" id="video-settings-regenerate">この設定で再生成</button>
+      </div>
+    </div>
     <div class="video-container" id="video-container"></div>
   </div>
 
@@ -6083,8 +6258,17 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       let startX, startY;
       let svgNaturalWidth = 0, svgNaturalHeight = 0;
       let minimapScale = 1;
+      let currentMermaidContainer = null; // Store the container for selection after close
+      let skipNextFullscreenOpen = false; // Flag to skip opening fullscreen after close
 
       function openFullscreen(mermaidEl) {
+        // Skip if this is triggered by the post-close click
+        if (skipNextFullscreenOpen) {
+          skipNextFullscreenOpen = false;
+          return;
+        }
+        // Store the container for selection when fullscreen closes
+        currentMermaidContainer = mermaidEl.closest('.mermaid-container');
         const svg = mermaidEl.querySelector('svg');
         if (!svg) return;
         fsWrapper.innerHTML = '';
@@ -6146,6 +6330,17 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
 
       function closeFullscreen() {
         fsOverlay.classList.remove('visible');
+        // Trigger selection of the mermaid container after closing fullscreen
+        if (currentMermaidContainer) {
+          const containerToSelect = currentMermaidContainer;
+          currentMermaidContainer = null;
+          // Set flags to skip reopening fullscreen and trigger selection
+          setTimeout(() => {
+            skipNextFullscreenOpen = true;
+            window._mermaidSelectAfterClose = containerToSelect;
+            containerToSelect.click();
+          }, 100);
+        }
       }
 
       function updateTransform() {
@@ -6375,6 +6570,82 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       }
     })();
 
+    // --- Heading Toggle ---
+    (function initHeadingToggle() {
+      const preview = document.querySelector('.md-preview');
+      if (!preview) return;
+
+      const STORAGE_PREFIX = 'reviw-heading-toggle-';
+
+      // Get all headings
+      const headings = Array.from(preview.querySelectorAll('.md-heading-toggle'));
+      if (headings.length === 0) return;
+
+      // Wrap content between headings into sections
+      headings.forEach((heading, idx) => {
+        const level = parseInt(heading.dataset.headingLevel) || 1;
+        const key = heading.dataset.headingKey;
+
+        // Collect all siblings until next heading of same or higher level
+        const content = [];
+        let sibling = heading.nextElementSibling;
+
+        while (sibling) {
+          // Check if it's a heading of same or higher level
+          if (sibling.classList.contains('md-heading-toggle')) {
+            const siblingLevel = parseInt(sibling.dataset.headingLevel) || 1;
+            if (siblingLevel <= level) break;
+          }
+          content.push(sibling);
+          sibling = sibling.nextElementSibling;
+        }
+
+        // Create wrapper for content
+        if (content.length > 0) {
+          const wrapper = document.createElement('div');
+          wrapper.className = 'heading-section-content';
+          wrapper.dataset.forHeading = key;
+
+          // Move content into wrapper
+          const firstContent = content[0];
+          heading.parentNode.insertBefore(wrapper, firstContent);
+          content.forEach(el => wrapper.appendChild(el));
+
+          // Restore state from localStorage
+          const savedState = localStorage.getItem(STORAGE_PREFIX + key);
+          if (savedState === 'collapsed') {
+            heading.classList.add('collapsed');
+            wrapper.classList.add('hidden');
+          }
+        }
+
+        // Add click handler to toggle icon only (not the whole heading)
+        // This allows the heading text to remain selectable for comments
+        const toggleIcon = heading.querySelector('.heading-toggle-icon');
+        if (toggleIcon) {
+          toggleIcon.addEventListener('click', (e) => {
+            e.stopPropagation(); // Prevent triggering comment selection
+            const wrapper = preview.querySelector(\`.heading-section-content[data-for-heading="\${key}"]\`);
+            if (!wrapper) return;
+
+            const isCollapsed = heading.classList.toggle('collapsed');
+            wrapper.classList.toggle('hidden', isCollapsed);
+
+            // Save state to localStorage
+            localStorage.setItem(STORAGE_PREFIX + key, isCollapsed ? 'collapsed' : 'expanded');
+
+            // Re-render Mermaid diagrams when expanding (they may not render while hidden)
+            if (!isCollapsed && typeof mermaid !== 'undefined') {
+              const mermaidDivs = wrapper.querySelectorAll('.mermaid');
+              if (mermaidDivs.length > 0) {
+                mermaid.run({ nodes: Array.from(mermaidDivs) }).catch(() => {});
+              }
+            }
+          });
+        }
+      });
+    })();
+
     // --- Image Fullscreen ---
     (function initImageFullscreen() {
       const preview = document.querySelector('.md-preview');
@@ -6526,8 +6797,23 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
         return mins + ':' + (secs < 10 ? '0' : '') + secs;
       }
 
+      // Default threshold values
+      const DEFAULT_SCENE_THRESHOLD = 0.005;
+      const DEFAULT_STABILIZATION_THRESHOLD = 0.5;
+      let currentSceneThreshold = DEFAULT_SCENE_THRESHOLD;
+      let currentStabilizationThreshold = DEFAULT_STABILIZATION_THRESHOLD;
+      let currentVideoPath = null;
+      let currentVideo = null;
+
       // Load video timeline via SSE
-      function loadVideoTimeline(videoPath, video) {
+      function loadVideoTimeline(videoPath, video, options = {}) {
+        const sceneThreshold = options.sceneThreshold || currentSceneThreshold;
+        const stabilizationThreshold = options.stabilizationThreshold || currentStabilizationThreshold;
+
+        // Store for regeneration
+        currentVideoPath = videoPath;
+        currentVideo = video;
+
         // Close existing connection
         if (currentTimelineEventSource) {
           currentTimelineEventSource.close();
@@ -6552,9 +6838,9 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
 
         videoContainer.appendChild(timeline);
 
-        // Start SSE connection
+        // Start SSE connection with threshold parameters
         const encodedPath = encodeURIComponent(videoPath);
-        const es = new EventSource('/video-timeline?path=' + encodedPath);
+        const es = new EventSource(\`/video-timeline?path=\${encodedPath}&scene=\${sceneThreshold}&stabilization=\${stabilizationThreshold}\`);
         currentTimelineEventSource = es;
 
         es.onmessage = function(e) {
@@ -6648,6 +6934,11 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       }
 
       function showVideo(index) {
+        // Skip if this is triggered by the post-close click
+        if (window._skipNextVideoFullscreen) {
+          window._skipNextVideoFullscreen = false;
+          return;
+        }
         if (index < 0 || index >= allVideoSources.length) return;
         currentVideoIndex = index;
         const source = allVideoSources[index];
@@ -6730,6 +7021,12 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
 
       function closeVideoOverlay() {
         videoOverlay.classList.remove('visible');
+
+        // Save source element before resetting index (for triggering selection)
+        const closedSource = currentVideoIndex >= 0 && currentVideoIndex < allVideoSources.length
+          ? allVideoSources[currentVideoIndex]
+          : null;
+
         currentVideoIndex = -1;
 
         // Close timeline SSE connection
@@ -6747,6 +7044,18 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           video.remove();
         }
 
+        // Trigger selection on the source element's parent table cell (if any)
+        if (closedSource && closedSource.element) {
+          const parentCell = closedSource.element.closest('td, th');
+          if (parentCell) {
+            // Set global flag to skip reopening fullscreen, then trigger click for selection
+            setTimeout(() => {
+              window._skipNextVideoFullscreen = true;
+              parentCell.click();
+            }, 100);
+          }
+        }
+
         // Remove timeline
         const timeline = videoContainer.querySelector('.video-timeline');
         if (timeline) timeline.remove();
@@ -6931,6 +7240,63 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           });
         }
       });
+
+      // --- Video Settings Panel ---
+      const settingsBtn = document.getElementById('video-settings-btn');
+      const settingsPanel = document.getElementById('video-settings-panel');
+      const sceneButtons = document.getElementById('scene-buttons');
+      const regenerateBtn = document.getElementById('video-settings-regenerate');
+
+      if (settingsBtn && settingsPanel) {
+        // Toggle settings panel
+        settingsBtn.addEventListener('click', (e) => {
+          e.stopPropagation();
+          settingsPanel.classList.toggle('visible');
+        });
+
+        // Prevent panel clicks from closing overlay
+        settingsPanel.addEventListener('click', (e) => e.stopPropagation());
+
+        // Handle 5-level button clicks
+        if (sceneButtons) {
+          const buttons = sceneButtons.querySelectorAll('button');
+          buttons.forEach(btn => {
+            btn.addEventListener('click', () => {
+              buttons.forEach(b => b.classList.remove('selected'));
+              btn.classList.add('selected');
+            });
+          });
+        }
+
+        // Regenerate timeline with selected settings
+        if (regenerateBtn) {
+          regenerateBtn.addEventListener('click', () => {
+            if (!currentVideoPath || !currentVideo) return;
+
+            // Get selected button's values
+            const selectedBtn = sceneButtons?.querySelector('button.selected');
+            if (selectedBtn) {
+              currentSceneThreshold = parseFloat(selectedBtn.dataset.scene) || DEFAULT_SCENE_THRESHOLD;
+              currentStabilizationThreshold = parseFloat(selectedBtn.dataset.stab) || DEFAULT_STABILIZATION_THRESHOLD;
+            }
+
+            loadVideoTimeline(currentVideoPath, currentVideo, {
+              sceneThreshold: currentSceneThreshold,
+              stabilizationThreshold: currentStabilizationThreshold
+            });
+
+            // Hide panel after regenerating
+            settingsPanel.classList.remove('visible');
+          });
+        }
+
+        // Close panel when overlay closes
+        const originalCloseVideoOverlay = closeVideoOverlay;
+        closeVideoOverlay = function() {
+          settingsPanel.classList.remove('visible');
+          originalCloseVideoOverlay();
+        };
+      }
     })();
 
     // --- Preview Commenting ---
@@ -6946,11 +7312,16 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
         .md-preview > p:hover, .md-preview > h1:hover, .md-preview > h2:hover,
         .md-preview > h3:hover, .md-preview > h4:hover, .md-preview > h5:hover,
         .md-preview > h6:hover, .md-preview > ul > li:hover, .md-preview > ol > li:hover,
-        .md-preview > pre:hover, .md-preview > blockquote:hover {
+        .md-preview > pre:hover, .md-preview > blockquote:hover,
+        .md-preview details summary:hover {
           background: rgba(99, 102, 241, 0.08);
           cursor: pointer;
           border-radius: 4px;
         }
+        .md-preview td:hover, .md-preview th:hover {
+          background: rgba(99, 102, 241, 0.12);
+          cursor: pointer;
+        }
         .md-preview img:hover {
           outline: 2px solid var(--accent);
           cursor: pointer;
@@ -7013,7 +7384,9 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
         }
 
         if (!text) return -1;
-        const normalized = text.trim().replace(/\\s+/g, ' ').slice(0, 100);
+        // Remove toggle icon characters (▼, ▶) that may be included from heading toggles
+        const cleanText = text.replace(/[▼▶]/g, '').trim();
+        const normalized = cleanText.replace(/\\s+/g, ' ').slice(0, 100);
         if (!normalized) return -1;
 
         for (let i = 0; i < DATA.length; i++) {
@@ -7033,6 +7406,19 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
             }
           }
 
+          // Check for HTML summary tags: <summary>text</summary>
+          const summaryMatch = lineText.match(/<summary>([^<]*)<\\/summary>/i);
+          if (summaryMatch) {
+            const summaryText = summaryMatch[1].trim();
+            if (summaryText === normalized || summaryText.toLowerCase() === normalized.toLowerCase()) {
+              return i + 1;
+            }
+            // Partial match for long summary text
+            if (summaryText.includes(normalized.slice(0, 30)) && normalized.length > 5) {
+              return i + 1;
+            }
+          }
+
           // Try stripping all markdown formatting (links, bold, italic, etc.)
           const strippedLine = stripMarkdown(lineText).replace(/\\s+/g, ' ').slice(0, 100);
           if (strippedLine === normalized) return i + 1;
@@ -7045,10 +7431,31 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
       // Helper: find matching source line for table cell (prioritizes table rows)
       function findTableSourceLine(text) {
         if (!text) return -1;
-        const normalized = text.trim().replace(/\\s+/g, ' ').slice(0, 100);
+        // Remove toggle icon characters (▼, ▶) that may be included from heading toggles
+        const cleanText = text.replace(/[▼▶]/g, '').trim();
+        const normalized = cleanText.replace(/\\s+/g, ' ').slice(0, 100);
         if (!normalized) return -1;
 
-        // First pass: look for table rows (lines starting with |) containing the text
+        // First pass: look for EXACT cell text match (not inside markdown syntax)
+        for (let i = 0; i < DATA.length; i++) {
+          const lineText = (DATA[i][0] || '').trim();
+          if (!lineText || !lineText.startsWith('|')) continue;
+
+          // Split into cells and check for exact match (excluding markdown syntax)
+          const cells = lineText.split('|').map(c => c.trim());
+          for (const cell of cells) {
+            // Skip cells that are markdown images/links (start with ![, contain []())
+            if (cell.match(/^!?\\[.*\\]\\(.*\\)$/)) continue;
+
+            // Check for exact cell text match
+            if (cell === normalized) return i + 1;
+
+            // For short text (like header cells), require exact word match
+            if (normalized.length <= 5 && cell === normalized) return i + 1;
+          }
+        }
+
+        // Second pass: look for partial matches (including inside markdown syntax)
         for (let i = 0; i < DATA.length; i++) {
           const lineText = (DATA[i][0] || '').trim();
           if (!lineText || !lineText.startsWith('|')) continue;
@@ -7137,6 +7544,51 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
         return { startLine: -1, endLine: -1 };
       }
 
+      // Helper: find source line range for mermaid diagram by its container
+      function findMermaidSourceLine(mermaidContainer) {
+        if (!mermaidContainer) return { startLine: -1, endLine: -1 };
+
+        // Find the index of this mermaid container among all mermaid containers
+        const allMermaidContainers = document.querySelectorAll('.mermaid-container');
+        let containerIndex = -1;
+        for (let i = 0; i < allMermaidContainers.length; i++) {
+          if (allMermaidContainers[i] === mermaidContainer) {
+            containerIndex = i;
+            break;
+          }
+        }
+        if (containerIndex < 0) return { startLine: -1, endLine: -1 };
+
+        // Find all mermaid code blocks in the source
+        const mermaidBlocks = [];
+        let currentBlock = null;
+
+        for (let i = 0; i < DATA.length; i++) {
+          const lineText = (DATA[i][0] || '').trim();
+
+          if (lineText.match(/^\`\`\`mermaid/i) && !currentBlock) {
+            // Start of a mermaid code block
+            currentBlock = { startLine: i + 1, lines: [] };
+          } else if (lineText === '\`\`\`' && currentBlock) {
+            // End of the code block
+            currentBlock.endLine = i + 1;
+            mermaidBlocks.push(currentBlock);
+            currentBlock = null;
+          } else if (currentBlock) {
+            // Inside the mermaid block
+            currentBlock.lines.push(DATA[i][0] || '');
+          }
+        }
+
+        // Return the block at the same index as the container
+        if (containerIndex < mermaidBlocks.length) {
+          const block = mermaidBlocks[containerIndex];
+          return { startLine: block.startLine, endLine: block.endLine };
+        }
+
+        return { startLine: -1, endLine: -1 };
+      }
+
       // Helper: find source line for image by src
       function findImageSourceLine(src) {
         if (!src) return -1;
@@ -7217,6 +7669,44 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           return;
         }
 
+        // Handle video/video-button clicks - select source line first
+        // This includes video elements, fullscreen buttons, video wrapper clicks,
+        // AND clicks on table cells containing videos (for post-fullscreen selection)
+        const videoElement = e.target.closest('video.video-preview');
+        const videoButton = e.target.closest('.video-fullscreen-btn');
+        const videoWrapper = e.target.closest('.video-fullscreen-wrapper');
+
+        // Also check if clicking on a td that contains a video
+        const targetCell = e.target.closest('td, th');
+        const cellContainsVideo = targetCell && targetCell.querySelector('video.video-preview');
+
+        if (videoElement || videoButton || videoWrapper || cellContainsVideo) {
+          // Check if this is a post-fullscreen click (for selection only, skip fullscreen)
+          const isPostFullscreenClick = window._skipNextVideoFullscreen;
+          if (isPostFullscreenClick) {
+            window._skipNextVideoFullscreen = false;
+          }
+
+          // Find the parent table cell
+          const refElement = videoElement || videoButton || videoWrapper || targetCell;
+          const parentCell = refElement.closest('td, th') || targetCell;
+          if (parentCell) {
+            // Use video src to find the source line
+            const video = videoElement || parentCell.querySelector('video');
+            if (video && video.src) {
+              const line = findImageSourceLine(video.src);
+              if (line > 0) {
+                selectSourceRange(line, null, parentCell);
+                // Return if this is a post-fullscreen click (for selection only)
+                if (isPostFullscreenClick) {
+                  return;
+                }
+              }
+            }
+          }
+          // Don't return for normal clicks - let fullscreen handler work too
+        }
+
         // Handle links - sync to source but let link work normally
         const link = e.target.closest('a');
         if (link) {
@@ -7233,8 +7723,23 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           return;
         }
 
-        // Ignore clicks on mermaid, video overlay
-        if (e.target.closest('.mermaid-container, .video-fullscreen-overlay')) return;
+        // Handle mermaid container clicks for post-fullscreen selection
+        const mermaidContainer = e.target.closest('.mermaid-container');
+        if (mermaidContainer) {
+          // Check if this is a post-fullscreen click for selection
+          if (window._mermaidSelectAfterClose === mermaidContainer) {
+            window._mermaidSelectAfterClose = null;
+            const { startLine, endLine } = findMermaidSourceLine(mermaidContainer);
+            if (startLine > 0) {
+              selectSourceRange(startLine, endLine, mermaidContainer);
+            }
+          }
+          // Always return to prevent fullscreen from reopening or other handling
+          return;
+        }
+
+        // Ignore clicks on video overlay
+        if (e.target.closest('.video-fullscreen-overlay')) return;
 
         // Handle code blocks - select entire block
         const pre = e.target.closest('pre');
@@ -7248,15 +7753,31 @@ function htmlTemplate(dataRows, cols, projectRoot, relativePath, mode, previewHt
           return;
         }
 
-        const target = e.target.closest('p, h1, h2, h3, h4, h5, h6, li, blockquote, td, th');
+        const target = e.target.closest('p, h1, h2, h3, h4, h5, h6, li, blockquote, td, th, summary');
         if (!target) return;
 
+        // Get text content, excluding toggle icon if present (for heading toggles)
+        let searchText = target.textContent;
+
+        // For summary elements, clean up the text (may have disclosure triangle)
+        if (target.tagName === 'SUMMARY') {
+          searchText = searchText.replace(/^[▶▼◀►◆◇]?\\s*/, '').trim();
+        }
+        const toggleIcon = target.querySelector('.heading-toggle-icon');
+        if (toggleIcon) {
+          // Exclude the toggle icon text from search
+          searchText = searchText.replace(toggleIcon.textContent, '').trim();
+        }
+
         // Use table-specific search for table cells, otherwise use element-aware search
         const isTableCell = target.tagName === 'TD' || target.tagName === 'TH';
-        const line = isTableCell ? findTableSourceLine(target.textContent) : findSourceLine(target.textContent, target);
+        const line = isTableCell ? findTableSourceLine(searchText) : findSourceLine(searchText, target);
         if (line <= 0) return;
 
-        e.preventDefault();
+        // Don't prevent default for summary elements - let native <details> toggle work
+        if (target.tagName !== 'SUMMARY') {
+          e.preventDefault();
+        }
         selectSourceRange(line, null, target);
       });
 
@@ -8090,6 +8611,8 @@ function createFileServer(filePath, fileIndex = 0) {
       if (req.method === "GET" && req.url.startsWith("/video-timeline?")) {
         const urlParams = new URL(req.url, `http://localhost`);
         const videoPath = urlParams.searchParams.get("path");
+        const sceneThreshold = parseFloat(urlParams.searchParams.get("scene")) || 0.005;
+        const stabilizationThreshold = parseFloat(urlParams.searchParams.get("stabilization")) || 0.5;
 
         if (!videoPath) {
           res.writeHead(400, { "Content-Type": "text/plain" });
@@ -8148,7 +8671,7 @@ function createFileServer(filePath, fileIndex = 0) {
               // Ignore cleanup errors
             }
           });
-        });
+        }, { sceneThreshold, stabilizationThreshold });
         return;
       }
 

package/package.json

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