Skip to content

v0.6 — Desloppify: Code Health Campaign

Choose a tag to compare

@docxology docxology released this 10 Mar 23:19
· 2805 commits to main since this release

🧹 Code Health: Desloppify Campaign

The largest code-quality improvement cycle since the template's inception. 162 commits across 948 files, systematically eliminating technical debt from all 8 infrastructure packages through 26 rounds of blind review.

Fixes

  • Import hygiene — Removed unused imports across 20+ files; eliminated sys.path mutations from CLI modules; proper TYPE_CHECKING guards throughout
  • Exception handling — Narrowed broad except Exception / bare except clauses in integrity.py, logging_utils, config_loader, and llm modules; fixed silent JSONDecodeError swallowing; restored exception context with raise ... from exc
  • Dead code removal — Deleted orphaned coverage_reporter.py (zero importers); removed stub/passthrough wrapper methods across 10+ modules; eliminated dead HTML-entities dict from InputSanitizer
  • Type annotations — Modernised legacy typing imports (List[x] → list[x], Optional[x] → x | None) across 30+ modules; added TypedDict returns for integrity results; annotated CLI re-exports
  • API surface — Consolidated LLMConfig env-read wrappers (ABS-001); merged duplicate PerformanceMetrics naming conflict; removed ProjectLogger pure-forwarder abstraction; eliminated calculate_file_hash re-export from publishing boundary
  • Bug fixes — Fixed inverted scan_errors bool in doc scanner; fixed stall-detection dead branch in pipeline reporter; fixed config_files path bug in config_cli; fixed clean_output_directory return type; fixed broken accessor imports after core.py hub elimination
  • Structural — Eliminated infrastructure/core/core.py hub; extracted _build_stage_list to remove stage-list duplication; moved MultiProjectResult to TYPE_CHECKING to break reporting→core circular dependency
  • Logging — Removed noisy debug logs from LLM and environment modules; downgraded verbose entry logs; added get_logger to logic modules lacking structured logging
  • Docstrings — Stripped AI-generated boilerplate from 40+ functions; removed restating comments and banner comments
  • Tests — Fixed test name collisions; added deterministic tests for validate_review_quality and exception types; added integration tests to testpaths; removed orphan test files
  • Dependencies — Removed scipy from infrastructure env check; resolved stale psutil guards; moved matplotlib to optional dep group

Quality Gates

Gate Status
Desloppify blind reviews 26 rounds
Commits 162
Files changed 948
ruff check ✅ 0 errors
mypy --strict ✅ 0 errors (all 8 packages)
bandit -ll ✅ 0 MEDIUM+ findings
pip-audit ✅ Blocking gate
pytest ✅ All pass

Documentation

  • Updated CHANGELOG.md with full v0.6.0 entry
  • Updated docs/development/roadmap.md with desloppify results and refreshed quality metrics
  • Updated docs/audit/documentation-review-report.md for v0.6 scope

See CHANGELOG.md for full details.