Skip to content

Do not apply omitted completion-generation methods - #2235

Merged
rolandwalker merged 1 commit into
mainfrom
RW/do-not-apply-omitted-completion-candidate-methods
Sep 12, 2026
Merged

Do not apply omitted completion-generation methods#2235
rolandwalker merged 1 commit into
mainfrom
RW/do-not-apply-omitted-completion-candidate-methods

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Previously, if a completion-generation method was omitted from the completion_match_order configuration value, that method was merely demoted in the priority.

With this change, is a method is omitted, then completions based on that method are not generated, and do not appear in the choices visible to the user.

Limitation: filename completions don't work well unless perfect is in the match-order list, and preferably slash_words as well. But realistically, would anyone configure this such that "perfect" matches were omitted?

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

Previously, if a completion-generation method was omitted from the
completion_match_order configuration value, that method was merely
demoted in the priority.

With this change, is a method is omitted, then completions based on
that method are not generated, and do not appear in the choices visible
to the user.

Limitation: filename completions don't work well unless "perfect" is in
the match-order list, and preferably "slash_words" as well.  But
realistically, would anyone configure this such that "perfect" matches
were omitted?
@rolandwalker rolandwalker self-assigned this Sep 12, 2026
@rolandwalker
rolandwalker merged commit 44bb532 into main Sep 12, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/do-not-apply-omitted-completion-candidate-methods branch September 12, 2026 18:52
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