Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 7, 2025

Code review identified that import paths weren't updated when services were reorganized into subpackages (memory, knowledge, tasks, code, utils, sql). Additionally, a syntax error in mcp/server.py prevented the file from compiling.

Export singleton instances from subpackages

  • Updated __init__.py files to export both classes and singleton instances (memory_store, task_queue, graph_service, etc.)
  • Enables cleaner imports: from codebase_rag.services.memory import memory_store instead of from codebase_rag.services.memory_store import memory_store

Update import paths across codebase

  • API routes (6 files): Use subpackage imports for all service dependencies
  • Core modules: Updated lifespan.py and mcp/server.py
  • Service internals: Changed memory_extractor.py to use relative import for memory_store

Fix syntax error in mcp/server.py

  • Removed orphaned code incorrectly placed inside InitializationOptions() call in main() function
  • Code from a different function was mixed into the initialization parameters

Example of new import pattern:

# Before
from codebase_rag.services.task_queue import task_queue, TaskStatus
from codebase_rag.services.task_storage import TaskType
from codebase_rag.services.task_processors import processor_registry

# After
from codebase_rag.services.tasks import task_queue, TaskStatus, TaskType, processor_registry

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ax error in mcp/server.py

Co-authored-by: royisme <350731+royisme@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor directory structure for better maintainability Fix import paths for reorganized services subpackage structure Nov 7, 2025
Copilot AI requested a review from royisme November 7, 2025 01:54
@royisme royisme marked this pull request as ready for review November 7, 2025 01:59
@royisme royisme merged commit f79bd0c into claude/refactor-directory-structure-011CUsSDTpBiMf1EiqBHxZrM Nov 7, 2025
@royisme royisme deleted the copilot/sub-pr-21 branch November 7, 2025 01:59
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.

2 participants