claude-dev-env 8.12.0 → 8.13.0

package/scripts/tests/test_sync_to_cursor.py

@@ -1,315 +0,0 @@
-"""Tests for sync_to_cursor.py: canonical docs copy, manifest, and truncation footer."""
-
-from __future__ import annotations
-
-import json
-import sys
-from pathlib import Path
-
-import pytest
-
-_SCRIPTS_DIR = Path(__file__).resolve().parent.parent
-if str(_SCRIPTS_DIR) not in sys.path:
-    sys.path.insert(0, str(_SCRIPTS_DIR))
-
-import sync_to_cursor as mod
-from sync_to_cursor.engine import run as run_sync_to_cursor
-from sync_to_cursor.rules import (
-    _code_standards_section_order,
-    _parse_h2_sections,
-    _read_paths_glob,
-    _test_quality_section_order,
-    build_mappings,
-    strip_leading_yaml_frontmatter,
-)
-
-_CLAUDE_DEV_ENV_DIR = _SCRIPTS_DIR.parent
-_REAL_TESTING_RULE = _CLAUDE_DEV_ENV_DIR / "rules" / "testing.md"
-_REAL_CODE_RULES_DOC = _CLAUDE_DEV_ENV_DIR / "docs" / "CODE_RULES.md"
-_REAL_TEST_QUALITY_DOC = _CLAUDE_DEV_ENV_DIR / "docs" / "TEST_QUALITY.md"
-
-
-def _minimal_rule_files(claude_rules: Path) -> None:
-    claude_rules.mkdir(parents=True, exist_ok=True)
-    (claude_rules / "code-standards.md").write_text(
-        "# Code standards stub\n", encoding="utf-8"
-    )
-    (claude_rules / "tasklings-preferences.md").write_text(
-        "---\npaths:\n  - \"Y:/x/**\"\n---\n\n# Tasklings\n",
-        encoding="utf-8",
-    )
-    (claude_rules / "bdd.md").write_text("# BDD\n", encoding="utf-8")
-    (claude_rules / "testing.md").write_text(
-        "---\npaths:\n  - \"**/test_*.py\"\n---\n\n# Testing\n",
-        encoding="utf-8",
-    )
-    (claude_rules / "research-mode.md").write_text("# RM\n", encoding="utf-8")
-    (claude_rules / "explore-thoroughly.md").write_text("# ET\n", encoding="utf-8")
-
-
-def _minimal_code_rules_and_test_quality(claude_docs: Path) -> tuple[bytes, bytes]:
-    claude_docs.mkdir(parents=True, exist_ok=True)
-    cr = (
-        "\n\n".join(f"## {title}\n\nalpha" for title in _code_standards_section_order) + "\n"
-    ).encode("utf-8")
-    tq = (
-        "\n\n".join(f"## {title}\n\nbeta" for title in _test_quality_section_order) + "\n"
-    ).encode("utf-8")
-    (claude_docs / "CODE_RULES.md").write_bytes(cr)
-    (claude_docs / "TEST_QUALITY.md").write_bytes(tq)
-    return cr, tq
-
-
-def test_limit_lines_footer_mentions_cursor_docs() -> None:
-    long_body = "\n".join(f"line {index}" for index in range(mod.MAX_RULE_BODY_LINES + 5))
-    out = mod._limit_lines(long_body, mod.MAX_RULE_BODY_LINES)
-    assert ".cursor/docs" in out
-    assert "synced reference" in out
-    assert "~/.claude/system-prompts/software-engineer.xml" in out
-
-
-def test_sync_canonical_docs_copies_byte_identical(tmp_path: Path) -> None:
-    claude = tmp_path / ".claude"
-    cursor = tmp_path / ".cursor"
-    docs = claude / "docs"
-    cr, tq = _minimal_code_rules_and_test_quality(docs)
-    mod._sync_canonical_docs(claude, cursor, dry_run=False, quiet=True)
-    assert (cursor / "docs" / "CODE_RULES.md").read_bytes() == cr
-    assert (cursor / "docs" / "TEST_QUALITY.md").read_bytes() == tq
-
-
-def test_sync_canonical_docs_skips_missing_with_no_dst(tmp_path: Path) -> None:
-    claude = tmp_path / ".claude"
-    cursor = tmp_path / ".cursor"
-    (claude / "docs").mkdir(parents=True)
-    (claude / "docs" / "CODE_RULES.md").write_text("x", encoding="utf-8")
-    mod._sync_canonical_docs(claude, cursor, dry_run=False, quiet=True)
-    assert (cursor / "docs" / "CODE_RULES.md").read_text(encoding="utf-8") == "x"
-    assert not (cursor / "docs" / "TEST_QUALITY.md").is_file()
-
-
-def test_check_fails_when_doc_source_changes_without_resync(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    claude = tmp_path / ".claude"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    monkeypatch.setenv("LLM_SETTINGS_ROOT", str(tmp_path))
-    assert run_sync_to_cursor(["--force"]) == 0
-    (claude / "docs" / "CODE_RULES.md").write_bytes(b"changed\n")
-    assert run_sync_to_cursor(["--check"]) != 0
-
-
-def test_check_passes_after_resync(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    claude = tmp_path / ".claude"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    monkeypatch.setenv("LLM_SETTINGS_ROOT", str(tmp_path))
-    assert run_sync_to_cursor(["--force"]) == 0
-    assert run_sync_to_cursor(["--check"]) == 0
-
-
-def test_manifest_includes_docs_entries(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    claude = tmp_path / ".claude"
-    cursor = tmp_path / ".cursor"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    monkeypatch.setenv("LLM_SETTINGS_ROOT", str(tmp_path))
-    assert run_sync_to_cursor(["--force"]) == 0
-    manifest = json.loads((cursor / ".sync-manifest.json").read_text(encoding="utf-8"))
-    assert "docs_entries" in manifest
-    docs_entries = manifest["docs_entries"]
-    assert "docs/CODE_RULES.md" in docs_entries
-    assert "docs/TEST_QUALITY.md" in docs_entries
-    assert "sources_hash" in docs_entries["docs/CODE_RULES.md"]
-    assert "output_hash" in docs_entries["docs/CODE_RULES.md"]
-
-
-def test_merge_code_standards_with_pointer_style_code_rules(tmp_path: Path) -> None:
-    rules_directory = tmp_path / "rules"
-    docs_directory = tmp_path / "docs"
-    rules_directory.mkdir(parents=True, exist_ok=True)
-    docs_directory.mkdir(parents=True, exist_ok=True)
-    (rules_directory / "code-standards.md").write_text(
-        "# Code standards stub\n\n- Use full words\n", encoding="utf-8"
-    )
-    (docs_directory / "CODE_RULES.md").write_text(
-        "# CODE_RULES pointer: canonical code-quality policy lives in"
-        " `~/.claude/system-prompts/software-engineer.xml` under `<code_quality>`.\n",
-        encoding="utf-8",
-    )
-    merged = mod.merge_code_standards(
-        (rules_directory / "code-standards.md", docs_directory / "CODE_RULES.md")
-    )
-    assert merged.strip(), "output must not be empty for pointer-style CODE_RULES.md"
-    assert "software-engineer.xml" in merged
-    assert "code_quality" in merged
-
-
-def test_sync_canonical_docs_deletes_stale_copy_when_source_removed(tmp_path: Path) -> None:
-    claude = tmp_path / ".claude"
-    cursor = tmp_path / ".cursor"
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    mod._sync_canonical_docs(claude, cursor, dry_run=False, quiet=True)
-    assert (cursor / "docs" / "CODE_RULES.md").is_file()
-    (claude / "docs" / "CODE_RULES.md").unlink()
-    mod._sync_canonical_docs(claude, cursor, dry_run=False, quiet=True)
-    assert not (cursor / "docs" / "CODE_RULES.md").is_file()
-
-
-def test_dry_run_does_not_create_output_directory(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    claude = tmp_path / ".claude"
-    cursor = tmp_path / ".cursor"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    monkeypatch.setenv("LLM_SETTINGS_ROOT", str(tmp_path))
-    assert run_sync_to_cursor(["--dry-run", "--quiet"]) == 0
-    assert not (cursor / "rules").exists(), "--dry-run must not create the output directory"
-
-
-def test_check_skips_optional_mapping_when_source_missing(
-    tmp_path: Path, monkeypatch: pytest.MonkeyPatch
-) -> None:
-    claude = tmp_path / ".claude"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    monkeypatch.setenv("LLM_SETTINGS_ROOT", str(tmp_path))
-    assert run_sync_to_cursor(["--force"]) == 0
-    (claude / "rules" / "tasklings-preferences.md").unlink()
-    assert run_sync_to_cursor(["--check"]) == 0, "--check must pass when only optional sources are missing"
-
-
-def test_test_quality_glob_derived_from_testing_frontmatter() -> None:
-    expected_glob = _read_paths_glob(_REAL_TESTING_RULE)
-    assert expected_glob, "testing.md must declare a non-empty paths frontmatter"
-    pinned_glob = "**/test_*.py,**/*_test.py,**/*.test.*,**/*.spec.*,**/conftest.py,**/tests/**"
-    assert expected_glob == pinned_glob
-    test_quality_mapping = next(
-        mapping for mapping in build_mappings(_CLAUDE_DEV_ENV_DIR) if mapping.key == "test-quality"
-    )
-    assert test_quality_mapping.globs == expected_glob
-    assert test_quality_mapping.globs == pinned_glob
-
-
-def test_tasklings_glob_derived_from_frontmatter(tmp_path: Path) -> None:
-    rules_directory = tmp_path / "rules"
-    rules_directory.mkdir(parents=True)
-    (rules_directory / "tasklings-preferences.md").write_text(
-        '---\npaths:\n  - "Y:/MyProject/**"\n  - "Z:/Other/**"\n---\n\n# Tasklings\n',
-        encoding="utf-8",
-    )
-    glob_value = _read_paths_glob(rules_directory / "tasklings-preferences.md")
-    assert glob_value == "Y:/MyProject/**,Z:/Other/**"
-
-
-def test_merge_reference_headers_point_at_cursor_docs(tmp_path: Path) -> None:
-    rules_directory = tmp_path / "rules"
-    docs_directory = tmp_path / "docs"
-    rules_directory.mkdir(parents=True, exist_ok=True)
-    docs_directory.mkdir(parents=True, exist_ok=True)
-    (rules_directory / "code-standards.md").write_text("# CS\n", encoding="utf-8")
-    cr_body = "\n\n".join(f"## {t}\n\nbody" for t in _code_standards_section_order)
-    (docs_directory / "CODE_RULES.md").write_text(cr_body, encoding="utf-8")
-    merged = mod.merge_code_standards(
-        (rules_directory / "code-standards.md", docs_directory / "CODE_RULES.md")
-    )
-    assert ".cursor/docs/CODE_RULES.md" in merged
-
-    (rules_directory / "testing.md").write_text("# T\n", encoding="utf-8")
-    tq_body = "\n\n".join(f"## {t}\n\nbody" for t in _test_quality_section_order)
-    (docs_directory / "TEST_QUALITY.md").write_text(tq_body, encoding="utf-8")
-    merged_tq = mod.merge_test_quality(
-        (rules_directory / "testing.md", docs_directory / "TEST_QUALITY.md")
-    )
-    assert ".cursor/docs/TEST_QUALITY.md" in merged_tq
-
-
-def test_code_standards_section_order_titles_all_exist_in_real_doc() -> None:
-    real_headings = set(_parse_h2_sections(_REAL_CODE_RULES_DOC.read_text(encoding="utf-8")))
-    missing_titles = [title for title in _code_standards_section_order if title not in real_headings]
-    assert missing_titles == [], (
-        f"_code_standards_section_order titles absent from {_REAL_CODE_RULES_DOC.name}: "
-        f"{missing_titles}"
-    )
-
-
-def test_test_quality_section_order_titles_all_exist_in_real_doc() -> None:
-    real_headings = set(_parse_h2_sections(_REAL_TEST_QUALITY_DOC.read_text(encoding="utf-8")))
-    missing_titles = [title for title in _test_quality_section_order if title not in real_headings]
-    assert missing_titles == [], (
-        f"_test_quality_section_order titles absent from {_REAL_TEST_QUALITY_DOC.name}: "
-        f"{missing_titles}"
-    )
-
-
-def test_merge_code_standards_raises_when_expected_section_absent(tmp_path: Path) -> None:
-    rules_directory = tmp_path / "rules"
-    docs_directory = tmp_path / "docs"
-    rules_directory.mkdir(parents=True, exist_ok=True)
-    docs_directory.mkdir(parents=True, exist_ok=True)
-    (rules_directory / "code-standards.md").write_text("# CS\n", encoding="utf-8")
-    present_titles = [
-        title for title in _code_standards_section_order if title != "5. NO ABBREVIATIONS"
-    ]
-    cr_body = "\n\n".join(f"## {title}\n\nbody" for title in present_titles)
-    (docs_directory / "CODE_RULES.md").write_text(cr_body, encoding="utf-8")
-    with pytest.raises(AssertionError, match="5. NO ABBREVIATIONS"):
-        mod.merge_code_standards(
-            (rules_directory / "code-standards.md", docs_directory / "CODE_RULES.md")
-        )
-
-
-def test_build_mappings_raises_when_testing_lacks_paths_frontmatter(tmp_path: Path) -> None:
-    claude = tmp_path / ".claude"
-    _minimal_rule_files(claude / "rules")
-    _minimal_code_rules_and_test_quality(claude / "docs")
-    (claude / "rules" / "testing.md").write_text("# Testing\n", encoding="utf-8")
-    with pytest.raises(AssertionError, match="testing.md"):
-        build_mappings(claude)
-
-
-def test_merge_test_quality_raises_when_expected_section_absent(tmp_path: Path) -> None:
-    rules_directory = tmp_path / "rules"
-    docs_directory = tmp_path / "docs"
-    rules_directory.mkdir(parents=True, exist_ok=True)
-    docs_directory.mkdir(parents=True, exist_ok=True)
-    (rules_directory / "testing.md").write_text("# T\n", encoding="utf-8")
-    present_titles = [
-        title for title in _test_quality_section_order if title != "Core Testing Principles"
-    ]
-    tq_body = "\n\n".join(f"## {title}\n\nbody" for title in present_titles)
-    (docs_directory / "TEST_QUALITY.md").write_text(tq_body, encoding="utf-8")
-    with pytest.raises(AssertionError, match="Core Testing Principles"):
-        mod.merge_test_quality(
-            (rules_directory / "testing.md", docs_directory / "TEST_QUALITY.md")
-        )
-
-
-def test_merge_test_quality_strips_leading_paths_frontmatter(tmp_path: Path) -> None:
-    docs_directory = tmp_path / "docs"
-    docs_directory.mkdir(parents=True, exist_ok=True)
-    test_quality_body = "\n\n".join(
-        f"## {title}\n\nbody" for title in _test_quality_section_order
-    )
-    (docs_directory / "TEST_QUALITY.md").write_text(test_quality_body, encoding="utf-8")
-    merged = mod.merge_test_quality(
-        (_REAL_TESTING_RULE, docs_directory / "TEST_QUALITY.md")
-    )
-    testing_text = _REAL_TESTING_RULE.read_text(encoding="utf-8")
-    first_content_line = next(
-        line for line in strip_leading_yaml_frontmatter(testing_text).splitlines()
-        if line.strip()
-    )
-    merged_lines = merged.splitlines()
-    assert merged_lines[0] != "---", "merged body must not open with a leaked frontmatter fence"
-    assert merged_lines[0] == first_content_line
-    body_before_reference = "\n".join(
-        merged_lines[: merged_lines.index("## Reference (full text: `.cursor/docs/TEST_QUALITY.md`)")]
-    )
-    assert "paths:" not in body_before_reference, "testing.md paths frontmatter leaked into rule body"