Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add fzf-based model autocomplete to duckdbt #524

Merged
merged 3 commits into from
Mar 31, 2025
Merged

Conversation

jwills
Copy link
Collaborator

@jwills jwills commented Mar 22, 2025

Had Claude take a crack at this based on feedback from @b-per ; the duckdbt CLI will now auto-complete models using fzf if iterfzf is installed in the python environment.

(We update the cache of models in the project based on the manifest whenever parse is run)

jwills and others added 3 commits March 22, 2025 08:36
Store the manifest object in the DuckdbtShell class to enable future
features that need access to project information like model names.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Implemented a model name autocomplete feature that activates when
typing '**' and pressing TAB. This allows users to quickly select
models from the project. The feature:

- Requires iterfzf to be installed
- Caches model names from the manifest
- Shows a helpful message if iterfzf is not available
- Filters models based on text before the ** marker

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link
Collaborator

@guenp guenp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks vibe Claude!
looks good to me

@jwills jwills merged commit c86ef32 into master Mar 31, 2025
33 checks passed
@jwills jwills deleted the jwills_fzf_duckdbt branch March 31, 2025 18:17
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.

2 participants