#99 Rename get_source_includes hook to get_class_cpp_source_includes

[kwabenantim]

Follow up to #99.

Align the custom-generator include hook with its siblings (get_class_cpp_pre_code, get_class_cpp_def_code): it is class-scoped and targets the .cppwg.cpp include block, so the name now carries both the class scope and cpp file tokens..

[Copilot]

Pull request overview

This PR renames the custom-generator hook used to contribute per-class .cpp wrapper includes from get_source_includes to get_class_cpp_source_includes, aligning it with the other class-scoped hooks (get_class_cpp_pre_code, get_class_cpp_def_code).

Changes:

• Renames the include hook in the Custom base class and in the class writer call site.
• Updates unit tests to use the new hook name.
• Updates custom-generator documentation to reflect the new hook name.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/test_utils.py	Updates hook name used in call_generator_hook test.
tests/test_class_writer.py	Renames hook implementations and test docstrings to the new hook name.
doc/custom-generators.md	Documents the new hook name and updates example code.
cppwg/writers/class_writer.py	Calls the new hook when emitting generator-provided includes.
cppwg/templates/custom.py	Renames the base hook method in the Custom template class.

Comments suppressed due to low confidence (1)

cppwg/templates/custom.py:32

• Custom now defines only get_class_cpp_source_includes(). Generators that subclass Custom but still override the older get_source_includes() hook will no longer have their includes picked up (since the base implementation returns []). Consider keeping a deprecated get_source_includes() method and having the new hook delegate to it by default, so older subclasses keep working.

    def get_class_cpp_source_includes(self, *args, **kwargs) -> list:
        """
        Return headers to add to the #include block of the class wrapper.

        Use this for headers the generated code needs but that cppwg cannot

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.