Skip to content

feat(review): review any source: stack, uncommitted, ref, range, or PR - #83

Draft
lettertwo wants to merge 24 commits into
review/05-config-themefrom
review/06-any-source
Draft

feat(review): review any source: stack, uncommitted, ref, range, or PR#83
lettertwo wants to merge 24 commits into
review/05-config-themefrom
review/06-any-source

Conversation

@lettertwo

@lettertwo lettertwo commented Sep 4, 2026

Copy link
Copy Markdown
Owner

This PR adds the source argument: git workon review [stack | uncommitted | <ref> | <a..b> | <pr>]. Resolution follows ADR-036 (source grammar): one sniffed positional, keyword-over-ref precedence, git-diff dot semantics for ranges, gh-backed PR resolution, and failure before terminal takeover. Refresh re-resolves the launched source and no-ops for a PR.

Outside the crate: git-workon-lib renames ChangesetSource to ChangesetSpan and fetches a PR's head and base fresh in pr.rs; git-workon gains source completion for review and delegates completion to known responder binaries with stdin nulled.

The tail of the PR is the first responsiveness work: buffered nav input is coalesced, changeset diffs are acquired in parallel, file loads defer to an input-idle window, and a splash shows before acquisition. A PTY harness pins launch and nav-burst responsiveness.

Execution plan: docs/plans/review-any-source.md.

@lettertwo lettertwo changed the title review/06 any source feat(review): review any source: stack, uncommitted, ref, range, or PR Sep 4, 2026
@lettertwo
lettertwo force-pushed the review/06-any-source branch from 46358a6 to eec964f Compare September 4, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant