Skip to content

feat: auto commit refresh indexes#104

Merged
davidkelley merged 2 commits intomainfrom
feat/issue-92-auto-commit-refresh-indexes
Mar 3, 2026
Merged

feat: auto commit refresh indexes#104
davidkelley merged 2 commits intomainfrom
feat/issue-92-auto-commit-refresh-indexes

Conversation

@davidkelley
Copy link
Owner

Copilot AI review requested due to automatic review settings March 2, 2026 16:29
@davidkelley davidkelley changed the title feat/issue 92 auto commit refresh indexes feat: auto commit refresh indexes Mar 2, 2026
@davidkelley
Copy link
Owner Author

@codex review

Copy link

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

This PR extends searchlite-http with per-index runtime configuration for background maintenance (auto-commit/auto-refresh intervals) and enriches the GET /indexes endpoint with operational metadata (existence, commit timestamp, doc counts, and runtime flags).

Changes:

  • Extend index mount parsing and server flags to support per-index and global auto_commit / auto_refresh interval configuration.
  • Add a maintenance runtime to schedule periodic auto-commit work and expand /indexes responses with manifest/runtime metadata.
  • Add helper logic + tests around /indexes metadata and auto-commit behavior; add a feature-hardening matrix doc for issue #92.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

File Description
searchlite-http/src/lib.rs Adds interval config plumbing, maintenance runtime, /indexes metadata enrichment, live doc count helper, and new integration/unit tests.
docs/feature-hardening/issue-92-auto-commit-refresh-indexes/matrix.md Adds the issue-92 feature hardening matrix documenting intended behaviors and verification status.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 431abe848f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@davidkelley
Copy link
Owner Author

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Hooray!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@davidkelley davidkelley linked an issue Mar 3, 2026 that may be closed by this pull request
@davidkelley davidkelley merged commit beca48b into main Mar 3, 2026
6 checks passed
@davidkelley davidkelley deleted the feat/issue-92-auto-commit-refresh-indexes branch March 3, 2026 13:24
This was referenced Mar 3, 2026
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.

#6: Auto-commit/refresh + /indexes endpoint

2 participants