Fix/pipeline test and coverage - #1
Closed
huyenhothi wants to merge 38 commits into
Closed
Conversation
- Fix cross-platform path handling in Git connector tests (9 test failures) * Update test_metadata_extractor.py to use os.path.join with temp_dir * Update test_metadata_extractor_enhanced.py for all file paths * Fix test_git_connector.py with proper mocking and temp_dir paths - Fix pytest configuration conflicts (172 root tests) * Update pyproject.toml to exclude packages/ from root test discovery * Add norecursedirs to prevent conftest import conflicts - Fix config loader test isolation (1 test failure) * Add monkeypatch.chdir(tmp_path) to isolate from project root * Prevent finding config.yaml in workspace during tests - Fix website build system tests (19 failures/errors) * Add UTF-8 encoding to all file operations (2 unicode errors) * Use Path.as_posix() for sitemap URL generation (1 path separator issue) * Implement smart colon detection for Windows paths in asset copying (5 failures) * Add retry logic with delays for temp directory cleanup (15 permission errors) * Update test assertions for cross-platform compatibility (2 assertions) - Update test timing assertions * Change > 0 to >= 0 for processing_time_ms assertions * Fix directory merge expectations in coverage boost tests All changes maintain full compatibility with Mac/Linux platforms. Tests verified: 1899+ passing (172 root + 33 git + 120 website + 1674 loader) Resolves cbtw-apac#57
fix: Windows compatibility fixes
fix: update prometheus-client dependency. Fix cbtw-apac#60
fix: log level not reconfigurable at runtime Fix cbtw-apac#58
Migrate from deprecated search() method to query_points() API for compatibility with qdrant-client 1.10+. Changes: - Update vector_search_service.py to use query_points() - Update qdrant_manager.py search methods to use query_points() - Update test mocks to use AsyncMock for query_points() - Add third mock result to conftest for test compatibility Fixes cbtw-apac#74
chore: split oversized file to comply with model size constraints
fix: update qdrant-client API from search() to query_points() (cbtw-apac#74)
Fix missing spacy in mcp-server dependencies
The level parameter was accepted but never used in the MCP server's LoggingConfig.reconfigure() method. This fix: - Passes level to CoreLoggingConfig.reconfigure() - Uses level in fallback path when core is unavailable - Fixes variable shadowing (old_level instead of level)
…-with-level fix: use level parameter in MCP wrapper reconfigure()
Add comprehensive release notes for v0.7.4 covering 5 bug fixes: - Windows compatibility (cbtw-apac#57): Fixed asyncio event loop crashes, cross-platform stdin handler, signal handler platform checks - Logging system (cbtw-apac#58): Fixed log level not reconfigurable at runtime - prometheus-client dependency (cbtw-apac#60): Restored missing runtime dependency - spacy dependency (cbtw-apac#67): Added missing spacy to MCP server dependencies - qdrant-client API (cbtw-apac#74): Migrated from deprecated search() to query_points() Update README version reference to v0.7.4
- Update version to 0.7.4 in all 4 pyproject.toml files: - pyproject.toml (workspace) - packages/qdrant-loader/pyproject.toml - packages/qdrant-loader-core/pyproject.toml - packages/qdrant-loader-mcp-server/pyproject.toml - Update internal dependency pins: - qdrant-loader-core[openai]==0.7.4 (in qdrant-loader) - qdrant-loader-core==0.7.4 (in mcp-server)
Hotfix/0.7.4: Windows MCP integration and logging level CLI support
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
6 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.
Pull Request
Summary
Describe the change in 1–2 sentences.
Type of change
Docs Impact (required for any code or docs changes)
docs/?python website/build.py+python website/check_links.py)Testing
Describe how you tested this change. Include commands and results.
Checklist
pytest -v)