refactor(pickers): split PR and issue surfaces out of pickers.lua #528

New issue

Closed

opened 2026-04-29 02:09:38 +00:00 by barrettruth · 0 comments

barrettruth commented

2026-04-29 02:09:38 +00:00

lua/forge/pickers.lua currently contains the full PR and issue picker surfaces inline.

That includes:

M.pr()
M.issue()
PR/issue-specific header ordering and action wiring
close / reopen helpers
PR action entry helpers that still live beside unrelated picker code

This issue is for moving the PR and issue picker surfaces into dedicated modules while preserving the current interactive behavior.

A good end state is that the top-level picker module mainly routes into focused surface implementations instead of owning the full PR and issue UI flows directly.

`lua/forge/pickers.lua` currently contains the full PR and issue picker surfaces inline. That includes: - `M.pr()` - `M.issue()` - PR/issue-specific header ordering and action wiring - close / reopen helpers - PR action entry helpers that still live beside unrelated picker code This issue is for moving the PR and issue picker surfaces into dedicated modules while preserving the current interactive behavior. A good end state is that the top-level picker module mainly routes into focused surface implementations instead of owning the full PR and issue UI flows directly.