From 872bf34ba89121371e7df6d232341a8dd08477e7 Mon Sep 17 00:00:00 2001 From: Sinity Date: Sun, 26 Jul 2026 20:59:53 +0200 Subject: [PATCH] fix(rebuild): drop stale resume_from_empty_message_index kwarg MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Problem: rebuild_index.py's bulk-build derived-state repopulate still called rebuild_fts_index_sync with a resume_from_empty_message_index kwarg that master already dropped from the function signature via a separately-merged FTS convergence pass — this branch had gone stale enough that its own remaining net diff (once rebased past everything master already absorbed in parallel) reduced to this one call. What changed: call rebuild_fts_index_sync with no extra kwarg, matching its current signature. Verification: devtools test tests/unit/maintenance/test_rebuild_index_bulk_build.py tests/unit/storage/test_planner_statistics_seed.py; devtools verify --quick. Co-Authored-By: Codex --- polylogue/maintenance/rebuild_index.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/polylogue/maintenance/rebuild_index.py b/polylogue/maintenance/rebuild_index.py index 3c270b2866..26d5ca8bfb 100644 --- a/polylogue/maintenance/rebuild_index.py +++ b/polylogue/maintenance/rebuild_index.py @@ -118,7 +118,7 @@ def _repopulate_bulk_build_derived_state(index_path: Path) -> dict[str, float]: with contextlib.closing(sqlite3.connect(index_path, timeout=600)) as conn: conn.execute("PRAGMA busy_timeout = 600000") started_at = time.perf_counter() - rebuild_fts_index_sync(conn, resume_from_empty_message_index=True) + rebuild_fts_index_sync(conn) timings_s["fts"] = time.perf_counter() - started_at started_at = time.perf_counter() rebuild_command_trigram_index_sync(conn)