config: route /review clones into local_cache_dir (default local-cache/)

/review and /refactor previously cloned into Path.cwd() — running
luxe from the project root scattered repo clones (elara, neon-rain,
zoleb, never-say-yes) at /luxe/<name>/. They were gitignored
individually but littered the project tree.

LuxeConfig.local_cache_dir (default "local-cache") plus
LuxeConfig.cache_dir() helper expand the path (absolute, ~/, or
relative to cwd) and ensure the directory exists. Both resolve_repo
callsites in luxe_cli/repl/review.py and luxe_cli/review.py now
pass cfg.cache_dir() instead of Path.cwd().

.gitignore: replace the per-repo entries (/ecliptic/, /luxe/zoleb/,
/luxe/elara/, etc.) with a single `local-cache/` rule. Sibling
projects /ecliptic/ and /warrens/ that aren't luxe-managed get
their own short ignore block.

Cleaned up the existing local tree: moved luxe/{elara,neon-rain,
never-say-yes,zoleb} → luxe/local-cache/. Nothing was tracked in
those dirs, so the move is a pure filesystem reorg.

agents.yaml documents the new field with a one-line comment +
points users at ~/.luxe/cache as an alternative.

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

Author: michaeldtimpe

.gitignore

@@ -75,15 +75,14 @@ luxe/scripts/results/
 # Uv / pip local state
 .uv/
 
-# Local clones pulled in by `/review` and `/refactor`. These target
-# external repos the user happens to be analyzing; their source of
-# truth is elsewhere. Add new subdirectories here when /review clones
-# a new target into the cwd.
+# Local clones pulled in by `/review` and `/refactor`. Configured via
+# LuxeConfig.local_cache_dir (defaults to "local-cache" relative to
+# the cwd luxe was invoked from). One blanket rule replaces the
+# previous per-repo entries; set local_cache_dir to ~/.luxe/cache or
+# elsewhere if you'd rather keep clones outside the project tree.
+local-cache/
+
+# Sibling projects that share this workspace but aren't part of luxe
+# (Godot game, personal writing). Not luxe-managed clones.
 /ecliptic/
-/luxe/zoleb/
-/luxe/elara/
-/luxe/never-say-yes/
-/luxe/neon-rain/
-
-# Personal writing outside any tracked project (story drafts, outlines).
 /warrens/

ARCHITECTURE.md

@@ -211,7 +211,11 @@ Missing binaries or missing project markers produce a helpful
 
 `configs/agents.yaml` holds:
 - Top-level: `ollama_base_url` (legacy fallback URL),
-  `draw_things_url`, `image_output_dir`, `session_dir`
+  `draw_things_url`, `image_output_dir`, `session_dir`,
+  `local_cache_dir` (where `/review` and `/refactor` clone target
+  repos — defaults to `local-cache` next to wherever luxe was launched;
+  blanket-ignored in `.gitignore` so clones never accidentally land in
+  a tracked tree)
 - `providers:` — named backend endpoints, e.g.
   `lmstudio: { base_url: "http://127.0.0.1:1234", kind: lmstudio }`.
   Agents reference these by key.

luxe/configs/agents.yaml

@@ -7,6 +7,11 @@ ollama_base_url: "http://127.0.0.1:11434"
 draw_things_url: "http://127.0.0.1:7859"    # Draw Things HTTP server port
 image_output_dir: "~/luxe-images"
 session_dir: "~/.luxe/sessions"
+# Where /review and /refactor clone target repos. Default is
+# `local-cache` relative to wherever you launched luxe; set to
+# `~/.luxe/cache` (or any absolute path) to keep clones out of the
+# project tree entirely.
+local_cache_dir: "local-cache"
 
 # Named backend providers. Agents reference these by key via the
 # `provider:` field. Each agent may also set `endpoint:` directly to

luxe/luxe_cli/README.md

@@ -322,9 +322,13 @@ forceful and reconciles state to `aborted`.
 
 ## Code intelligence — `/review` and `/refactor`
 
-Both take a git URL. They scan the **current folder** for an existing
-clone matching the URL's `origin` (pulled via `git pull --ff-only`);
-if none found, they `git clone` into the cwd as a subdirectory.
+Both take a git URL. They scan **`local_cache_dir`** (default
+`local-cache/` under cwd) for an existing clone matching the URL's
+`origin` and `git pull --ff-only` it; otherwise `git clone` into
+`local_cache_dir/<repo-name>`. Set `local_cache_dir: ~/.luxe/cache`
+in `agents.yaml` to keep clones outside the project tree entirely.
+The default path is in `.gitignore` so clones never accidentally
+land in a tracked repo.
 
 The runtime wraps the repo in a background task that pins every
 subtask to the dedicated agent:

luxe/luxe_cli/registry.py

@@ -103,6 +103,11 @@ class LuxeConfig(BaseModel):
     session_dir: str = "~/.luxe/sessions"
     draw_things_url: str = "http://127.0.0.1:7860"
     image_output_dir: str = "~/luxe-images"
+    # Where /review and /refactor clone target repos. Path is resolved
+    # relative to cwd unless absolute or using ~. Default keeps clones
+    # out of the project root (and out of git, via .gitignore) instead
+    # of dropping them next to the user's source tree.
+    local_cache_dir: str = "local-cache"
     # Deterministic pre-router: when enabled, a keyword/regex scorer
     # short-circuits the LLM router on decisive prompts. Falls through
     # to the LLM on low-confidence decisions. Disable for A/B testing.
@@ -119,6 +124,19 @@ class LuxeConfig(BaseModel):
     default_provider: str | None = None
     agents: list[AgentConfig]
 
+    def cache_dir(self) -> Path:
+        """Resolve local_cache_dir to an absolute path and ensure it exists.
+
+        Absolute paths and ~/ paths are honored as-given; relative
+        paths resolve against the current working directory. The
+        directory is created if missing — clones land directly inside.
+        """
+        p = Path(self.local_cache_dir).expanduser()
+        if not p.is_absolute():
+            p = Path.cwd() / p
+        p.mkdir(parents=True, exist_ok=True)
+        return p
+
     def get(self, name: str) -> AgentConfig:
         for a in self.agents:
             if a.name == name:

luxe/luxe_cli/repl/review.py

@@ -22,7 +22,7 @@ def _start_review(url: str, mode: str, state: "ReplState", cfg: LuxeConfig) -> N
 
     console.print(f"[dim]resolving[/dim] [cyan]{url}[/cyan][dim]...[/dim]")
     with console.status("[dim]git[/dim]", spinner="dots"):
-        repo_path, status_msg = resolve_repo(url, Path.cwd())
+        repo_path, status_msg = resolve_repo(url, cfg.cache_dir())
     if repo_path is None:
         console.print(f"[red]{status_msg}[/red]")
         return

luxe/luxe_cli/review.py

@@ -80,7 +80,7 @@ def start_review_task(
     """Plan + persist + spawn a review/refactor task. Returns task id.
 
     Headless — no ReplState needed. Used by `luxe analyze --review`."""
-    repo_path, status_msg = resolve_repo(str(url_or_path), Path.cwd())
+    repo_path, status_msg = resolve_repo(str(url_or_path), cfg.cache_dir())
     if repo_path is None:
         raise RuntimeError(status_msg)
 

luxe/tests/test_cache_dir.py

@@ -0,0 +1,58 @@
+"""Tests for LuxeConfig.cache_dir() resolution + dispatch wiring."""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+
+from luxe_cli.registry import LuxeConfig
+
+
+def _cfg(**overrides) -> LuxeConfig:
+    base = dict(agents=[])
+    base.update(overrides)
+    return LuxeConfig(**base)
+
+
+def test_cache_dir_default_is_local_cache_under_cwd(tmp_path, monkeypatch):
+    monkeypatch.chdir(tmp_path)
+    cfg = _cfg()
+    p = cfg.cache_dir()
+    assert p == tmp_path / "local-cache"
+    assert p.is_dir()
+
+
+def test_cache_dir_honors_absolute_path(tmp_path):
+    target = tmp_path / "elsewhere"
+    cfg = _cfg(local_cache_dir=str(target))
+    p = cfg.cache_dir()
+    assert p == target
+    assert p.is_dir()
+
+
+def test_cache_dir_expands_tilde(tmp_path, monkeypatch):
+    monkeypatch.setenv("HOME", str(tmp_path))
+    cfg = _cfg(local_cache_dir="~/my-cache")
+    p = cfg.cache_dir()
+    assert p == tmp_path / "my-cache"
+    assert p.is_dir()
+
+
+def test_cache_dir_creates_missing_dir(tmp_path):
+    target = tmp_path / "fresh" / "nested"
+    cfg = _cfg(local_cache_dir=str(target))
+    assert not target.exists()
+    cfg.cache_dir()
+    assert target.is_dir()
+
+
+def test_real_agents_yaml_resolves_local_cache(tmp_path, monkeypatch):
+    """The shipped config defaults to local-cache/, not cwd."""
+    monkeypatch.chdir(tmp_path)
+    from luxe_cli.registry import load_config
+
+    cfg = load_config()
+    p = cfg.cache_dir()
+    assert p.name == "local-cache"
+    assert p.parent == tmp_path