Collection: Adding input parameter "provider"

[nishika26]

Summary

Target issue is #490

Checklist

Before submitting a pull request, please ensure that you mark these task.

• Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.

Notes

• Just adding an input parameter, "provider" as we will be extending the collection module to be able to make knowledge base with providers other than openai

Summary by CodeRabbit

Release Notes

• New Features
  • Collection creation now supports explicit LLM provider configuration options. Users can specify their preferred LLM service provider when creating new collections, with OpenAI currently available as the primary provider option. This enhancement enables greater control over service dependencies and ensures proper provider configuration during the collection creation and initialization process.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

A new ProviderOptions model was added to the collection module, defining an LLM provider field constrained to "openai". The CreationRequest class was updated to inherit from ProviderOptions, extending its public API. A Literal type was imported to support type constraints, and a field comment was refined.

Changes

Cohort / File(s)	Summary
Collection Model Enhancements backend/app/models/collection.py	Added Literal import from typing; created new ProviderOptions model with provider: Literal["openai"] field; extended CreationRequest to inherit from ProviderOptions; updated llm_service_name comment from "Name of the LLM service provider" to "Name of the LLM service"

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• New public API surface introduced via ProviderOptions model—verify naming, field defaults, and description accuracy
• Verify CreationRequest inheritance chain maintains backward compatibility and expected behavior
• Confirm Literal["openai"] constraint aligns with service architecture and future extensibility needs

Poem

🐰 A provider hops into view,
OpenAI constrained, tried and true!
Collections now know whom to trust,
With Literal types—it's a must!
Inheritance flows, clean and bright,
Our API takes flight! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Collection: Adding input parameter "provider"' directly and clearly describes the main change - adding a new 'provider' parameter to the collection module.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch enhancement/provider_param_collection

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[coderabbitai]

Actionable comments posted: 3

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d26401b and 0c3f4be.

📒 Files selected for processing (1)

• backend/app/models/collection.py (3 hunks)

🧰 Additional context used

📓 Path-based instructions (2)

**/*.py

📄 CodeRabbit inference engine (CLAUDE.md)

Use type hints in Python code (Python 3.11+ project)

Files:

• backend/app/models/collection.py

backend/app/models/**/*.py

📄 CodeRabbit inference engine (CLAUDE.md)

Define SQLModel entities (database tables and domain objects) in backend/app/models/

Files:

• backend/app/models/collection.py

🧬 Code graph analysis (1)

backend/app/models/collection.py (2)

backend/app/tests/services/llm/providers/test_openai.py (1)

• provider (27-29)

backend/app/crud/rag/open_ai.py (1)

• default (31-32)

🔇 Additional comments (1)

backend/app/models/collection.py (1)

2-2: LGTM!

The Literal import is correctly added to support the type constraint in the new ProviderOptions class.