claude-dev-env 2.28.0 → 2.28.1

package/hooks/blocking/test_code_rules_enforcer_docstring_args_span_scope.py

@@ -1,425 +0,0 @@
-"""Span-scope tests for docstring checks.
-
-Covers two surfaces:
-
-1. ``check_docstring_args_single_line_scope_vs_span`` (Category O6) — an Args
-   entry that claims single-line (block-anchor) scope while the body builds a
-   multi-line span and routes it through span intersection.
-
-2. Changed-span grading for ``check_docstring_runon_sentence`` and
-   ``check_docstring_prose_wall_without_illustration`` (issue #237) — far-away
-   no-op grandfathering, introduced-violation blocking, and
-   ``defer_scope_to_caller`` pass-through.
-"""
-
-from __future__ import annotations
-
-import importlib.util
-from pathlib import Path
-from types import ModuleType
-
-
-def _load_enforcer_module() -> ModuleType:
-    module_path = Path(__file__).parent / "code_rules_enforcer.py"
-    spec = importlib.util.spec_from_file_location("code_rules_enforcer", module_path)
-    assert spec is not None
-    assert spec.loader is not None
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-code_rules_enforcer = _load_enforcer_module()
-
-
-def check_docstring_args_single_line_scope_vs_span(content: str, file_path: str) -> list[str]:
-    return code_rules_enforcer.check_docstring_args_single_line_scope_vs_span(content, file_path)
-
-
-PRODUCTION_FILE_PATH = "/project/scripts/check_import_block_sorted.py"
-TEST_FILE_PATH = "/project/scripts/test_check_import_block_sorted.py"
-HOOK_INFRASTRUCTURE_PATH = "/home/user/.claude/hooks/blocking/example.py"
-
-
-SPAN_SCOPING_BODY = (
-    "    span_range = range(line_number, block_end_line_number + 1)\n"
-    "    all_violations.append((span_range, message))\n"
-    "    return _scope_violations_to_changed_lines(\n"
-    "        all_violations, all_changed_lines, defer_scope_to_caller\n"
-    "    )\n"
-)
-
-
-def test_flags_anchor_line_scope_against_span_intersection_body() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block scoped to the changed lines.\n'
-        "\n"
-        "    A finding is returned when any line in that block span is among\n"
-        "    all_changed_lines.\n"
-        "\n"
-        "    Args:\n"
-        "        content: The full file content the write would leave on disk.\n"
-        "        file_path: The destination path used to gate by extension.\n"
-        "        all_changed_lines: Post-edit line numbers the current edit touched, or\n"
-        "            None to treat the whole file as in scope. When provided, a finding\n"
-        "            blocks only when its block-anchor line is among the changed lines.\n"
-        "\n"
-        "    Returns:\n"
-        "        One issue string per detected finding.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    issues = check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH)
-    assert len(issues) == 1
-    assert "check_import_block_sorted" in issues[0]
-    assert "all_changed_lines" in issues[0]
-
-
-def test_flags_single_line_scope_against_stepped_range_span_body() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block scoped to the changed lines.\n'
-        "\n"
-        "    Args:\n"
-        "        content: The full file content the write would leave on disk.\n"
-        "        file_path: The destination path used to gate by extension.\n"
-        "        all_changed_lines: Post-edit line numbers, or None. When provided,\n"
-        "            a finding blocks only when its block-anchor line is among the\n"
-        "            changed lines.\n"
-        "\n"
-        "    Returns:\n"
-        "        One issue string per detected finding.\n"
-        '    """\n'
-        "    span_range = range(line_number, block_end_line_number + 1, 1)\n"
-        "    all_violations.append((span_range, message))\n"
-        "    return _scope_violations_to_changed_lines(\n"
-        "        all_violations, all_changed_lines, defer_scope_to_caller\n"
-        "    )\n"
-    )
-    issues = check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH)
-    assert len(issues) == 1
-    assert "check_import_block_sorted" in issues[0]
-    assert "all_changed_lines" in issues[0]
-
-
-def test_flags_the_line_is_among_phrasing() -> None:
-    content = (
-        "def check_block(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted block scoped to the diff.\n'
-        "\n"
-        "    Args:\n"
-        "        content: The file content.\n"
-        "        all_changed_lines: When provided, a finding blocks only when the\n"
-        "            anchor line is among the changed lines.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    assert len(check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH)) == 1
-
-
-def test_passes_when_args_says_any_line_of_the_span() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block scoped to the changed lines.\n'
-        "\n"
-        "    Args:\n"
-        "        content: The full file content.\n"
-        "        all_changed_lines: Post-edit line numbers the current edit touched, or\n"
-        "            None to treat the whole file as in scope. When provided, a finding\n"
-        "            blocks only when any line of its block span is among the changed lines.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    assert check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH) == []
-
-
-def test_passes_when_body_scopes_by_single_line_not_a_span() -> None:
-    content = (
-        "def check_one_line(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag a violation on a single line scoped to the diff.\n'
-        "\n"
-        "    Args:\n"
-        "        content: The file content.\n"
-        "        all_changed_lines: When provided, a finding blocks only when its\n"
-        "            anchor line is among the changed lines.\n"
-        '    """\n'
-        "    if all_changed_lines is not None and line_number not in all_changed_lines:\n"
-        "        return []\n"
-        "    return [message]\n"
-    )
-    assert check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH) == []
-
-
-def test_passes_when_no_single_line_scope_phrase() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block scoped to the changed lines.\n'
-        "\n"
-        "    Args:\n"
-        "        content: The full file content.\n"
-        "        all_changed_lines: Post-edit line numbers the current edit touched.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    assert check_docstring_args_single_line_scope_vs_span(content, PRODUCTION_FILE_PATH) == []
-
-
-def test_test_files_are_exempt() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block.\n'
-        "\n"
-        "    Args:\n"
-        "        all_changed_lines: A finding blocks only when its anchor line is\n"
-        "            among the changed lines.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    assert check_docstring_args_single_line_scope_vs_span(content, TEST_FILE_PATH) == []
-
-
-def test_hook_infrastructure_is_not_exempt() -> None:
-    content = (
-        "def check_import_block_sorted(\n"
-        "    content: str, file_path: str, all_changed_lines: set[int] | None\n"
-        ") -> list[str]:\n"
-        '    """Flag an unsorted import block.\n'
-        "\n"
-        "    Args:\n"
-        "        all_changed_lines: A finding blocks only when its block-anchor line is\n"
-        "            among the changed lines.\n"
-        '    """\n' + SPAN_SCOPING_BODY
-    )
-    assert (
-        len(check_docstring_args_single_line_scope_vs_span(content, HOOK_INFRASTRUCTURE_PATH)) == 1
-    )
-
-# ---------------------------------------------------------------------------
-# Changed-span grading for the two whole-file docstring checks (issue #237 / P-41)
-# check_docstring_runon_sentence and check_docstring_prose_wall_without_illustration
-# ---------------------------------------------------------------------------
-
-
-def check_docstring_runon_sentence(
-    content: str,
-    file_path: str,
-    all_changed_lines: set[int] | None = None,
-    defer_scope_to_caller: bool = False,
-) -> list[str]:
-    return code_rules_enforcer.check_docstring_runon_sentence(
-        content, file_path, all_changed_lines, defer_scope_to_caller
-    )
-
-
-def check_docstring_prose_wall_without_illustration(
-    content: str,
-    file_path: str,
-    all_changed_lines: set[int] | None = None,
-    defer_scope_to_caller: bool = False,
-) -> list[str]:
-    return code_rules_enforcer.check_docstring_prose_wall_without_illustration(
-        content, file_path, all_changed_lines, defer_scope_to_caller
-    )
-
-
-_CLEAN_HELPER_TAIL = (
-    "\n"
-    "def clean_helper() -> str:\n"
-    '    """Return a short status token for the board.\n'
-    "\n"
-    "    Each call names one vessel and its final port.\n"
-    '    """\n'
-    '    return "ok"\n'
-)
-
-
-def _module_with_far_away_runon_and_clean_helper() -> str:
-    return (
-        '"""Owns the SIGINT install/restore/installability check, the atexit terminal-record\n'
-        "registration, and the interrupted-run finalizer — the non-promoter-specific\n"
-        "machinery that brackets a run so the JSONL artifact always carries a terminal\n"
-        "record and an in-flight theme record on interrupt.\n"
-        '"""\n'
-        + _CLEAN_HELPER_TAIL
-    )
-
-
-def _module_with_far_away_prose_wall_and_clean_helper() -> str:
-    return (
-        '"""Assemble the nightly voyage tally from the harbor scans.\n'
-        "\n"
-        "A scan names one vessel and where it dropped anchor.\n"
-        "The tally walks the scans in arrival order and keeps that order.\n"
-        "A calm voyage ends well for every vessel it carried.\n"
-        "A halted voyage marks the vessel it was near when the storm arrived.\n"
-        "A wrecked voyage marks the vessel that sank and stops the walk there.\n"
-        "The tally groups the vessels by their final port for the harbor.\n"
-        "The harbor reads the tally and sees every arrival at a glance.\n"
-        '"""\n'
-        + _CLEAN_HELPER_TAIL
-    )
-
-
-def test_runon_far_away_no_op_is_grandfathered() -> None:
-    content = _module_with_far_away_runon_and_clean_helper()
-    clean_helper_line = content.splitlines().index("def clean_helper() -> str:") + 1
-    all_changed_lines = set(range(clean_helper_line, clean_helper_line + 6))
-    unscoped = check_docstring_runon_sentence(content, PRODUCTION_FILE_PATH)
-    assert any("run-on" in each for each in unscoped), (
-        f"control: unscoped must still see the far-away run-on, got {unscoped!r}"
-    )
-    scoped = check_docstring_runon_sentence(
-        content, PRODUCTION_FILE_PATH, all_changed_lines=all_changed_lines
-    )
-    assert scoped == [], (
-        f"far-away run-on must be grandfathered on a clean-helper edit, got {scoped!r}"
-    )
-
-
-def test_runon_introduced_violation_blocks() -> None:
-    content = _module_with_far_away_runon_and_clean_helper()
-    unscoped = check_docstring_runon_sentence(content, PRODUCTION_FILE_PATH)
-    assert any("run-on" in each for each in unscoped)
-    scoped = check_docstring_runon_sentence(
-        content, PRODUCTION_FILE_PATH, all_changed_lines={1}
-    )
-    assert any("run-on" in each for each in scoped), (
-        f"introduced/touched run-on must block, got {scoped!r}"
-    )
-
-
-def test_prose_wall_far_away_no_op_is_grandfathered() -> None:
-    content = _module_with_far_away_prose_wall_and_clean_helper()
-    clean_helper_line = content.splitlines().index("def clean_helper() -> str:") + 1
-    all_changed_lines = set(range(clean_helper_line, clean_helper_line + 6))
-    unscoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH
-    )
-    assert any("worked example" in each for each in unscoped), (
-        f"control: unscoped must still see the far-away wall, got {unscoped!r}"
-    )
-    scoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH, all_changed_lines=all_changed_lines
-    )
-    assert scoped == [], (
-        f"far-away wall must be grandfathered on a clean-helper edit, got {scoped!r}"
-    )
-
-
-def test_prose_wall_introduced_violation_blocks() -> None:
-    content = _module_with_far_away_prose_wall_and_clean_helper()
-    unscoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH
-    )
-    assert any("worked example" in each for each in unscoped)
-    scoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH, all_changed_lines={1}
-    )
-    assert any("worked example" in each for each in scoped), (
-        f"introduced/touched wall must block, got {scoped!r}"
-    )
-
-
-def test_runon_defer_scope_returns_all_violations() -> None:
-    content = _module_with_far_away_runon_and_clean_helper()
-    deferred = check_docstring_runon_sentence(
-        content,
-        PRODUCTION_FILE_PATH,
-        all_changed_lines={999},
-        defer_scope_to_caller=True,
-    )
-    assert any("run-on" in each for each in deferred)
-
-
-def _function_with_runon_docstring() -> str:
-    return (
-        "def clean_helper() -> str:\n"
-        '    """Owns the SIGINT install/restore/installability check, the atexit terminal-record\n'
-        "    registration, and the interrupted-run finalizer — the non-promoter-specific\n"
-        "    machinery that brackets a run so the JSONL artifact always carries a terminal\n"
-        "    record and an in-flight theme record on interrupt.\n"
-        '    """\n'
-        '    return "ok"\n'
-    )
-
-
-def test_runon_docstring_body_edit_blocks_without_def_line() -> None:
-    content = _function_with_runon_docstring()
-    unscoped = check_docstring_runon_sentence(content, PRODUCTION_FILE_PATH)
-    assert any("run-on" in each for each in unscoped), (
-        f"control: unscoped must see the function run-on, got {unscoped!r}"
-    )
-    docstring_body_lines = {2, 3, 4, 5}
-    scoped = check_docstring_runon_sentence(
-        content, PRODUCTION_FILE_PATH, all_changed_lines=docstring_body_lines
-    )
-    assert any("run-on" in each for each in scoped), (
-        f"docstring-body edit must block even when the def line is untouched, got {scoped!r}"
-    )
-
-
-def test_runon_post_docstring_body_edit_is_grandfathered() -> None:
-    content = _function_with_runon_docstring()
-    return_line = content.splitlines().index('    return "ok"') + 1
-    scoped = check_docstring_runon_sentence(
-        content, PRODUCTION_FILE_PATH, all_changed_lines={return_line}
-    )
-    assert scoped == [], (
-        f"post-docstring body edit must grandfather the pre-existing run-on, got {scoped!r}"
-    )
-
-
-def _function_with_prose_wall_docstring() -> str:
-    return (
-        "def clean_helper() -> str:\n"
-        '    """Assemble the nightly voyage tally from the harbor scans.\n'
-        "\n"
-        "    A scan names one vessel and where it dropped anchor.\n"
-        "    The tally walks the scans in arrival order and keeps that order.\n"
-        "    A calm voyage ends well for every vessel it carried.\n"
-        "    A halted voyage marks the vessel it was near when the storm arrived.\n"
-        "    A wrecked voyage marks the vessel that sank and stops the walk there.\n"
-        "    The tally groups the vessels by their final port for the harbor.\n"
-        "    The harbor reads the tally and sees every arrival at a glance.\n"
-        '    """\n'
-        '    return "ok"\n'
-    )
-
-
-def test_prose_wall_docstring_body_edit_blocks_without_def_line() -> None:
-    content = _function_with_prose_wall_docstring()
-    unscoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH
-    )
-    assert any("worked example" in each for each in unscoped), (
-        f"control: unscoped must see the function wall, got {unscoped!r}"
-    )
-    docstring_body_lines = set(range(2, 12))
-    scoped = check_docstring_prose_wall_without_illustration(
-        content, PRODUCTION_FILE_PATH, all_changed_lines=docstring_body_lines
-    )
-    assert any("worked example" in each for each in scoped), (
-        f"docstring-body edit must block even when the def line is untouched, got {scoped!r}"
-    )
-
-
-def test_prose_wall_defer_scope_returns_all_violations() -> None:
-    content = _module_with_far_away_prose_wall_and_clean_helper()
-    deferred = check_docstring_prose_wall_without_illustration(
-        content,
-        PRODUCTION_FILE_PATH,
-        all_changed_lines={999},
-        defer_scope_to_caller=True,
-    )
-    assert any("worked example" in each for each in deferred)
-

package/hooks/blocking/test_code_rules_enforcer_docstring_cardinal_family.py

@@ -1,176 +0,0 @@
-"""Tests for check_docstring_cardinal_count_matches_constant_family — O6 drift.
-
-A docstring states a cardinal count of an outcome family ("Covers the four
-outcome branches: ...") and enumerates the family members in prose, while the
-module references more members of that constant family than the count names. The
-prose under-describes the code: a reader trusts "four" and the list to be the
-full set, but the module imports and exercises a fifth outcome. This is the
-deterministic cardinal-count slice of Category O6 docstring-prose-vs-
-implementation drift, the shape that appears when a producer adds an outcome
-branch but leaves the test-module summary at the old count. The check runs on
-test files because the drift class lives in a test-module docstring.
-"""
-
-from __future__ import annotations
-
-import importlib.util
-from pathlib import Path
-from types import ModuleType
-
-
-def _load_enforcer_module() -> ModuleType:
-    module_path = Path(__file__).parent / "code_rules_enforcer.py"
-    spec = importlib.util.spec_from_file_location("code_rules_enforcer", module_path)
-    assert spec is not None
-    assert spec.loader is not None
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-code_rules_enforcer = _load_enforcer_module()
-
-
-def check_docstring_cardinal_count_matches_constant_family(
-    content: str, file_path: str
-) -> list[str]:
-    return code_rules_enforcer.check_docstring_cardinal_count_matches_constant_family(
-        content, file_path
-    )
-
-
-def validate_content(content: str, file_path: str, old_content: str) -> list[str]:
-    return code_rules_enforcer.validate_content(content, file_path, old_content)
-
-
-PRODUCTION_FILE_PATH = "/project/src/dropdown_fix_rule.py"
-TEST_FILE_PATH = "/project/src/tests/test_dropdown_fix_rule.py"
-
-
-def _test_module(summary_line: str, enumeration_line: str) -> str:
-    return (
-        '"""Behavioral tests for classify_dropdown_fix.\n'
-        "\n"
-        f"{summary_line}\n"
-        f"{enumeration_line}\n"
-        '"""\n'
-        "\n"
-        "from stp_dropdown_color_fix.config.status_codes import (\n"
-        "    OUTCOME_ALREADY_OK,\n"
-        "    OUTCOME_OFFENDER_LOW_CONTRAST,\n"
-        "    OUTCOME_OFFENDER_MISSING,\n"
-        "    OUTCOME_OFFENDER_UNREADABLE,\n"
-        "    OUTCOME_SKIPPED_NO_SAFE_SOURCE,\n"
-        ")\n"
-        "\n"
-        "def test_skipped() -> None:\n"
-        "    assert classify() == OUTCOME_SKIPPED_NO_SAFE_SOURCE\n"
-        "\n"
-        "def test_missing() -> None:\n"
-        "    assert classify() == OUTCOME_OFFENDER_MISSING\n"
-        "\n"
-        "def test_low_contrast() -> None:\n"
-        "    assert classify() == OUTCOME_OFFENDER_LOW_CONTRAST\n"
-        "\n"
-        "def test_already_ok() -> None:\n"
-        "    assert classify() == OUTCOME_ALREADY_OK\n"
-        "\n"
-        "def test_unreadable() -> None:\n"
-        "    assert classify() == OUTCOME_OFFENDER_UNREADABLE\n"
-    )
-
-
-def _drifted_module() -> str:
-    return _test_module(
-        "Covers the four outcome branches: skipped_no_safe_source,",
-        "offender_missing (case A), offender_low_contrast (case B), already_ok.",
-    )
-
-
-def _matching_module() -> str:
-    return _test_module(
-        "Covers the five outcome branches: skipped_no_safe_source, offender_missing,",
-        "offender_low_contrast, already_ok, offender_unreadable.",
-    )
-
-
-def test_should_flag_test_module_omitting_an_outcome() -> None:
-    issues = check_docstring_cardinal_count_matches_constant_family(
-        _drifted_module(), TEST_FILE_PATH
-    )
-    assert any("offender_unreadable" in each for each in issues), (
-        "A test-module docstring claiming four outcome branches while the module "
-        f"references five OUTCOME_ constants must be flagged, got: {issues!r}"
-    )
-
-
-def test_should_report_category_o6_in_the_message() -> None:
-    issues = check_docstring_cardinal_count_matches_constant_family(
-        _drifted_module(), TEST_FILE_PATH
-    )
-    assert any("O6" in each for each in issues), (
-        f"Expected the Category O6 label in the message, got: {issues!r}"
-    )
-
-
-def test_should_also_flag_production_module() -> None:
-    issues = check_docstring_cardinal_count_matches_constant_family(
-        _drifted_module(), PRODUCTION_FILE_PATH
-    )
-    assert issues != [], (
-        f"The cardinal drift must be flagged on a production file too, got: {issues!r}"
-    )
-
-
-def test_should_not_flag_when_count_and_enumeration_match_family() -> None:
-    issues = check_docstring_cardinal_count_matches_constant_family(
-        _matching_module(), TEST_FILE_PATH
-    )
-    assert issues == [], (
-        f"A docstring naming all five referenced outcomes must not be flagged, got: {issues!r}"
-    )
-
-
-def test_should_not_flag_without_a_cardinal_phrase() -> None:
-    source = _test_module(
-        "Covers these outcome branches: skipped_no_safe_source,",
-        "offender_missing, offender_low_contrast, already_ok.",
-    )
-    issues = check_docstring_cardinal_count_matches_constant_family(source, TEST_FILE_PATH)
-    assert issues == [], (
-        f"An enumeration with no cardinal count word must not bind, got: {issues!r}"
-    )
-
-
-def test_should_not_flag_single_passing_mention() -> None:
-    source = _test_module(
-        "Asserts the four guard paths reach already_ok when the dropdown is",
-        "absent and the dialer keeps its painted swatch.",
-    )
-    issues = check_docstring_cardinal_count_matches_constant_family(source, TEST_FILE_PATH)
-    assert issues == [], f"A single overlapping token must not bind the family, got: {issues!r}"
-
-
-def test_should_not_flag_when_family_has_one_referenced_member() -> None:
-    source = (
-        '"""Covers the four outcome branches: already_ok, more, words, here."""\n'
-        "from status_codes import OUTCOME_ALREADY_OK\n"
-        "\n"
-        "def test_only() -> None:\n"
-        "    assert classify() == OUTCOME_ALREADY_OK\n"
-    )
-    issues = check_docstring_cardinal_count_matches_constant_family(source, TEST_FILE_PATH)
-    assert issues == [], f"A family with one referenced member must not bind, got: {issues!r}"
-
-
-def test_should_handle_syntax_error_gracefully() -> None:
-    issues = check_docstring_cardinal_count_matches_constant_family("def fetch(\n", TEST_FILE_PATH)
-    assert issues == [], f"Syntax error must yield no issues, got: {issues!r}"
-
-
-def test_validate_content_surfaces_cardinal_family_drift() -> None:
-    issues = validate_content(_drifted_module(), TEST_FILE_PATH, old_content="")
-    matching_issues = [each for each in issues if "offender_unreadable" in each and "O6" in each]
-    assert matching_issues, (
-        f"Expected validate_content to surface the O6 cardinal-family drift, got: {issues!r}"
-    )