npm - claude-dev-env - Versions diffs - 1.42.0 → 1.43.0 - Mend

@@ -1,60 +0,0 @@
-"""Test fixtures for skills/pr-converge/scripts/.
-Two unrelated Python packages live under the name ``config`` in this repo:
-  - ``skills/pr-converge/config/`` (constants for the pr-converge scripts)
-  - ``hooks/config/`` (constants for the code-rules enforcer and other hooks)
-When tests under this directory exercise pr-converge scripts that load
-``from config.constants import ...`` and other code paths in the same
-pytest process also load a different ``config`` package,
-``sys.modules['config']`` and ``sys.modules['config.<submodule>']`` cache
-entries from one package leak into the other. The next
-``from config.<submodule> import ...`` then fails with
-``ModuleNotFoundError`` because the cached parent package does not
-expose that submodule.
-Independently, several scripts in this folder do
-``Path(__file__).resolve()`` then prepend the resulting directory to
-``sys.path``. On Windows when the working tree lives under a mapped drive
-backed by a UNC share (``Y:`` -> ``\\\\server\\share\\...``), ``.resolve()``
-returns the UNC form, and Python's import machinery on this host cannot
-locate ``config`` packages from a UNC ``sys.path`` entry. The Y:-form entry
-gets pushed to a later index by subsequent inserts, making
-``from config.<submodule> import ...`` fail.
-This autouse fixture restores both invariants once per pytest session,
-immediately before the first test executes (after collection and module
-imports have completed; session-scoped fixtures run after import, not
-before, so test-module-level ``import`` of pr-converge scripts is
-isolated by each module's own ``_load_module()`` helper rather than by
-this fixture):
-  1. evict every ``config`` and ``config.*`` entry from ``sys.modules``
-  2. prepend the drive-letter (``.absolute()``) form of the pr-converge
-     directory to ``sys.path`` so package resolution always has a
-     non-UNC path to search first
-"""
-from __future__ import annotations
-import sys
-from pathlib import Path
-import pytest
-PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM = str(Path(__file__).absolute().parent.parent)
-if PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM not in sys.path:
-    sys.path.insert(0, PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM)
-@pytest.fixture(scope="session", autouse=True)
-def _evict_config_namespace_at_session_start() -> None:
-    for each_module_name in [
-        each_key
-        for each_key in list(sys.modules)
-        if each_key == "config" or each_key.startswith("config.")
-    ]:
-        sys.modules.pop(each_module_name, None)
-    if PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM in sys.path:
-        sys.path.remove(PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM)
-    sys.path.insert(0, PR_CONVERGE_DIRECTORY_DRIVE_LETTER_FORM)

@@ -1,55 +0,0 @@
-"""Regression tests for revoke_project_claude_permissions module-import behavior.
-Pins the loop1-3 fix: a defensive cache pop above sys.path.insert evicts every
-cached `config` and `config.<submodule>` entry so the from-import binds against
-scripts/config/ rather than a stale parent package shadowing it.
-"""
-from __future__ import annotations
-import importlib
-import sys
-from pathlib import Path
-from types import ModuleType
-_script_directory = str(Path(__file__).resolve().parent)
-if _script_directory not in sys.path:
-    sys.path.insert(0, _script_directory)
-def _reload_revoke_with_stale_config_cache() -> 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("revoke_project_claude_permissions")
-        if target_module is None:
-            target_module = importlib.import_module("revoke_project_claude_permissions")
-        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_revoke_module_imports_when_config_is_already_cached(tmp_path: Path) -> None:
-    """Module import must succeed even when sys.modules carries a stale `config`.
-    Regression for loop1-3 — invokes is_valid_project_root after reload to
-    prove the binding came from scripts/config/ rather than the sentinel.
-    """
-    reloaded_module = _reload_revoke_with_stale_config_cache()
-    not_a_project_root = tmp_path / "empty_dir"
-    not_a_project_root.mkdir()
-    assert reloaded_module.is_valid_project_root(not_a_project_root) is False, (
-        "is_valid_project_root must run normally after the reload — proof that "
-        "the from-import bound real constants, not the stale cached ones"
-    )
-    a_claude_project_root = tmp_path / "claude_project"
-    (a_claude_project_root / ".claude").mkdir(parents=True)
-    assert reloaded_module.is_valid_project_root(a_claude_project_root) is True

@@ -1,20 +0,0 @@
-"""Evict cached ``config`` package bindings before local ``config`` imports.
-Mirrors ``_evict_config_module`` in the repository root ``conftest.py``: stale
-``config`` or ``config.*`` entries from other packages must not satisfy
-``from config.…`` in these scripts after ``sys.path`` inserts the script directory.
-"""
-from __future__ import annotations
-import importlib
-import sys
-def evict_cached_config_modules() -> None:
-    for each_cached_module_name in list(sys.modules):
-        is_config_root = each_cached_module_name == "config"
-        is_config_submodule = each_cached_module_name.startswith("config.")
-        if is_config_root or is_config_submodule:
-            sys.modules.pop(each_cached_module_name, None)
-    importlib.invalidate_caches()

@@ -1,22 +0,0 @@
-"""Tests for evict_cached_config_modules."""
-from __future__ import annotations
-import sys
-import types
-from pathlib import Path
-_scripts_directory = Path(__file__).absolute().parent
-if str(_scripts_directory) not in sys.path:
-    sys.path.insert(0, str(_scripts_directory))
-from evict_cached_config_modules import evict_cached_config_modules
-def test_should_remove_root_config_and_submodules() -> None:
-    fake = types.ModuleType("config")
-    sys.modules["config"] = fake
-    sys.modules["config.stale_submodule"] = types.ModuleType("config.stale_submodule")
-    evict_cached_config_modules()
-    assert "config" not in sys.modules
-    assert "config.stale_submodule" not in sys.modules

claude-dev-env 1.42.0 → 1.43.0