claude-dev-env 1.42.0 → 1.44.0

package/scripts/sweep_empty_dirs.py

@@ -16,7 +16,7 @@ import os
 import sys
 import time
 
-from config.timing import DEFAULT_AGE_SECONDS, DEFAULT_POLL_INTERVAL
+from dev_env_scripts_constants.timing import DEFAULT_AGE_SECONDS, DEFAULT_POLL_INTERVAL
 
 
 def _positive_int(raw_argument: str) -> int:
@@ -42,6 +42,15 @@ def sweep(root: str, min_age_seconds: int) -> list[str]:
     Walks bottom-up so nested empty directories are cleaned from the leaves
     inward.  Relies on os.rmdir to fail harmlessly for non-empty directories
     instead of checking snapshotted subdirectory lists.
+
+    Args:
+        root: Root directory to walk; directories below the root are
+            candidates for removal.
+        min_age_seconds: Minimum age in seconds for a directory to qualify
+            for removal.
+
+    Returns:
+        Sorted list of directory paths that were removed during the sweep.
     """
 
     all_removed: list[str] = []

package/scripts/test_setup_project_paths.py

@@ -22,8 +22,8 @@ for each_sys_path_entry in (
 
 import setup_project_paths as setup
 import untracked_repo_detector as detector_module
-from config.project_paths_reader import registry_file_path
-from config.setup_project_paths_constants import (
+from hooks_constants.project_paths_reader import registry_file_path
+from hooks_constants.setup_project_paths_constants import (
     ABORTED_NOTHING_WRITTEN_MESSAGE,
     CONFIRMATION_PROMPT_TEXT,
     ES_EXE_FOLDERS_ONLY_QUERY_ARGUMENTS,

package/scripts/test_sweep_empty_dirs.py

@@ -15,10 +15,6 @@ _SCRIPTS_DIR = Path(os.path.abspath(__file__)).parent
 if str(_SCRIPTS_DIR) not in sys.path:
     sys.path.insert(0, str(_SCRIPTS_DIR))
 
-for _cached in list(sys.modules):
-    if _cached == "config" or _cached.startswith("config."):
-        del sys.modules[_cached]
-
 from sweep_empty_dirs import _build_parser, _positive_int, sweep  # noqa: E402
 
 _OLD_TIMESTAMP = time.time() - 300
@@ -53,7 +49,7 @@ def test_positive_int_rejects_non_integer() -> None:
 
 
 def test_build_parser_sets_age_default_from_timing_config() -> None:
-    """_build_parser uses DEFAULT_AGE_SECONDS from config.timing as --age default."""
+    """_build_parser uses DEFAULT_AGE_SECONDS from dev_env_scripts_constants.timing as --age default."""
     parser = _build_parser()
     default_age = parser.get_default("age")
     assert isinstance(default_age, int)
@@ -61,7 +57,7 @@ def test_build_parser_sets_age_default_from_timing_config() -> None:
 
 
 def test_build_parser_sets_interval_default_from_timing_config() -> None:
-    """_build_parser uses DEFAULT_POLL_INTERVAL from config.timing as --interval default."""
+    """_build_parser uses DEFAULT_POLL_INTERVAL from dev_env_scripts_constants.timing as --interval default."""
     parser = _build_parser()
     default_interval = parser.get_default("interval")
     assert isinstance(default_interval, int)

package/skills/_shared/pr-loop/scripts/_path_resolver.py

@@ -9,7 +9,7 @@ from __future__ import annotations
 import tempfile
 from pathlib import Path
 
-from config.path_resolver_constants import (
+from skills_pr_loop_constants.path_resolver_constants import (
     DIFF_PATCH_TEMPLATE,
     FIX_OUTCOME_XML_TEMPLATE,
     MULTI_PR_SLUG_TEMPLATE,

package/skills/_shared/pr-loop/scripts/build_audit_prompt.py

@@ -19,7 +19,7 @@ if str(_self_dir) not in sys.path:
     sys.path.insert(0, str(_self_dir))
 
 from _xml_utils import emit_pretty_xml
-from config.path_resolver_constants import (
+from skills_pr_loop_constants.path_resolver_constants import (
     ALL_AUDIT_CATEGORY_ENTRIES,
     ALL_AUDIT_CONSTRAINT_TEXTS,
 )

package/skills/_shared/pr-loop/scripts/build_fix_prompt.py

@@ -21,7 +21,7 @@ if str(_self_dir) not in sys.path:
 
 from _cli_utils import require_file
 from _xml_utils import emit_pretty_xml
-from config.path_resolver_constants import (
+from skills_pr_loop_constants.path_resolver_constants import (
     ALL_FIX_CONSTRAINT_TEXTS,
     ALL_FIX_EXECUTION_STEPS,
 )

package/skills/_shared/pr-loop/scripts/init_loop_state.py

@@ -20,7 +20,7 @@ from _path_resolver import (
     per_pr_workspace,
     resolve_run_temp_dir,
 )
-from config.path_resolver_constants import LOOP_STATE_JSON_INDENT
+from skills_pr_loop_constants.path_resolver_constants import LOOP_STATE_JSON_INDENT
 
 
 def create_loop_state(

package/skills/_shared/pr-loop/scripts/teardown_worktrees.py

@@ -24,7 +24,7 @@ if str(_self_dir) not in sys.path:
     sys.path.insert(0, str(_self_dir))
 
 from _path_resolver import per_pr_workspace
-from config.path_resolver_constants import ALL_PYTHON_ONEXC_VERSION
+from skills_pr_loop_constants.path_resolver_constants import ALL_PYTHON_ONEXC_VERSION
 
 
 def _remove_readonly_attribute(

package/skills/_shared/pr-loop/scripts/write_audit_outcomes.py

@@ -19,7 +19,7 @@ if str(_self_dir) not in sys.path:
 from _cli_utils import require_file
 from _path_resolver import outcome_xml_path
 from _xml_utils import emit_pretty_xml
-from config.path_resolver_constants import ALL_FINDING_BODY_ELEMENT_KEYS
+from skills_pr_loop_constants.path_resolver_constants import ALL_FINDING_BODY_ELEMENT_KEYS
 
 
 def build_audit_xml(
@@ -70,7 +70,7 @@ def _populate_findings(parent: Element, findings_data: list[dict[str, object]])
 
     Scalar finding fields become XML attributes on `<finding>`; the
     body fields named in `ALL_FINDING_BODY_ELEMENT_KEYS` (defined in
-    `packages/claude-dev-env/skills/_shared/pr-loop/scripts/config/path_resolver_constants.py`
+    `packages/claude-dev-env/skills/_shared/pr-loop/scripts/skills_pr_loop_constants/path_resolver_constants.py`
     and currently `("title", "excerpt", "description")`) become child elements.
     Nested dicts or lists in scalar slots are flattened to string form
     so attribute serialization stays well-defined.

package/skills/_shared/pr-loop/scripts/write_fix_outcomes.py

@@ -1,7 +1,7 @@
 """Validate status enum values and write <bugteam_fix> XML at the canonical path.
 
 Status enum (canonical source: `ALL_VALID_FIX_STATUSES` in
-`packages/claude-dev-env/skills/_shared/pr-loop/scripts/config/path_resolver_constants.py`):
+`packages/claude-dev-env/skills/_shared/pr-loop/scripts/skills_pr_loop_constants/path_resolver_constants.py`):
 fixed | could_not_address | hook_blocked | unverified_fixed.
 
 Each outcome's scalar fields become XML attributes on `<outcome>`; the
@@ -27,7 +27,7 @@ if str(_self_dir) not in sys.path:
 from _cli_utils import require_file
 from _path_resolver import fix_outcome_xml_path
 from _xml_utils import emit_pretty_xml
-from config.path_resolver_constants import (
+from skills_pr_loop_constants.path_resolver_constants import (
     ALL_FIX_OUTCOME_BODY_ELEMENT_KEYS,
     ALL_VALID_FIX_STATUSES,
 )

package/skills/bugteam/PROMPTS.md

@@ -108,7 +108,7 @@ cd into `<worktree_path>` before any git or file operation.
      teammate does NOT author the inline-comment body directly:
      `post_audit_thread.py` renders every body from
      `INLINE_COMMENT_BODY_TEMPLATE` (defined in
-     [`_shared/pr-loop/scripts/config/post_audit_thread_constants.py`](../../_shared/pr-loop/scripts/config/post_audit_thread_constants.py))
+     [`_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py`](../../_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py))
      — the template prepends `**[<severity>] <Skill> audit finding**`
      and renders the suggested-fix block, so a teammate who hand-formats
      a title or footer wastes the work.

package/skills/bugteam/SKILL.md

@@ -103,7 +103,7 @@ narrative) becomes `description`, and the suffix starting at `Fix:`
 When the agent omits the `Fix:` heading on a given finding, write the
 full `failure_mode` text to BOTH `description` and `fix_summary` so the
 script's body template (`INLINE_COMMENT_BODY_TEMPLATE` in
-[`_shared/pr-loop/scripts/config/post_audit_thread_constants.py`](../../_shared/pr-loop/scripts/config/post_audit_thread_constants.py))
+[`_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py`](../../_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py))
 still renders coherently. Set `side="RIGHT"` for every entry. On CLEAN,
 pass an empty array (`[]`) so the script posts an APPROVE review
 (GitHub stores it as `state=APPROVED`) with a "no findings" summary and

package/skills/bugteam/reference/github-pr-reviews.md

@@ -22,7 +22,7 @@ python "${CLAUDE_SKILL_DIR}/../../_shared/pr-loop/scripts/post_audit_thread.py"
 
 Capture `<head_sha>` via `git rev-parse HEAD` in the subagent cwd immediately before this call so the review attaches to the commit the audit actually scoped.
 
-`--findings-json` points to a JSON file whose root is a list of objects shaped `{path, line, side, severity, description, fix_summary}`. Build it from the merged Shape A findings: finding `file` → `path`. Each finding's `failure_mode` carries the full audit-to-fix handoff text per [`agents/code-quality-agent.md`](../../../agents/code-quality-agent.md); split `failure_mode` at the literal `Fix:` heading so the failure narrative becomes `description` and the suffix beginning at `Fix:` (including the trailing `Validation:` clause) becomes `fix_summary`. When a finding's `failure_mode` omits the `Fix:` heading, write the full text to BOTH `description` and `fix_summary` so the script's body template (`INLINE_COMMENT_BODY_TEMPLATE` in [`packages/claude-dev-env/_shared/pr-loop/scripts/config/post_audit_thread_constants.py`](../../../_shared/pr-loop/scripts/config/post_audit_thread_constants.py)) renders coherently. Set `side="RIGHT"` for every entry. On CLEAN the list is empty (`[]`); on DIRTY the list carries one entry per finding.
+`--findings-json` points to a JSON file whose root is a list of objects shaped `{path, line, side, severity, description, fix_summary}`. Build it from the merged Shape A findings: finding `file` → `path`. Each finding's `failure_mode` carries the full audit-to-fix handoff text per [`agents/code-quality-agent.md`](../../../agents/code-quality-agent.md); split `failure_mode` at the literal `Fix:` heading so the failure narrative becomes `description` and the suffix beginning at `Fix:` (including the trailing `Validation:` clause) becomes `fix_summary`. When a finding's `failure_mode` omits the `Fix:` heading, write the full text to BOTH `description` and `fix_summary` so the script's body template (`INLINE_COMMENT_BODY_TEMPLATE` in [`packages/claude-dev-env/_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py`](../../../_shared/pr-loop/scripts/pr_loop_shared_constants/post_audit_thread_constants.py)) renders coherently. Set `side="RIGHT"` for every entry. On CLEAN the list is empty (`[]`); on DIRTY the list carries one entry per finding.
 
 The script handles retries internally — 1s / 4s / 16s backoff across four attempts (one initial plus three retries). Exit codes:
 

package/skills/bugteam/scripts/{_claude_permissions_common.py → _bugteam_permissions_common.py}

@@ -16,25 +16,13 @@ from collections.abc import Callable
 from pathlib import Path
 from typing import NoReturn
 
-_previously_cached_config = {}
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    _previously_cached_config[each_cached_module_name] = sys.modules.pop(
-        each_cached_module_name
-    )
-
-from config.claude_permissions_common_constants import (
+from bugteam_scripts_constants.claude_permissions_common_constants import (
     ALL_TRUST_ENTRY_PROJECT_PATH_BOUNDARY_QUOTE_CHARACTERS,
     ATOMIC_WRITE_TEMPORARY_SUFFIX,
     DEFAULT_SETTINGS_FILE_MODE,
     TEXT_FILE_ENCODING,
 )
 
-sys.modules.update(_previously_cached_config)
-
 
 def exit_with_error(message: str) -> NoReturn:
     print(f"Error: {message}", file=sys.stderr)

package/skills/bugteam/scripts/bugteam_code_rules_gate.py

@@ -11,17 +11,7 @@ from pathlib import Path
 
 ValidateContentCallable = Callable[..., list[str]]
 
-_previously_cached_config = {}
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    _previously_cached_config[each_cached_module_name] = sys.modules.pop(
-        each_cached_module_name
-    )
-
-from config.bugteam_code_rules_gate_constants import (
+from bugteam_scripts_constants.bugteam_code_rules_gate_constants import (
     ALL_CODE_FILE_EXTENSIONS,
     ALL_COLUMN_MAGIC_FALSE_VALUES,
     ALL_GIT_DIFF_CACHED_ARGS,
@@ -34,8 +24,6 @@ from config.bugteam_code_rules_gate_constants import (
     VIOLATION_LINE_RAW_PATTERN,
 )
 
-sys.modules.update(_previously_cached_config)
-
 
 def hunk_header_pattern() -> re.Pattern[str]:
     return re.compile(HUNK_HEADER_RAW_PATTERN)

package/skills/bugteam/scripts/bugteam_fix_hookspath.py

@@ -5,22 +5,7 @@ import subprocess
 import sys
 from pathlib import Path
 
-parent_directory = str(Path(__file__).resolve().parent)
-try:
-    sys.path.remove(parent_directory)
-except ValueError:
-    pass
-if parent_directory not in sys.path:
-    sys.path.insert(0, parent_directory)
-
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    sys.modules.pop(each_cached_module_name, None)
-
-from config.bugteam_fix_hookspath_constants import (
+from bugteam_scripts_constants.bugteam_fix_hookspath_constants import (
     ALL_CANONICAL_HOOKS_DIRECTORY_COMPONENTS,
     ALL_GLOBAL_HOOKS_PATH_ARGUMENTS,
     ALL_HOME_ENV_VAR_NAMES,

package/skills/bugteam/scripts/bugteam_preflight.py

@@ -6,19 +6,13 @@ import subprocess
 import sys
 from pathlib import Path
 
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    sys.modules.pop(each_cached_module_name, None)
 _bugteam_scripts_directory = str(Path(__file__).absolute().parent)
 while _bugteam_scripts_directory in sys.path:
     sys.path.remove(_bugteam_scripts_directory)
 if _bugteam_scripts_directory not in sys.path:
     sys.path.insert(0, _bugteam_scripts_directory)
 
-from config.bugteam_preflight_constants import (
+from bugteam_scripts_constants.bugteam_preflight_constants import (
     ALL_DISCOVERY_IGNORE_DIRECTORIES,
     ALL_GIT_CONFIG_HOOKS_PATH_ARGUMENTS,
     ALL_PRE_COMMIT_ARGUMENTS,
@@ -35,12 +29,6 @@ from config.bugteam_preflight_constants import (
     PYTEST_INI_FILENAME,
 )
 
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    sys.modules.pop(each_cached_module_name, None)
 _shared_pr_loop_scripts_directory = (
     Path(__file__).absolute().parent
     / ".." / ".." / ".." / "_shared" / "pr-loop" / "scripts"

package/skills/bugteam/scripts/grant_project_claude_permissions.py

@@ -9,17 +9,11 @@ the changes applied. No-op when the entries already exist.
 import sys
 from pathlib import Path
 
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    sys.modules.pop(each_cached_module_name, None)
 parent_directory = str(Path(__file__).resolve().parent)
 if parent_directory not in sys.path:
     sys.path.insert(0, parent_directory)
 
-from _claude_permissions_common import (  # noqa: E402
+from _bugteam_permissions_common import (  # noqa: E402
     append_if_missing,
     build_agent_config_deny_rules,
     build_permission_rules,
@@ -32,7 +26,7 @@ from _claude_permissions_common import (  # noqa: E402
     remove_matching_entries_from_list,
     save_settings,
 )
-from config.claude_permissions_common_constants import (  # noqa: E402
+from bugteam_scripts_constants.claude_permissions_common_constants import (  # noqa: E402
     ALL_AGENT_CONFIG_DENY_TOOLS,
     ALL_AGENT_CONFIG_PATH_PATTERNS,
     ALL_PERMISSION_ALLOW_TOOLS,

package/skills/bugteam/scripts/probe_code_rules_enforcer_check.py

@@ -19,7 +19,7 @@ import sys
 from pathlib import Path
 from types import ModuleType
 
-from config.probe_code_rules_enforcer_check_constants import (
+from bugteam_scripts_constants.probe_code_rules_enforcer_check_constants import (
     DEFAULT_REPORTED_PATH,
     ENFORCER_MODULE_NAME,
     ENFORCER_RELATIVE_PATH,

package/skills/bugteam/scripts/reflow_skill_md.py

@@ -18,7 +18,7 @@ from __future__ import annotations
 import re
 import textwrap
 
-from config.reflow_skill_md_constants import (
+from bugteam_scripts_constants.reflow_skill_md_constants import (
     BASH_CONTINUATION_MARKER_WIDTH,
     BULLET_LIST_ITEM_PATTERN as BULLET_RE,
     MAXIMUM_LINE_WIDTH as MAX_WIDTH,

package/skills/bugteam/scripts/revoke_project_claude_permissions.py

@@ -10,17 +10,11 @@ autoMode sections so repeated grant/revoke cycles leave no dead structure.
 import sys
 from pathlib import Path
 
-for each_cached_module_name in [
-    each_module_key
-    for each_module_key in list(sys.modules)
-    if each_module_key == "config" or each_module_key.startswith("config.")
-]:
-    sys.modules.pop(each_cached_module_name, None)
 parent_directory = str(Path(__file__).resolve().parent)
 if parent_directory not in sys.path:
     sys.path.insert(0, parent_directory)
 
-from _claude_permissions_common import (  # noqa: E402
+from _bugteam_permissions_common import (  # noqa: E402
     build_agent_config_deny_rules,
     build_permission_rules,
     exit_with_error,
@@ -31,7 +25,7 @@ from _claude_permissions_common import (  # noqa: E402
     remove_matching_entries_from_list,
     save_settings,
 )
-from config.claude_permissions_common_constants import (  # noqa: E402
+from bugteam_scripts_constants.claude_permissions_common_constants import (  # noqa: E402
     ALL_AGENT_CONFIG_DENY_TOOLS,
     ALL_AGENT_CONFIG_PATH_PATTERNS,
     ALL_PERMISSION_ALLOW_TOOLS,

package/skills/bugteam/scripts/{test__claude_permissions_common.py → test__bugteam_permissions_common.py}

@@ -1,10 +1,10 @@
-"""TDD-pair tests for the underscore-prefixed _claude_permissions_common module.
+"""TDD-pair tests for the underscore-prefixed _bugteam_permissions_common module.
 
 The TDD enforcer matches a production filename ``X.py`` to ``test_X.py``;
-``_claude_permissions_common.py`` carries a leading underscore that the
+``_bugteam_permissions_common.py`` carries a leading underscore that the
 enforcer treats as part of the name. This file's tests are the canonical
 match. The broader behavioral suite continues to live alongside, in
-``test_claude_permissions_common.py``.
+``test_bugteam_permissions_common.py``.
 """
 from __future__ import annotations
 
@@ -18,7 +18,7 @@ _script_directory = str(Path(__file__).resolve().parent)
 if _script_directory not in sys.path:
     sys.path.insert(0, _script_directory)
 
-import _claude_permissions_common as common_module
+import _bugteam_permissions_common as common_module
 import grant_project_claude_permissions as grant_module
 import revoke_project_claude_permissions as revoke_module
 

package/skills/bugteam/scripts/test_agent_config_carveout.py

@@ -21,10 +21,10 @@ _script_directory = str(Path(__file__).resolve().parent)
 if _script_directory not in sys.path:
     sys.path.insert(0, _script_directory)
 
-import _claude_permissions_common as common_module
+import _bugteam_permissions_common as common_module
 import grant_project_claude_permissions as grant_module
 import revoke_project_claude_permissions as revoke_module
-from config.claude_permissions_common_constants import (
+from bugteam_scripts_constants.claude_permissions_common_constants import (
     ALL_AGENT_CONFIG_DENY_TOOLS,
     ALL_AGENT_CONFIG_PATH_PATTERNS,
     ALL_PERMISSION_ALLOW_TOOLS,

package/skills/bugteam/scripts/test_bugteam_fix_hookspath.py

@@ -356,29 +356,3 @@ def test_read_global_core_hooks_path_returns_empty_when_key_unset() -> None:
     with patch.object(subprocess, "run", fake_run):
         result = bugteam_fix_hookspath.read_global_core_hooks_path(None)
     assert result == ""
-
-
-def test_module_import_evicts_cached_config_submodules() -> None:
-    """Importing bugteam_fix_hookspath must evict cached `config.*` submodules.
-
-    Regression for loop1-1: without a defensive cache pop above sys.path.insert,
-    a previously-cached `config` package shadows scripts/config/ and the
-    from-import raises ModuleNotFoundError.
-    """
-    fake_submodule_name = "config.bugteam_fix_hookspath_constants"
-    fake_parent_name = "config"
-    sentinel_module_a = ModuleType(fake_parent_name)
-    sentinel_module_b = ModuleType(fake_submodule_name)
-    sys.modules[fake_parent_name] = sentinel_module_a
-    sys.modules[fake_submodule_name] = sentinel_module_b
-    try:
-        _load_fix_module()
-    finally:
-        sys.modules.pop(fake_parent_name, None)
-        sys.modules.pop(fake_submodule_name, None)
-    assert sys.modules.get(fake_parent_name) is not sentinel_module_a, (
-        "parent `config` cache entry must be evicted on module import"
-    )
-    assert sys.modules.get(fake_submodule_name) is not sentinel_module_b, (
-        "cached `config.<submodule>` entries must be evicted on module import"
-    )

package/skills/bugteam/scripts/{test_claude_permissions_common.py → test_bugteam_permissions_common.py}

@@ -1,7 +1,5 @@
-import importlib
 import sys
 from pathlib import Path
-from types import ModuleType
 from unittest.mock import patch
 
 import pytest
@@ -10,14 +8,14 @@ _script_directory = str(Path(__file__).resolve().parent)
 if _script_directory not in sys.path:
     sys.path.insert(0, _script_directory)
 
-import _claude_permissions_common as common_module
-from _claude_permissions_common import (
+import _bugteam_permissions_common as common_module
+from _bugteam_permissions_common import (
     build_permission_rule,
     get_current_project_path,
     path_contains_glob_metacharacters,
     save_settings,
 )
-from config.claude_permissions_common_constants import DEFAULT_SETTINGS_FILE_MODE
+from bugteam_scripts_constants.claude_permissions_common_constants import DEFAULT_SETTINGS_FILE_MODE
 import grant_project_claude_permissions as grant_module
 import revoke_project_claude_permissions as revoke_module
 
@@ -140,64 +138,3 @@ def test_is_valid_project_root_exported_from_consumer_modules(
     assert revoke_module.is_valid_project_root(bare_directory) is False
 
 
-def _reload_with_stale_config_cache(module_name: str) -> ModuleType:
-    fake_submodule_name = "config.claude_permissions_common_constants"
-    fake_parent_name = "config"
-    sentinel_module_a = ModuleType(fake_parent_name)
-    sentinel_module_b = ModuleType(fake_submodule_name)
-    sys.modules[fake_parent_name] = sentinel_module_a
-    sys.modules[fake_submodule_name] = sentinel_module_b
-    try:
-        target_module = sys.modules.get(module_name)
-        if target_module is None:
-            target_module = importlib.import_module(module_name)
-        else:
-            target_module = importlib.reload(target_module)
-    finally:
-        sys.modules.pop(fake_parent_name, None)
-        sys.modules.pop(fake_submodule_name, None)
-    return target_module
-
-
-def test_grant_module_import_evicts_cached_config_submodules(
-    tmp_path: Path,
-) -> None:
-    """grant_project_claude_permissions must evict cached `config.*` on import.
-
-    Regression for loop1-2: without a defensive cache pop above sys.path.insert,
-    a cached `config` package shadows scripts/config/ and the from-import raises.
-    Calls the rebound `is_valid_project_root` to confirm the real implementation
-    survived the cache eviction (a stale shadow would either raise on import or
-    bind a placeholder that returns the wrong value).
-    """
-    reloaded_module = _reload_with_stale_config_cache(
-        "grant_project_claude_permissions"
-    )
-    real_project_root = tmp_path / "project_root"
-    (real_project_root / ".claude").mkdir(parents=True)
-    bare_directory = tmp_path / "no_claude_marker"
-    bare_directory.mkdir()
-    assert reloaded_module.is_valid_project_root(real_project_root) is True
-    assert reloaded_module.is_valid_project_root(bare_directory) is False
-
-
-def test_revoke_module_import_evicts_cached_config_submodules(
-    tmp_path: Path,
-) -> None:
-    """revoke_project_claude_permissions must evict cached `config.*` on import.
-
-    Regression for loop1-3: without a defensive cache pop above sys.path.insert,
-    a cached `config` package shadows scripts/config/ and the from-import raises.
-    Calls the rebound `is_valid_project_root` to confirm the real implementation
-    survived the cache eviction (a stale shadow would either raise on import or
-    bind a placeholder that returns the wrong value).
-    """
-    reloaded_module = _reload_with_stale_config_cache(
-        "revoke_project_claude_permissions"
-    )
-    real_project_root = tmp_path / "project_root"
-    (real_project_root / ".claude").mkdir(parents=True)
-    bare_directory = tmp_path / "no_claude_marker"
-    bare_directory.mkdir()
-    assert reloaded_module.is_valid_project_root(real_project_root) is True
-    assert reloaded_module.is_valid_project_root(bare_directory) is False

package/skills/bugteam/scripts/test_bugteam_preflight.py

@@ -210,38 +210,13 @@ def test_should_exit_nonzero_when_subprocess_run_raises_os_error(
     )
 
 
-def test_module_import_evicts_cached_config_submodules() -> None:
-    """Importing bugteam_preflight must evict cached `config.*` submodules.
-
-    Regression for loop1-4: a single `sys.modules.pop("config", None)` only
-    removes the parent key, leaving stale `config.<submodule>` entries that
-    satisfy the next from-import with the wrong bindings.
-    """
-    fake_submodule_name = "config.bugteam_preflight_constants"
-    fake_parent_name = "config"
-    sentinel_module_a = ModuleType(fake_parent_name)
-    sentinel_module_b = ModuleType(fake_submodule_name)
-    sys.modules[fake_parent_name] = sentinel_module_a
-    sys.modules[fake_submodule_name] = sentinel_module_b
-    try:
-        _load_preflight_module()
-    finally:
-        sys.modules.pop(fake_parent_name, None)
-        sys.modules.pop(fake_submodule_name, None)
-    assert sys.modules.get(fake_parent_name) is not sentinel_module_a, (
-        "parent `config` cache entry must be evicted on module import"
-    )
-    assert sys.modules.get(fake_submodule_name) is not sentinel_module_b, (
-        "cached `config.<submodule>` entries must be evicted on module import"
-    )
-
-
 def test_has_pytest_configuration_finds_pytest_ini(tmp_path: Path) -> None:
     """has_pytest_configuration must detect pytest.ini at the repo root.
 
     Regression for loop1-17/loop1-18: the literals "pytest.ini",
     "pyproject.toml", and "[tool.pytest" were inlined in production function
-    bodies; centralizing them in config and importing here pins the contract.
+    bodies; centralizing them in bugteam_scripts_constants and importing here
+    pins the contract.
     """
     repository_root = tmp_path / "repo"
     repository_root.mkdir()

package/skills/bugteam/scripts/windows_safe_rmtree.py

@@ -16,7 +16,7 @@ import stat
 import sys
 from collections.abc import Callable
 
-from config.windows_safe_rmtree_constants import (
+from bugteam_scripts_constants.windows_safe_rmtree_constants import (
     EXIT_CODE_REMOVE_TREE_FAILURE,
     EXIT_CODE_USAGE_ERROR,
     EXPECTED_ARGUMENT_COUNT,

package/skills/doc-gist/SKILL.md

@@ -94,6 +94,6 @@ Read the matching example for the artifact you're designing. Crib palette, typog
 
 - `SKILL.md` — this file.
 - `skills/doc-gist/scripts/gist_upload.py` — transport: HTML in, gist + preview URLs out.
-- `skills/doc-gist/scripts/config/gist_upload_constants.py` — the URL prefixes and template strings.
+- `skills/doc-gist/scripts/doc_gist_scripts_constants/gist_upload_constants.py` — the URL prefixes and template strings.
 - `references/examples/` — Thariq's 20 html-effectiveness prototypes.
 - (PostToolUse hook lives in `packages/claude-dev-env/hooks/workflow/doc_gist_auto_publish.py` — wired into the plugin's `hooks.json`.)

package/skills/doc-gist/scripts/gist_upload.py

@@ -30,7 +30,7 @@ _script_directory = str(Path(__file__).resolve().parent)
 if _script_directory not in sys.path:
     sys.path.insert(0, _script_directory)
 
-from config.gist_upload_constants import (  # noqa: E402
+from doc_gist_scripts_constants.gist_upload_constants import (  # noqa: E402
     GIST_DEFAULT_FILENAME,
     GIST_HOST_PREFIX,
     MINIMUM_GIST_URL_PARTS,

package/skills/implement/SKILL.md

@@ -62,5 +62,5 @@ Append entries as decisions are made — do not batch them until the end.
 | File | Purpose |
 |---|---|
 | `SKILL.md` | This hub |
-| `scripts/append_note.py` | CLI to append one entry to a section |
-| `scripts/config/notes_constants.py` | Section slugs → headings and default filename |
+| `packages/claude-dev-env/skills/implement/scripts/append_note.py` | CLI to append one entry to a section |
+| `packages/claude-dev-env/skills/implement/scripts/implement_scripts_constants/notes_constants.py` | Section slugs → headings and default filename |

package/skills/implement/scripts/append_note.py

@@ -16,7 +16,7 @@ import html
 import sys
 from pathlib import Path
 
-from config.notes_constants import DEFAULT_NOTES_FILENAME, HEADING_BY_SLUG
+from implement_scripts_constants.notes_constants import DEFAULT_NOTES_FILENAME, HEADING_BY_SLUG
 
 
 def _build_skeleton() -> str:

package/skills/pr-converge/{config → pr_converge_skill_constants}/constants.py

@@ -2,7 +2,7 @@
 
 All runtime and API constants live here. Script-specific constants
 (CLI args, markdown patterns, reflow settings) stay in
-``packages/claude-dev-env/skills/pr-converge/scripts/config/pr_converge_constants.py``,
+``packages/claude-dev-env/skills/pr-converge/scripts/pr_converge_scripts_constants/pr_converge_constants.py``,
 which imports from here.
 """