Skip to content

Include keywords in completion candidates#681

Merged
vinistock merged 1 commit intomainfrom
03-19-include_keywords_in_completion_candidates
Mar 23, 2026
Merged

Include keywords in completion candidates#681
vinistock merged 1 commit intomainfrom
03-19-include_keywords_in_completion_candidates

Conversation

@vinistock
Copy link
Copy Markdown
Member

@vinistock vinistock commented Mar 19, 2026

Start including keywords as part of expression completion.

Note: there are certain scenarios where a keyword isn't valid. We don't currently try to filter those out.

Copy link
Copy Markdown
Member Author

vinistock commented Mar 19, 2026

@vinistock vinistock mentioned this pull request Mar 19, 2026
@vinistock vinistock self-assigned this Mar 19, 2026
@vinistock vinistock force-pushed the 03-19-include_keywords_in_completion_candidates branch from 39d3a9e to 3d8dedd Compare March 19, 2026 20:04
@vinistock vinistock added the enhancement New feature or request label Mar 19, 2026
@vinistock vinistock marked this pull request as ready for review March 19, 2026 20:05
@vinistock vinistock requested a review from a team as a code owner March 19, 2026 20:05
@vinistock vinistock force-pushed the 03-19-add_keywords_to_model branch from d2fea3b to 95a68ed Compare March 20, 2026 19:48
@vinistock vinistock force-pushed the 03-19-include_keywords_in_completion_candidates branch from 3d8dedd to f925fef Compare March 20, 2026 19:48
Copy link
Copy Markdown
Contributor

@alexcrocha alexcrocha left a comment

Choose a reason for hiding this comment

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

Note: there are certain scenarios where a keyword isn't valid. We don't currently try to filter those out.

Piqued my attention, curious about intent. Is this something we'd want to tackle in Rubydex later, or are we leaving it for the consumer to handle?

Base automatically changed from 03-19-add_keywords_to_model to main March 23, 2026 13:43
@vinistock
Copy link
Copy Markdown
Member Author

That's a good question. It might be better suited for the consumer to filter out which keywords don't apply since they have context of the surrounding code.

I haven't really explored this in much detail, so I don't yet know if it would be feasible to filter it out on the Rubydex side.

@vinistock vinistock merged commit 97ddff9 into main Mar 23, 2026
47 checks passed
@vinistock vinistock deleted the 03-19-include_keywords_in_completion_candidates branch March 23, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants