drush search-api-reindex should use modulename:command format

Thanks for creating this issue, and sorry it took me so long to respond.
It seems the command name already is namespaced properly, and that aliases generally aren’t, as far as I can see. Therefore, I don’t really see the benefit of adding another alias, same as the old only with a colon in place of one dash. Or is there some documentation, or some examples, that state that aliases should also use the colon namespacing?

-   * @aliases search-api-mark-all,search-api-reindex,sapi-r,search-api-reset-tracker
+   * @aliases search-api-mark-all,search-api:reindex,search-api-reindex,sapi-r,search-api-reset-tracker

This adds a new alias, and does nothing else.

Aliases have two uses:

1. To shorten command names for easier typing.
2. To provide backwards compatibility for old-style command names and previous command names that people might have gotten used to using.

The second use-case is the most important IMO.

The actual command here is search-api:reset-tracker, and that's in the proper form and that's what shows up if you list the commands using drush --filter=search-api.

The search-api-reindex alias is for backwards compatibility with the Drupal 7 search-api-reindex Drush command.

In my opinion we don't need a new alias that "looks like" a top-level command name. There are plenty of aliases for this command. If you were used to using search-api-reindex in D7 that will still work, as designed. Moving forward, you can keep using that or switch to the the current command (not alias) which is search-api:reset-tracker

I recommend closing this as "works as designed" or "won't fix".

I agree, thanks a lot for weighing in!