feat: enhance dry run diff view to show friendly names (resolves #51)

src/Engine.cs

@@ -340,7 +340,7 @@ public async Task PrintDiffAsync(Configuration config)
                 _logger.LogError("\n[-] Items to Remove:");
                 foreach (var item in itemsToRemove)
                 {
-                    _logger.LogError($"  - {item}");
+                    _logger.LogError($"  - {FormatFriendlyName(item)}");
                 }
             }
 
@@ -349,7 +349,7 @@ public async Task PrintDiffAsync(Configuration config)
                 _logger.LogSuccess("\n[+] Items to Add:");
                 foreach (var item in itemsToAdd)
                 {
-                    _logger.LogSuccess($"  + {item}");
+                    _logger.LogSuccess($"  + {FormatFriendlyName(item)}");
                 }
             }
 
@@ -358,11 +358,39 @@ public async Task PrintDiffAsync(Configuration config)
                 _logger.LogInfo("\n[=] Unchanged Items:");
                 foreach (var item in unchangedItems)
                 {
-                    _logger.LogInfo($"  = {item}");
+                    _logger.LogInfo($"  = {FormatFriendlyName(item)}");
                 }
             }
         }
 
+        private string FormatFriendlyName(string item)
+        {
+            if (item.StartsWith("reg:"))
+            {
+                var parts = item.Substring(4).Split('|', 2);
+                if (parts.Length == 2)
+                {
+                    string path = parts[0];
+                    string name = parts[1];
+                    string? settingKey = _systemSettings.GetFriendlyName(path, name);
+                    if (settingKey != null)
+                    {
+                        return $"System Setting: {settingKey}";
+                    }
+                    return $"Registry Tweak: {path} -> {name}";
+                }
+            }
+            else
+            {
+                var parts = item.Split(':', 2);
+                if (parts.Length == 2)
+                {
+                    return $"App ({parts[0]}): {parts[1]}";
+                }
+            }
+            return item;
+        }
+
         private async Task<HashSet<string>> BuildStateFromConfig(Configuration config)
         {
             var state = new HashSet<string>();

src/Interfaces/ISystemSettingsService.cs

@@ -9,5 +9,6 @@ public interface ISystemSettingsService
         Task<IEnumerable<RegistryTweak>> GetTweaksAsync(Dictionary<string, object> settings);
         Task ApplyNonRegistrySettingsAsync(Dictionary<string, object> settings, bool dryRun);
         Task<Dictionary<string, object>> GetCapturedSettingsAsync();
+        string? GetFriendlyName(string registryPath, string registryName);
     }
 }

src/Services/System/SystemSettingsService.cs

@@ -179,6 +179,15 @@ public async Task<Dictionary<string, object>> GetCapturedSettingsAsync()
             });
         }
 
+        public string? GetFriendlyName(string registryPath, string registryName)
+        {
+            var match = _catalog.FirstOrDefault(d =>
+                d.RegistryPath.Equals(registryPath, StringComparison.OrdinalIgnoreCase) &&
+                d.RegistryName.Equals(registryName, StringComparison.OrdinalIgnoreCase));
+
+            return match?.SettingKey;
+        }
+
         public Task ApplyNonRegistrySettingsAsync(Dictionary<string, object> settings, bool dryRun)
         {
             if (settings == null) return Task.CompletedTask;

tests/WinHome.Tests/EngineTest.cs

@@ -162,11 +162,11 @@ public async Task PrintDiffAsync_ShouldPrintCorrectDiff()
 
             // Assert
             mockLogger.Verify(l => l.LogError(It.Is<string>(s => s.Contains("Items to Remove"))), Times.Once);
-            mockLogger.Verify(l => l.LogError(It.Is<string>(s => s.Contains("winget:OldApp"))), Times.Once);
+            mockLogger.Verify(l => l.LogError(It.Is<string>(s => s.Contains("App (winget): OldApp"))), Times.Once);
             mockLogger.Verify(l => l.LogSuccess(It.Is<string>(s => s.Contains("Items to Add"))), Times.Once);
-            mockLogger.Verify(l => l.LogSuccess(It.Is<string>(s => s.Contains("winget:NewApp"))), Times.Once);
+            mockLogger.Verify(l => l.LogSuccess(It.Is<string>(s => s.Contains("App (winget): NewApp"))), Times.Once);
             mockLogger.Verify(l => l.LogInfo(It.Is<string>(s => s.Contains("Unchanged Items"))), Times.Once);
-            mockLogger.Verify(l => l.LogInfo(It.Is<string>(s => s.Contains("winget:UnchangedApp"))), Times.Once);
+            mockLogger.Verify(l => l.LogInfo(It.Is<string>(s => s.Contains("App (winget): UnchangedApp"))), Times.Once);
         }
     }
 }