Skip to content

Revert draft BM25 ranking changes from PR #54 - #56

Draft
TeoSlayer wants to merge 1 commit into
mainfrom
revert/draft-ranking-pr54
Draft

TeoSlayer wants to merge 1 commit into
mainfrom
revert/draft-ranking-pr54

Conversation

@TeoSlayer

@TeoSlayer TeoSlayer commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Current status: production is restored to the original v0.2.5 binary/config and public routing. The community service is stopped and automatic updates are disabled. This change is consolidated into #58 for review; this PR remains a draft reference. The user explicitly requested PR-only work with no merge, release or deployment.

Reverts the merge of PR #54 (616ea1513a5f1bbb51e734aa5caededf4ea3f3c9). It was still a draft, and its author reported that production testing was not promising and further work was needed. Passing CI did not establish production readiness.

This is the complete mechanical revert of #54: the BM25 implementation, its tests, and the three affected documentation files return to their pre-PR contents. Security hardening, dependency updates, and community/CLI contribution features remain in the branch.

Production was rolled back from v0.2.6 to the previous v0.2.5 binary and configuration. The community web app was not enabled during the v0.2.6 engine rollout. The subsequent auth-only rollout in #57 used a separate signed binary and has now also been rolled back. The original public routing is restored and the portal is stopped. v0.2.6 is marked withdrawn/prerelease, v0.2.5 is again the latest stable release, and automatic updates remain stopped pending review.

Validation: exact comparison confirms all five reverted files match the first parent of #54's merge. Go vet and the full race suites for internal/index, internal/community, internal/crawler, and cmd/cosift pass. GitHub CI and the security check pass. The advisory Codecov patch-coverage check is below target. The production vector index has fully reloaded, and the three comparison searches match the pre-deployment ordering. This PR remains deliberately a draft for review and does not trigger a release.

…re-authority"

This reverts commit 616ea15, reversing
changes made to 79e20f9.
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/index/pebble_bm25.go 62.50% 2 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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