claude-dev-env 1.83.0 → 1.85.0

package/hooks/blocking/test_md_to_html_blocker_path_resolution.py

@@ -1,336 +0,0 @@
-"""Tests for md_to_html_blocker path resolution, denial payload, and config.
-
-Covers home-relative and tilde path canonicalization, the OS temp-directory
-exemption, cwd-relative resolution against repo and plugin roots, the structure
-of the denial payload (system message, additional context, redirect reason),
-and the module-introspection contracts that pin centralized constants.
-"""
-
-import importlib
-import json
-import os
-import subprocess
-import sys
-
-_BLOCKING_DIRECTORY = os.path.dirname(__file__)
-
-if _BLOCKING_DIRECTORY not in sys.path:
-    sys.path.insert(0, _BLOCKING_DIRECTORY)
-
-from _md_to_html_blocker_test_support import (  # noqa: E402
-    HOOK_SCRIPT_PATH,
-    _get_sandbox_parent_directory,
-    _run_hook,
-)
-
-_ALL_HOME_ENVIRONMENT_VARIABLE_NAMES = ("USERPROFILE", "HOME")
-_ALL_TEMP_ENVIRONMENT_VARIABLE_NAMES = ("TMPDIR", "TEMP", "TMP")
-
-
-def _redirect_home_to(monkeypatch, fake_home_directory):
-    """Point every home-directory env read at *fake_home_directory*.
-
-    The hook runs as a subprocess that inherits os.environ, so setting the
-    home env vars here propagates to the hook process. Windows ntpath reads
-    USERPROFILE while POSIX reads HOME, so both are set to keep the test's
-    expected path and the hook's resolution aligned on either platform.
-
-    Args:
-        monkeypatch: The pytest monkeypatch fixture used to set env vars.
-        fake_home_directory: An existing sandbox directory to treat as home.
-
-    Returns:
-        The canonical (realpath) form of *fake_home_directory*, matching the
-        canonicalization the exemption resolver applies before comparison.
-    """
-    for each_home_variable_name in _ALL_HOME_ENVIRONMENT_VARIABLE_NAMES:
-        monkeypatch.setenv(each_home_variable_name, fake_home_directory)
-    return os.path.realpath(fake_home_directory)
-
-
-def _redirect_temp_to(monkeypatch, fake_temp_directory):
-    """Point every temp-directory env read at *fake_temp_directory*.
-
-    tempfile.gettempdir() consults TMPDIR, then TEMP, then TMP, so all three
-    are set. The hook subprocess is a fresh process, so its
-    tempfile.gettempdir() reads these env vars rather than a cached value.
-
-    Args:
-        monkeypatch: The pytest monkeypatch fixture used to set env vars.
-        fake_temp_directory: An existing sandbox directory to treat as temp.
-
-    Returns:
-        The canonical (realpath) form of *fake_temp_directory*, matching the
-        canonicalization the exemption resolver applies before comparison.
-    """
-    for each_temp_variable_name in _ALL_TEMP_ENVIRONMENT_VARIABLE_NAMES:
-        monkeypatch.setenv(each_temp_variable_name, fake_temp_directory)
-    return os.path.realpath(fake_temp_directory)
-
-
-def _isolate_home_away_from_temp(monkeypatch, base_directory):
-    """Redirect home and temp at disjoint subdirectories under *base_directory*.
-
-    Pointing home and temp at separate trees keeps a home-relative test path
-    from also matching the temp-directory exemption, which the resolver checks
-    after the home exemption. Both env groups are set so the hook subprocess
-    and this test process resolve the same fake home and temp.
-
-    Args:
-        monkeypatch: The pytest monkeypatch fixture used to set env vars.
-        base_directory: An existing sandbox directory whose `home` and `temp`
-            subdirectories become the fake home and fake temp.
-
-    Returns:
-        The canonical (realpath) form of the fake home directory.
-    """
-    fake_home_directory = os.path.join(base_directory, "home")
-    fake_temp_directory = os.path.join(base_directory, "temp")
-    os.makedirs(fake_home_directory, exist_ok=True)
-    os.makedirs(fake_temp_directory, exist_ok=True)
-    _redirect_temp_to(monkeypatch, fake_temp_directory)
-    return _redirect_home_to(monkeypatch, fake_home_directory)
-
-
-def test_block_messages_mention_claude_dev_env_source_exemptions():
-    """Block messages must surface the `packages/claude-dev-env/<dir>/` anchored
-    exemption so contributors aren't misled when a `.md` write is denied
-    elsewhere. Ensures docs/, rules/, and system-prompts/ source files
-    render as writable in the user-facing message."""
-    hook_dir = os.path.dirname(HOOK_SCRIPT_PATH)
-    if hook_dir not in sys.path:
-        sys.path.insert(0, hook_dir)
-    blocker_module = importlib.import_module("md_to_html_blocker")
-    importlib.reload(blocker_module)
-
-    context_message = blocker_module._block_context()
-    system_message = blocker_module._block_system_message()
-    combined_messages = context_message + " " + system_message
-    assert "claude-dev-env" in combined_messages, (
-        "Block messages must mention claude-dev-env source-directory exemption; "
-        f"got context={context_message!r} system={system_message!r}"
-    )
-
-
-def test_module_imports_path_segments_from_hooks_constants():
-    """The blocker pulls the two leading path segments (`packages` and
-    `claude-dev-env`) through the centralised hooks_constants module rather
-    than inlining them as raw string literals."""
-    hook_dir = os.path.dirname(HOOK_SCRIPT_PATH)
-    if hook_dir not in sys.path:
-        sys.path.insert(0, hook_dir)
-    blocker_module = importlib.import_module("md_to_html_blocker")
-    importlib.reload(blocker_module)
-    assert blocker_module.PACKAGES_TOP_LEVEL_SEGMENT == "packages"
-    assert blocker_module.CLAUDE_DEV_ENV_REPO_NAME_SEGMENT == "claude-dev-env"
-
-
-def test_module_imports_top_directories_from_hooks_constants():
-    """The exempt-top-directories set must live in `hooks_constants/` rather
-    than as a file-global single-use constant in the blocker module. The
-    blocker imports the centralized constant; a regression that reintroduces
-    a local module-scope copy would fail this assertion."""
-    hook_dir = os.path.dirname(HOOK_SCRIPT_PATH)
-    if hook_dir not in sys.path:
-        sys.path.insert(0, hook_dir)
-    blocker_module = importlib.import_module("md_to_html_blocker")
-    importlib.reload(blocker_module)
-    assert hasattr(blocker_module, "ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES"), (
-        "Blocker module must import ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES from "
-        "hooks_constants/ (file-global single-use rule)."
-    )
-    assert not hasattr(blocker_module, "_claude_code_source_top_directories"), (
-        "Local _claude_code_source_top_directories must not be re-introduced; "
-        "use the imported constant from hooks_constants/ instead."
-    )
-
-
-def test_blocks_relative_readme_when_cwd_is_not_repo_root():
-    sandbox_parent = _get_sandbox_parent_directory()
-    non_repo_cwd = os.path.join(sandbox_parent, "not-a-repo")
-    os.makedirs(non_repo_cwd, exist_ok=True)
-    payload = json.dumps(
-        {
-            "tool_name": "Write",
-            "tool_input": {"file_path": "README.md", "content": "# README"},
-        }
-    )
-    result = subprocess.run(
-        [sys.executable, HOOK_SCRIPT_PATH],
-        input=payload,
-        capture_output=True,
-        text=True,
-        check=False,
-        cwd=non_repo_cwd,
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    assert output["hookSpecificOutput"]["permissionDecision"] == "deny"
-
-
-def test_denial_has_system_message():
-    result = _run_hook(
-        "Write",
-        {"file_path": "docs/guide.md", "content": "# Hello"},
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    assert output["suppressOutput"] is True
-    assert isinstance(output["systemMessage"], str)
-    assert len(output["systemMessage"]) > 0
-
-
-def test_denial_has_additional_context():
-    result = _run_hook(
-        "Write",
-        {"file_path": "docs/guide.md", "content": "# Hello"},
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    ctx = output["hookSpecificOutput"].get("additionalContext", "")
-    assert "HTML" in ctx
-    assert "thariqs.github.io" in output["hookSpecificOutput"]["permissionDecisionReason"]
-
-
-def test_denial_reason_mentions_html_redirect():
-    result = _run_hook(
-        "Write",
-        {"file_path": "docs/guide.md", "content": "# Hello"},
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    reason = output["hookSpecificOutput"]["permissionDecisionReason"]
-    assert ".html" in reason.lower()
-
-
-def test_passes_home_session_log_directory(monkeypatch, tmp_path):
-    home_directory = _isolate_home_away_from_temp(monkeypatch, str(tmp_path))
-    session_log_path = os.path.join(home_directory, "SessionLog", "decisions", "note.md")
-    result = _run_hook(
-        "Write",
-        {"file_path": session_log_path, "content": "# Note"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_passes_home_claude_plans_directory(monkeypatch, tmp_path):
-    home_directory = _isolate_home_away_from_temp(monkeypatch, str(tmp_path))
-    plans_path = os.path.join(home_directory, ".claude", "plans", "plan.md")
-    result = _run_hook(
-        "Write",
-        {"file_path": plans_path, "content": "# Plan"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_blocks_home_directory_other_md_file(monkeypatch, tmp_path):
-    home_directory = _isolate_home_away_from_temp(monkeypatch, str(tmp_path))
-    other_path = os.path.join(home_directory, "docs", "guide.md")
-    result = _run_hook(
-        "Write",
-        {"file_path": other_path, "content": "# Guide"},
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    assert output["hookSpecificOutput"]["permissionDecision"] == "deny"
-
-
-def test_passes_tilde_session_log_path():
-    result = _run_hook(
-        "Write",
-        {"file_path": "~/SessionLog/decisions/note.md", "content": "# Note"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_passes_tilde_claude_plans_path():
-    result = _run_hook(
-        "Write",
-        {"file_path": "~/.claude/plans/plan.md", "content": "# Plan"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_blocks_tilde_other_home_md_file():
-    result = _run_hook(
-        "Write",
-        {"file_path": "~/docs/guide.md", "content": "# Guide"},
-    )
-    assert result.returncode == 0
-    output = json.loads(result.stdout)
-    assert output["hookSpecificOutput"]["permissionDecision"] == "deny"
-
-
-def test_passes_system_temp_directory(monkeypatch, tmp_path):
-    temp_directory = _redirect_temp_to(monkeypatch, str(tmp_path))
-    temp_md_path = os.path.join(temp_directory, "bugteam-scratch", "pr-body.md")
-    result = _run_hook(
-        "Write",
-        {"file_path": temp_md_path, "content": "# Scratch"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_passes_relative_path_from_home_cwd(monkeypatch, tmp_path):
-    home_directory = _isolate_home_away_from_temp(monkeypatch, str(tmp_path))
-    payload = json.dumps(
-        {
-            "tool_name": "Write",
-            "tool_input": {
-                "file_path": "SessionLog/decisions/note.md",
-                "content": "# Note",
-            },
-        }
-    )
-    result = subprocess.run(
-        [sys.executable, HOOK_SCRIPT_PATH],
-        input=payload,
-        capture_output=True,
-        text=True,
-        check=False,
-        cwd=home_directory,
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_passes_canonicalized_home_path(monkeypatch, tmp_path):
-    canonical_home = _isolate_home_away_from_temp(monkeypatch, str(tmp_path))
-    canonical_path = os.path.join(canonical_home, "SessionLog", "canonical-note.md")
-    result = _run_hook(
-        "Write",
-        {"file_path": canonical_path, "content": "# Canonical"},
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""
-
-
-def test_passes_relative_path_under_cwd_plugin_root_marker(tmp_path):
-    plugin_root = tmp_path / "plugin-cwd-repo"
-    (plugin_root / ".claude-plugin").mkdir(parents=True)
-    (plugin_root / "subdir").mkdir(parents=True)
-
-    payload = json.dumps(
-        {
-            "tool_name": "Write",
-            "tool_input": {
-                "file_path": "subdir/design.md",
-                "content": "# Design",
-            },
-        }
-    )
-    result = subprocess.run(
-        [sys.executable, HOOK_SCRIPT_PATH],
-        input=payload,
-        capture_output=True,
-        text=True,
-        check=False,
-        cwd=str(plugin_root),
-    )
-    assert result.returncode == 0
-    assert result.stdout == ""

package/hooks/hooks_constants/doc_gist_auto_publish_constants.py

@@ -1,18 +0,0 @@
-"""Constants for the doc-gist auto-publish PostToolUse hook.
-
-PUBLISH_SENTINEL: HTML comment marker. Claude includes it in HTML it intends to share;
-absent in HTML that is part of code, tests, or fixtures. Hook is a no-op without it.
-
-HTML_FILE_EXTENSION: only files ending in this extension are candidates for the hook;
-Markdown, source files, etc. are skipped.
-
-ALL_TARGET_TOOL_NAMES: tool names the hook fires after. The hook is a PostToolUse
-listener; only Write and Edit produce a writable file path that the marker check
-can be applied to.
-"""
-
-PUBLISH_SENTINEL = "<!-- @publish-as-gist -->"
-HTML_FILE_EXTENSION = ".html"
-ALL_TARGET_TOOL_NAMES = ("Write", "Edit")
-HOOK_SUBPROCESS_TIMEOUT_SECONDS = 50
-UPLOAD_SCRIPT_RELATIVE_PATH = "skills/doc-gist/scripts/gist_upload.py"

package/hooks/hooks_constants/html_companion_constants.py

@@ -1,20 +0,0 @@
-"""CSS theme constants and URL scheme denylist for md-to-html companion hook."""
-
-CSS_BG_COLOR = "13, 17, 23"
-CSS_FG_COLOR = "201, 209, 217"
-CSS_BORDER_COLOR = "48, 54, 61"
-CSS_ACCENT_COLOR = "88, 166, 255"
-CSS_MUTED_COLOR = "139, 148, 158"
-CSS_SURFACE_COLOR = "22, 27, 34"
-CSS_STRONG_COLOR = "240, 246, 252"
-CSS_LINE_HEIGHT = "1.6"
-CSS_BODY_PADDING = "2rem"
-CSS_MAX_WIDTH = "960px"
-CSS_H1_SIZE = "1.6rem"
-CSS_H2_SIZE = "1.25rem"
-CSS_H3_SIZE = "1.1rem"
-CSS_CODE_SIZE = "0.85rem"
-CSS_TABLE_WIDTH = "100%"
-CSS_TH_WEIGHT = "600"
-
-BLOCKED_URL_SCHEMES = frozenset({"javascript", "data"})

package/hooks/hooks_constants/md_to_html_blocker_constants.py

@@ -1,76 +0,0 @@
-"""Configuration constants for the md_to_html_blocker PreToolUse hook
-and its shared exemption helpers (`md_path_exemptions`)."""
-
-from __future__ import annotations
-
-import os
-import tempfile
-
-
-ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES: frozenset[str] = frozenset(
-    {"agents", "docs", "skills", "rules", "system-prompts", "commands"}
-)
-
-PACKAGES_TOP_LEVEL_SEGMENT: str = "packages"
-CLAUDE_DEV_ENV_REPO_NAME_SEGMENT: str = "claude-dev-env"
-
-MINIMUM_SEGMENT_COUNT_TO_MATCH_INDICATOR: int = 4
-
-ALL_EXEMPT_ANYWHERE_FILENAMES: tuple[str, ...] = ("SKILL.md",)
-ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS: tuple[str, ...] = ("agents", "skills", "commands")
-ALL_EXEMPT_HOME_RELATIVE_DIRECTORIES: tuple[str, ...] = ("SessionLog",)
-ALL_EXEMPT_ROOT_FILENAMES: tuple[str, ...] = ("readme.md", "changelog.md", "claude.md", "agents.md")
-REPO_ROOT_MARKER_NAME: str = ".git"
-CLAUDE_DIRECTORY_NAME: str = ".claude"
-PLUGIN_ROOT_MARKER_DIRECTORY_NAME: str = ".claude-plugin"
-
-CLAUDE_PROFILE_DIRECTORY_NAME_PREFIX: str = f"{CLAUDE_DIRECTORY_NAME}-"
-
-ALL_EXEMPT_ANYWHERE_FILENAMES_LOWER: frozenset[str] = frozenset(
-    each_filename.lower() for each_filename in ALL_EXEMPT_ANYWHERE_FILENAMES
-)
-ALL_EXEMPT_ROOT_FILENAMES_LOWER: frozenset[str] = frozenset(
-    each_filename.lower() for each_filename in ALL_EXEMPT_ROOT_FILENAMES
-)
-
-
-def _resolve_canonical_directory_lowercase(directory_path: str) -> str:
-    return (
-        os.path.realpath(directory_path).replace("\\", "/").rstrip("/").lower()
-    )
-
-
-RESOLVED_HOME_DIRECTORY_LOWER: str = _resolve_canonical_directory_lowercase(
-    os.path.expanduser("~")
-)
-RESOLVED_TEMP_DIRECTORY_LOWER: str = _resolve_canonical_directory_lowercase(
-    tempfile.gettempdir()
-)
-
-ALL_EXEMPT_HOME_DIRECTORY_PATH_PREFIXES: tuple[str, ...] = tuple(
-    f"{RESOLVED_HOME_DIRECTORY_LOWER}/{each_relative_directory.lower()}/"
-    for each_relative_directory in ALL_EXEMPT_HOME_RELATIVE_DIRECTORIES
-)
-RESOLVED_TEMP_DIRECTORY_PATH_PREFIX: str = f"{RESOLVED_TEMP_DIRECTORY_LOWER}/"
-
-
-__all__ = [
-    "ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES",
-    "ALL_EXEMPT_ANYWHERE_FILENAMES",
-    "ALL_EXEMPT_ANYWHERE_FILENAMES_LOWER",
-    "ALL_EXEMPT_HOME_DIRECTORY_PATH_PREFIXES",
-    "ALL_EXEMPT_HOME_RELATIVE_DIRECTORIES",
-    "ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS",
-    "ALL_EXEMPT_ROOT_FILENAMES",
-    "ALL_EXEMPT_ROOT_FILENAMES_LOWER",
-    "CLAUDE_DEV_ENV_REPO_NAME_SEGMENT",
-    "CLAUDE_DIRECTORY_NAME",
-    "CLAUDE_PROFILE_DIRECTORY_NAME_PREFIX",
-    "MINIMUM_SEGMENT_COUNT_TO_MATCH_INDICATOR",
-    "PACKAGES_TOP_LEVEL_SEGMENT",
-    "PLUGIN_ROOT_MARKER_DIRECTORY_NAME",
-    "REPO_ROOT_MARKER_NAME",
-    "RESOLVED_HOME_DIRECTORY_LOWER",
-    "RESOLVED_TEMP_DIRECTORY_LOWER",
-    "RESOLVED_TEMP_DIRECTORY_PATH_PREFIX",
-]

package/hooks/hooks_constants/test_md_to_html_blocker_constants.py

@@ -1,125 +0,0 @@
-"""Behavior tests for md_to_html_blocker_constants module."""
-
-from __future__ import annotations
-
-import sys
-from pathlib import Path
-
-_HOOKS_ROOT = Path(__file__).resolve().parent.parent
-if str(_HOOKS_ROOT) not in sys.path:
-    sys.path.insert(0, str(_HOOKS_ROOT))
-
-from hooks_constants import md_to_html_blocker_constants as constants_module
-
-
-def test_indicator_path_segments_are_named_constants() -> None:
-    """The two leading segments of the Claude Code source indicator
-    (`packages/claude-dev-env`) must live as named constants so
-    `_is_exempt_path` references symbols, not inline literals. Bugbot flagged
-    these as magic strings even though the third segment was already a named
-    constant; align all three."""
-    assert constants_module.PACKAGES_TOP_LEVEL_SEGMENT == "packages"
-    assert constants_module.CLAUDE_DEV_ENV_REPO_NAME_SEGMENT == "claude-dev-env"
-    for each_name in ("PACKAGES_TOP_LEVEL_SEGMENT", "CLAUDE_DEV_ENV_REPO_NAME_SEGMENT"):
-        assert each_name in constants_module.__all__
-
-
-def test_claude_code_source_top_directories_enumerates_six_canonical_dirs() -> None:
-    """The exempt top-level directory set must match the six canonical Claude
-    Code source directories (agents, docs, skills, rules, system-prompts,
-    commands). Drift in either direction silently changes the exemption
-    surface."""
-    expected_directories = frozenset(
-        {"agents", "docs", "skills", "rules", "system-prompts", "commands"}
-    )
-    assert constants_module.ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES == expected_directories
-    assert "ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES" in constants_module.__all__
-
-
-def test_minimum_segment_count_to_match_indicator_is_four() -> None:
-    """Matching `packages/claude-dev-env/<dir>/<file>` requires at least 4
-    consecutive path segments from the starting index. The constant documents
-    that requirement explicitly."""
-    assert constants_module.MINIMUM_SEGMENT_COUNT_TO_MATCH_INDICATOR == 4
-    assert "MINIMUM_SEGMENT_COUNT_TO_MATCH_INDICATOR" in constants_module.__all__
-
-
-def test_exempt_anywhere_filenames_include_skill_md() -> None:
-    """`SKILL.md` files are exempt anywhere in the tree. The constant stores
-    the display-case spelling; the lookup at use sites lowercases the
-    candidate basename, so casing here documents the human-facing form."""
-    assert "SKILL.md" in constants_module.ALL_EXEMPT_ANYWHERE_FILENAMES
-    assert "ALL_EXEMPT_ANYWHERE_FILENAMES" in constants_module.__all__
-
-
-def test_exempt_plugin_directory_segments_match_claude_code_layout() -> None:
-    """The three plugin-layout directory names recognized anywhere in a path:
-    agents/, skills/, commands/. Drift in either direction silently changes
-    the exemption surface."""
-    expected_segments = ("agents", "skills", "commands")
-    assert constants_module.ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS == expected_segments
-    assert "ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS" in constants_module.__all__
-
-
-def test_exempt_plugin_segments_subset_of_claude_code_source_top_directories() -> None:
-    """ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS (matched anywhere in the path)
-    must be a subset of ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES (matched
-    only at the anchored claude-dev-env source root). If a future change
-    adds a segment to the anywhere list that is not in the anchored set,
-    the anywhere rule would let writes through that the anchored rule
-    would still block — surprising and asymmetric."""
-    anywhere_segments = set(constants_module.ALL_EXEMPT_PLUGIN_DIRECTORY_SEGMENTS)
-    anchored_source_directories = set(
-        constants_module.ALL_CLAUDE_CODE_SOURCE_TOP_DIRECTORIES
-    )
-    assert anywhere_segments.issubset(anchored_source_directories)
-
-
-def test_exempt_home_relative_directories_include_session_log() -> None:
-    """SessionLog/ under the user's home directory is the canonical Obsidian
-    vault entrypoint and must be writable as .md."""
-    assert "SessionLog" in constants_module.ALL_EXEMPT_HOME_RELATIVE_DIRECTORIES
-    assert "ALL_EXEMPT_HOME_RELATIVE_DIRECTORIES" in constants_module.__all__
-
-
-def test_exempt_root_filenames_cover_readme_changelog_claude_and_agents() -> None:
-    """README.md, CHANGELOG.md, CLAUDE.md, and AGENTS.md at a repo root are
-    universally exempt; every repo with a `.git` marker satisfies the root
-    check. CLAUDE.md and AGENTS.md are functional agent-instruction files that
-    Claude Code loads by name and must stay Markdown."""
-    assert constants_module.ALL_EXEMPT_ROOT_FILENAMES == (
-        "readme.md",
-        "changelog.md",
-        "claude.md",
-        "agents.md",
-    )
-    assert "ALL_EXEMPT_ROOT_FILENAMES" in constants_module.__all__
-
-
-def test_repo_root_marker_name_is_dot_git() -> None:
-    """A directory containing `.git` (file or directory) is treated as a repo
-    root for the README/CHANGELOG exemption."""
-    assert constants_module.REPO_ROOT_MARKER_NAME == ".git"
-    assert "REPO_ROOT_MARKER_NAME" in constants_module.__all__
-
-
-def test_claude_directory_name_is_dot_claude() -> None:
-    """Any path containing a `.claude/` segment bypasses the .md block
-    (project-level Claude Code infrastructure)."""
-    assert constants_module.CLAUDE_DIRECTORY_NAME == ".claude"
-    assert "CLAUDE_DIRECTORY_NAME" in constants_module.__all__
-
-
-def test_plugin_root_marker_directory_name_is_dot_claude_plugin() -> None:
-    """Any directory whose ancestor contains `.claude-plugin/` is treated as
-    a plugin repo root and exempted."""
-    assert constants_module.PLUGIN_ROOT_MARKER_DIRECTORY_NAME == ".claude-plugin"
-    assert "PLUGIN_ROOT_MARKER_DIRECTORY_NAME" in constants_module.__all__
-
-
-def test_claude_profile_directory_name_prefix_is_dot_claude_hyphen() -> None:
-    """A directory whose name carries the `.claude-` prefix (profile
-    directories like `.claude-mel/`, plus `.claude-plugin/`) is Claude
-    infrastructure; any path inside one bypasses the .md block."""
-    assert constants_module.CLAUDE_PROFILE_DIRECTORY_NAME_PREFIX == ".claude-"
-    assert "CLAUDE_PROFILE_DIRECTORY_NAME_PREFIX" in constants_module.__all__