Rename TimeZoneTimeProvider to RequestTimeProvider for better clarity and consistency (#32)

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.

<!-- START COPILOT CODING AGENT SUFFIX -->



<details>

<summary>Original prompt</summary>

> 
> ----
> 
> *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)
> 
> <comments>
> </comments>
> 


</details>
Fixes #31

<!-- START COPILOT CODING AGENT TIPS -->
---

💡 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](https://gh.io/copilot-coding-agent-tips) in the docs.

Author: marcominerva

src/PdfSmith.BusinessLayer/Services/TimeZoneTimeProvider.cs renamed to src/PdfSmith.BusinessLayer/Services/RequestTimeProvider.cs

@@ -2,7 +2,7 @@
 
 namespace PdfSmith.BusinessLayer.Services;
 
-public class TimeZoneTimeProvider(ITimeZoneService timeZoneService) : TimeProvider
+public class RequestTimeProvider(ITimeZoneService timeZoneService) : TimeProvider
 {
     public override TimeZoneInfo LocalTimeZone => timeZoneService.GetTimeZone() ?? TimeZoneInfo.Utc;
 }

src/PdfSmith.BusinessLayer/Templating/HandlebarsTemplateEngine.cs

@@ -6,9 +6,9 @@
 
 namespace PdfSmith.BusinessLayer.Templating;
 
-public class HandlebarsTemplateEngine(TimeZoneTimeProvider timeZoneTimeProvider) : ITemplateEngine
+public class HandlebarsTemplateEngine(RequestTimeProvider requestTimeProvider) : ITemplateEngine
 {
-    private readonly Lazy<IHandlebars> handlebarsInstance = new(() => CreateHandlebarsInstance(timeZoneTimeProvider));
+    private readonly Lazy<IHandlebars> handlebarsInstance = new(() => CreateHandlebarsInstance(requestTimeProvider));
 
     public Task<string> RenderAsync(string template, object? model, CultureInfo culture, CancellationToken cancellationToken = default)
     {
@@ -32,7 +32,7 @@ public Task<string> RenderAsync(string template, object? model, CultureInfo cult
         }
     }
 
-    private static IHandlebars CreateHandlebarsInstance(TimeZoneTimeProvider timeZoneTimeProvider)
+    private static IHandlebars CreateHandlebarsInstance(RequestTimeProvider requestTimeProvider)
     {
         var handlebars = Handlebars.Create();
 
@@ -92,7 +92,7 @@ private static IHandlebars CreateHandlebarsInstance(TimeZoneTimeProvider timeZon
             var format = arguments.ElementAtOrDefault(0)?.ToString() ??
                 $"{CultureInfo.CurrentCulture.DateTimeFormat.ShortDatePattern} {CultureInfo.CurrentCulture.DateTimeFormat.LongTimePattern}";
 
-            var now = timeZoneTimeProvider.GetLocalNow().DateTime;
+            var now = requestTimeProvider.GetLocalNow().DateTime;
             return now.ToString(format, CultureInfo.CurrentCulture);
         });
 
@@ -101,7 +101,7 @@ private static IHandlebars CreateHandlebarsInstance(TimeZoneTimeProvider timeZon
             var format = arguments.ElementAtOrDefault(0)?.ToString() ??
                 $"{CultureInfo.CurrentCulture.DateTimeFormat.ShortDatePattern} {CultureInfo.CurrentCulture.DateTimeFormat.LongTimePattern}";
 
-            var now = timeZoneTimeProvider.GetUtcNow().DateTime;
+            var now = requestTimeProvider.GetUtcNow().DateTime;
             return now.ToString(format, CultureInfo.CurrentCulture);
         });
 

src/PdfSmith.BusinessLayer/Templating/RazorTemplateEngine.cs

@@ -13,14 +13,14 @@ public async Task<string> RenderAsync(string template, object? model, CultureInf
     {
         try
         {
-            var sanitizedTemplate = DateTimeNowRegex.Replace(template, "@timeZoneTimeProvider.GetLocalNow().DateTime");
-            sanitizedTemplate = DateTimeOffsetNowRegex.Replace(sanitizedTemplate, "@timeZoneTimeProvider.GetLocalNow()");
+            var sanitizedTemplate = DateTimeNowRegex.Replace(template, "@requestTimeProvider.GetLocalNow().DateTime");
+            sanitizedTemplate = DateTimeOffsetNowRegex.Replace(sanitizedTemplate, "@requestTimeProvider.GetLocalNow()");
 
             var content = $"""
                 @using System
                 @using System.Collections.Generic
                 @using System.Linq
-                @inject PdfSmith.BusinessLayer.Services.TimeZoneTimeProvider timeZoneTimeProvider
+                @inject PdfSmith.BusinessLayer.Services.RequestTimeProvider requestTimeProvider
                 {sanitizedTemplate}
                 """;
 

src/PdfSmith.BusinessLayer/Templating/ScribanTemplateEngine.cs

@@ -8,7 +8,7 @@
 
 namespace PdfSmith.BusinessLayer.Templating;
 
-public partial class ScribanTemplateEngine(TimeZoneTimeProvider timeZoneTimeProvider) : ITemplateEngine
+public partial class ScribanTemplateEngine(RequestTimeProvider requestTimeProvider) : ITemplateEngine
 {
     private const string DateTimeZonePlaceholder = "datetime_withzone";
 
@@ -27,7 +27,7 @@ public async Task<string> RenderAsync(string text, object? model, CultureInfo cu
         context.PushCulture(culture);
 
         var dateWithTimeZoneScript = new ScriptObject();
-        dateWithTimeZoneScript.Import(DateTimeZonePlaceholder, new Func<DateTime>(() => timeZoneTimeProvider.GetLocalNow().DateTime));
+        dateWithTimeZoneScript.Import(DateTimeZonePlaceholder, new Func<DateTime>(() => requestTimeProvider.GetLocalNow().DateTime));
         context.PushGlobal(dateWithTimeZoneScript);
 
         var result = await template.RenderAsync(context);

src/PdfSmith/Program.cs

@@ -37,7 +37,7 @@
 builder.Services.AddHttpContextAccessor();
 
 builder.Services.AddSingleton(TimeProvider.System);
-builder.Services.AddSingleton<TimeZoneTimeProvider>();
+builder.Services.AddSingleton<RequestTimeProvider>();
 
 builder.Services.AddSingleton<ITimeZoneService, TimeZoneService>();