Skip to content

refacto(rerank): rerank endpoint clean archi refactoring#866

Draft
leoguillaume wants to merge 16 commits into
mainfrom
739-refacto-refacto-post-v1rerank-endpoint-toward-clean-architecture
Draft

refacto(rerank): rerank endpoint clean archi refactoring#866
leoguillaume wants to merge 16 commits into
mainfrom
739-refacto-refacto-post-v1rerank-endpoint-toward-clean-architecture

Conversation

@leoguillaume
Copy link
Copy Markdown
Member

Overview

Please precise the issue that you are resolving and provide a summarized description of what was done in this PR. Then provide the Definition of Done (DoD) criteria that apply to this PR:

  • DoD 1
  • DoD 2
  • DoD 3
  • ...

Breaking changes:

  • No breaking changes
  • This PR contains breaking changes (explain below)

Check lists

Review checklist

Before requesting a review, please take a moment to confirm that the following aspects have been considered and addressed. This section helps ensure the PR is ready for review, safe to merge, and deployable. If any items are left unchecked, please add a brief explanation for context.

  • Updated or added documentation
  • Updated or added unit tests
  • Updated or added integration tests
  • No debug logs or commented-out code left
  • No secrets or environment variables committed in clear text
  • Code is linted and formatted using the project pre-commit hooks

If api/sql/models.py has been modified, please confirm that the following steps have been completed:

  • Alembic migration has been generated
  • Alembic migration upgrade has been tested locally
  • Alembic migration downgrade has been tested locally

Deployment checklist

For each of the following items, please confirm if the PR concerns the deployment of the changes:

  • Alembic migration has been generated
  • Configuration file has been modified
  • Environment variables have been modified

If new or updated environment variables are required, please list them here, otherwise delete this part. If other special deployment steps are required, please describe them here, otherwise delete this part.

Additional Notes

Please provide any additional information or context that may be relevant to this PR, otherwise delete this part. This could be any specific areas you would like the reviewers to focus on during their review of this PR (complex logic, risky changes, performance-sensitive code, etc.)

@leoguillaume leoguillaume force-pushed the 739-refacto-refacto-post-v1rerank-endpoint-toward-clean-architecture branch 3 times, most recently from 9e1b082 to d8cd010 Compare May 13, 2026 21:24
@leoguillaume leoguillaume changed the title wip refacto(rerank): rerank endpoint clean archi refactoring May 18, 2026
@leoguillaume leoguillaume force-pushed the 739-refacto-refacto-post-v1rerank-endpoint-toward-clean-architecture branch from 55af741 to 550e1cd Compare May 21, 2026 13:08
@leoguillaume leoguillaume force-pushed the 739-refacto-refacto-post-v1rerank-endpoint-toward-clean-architecture branch from 550e1cd to 7c7b18e Compare May 27, 2026 16:21
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.

 [refacto] Refacto POST /v1/rerank endpoint toward clean architecture

1 participant