Claude/refactor directory structure 011 c us sd tp bi mf1 eiq b hx zr m

[royisme]

This pull request refactors the project’s source code layout to consolidate all core modules under the src/codebase_rag package, updates documentation and build scripts to match the new structure, and standardizes the application startup method. These changes improve maintainability, simplify imports, and make Docker builds more consistent.

Source Code Structure Refactor:

• Consolidated all main modules (api, core, services, mcp_tools, etc.) under a single package: src/codebase_rag. All Dockerfiles now copy only the src directory, and startup commands use python -m codebase_rag instead of various script files. [1] [2] [3] [4] [5] [6] [7] [8]

Documentation Updates:

• Updated all import paths in documentation to reference the new src/codebase_rag package, ensuring consistency and clarity for developers using the Python SDK or contributing to the project. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21]

Build & Versioning Adjustments:

• Updated .bumpversion.toml and GitHub Actions to reference the new location of __version__.py in src/codebase_rag/__version__.py, ensuring versioning and CI workflows operate correctly with the new structure. [1] [2]

Startup Method Standardization:

• Changed documentation and Docker startup commands to use python -m codebase_rag for launching the application, replacing previous usage of start.py, start_mcp.py, and other entry points. This makes running the app more predictable and easier to maintain. [1] [2] [3] [4]

Miscellaneous Documentation Fixes:

• Updated deployment notification URLs and removed the old CNAME file to reflect the new documentation hosting location. [1] [2]

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Replace src. imports inside package*

Package modules are still importing via the src.codebase_rag namespace even though the package is now exposed as codebase_rag (see pyproject console scripts). For example, __init__.py imports from src.codebase_rag.__version__ import …. When the user runs python -m codebase_rag or the installed codebase-rag CLI, Python cannot resolve a top-level src package and raises ModuleNotFoundError, so the application cannot start at all. These imports should be relative or use the codebase_rag.* namespace.

Useful? React with 👍 / 👎.

[Copilot]

Pull Request Overview

This PR refactors the project to adopt Python's standard src-layout by consolidating all source code under src/codebase_rag/, removing legacy entry scripts, and standardizing the application startup method to python -m codebase_rag.

Key Changes:

• Moved all modules (api, core, services, mcp_tools) into src/codebase_rag/ package structure
• Replaced entry scripts (start.py, start_mcp.py) with module-based invocation (python -m codebase_rag)
• Updated all import paths from relative imports to absolute src.codebase_rag.* imports

Reviewed Changes

Copilot reviewed 86 out of 120 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
tests/*.py	Updated test imports to reference new src.codebase_rag package structure
src/codebase_rag/**/*.py	Created new package structure with updated internal imports
start.py, start_mcp.py	Removed legacy entry scripts
pyproject.toml	Updated package discovery and console scripts
docker/Dockerfile.*	Updated COPY commands and CMD to use new structure
scripts/*.sh	Updated references to new file paths
docs/**/*.md	Updated documentation with new import paths and commands
.github/workflows/*.yml	Updated CI/CD to reference new version file location

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Import path is incorrect. Should be from codebase_rag.services.memory.memory_store import memory_store to match the new package structure, or use relative import from .memory_store import memory_store.

Suggested change

	from codebase_rag.services.memory_store import memory_store
	from .memory_store import memory_store

[Copilot]

Import paths don't match the new subpackage structure. Based on the new organization, these should be:\n- from codebase_rag.services.knowledge import Neo4jKnowledgeService\n- from codebase_rag.services.memory import memory_store, memory_extractor\n- from codebase_rag.services.tasks import task_queue, TaskStatus, submit_document_processing_task, submit_directory_processing_task, processor_registry\n- from codebase_rag.services.code import graph_service, get_code_ingestor, pack_builder\n- from codebase_rag.services.utils import ranker, git_utils

Suggested change

	from codebase_rag.services.neo4j_knowledge_service import Neo4jKnowledgeService
	from codebase_rag.services.memory_store import memory_store
	from codebase_rag.services.memory_extractor import memory_extractor
	from codebase_rag.services.task_queue import task_queue, TaskStatus, submit_document_processing_task, submit_directory_processing_task
	from codebase_rag.services.task_processors import processor_registry
	from codebase_rag.services.graph_service import graph_service
	from codebase_rag.services.code_ingestor import get_code_ingestor
	from codebase_rag.services.ranker import ranker
	from codebase_rag.services.pack_builder import pack_builder
	from codebase_rag.services.git_utils import git_utils
	from codebase_rag.services.knowledge import Neo4jKnowledgeService
	from codebase_rag.services.memory import memory_store, memory_extractor
	from codebase_rag.services.tasks import task_queue, TaskStatus, submit_document_processing_task, submit_directory_processing_task, processor_registry
	from codebase_rag.services.code import graph_service, get_code_ingestor, pack_builder
	from codebase_rag.services.utils import ranker, git_utils

[Copilot]

Import paths don't match the new subpackage structure. Should be:\n- from codebase_rag.services.knowledge import neo4j_knowledge_service\n- from codebase_rag.services.tasks import task_queue, processor_registry\n- from codebase_rag.services.memory import memory_store

Suggested change

	from codebase_rag.services.neo4j_knowledge_service import neo4j_knowledge_service
	from codebase_rag.services.task_queue import task_queue
	from codebase_rag.services.task_processors import processor_registry
	from codebase_rag.services.memory_store import memory_store
	from codebase_rag.services.knowledge import neo4j_knowledge_service
	from codebase_rag.services.tasks import task_queue, processor_registry
	# processor_registry is now imported from tasks subpackage
	from codebase_rag.services.memory import memory_store

[Copilot]

Import paths don't match the new subpackage structure. Should use the reorganized service modules:\n- from codebase_rag.services.sql import sql_analyzer, parse_sql_schema_smart\n- from codebase_rag.services.code import graph_service, get_code_ingestor, pack_builder\n- from codebase_rag.services.knowledge import Neo4jKnowledgeService\n- from codebase_rag.services.tasks import task_queue\n- from codebase_rag.services.utils import git_utils, ranker, metrics_service

Suggested change

	from codebase_rag.services.sql_parser import sql_analyzer
	from codebase_rag.services.graph_service import graph_service
	from codebase_rag.services.neo4j_knowledge_service import Neo4jKnowledgeService
	from codebase_rag.services.universal_sql_schema_parser import parse_sql_schema_smart
	from codebase_rag.services.task_queue import task_queue
	from codebase_rag.services.code_ingestor import get_code_ingestor
	from codebase_rag.services.git_utils import git_utils
	from codebase_rag.services.ranker import ranker
	from codebase_rag.services.pack_builder import pack_builder
	from codebase_rag.services.metrics import metrics_service
	from codebase_rag.services.sql import sql_analyzer, parse_sql_schema_smart
	from codebase_rag.services.code import graph_service, get_code_ingestor, pack_builder
	from codebase_rag.services.knowledge import Neo4jKnowledgeService
	from codebase_rag.services.tasks import task_queue
	from codebase_rag.services.utils import git_utils, ranker, metrics_service

[Copilot]

Import path should be from codebase_rag.services.knowledge import neo4j_knowledge_service to match the new subpackage structure.

Suggested change

	from codebase_rag.services.neo4j_knowledge_service import neo4j_knowledge_service
	from codebase_rag.services.knowledge import neo4j_knowledge_service

[Copilot]

Import paths should be from codebase_rag.services.memory import memory_store, memory_extractor to match the new subpackage structure.

Suggested change

	from codebase_rag.services.memory_store import memory_store
	from codebase_rag.services.memory_extractor import memory_extractor
	from codebase_rag.services.memory import memory_store, memory_extractor

[Copilot]

Import paths should be from codebase_rag.services.tasks import task_queue, TaskStatus, TaskType to match the new subpackage structure. Note: TaskType should be exported from the tasks subpackage.

Suggested change

	from codebase_rag.services.task_storage import TaskType
	from codebase_rag.services.tasks import TaskType

[Copilot]

Import path should be from codebase_rag.services.tasks import task_queue, TaskStatus to match the new subpackage structure.

Suggested change

	from codebase_rag.services.task_queue import task_queue, TaskStatus
	from codebase_rag.services.tasks import task_queue, TaskStatus

[Copilot]

Import path should be from codebase_rag.services.tasks import task_queue to match the new subpackage structure.

Suggested change

	from codebase_rag.services.task_queue import task_queue
	from codebase_rag.services.tasks import task_queue

[royisme]

@copilot open a new pull request to apply changes based on the comments in this thread

[Copilot]

@royisme I've opened a new pull request, #22, to work on those changes. Once the pull request is ready, I'll request review from you.