fix(ci): restore repo-root import path for tests via root conftest#163
Merged
Conversation
CI (pip install -e ".[dev,mcp,security]") went red on every run since the P6 packaging fix scoped packages.find to memorymaster*. That correctly stops shipping scripts/, but it also drops the repo root from sys.path under an editable install — so tests/test_connectors.py's `from scripts import ...` raises ModuleNotFoundError during collection, failing the whole suite. Add a root conftest.py that inserts the repo root on sys.path before collection. Restores import access to scripts/ for tests without changing what the wheel ships. Pre-existing main breakage, unrelated to feature work.
wolverin0
added a commit
that referenced
this pull request
Jun 24, 2026
…+ document v4.0.0→ work) (#167) * release: v4.1.0 — local-filesystem (Everything) bridge + LLM Atlas extractor Surfaces the previously-merged-but-undocumented work since v4.0.0: - #161 local-filesystem search bridge (resolve_project, Everything ES.exe, path redaction) - #166 LLM typed-entity Atlas extractor - #162 detect-first installer; #165 steward tiers-every-cycle; #163 CI conftest; #164 IP scrub README Key features + CHANGELOG now document the Everything integration. * docs(release): frame local-search honestly — optional, Windows/Codex-focused, graceful-degrade Re-verified the Everything bridge works end-to-end (es.exe 1.1.0.27): resolve-project finds real projects with explainable scoring, memory-first cache hits on repeat, paths redacted. But the value is narrow (path-resolution for weak-file-search agents + cross- session recall), not a headline; speed is sub-second either way. Docs now say so.
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
1 participant
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.
Pre-existing main CI breakage (red since the P6 packaging fix), unrelated to feature work.
Cause: CI runs
pip install -e ".[dev,mcp,security]". Sincepackages.findwas scoped tomemorymaster*, an editable install no longer puts the repo root onsys.path, sotests/test_connectors.py'sfrom scripts import ...raises ModuleNotFoundError during collection — failing the entire suite on every Python/OS combo.Fix: a root
conftest.pythat inserts the repo root ontosys.pathbefore collection. Restores import access toscripts/for tests without changing what the wheel ships. Unblocks #161 and #162.