Add exception logging to service classes for improved debuggability

[Copilot]

GitHub Issue (If applicable): #145

PR Type

What kind of change does this PR introduce?

• Bugfix

What is the current behavior?

Exceptions thrown in service classes under Chefs.Business.Services are swallowed silently with no console output, making NativeAOT debugging nearly impossible. Users see "An error occurred" with no actionable information in logs.

What is the new behavior?

All public async methods in service classes now log exceptions before rethrowing. Adds ILogger<T> to:

• ShareService
• RecipeService
• UserService
• CookbookService
• NotificationService

Pattern applied:

public async ValueTask<IImmutableList<Recipe>> GetAll(CancellationToken ct)
{
    try
    {
        return await GetAllCore(ct);
    }
    catch (Exception ex)
    {
        logger.LogError(ex, "Failed to get all recipes");
        throw;
    }
}

private async ValueTask<IImmutableList<Recipe>> GetAllCore(CancellationToken ct)
{
    // original implementation
}

Internal method calls use *Core variants directly to avoid redundant logging.

PR Checklist

Please check if your PR fulfills the following requirements:

• Tested code with current supported SDKs
• Docs have been added/updated which fit documentation template. (for bug fixes / features)
• Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
• Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
• Contains NO breaking changes
• Updated the Release Notes
• Associated with an issue (GitHub or internal)

Other information

No breaking changes—loggers are injected via existing DI infrastructure.

Internal Issue (If applicable):

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• crl.godaddy.com
  • Triggering command: /usr/bin/dotnet dotnet build Chefs/Chefs.csproj -p:TargetFrameworkOverride=net9.0-desktop (dns block)
  • Triggering command: /usr/bin/dotnet dotnet restore --no-dependencies /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/2B9D6FF9D398549B5FBB8065A02F803E/missingpackages_workingdir --packages /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/missingpackages /p:DisableImplicitNuGetFallbackFolder=true --verbosity normal --configfile /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/nugetconfig/nuget.config --force (dns block)
• ocsp.godaddy.com
  • Triggering command: /usr/bin/dotnet dotnet build Chefs/Chefs.csproj -p:TargetFrameworkOverride=net9.0-desktop (dns block)
  • Triggering command: /usr/bin/dotnet dotnet restore --no-dependencies /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/2B9D6FF9D398549B5FBB8065A02F803E/missingpackages_workingdir --packages /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/missingpackages /p:DisableImplicitNuGetFallbackFolder=true --verbosity normal --configfile /home/REDACTED/work/uno.chefs/.codeql-scratch/dbs/csharp/working/nugetconfig/nuget.config --force (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Exceptions can be swallowed with no actionable output</issue_title>
<issue_description>(All the interesting bugs are under NativeAOT!)

Build and run Uno.Chefs for macOS desktop under NativeAOT, which in turn requires using Uno.Sdk 6.6.0:

sed -i '' 's/"Uno.Sdk": ".*"/"Uno.Sdk": "6.6.0-dev.3"/g' global.json
dotnet publish -c Release -r osx-x64 -f net10.0-desktop -p:TargetFrameworkOverride=net10.0-desktop \
  -bl Chefs/Chefs.csproj -p:UseSkiaRendering=true -p:SkiaPublishAot=true \
  -p:SkiaPublishAot=true \
  -p:IlcGenerateMapFile=true -p:IlcGenerateMstatFile=true -p:IlcGenerateDgmlFile=true -p:IlcGenerateMetadataLog=true \
  -p:EmitCompilerGeneratedFiles=true -p:CompilerGeneratedFilesOutputPath=`pwd`/_gen
Chefs/bin/Release/net10.0-desktop/osx-x64/publish/Chefs

From the launch screen:

1. Click Skip
2. Click  Sign in with Apple
3. Click 🔍Search on the left
4. In the Search box, type m

Current behavior

The search results screen shows:

An error occurred.

The console log shows nothing.

The lack of any information about the error that occurred makes it difficult to fix the bug.

Desired behavior

The console log should show the exception that was thrown.

Implementation

For all classes underneath the Chefs.Business.Services namespace, including:

• Chefs/Business/Services/Sharing/ShareService.cs
• Chefs/Business/Services/Recipes/RecipeService.cs
• Chefs/Business/Services/Users/UserService.cs
• Chefs/Business/Services/Cookbooks/CookbookService.cs
• Chefs/Business/Services/Notifications/NotificationService.cs

Perform the following changes:

1. Add an ILogger<T> logger constructor parameter, in which T is the declaring type:

   diff --git a/Chefs/Business/Services/Sharing/ShareService.cs b/Chefs/Business/Services/Sharing/ShareService.cs
   index ec38f458..0e5a6e07 100644
   --- a/Chefs/Business/Services/Sharing/ShareService.cs
   +++ b/Chefs/Business/Services/Sharing/ShareService.cs
   @@ -6,7 +6,7 @@ using WinRT.Interop;
    
    namespace Chefs.Business.Services.Sharing;
    
   -public class ShareService() : IShareService
   +public class ShareService(ILogger<ShareService> logger) : IShareService
    {
           private Recipe? _recipe;
           private IImmutableList<Step>? _steps;

2. For each public method which returns Task, Task<T>, ValueTask, or ValueTask<T>, rename the method by appending a Core suffix, and change the visibility to private:

   diff --git a/Chefs/Business/Services/Sharing/ShareService.cs b/Chefs/Business/Services/Sharing/ShareService.cs
   index ec38f458..fb82cef4 100644
   --- a/Chefs/Business/Services/Sharing/ShareService.cs
   +++ b/Chefs/Business/Services/Sharing/ShareService.cs
   @@ -16,7 +16,7 @@ public class ShareService() : IShareService
           static IDataTransferManagerInterop DataTransferManagerInterop => DataTransferManager.As<IDataTransferManagerInterop>();
    #endif
    
   -       public async Task ShareRecipe(Recipe recipe, IImmutableList<Step> steps, CancellationToken ct)
   +       private async Task ShareRecipeCore(Recipe recipe, IImmutableList<Step> steps, CancellationToken ct)
           {
                   _recipe = recipe;
                   _steps = steps;

3. For each method changed in (2), write a method with the original method declaration which invokes the Core suffixed method, wrapping the invocation in a try/catch block. The catch block should write the exception to the logger parameter, and re-throw the exception:

   	public async Task ShareRecipe(Recipe recipe, IImmutableList<Step> steps, CancellationToken ct)
   	{
   		try
   		{
   			await ShareRecipeCore(recipe, steps, ct);
   		}
   		catch (Exception ex)
   		{
   			logger.LogError(ex, "Failed to share recipe");
   			throw;
   		}
   	}

For example, the complete diff to ShareService is:

diff --git a/Chefs/Business/Services/Sharing/ShareService.cs b/Chefs/Business/Services/Sharing/ShareService.cs
index ec38f458..7149430e 100644
--- a/Chefs/Business/Services/Sharing/ShareService.cs
+++ b/Chefs/Business/Services/Sharing/ShareService.cs
@@ -6,7 +6,7 @@ using WinRT.Interop;
 
 namespace Chefs.Business.Services.Sharing;
 
-public class ShareService() : IShareService
+public class ShareService(ILogger<ShareService> logger) : IShareService
 {
 	private Recipe? _recipe;
 	private IImmutableList<Step>? _steps;
@@ -17,6 +17,19 @@ public class ShareService() : IShareService
 #endif
 
 	public async Task ShareRecipe(Recipe recipe, IImmutableList<Step> steps, CancellationToken ct)
+	{
+		try
+		{
+			await ShareRecipeCore(recipe, steps, ct);
+		}
+		catch (Ex...

</details>



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

- Fixes unoplatform/uno.chefs#1726

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

💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey).

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}