Skip to content

feat(phase-2): implement file cleanup and fix warnings#285

Merged
laynepenney merged 2 commits intomainfrom
feat/production-readiness-phase-2
Feb 8, 2026
Merged

feat(phase-2): implement file cleanup and fix warnings#285
laynepenney merged 2 commits intomainfrom
feat/production-readiness-phase-2

Conversation

@laynepenney
Copy link
Copy Markdown
Collaborator

@laynepenney laynepenney commented Feb 8, 2026

Summary

Phase 2 of Production Readiness Plan: Clean up warnings and implement high-priority TODOs.

Changes

Phase 2 Complete ✅

Warnings Fixed:

  • Removed unused load_session function from main.rs
  • Clean build with zero warnings

HIGH Priority TODOs Resolved:

  1. File Cleanup (symbol_index/indexer.rs:561)

    • Added get_all_files() method to SymbolDatabase
    • Implemented cleanup_deleted() to remove stale entries
    • Files checked against disk and deleted from DB if missing
  2. Usage Detection (symbol_index/service.rs:206)

    • Added find_imports_with_symbol() database method
    • find_references() now finds all imports referencing a symbol
    • Returns import locations with file paths
  3. Dependency Graph (symbol_index/service.rs:229)

    • Added get_file_dependencies() and get_file_dependents() database methods
    • Implemented BFS traversal in get_dependencies()
    • Supports both Imports and ImportedBy directions
    • Configurable max depth (default: 3 levels)

Documentation Updated:

  • Updated PRODUCTION_READINESS_PLAN.md with Phase 2 progress
  • Marked Phase 1 as complete
  • Documented remaining LOW priority TODOs for Phase 4

Verification

  • ✅ Clean build with zero warnings
  • ✅ All HIGH priority TODOs resolved
  • ✅ All 516 tests pass
  • ✅ No behavioral regressions

Files Changed

  • codi-rs/docs/PRODUCTION_READINESS_PLAN.md
  • codi-rs/src/main.rs
  • codi-rs/src/symbol_index/database.rs
  • codi-rs/src/symbol_index/indexer.rs
  • codi-rs/src/symbol_index/service.rs

Related

  • Implements Phase 2 of docs/PRODUCTION_READINESS_PLAN.md

- Remove unused load_session function from main.rs
- Add get_all_files() method to SymbolDatabase
- Implement cleanup_deleted() to remove stale file entries
- Update PRODUCTION_READINESS_PLAN with Phase 2 progress

Progress on Phase 2:
- ✅ Clean build with zero warnings
- ✅ HIGH priority TODO: File cleanup for deleted/renamed files
- Add database methods: find_imports_with_symbol(), get_file_dependencies(), get_file_dependents()
- Implement usage detection in find_references() by finding imports
- Implement dependency graph traversal in get_dependencies() with BFS
- Add support for both Imports and ImportedBy directions
- Fix unused variable warnings

Phase 2 complete:
- ✅ Clean build with zero warnings
- ✅ HIGH priority TODOs resolved (file cleanup, usage detection, dependency graph)
- ✅ All 516 tests pass
@laynepenney laynepenney merged commit 58a8f0f into main Feb 8, 2026
3 checks passed
@laynepenney laynepenney deleted the feat/production-readiness-phase-2 branch February 8, 2026 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant