Skip to content

Add FT.ALIASLIST search command support - #518

Open
uglide wants to merge 6 commits into
masterfrom
im/8_10-ft-alias-list
Open

Add FT.ALIASLIST search command support#518
uglide wants to merge 6 commits into
masterfrom
im/8_10-ft-alias-list

Conversation

@uglide

@uglide uglide commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Add support for the FT.ALIASLIST command (Query Engine 8.10), which lists all aliases associated with a given index.

  • Add ALIASLIST constant and SearchCommandBuilder.AliasList(index)
  • Expose AliasList / AliasListAsync on ISearchCommands and ISearchCommandsAsync (returning RedisResult[], matching _List/TagVals)
  • Register the new public API surface in PublicAPI.Unshipped.txt
  • Add sync and async integration tests (guarded to Redis 8.10+)

Note

Low Risk
Additive search API and tests only; no changes to auth, data paths, or existing command behavior.

Overview
Adds FT.ALIASLIST (Query Engine 8.10+) so callers can list all aliases bound to a search index name.

AliasList / AliasListAsync are exposed on the search command interfaces and implementations, wired through SearchCommandBuilder.AliasList and returning RedisResult[] (same shape as _List / TagVals). The new public API is registered in PublicAPI.Unshipped.txt.

Integration tests cover empty lists, multiple aliases after add/del, and server errors for missing indexes, gated to Redis 8.10+. Existing alias tests reuse shared helpers that skip on legacy cluster CROSSSLOT failures from FT.ALIASADD.

Reviewed by Cursor Bugbot for commit 37c11fa. Bugbot is set up for automated code reviews on this repo. Configure here.

uglide added 2 commits July 14, 2026 17:26
Add support for the FT.ALIASLIST command (Query Engine 8.10), which
lists all aliases associated with a given index.

- Add ALIASLIST constant and SearchCommandBuilder.AliasList(index)
- Expose AliasList / AliasListAsync on ISearchCommands and
  ISearchCommandsAsync (returning RedisResult[], matching _List/TagVals)
- Register the new public API surface in PublicAPI.Unshipped.txt
- Add sync and async integration tests (guarded to Redis 8.10+)

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit d536e0d. Configure here.

Comment thread tests/NRedisStack.Tests/Search/SearchTests.cs
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