Skip to content

merge stage into main#110

Merged
aditya-arcot merged 3 commits intomainfrom
stage
Mar 27, 2026
Merged

merge stage into main#110
aditya-arcot merged 3 commits intomainfrom
stage

Conversation

@aditya-arcot
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 27, 2026 14:10
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

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

Merges staging changes into main, primarily updating dependency lockfiles and adding an automated Meilisearch reindex workflow (service code + infra wiring) to ensure search indexes are rebuilt in deployment environments.

Changes:

  • Bump server/client/tooling dependencies (Python uv.lock, root/client npm lockfiles).
  • Update Meilisearch reindex behavior to delete existing indexes before rebuilding, with corresponding test updates.
  • Add a standalone reindex script and wire it into Docker Compose as a one-off service run before the API starts.

Reviewed changes

Copilot reviewed 8 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
server/uv.lock Updates Python dependency lock entries (fastapi-cloud-cli, meilisearch sdk, ruff).
server/pyproject.toml Excludes app/scripts/* from coverage collection.
server/app/services/search.py Changes reindex() to delete existing Meilisearch indexes up-front; removes per-index deletions.
server/app/tests/services/search/test_reindex.py Updates unit test to assert deletion happens prior to indexing calls.
server/app/scripts/reindex_meilisearch.py Adds an async script entrypoint for full Meilisearch reindex using app settings/dependencies.
server/app/scripts/init.py Declares scripts package (empty).
config/infra/docker-compose.yml Renames migrations service, adds reindex-meilisearch one-off job, and makes server depend on it.
config/infra/docker-compose.override.yml Adds dev build config for the renamed migration service and the reindex job.
package.json Bumps npm-check-updates.
package-lock.json Lockfile updates for npm-check-updates and transitive yaml.
client/package.json Bumps vite.
client/package-lock.json Lockfile updates for Vite and transitive rolldown bindings/pluginutils/picomatch.
Files not reviewed (1)
  • client/package-lock.json: Language not supported

@aditya-arcot aditya-arcot merged commit e212146 into main Mar 27, 2026
18 of 20 checks passed
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.

2 participants