Skip to content

feat(autocomplete): do not set default filter for options getters#496

Merged
43081j merged 1 commit intomainfrom
double-tap-without-bots
Mar 23, 2026
Merged

feat(autocomplete): do not set default filter for options getters#496
43081j merged 1 commit intomainfrom
double-tap-without-bots

Conversation

@43081j
Copy link
Collaborator

@43081j 43081j commented Mar 23, 2026

Basically:

  • If options is a function, do not set the default filter
  • If a filter is explicitly set, keep it
  • If options is not a function, set the default filter

Closes #488.
Closes #489.

Basically:

- If `options` is a function, do not set the default filter
- If a filter is explicitly set, keep it
- If `options` is not a function, set the default filter
@changeset-bot
Copy link

changeset-bot bot commented Mar 23, 2026

🦋 Changeset detected

Latest commit: d54e568

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@clack/core Patch
@clack/prompts Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 23, 2026

commit: d54e568

@43081j
Copy link
Collaborator Author

43081j commented Mar 23, 2026

@dreyfus92 the other PR went stale and got flagged as bot-like, which may explain the staleness

either way, here it is from scratch so we can get it landed

Copy link
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

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

nice, awesome work @43081j 😄

@43081j 43081j merged commit 417b451 into main Mar 23, 2026
9 checks passed
@43081j 43081j deleted the double-tap-without-bots branch March 23, 2026 23:58
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.

[Bug] filter + options() double-filtering issue for autocomplete

2 participants