Fix: add back default local repo type in ChatCompletionRequest#556
Fix: add back default local repo type in ChatCompletionRequest#556GdoongMathew wants to merge 2 commits into
local repo type in ChatCompletionRequest#556Conversation
There was a problem hiding this comment.
Code Review
This pull request adds "local" to the allowed repository types in the ChatCompletionRequest model. The review feedback suggests updating the field's description to include "local" to ensure the documentation remains accurate.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| filePath: Optional[str] = Field(None, description="Optional path to a file in the repository to include in the prompt") | ||
| token: Optional[str] = Field(None, description="Personal access token for private repositories") | ||
| type: Optional[Literal["github", "gitlab", "bitbucket"]] = Field( | ||
| type: Optional[Literal["local", "github", "gitlab", "bitbucket"]] = Field( |
There was a problem hiding this comment.
The repository type "local" has been added to the type field's allowed Literal values, but the field's description on line 21 has not been updated to reflect this change.\n\nTo ensure the generated OpenAPI/Swagger documentation is accurate and complete, please update the description to include 'local'.\n\nFor example:\npython\n type: Optional[Literal["local", "github", "gitlab", "bitbucket"]] = Field(\n "github",\n description="Type of repository (e.g., 'local', 'github', 'gitlab', 'bitbucket')",\n )\n
Summary
in #550, I missed adding
localrepo type inChatCompletionRequest. This would cause pydantic validation error when sending request for indexing a new local repo.