fix: Replace hardcoded GitHub credentials with configurable options in GitHelpers

[thomhurst]

Summary

• Adds GitUserName, GitUserEmail, RepositoryOwner, and RepositoryName configurable properties to GitHubSettings
• Updates GitHelpers.cs to use these settings instead of hardcoded values like "Tom Longhurst" and "thomhurst@users.noreply.github.com"
• Provides sensible defaults (github-actions[bot] for automated commits) for backward compatibility
• Adds configuration options to appsettings.json

Test plan

• Verify the build compiles successfully
• Verify existing functionality is not affected (default values maintain current behavior)
• Test with custom configuration values if desired

Closes #1494

🤖 Generated with Claude Code

[thomhurst]

Summary

Replaces hardcoded GitHub credentials (user name, email, repository owner/name) with configurable options in GitHubSettings.

Critical Issues

None found ✅

Suggestions

None - the implementation is clean and maintains backward compatibility through sensible defaults.

Verdict

✅ APPROVE - No critical issues

The refactoring successfully:

• Removes hardcoded values ("Tom Longhurst", "thomhurst@users.noreply.github.com", "thomhurst", "ModularPipelines")
• Provides configurable alternatives via GitHubSettings
• Maintains backward compatibility with appropriate defaults
• Follows existing DI and configuration patterns
• Properly handles null cases with fallback to new instance

[Copilot]

Pull request overview

This PR successfully refactors hardcoded GitHub credentials and repository information into configurable settings, improving the flexibility and reusability of the ModularPipelines build system. The changes replace hardcoded values like "Tom Longhurst" and "thomhurst@users.noreply.github.com" with configurable properties while maintaining backward compatibility through sensible defaults.

Key changes:

• Adds four new configurable properties to GitHubSettings for git user info and repository details
• Refactors GitHelpers.cs to consistently use settings instead of hardcoded values
• Provides appropriate defaults (github-actions[bot] for automation, existing repo values for continuity)

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
src/ModularPipelines.Build/Settings/GitHubSettings.cs	Adds four new properties with XML documentation and default values for git user credentials and repository identification
src/ModularPipelines.Build/GitHelpers.cs	Refactors all methods to use configurable settings via new GetGitHubSettings helper method instead of hardcoded strings
src/ModularPipelines.Build/appsettings.json	Adds configuration entries for the new settings with appropriate default values matching the code