Improve select prompt (with search)

Hi,

I’ve finally found the time to experiment with the select prompt with auto-complete and here is the feedback.

Ctrl-C keeps the console dirty

When someone ctrl-c’s instead of validating the prompt, the options stay displayed and clutter the terminal.

If it is possible, it’d be nice to catch the SIGINT signal and cleanup.

Less-strict search method

The current search method shows a few improvement opportunities IMO:

• it currently only matches the values. This has the drawback that when searching in options with names, the values are not displayed and may not share a single character in common with the values (ex: {name: 'Blue', value: '#0000AA'}).
• is searches at the beginning of the value and not everywhere (meaning new does not match Add a new feature).

I think it would be better to search on names when provided and to use a case-insensitive-match-anywhere search method.

Something like:

function searchMethod (allOptions, searchText) {
  return allOptions
    .filter(option => !option.disabled)
    .reduce(
      (seed, option) => {
        const lowerCaseName = option.name.toLocaleLowerCase();
        const lowerCaseValue = option.value.toLocaleLowerCase();
        const lowerCaseSearch = searchText.toLocaleLowerCase();
        
        const matchesPerfectly = lowerCaseName === lowerCaseSearch || lowerCaseValue === lowerCaseSearch;
        if (matchesPerfectly) { // Put perfect match first!
          return [ option, ...seed ];
        }
        
        const matches = lowerCaseName.includes(lowerCaseSearch) || lowerCaseValue.includes(lowerCaseSearch);
        return matches ? [ ...seed, option ] : seed;
      },
      [],
    );
}

WDYT?

I can help with implementation if you want to take these, and can split issues etc… 😃

[quilicicf]

Noice! I’ve tested it and it works on commit a6b5d36 thanks 😃

[c4spar]

@quilicicf the issue with the stdout should be fixed now. i realized, there was realy no logic for clearing the stdout. i think, it was just my zsh and docker which cleared the stdout automatically. With the bash shell i got the same issue.