Skip to content

Only ignore min completion trigger for /command term itself#2059

Merged
rolandwalker merged 1 commit into
mainfrom
RW/completion-trigger-command-finesse
Jul 25, 2026
Merged

Only ignore min completion trigger for /command term itself#2059
rolandwalker merged 1 commit into
mainfrom
RW/completion-trigger-command-finesse

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Ignoring min_completion_trigger when the line starts with forward slash helps prompt for possible commands, but the previous implementation suppressed the setting throughout the entire line, applying also to the subcommands and further arguments of /command.

Instead, the user's setting should be generally respected, and ignored only for the first term, when /command is typed.

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 .

@rolandwalker rolandwalker self-assigned this Jul 25, 2026
@rolandwalker
rolandwalker force-pushed the RW/completion-trigger-command-finesse branch from 6c9c2a4 to ec90be5 Compare July 25, 2026 18:45
Ignoring min_completion_trigger when the line starts with forward slash
helps prompt for possible commands, but the previous implementation
suppressed the setting throughout the entire line, applying also to the
subcommands and further arguments of /command.

Instead, the user's setting should be generally respected, and ignored
_only_ for the first term, when /command is typed.
@rolandwalker
rolandwalker force-pushed the RW/completion-trigger-command-finesse branch from ec90be5 to cd1298c Compare July 25, 2026 18:48
@rolandwalker
rolandwalker merged commit 428a314 into main Jul 25, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/completion-trigger-command-finesse branch July 25, 2026 18:54
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