Skip to content

Fix pathname completions not descending into directories - #2211

Merged
rolandwalker merged 1 commit into
mainfrom
RW/fix-broken-file-completions
Sep 7, 2026
Merged

Fix pathname completions not descending into directories#2211
rolandwalker merged 1 commit into
mainfrom
RW/fix-broken-file-completions

Conversation

@rolandwalker

Copy link
Copy Markdown
Contributor

Description

Make pathname completions as for eg /source much better

  • Fix a bug in which completions would complete a directory, but not then generate candidates on files from within that directory.
  • Fix a bug in which completions were not generated if the pathname started with ~/ or /.

Limitations and future work: this is perfect for the usecase of /source, but other usecases such as /tee and /once should not be limited to completing on pathnames ending with .sql.

Example which previously did not show completions:

last image

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 .

Make pathname completions as for _eg_ /source much better

 * fix a bug in which completions would complete a directory, but not
   then generate candidates on files from within that directory
 * fix a bug in which completions were not generated if the pathname
   started with "~/" or "/".

Limitations and future work: this is perfect for the usecase of /source,
but other usecases such as /tee and /once should not be limited to
completing on pathnames ending with ".sql".
@rolandwalker rolandwalker self-assigned this Sep 7, 2026
@rolandwalker
rolandwalker merged commit 1616e54 into main Sep 7, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/fix-broken-file-completions branch September 7, 2026 16:59
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