overkill 0.9.0__tar.gz → 0.9.1__tar.gz

{overkill-0.9.0 → overkill-0.9.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: overkill
-Version: 0.9.0
+Version: 0.9.1
 Summary: AI-powered code review loop — automates Codex/Gemini review + Claude fix cycles
 Project-URL: Repository, https://github.com/modocai/mr-overkill
 Project-URL: Issues, https://github.com/modocai/mr-overkill/issues

{overkill-0.9.0 → overkill-0.9.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "overkill"
-version = "0.9.0"
+version = "0.9.1"
 description = "AI-powered code review loop — automates Codex/Gemini review + Claude fix cycles"
 readme = "README.md"
 license = { text = "MIT" }

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/budget/claude.py

@@ -45,21 +45,42 @@ _TIER_LIMITS: dict[str, int] = {
 }
 
 _TIER_MAP: dict[str, str] = {
+    "default": "pro",
     "default_claude_max_20x": "max20",
     "default_claude_max_5x": "max5",
 }
 
 
-def detect_tier(telemetry_dir: Path | None = None) -> str:
+#: Reported in place of a tier that could not be determined.
+UNKNOWN_TIER = "unknown"
+
+
+def detect_tier(
+    telemetry_dir: Path | None = None, *, warn: bool = True
+) -> str | None:
     """Read rateLimitTier from Claude Code telemetry files.
 
-    Returns ``'pro'``, ``'max5'``, or ``'max20'``.
+    Returns ``'pro'``, ``'max5'``, ``'max20'``, or ``None`` when the tier
+    cannot be determined.
+
+    Pass ``warn=False`` from callers that only use the tier as a display
+    label.  The warning below says the budget cannot be sized, which is only
+    true where the tier is a denominator; on the OAuth path it is noise, and
+    the budget gate re-checks per agent invocation, so it repeats.
+
+    ``None`` rather than a default on purpose.  This value is only ever used
+    as the denominator of a usage percentage, so guessing it does not degrade
+    gracefully: guessing ``pro`` for a max20 account reports 995% where the
+    truth is 49%, and the run blocks on a budget it has not used.  Telemetry
+    formats change — ``user_attributes`` disappeared from the files once
+    already — so not knowing has to be sayable.
     """
     if telemetry_dir is None:
         telemetry_dir = Path.home() / ".claude" / "telemetry"
 
     if not telemetry_dir.is_dir():
-        return "pro"
+        logger.debug("No telemetry directory at %s — tier unknown.", telemetry_dir)
+        return None
 
     best_ts = ""
     best_tier = ""
@@ -113,7 +134,22 @@ def detect_tier(telemetry_dir: Path | None = None) -> str:
                 best_ts = ts
                 best_tier = str(tier_raw)
 
-    return _TIER_MAP.get(best_tier, "pro")
+    if not best_tier:
+        # Debug, not a warning: the only caller that sizes a limit with this
+        # is check_local(), which logs its own accurate message when the
+        # missing tier actually costs it a percentage.
+        logger.debug(
+            "No rateLimitTier found in %s — cannot size the token limit.",
+            telemetry_dir,
+        )
+        return None
+    tier = _TIER_MAP.get(best_tier)
+    if tier is None and warn:
+        logger.warning(
+            "Unrecognised rateLimitTier %r — cannot size the token limit.",
+            best_tier,
+        )
+    return tier
 
 
 def check_oauth() -> BudgetStatus | None:
@@ -179,7 +215,9 @@ def check_oauth() -> BudgetStatus | None:
             logger.debug("OAuth: no utilization in response")
             return None
 
-        tier = detect_tier()
+        # Only a label here: OAuth reports utilization directly, so an
+        # unknown tier costs nothing on this path — hence warn=False.
+        tier = detect_tier(warn=False) or UNKNOWN_TIER
         seven_day = data.get("seven_day")
 
         return BudgetStatus(
@@ -207,9 +245,15 @@ def check_local(projects_dir: Path | None = None) -> BudgetStatus:
 
     Scans files modified in the last 5 hours, sums token usage
     (deduplicating by message.id), and calculates percentage.
+
+    ``five_hour_used_pct`` is ``None`` when tokens were used but the tier —
+    and so the limit to measure them against — could not be determined.  A
+    percentage from a guessed limit is worse than no percentage at all:
+    ``budget_sufficient`` treats ``None`` as "assume OK", where a wrong number
+    blocks the run outright.
     """
     tier = detect_tier()
-    limit = _TIER_LIMITS.get(tier, _TIER_LIMITS["pro"])
+    limit = _TIER_LIMITS.get(tier) if tier else None
 
     if projects_dir is None:
         projects_dir = Path.home() / ".claude" / "projects"
@@ -260,24 +304,39 @@ def check_local(projects_dir: Path | None = None) -> BudgetStatus:
     for usage in seen_ids.values():
         total_tokens += _sum_usage(usage)
 
-    pct = (int(total_tokens) * 100 // limit) if total_tokens > 0 and limit > 0 else 0
-
-    logger.info(
-        "Local estimate: %d%% (%d weighted tokens / %d limit, "
-        "tier=%s, %d deduplicated messages)",
-        pct,
-        int(total_tokens),
-        limit,
-        tier,
-        len(seen_ids),
-    )
+    pct: int | None
+    if total_tokens <= 0:
+        # Nothing used needs no denominator.
+        pct = 0
+    elif limit:
+        pct = int(total_tokens) * 100 // limit
+    else:
+        pct = None
+
+    if pct is None:
+        logger.warning(
+            "Local estimate unavailable: %d weighted tokens over %d "
+            "deduplicated messages, but no tier to size the limit against.",
+            int(total_tokens),
+            len(seen_ids),
+        )
+    else:
+        logger.info(
+            "Local estimate: %d%% (%d weighted tokens / %s limit, "
+            "tier=%s, %d deduplicated messages)",
+            pct,
+            int(total_tokens),
+            limit if limit else "unknown",
+            tier or UNKNOWN_TIER,
+            len(seen_ids),
+        )
 
     return BudgetStatus(
         five_hour_used_pct=pct,
         seven_day_used_pct=None,
         tokens_used=int(total_tokens),
         mode="local",
-        tier=tier,
+        tier=tier or UNKNOWN_TIER,
         resets_at=None,
         seven_day_resets_at=None,
         estimated=True,
@@ -365,16 +424,28 @@ def check_token_budget() -> BudgetStatus:
 
     if cached is not None:
         if cached.five_hour_used_pct is not None:
-            # Use the higher 5h value between cached and local to avoid
-            # underreporting when cache is stale.
+            # Prefer the higher of the two, so a stale cache cannot
+            # under-report.  Only when the local estimate has a percentage at
+            # all: without a known tier it has no denominator, and letting it
+            # win then would replace a real reading with a guess — which is
+            # how a 44% cache once became a 995% refusal.
             if (
                 local.five_hour_used_pct is not None
                 and local.five_hour_used_pct > cached.five_hour_used_pct
             ):
-                cached = dataclasses.replace(
+                logger.info(
+                    "Using cached OAuth budget data, raised from %d%% to the "
+                    "higher local estimate of %d%%.",
+                    cached.five_hour_used_pct,
+                    local.five_hour_used_pct,
+                )
+                return dataclasses.replace(
                     cached, five_hour_used_pct=local.five_hour_used_pct,
                 )
-            logger.info("Using cached OAuth budget data.")
+            logger.info(
+                "Using cached OAuth budget data (%d%% used).",
+                cached.five_hour_used_pct,
+            )
             return cached
         # 5-hour expired but 7-day still valid — merge 7-day into local.
         logger.info(

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/cli.py

@@ -13,7 +13,7 @@ import subprocess
 from dataclasses import dataclass
 from pathlib import Path
 
-from mr_overkill import __version__
+from mr_overkill import __version__, workspace_policy
 from mr_overkill.commit_scope import resolve_commit
 from mr_overkill.models import BudgetScope, LoopConfig
 
@@ -84,17 +84,21 @@ def _load_rc_file(rc_name: str) -> dict[str, str]:
         return {}
 
     git_root = Path(result.stdout.strip())
-    rc_path = git_root / ".overkill" / rc_name
+    rc_path = workspace_policy.workspace_path(git_root, rc_name)
+    legacy_dir = workspace_policy.legacy_workspace_path(git_root, rc_name)
     if not rc_path.is_file():
-        # Fall back to the legacy .review-loop/ location
-        legacy_dir = git_root / ".review-loop" / rc_name
-        # Also check for legacy .reviewlooprc name in .review-loop/
+        # Also check for the legacy rc name in the legacy directory
         legacy_old_name = (
-            git_root / ".review-loop" / ".reviewlooprc"
-            if rc_name == ".overkillrc" else None
+            workspace_policy.legacy_workspace_path(
+                git_root, workspace_policy.LEGACY_RC_NAME
+            )
+            if rc_name == workspace_policy.RC_NAME else None
         )
         # Legacy repo-root name: .overkillrc was .reviewlooprc
-        legacy_root_name = ".reviewlooprc" if rc_name == ".overkillrc" else rc_name
+        legacy_root_name = (
+            workspace_policy.LEGACY_RC_NAME
+            if rc_name == workspace_policy.RC_NAME else rc_name
+        )
         legacy_root = git_root / legacy_root_name
         if legacy_dir.is_file():
             logging.getLogger(__name__).warning(
@@ -177,9 +181,11 @@ def _resolve_prompts_dir(prompts_dir: str) -> Path:
         git_root = Path(result.stdout.strip())
         resolved = git_root / prompts_dir
         # Legacy fallback: .overkill/... → .review-loop/...
-        if not resolved.is_dir() and ".overkill/" in prompts_dir:
+        current_prefix = f"{workspace_policy.WORKSPACE_DIR}/"
+        legacy_prefix = f"{workspace_policy.LEGACY_WORKSPACE_DIR}/"
+        if not resolved.is_dir() and current_prefix in prompts_dir:
             legacy = git_root / prompts_dir.replace(
-                ".overkill/", ".review-loop/", 1
+                current_prefix, legacy_prefix, 1
             )
             if legacy.is_dir():
                 logging.getLogger(__name__).warning(
@@ -189,9 +195,9 @@ def _resolve_prompts_dir(prompts_dir: str) -> Path:
                 )
                 return legacy
         # Inverse fallback: .review-loop/... → .overkill/...
-        if not resolved.is_dir() and ".review-loop/" in prompts_dir:
+        if not resolved.is_dir() and legacy_prefix in prompts_dir:
             migrated = git_root / prompts_dir.replace(
-                ".review-loop/", ".overkill/", 1
+                legacy_prefix, current_prefix, 1
             )
             if migrated.is_dir():
                 logging.getLogger(__name__).warning(
@@ -204,6 +210,24 @@ def _resolve_prompts_dir(prompts_dir: str) -> Path:
     return p
 
 
+def _resolve_log_dir(git_root: Path, *parts: str, resume: bool) -> Path:
+    """Log directory for this run, preferring the current workspace layout.
+
+    A repo half-migrated between the two layouts can hold both, so a resume
+    additionally prefers whichever directory actually has the run metadata —
+    picking the empty one would report there is nothing to resume.
+    """
+    candidates = workspace_policy.workspace_paths(git_root, "logs", *parts)
+    log_dir = next((c for c in candidates if c.is_dir()), candidates[0])
+    if resume and not (log_dir / "max-loop.txt").is_file():
+        with_metadata = next(
+            (c for c in candidates if (c / "max-loop.txt").is_file()), None
+        )
+        if with_metadata is not None:
+            log_dir = with_metadata
+    return log_dir
+
+
 def _int_from_rc(
     rc: dict[str, str],
     key: str,
@@ -403,7 +427,7 @@ def parse_review_loop_args(
             )
 
     # Load rc file defaults
-    rc = _load_rc_file(".overkillrc")
+    rc = _load_rc_file(workspace_policy.RC_NAME)
 
     # Resolve values with precedence: CLI > rc file > defaults
     target = args.target or rc.get("TARGET_BRANCH", "develop")
@@ -449,7 +473,7 @@ def parse_review_loop_args(
     budget_scope_str = rc.get("BUDGET_SCOPE", "micro")
 
     prompts_dir = _resolve_prompts_dir(
-        rc.get("PROMPTS_DIR", ".overkill/prompts/active")
+        rc.get("PROMPTS_DIR", workspace_policy.DEFAULT_PROMPTS_DIR)
     )
 
     current_branch = _detect_current_branch()
@@ -462,18 +486,7 @@ def parse_review_loop_args(
         check=False,
     )
     git_root = Path(result.stdout.strip()) if result.returncode == 0 else Path(".")
-    log_dir = git_root / ".overkill" / "logs"
-    if not log_dir.is_dir():
-        legacy_log = git_root / ".review-loop" / "logs"
-        if legacy_log.is_dir():
-            log_dir = legacy_log
-
-    # In mixed-state repos, fall back to legacy logs when resume metadata
-    # is missing from the preferred directory.
-    if args.resume and not (log_dir / "max-loop.txt").is_file():
-        legacy_log = git_root / ".review-loop" / "logs"
-        if (legacy_log / "max-loop.txt").is_file():
-            log_dir = legacy_log
+    log_dir = _resolve_log_dir(git_root, resume=args.resume)
 
     # Restore saved values on resume when not explicitly given
     restored_target = None
@@ -818,7 +831,7 @@ def parse_refactor_suggest_args(
     args = parser.parse_args(argv)
 
     # Load rc file defaults
-    rc = _load_rc_file(".refactorsuggestrc")
+    rc = _load_rc_file(workspace_policy.REFACTOR_RC_NAME)
 
     # Resolve values: CLI > rc > defaults
     scope = args.scope or rc.get("SCOPE", "auto")
@@ -880,7 +893,7 @@ def parse_refactor_suggest_args(
     budget_scope_str = rc.get("BUDGET_SCOPE", "module")
 
     prompts_dir = _resolve_prompts_dir(
-        rc.get("PROMPTS_DIR", ".overkill/prompts/active")
+        rc.get("PROMPTS_DIR", workspace_policy.DEFAULT_PROMPTS_DIR)
     )
 
     current_branch = _detect_current_branch()
@@ -896,18 +909,7 @@ def parse_refactor_suggest_args(
         Path(result.stdout.strip()) if result.returncode == 0
         else Path(".")
     )
-    log_dir = git_root / ".overkill" / "logs" / "refactor"
-    if not log_dir.is_dir():
-        legacy_log = git_root / ".review-loop" / "logs" / "refactor"
-        if legacy_log.is_dir():
-            log_dir = legacy_log
-
-    # In mixed-state repos, fall back to legacy logs when resume metadata
-    # is missing from the preferred directory.
-    if args.resume and not (log_dir / "max-loop.txt").is_file():
-        legacy_log = git_root / ".review-loop" / "logs" / "refactor"
-        if (legacy_log / "max-loop.txt").is_file():
-            log_dir = legacy_log
+    log_dir = _resolve_log_dir(git_root, "refactor", resume=args.resume)
 
     # Restore saved values on resume when not explicitly given
     if args.resume:

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/commit_scope.py

@@ -20,9 +20,10 @@ from pathlib import Path
 
 logger = logging.getLogger(__name__)
 
-# ``git hash-object -t tree /dev/null`` — the canonical empty tree.  Diffing
-# against it turns a root commit (which has no parent to diff against) into a
-# plain "everything was added" patch.
+#: The empty tree under SHA-1, git's default object format.  Only a fallback
+#: for when git cannot be asked — :func:`empty_tree_sha` is the real answer,
+#: because a repository created with ``--object-format=sha256`` has a
+#: different one and this id does not resolve there at all.
 EMPTY_TREE_SHA = "4b825dc642cb6eb9a060e54bf8d69288fbee4904"
 
 
@@ -30,6 +31,26 @@ def _run(cmd: list[str], cwd: Path | None = None) -> subprocess.CompletedProcess
     return subprocess.run(cmd, cwd=cwd, capture_output=True, text=True, check=False)
 
 
+def empty_tree_sha(cwd: Path | None = None) -> str:
+    """The empty tree's object id in *this* repository's object format.
+
+    Diffing against it turns a root commit — which has no parent to diff
+    against — into a plain "everything was added" patch.  Asking git rather
+    than hardcoding the SHA-1 value is what makes that work in a repository
+    using SHA-256, where the hardcoded id resolves to nothing and the diff
+    fails with a message about an empty commit instead.
+    """
+    result = _run(["git", "hash-object", "-t", "tree", "/dev/null"], cwd)
+    sha = result.stdout.strip()
+    if result.returncode != 0 or not sha:
+        logger.debug(
+            "Could not compute the empty tree id (%s); falling back to SHA-1.",
+            result.stderr.strip() or f"exit {result.returncode}",
+        )
+        return EMPTY_TREE_SHA
+    return sha
+
+
 def resolve_commit(rev: str, cwd: Path | None = None) -> str | None:
     """Resolve *rev* to a full commit SHA, or ``None`` if it is not a commit.
 
@@ -57,7 +78,7 @@ def commit_base(sha: str, cwd: Path | None = None) -> str:
     result = _run(["git", "rev-list", "--parents", "-n", "1", sha], cwd)
     fields = result.stdout.split()
     if len(fields) < 2:
-        return EMPTY_TREE_SHA
+        return empty_tree_sha(cwd)
     return fields[1]
 
 

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/git_ops.py

@@ -16,6 +16,7 @@ import subprocess
 import uuid
 from pathlib import Path
 
+from mr_overkill import workspace_policy
 from mr_overkill.models import WorktreeSnapshot
 
 logger = logging.getLogger(__name__)
@@ -222,7 +223,7 @@ def commit_and_push(
     """
     changed = changed_files_since_snapshot(
         snapshot, cwd=cwd,
-        exclude_prefixes=(".overkill/logs/", ".review-loop/logs/"),
+        exclude_prefixes=workspace_policy.LOG_PREFIXES,
     )
     if not changed:
         logger.info("No file changes after fix — nothing to commit.")

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/init.py

@@ -9,10 +9,12 @@ from contextlib import contextmanager
 from importlib.resources import as_file, files
 from pathlib import Path
 
+from mr_overkill import workspace_policy
+
 logger = logging.getLogger(__name__)
 
-_OVERKILL_DIR = ".overkill"
-_LEGACY_DIR = ".review-loop"
+_OVERKILL_DIR = workspace_policy.WORKSPACE_DIR
+_LEGACY_DIR = workspace_policy.LEGACY_WORKSPACE_DIR
 
 
 @contextmanager
@@ -69,8 +71,16 @@ def _copy_rc_files(data: Path, dest: Path) -> list[str]:
     manifest: list[str] = []
     project_root = dest.parent
     rc_pairs = [
-        (".overkillrc.example", ".overkillrc", ".reviewlooprc"),
-        (".refactorsuggestrc.example", ".refactorsuggestrc", None),
+        (
+            f"{workspace_policy.RC_NAME}.example",
+            workspace_policy.RC_NAME,
+            workspace_policy.LEGACY_RC_NAME,
+        ),
+        (
+            f"{workspace_policy.REFACTOR_RC_NAME}.example",
+            workspace_policy.REFACTOR_RC_NAME,
+            None,
+        ),
     ]
     for example_name, live_name, legacy_name in rc_pairs:
         live_path = dest / live_name
@@ -98,8 +108,8 @@ def _copy_rc_files(data: Path, dest: Path) -> list[str]:
 def _ensure_gitignore(project_root: Path) -> None:
     """Add ``.overkill/`` to ``.gitignore`` if absent."""
     gitignore = project_root / ".gitignore"
-    marker = ".overkill/"
-    legacy_marker = ".review-loop/"
+    marker = f"{_OVERKILL_DIR}/"
+    legacy_marker = f"{_LEGACY_DIR}/"
 
     if gitignore.is_file():
         content = gitignore.read_text(encoding="utf-8")
@@ -167,8 +177,8 @@ def _migrate_legacy_dir(target_dir: Path) -> None:
     logger.info("Migrated %s/ → %s/", _LEGACY_DIR, _OVERKILL_DIR)
 
     # Rename legacy RC file inside the migrated directory
-    old_rc = dest / ".reviewlooprc"
-    new_rc = dest / ".overkillrc"
+    old_rc = dest / workspace_policy.LEGACY_RC_NAME
+    new_rc = dest / workspace_policy.RC_NAME
     if old_rc.is_file() and not new_rc.is_file():
         old_rc.rename(new_rc)
         logger.info("Renamed .reviewlooprc → .overkillrc")

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/loop_engine.py

@@ -12,7 +12,7 @@ import subprocess
 from pathlib import Path
 from typing import Protocol
 
-from mr_overkill import wip_scope
+from mr_overkill import wip_scope, workspace_policy
 from mr_overkill.git_ops import (
     commit_and_push,
     diff_hash,
@@ -513,16 +513,7 @@ def review_fix_loop(
         # f. Stash allowlisted files
         try:
             allowed_stashed = stash_allowlisted(
-                [
-                    ".gitignore",
-                    ".overkill/.overkillrc",
-                    ".overkill/.refactorsuggestrc",
-                    # Legacy paths for pre-migration users
-                    ".reviewlooprc",
-                    ".refactorsuggestrc",
-                    ".review-loop/.reviewlooprc",
-                    ".review-loop/.refactorsuggestrc",
-                ],
+                sorted(workspace_policy.ALLOWLISTED_FILES),
                 cwd=cwd,
             )
         except RuntimeError:
@@ -697,24 +688,7 @@ def _validate_target_branch(target: str, cwd: Path | None) -> bool:
 
 def _reject_dirty_worktree(cwd: Path | None) -> list[str]:
     """Return non-allowlisted dirty files, or empty list if clean."""
-    allowlisted = {
-        ".gitignore",
-        ".overkill/.overkillrc",
-        ".overkill/.refactorsuggestrc",
-        # Legacy paths for pre-migration users
-        ".overkillrc",
-        ".reviewlooprc",
-        ".refactorsuggestrc",
-        ".review-loop/.overkillrc",
-        ".review-loop/.reviewlooprc",
-        ".review-loop/.refactorsuggestrc",
-    }
-    return [
-        f for f in git_all_dirty(cwd)
-        if f not in allowlisted
-        and not f.startswith(".overkill/logs/")
-        and not f.startswith(".review-loop/logs/")
-    ]
+    return workspace_policy.non_allowlisted(git_all_dirty(cwd))
 
 
 def _no_diff(target: str, current: str, cwd: Path | None) -> bool:

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/models.py

@@ -80,8 +80,11 @@ class BudgetStatus:
     five_hour_used_pct: int | None
     seven_day_used_pct: int | None
     tokens_used: int  # weighted sum for local estimation (not raw count)
-    mode: str  # "oauth" | "local" | "session_log" | "apikey" | "no_data"
-    tier: str  # "pro" | "max5" | "max20"
+    # "oauth" | "local" | "cached" | "session_log"
+    # | "apikey" | "no_data" | "synthetic" | "unknown"
+    mode: str
+    # "" means the checker has no tier concept; budget_report relies on it.
+    tier: str  # "pro" | "max5" | "max20" | "unknown" | ""
     resets_at: str | None
     seven_day_resets_at: str | None = None
     estimated: bool = False

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/refactor_suggest.py

@@ -10,6 +10,7 @@ import logging
 import subprocess
 from datetime import UTC, datetime
 
+from mr_overkill import workspace_policy
 from mr_overkill.agents import (
     create_fix_agent,
     create_review_agent,
@@ -18,7 +19,11 @@ from mr_overkill.agents import (
 from mr_overkill.budget import budget_gate_disabled, budget_sufficient
 from mr_overkill.budget.claude import check_token_budget as claude_budget
 from mr_overkill.budget.codex import check_token_budget as codex_budget
-from mr_overkill.git_ops import git_all_dirty, stash_allowlisted, unstash_allowlisted
+from mr_overkill.git_ops import (
+    git_all_dirty,
+    stash_allowlisted,
+    unstash_allowlisted,
+)
 from mr_overkill.loop_engine import PreFixConfirmFn, review_fix_loop
 from mr_overkill.models import (
     BudgetScope,
@@ -108,18 +113,7 @@ def _get_budget_status(tool: str) -> BudgetStatus:
 # ── Branch management ────────────────────────────────────────────────
 
 
-_ALLOWLISTED_FILES = [
-    ".gitignore",
-    ".overkill/.overkillrc",
-    ".overkill/.refactorsuggestrc",
-    # Legacy paths for pre-migration users
-    ".overkillrc",
-    ".reviewlooprc",
-    ".refactorsuggestrc",
-    ".review-loop/.overkillrc",
-    ".review-loop/.reviewlooprc",
-    ".review-loop/.refactorsuggestrc",
-]
+_ALLOWLISTED_FILES = sorted(workspace_policy.ALLOWLISTED_FILES)
 
 
 def create_refactor_branch(
@@ -348,13 +342,7 @@ def run(config: LoopConfig, scope: str, *, create_pr: bool = False) -> int:
 
     # Reject non-allowlisted dirty files before creating a branch
     if not config.dry_run and not config.resume:
-        allowlisted = set(_ALLOWLISTED_FILES)
-        non_allowed = [
-            f for f in git_all_dirty(None)
-            if f not in allowlisted
-            and not f.startswith(".overkill/logs/")
-            and not f.startswith(".review-loop/logs/")
-        ]
+        non_allowed = workspace_policy.non_allowlisted(git_all_dirty(None))
         if non_allowed:
             logger.error(
                 "Working tree is not clean. Commit or stash your changes "

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/self_review.py

@@ -11,6 +11,7 @@ import string
 import subprocess
 from pathlib import Path
 
+from mr_overkill import workspace_policy
 from mr_overkill.git_ops import changed_files_since_snapshot
 from mr_overkill.json_extract import (
     inject_refactoring_plan,
@@ -104,7 +105,7 @@ def self_review_subloop(
         # Check if fix produced changes
         changed = changed_files_since_snapshot(
             pre_fix_snapshot, cwd=cwd,
-            exclude_prefixes=(".overkill/logs/", ".review-loop/logs/"),
+            exclude_prefixes=workspace_policy.LOG_PREFIXES,
         )
         if not changed:
             logger.info("No working tree changes from fix — skipping self-review.")

{overkill-0.9.0 → overkill-0.9.1}/src/mr_overkill/wip_scope.py

@@ -22,15 +22,15 @@ import logging
 import subprocess
 from pathlib import Path
 
+from mr_overkill import workspace_policy
 from mr_overkill.git_ops import git_all_dirty
 
 logger = logging.getLogger(__name__)
 
 SCAFFOLD_MESSAGE = "wip: review-loop scaffolding [skip ci]"
 
-# Log artefacts are not part of anyone's work in progress.  Same list as
-# ``git_ops.commit_and_push``'s ``exclude_prefixes``.
-_LOG_PREFIXES = (".overkill/logs/", ".review-loop/logs/")
+# Log artefacts are not part of anyone's work in progress.
+_LOG_PREFIXES = workspace_policy.LOG_PREFIXES
 
 
 def _run(