Skip to content

completion: Use same config as list-topics for topic completion - #288

Merged
jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fixcomp
Sep 11, 2026
Merged

jerry-skydio merged 1 commit into
mainfrom
jerry/revup/main/fixcomp

Conversation

@jerry-skydio

Copy link
Copy Markdown
Collaborator

Tab completion built git with hardcoded values (origin/main, empty base
branch globs) instead of the repo config, so it picked the wrong base
branch and surfaced thousands of stale historical topics. Two fixes:

  • Apply config to the parsers before argcomplete runs, so completers see
    the same resolved settings normal commands do.
  • topic_completer now builds git and topics via make_git/get_topics
    exactly as list-topics does, from the parsed namespace.

Consolidate get_git into make_git (now takes the namespace directly),
which also breaks the completion -> revup cyclic import.

@jerry-skydio

Copy link
Copy Markdown
Collaborator Author

Reviews in this chain:
└#288 completion: Use same config as list-topics for topic completion

@jerry-skydio

jerry-skydio commented Aug 7, 2026 •

Copy link
Copy Markdown
Collaborator Author
# head base diff date summary
0 63a0e0d4 69101c6b diff Aug 7 14:50 PM 3 files changed, 35 insertions(+), 41 deletions(-)
1 28fc590d 15963e0a diff Sep 10 17:29 PM 1 file changed, 2 insertions(+), 2 deletions(-)

Comment thread revup/completion.py Outdated
Tab completion built git with hardcoded values (origin/main, empty base
branch globs) instead of the repo config, so it picked the wrong base
branch and surfaced thousands of stale historical topics. Two fixes:

- Apply config to the parsers before argcomplete runs, so completers see
  the same resolved settings normal commands do.
- topic_completer now builds git and topics via make_git/get_topics
  exactly as list-topics does, from the parsed namespace.

Consolidate get_git into make_git (now takes the namespace directly),
which also breaks the completion -> revup cyclic import.
@jerry-skydio
jerry-skydio force-pushed the jerry/revup/main/fixcomp branch from 63a0e0d to 28fc590 Compare September 11, 2026 00:29
@jerry-skydio
jerry-skydio merged commit e5d0826 into main Sep 11, 2026
6 checks passed
@jerry-skydio
jerry-skydio deleted the jerry/revup/main/fixcomp branch September 11, 2026 00:30
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.

3 participants