Skip to content

Python: fix(redis): pass collection prefix as a list#14012

Open
MukundaKatta wants to merge 1 commit into
microsoft:mainfrom
MukundaKatta:codex/semantic-kernel-redis-prefix
Open

Python: fix(redis): pass collection prefix as a list#14012
MukundaKatta wants to merge 1 commit into
microsoft:mainfrom
MukundaKatta:codex/semantic-kernel-redis-prefix

Conversation

@MukundaKatta
Copy link
Copy Markdown

No description provided.

@moonbox3 moonbox3 added the python Pull requests for the Python Semantic Kernel label May 15, 2026
@github-actions github-actions Bot changed the title fix(redis): pass collection prefix as a list Python: fix(redis): pass collection prefix as a list May 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes Redis search index creation by passing the IndexDefinition.prefix as a list (instead of a string), matching the expected Redis client API and preventing incorrect prefix handling.

Changes:

  • Update Redis connector index creation to pass prefix=[f"{collection_name}:"] to IndexDefinition.
  • Update the existing unit test that creates an IndexDefinition manually to use a list prefix.
  • Add a unit test asserting ensure_collection_exists() constructs IndexDefinition with a single-item prefix list for both HASH and JSON collections.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
python/semantic_kernel/connectors/redis.py Passes collection prefix to IndexDefinition as a list during index creation.
python/tests/unit/connectors/memory/test_redis_store.py Updates manual index-definition test and adds coverage ensuring list-prefix behavior for both collection types.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants