claude-dev-env 2.12.0 → 2.14.0

package/hooks/hooks_constants/pr_description_enforcer_constants.py

@@ -1,167 +0,0 @@
-"""Configuration constants for the pr_description_enforcer PreToolUse hook."""
-
-from __future__ import annotations
-
-import os
-import re
-from dataclasses import dataclass
-from pathlib import Path
-
-
-_PLUGIN_ROOT: str = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
-PR_GUIDE_PATH: str = os.path.join(_PLUGIN_ROOT, "docs", "PR_DESCRIPTION_GUIDE.md")
-
-MINIMUM_SUBSTANTIVE_PROSE_CHARS: int = 40
-
-FENCED_CODE_BLOCK_PATTERN: re.Pattern[str] = re.compile(r"```.*?```", re.DOTALL)
-INLINE_CODE_PATTERN: re.Pattern[str] = re.compile(r"`[^`]*`")
-HEADING_LINE_PATTERN: re.Pattern[str] = re.compile(r"^#+[ \t].*$", re.MULTILINE)
-BOLD_PAIR_PATTERN: re.Pattern[str] = re.compile(r"\*\*([^*]+?)\*\*")
-BULLET_MARKER_PATTERN: re.Pattern[str] = re.compile(r"^\s*[-*+]\s+", re.MULTILINE)
-BLOCKQUOTE_MARKER_PATTERN: re.Pattern[str] = re.compile(r"^\s*>\s+", re.MULTILINE)
-BLOCKQUOTE_LINE_PATTERN: re.Pattern[str] = re.compile(r"^\s*>.*$", re.MULTILINE)
-TABLE_ROW_LINE_PATTERN: re.Pattern[str] = re.compile(r"^\s*\|.*\|.*$", re.MULTILINE)
-LINK_TEXT_PATTERN: re.Pattern[str] = re.compile(r"\[([^\]]+)\]\([^)]+\)")
-WHITESPACE_RUN_PATTERN: re.Pattern[str] = re.compile(r"\s+")
-VAGUE_LANGUAGE_PATTERN: re.Pattern[str] = re.compile(
-    r"\b(fix(?:ed)? (?:bug|issue|it)|update(?:d)? code|minor changes|various (?:fixes|updates|improvements))\b",
-    re.IGNORECASE,
-)
-
-SUMMARY_HEADER: str = "## Summary"
-PROBLEM_HEADER: str = "## Problem"
-TEST_PLAN_HEADER: str = "## Test plan"
-TESTS_HEADER: str = "## Tests"
-TESTING_HEADER: str = "## Testing"
-VERIFICATION_HEADER: str = "## Verification"
-VALIDATION_HEADER: str = "## Validation"
-
-ALL_HEAVY_OPENING_HEADERS: frozenset[str] = frozenset({PROBLEM_HEADER, SUMMARY_HEADER})
-ALL_HEAVY_TESTING_HEADERS: frozenset[str] = frozenset(
-    {TEST_PLAN_HEADER, TESTING_HEADER, TESTS_HEADER, VERIFICATION_HEADER, VALIDATION_HEADER}
-)
-GH_PR_COMMAND_MIN_TOKEN_COUNT: int = 3
-ALL_PR_NUMBER_BEARING_GH_PR_SUBCOMMANDS: frozenset[str] = frozenset({"edit", "comment", "ready"})
-BODY_FILE_STDIN_SENTINEL: str = "-"
-ATOMIC_WRITE_TEMP_SUFFIX: str = ".tmp"
-SELF_CLOSING_REFERENCE_MESSAGE_PREFIX: str = "PR body references its own PR number #"
-SELF_CLOSING_REFERENCE_MESSAGE_SUFFIX: str = (
-    " as a self-closing keyword (Fixes/Closes/Resolves) -- remove the self-reference"
-)
-
-SELF_REFERENCE_PATTERN_TEMPLATE: str = (
-    r"\b(?:Close|Closes|Closed|Fix|Fixes|Fixed|Resolve|Resolves|Resolved)"
-    r"\s+#{pr_number}\b"
-)
-
-THIS_PR_OPENING_PATTERN: re.Pattern[str] = re.compile(
-    r"^\s*(?:#[^\n]*\n\s*)*This PR\b",
-    re.IGNORECASE,
-)
-
-TRIVIAL_SHAPE: str = "trivial"
-STANDARD_SHAPE: str = "standard"
-HEAVY_SHAPE: str = "heavy"
-
-TRIVIAL_BODY_CHAR_THRESHOLD: int = 200
-HEAVY_MIN_BODY_CHARS_FOR_CLASSIFICATION: int = 500
-
-READABILITY_MAX_SENTENCE_WORDS: int = 28
-READABILITY_AVG_SENTENCE_WORDS: int = 18
-READABILITY_MIN_FLESCH: int = 50
-READABILITY_STRIKE_THRESHOLD: int = 3
-
-READABILITY_LOOSEN_CAP: int = 3
-READABILITY_MIN_FLESCH_FLOOR: int = 30
-READABILITY_MAX_SENTENCE_WORDS_CEILING: int = 60
-READABILITY_AVG_SENTENCE_WORDS_CEILING: int = 40
-
-READABILITY_FLESCH_LOOSEN_FACTOR: float = 0.9
-READABILITY_SENTENCE_WORDS_LOOSEN_FACTOR: float = 10 / 9
-
-ALL_READABILITY_CLI_FLAG_TOKENS: frozenset[str] = frozenset(
-    {
-        "--readability-loosen",
-        "--readability-reset",
-        "--readability-disable",
-        "--readability-enable",
-    }
-)
-
-FLESCH_BASE_SCORE: float = 206.835
-FLESCH_WORDS_PER_SENTENCE_COEFFICIENT: float = 1.015
-FLESCH_SYLLABLES_PER_WORD_COEFFICIENT: float = 84.6
-FLESCH_PERFECT_SCORE: float = 100.0
-
-READABILITY_STATE_FILE: Path = Path.home() / ".claude" / "state" / "pr_description_readability_strikes.json"
-READABILITY_THRESHOLD_OVERRIDE_FILE: Path = (
-    Path.home() / ".claude" / "state" / "pr_description_readability_overrides.json"
-)
-READABILITY_ENABLED_STATE_FILE: Path = (
-    Path.home() / ".claude" / "state" / "pr_description_readability_enabled.json"
-)
-
-
-@dataclass(frozen=True)
-class ReadabilityThresholds:
-    """Three readability thresholds carried as a single typed structure."""
-
-    flesch_min: int
-    max_sentence_words: int
-    avg_sentence_words: int
-
-
-DEFAULT_READABILITY_THRESHOLDS: ReadabilityThresholds = ReadabilityThresholds(
-    flesch_min=READABILITY_MIN_FLESCH,
-    max_sentence_words=READABILITY_MAX_SENTENCE_WORDS,
-    avg_sentence_words=READABILITY_AVG_SENTENCE_WORDS,
-)
-
-
-__all__ = [
-    "ALL_HEAVY_OPENING_HEADERS",
-    "ALL_HEAVY_TESTING_HEADERS",
-    "ALL_PR_NUMBER_BEARING_GH_PR_SUBCOMMANDS",
-    "ALL_READABILITY_CLI_FLAG_TOKENS",
-    "ATOMIC_WRITE_TEMP_SUFFIX",
-    "BLOCKQUOTE_LINE_PATTERN",
-    "BLOCKQUOTE_MARKER_PATTERN",
-    "BODY_FILE_STDIN_SENTINEL",
-    "BOLD_PAIR_PATTERN",
-    "BULLET_MARKER_PATTERN",
-    "DEFAULT_READABILITY_THRESHOLDS",
-    "FENCED_CODE_BLOCK_PATTERN",
-    "FLESCH_BASE_SCORE",
-    "FLESCH_PERFECT_SCORE",
-    "FLESCH_SYLLABLES_PER_WORD_COEFFICIENT",
-    "FLESCH_WORDS_PER_SENTENCE_COEFFICIENT",
-    "GH_PR_COMMAND_MIN_TOKEN_COUNT",
-    "HEADING_LINE_PATTERN",
-    "HEAVY_MIN_BODY_CHARS_FOR_CLASSIFICATION",
-    "HEAVY_SHAPE",
-    "INLINE_CODE_PATTERN",
-    "LINK_TEXT_PATTERN",
-    "MINIMUM_SUBSTANTIVE_PROSE_CHARS",
-    "PR_GUIDE_PATH",
-    "READABILITY_AVG_SENTENCE_WORDS_CEILING",
-    "READABILITY_ENABLED_STATE_FILE",
-    "READABILITY_FLESCH_LOOSEN_FACTOR",
-    "READABILITY_LOOSEN_CAP",
-    "READABILITY_MAX_SENTENCE_WORDS_CEILING",
-    "READABILITY_MIN_FLESCH_FLOOR",
-    "READABILITY_SENTENCE_WORDS_LOOSEN_FACTOR",
-    "READABILITY_STATE_FILE",
-    "READABILITY_STRIKE_THRESHOLD",
-    "READABILITY_THRESHOLD_OVERRIDE_FILE",
-    "ReadabilityThresholds",
-    "SELF_CLOSING_REFERENCE_MESSAGE_PREFIX",
-    "SELF_CLOSING_REFERENCE_MESSAGE_SUFFIX",
-    "SELF_REFERENCE_PATTERN_TEMPLATE",
-    "STANDARD_SHAPE",
-    "TABLE_ROW_LINE_PATTERN",
-    "THIS_PR_OPENING_PATTERN",
-    "TRIVIAL_BODY_CHAR_THRESHOLD",
-    "TRIVIAL_SHAPE",
-    "VAGUE_LANGUAGE_PATTERN",
-    "WHITESPACE_RUN_PATTERN",
-]

package/hooks/hooks_constants/pr_description_proof_of_work_constants.py

@@ -1,107 +0,0 @@
-"""Configuration constants for the proof-of-work PR-comment audit.
-
-Holds the proof-part detection patterns, the gap and linkage word lists,
-the gh command tokens the audit runs to read a PR's comments and diff, and
-the corrective messages returned when a proof part is missing or when
-``gh pr ready`` runs with no passing proof comment on the PR.
-"""
-
-from __future__ import annotations
-
-import re
-
-ALL_PROOF_HEADING_KEYWORDS: tuple[str, ...] = ("proof", "verification")
-
-PR_READY_INVOCATION_PATTERN: re.Pattern[str] = re.compile(r"(?:^|[;&|])\s*gh\s+pr\s+ready\b")
-PR_READY_UNDO_FLAG: str = "--undo"
-
-GH_EXECUTABLE: str = "gh"
-GH_API_SUBCOMMAND: str = "api"
-GH_PAGINATE_FLAG: str = "--paginate"
-GH_SLURP_FLAG: str = "--slurp"
-PR_COMMENTS_API_PATH_TEMPLATE: str = "repos/{{owner}}/{{repo}}/issues/{pr_number}/comments"
-ALL_PR_DIFF_SUBCOMMANDS: tuple[str, ...] = ("pr", "diff")
-PR_DIFF_NAME_ONLY_FLAG: str = "--name-only"
-COMMENT_BODY_JSON_FIELD: str = "body"
-GH_COMMAND_TIMEOUT_SECONDS: int = 6
-MAX_DIFF_SCAN_CHARS: int = 200000
-
-ALL_VISUAL_FILE_SUFFIXES: frozenset[str] = frozenset(
-    {".png", ".jpg", ".jpeg", ".gif", ".webp", ".avif", ".svg", ".ico", ".html", ".css"}
-)
-HEX_COLOR_ADDED_LINE_PATTERN: re.Pattern[str] = re.compile(r"^\+.*#[0-9A-Fa-f]{6}\b", re.MULTILINE)
-IMAGE_EMBED_PATTERN: re.Pattern[str] = re.compile(r"!\[[^\]]*\]\([^)\s]+\)")
-ISSUE_REFERENCE_PATTERN: re.Pattern[str] = re.compile(r"#\d+")
-PLAN_LINKAGE_KEYWORD_PATTERN: re.Pattern[str] = re.compile(
-    r"\b(?:issue|phase|plan|parent|advanc\w*|milestone|part of)\b", re.IGNORECASE
-)
-DIGIT_PATTERN: re.Pattern[str] = re.compile(r"\d")
-ALL_HONEST_GAP_PHRASES: tuple[str, ...] = (
-    "gap",
-    "limitation",
-    "cannot",
-    "does not show",
-    "not shown",
-    "unverified",
-    "not covered",
-)
-
-PROOF_PART_COMMAND_MESSAGE: str = (
-    "proof comment missing a fenced command block showing the exact command(s) run on real data"
-)
-PROOF_PART_MEASURED_MESSAGE: str = (
-    "proof comment missing a measured-value element -- add a table row or bullet fact carrying "
-    "the numbers read from the produced artifact"
-)
-PROOF_PART_PLAN_LINKAGE_MESSAGE: str = (
-    "proof comment missing a plan-linkage sentence -- add one line naming the parent issue or "
-    "phase this PR advances, with its #number"
-)
-PROOF_PART_VISUAL_MESSAGE: str = (
-    "proof comment missing a visual element on a visual change -- embed an image (color "
-    "swatches, before/after screenshots) for values a human cannot read at a glance"
-)
-PROOF_PART_HONEST_GAPS_MESSAGE: str = (
-    "proof comment missing an honest-gaps statement -- say plainly what this offline proof "
-    "cannot show and what covers that gap"
-)
-
-PR_READY_GATE_MESSAGE_TEMPLATE: str = (
-    "BLOCKED: [PROOF_OF_WORK] PR #{pr_number} carries no passing proof-of-work comment. "
-    "Post one comment with a proof or verification heading carrying: "
-    "(1) the exact command(s) run on real data in a fenced code block, "
-    "(2) measured outcomes from the produced artifact as a table row or bullet facts, "
-    "(3) one sentence naming the parent issue or phase this PR advances, with its #number, "
-    "(4) an image embed when the diff touches images, HTML, CSS, or hex color values, and "
-    "(5) an honest statement of what the offline proof cannot show. "
-    "Then re-run gh pr ready."
-)
-
-__all__ = [
-    "ALL_HONEST_GAP_PHRASES",
-    "ALL_PROOF_HEADING_KEYWORDS",
-    "ALL_PR_DIFF_SUBCOMMANDS",
-    "ALL_VISUAL_FILE_SUFFIXES",
-    "COMMENT_BODY_JSON_FIELD",
-    "DIGIT_PATTERN",
-    "GH_API_SUBCOMMAND",
-    "GH_COMMAND_TIMEOUT_SECONDS",
-    "GH_EXECUTABLE",
-    "GH_PAGINATE_FLAG",
-    "GH_SLURP_FLAG",
-    "HEX_COLOR_ADDED_LINE_PATTERN",
-    "IMAGE_EMBED_PATTERN",
-    "ISSUE_REFERENCE_PATTERN",
-    "MAX_DIFF_SCAN_CHARS",
-    "PLAN_LINKAGE_KEYWORD_PATTERN",
-    "PROOF_PART_COMMAND_MESSAGE",
-    "PROOF_PART_HONEST_GAPS_MESSAGE",
-    "PROOF_PART_MEASURED_MESSAGE",
-    "PROOF_PART_PLAN_LINKAGE_MESSAGE",
-    "PROOF_PART_VISUAL_MESSAGE",
-    "PR_COMMENTS_API_PATH_TEMPLATE",
-    "PR_DIFF_NAME_ONLY_FLAG",
-    "PR_READY_GATE_MESSAGE_TEMPLATE",
-    "PR_READY_INVOCATION_PATTERN",
-    "PR_READY_UNDO_FLAG",
-]

package/hooks/hooks_constants/reviewer_spawn_gate_constants.py

@@ -1,41 +0,0 @@
-"""Configuration constants for the reviewer_spawn_gate PreToolUse hook."""
-
-from __future__ import annotations
-
-import re
-from pathlib import Path
-
-BASH_TOOL_NAME: str = "Bash"
-
-GATE_SENTINEL_MARKER: str = "CLAUDE_REVIEWER_GATE=autoconverge"
-
-COPILOT_REVIEWER_TOKEN: str = "copilot"
-BUGBOT_REVIEWER_TOKEN: str = "bugbot"
-
-COPILOT_REVIEWER_LABEL: str = "GitHub Copilot"
-BUGBOT_REVIEWER_LABEL: str = "Cursor Bugbot"
-
-ALL_COPILOT_TRIGGER_MARKERS: tuple[str, ...] = (
-    "requested_reviewers",
-    "copilot-pull-request-reviewer[bot]",
-)
-
-BUGBOT_TRIGGER_SCRIPT_MARKER: str = "post_fix_reply.py"
-BUGBOT_RUN_BODY_PATTERN: re.Pattern[str] = re.compile(
-    r'--body\s+["\']bugbot run["\']', re.IGNORECASE
-)
-
-AVAILABILITY_SCRIPT_RELATIVE_PATH: Path = (
-    Path("_shared") / "pr-loop" / "scripts" / "reviewer_availability.py"
-)
-AVAILABILITY_SCRIPT_PATH_ENV_VAR_NAME: str = "REVIEWER_SPAWN_GATE_AVAILABILITY_SCRIPT_PATH"
-AVAILABILITY_SCRIPT_TIMEOUT_SECONDS: int = 15
-AVAILABILITY_REVIEWER_FLAG: str = "--reviewer"
-AVAILABILITY_DOWN_EXIT_CODE: int = 3
-
-DENY_REASON_TEMPLATE: str = (
-    "BLOCKED [reviewer-spawn-gate]: {reviewer_label} is down or out of quota "
-    "this run. Skip requesting {reviewer_label} on this trigger — "
-    "autoconverge requests it again once the availability check reports it "
-    "back.\n\nAvailability check output: {availability_detail}"
-)

package/hooks/hooks_constants/test_pr_description_enforcer_constants.py

@@ -1,292 +0,0 @@
-"""Behavior tests for pr_description_enforcer_constants module."""
-
-from __future__ import annotations
-
-import os
-import sys
-from pathlib import Path
-
-import pytest
-
-_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 pr_description_enforcer_constants as constants_module
-
-
-def test_plugin_root_is_private_module_attribute() -> None:
-    assert hasattr(constants_module, "_PLUGIN_ROOT")
-    assert isinstance(constants_module._PLUGIN_ROOT, str)
-    assert os.path.isabs(constants_module._PLUGIN_ROOT)
-
-
-def test_plugin_root_public_name_is_not_exported() -> None:
-    assert not hasattr(constants_module, "PLUGIN_ROOT")
-
-
-def test_pr_guide_path_resolves_under_plugin_root_docs() -> None:
-    expected_pr_guide_path = os.path.join(
-        constants_module._PLUGIN_ROOT, "docs", "PR_DESCRIPTION_GUIDE.md"
-    )
-    assert constants_module.PR_GUIDE_PATH == expected_pr_guide_path
-
-
-def test_fenced_code_block_pattern_matches_triple_backtick_block() -> None:
-    sample_markdown = "before ```python\ncode\n``` after"
-    match = constants_module.FENCED_CODE_BLOCK_PATTERN.search(sample_markdown)
-    assert match is not None
-    assert match.group(0).startswith("```")
-    assert match.group(0).endswith("```")
-
-
-def test_inline_code_pattern_matches_single_backtick_span() -> None:
-    match = constants_module.INLINE_CODE_PATTERN.search("see `value` here")
-    assert match is not None
-    assert match.group(0) == "`value`"
-
-
-def test_heading_line_pattern_matches_atx_heading() -> None:
-    match = constants_module.HEADING_LINE_PATTERN.search("## Description\n")
-    assert match is not None
-    assert match.group(0).strip() == "## Description"
-
-
-def test_bold_pair_pattern_captures_inner_text() -> None:
-    match = constants_module.BOLD_PAIR_PATTERN.search("this is **bold** text")
-    assert match is not None
-    assert match.group(1) == "bold"
-
-
-def test_bullet_marker_pattern_strips_dash_bullet_from_line() -> None:
-    stripped_line = constants_module.BULLET_MARKER_PATTERN.sub("", "- first item")
-    assert stripped_line == "first item"
-
-
-def test_blockquote_marker_pattern_strips_quote_marker_from_line() -> None:
-    stripped_line = constants_module.BLOCKQUOTE_MARKER_PATTERN.sub("", "> quoted line")
-    assert stripped_line == "quoted line"
-
-
-def test_link_text_pattern_captures_anchor_text() -> None:
-    match = constants_module.LINK_TEXT_PATTERN.search("See [the docs](https://example.com) now")
-    assert match is not None
-    assert match.group(1) == "the docs"
-
-
-def test_whitespace_run_pattern_collapses_multiple_spaces() -> None:
-    collapsed_text = constants_module.WHITESPACE_RUN_PATTERN.sub(" ", "a   b\t\tc\n\nd")
-    assert collapsed_text == "a b c d"
-
-
-def test_this_pr_opening_pattern_matches_any_verb() -> None:
-    """The guide says any `This PR ...` opening is a hard block. The pattern
-    must match regardless of the verb that follows, not a short allowlist."""
-    test_inputs = [
-        "This PR introduces a new feature.",
-        "This PR improves the algorithm.",
-        "This PR enables the cache.",
-        "This PR documents the contract.",
-        "This PR adds a check.",
-        "This PR fixes the bug.",
-    ]
-    for each_input in test_inputs:
-        assert constants_module.THIS_PR_OPENING_PATTERN.match(each_input), (
-            f"`{each_input}` must match THIS_PR_OPENING_PATTERN regardless of verb"
-        )
-
-
-def test_this_pr_opening_pattern_does_not_match_within_prose() -> None:
-    """The block applies to body openings, not mid-paragraph mentions."""
-    text_with_mid_mention = "Adds caching. This PR follows the playbook."
-    assert constants_module.THIS_PR_OPENING_PATTERN.match(text_with_mid_mention) is None
-
-
-def test_all_list_is_alphabetically_sorted() -> None:
-    """`__all__` is the export surface; alphabetical order makes it easy to
-    verify completeness, spot duplicates, and bisect missing entries. Pin
-    sorted order so drift cannot reintroduce the H-before-G / appended-at-end
-    inconsistencies Bugbot flagged."""
-    actual_all_list = constants_module.__all__
-    expected_sorted = sorted(actual_all_list)
-    if actual_all_list != expected_sorted:
-        first_divergence_index = next(
-            each_index
-            for each_index, (each_actual_value, each_expected_value) in enumerate(zip(actual_all_list, expected_sorted))
-            if each_actual_value != each_expected_value
-        )
-        raise AssertionError(
-            "constants_module.__all__ must be alphabetically sorted; "
-            f"the first divergence is at index {first_divergence_index}: "
-            f"got {actual_all_list[first_divergence_index]!r}, "
-            f"expected {expected_sorted[first_divergence_index]!r}"
-        )
-
-
-def test_dead_heavy_detection_constants_are_removed() -> None:
-    """`ALL_HEAVY_DETECTION_HEADERS` and `HEAVY_DETECTION_HEADER_COUNT_MIN`
-    were remnants of the prior two-condition shape classifier (length AND
-    detection-header count). e137dee9 simplified the classifier to use length
-    alone, leaving both names as dead exports. Pin their removal so they
-    cannot drift back as misleading vestigial constants."""
-    for each_name in ("ALL_HEAVY_DETECTION_HEADERS", "HEAVY_DETECTION_HEADER_COUNT_MIN"):
-        assert not hasattr(constants_module, each_name), (
-            f"{each_name} must be deleted; the classifier now uses "
-            "HEAVY_MIN_BODY_CHARS_FOR_CLASSIFICATION alone."
-        )
-        assert each_name not in constants_module.__all__
-
-
-def test_self_closing_reference_message_prefix_and_suffix_compose_full_message() -> None:
-    pr_number = 42
-    composed_message = (
-        f"{constants_module.SELF_CLOSING_REFERENCE_MESSAGE_PREFIX}"
-        f"{pr_number}"
-        f"{constants_module.SELF_CLOSING_REFERENCE_MESSAGE_SUFFIX}"
-    )
-    assert "#42" in composed_message
-    assert "Fixes/Closes/Resolves" in composed_message
-    assert "self-reference" in composed_message
-
-
-def test_loosen_factors_are_inverse_paired() -> None:
-    flesch_factor = constants_module.READABILITY_FLESCH_LOOSEN_FACTOR
-    sentence_factor = constants_module.READABILITY_SENTENCE_WORDS_LOOSEN_FACTOR
-    assert flesch_factor * sentence_factor == pytest.approx(1.0)
-
-
-def test_unused_header_constants_are_removed_from_module() -> None:
-    """The four header constants FIX_HEADER, CHANGES_HEADER, APPROACH_HEADER,
-    and ROOT_CAUSE_HEADER were dead exports with no call sites and must not be
-    re-introduced as either module attributes or __all__ entries."""
-    for each_dead_name in ("FIX_HEADER", "CHANGES_HEADER", "APPROACH_HEADER", "ROOT_CAUSE_HEADER"):
-        assert not hasattr(constants_module, each_dead_name), (
-            f"{each_dead_name} was re-introduced; the four header constants "
-            "had zero call sites and were deleted."
-        )
-        assert each_dead_name not in constants_module.__all__, (
-            f"{each_dead_name} re-appeared in __all__ even though it has no consumers."
-        )
-
-
-def test_module_internal_header_constants_are_not_publicly_exported() -> None:
-    """The seven header-name constants and three default-readability-threshold scalars
-    are consumed only inside this same module (used to build ALL_HEAVY_* frozensets
-    and DEFAULT_READABILITY_THRESHOLDS). They must remain module attributes for the
-    builder expressions, but they must NOT appear in __all__ -- callers should import
-    the public aggregates (ALL_HEAVY_OPENING_HEADERS / ALL_HEAVY_TESTING_HEADERS /
-    DEFAULT_READABILITY_THRESHOLDS) instead of the individual scalars."""
-    all_internal_only_names = (
-        "SUMMARY_HEADER",
-        "PROBLEM_HEADER",
-        "TEST_PLAN_HEADER",
-        "TESTS_HEADER",
-        "TESTING_HEADER",
-        "VERIFICATION_HEADER",
-        "VALIDATION_HEADER",
-        "READABILITY_MAX_SENTENCE_WORDS",
-        "READABILITY_AVG_SENTENCE_WORDS",
-        "READABILITY_MIN_FLESCH",
-    )
-    for each_internal_name in all_internal_only_names:
-        assert hasattr(constants_module, each_internal_name), (
-            f"{each_internal_name} must remain as a module attribute -- "
-            "the aggregate builders reference it."
-        )
-        assert each_internal_name not in constants_module.__all__, (
-            f"{each_internal_name} is consumed only inside the module and "
-            "must not be advertised in __all__."
-        )
-
-
-def test_self_reference_pattern_covers_all_nine_github_closing_keywords() -> None:
-    """GitHub recognizes nine closing keywords: close/closes/closed,
-    fix/fixes/fixed, resolve/resolves/resolved. The self-reference template
-    must enumerate all nine so no variant bypasses the self-closing block."""
-    pattern_source = constants_module.SELF_REFERENCE_PATTERN_TEMPLATE
-    for each_keyword in (
-        "Close", "Closes", "Closed",
-        "Fix", "Fixes", "Fixed",
-        "Resolve", "Resolves", "Resolved",
-    ):
-        assert each_keyword in pattern_source, (
-            f"SELF_REFERENCE_PATTERN_TEMPLATE must enumerate `{each_keyword}` "
-            f"to catch every GitHub closing-keyword variant; got: {pattern_source!r}"
-        )
-
-
-def test_body_shape_string_literals_are_named_constants() -> None:
-    """The three PR-body shape names (`trivial`, `standard`, `heavy`) are used
-    by both `_compute_pr_body_shape` (return values) and `validate_pr_body`
-    (string comparisons). Bugbot flagged the inline literals as cross-function
-    magic strings. Pin the canonical names in `hooks_constants/` so a typo in
-    either site cannot silently misclassify shape."""
-    assert constants_module.TRIVIAL_SHAPE == "trivial"
-    assert constants_module.STANDARD_SHAPE == "standard"
-    assert constants_module.HEAVY_SHAPE == "heavy"
-    for each_name in ("TRIVIAL_SHAPE", "STANDARD_SHAPE", "HEAVY_SHAPE"):
-        assert each_name in constants_module.__all__
-
-
-def test_readability_cli_flag_tokens_exposes_four_flags() -> None:
-    """ALL_READABILITY_CLI_FLAG_TOKENS centralises the four CLI flags the
-    dispatcher recognises (loosen/reset/disable/enable). Pinning the set keeps
-    main()'s dispatcher loop and the dispatcher itself in sync; adding a flag
-    requires updating both the constant and the dispatcher in the same commit."""
-    expected_tokens = frozenset(
-        {
-            "--readability-loosen",
-            "--readability-reset",
-            "--readability-disable",
-            "--readability-enable",
-        }
-    )
-    assert constants_module.ALL_READABILITY_CLI_FLAG_TOKENS == expected_tokens
-    assert "ALL_READABILITY_CLI_FLAG_TOKENS" in constants_module.__all__
-
-
-def test_ceremony_header_pattern_is_removed() -> None:
-    """`CEREMONY_HEADER_PATTERN` is replaced by the broader `HEADING_LINE_PATTERN`
-    check in the Trivial-ceremony violation; the constant must not linger as a
-    dead export to drift back into use."""
-    assert not hasattr(constants_module, "CEREMONY_HEADER_PATTERN"), (
-        "CEREMONY_HEADER_PATTERN must be deleted; the ceremony-on-Trivial check "
-        "now uses HEADING_LINE_PATTERN to cover every heading level."
-    )
-    assert "CEREMONY_HEADER_PATTERN" not in constants_module.__all__
-
-
-def test_flesch_formula_coefficients_are_named_constants() -> None:
-    """The Flesch Reading Ease formula coefficients (206.835, 1.015, 84.6) and
-    the perfect-score default (100.0) must live as named UPPER_SNAKE constants
-    in `hooks_constants/pr_description_enforcer_constants.py` so the production
-    function body has zero magic numeric literals."""
-    assert constants_module.FLESCH_BASE_SCORE == 206.835
-    assert constants_module.FLESCH_WORDS_PER_SENTENCE_COEFFICIENT == 1.015
-    assert constants_module.FLESCH_SYLLABLES_PER_WORD_COEFFICIENT == 84.6
-    assert constants_module.FLESCH_PERFECT_SCORE == 100.0
-    for each_name in (
-        "FLESCH_BASE_SCORE",
-        "FLESCH_WORDS_PER_SENTENCE_COEFFICIENT",
-        "FLESCH_SYLLABLES_PER_WORD_COEFFICIENT",
-        "FLESCH_PERFECT_SCORE",
-    ):
-        assert each_name in constants_module.__all__, (
-            f"{each_name} must appear in __all__ so the enforcer can import it."
-        )
-
-
-def test_all_heavy_testing_headers_enumerates_five_canonical_forms() -> None:
-    """The frozenset ALL_HEAVY_TESTING_HEADERS drives the violation message that
-    tells the writer which headers satisfy the heavy-shape testing-category
-    requirement. The set must include all five canonical forms (Test plan,
-    Testing, Tests, Verification, Validation) so the writer's documented
-    vocabulary matches the message it receives on a violation."""
-    expected_canonical_headers = [
-        "## Test plan",
-        "## Testing",
-        "## Tests",
-        "## Validation",
-        "## Verification",
-    ]
-    assert sorted(constants_module.ALL_HEAVY_TESTING_HEADERS) == expected_canonical_headers