Regression fixes: Per repo fixes - #133
Merged
Merged
Conversation
- scripts/ingest/cli.py: stop passing collection=None in multi-repo mode; iterate repo dirs and call index_repo with each repo’s derived collection so Qdrant always receives a concrete name. - scripts/workspace_state.py: in multi-repo mode, ignore COLLECTION_NAME overrides when repo_name is provided, preventing codebase/codebase_old hijacking; add regression tests covering both multi- and single-repo precedence. - scripts/watch_index.py: gate health-check/default-collection routines behind explicit env flags when multi-repo is enabled so watcher no longer auto-creates “codebase”. - tests/test_workspace_state.py: add coverage for the new get_collection_name precedence rules.
- scripts/ingest/pipeline.py: always enumerate files up front and print `[index] Found N files…` plus `[index] X/Y files processed` heartbeats whenever stdout isn’t a TTY, while keeping tqdm for interactive sessions. This mirrors the pre-refactor logging so K8s logs show real progress again.
- scripts/ingest/pipeline.py: always wrap the file iterator with tqdm (matching pre-refactor behavior) and gate the new `[index] …` log heartbeats behind INDEX_PROGRESS_LOG (default on) so headless logs stay readable without suppressing tqdm in interactive sessions.
…se” collection
- watch_index_core/utils.py
* treat per-repo state entries that point at placeholder collections (codebase,
etc.) as invalid in multi-repo mode, derive the correct collection via
get_collection_name, and persist the repaired mapping back to state
- workspace_state.py
* stop auto-populating serving_collection from qdrant_collection when
multi-repo is enabled and staging is off, and never propagate placeholder
collection names into serving_collection
- tests/test_watcher_collection_resolution.py
* add regression test ensuring stale state.json values (codebase) are ignored
for per-repo routing in multi-repo mode
voarsh2
marked this pull request as ready for review
December 29, 2025 22:38
Merged
m1rl0k
added a commit
that referenced
this pull request
Mar 1, 2026
Regression fixes: Per repo fixes
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.