Add a changed-files review index for PR review #83

New issue

Closed

opened 2026-04-07 17:04:12 +00:00 by barrettruth · 1 comment

barrettruth commented

2026-04-07 17:04:12 +00:00

Summary

A local review workflow needs a changed-files backbone. Right now forge review leans on the diff tooling's side effects instead of presenting changed files as a first-class review surface.

Goals

land PR review in a changed-files index
keep the index picker-agnostic
make file-to-file navigation explicit rather than incidental

Scope

derive changed files for the current PR review subject
present them as a first-class review index/picker
support opening the selected file in the current review mode
keep the implementation independent from any one picker backend

Out of scope

branch/commit review subjects
full file/hunk navigation model
worktree materialization changes

Part of #63.

## Summary A local review workflow needs a changed-files backbone. Right now forge review leans on the diff tooling's side effects instead of presenting changed files as a first-class review surface. ## Goals - land PR review in a changed-files index - keep the index picker-agnostic - make file-to-file navigation explicit rather than incidental ## Scope - derive changed files for the current PR review subject - present them as a first-class review index/picker - support opening the selected file in the current review mode - keep the implementation independent from any one picker backend ## Out of scope - branch/commit review subjects - full file/hunk navigation model - worktree materialization changes Part of #63.

barrettruth commented

2026-04-10 22:00:33 +00:00

Implemented on main in #90 with a first-class changed-files review index.