Function _check_cpp_hpp_files

ci/lint_cpp/test_unity_build.py:311–347

Check that all .cpp.hpp files are referenced in their directory's _build.cpp.hpp. Returns: list of violation strings

(src_dir: Path, cpp_hpp_by_dir: CppHppByDir)

def _check_cpp_hpp_files(src_dir: Path, cpp_hpp_by_dir: CppHppByDir) -> list[str]:
    """
    Check that all .cpp.hpp files are referenced in their directory's _build.cpp.hpp.

    Returns:
        list of violation strings
    """
    violations: list[str] = []

    for dir_path, cpp_hpp_files in sorted(cpp_hpp_by_dir.items()):
        # Skip the build directory itself — build files aren't tracked by _build.cpp.hpp
        build_dir = src_dir / "fl" / "build"
        if dir_path == build_dir or str(dir_path).startswith(str(build_dir)):
            continue

        build_hpp = dir_path / BUILD_HPP

        if not build_hpp.exists():
            rel_dir = dir_path.relative_to(PROJECT_ROOT)
            violations.append(f"Missing {BUILD_HPP} in {rel_dir.as_posix()}/")
            continue

        content = build_hpp.read_text(encoding="utf-8")

        for cpp_hpp in sorted(cpp_hpp_files):
            # Skip the _build.cpp.hpp file itself (shouldn't include itself)
            if cpp_hpp.name == BUILD_HPP:
                continue

            rel_path = cpp_hpp.relative_to(src_dir)
            include_path = rel_path.as_posix()

            if include_path not in content:
                rel_build = build_hpp.relative_to(PROJECT_ROOT)
                violations.append(f"{rel_build.as_posix()}: missing {include_path}")

    return violations


SECTION_COMMENT_CURRENT_DIR = "// begin current directory includes"