Rename TimeZoneTimeProvider to RequestTimeProvider for better clarity and consistency

[Copilot]

This PR renames the TimeZoneTimeProvider class to RequestTimeProvider to improve code readability and better reflect its purpose. The class is primarily used to provide timezone-aware time information based on HTTP request context, making "RequestTimeProvider" a more descriptive and intuitive name.

Changes Made

• Renamed class: TimeZoneTimeProvider → RequestTimeProvider
• Renamed file: TimeZoneTimeProvider.cs → RequestTimeProvider.cs
• Updated dependency injection: Modified Program.cs to register the renamed class
• Updated constructor parameters: Updated all template engines (HandlebarsTemplateEngine, ScribanTemplateEngine, RazorTemplateEngine) to use the new class name
• Updated variable names: Changed all instances of timeZoneTimeProvider to requestTimeProvider throughout the codebase for consistency
• Updated Razor template injection: Modified the @inject directive in RazorTemplateEngine to reference the new class and variable names

Benefits

• Improved clarity: The new name better describes the class's role in providing request-scoped time information
• Better consistency: Variable names now follow camelCase convention with the new class name
• Enhanced maintainability: More descriptive naming makes the codebase easier to understand for new developers

All existing functionality remains unchanged - this is purely a refactoring change to improve code quality and readability.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Rename TimeZoneTimeProvider class to RequestTimeProvider and update related variables</issue_title>
<issue_description>The TimeZoneTimeProvider class should be renamed to RequestTimeProvider for better clarity and consistency. Additionally, any variables of this type (e.g., timeZoneTimeProvider, tztm, or similar) should be updated to reflect the new class name throughout the codebase.

Tasks:

• Rename the TimeZoneTimeProvider class to RequestTimeProvider.
• Update all variable names that reference this type to follow the new naming (e.g., requestTimeProvider).
• Ensure all references in documentation and comments are also updated.
• Run all tests to confirm there are no regressions after the rename.

This refactor will help improve code readability and maintainability.</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #31

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull Request Overview

This PR refactors the codebase by renaming TimeZoneTimeProvider to RequestTimeProvider to better reflect its purpose of providing timezone-aware time information based on HTTP request context. The change improves code readability and maintains consistency throughout the application.

• Renamed the core time provider class and updated all references
• Updated dependency injection registration in the main application setup
• Synchronized variable naming conventions across all template engines

Reviewed Changes

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

Show a summary per file

File	Description
src/PdfSmith/Program.cs	Updated DI registration to use the new RequestTimeProvider class name
src/PdfSmith.BusinessLayer/Services/RequestTimeProvider.cs	Renamed the main class from TimeZoneTimeProvider to RequestTimeProvider
src/PdfSmith.BusinessLayer/Templating/ScribanTemplateEngine.cs	Updated constructor parameter and usage to use requestTimeProvider variable naming
src/PdfSmith.BusinessLayer/Templating/RazorTemplateEngine.cs	Updated @Inject directive and template replacement patterns to use new class and variable names
src/PdfSmith.BusinessLayer/Templating/HandlebarsTemplateEngine.cs	Updated constructor parameter and all internal references to use requestTimeProvider naming