claude-dev-env 1.80.0 → 1.82.0

package/_shared/pr-loop/scripts/tests/test_code_rules_gate.py

@@ -1953,3 +1953,342 @@ def test_main_staged_mode_passes_on_staged_deletion_of_clean_file(
         "a staged deletion has no staged blob; the gate must skip it cleanly "
         "rather than fail closed as if the file were unreadable"
     )
+
+
+def test_run_staged_test_files_passes_for_a_passing_staged_test(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(
+        temporary_git_repository / "test_ok.py",
+        "def test_ok() -> None:\n    assert True\n",
+    )
+    stage_file(temporary_git_repository, "test_ok.py")
+
+    exit_code = gate_module.run_staged_test_files(temporary_git_repository)
+
+    assert exit_code == 0
+
+
+def test_run_staged_test_files_fails_for_a_failing_staged_test(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(
+        temporary_git_repository / "test_bad.py",
+        "def test_bad() -> None:\n    assert False\n",
+    )
+    stage_file(temporary_git_repository, "test_bad.py")
+
+    exit_code = gate_module.run_staged_test_files(temporary_git_repository)
+
+    assert exit_code != 0
+
+
+def test_run_staged_test_files_zero_when_no_test_file_staged(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "helper.py", "first_value = 1\n")
+    stage_file(temporary_git_repository, "helper.py")
+
+    assert gate_module.run_staged_test_files(temporary_git_repository) == 0
+
+
+def test_run_staged_test_files_zero_when_staged_conftest_collects_no_tests(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(
+        temporary_git_repository / "conftest.py",
+        "import pytest\n\n\n@pytest.fixture()\ndef sample_fixture() -> int:\n"
+        "    return 1\n",
+    )
+    stage_file(temporary_git_repository, "conftest.py")
+
+    assert gate_module.run_staged_test_files(temporary_git_repository) == 0
+
+
+def test_hunk_header_pattern_captures_new_start_and_count() -> None:
+    header_match = gate_module.hunk_header_pattern().match("@@ -12,3 +45,6 @@ def scope")
+    assert header_match is not None
+    assert header_match.group(1) == "45"
+    assert header_match.group(2) == "6"
+
+
+def test_hunk_header_pattern_leaves_count_group_none_for_single_line_hunk() -> None:
+    header_match = gate_module.hunk_header_pattern().match("@@ -0,0 +7 @@")
+    assert header_match is not None
+    assert header_match.group(1) == "7"
+    assert header_match.group(2) is None
+
+
+def test_violation_line_pattern_captures_leading_line_number() -> None:
+    violation_match = gate_module.violation_line_pattern().match(
+        "Line 88: banned identifier"
+    )
+    assert violation_match is not None
+    assert violation_match.group(1) == "88"
+
+
+def test_violation_line_pattern_does_not_match_unprefixed_text() -> None:
+    assert gate_module.violation_line_pattern().match("Function foo is too long") is None
+
+
+def test_filter_paths_under_prefixes_keeps_only_nested_paths(tmp_path: Path) -> None:
+    kept_path = tmp_path / "packages" / "core" / "module.py"
+    dropped_path = tmp_path / "docs" / "guide.py"
+    kept_path.parent.mkdir(parents=True)
+    dropped_path.parent.mkdir(parents=True)
+    kept_path.write_text("kept = 1\n", encoding="utf-8")
+    dropped_path.write_text("dropped = 1\n", encoding="utf-8")
+
+    filtered_paths = gate_module.filter_paths_under_prefixes(
+        [kept_path, dropped_path], tmp_path, ["packages/core"]
+    )
+
+    assert filtered_paths == [kept_path]
+
+
+def test_filter_paths_under_prefixes_returns_input_when_no_prefixes(
+    tmp_path: Path,
+) -> None:
+    only_path = tmp_path / "anywhere.py"
+    only_path.write_text("value = 1\n", encoding="utf-8")
+
+    filtered_paths = gate_module.filter_paths_under_prefixes([only_path], tmp_path, [])
+
+    assert filtered_paths == [only_path]
+
+
+def test_is_code_path_accepts_python_and_typescript_suffixes() -> None:
+    assert gate_module.is_code_path(Path("pkg/module.py"))
+    assert gate_module.is_code_path(Path("pkg/component.tsx"))
+
+
+def test_is_code_path_rejects_non_code_suffix() -> None:
+    assert not gate_module.is_code_path(Path("docs/readme.md"))
+
+
+def test_parse_added_line_numbers_collects_every_added_line() -> None:
+    unified_diff_text = (
+        "diff --git a/target.py b/target.py\n"
+        "@@ -1,0 +2,3 @@\n"
+        "+second = 2\n"
+        "+third = 3\n"
+        "+fourth = 4\n"
+    )
+
+    added_line_numbers = gate_module.parse_added_line_numbers(unified_diff_text)
+
+    assert added_line_numbers == {2, 3, 4}
+
+
+def test_parse_added_line_numbers_ignores_zero_count_hunks() -> None:
+    unified_diff_text = "@@ -5,2 +5,0 @@\n"
+
+    assert gate_module.parse_added_line_numbers(unified_diff_text) == set()
+
+
+def test_is_file_new_at_base_true_for_file_added_after_base(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "original.py", "first_count = 1\n")
+    commit_all_files(temporary_git_repository, "baseline")
+    base_sha = run_git_in_repository(
+        temporary_git_repository, "rev-parse", "HEAD"
+    ).strip()
+    write_file(temporary_git_repository / "added.py", "second_count = 2\n")
+    commit_all_files(temporary_git_repository, "add second file")
+
+    assert gate_module.is_file_new_at_base(temporary_git_repository, base_sha, "added.py")
+    assert not gate_module.is_file_new_at_base(
+        temporary_git_repository, base_sha, "original.py"
+    )
+
+
+def test_added_lines_for_file_reports_lines_added_since_base(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "target.py", "first = 1\nsecond = 2\n")
+    commit_all_files(temporary_git_repository, "baseline")
+    base_sha = run_git_in_repository(
+        temporary_git_repository, "rev-parse", "HEAD"
+    ).strip()
+    write_file(
+        temporary_git_repository / "target.py",
+        "first = 1\nsecond = 2\nthird = 3\nfourth = 4\n",
+    )
+    commit_all_files(temporary_git_repository, "append lines")
+
+    added_line_numbers = gate_module.added_lines_for_file(
+        temporary_git_repository, base_sha, "target.py"
+    )
+
+    assert 3 in added_line_numbers
+    assert 4 in added_line_numbers
+    assert 1 not in added_line_numbers
+
+
+def test_extract_violation_line_number_returns_parsed_line() -> None:
+    assert gate_module.extract_violation_line_number("Line 17: magic value") == 17
+
+
+def test_extract_violation_line_number_returns_none_for_unprefixed_text() -> None:
+    assert gate_module.extract_violation_line_number("no line prefix here") is None
+
+
+def test_function_length_span_range_covers_declared_span() -> None:
+    violation_text = "Function 'compute_total' (defined at line 10) is 25 lines - split it"
+
+    span_range = gate_module.function_length_span_range(violation_text)
+
+    assert span_range == range(10, 35)
+
+
+def test_function_length_span_range_returns_none_for_other_message() -> None:
+    assert gate_module.function_length_span_range("Line 3: banned identifier") is None
+
+
+def test_isolation_span_range_covers_enclosing_test_span() -> None:
+    violation_text = (
+        "Line 5: Test 'test_probe' (defined at line 12, spanning 8 lines) probes HOME"
+    )
+
+    span_range = gate_module.isolation_span_range(violation_text)
+
+    assert span_range == range(12, 20)
+
+
+def test_isolation_span_range_returns_none_for_other_message() -> None:
+    assert gate_module.isolation_span_range("Function foo is 5 lines") is None
+
+
+def test_enclosing_span_range_dispatches_to_function_length_extractor() -> None:
+    violation_text = "Function 'render' (defined at line 4) is 6 lines - too long"
+
+    span_range = gate_module.enclosing_span_range(violation_text)
+
+    assert span_range == range(4, 10)
+
+
+def test_enclosing_span_range_returns_none_when_no_span_fragment() -> None:
+    assert gate_module.enclosing_span_range("Line 9: plain violation") is None
+
+
+def test_print_violation_section_writes_header_and_grouped_paths(
+    tmp_path: Path,
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    offending_path = tmp_path / "package" / "module.py"
+    offending_path.parent.mkdir(parents=True)
+    offending_path.write_text("value = 1\n", encoding="utf-8")
+
+    gate_module.print_violation_section(
+        "BLOCKING violations:",
+        {offending_path.resolve(): ["Line 3: magic value", "Line 4: banned name"]},
+        tmp_path,
+    )
+
+    captured = capsys.readouterr()
+    assert "BLOCKING violations:" in captured.err
+    assert "package/module.py" in captured.err.replace("\\", "/")
+    assert "Line 3: magic value" in captured.err
+    assert "Line 4: banned name" in captured.err
+
+
+def test_read_prior_committed_content_returns_head_version(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "tracked.py", "committed = 1\n")
+    commit_all_files(temporary_git_repository, "commit tracked")
+    write_file(temporary_git_repository / "tracked.py", "working_tree_edit = 2\n")
+
+    committed_content = gate_module.read_prior_committed_content(
+        temporary_git_repository, "tracked.py"
+    )
+
+    assert committed_content == "committed = 1\n"
+
+
+def test_read_prior_committed_content_returns_empty_for_untracked_path(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "anchor.py", "anchor = 1\n")
+    commit_all_files(temporary_git_repository, "baseline")
+
+    assert (
+        gate_module.read_prior_committed_content(temporary_git_repository, "missing.py")
+        == ""
+    )
+
+
+def test_read_staged_content_returns_staged_blob(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "staged.py", "staged_value = 1\n")
+    stage_file(temporary_git_repository, "staged.py")
+
+    staged_content = gate_module.read_staged_content(
+        temporary_git_repository, "staged.py"
+    )
+
+    assert staged_content == "staged_value = 1\n"
+
+
+def test_read_staged_content_returns_none_for_unstaged_path(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "anchor.py", "anchor = 1\n")
+    commit_all_files(temporary_git_repository, "baseline")
+
+    assert (
+        gate_module.read_staged_content(temporary_git_repository, "never_staged.py")
+        is None
+    )
+
+
+def test_staged_blob_exists_true_for_staged_file(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "present.py", "present_value = 1\n")
+    stage_file(temporary_git_repository, "present.py")
+
+    assert gate_module.staged_blob_exists(temporary_git_repository, "present.py")
+
+
+def test_staged_blob_exists_false_for_staged_deletion(
+    temporary_git_repository: Path,
+) -> None:
+    write_file(temporary_git_repository / "removable.py", "first_count = 1\n")
+    commit_all_files(temporary_git_repository, "initial")
+    run_git_in_repository(temporary_git_repository, "rm", "--", "removable.py")
+
+    assert not gate_module.staged_blob_exists(temporary_git_repository, "removable.py")
+
+
+def test_parse_arguments_reads_staged_base_and_prefix_flags() -> None:
+    parsed_arguments = gate_module.parse_arguments(
+        [
+            "--staged",
+            "--base",
+            "main",
+            "--repo-root",
+            "some/root",
+            "--only-under",
+            "packages/core",
+            "explicit_file.py",
+        ]
+    )
+
+    assert parsed_arguments.staged is True
+    assert parsed_arguments.base == "main"
+    assert parsed_arguments.repo_root == Path("some/root")
+    assert parsed_arguments.only_under == ["packages/core"]
+    assert parsed_arguments.paths == [Path("explicit_file.py")]
+
+
+def test_parse_arguments_applies_documented_defaults() -> None:
+    parsed_arguments = gate_module.parse_arguments([])
+
+    assert parsed_arguments.staged is False
+    assert parsed_arguments.base == "origin/main"
+    assert parsed_arguments.repo_root is None
+    assert parsed_arguments.only_under == []
+    assert parsed_arguments.paths == []

package/_shared/pr-loop/scripts/tests/test_terminology_sweep.py

@@ -0,0 +1,297 @@
+"""Behavioral tests for the commit-time terminology sweep."""
+
+import importlib.util
+import subprocess
+from pathlib import Path
+from types import ModuleType
+
+
+def _load_sweep_module() -> ModuleType:
+    module_path = Path(__file__).parent.parent / "terminology_sweep.py"
+    spec = importlib.util.spec_from_file_location("terminology_sweep", 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
+
+
+sweep_module = _load_sweep_module()
+sweep_diff = sweep_module.sweep_diff
+staged_terminology_findings = sweep_module.staged_terminology_findings
+main = sweep_module.main
+parse_added_lines = sweep_module._parse_added_lines
+
+
+def _init_git_repository(repository_path: Path) -> None:
+    for each_command in (
+        ["git", "init"],
+        ["git", "config", "user.email", "test@example.com"],
+        ["git", "config", "user.name", "Test"],
+    ):
+        subprocess.run(each_command, cwd=repository_path, check=True, capture_output=True)
+
+
+CODE_AND_PROSE_DIFF = (
+    "diff --git a/api/quota.py b/api/quota.py\n"
+    "--- a/api/quota.py\n"
+    "+++ b/api/quota.py\n"
+    "@@ -0,0 +1,2 @@\n"
+    "+def read_quota(account):\n"
+    '+    return account["premium_interactions"]\n'
+    "diff --git a/docs/README.md b/docs/README.md\n"
+    "--- a/docs/README.md\n"
+    "+++ b/docs/README.md\n"
+    "@@ -0,0 +1,1 @@\n"
+    "+The premium-request budget gates the run.\n"
+)
+
+
+def test_flags_prose_near_miss_of_code_identifier() -> None:
+    findings = sweep_diff(CODE_AND_PROSE_DIFF)
+    assert len(findings) == 1
+    assert "docs/README.md:1" in findings[0]
+    assert "premium-request" in findings[0]
+    assert "premium_interactions" in findings[0]
+
+
+def test_does_not_flag_exact_hyphen_variant() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+premium_interactions = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The premium-interactions budget gates the run.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def _code_and_prose_collision_diff(code_identifier: str, prose_compound: str) -> str:
+    return (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        f"+{code_identifier} = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        f"+This is a {prose_compound} cache.\n"
+    )
+
+
+def test_does_not_flag_english_compound_tail_read_only() -> None:
+    diff = _code_and_prose_collision_diff("read_config", "read-only")
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_english_compound_tail_data_driven() -> None:
+    diff = _code_and_prose_collision_diff("data_source", "data-driven")
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_english_compound_tail_type_safe() -> None:
+    diff = _code_and_prose_collision_diff("type_check", "type-safe")
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_english_compound_tail_test_driven() -> None:
+    diff = _code_and_prose_collision_diff("test_case", "test-driven")
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_english_compound_tail_high_quality() -> None:
+    diff = _code_and_prose_collision_diff("high_level", "high-quality")
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_unrelated_hyphenated_prose() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+premium_interactions = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+A well-known open-source project.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_plural_prose_of_singular_identifier() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+test_file = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+We scan the test files for coverage.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_plural_prose_of_singular_identifier_audit_prompt() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+audit_prompt = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The audit prompts drive each round.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_singular_prose_of_plural_identifier() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+audit_prompts = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The audit prompt drives each round.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_does_not_flag_y_to_ies_plural_variant() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+retry_policy = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The retry policies bound each attempt.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_still_flags_non_plural_divergent_tail() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+premium_interactions = 5\n"
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The premium request budget gates the run.\n"
+    )
+    findings = sweep_diff(diff)
+    assert len(findings) == 1
+    assert "premium request" in findings[0]
+
+
+def test_flags_near_miss_inside_code_comment() -> None:
+    diff = (
+        "diff --git a/api/quota.py b/api/quota.py\n"
+        "--- a/api/quota.py\n"
+        "+++ b/api/quota.py\n"
+        "@@ -0,0 +1,2 @@\n"
+        "+premium_interactions = 5\n"
+        "+# the premium-request path resets the counter\n"
+    )
+    findings = sweep_diff(diff)
+    assert len(findings) == 1
+    assert "premium-request" in findings[0]
+
+
+def test_no_findings_when_no_multiword_identifier_introduced() -> None:
+    diff = (
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+The premium-request budget gates the run.\n"
+    )
+    assert sweep_diff(diff) == []
+
+
+def test_main_exits_one_when_findings(tmp_path: Path) -> None:
+    diff_file = tmp_path / "change.diff"
+    diff_file.write_text(CODE_AND_PROSE_DIFF, encoding="utf-8")
+    exit_code = main(["--diff-file", str(diff_file)])
+    assert exit_code == 1
+
+
+def test_main_exits_zero_when_clean(tmp_path: Path) -> None:
+    diff_file = tmp_path / "clean.diff"
+    diff_file.write_text(
+        "diff --git a/docs/README.md b/docs/README.md\n"
+        "--- a/docs/README.md\n"
+        "+++ b/docs/README.md\n"
+        "@@ -0,0 +1,1 @@\n"
+        "+Nothing to see here.\n",
+        encoding="utf-8",
+    )
+    exit_code = main(["--diff-file", str(diff_file)])
+    assert exit_code == 0
+
+
+def test_staged_terminology_findings_flags_staged_prose(tmp_path: Path) -> None:
+    _init_git_repository(tmp_path)
+    (tmp_path / "quota.py").write_text(
+        "premium_interactions = 5\n", encoding="utf-8"
+    )
+    (tmp_path / "README.md").write_text(
+        "The premium-request budget gates the run.\n", encoding="utf-8"
+    )
+    subprocess.run(["git", "add", "-A"], cwd=tmp_path, check=True, capture_output=True)
+    findings = staged_terminology_findings(tmp_path)
+    assert any("premium-request" in each_finding for each_finding in findings)
+
+
+def test_staged_terminology_findings_empty_when_clean(tmp_path: Path) -> None:
+    _init_git_repository(tmp_path)
+    (tmp_path / "README.md").write_text("Nothing notable here.\n", encoding="utf-8")
+    subprocess.run(["git", "add", "-A"], cwd=tmp_path, check=True, capture_output=True)
+    assert staged_terminology_findings(tmp_path) == []
+
+
+def test_parse_added_lines_counts_pre_increment_content_as_added_line() -> None:
+    diff_with_pre_increment = (
+        "diff --git a/app.mjs b/app.mjs\n"
+        "--- a/app.mjs\n"
+        "+++ b/app.mjs\n"
+        "@@ -0,0 +1,3 @@\n"
+        "+++counter;\n"
+        "+let first_total = read_first_value();\n"
+        "+let second_total = read_second_value();\n"
+    )
+
+    all_added_lines = parse_added_lines(diff_with_pre_increment)
+
+    assert [
+        (each_line_number, each_text) for _, each_line_number, each_text in all_added_lines
+    ] == [
+        (1, "++counter;"),
+        (2, "let first_total = read_first_value();"),
+        (3, "let second_total = read_second_value();"),
+    ]

package/audit-rubrics/CLAUDE.md

@@ -1,6 +1,6 @@
 # audit-rubrics
 
-Audit rubrics for the PR-loop code-review suite. The rubrics define the 16 bug categories (A–P), their sub-bucket decompositions, and the prompt templates agents use during an audit pass. Installed into `~/.claude/audit-rubrics/` by `bin/install.mjs`.
+Audit rubrics for the PR-loop code-review suite. The rubrics define the 17 bug categories (A–Q), their sub-bucket decompositions, and the prompt templates agents use during an audit pass. Installed into `~/.claude/audit-rubrics/` by `bin/install.mjs`.
 
 ## Key file
 
@@ -12,7 +12,7 @@ Audit rubrics for the PR-loop code-review suite. The rubrics define the 16 bug c
 
 | Entry | Description |
 |---|---|
-| `category_rubrics/` | One `.md` per category (A–P): defines what the category audits, example findings, and a sub-bucket decomposition table |
+| `category_rubrics/` | One `.md` per category (A–Q): defines what the category audits, example findings, and a sub-bucket decomposition table |
 | `prompts/` | One `.md` per category: the ready-to-use audit prompt template an agent inlines the artifact into |
 
 ## Categories
@@ -35,6 +35,7 @@ Audit rubrics for the PR-loop code-review suite. The rubrics define the 16 bug c
 | N | Test name / scenario verifier |
 | O | Docstring vs implementation drift |
 | P | Name vs behavior contract |
+| Q | Cross-surface claim consistency |
 
 ## Breaking-change rule
 

package/audit-rubrics/category_rubrics/CLAUDE.md

@@ -1,6 +1,6 @@
 # audit-rubrics/category_rubrics
 
-One rubric file per audit category (A–P). Each file defines what the category covers, gives concrete examples of findings, and provides the sub-bucket decomposition an audit agent uses to structure its pass.
+One rubric file per audit category (A–Q). Each file defines what the category covers, gives concrete examples of findings, and provides the sub-bucket decomposition an audit agent uses to structure its pass.
 
 ## Files
 
@@ -22,6 +22,7 @@ One rubric file per audit category (A–P). Each file defines what the category
 | `category-n-test-name-scenario-verifier.md` | N — Test name / scenario verifier |
 | `category-o-docstring-vs-impl-drift.md` | O — Docstring vs implementation drift |
 | `category-p-name-vs-behavior-contract.md` | P — Name vs behavior contract |
+| `category-q-cross-surface-claims.md` | Q — Cross-surface claim consistency |
 
 ## Rubric structure