refactor: consolidate hardcoded file paths into settings helpers (#118)

Add target_sqlite_db_path() and cocoindex_db_path() to settings.py as
the single source of truth for database paths, replacing scattered
hardcoded "target_sqlite.db" and "cocoindex.db" strings across cli.py,
daemon.py, project.py, and config.py. Also use daemon_log_path() for
daemon log references and project_settings_path() for settings file
references.

Remove config.py (unused legacy module) and its tests — no production
code imported it.

Add settings/index-db location display to `ccc status`.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: georgeh0

CLAUDE.md

@@ -44,6 +44,16 @@ Avoid `Any` whenever feasible. Use specific types — including concrete types f
 
 For functions returning multiple values, use `NamedTuple` instead of plain tuples. At call sites, access fields by name (`result.can_reuse`) rather than positional unpacking — this prevents misreading fields in the wrong order.
 
+### Single Source of Truth
+
+When the same value or logic appears in multiple places, consolidate it into one canonical definition. Don't scatter literals, constants, or path construction across files.
+
+### Dead Code
+
+When changes make code unreachable or unused, delete it along with its tests. Don't leave orphaned modules around.
+
 ### Testing Guidelines
 
 We prefer end-to-end tests on user-facing APIs, over unit tests on smaller internal functions. With this said, there're cases where unit tests are necessary, e.g. for internal logic with various situations and edge cases, in which case it's usually easier to cover various scenarios with unit tests.
+
+When tests fail, fix the underlying issue. Don't skip, ignore, or exclude to get a green result.

src/cocoindex_code/cli.py

@@ -12,13 +12,16 @@
 from .client import DaemonStartError
 from .protocol import DoctorCheckResult, IndexingProgress, ProjectStatusResponse, SearchResponse
 from .settings import (
+    cocoindex_db_path,
     default_project_settings,
     default_user_settings,
     find_parent_with_marker,
     find_project_root,
+    project_settings_path,
     resolve_db_dir,
     save_project_settings,
     save_user_settings,
+    target_sqlite_db_path,
     user_settings_path,
 )
 
@@ -284,10 +287,8 @@ def init(
     force: bool = _typer.Option(False, "-f", "--force", help="Skip parent directory warning"),
 ) -> None:
     """Initialize a project for cocoindex-code."""
-    from .settings import project_settings_path as _project_settings_path
-
     cwd = Path.cwd().resolve()
-    settings_file = _project_settings_path(cwd)
+    settings_file = project_settings_path(cwd)
 
     # Always ensure user settings exist
     user_path = user_settings_path()
@@ -377,8 +378,15 @@ def status() -> None:
     """Show project status."""
     from . import client as _client
 
-    project_root = str(require_project_root())
+    project_root_path = require_project_root()
+    project_root = str(project_root_path)
     print_project_header(project_root)
+
+    _typer.echo(f"Settings: {project_settings_path(project_root_path)}")
+    db_path = target_sqlite_db_path(project_root_path)
+    if db_path.exists():
+        _typer.echo(f"Index DB: {db_path}")
+
     print_index_stats(_client.project_status(project_root))
 
 
@@ -393,10 +401,10 @@ def reset(
     db_dir = resolve_db_dir(project_root)
 
     db_files = [
-        db_dir / "cocoindex.db",
-        db_dir / "target_sqlite.db",
+        cocoindex_db_path(project_root),
+        target_sqlite_db_path(project_root),
     ]
-    settings_file = cocoindex_dir / "settings.yml"
+    settings_file = project_settings_path(project_root)
 
     # Determine what will be deleted
     to_delete = [f for f in db_files if f.exists()]
@@ -503,16 +511,10 @@ def doctor() -> None:
     from .settings import (
         load_user_settings as _load_user_settings,
     )
-    from .settings import (
-        project_settings_path as _project_settings_path,
-    )
-    from .settings import (
-        user_settings_path as _user_settings_path,
-    )
 
     # --- 1. Global settings (local, no daemon needed) ---
     _print_section("Global Settings")
-    settings_path = _user_settings_path()
+    settings_path = user_settings_path()
     _typer.echo(f"  Settings: {settings_path}")
     try:
         user_settings = _load_user_settings()
@@ -570,7 +572,7 @@ def doctor() -> None:
     # --- 6. Project settings (local, no daemon needed) ---
     if project_root is not None:
         _print_section("Project Settings")
-        ps_path = _project_settings_path(project_root)
+        ps_path = project_settings_path(project_root)
         _typer.echo(f"  Settings: {ps_path}")
         try:
             ps = _load_project_settings(project_root)
@@ -597,10 +599,9 @@ def doctor() -> None:
 
     # --- 8. Log files ---
     _print_section("Log Files")
-    from .daemon import daemon_dir as _daemon_dir
+    from .daemon import daemon_log_path as _daemon_log_path
 
-    log_dir = _daemon_dir()
-    _typer.echo(f"  Daemon logs: {log_dir / 'daemon.log'}")
+    _typer.echo(f"  Daemon logs: {_daemon_log_path()}")
     _typer.echo("  Check logs above for further troubleshooting.")
 
 

src/cocoindex_code/client.py

@@ -343,10 +343,10 @@ def start_daemon() -> subprocess.Popen[bytes]:
     Returns the ``Popen`` object so callers can detect early process death
     (via ``proc.poll()``) instead of waiting for a full timeout.
     """
-    from .daemon import daemon_dir
+    from .daemon import daemon_dir, daemon_log_path
 
     daemon_dir().mkdir(parents=True, exist_ok=True)
-    log_path = daemon_dir() / "daemon.log"
+    log_path = daemon_log_path()
 
     ccc_path = _find_ccc_executable()
     if ccc_path:

src/cocoindex_code/config.py

@@ -48,7 +48,7 @@
 from .settings import (
     global_settings_mtime_us,
     load_user_settings,
-    resolve_db_dir,
+    target_sqlite_db_path,
     user_settings_dir,
 )
 from .shared import Embedder, create_embedder
@@ -346,7 +346,7 @@ async def _check_index_status(project_root_str: str) -> DoctorCheckResult:
     from cocoindex.connectors import sqlite as coco_sqlite
 
     project_root = Path(project_root_str)
-    db_path = resolve_db_dir(project_root) / "target_sqlite.db"
+    db_path = target_sqlite_db_path(project_root)
     details = [f"Index: {db_path}"]
 
     if not db_path.exists():

src/cocoindex_code/daemon.py

@@ -21,7 +21,15 @@
     SearchResult,
 )
 from .query import query_codebase
-from .settings import resolve_db_dir
+from .settings import (
+    cocoindex_db_path as _cocoindex_db_path,
+)
+from .settings import (
+    resolve_db_dir,
+)
+from .settings import (
+    target_sqlite_db_path as _target_sqlite_db_path,
+)
 from .shared import (
     CODEBASE_DIR,
     EMBEDDER,
@@ -171,7 +179,7 @@ async def search(
         offset: int = 0,
     ) -> list[SearchResult]:
         """Search within this project."""
-        target_db = resolve_db_dir(self._project_root) / "target_sqlite.db"
+        target_db = _target_sqlite_db_path(self._project_root)
         results = await query_codebase(
             query=query,
             target_sqlite_db_path=target_db,
@@ -261,14 +269,14 @@ async def create(
         db_dir = resolve_db_dir(project_root)
         db_dir.mkdir(parents=True, exist_ok=True)
 
-        cocoindex_db_path = db_dir / "cocoindex.db"
-        target_sqlite_db_path = db_dir / "target_sqlite.db"
+        cocoindex_db = _cocoindex_db_path(project_root)
+        target_sqlite_db = _target_sqlite_db_path(project_root)
 
-        settings = coco.Settings.from_env(cocoindex_db_path)
+        settings = coco.Settings.from_env(cocoindex_db)
 
         context = coco.ContextProvider()
         context.provide(CODEBASE_DIR, project_root)
-        context.provide(SQLITE_DB, coco_sqlite.connect(str(target_sqlite_db_path), load_vec=True))
+        context.provide(SQLITE_DB, coco_sqlite.connect(str(target_sqlite_db), load_vec=True))
         context.provide(EMBEDDER, embedder)
 
         env = coco.Environment(settings, context_provider=context)

src/cocoindex_code/project.py

@@ -194,6 +194,20 @@ def _reset_db_path_mapping_cache() -> None:
     _db_path_mapping = None
 
 
+_TARGET_SQLITE_DB_NAME = "target_sqlite.db"
+_COCOINDEX_DB_NAME = "cocoindex.db"
+
+
+def target_sqlite_db_path(project_root: Path) -> Path:
+    """Return the path to the vector index SQLite database for a project."""
+    return resolve_db_dir(project_root) / _TARGET_SQLITE_DB_NAME
+
+
+def cocoindex_db_path(project_root: Path) -> Path:
+    """Return the path to the CocoIndex state database for a project."""
+    return resolve_db_dir(project_root) / _COCOINDEX_DB_NAME
+
+
 def user_settings_dir() -> Path:
     """Return ``~/.cocoindex_code/``.
 
@@ -238,7 +252,7 @@ def find_legacy_project_root(start: Path) -> Path | None:
     """
     current = start.resolve()
     while True:
-        if (current / _SETTINGS_DIR_NAME / "cocoindex.db").exists():
+        if (current / _SETTINGS_DIR_NAME / _COCOINDEX_DB_NAME).exists():
             return current
         parent = current.parent
         if parent == current: