firefrost-gaming/skill-seekers-reference
3
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity
693 Commits 1 Branch 0 Tags
main
Commit Graph

4 Commits

Author SHA1 Message Date
yusyus
bcc2ef6a7f test: Skip tests requiring optional dependencies 
- Skip test_benchmark.py if psutil not installed
- Skip test_embedding.py if numpy not installed
- Skip test_embedding_pipeline.py if numpy not installed
- Uses pytest.importorskip() for clean dependency handling
- Fixes CI test collection errors for optional features
 2026-02-08 14:49:45 +03:00
yusyus
0265de5816 style: Format all Python files with ruff 
- Formatted 103 files to comply with ruff format requirements
- No code logic changes, only formatting/whitespace
- Fixes CI formatting check failures
 2026-02-08 14:42:27 +03:00
yusyus
51787e57bc style: Fix 411 ruff lint issues (Kimi's issue #4) 
Auto-fixed lint issues with ruff --fix and --unsafe-fixes:

Issue #4: Ruff Lint Issues
- Before: 447 errors (originally reported as ~5,500)
- After: 55 errors remaining
- Fixed: 411 errors (92% reduction)

Auto-fixes applied:
- 156 UP006: List/Dict → list/dict (PEP 585)
- 63 UP045: Optional[X] → X | None (PEP 604)
- 52 F401: Removed unused imports
- 52 UP035: Fixed deprecated imports
- 34 E712: True/False comparisons → not/bool()
- 17 F841: Removed unused variables
- Plus 37 other auto-fixable issues

Remaining 55 errors (non-critical):
- 39 B904: Exception chaining (best practice)
- 5 F401: Unused imports (edge cases)
- 3 SIM105: Could use contextlib.suppress
- 8 other minor style issues

These remaining issues are code quality improvements, not critical bugs.

Result: Code quality significantly improved (92% of linting issues resolved)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
 2026-02-08 12:46:38 +03:00
yusyus
8b3f31409e fix: Enforce min_chunk_size in RAG chunker 
- Filter out chunks smaller than min_chunk_size (default 100 tokens)
- Exception: Keep all chunks if entire document is smaller than target size
- All 15 tests passing (100% pass rate)

Fixes edge case where very small chunks (e.g., 'Short.' = 6 chars) were
being created despite min_chunk_size=100 setting.

Test: pytest tests/test_rag_chunker.py -v
 2026-02-07 20:59:03 +03:00