Skip to content

Commit cd2898b

Browse files
committed
docs: update docs for line number gutter and ]c/[c change jump
Update requirements.md, roadmap.md, README.md, and design doc to reflect the implemented line number gutter and ]c/[c change line jump features. https://claude.ai/code/session_015NbnHYsXMH45WpAakjZoki
1 parent 37a8fc2 commit cd2898b

4 files changed

Lines changed: 25 additions & 12 deletions

File tree

README.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ review-codecommit lets you browse AWS CodeCommit repositories, view pull request
1414
- Browse pull requests with status filter (Open / Closed / Merged, `f` key cycle)
1515
- Search pull requests by title or author (`/` key)
1616
- Paginate through PR lists (`n`/`p` keys)
17-
- View PR details with color-coded unified diffs (green for additions, red for deletions)
17+
- View PR details with color-coded unified diffs (green for additions, red for deletions) with line number gutter
1818
- Read PR comments inline
1919
- Post comments on pull requests (`c` key in PR detail view)
2020
- Post inline comments on specific diff lines (`C` key at cursor position)
@@ -32,6 +32,7 @@ review-codecommit lets you browse AWS CodeCommit repositories, view pull request
3232
- Close PRs without merging (`x` key)
3333
- Commit-level review with Tab/Shift+Tab navigation between "All changes" and individual commits
3434
- Cursor-based diff navigation with `>` marker
35+
- Jump to next/previous change line with `]c`/`[c` (Vim diff-style)
3536
- View PR activity timeline (created, approved, merged, status changes, etc.) (`A` key)
3637
- Shell completion for bash, zsh, and fish (`--completions` option)
3738
- Vim-style keybindings (`j`/`k` navigation)
@@ -107,6 +108,9 @@ Completions support:
107108
| `Ctrl+d` | Half page down | PR Detail |
108109
| `Ctrl+u` | Half page up | PR Detail |
109110
| `G` | Jump to end | PR Detail |
111+
| `n` / `N` | Next / previous file | PR Detail |
112+
| `]c` | Jump to next change (add/delete line) | PR Detail |
113+
| `[c` | Jump to previous change (add/delete line) | PR Detail |
110114
| `Enter` | Select / confirm / submit comment | List screens / Comment input |
111115
| `q` / `Esc` | Go back / cancel | All / Comment input / Confirm prompt |
112116
| `c` | Post comment | PR Detail |

docs/design-review-readability.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# レビュー画面の可読性改善設計書
22

3-
> **ステータス**: 設計完了(2026-04-09
3+
> **✅ 実装完了** (2026-04-09)
44
>
55
> PR詳細画面のdiff表示に行番号ガターと変更行ジャンプ(`]c`/`[c`)を追加し、コードレビュー体験を改善する。
66

docs/requirements.md

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,10 @@ npx review-codecommit --completions <shell> # シェル補完スクリプト生
170170
| `Ctrl+d` | 半ページ下スクロール | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
171171
| `Ctrl+u` | 半ページ上スクロール | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
172172
| `G` | 最終行へジャンプ | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
173+
| `n` | 次のファイルへジャンプ | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
174+
| `N` | 前のファイルへジャンプ | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
175+
| `]c` | 次の変更行(add/delete)へジャンプ | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
176+
| `[c` | 前の変更行(add/delete)へジャンプ | PR詳細画面(コメント入力中・確認プロンプト中は無効) |
173177
| `Enter` | 選択・決定 / コメント送信 | リスト画面 / コメント入力 |
174178
| `q` / `Esc` | 前の画面に戻る / コメント入力キャンセル | 全画面 / コメント入力 |
175179
| `Ctrl+C` | 即座に終了 | 全画面 |
@@ -258,12 +262,12 @@ npx review-codecommit --completions <shell> # シェル補完スクリプト生
258262
│ src/auth.ts │
259263
│ │
260264
│ @@ -15,7 +15,7 @@ │
261-
const config = {
262-
│ > - timeout: 3000,
265+
15 15 │ const config = {
266+
│ > 16 │ - timeout: 3000,
263267
│ 💬 taro: この値はconfigから取る方が良さそう │
264268
│ └ watany: 次のPRで修正します │
265-
+ timeout: 10000,
266-
};
269+
16 │ + timeout: 10000,
270+
17 17 │ };
267271
│ │
268272
│──────────────────────────────────────────────│
269273
│ Comments (3): │
@@ -282,6 +286,7 @@ npx review-codecommit --completions <shell> # シェル補完スクリプト生
282286

283287
- **形式**: unified diff (`git diff` 同等)
284288
- **色付け**: 追加行=緑、削除行=赤、コンテキスト行=デフォルト
289+
- **行番号ガター**: 変更前/変更後の行番号を各4桁右寄せで表示(例: ` 3 4 │ `)。add/delete/context行のみ。dimColorで本文色と分離
285290
- **構造**: ファイル単位でセクション分け
286291

287292
## ページネーション

docs/roadmap.md

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -104,17 +104,21 @@ PR レビューの完全なワークフロー(閲覧→コメント→承認
104104

105105
#### 機能
106106

107-
| 機能 | 内容 |
108-
|------|------|
109-
| シンタックスハイライト | diff 内のコードをファイル拡張子に応じて色付け表示 |
110-
| ファイルツリー | PR 内の変更ファイルをツリー表示し、ファイル間をジャンプ(`t` キー) |
111-
| diff 統計 | ファイル別・合計の追加行/削除行数を表示 |
112-
| ファイルジャンプ | ファイルツリーで選択したファイルの diff 位置へ直接移動 |
107+
| 機能 | 内容 | 状態 |
108+
|------|------|------|
109+
| 行番号ガター | diff の add/delete/context 行に変更前/変更後の行番号を表示(` 3 4 │ ` 形式) ||
110+
| 変更行ジャンプ | `]c` で次の変更行、`[c` で前の変更行へジャンプ(Vim diff 風 2キーシーケンス) ||
111+
| シンタックスハイライト | diff 内のコードをファイル拡張子に応じて色付け表示 | |
112+
| ファイルツリー | PR 内の変更ファイルをツリー表示し、ファイル間をジャンプ(`t` キー) | |
113+
| diff 統計 | ファイル別・合計の追加行/削除行数を表示 | |
114+
| ファイルジャンプ | ファイルツリーで選択したファイルの diff 位置へ直接移動 | |
113115

114116
#### キーバインド追加
115117

116118
| キー | 動作 | 画面 |
117119
|------|------|------|
120+
| `]c` | 次の変更行(add/delete)へジャンプ | PR 詳細 |
121+
| `[c` | 前の変更行(add/delete)へジャンプ | PR 詳細 |
118122
| `t` | ファイルツリー表示 | PR 詳細 |
119123

120124
#### 設計上の考慮点

0 commit comments

Comments
 (0)