Spaces:
Runtime error
Runtime error
Commit History
-
refactor: Simplify codebase and consolidate testsβ
Julien Simon +1: Claude Opus 4.5- Extract shared utilities: get_top_chunk_index(), get_document_source() - Add HYBRID_ALPHA_UI_DEFAULT to centralize config - Simplify score normalization in HybridRetriever - Fix redundant branches in hybrid search - Simplify update_rag_controls() in handlers - Consolidate 10 fragmented test files into 4 main files - Remove duplicate fixtures and test code Net reduction: 115 lines (-853/+738) Test files: 28 β 20 Coverage: 98.86% Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>b39a3c2
-
refactor: Reorganize PDFs and improve error handlingβ
Julien Simon +1: Claude Opus 4.5- Move arXiv research papers to pdf_research/ directory - Add IEA energy reports and publications to pdf_energy/ - Update README to reflect new document content (clean energy, renewables, EVs, CCUS) - Create pdf symlink for default document source - Improve security by hiding internal error details from users in stream output - Replace print statements with proper logger.warning/error calls - Remove TEST_REVIEW.md and .DS_Store files - Add consistent trailing newlines to source files Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>ed005f8
-
Add comprehensive test suite with 98.28% coverageβ
Julien Simon- Add 34 new high-priority tests covering critical workflows: * Multi-turn conversations with chat history (4 tests) * Mode switching between RAG and Vanilla LLM (4 tests) * Network and API failure scenarios (9 tests) * Empty vectorstore and initial state handling (8 tests) * Input validation and edge cases (8 tests) - Add test review document (TEST_REVIEW.md) with critical analysis - Add coverage configuration (.coveragerc) to exclude test files - Update pytest.ini with coverage configuration Test Results: - 153 tests passing (up from 119) - 98.28% code coverage (maintained) - All modules at 100% except qa_chain (98%) and retrievers (93%) Covers all critical user workflows and production edge cases.7ab7df1