greview: improve missing-ref error handling #131

New issue

Closed

opened 2026-09-21 18:12:26 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-09-21 18:12:26 +00:00

Owner

Original issue: barrettruth/diffs.nvim#297
Original author: barrettruth
Original date: 2026-05-10T17:02:43Z

Problem

:Greview has poor error handling and user-facing messages when the requested ref or review behavior does not exist. The failure should be caught and explained clearly instead of surfacing confusing Git/ref-resolution behavior.

Expected

Detect missing or invalid refs before trying to render the review workspace.
Include the ref/behavior that failed in the message.
Keep the message actionable and concise.
Avoid stack traces or generic low-context Git output for normal missing-ref mistakes.

Scope

This is about :Greview ref/behavior resolution and error messaging only, not the split :Gdiff work.

> Original issue: barrettruth/diffs.nvim#297 > Original author: `barrettruth` > Original date: 2026-05-10T17:02:43Z ## Problem `:Greview` has poor error handling and user-facing messages when the requested ref or review behavior does not exist. The failure should be caught and explained clearly instead of surfacing confusing Git/ref-resolution behavior. ## Expected - Detect missing or invalid refs before trying to render the review workspace. - Include the ref/behavior that failed in the message. - Keep the message actionable and concise. - Avoid stack traces or generic low-context Git output for normal missing-ref mistakes. ## Scope This is about `:Greview` ref/behavior resolution and error messaging only, not the split `:Gdiff` work.

barrettruth

2026-09-21 18:12:26 +00:00