Skip to content

rag-cli: ensure chat_engine validator runs#20833

Open
noor-malaika wants to merge 1 commit intorun-llama:mainfrom
noor-malaika:bugfix/rag-cli-chat-failed
Open

rag-cli: ensure chat_engine validator runs#20833
noor-malaika wants to merge 1 commit intorun-llama:mainfrom
noor-malaika:bugfix/rag-cli-chat-failed

Conversation

@noor-malaika
Copy link

@noor-malaika noor-malaika commented Mar 1, 2026

Description

Add validate_default=True to chat_engine Field so the
@field_validator(..., mode="before") fires even when the default is None.

Fixes # (20783)

Version Bump?

Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

  • Yes
  • No

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Mar 1, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 2, 2026
@AstraBert
Copy link
Member

You need to bump the version of the CLI before merging so that it will be published

@noor-malaika noor-malaika force-pushed the bugfix/rag-cli-chat-failed branch from 74d5d34 to 93e0672 Compare March 2, 2026 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants