fix: use = delimiter for DB path mapping (avoid Windows colon conflict)

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

Author: georgeh0

README.md

@@ -425,15 +425,15 @@ By default, index databases (`cocoindex.db` and `target_sqlite.db`) live alongsi
 Set `COCOINDEX_CODE_DB_PATH_MAPPING` to remap database locations by path prefix:
 
 ```bash
-COCOINDEX_CODE_DB_PATH_MAPPING=/workspace:/db-files
+COCOINDEX_CODE_DB_PATH_MAPPING=/workspace=/db-files
 ```
 
 With this mapping, a project at `/workspace/myrepo` stores its databases in `/db-files/myrepo/` instead of `/workspace/myrepo/.cocoindex_code/`. Settings files remain in the original location.
 
 Multiple mappings are comma-separated and resolved in order (first match wins):
 
 ```bash
-COCOINDEX_CODE_DB_PATH_MAPPING=/workspace:/db-files,/workspace2:/db-files2
+COCOINDEX_CODE_DB_PATH_MAPPING=/workspace=/db-files,/workspace2=/db-files2
 ```
 
 Both source and target must be absolute paths. If no mapping matches, the default location is used.

src/cocoindex_code/settings.py

@@ -131,7 +131,7 @@ class DbPathMapping:
 def _parse_db_path_mapping() -> list[DbPathMapping]:
     """Parse ``COCOINDEX_CODE_DB_PATH_MAPPING`` env var.
 
-    Format: ``/src1:/dst1,/src2:/dst2``
+    Format: ``/src1=/dst1,/src2=/dst2``
     Both source and target must be absolute paths.
     """
     raw = os.environ.get(_ENV_DB_PATH_MAPPING, "")
@@ -143,11 +143,10 @@ def _parse_db_path_mapping() -> list[DbPathMapping]:
         entry = entry.strip()
         if not entry:
             continue
-        # Split on first colon only (to be robust with edge cases)
-        parts = entry.split(":", 1)
+        parts = entry.split("=", 1)
         if len(parts) != 2 or not parts[0] or not parts[1]:
             raise ValueError(
-                f"{_ENV_DB_PATH_MAPPING}: invalid entry {entry!r}, expected format 'source:target'"
+                f"{_ENV_DB_PATH_MAPPING}: invalid entry {entry!r}, expected format 'source=target'"
             )
         source = Path(parts[0])
         target = Path(parts[1])

tests/test_e2e.py

@@ -606,7 +606,7 @@ def e2e_project_with_db_mapping() -> Iterator[tuple[Path, Path]]:
     }
     os.environ["COCOINDEX_CODE_DIR"] = str(base_dir)
     workspace = str(base_dir / "workspace")
-    os.environ["COCOINDEX_CODE_DB_PATH_MAPPING"] = f"{workspace}:{db_base_dir}"
+    os.environ["COCOINDEX_CODE_DB_PATH_MAPPING"] = f"{workspace}={db_base_dir}"
     _reset_db_path_mapping_cache()
     old_cwd = os.getcwd()
     os.chdir(project_dir)

tests/test_settings.py

@@ -220,45 +220,45 @@ def test_no_mapping(self) -> None:
         )
 
     def test_single_mapping_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
         assert resolve_db_dir(Path("/workspace/myproject")) == Path("/db-files/myproject")
 
     def test_exact_root_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
         assert resolve_db_dir(Path("/workspace")) == Path("/db-files")
 
     def test_no_match_falls_back(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
         assert resolve_db_dir(Path("/other/myproject")) == Path("/other/myproject/.cocoindex_code")
 
     def test_multiple_mappings_first_wins(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db1,/workspace/sub:/db2")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db1,/workspace/sub=/db2")
         assert resolve_db_dir(Path("/workspace/sub/proj")) == Path("/db1/sub/proj")
 
     def test_multiple_mappings_second_matches(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db1,/other:/db2")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db1,/other=/db2")
         assert resolve_db_dir(Path("/other/proj")) == Path("/db2/proj")
 
     def test_no_partial_component_match(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
         assert resolve_db_dir(Path("/workspace2/proj")) == Path("/workspace2/proj/.cocoindex_code")
 
     def test_rejects_relative_source(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "relative/path:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "relative/path=/db-files")
         with pytest.raises(ValueError, match="source path must be absolute"):
             resolve_db_dir(Path("/anything"))
 
     def test_rejects_relative_target(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:relative/path")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=relative/path")
         with pytest.raises(ValueError, match="target path must be absolute"):
             resolve_db_dir(Path("/anything"))
 
     def test_skips_empty_entries(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files,,/other:/db2,")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files,,/other=/db2,")
         assert resolve_db_dir(Path("/other/proj")) == Path("/db2/proj")
 
     def test_nested_project(self, monkeypatch: pytest.MonkeyPatch) -> None:
-        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace:/db-files")
+        monkeypatch.setenv("COCOINDEX_CODE_DB_PATH_MAPPING", "/workspace=/db-files")
         assert resolve_db_dir(Path("/workspace/org/repo/subdir")) == Path(
             "/db-files/org/repo/subdir"
         )