fix: コメントハイライトとC-j表示バグを修正

- コメント入力時に即座にグレー表示されるよう修正
  - 差分描画でANSIエスケープが途切れる問題を回避
  - グレーハイライトがある行は常に行全体を再描画
- C-j (newline-and-indent) で行シフト時の表示崩れを修正
  - markDirty()をmarkFullRedraw()に変更
  - 行結合/移動/複製等の行シフト操作も同様に修正
- テストを統合: test_basic_operations.shをrun_all_tests.shにマージ
- テストハーネスに--expectオプションと\nエスケープ対応を追加

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: sanohiro

run_all_tests.sh

@@ -42,6 +42,28 @@ run_test() {
     fi
 }
 
+# 内容検証付きテスト実行ヘルパー（--expect使用）
+run_test_verify() {
+    local test_name="$1"
+    shift
+    # test_data/ のパスを /tmp/ze_test_data/ に置換
+    local args=()
+    for arg in "$@"; do
+        local replaced_arg=$(echo "$arg" | sed 's|test_data/|/tmp/ze_test_data/|g')
+        args+=("$replaced_arg")
+    done
+    local output
+    output=$($HARNESS "${args[@]}" 2>&1)
+    if echo "$output" | grep -q "PASS: Content matches!"; then
+        test_result "$test_name" "PASS"
+    elif echo "$output" | grep -q "Child exited with status: 0"; then
+        # --expectなしの場合は終了コードで判断
+        test_result "$test_name" "PASS"
+    else
+        test_result "$test_name" "FAIL"
+    fi
+}
+
 echo "========================================="
 echo "ze エディタ 統合テストスイート"
 echo "========================================="
@@ -317,6 +339,20 @@ run_test "28.6 ウィンドウ切り替え (C-Tab)" --file=test_data/test_multiw
 run_test "28.7 C-v ページダウン" --file=test_data/test_page_scroll.txt "C-v" "C-x" "C-c"
 run_test "28.8 M-v ページアップ" --file=test_data/test_page_scroll.txt "C-v" "M-v" "C-x" "C-c"
 
+echo
+echo "=== カテゴリ 29: 改行操作とC-j ==="
+run_test "29.1 行頭でEnter" --file=test_data/test_cursor_input.txt "Enter" "C-x" "C-c" "n"
+run_test "29.2 行途中でEnter" --file=test_data/test_cursor_input.txt "Right" "Right" "Enter" "C-x" "C-c" "n"
+run_test "29.3 行末でEnter" --file=test_data/test_cursor_input.txt "End" "Enter" "C-x" "C-c" "n"
+run_test "29.4 行末でC-j" --file=test_data/test_cursor_input.txt "End" "C-j" "C-x" "C-c" "n"
+run_test "29.5 行頭でC-j" --file=test_data/test_cursor_input.txt "C-j" "C-x" "C-c" "n"
+run_test "29.6 行途中でC-j" --file=test_data/test_cursor_input.txt "Right" "Right" "C-j" "C-x" "C-c" "n"
+run_test "29.7 C-j後に文字入力" --file=test_data/test_cursor_input.txt "End" "C-j" "new" "C-x" "C-c" "n"
+# 内容検証テスト
+run_test_verify "29.8 C-j内容検証" --file=test_data/test_newline.txt "End" "C-j" "inserted" "C-x" "C-s" "C-x" "C-c" '--expect=line1\ninserted'
+run_test_verify "29.9 行頭C-j内容検証" --file=test_data/test_newline.txt "C-j" "C-x" "C-s" "C-x" "C-c" '--expect=\nline1'
+run_test_verify "29.10 行複製内容検証 (C-c d)" --file=test_data/test_newline.txt "C-c" "d" "C-x" "C-s" "C-x" "C-c" '--expect=line1\nline1'
+
 echo
 echo "========================================="
 echo "統合テスト完了"

src/commands/edit.zig

@@ -374,7 +374,8 @@ pub fn joinLine(e: *Editor) !void {
         }
 
         buffer_state.editing_ctx.modified = true;
-        markDirtyAll(e, current_line - 1, null);
+        // joinLineは行数が減るため全画面再描画が必要
+        markFullRedrawAll(e);
 
         e.setCursorToPos(delete_start);
 
@@ -466,7 +467,8 @@ pub fn moveLineUp(e: *Editor) !void {
     try e.recordInsert(prev_line_start, line_content, view.getCursorBufferPos());
 
     buffer_state.editing_ctx.modified = true;
-    markDirtyAll(e, current_line - 1, null);
+    // moveLineUpは行の入れ替えのため全画面再描画が必要
+    markFullRedrawAll(e);
 
     view.moveCursorUp();
 }
@@ -500,7 +502,8 @@ pub fn moveLineDown(e: *Editor) !void {
     try e.recordInsert(new_insert_pos, line_content, view.getCursorBufferPos());
 
     buffer_state.editing_ctx.modified = true;
-    markDirtyAll(e, current_line, null);
+    // moveLineDownは行の入れ替えのため全画面再描画が必要
+    markFullRedrawAll(e);
 
     view.moveCursorDown();
 }
@@ -608,7 +611,8 @@ pub fn duplicateLine(e: *Editor) !void {
     try e.recordInsert(insert_pos, to_insert, view.getCursorBufferPos());
 
     buffer_state.editing_ctx.modified = true;
-    markDirtyAll(e, current_line, null);
+    // duplicateLineは行数が増えるため全画面再描画が必要
+    markFullRedrawAll(e);
 
     // カーソルを複製した行に移動
     view.moveCursorDown();

src/editor.zig

@@ -2405,12 +2405,13 @@ pub const Editor = struct {
             const max_screen_line = if (view.viewport_height >= 2) view.viewport_height - 2 else 0;
             if (view.cursor_y < max_screen_line) {
                 view.cursor_y += 1;
-                view.markDirty(current_line, null); // EOF まで再描画
             } else {
                 // 画面の最下部の場合はスクロール
                 view.top_line += 1;
-                view.markFullRedraw(); // スクロール時は全画面再描画
             }
+            // 改行は行シフトを起こすため、prev_screenキャッシュが無効になる
+            // markDirtyでは差分描画が壊れるので全画面再描画が必要
+            view.markFullRedraw();
             view.cursor_x = 0;
             view.top_col = 0; // 水平スクロールもリセット
         } else {
@@ -2593,9 +2594,16 @@ pub const Editor = struct {
         // カーソルを置換後の位置に移動
         self.setCursorToPos(match_pos + replacement.len);
 
-        // 置換した行から再描画（差分レンダリング）
-        const current_line = buffer.findLineByPos(match_pos);
-        self.getCurrentView().markDirty(current_line, null);
+        // 置換した行から再描画
+        // 検索文字列・置換文字列に改行が含まれる場合は行数が変わるため全画面再描画
+        const has_newline = std.mem.indexOf(u8, search, "\n") != null or
+            std.mem.indexOf(u8, replacement, "\n") != null;
+        if (has_newline) {
+            self.getCurrentView().markFullRedraw();
+        } else {
+            const current_line = buffer.findLineByPos(match_pos);
+            self.getCurrentView().markDirty(current_line, current_line);
+        }
     }
 
     // ========================================
@@ -2870,7 +2878,8 @@ pub const Editor = struct {
                                 self.getCurrentBuffer().editing_ctx.modified = true;
                                 self.setCursorToPos(start);
                                 self.window_manager.getCurrentWindow().mark_pos = null; // マークをクリア
-                                self.getCurrentView().markDirty(0, null);
+                                // シェル出力は行数が変わる可能性があるため全画面再描画
+                                self.getCurrentView().markFullRedraw();
                             }
                         } else {
                             // 選択なしの場合はカーソル位置に挿入（+> と同じ動作）
@@ -2880,7 +2889,8 @@ pub const Editor = struct {
                                 try buf.insertSlice(pos, stdout);
                                 try self.recordInsert(pos, stdout, pos);
                                 self.getCurrentBuffer().editing_ctx.modified = true;
-                                self.getCurrentView().markDirty(0, null);
+                                // シェル出力は行数が変わる可能性があるため全画面再描画
+                                self.getCurrentView().markFullRedraw();
                             }
                         }
                     },
@@ -2900,7 +2910,8 @@ pub const Editor = struct {
                         }
                         self.getCurrentBuffer().editing_ctx.modified = true;
                         self.setCursorToPos(0);
-                        self.getCurrentView().markDirty(0, null);
+                        // シェル出力は行数が変わる可能性があるため全画面再描画
+                        self.getCurrentView().markFullRedraw();
                     },
                     .current_line => {
                         // 現在行を置換
@@ -2921,7 +2932,8 @@ pub const Editor = struct {
                         }
                         self.getCurrentBuffer().editing_ctx.modified = true;
                         self.setCursorToPos(line_start);
-                        self.getCurrentView().markDirty(line_num, null);
+                        // シェル出力は行数が変わる可能性があるため全画面再描画
+                        self.getCurrentView().markFullRedraw();
                     },
                 }
             },
@@ -2933,7 +2945,8 @@ pub const Editor = struct {
                     try buf.insertSlice(pos, stdout);
                     try self.recordInsert(pos, stdout, pos);
                     self.getCurrentBuffer().editing_ctx.modified = true;
-                    self.getCurrentView().markDirty(0, null);
+                    // シェル出力は行数が変わる可能性があるため全画面再描画
+                    self.getCurrentView().markFullRedraw();
                 }
             },
             .new_buffer => {

src/view.zig

@@ -137,6 +137,16 @@ fn calculateScreenColumn(line: []const u8, start_byte: usize, end_byte: usize, i
     return screen_col;
 }
 
+/// 行の本文部分（行番号の後）にANSIグレーコード(\x1b[90m)が含まれているかチェック
+/// コメントハイライトの有無を判定するために使用
+fn hasAnsiGray(line: []const u8, content_start: usize) bool {
+    // 行番号の後ろの部分のみチェック
+    if (content_start >= line.len) return false;
+    const content = line[content_start..];
+    // グレーのANSIコード \x1b[90m を探す
+    return std.mem.indexOf(u8, content, "\x1b[90m") != null;
+}
+
 pub const View = struct {
     buffer: *Buffer,
     top_line: usize,
@@ -691,84 +701,97 @@ pub const View = struct {
                 }
             }
 
-            // 差分を検出
-            const min_len = @min(old_line.len, new_line.len);
-            var diff_start: ?usize = null;
-            var diff_end: usize = 0;
+            // 行番号の表示幅を取得
+            const line_num_display_width = self.getLineNumberWidth();
 
-            // 前方から差分を探す
-            for (0..min_len) |i| {
-                if (old_line[i] != new_line[i]) {
-                    if (diff_start == null) diff_start = i;
-                    diff_end = i + 1;
+            // 行番号がある場合、行番号部分のバイト範囲を推定
+            // 行番号フォーマット: "\x1b[90m  N\x1b[m  " (ANSIエスケープ付き)
+            const line_num_byte_end = if (line_num_display_width > 0) blk: {
+                // 行番号部分のバイト終端を探す（最初のリセットエスケープ \x1b[m の後のスペース2つ）
+                if (std.mem.indexOf(u8, new_line, "\x1b[m  ")) |pos| {
+                    break :blk pos + 5; // "\x1b[m  ".len
                 }
-            }
-
-            // 長さが違う場合は後ろも差分
-            if (old_line.len != new_line.len) {
-                if (diff_start == null) diff_start = min_len;
-                diff_end = @max(old_line.len, new_line.len);
-            }
-
-            if (diff_start) |start_raw| {
-                // 行番号の表示幅を取得
-                const line_num_display_width = self.getLineNumberWidth();
-
-                // 行番号がある場合、行番号部分のバイト範囲を推定してスキップ
-                // 行番号フォーマット: "\x1b[90m  N\x1b[m  " (ANSIエスケープ付き)
-                // 行番号がない、または差分が行番号より後から始まる場合は部分更新が可能
-                const line_num_byte_end = if (line_num_display_width > 0) blk: {
-                    // 行番号部分のバイト終端を探す（最初のリセットエスケープ \x1b[m の後のスペース2つ）
-                    if (std.mem.indexOf(u8, new_line, "\x1b[m  ")) |pos| {
-                        break :blk pos + 5; // "\x1b[m  ".len
-                    }
-                    // フォールバック：行全体を再描画
-                    break :blk 0;
-                } else 0;
-
-                // 差分が行番号部分にかかっている場合は行全体を再描画
-                if (line_num_display_width > 0 and start_raw < line_num_byte_end) {
-                    // 行頭から全体を再描画
+                // フォールバック：行全体を再描画
+                break :blk 0;
+            } else 0;
+
+            // シンタックスハイライト（ANSIエスケープ）の変化をチェック
+            // コメント色のANSIコードがある場合は行全体を再描画（差分描画ではカラーが正しく適用されない）
+            const old_has_gray = hasAnsiGray(old_line, line_num_byte_end);
+            const new_has_gray = hasAnsiGray(new_line, line_num_byte_end);
+            // グレーがある場合は行全体を再描画（差分描画だとカラーが途切れる問題を回避）
+            if (old_has_gray or new_has_gray) {
+                // 内容が異なる場合のみ再描画
+                if (!std.mem.eql(u8, old_line, new_line)) {
                     try term.moveCursor(abs_row, viewport_x);
                     try term.write(new_line);
-                    // 残りをスペースで埋める（CLEAR_LINEの代わり）
                     try self.padToWidth(term, new_line, viewport_width);
-                } else if (line_num_display_width > 0) {
-                    // 行番号があるが本文部分のみの差分
-                    // UTF-8文字境界に調整
-                    var start = start_raw;
-                    while (start > 0 and start < new_line.len and unicode.isUtf8Continuation(new_line[start])) {
-                        start -= 1;
+                }
+            } else {
+                // 差分を検出
+                const min_len = @min(old_line.len, new_line.len);
+                var diff_start: ?usize = null;
+                var diff_end: usize = 0;
+
+                // 前方から差分を探す
+                for (0..min_len) |i| {
+                    if (old_line[i] != new_line[i]) {
+                        if (diff_start == null) diff_start = i;
+                        diff_end = i + 1;
                     }
+                }
 
-                    // バイト位置から画面カラム位置を計算（行番号部分をスキップ）
-                    const screen_col = calculateScreenColumn(new_line, line_num_byte_end, start, line_num_display_width);
-
-                    // 差分部分のみ描画
-                    try term.moveCursor(abs_row, viewport_x + screen_col);
-                    try term.write(new_line[start..]);
+                // 長さが違う場合は後ろも差分
+                if (old_line.len != new_line.len) {
+                    if (diff_start == null) diff_start = min_len;
+                    diff_end = @max(old_line.len, new_line.len);
+                }
 
-                    // 古い行の方が長い場合は残りをスペースで埋める
-                    if (old_line.len > new_line.len) {
+                if (diff_start) |start_raw| {
+                    // 差分が行番号部分にかかっている場合は行全体を再描画
+                    if (line_num_display_width > 0 and start_raw < line_num_byte_end) {
+                        // 行頭から全体を再描画
+                        try term.moveCursor(abs_row, viewport_x);
+                        try term.write(new_line);
+                        // 残りをスペースで埋める（CLEAR_LINEの代わり）
                         try self.padToWidth(term, new_line, viewport_width);
-                    }
-                } else {
-                    // UTF-8文字境界に調整（継続バイトの途中から始まらないように）
-                    var start = start_raw;
-                    while (start > 0 and start < new_line.len and unicode.isUtf8Continuation(new_line[start])) {
-                        start -= 1;
-                    }
+                    } else if (line_num_display_width > 0) {
+                        // 行番号があるが本文部分のみの差分
+                        // UTF-8文字境界に調整
+                        var start = start_raw;
+                        while (start > 0 and start < new_line.len and unicode.isUtf8Continuation(new_line[start])) {
+                            start -= 1;
+                        }
 
-                    // バイト位置から画面カラム位置を計算（ANSIエスケープシーケンスをスキップ）
-                    const screen_col = calculateScreenColumn(new_line, 0, start, 0);
+                        // バイト位置から画面カラム位置を計算（行番号部分をスキップ）
+                        const screen_col = calculateScreenColumn(new_line, line_num_byte_end, start, line_num_display_width);
 
-                    // 差分部分のみ描画（viewport_xオフセットを加算）
-                    try term.moveCursor(abs_row, viewport_x + screen_col);
-                    try term.write(new_line[start..]);
+                        // 差分部分のみ描画
+                        try term.moveCursor(abs_row, viewport_x + screen_col);
+                        try term.write(new_line[start..]);
 
-                    // 古い行の方が長い場合は残りをスペースで埋める
-                    if (old_line.len > new_line.len) {
-                        try self.padToWidth(term, new_line, viewport_width);
+                        // 古い行の方が長い場合は残りをスペースで埋める
+                        if (old_line.len > new_line.len) {
+                            try self.padToWidth(term, new_line, viewport_width);
+                        }
+                    } else {
+                        // UTF-8文字境界に調整（継続バイトの途中から始まらないように）
+                        var start = start_raw;
+                        while (start > 0 and start < new_line.len and unicode.isUtf8Continuation(new_line[start])) {
+                            start -= 1;
+                        }
+
+                        // バイト位置から画面カラム位置を計算（ANSIエスケープシーケンスをスキップ）
+                        const screen_col = calculateScreenColumn(new_line, 0, start, 0);
+
+                        // 差分部分のみ描画（viewport_xオフセットを加算）
+                        try term.moveCursor(abs_row, viewport_x + screen_col);
+                        try term.write(new_line[start..]);
+
+                        // 古い行の方が長い場合は残りをスペースで埋める
+                        if (old_line.len > new_line.len) {
+                            try self.padToWidth(term, new_line, viewport_width);
+                        }
                     }
                 }
             }