Report UX improvements: table format conversion, semantic changes output, HTML styling

FolderDiffIL4DotNet.Tests/Services/HtmlReportGenerateServiceTests.cs

@@ -131,10 +131,11 @@ public void GenerateDiffReportHtml_SummarySection_ContainsCorrectCounts()
                 computerName: "test-host", config);
 
             var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
-            Assert.Contains("stat-label\">Unchanged</td><td class=\"stat-value\">1</td>", html);
-            Assert.Contains("stat-label\">Added</td><td class=\"stat-value\">1</td>", html);
-            Assert.Contains("stat-label\">Removed</td><td class=\"stat-value\">1</td>", html);
-            Assert.Contains("stat-label\">Modified</td><td class=\"stat-value\">1</td>", html);
+            Assert.Contains("stat-label\">Unchanged</td>", html);
+            Assert.Contains("stat-label\">Added</td>", html);
+            Assert.Contains("stat-label\">Removed</td>", html);
+            Assert.Contains("stat-label\">Modified</td>", html);
+            Assert.Contains("stat-value\">1</td>", html);
             Assert.Contains("3 (Old) vs 3 (New)", html);
         }
 
@@ -476,9 +477,10 @@ public void GenerateDiffReportHtml_InlineDiffSummary_UsesSameOneBasedNumberAsLef
             var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
             Assert.Contains("<td class=\"col-no\">1</td>", html);
             Assert.Contains("<td class=\"col-no\">2</td>", html);
-            Assert.Contains("#1 Show diff", html);
-            Assert.Contains("#2 Show diff", html);
-            Assert.DoesNotContain("#0 Show diff", html);
+            Assert.Contains(">#1 ", html);
+            Assert.Contains(">#2 ", html);
+            Assert.DoesNotContain(">#0 ", html);
+            Assert.Contains("Show diff", html);
         }
 
         // Verify DIFF REASON / Location / Timestamp body cells have text-align: center, but Notes column does not
@@ -500,7 +502,7 @@ public void GenerateDiffReportHtml_BodyCells_ColReasonPathTs_HaveCenterAlignment
             Assert.Contains("td.col-ts    { white-space: nowrap; text-align: center; }", html);
             Assert.Contains("min-width: 9em; text-align: center; }", html); // col-diff has text-align: center
             // File Path is NOT center-aligned / File Path 列は中央揃えではない
-            Assert.Contains("td.col-path { white-space: nowrap; overflow: hidden; text-overflow: ellipsis; }", html);
+            Assert.Contains("td.col-path { white-space: nowrap; overflow: hidden; }", html);
             // Notes column is NOT center-aligned / Notes 列は中央揃えではない
             Assert.Contains("td.col-notes  { overflow: hidden; }", html);
         }
@@ -894,7 +896,7 @@ public void GenerateDiffReportHtml_AssemblySemanticChanges_KindAccessModifiersUs
             Assert.Contains("<code>Method</code>", html);        // Kind
             Assert.Contains("<code>public</code>", html);        // Access
             Assert.Contains("<code>virtual</code>", html);       // Modifiers
-            Assert.Contains("<code>Modified</code>", html);      // Change
+            Assert.Contains("style=\"background:#e3f2fd\">[ * ]", html); // Status cell with blue bg (no code emphasis)
             Assert.Contains("<code>Changed</code>", html);       // Body
         }
 
@@ -939,6 +941,128 @@ public void GenerateDiffReportHtml_AssemblySemanticChanges_TdHasWhiteBackground(
             Assert.Contains("background: #fff", html);
         }
 
+        // ── Req1: Legend table / 凡例テーブル ─────────────────────────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_LegendSection_UsesTableFormat()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("legend-table");
+            var config = CreateConfig();
+
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            Assert.Contains("legend-table", html);
+            Assert.Contains("<table class=\"legend-table\">", html);
+        }
+
+        // ── Req2: stat-table borders / 統計テーブルボーダー ────────────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_StatTable_HasVisibleBorders()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("stat-border");
+            var config = CreateConfig();
+
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            Assert.Contains("border: 1px solid #ddd", html);
+        }
+
+        // ── Req4: InlineDiffMaxEditDistance code tag / code タグ ──────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_EditDistanceSkipped_InlineDiffMaxEditDistanceHasCodeTag()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("edit-dist-code-tag");
+
+            File.WriteAllLines(Path.Combine(oldDir, "huge.txt"), Enumerable.Range(1, 2001).Select(i => $"old{i}"));
+            File.WriteAllLines(Path.Combine(newDir, "huge.txt"), Enumerable.Range(1, 2001).Select(i => $"new{i}"));
+
+            _resultLists.AddModifiedFileRelativePath("huge.txt");
+            _resultLists.RecordDiffDetail("huge.txt", FileDiffResultLists.DiffDetailResult.TextMismatch);
+
+            var config = CreateConfig(enableInlineDiff: true);
+
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            Assert.Contains("<code>InlineDiffMaxEditDistance</code>", html);
+            Assert.Contains("current value:", html);
+        }
+
+        // ── Req5: diff-row background / 差分行背景色 ─────────────────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_DiffRowBackground_UsesDarkerColor()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("diff-row-bg");
+            var config = CreateConfig();
+
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            // diff-row background should be #edf0f4, not the old #f6f8fa
+            Assert.Contains("tr.diff-row { background: #edf0f4; }", html);
+            Assert.DoesNotContain("tr.diff-row { background: #f6f8fa; }", html);
+        }
+
+        // ── Req7: Copy paths button / コピーボタン ──────────────────────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_FilePathRow_HasCopyButton()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("copy-btn");
+
+            _resultLists.AddModifiedFileRelativePath("src/app.dll");
+            _resultLists.RecordDiffDetail("src/app.dll", FileDiffResultLists.DiffDetailResult.MD5Mismatch);
+
+            var config = CreateConfig();
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            Assert.Contains("btn-copy-path", html);
+            Assert.Contains("copyPath", html);
+            Assert.Contains("path-text", html);
+        }
+
+        // ── Req8: Row hover highlight / 行ホバーハイライト ──────────────────────
+
+        [Fact]
+        public void GenerateDiffReportHtml_RowHover_HasLightPurpleHighlight()
+        {
+            var (oldDir, newDir, reportDir) = MakeDirs("hover-highlight");
+            var config = CreateConfig();
+
+            _service.GenerateDiffReportHtml(oldDir, newDir, reportDir,
+                appVersion: "1.0", elapsedTimeString: null,
+                computerName: "test-host", config);
+
+            var html = File.ReadAllText(Path.Combine(reportDir, HtmlReportGenerateService.DIFF_REPORT_HTML_FILE_NAME));
+            Assert.Contains(":not(.stat-table):not(.legend-table):not(.il-ignore-table) > tbody tr:not(.diff-row):not(.diff-hunk-tr):not(.diff-del-tr):not(.diff-add-tr):hover { background: #f3eef8; }", html);
+            Assert.Contains("table.semantic-changes-table tbody tr:hover td { background: #f3eef8 !important; }", html);
+        }
+
+        // ── Req9: Language toggle / 言語切り替え ────────────────────────────────
+
+        [Fact]
+        public void I18n_ReturnsHtmlEncodedEnglishText()
+        {
+            string result = HtmlReportGenerateService.I18n("Hello", "こんにちは");
+            Assert.Equal("Hello", result);
+        }
+
         private static ConfigSettings CreateConfig(bool enableInlineDiff = true, bool lazyRender = false) => new()
         {
             IgnoredExtensions = new List<string>(),

FolderDiffIL4DotNet.Tests/Services/ReportGenerateServiceTests.cs

@@ -179,9 +179,10 @@ public void GenerateDiffReport_HeaderShowsIlContainsIgnoreNote_WhenEnabled()
 
             var reportPath = Path.Combine(reportDir, "diff_report.md");
             var reportText = File.ReadAllText(reportPath);
-            Assert.Contains("lines containing any of the configured strings are ignored", reportText);
-            Assert.Contains("\"buildserver\"", reportText);
-            Assert.Contains("\"buildPath\"", reportText);
+            Assert.Contains("lines containing any of the configured strings are ignored:", reportText);
+            Assert.Contains("| Ignored String |", reportText);
+            Assert.Contains("| \"buildserver\" |", reportText);
+            Assert.Contains("| \"buildPath\" |", reportText);
         }
 
         [Fact]
@@ -385,7 +386,7 @@ public void GenerateDiffReport_WritesWarningsInSeverityOrder_WhenMd5MismatchAndT
                 Path.Combine("nested", "payload.bin"),
                 "2026-03-14 10:00:00",
                 "2026-03-14 09:00:00");
-            _resultLists.RecordDiffDetail("payload.bin", FileDiffResultLists.DiffDetailResult.MD5Mismatch);
+            _resultLists.RecordDiffDetail(Path.Combine("nested", "payload.bin"), FileDiffResultLists.DiffDetailResult.MD5Mismatch);
 
             var config = CreateConfig();
             _service.GenerateDiffReport(
@@ -402,9 +403,10 @@ public void GenerateDiffReport_WritesWarningsInSeverityOrder_WhenMd5MismatchAndT
             Assert.Contains("## Warnings", reportText);
             Assert.Contains($"- **WARNING:** {Constants.WARNING_MD5_MISMATCH}", reportText);
             Assert.Contains("- **WARNING:** One or more **modified** files in `new` have older last-modified timestamps than the corresponding files in `old`.", reportText);
-            Assert.Contains("  - nested", reportText);
-            Assert.Contains("[2026-03-14 10:00:00 → 2026-03-14 09:00:00]", reportText);
-            Assert.EndsWith("[2026-03-14 10:00:00 → 2026-03-14 09:00:00]", reportText.TrimEnd());
+            Assert.Contains("| Status | File Path | Timestamp | Legend | Disassembler |", reportText);
+            Assert.Contains("|:------:|-----------|:---------:|--------|--------------|", reportText);
+            Assert.Contains("| nested", reportText);
+            Assert.Contains("2026-03-14 10:00:00 → 2026-03-14 09:00:00", reportText);
             Assert.True(
                 reportText.IndexOf(Constants.WARNING_MD5_MISMATCH, StringComparison.Ordinal) <
                 reportText.IndexOf("**modified** files in `new` have older last-modified timestamps", StringComparison.Ordinal));
@@ -473,12 +475,12 @@ public void GenerateDiffReport_ILCacheStats_OutputBetweenSummaryAndWarnings_When
 
             var reportText = File.ReadAllText(Path.Combine(reportDir, "diff_report.md"));
             Assert.Contains("## IL Cache Stats", reportText);
-            Assert.Contains("- Hits    :", reportText);
-            Assert.Contains("- Misses  :", reportText);
-            Assert.Contains("- Hit Rate:", reportText);
-            Assert.Contains("- Stores  :", reportText);
-            Assert.Contains("- Evicted :", reportText);
-            Assert.Contains("- Expired :", reportText);
+            Assert.Contains("| Hits |", reportText);
+            Assert.Contains("| Misses |", reportText);
+            Assert.Contains("| Hit Rate |", reportText);
+            Assert.Contains("| Stores |", reportText);
+            Assert.Contains("| Evicted |", reportText);
+            Assert.Contains("| Expired |", reportText);
             // IL Cache Stats section must appear between Summary and Warnings
             // IL Cache Stats セクションは Summary と Warnings の間に出力されること
             int summaryIdx = reportText.IndexOf("## Summary", StringComparison.Ordinal);
@@ -587,10 +589,10 @@ public void GenerateDiffReport_LargeFileCount_SummaryStatisticsAreCorrect()
 
             var reportText = File.ReadAllText(Path.Combine(reportDir, "diff_report.md"));
 
-            // Summary counts must match (labels are left-padded to 10 chars)
-            // サマリーカウントが一致すること（ラベルは 10 文字に左寄せ）
-            Assert.Contains($"- {"Unchanged",-10}: {fileCount}", reportText, StringComparison.Ordinal);
-            Assert.Contains($"- {"Compared",-10}: {fileCount} (Old) vs {fileCount} (New)", reportText, StringComparison.Ordinal);
+            // Summary counts must match (table format)
+            // サマリーカウントが一致すること（テーブル形式）
+            Assert.Contains($"| Unchanged | {fileCount} |", reportText, StringComparison.Ordinal);
+            Assert.Contains($"| Compared | {fileCount} (Old) vs {fileCount} (New) |", reportText, StringComparison.Ordinal);
         }
 
         [Fact]

README.md

@@ -138,6 +138,8 @@ Column headers for Added / Removed / Modified use colour-coded backgrounds (**gr
 
 Inline diff `<summary>` labels also include a one-based `#N` prefix such as `#3 Show diff` / `#3 Show IL diff`; this number matches the leftmost `#` column for the same row.
 
+A **language toggle** button (`日本語` / `English`) in the controls bar switches all user-facing text between English and Japanese. The preference is persisted in `localStorage` and baked into the downloaded reviewed copy (which also retains the toggle button).
+
 See [doc/samples/diff_report.html](doc/samples/diff_report.html) for a live sample (open in a browser).
 
 ### Review workflow
@@ -622,6 +624,8 @@ Added / Removed / Modified の列ヘッダはそれぞれ**緑・赤・青**の
 
 インライン差分の `<summary>` ラベルにも `#3 Show diff` / `#3 Show IL diff` のような 1 始まりの `#N` プレフィックスが付き、この番号は同じ行の左端 `#` 列と一致します。
 
+コントロールバーの**言語切り替え**ボタン（`日本語` / `English`）で、すべてのユーザー向けテキストを英語と日本語に即座に切り替えられます。言語設定は `localStorage` に保持され、レビュー済みダウンロードにも焼き込まれます（切り替えボタンも引き継がれます）。
+
 ライブサンプルは [doc/samples/diff_report.html](doc/samples/diff_report.html) を参照してください（ブラウザで開いてください）。
 
 ### レビュー手順