Refactor template and CI setup to use PACKAGE_NAME

Updated internal CMake functions to require PACKAGE_NAME as a parameter instead of relying on PROJECT_NAME. This clarifies the preconditions for template and CI workflow generation, ensuring correct substitution and improving maintainability.

Author: sean-parent

cmake/cpp-library-ci.cmake

@@ -4,10 +4,11 @@
 #
 # This module handles GitHub Actions workflow generation with PROJECT_NAME substitution
 
-# Generates GitHub Actions CI workflow from template with PROJECT_NAME substitution.
+# Generates GitHub Actions CI workflow from template with PACKAGE_NAME substitution.
+# - Precondition: PACKAGE_NAME must be set in parent scope
 # - Postcondition: .github/workflows/ci.yml created from template if not present
 # - With force_init: overwrites existing workflow file
-function(_cpp_library_setup_ci force_init)
+function(_cpp_library_setup_ci PACKAGE_NAME force_init)
     set(ci_template "${CPP_LIBRARY_ROOT}/templates/.github/workflows/ci.yml.in")
     set(ci_dest "${CMAKE_CURRENT_SOURCE_DIR}/.github/workflows/ci.yml")
 

cmake/cpp-library-install.cmake

@@ -24,7 +24,7 @@ endfunction()
 # - Precondition: TARGET_NAME specifies existing target with INTERFACE_LINK_LIBRARIES
 # - Postcondition: OUTPUT_VAR contains newline-separated find_dependency() calls for public dependencies
 # - Uses cpp_library_map_dependency() mappings if registered, otherwise uses defaults
-# - Automatically handles cpp-library dependencies (matching NAMESPACE)
+# - Automatically handles cpp-library dependencies (namespace::package → find_dependency(package))
 function(_cpp_library_generate_dependencies OUTPUT_VAR TARGET_NAME NAMESPACE)
     get_target_property(LINK_LIBS ${TARGET_NAME} INTERFACE_LINK_LIBRARIES)
 

cmake/cpp-library-setup.cmake

@@ -111,9 +111,10 @@ function(_cpp_library_setup_core)
 endfunction()
 
 # Copies template files (.clang-format, .gitignore, etc.) to project root if not present.
-# - Postcondition: missing template files copied to project, CI workflow configured with PROJECT_NAME substitution
+# - Precondition: PACKAGE_NAME must be passed as first parameter
+# - Postcondition: missing template files copied to project, CI workflow configured with PACKAGE_NAME substitution
 # - With FORCE_INIT: overwrites existing files
-function(_cpp_library_copy_templates)
+function(_cpp_library_copy_templates PACKAGE_NAME)
     set(options FORCE_INIT)
     cmake_parse_arguments(ARG "${options}" "" "" ${ARGN})
 
@@ -141,6 +142,6 @@ function(_cpp_library_copy_templates)
         endif()
     endforeach()
 
-    # Setup CI workflow with PROJECT_NAME substitution
-    _cpp_library_setup_ci(${ARG_FORCE_INIT})
+    # Setup CI workflow with PACKAGE_NAME substitution
+    _cpp_library_setup_ci("${PACKAGE_NAME}" ${ARG_FORCE_INIT})
 endfunction()

cpp-library.cmake

@@ -211,9 +211,9 @@ function(cpp_library_setup)
 
     # Copy static template files (like .clang-format, .gitignore, CMakePresets.json, etc.)
     if(DEFINED CPP_LIBRARY_FORCE_INIT AND CPP_LIBRARY_FORCE_INIT)
-        _cpp_library_copy_templates(FORCE_INIT)
+        _cpp_library_copy_templates("${PACKAGE_NAME}" FORCE_INIT)
     else()
-        _cpp_library_copy_templates()
+        _cpp_library_copy_templates("${PACKAGE_NAME}")
     endif()
 
     # Setup testing (if tests are specified)