feat: prune removed Azure-hosted apps from apps.json during sync

Co-authored-by: Copilot <copilot@github.com>

Author: punkouter26

PoPunkouterSoftware.Client/wwwroot/data/apps.json

@@ -78,19 +78,6 @@
       "url": "https://porobotstocks-porobotstocks-web.azurewebsites.net",
       "githubRepo": "punkouter5/PoRobotStocks"
     },
-    {
-      "id": "poappidea",
-      "name": "PoAppIdea",
-      "description": "App idea brainstorming tool",
-      "category": "productivity",
-      "status": "disabled",
-      "technologies": [
-        "Azure App Service",
-        "Blazor"
-      ],
-      "url": "https://poappidea-web.azurewebsites.net",
-      "githubRepo": null
-    },
     {
       "id": "pocouplequiz",
       "name": "PoCoupleQuiz",

PoPunkouterSoftware/Features/Azure/AppsJsonSyncer.cs

@@ -49,13 +49,16 @@ public static async Task SyncAsync(AzureReport report, string appsJsonPath, Canc
 
         var services = report.WebServices?.Services ?? new List<WebService>();
         var discoveredIds = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
+        var discoveredHosts = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
 
         // ── Process every service discovered in Azure ─────────────────────────
         foreach (var svc in services)
         {
             if (string.IsNullOrWhiteSpace(svc.Url)) continue;
             var slug = Slugify(svc.Name);
             discoveredIds.Add(slug);
+            if (TryGetHost(svc.Url, out var discoveredHost))
+                discoveredHosts.Add(discoveredHost);
 
             var techs = InferTechnologies(svc.ResourceType);
 
@@ -103,19 +106,24 @@ public static async Task SyncAsync(AzureReport report, string appsJsonPath, Canc
             }
         }
 
-        // ── Apps in apps.json that no longer exist in Azure ───────────────────
-        // We deliberately do NOT auto-demote apps that aren't found — the user
-        // controls status. Mark removed apps with a warning annotation in the ID.
-        // (Deleted Azure resources stay visible until the user manually removes them.)
-        var skippedUrls = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
-        foreach (var app in apps)
+        // ── Prune Azure-hosted apps that were removed from Azure ───────────────
+        // Only prune when discovery returned at least one service to avoid wiping
+        // apps.json due to transient auth/config failures that produce empty data.
+        if (services.Count > 0)
         {
-            // If this app's URL matches a discovered URL, skip marking
-            if (!string.IsNullOrWhiteSpace(app.Url) && discoveredIds.Contains(app.Id)) continue;
-            // If this app's slug matches a discovered slug, skip
-            if (discoveredIds.Contains(app.Id)) continue;
-            // Otherwise it's a stale entry — we leave it in place but flag it
-            // (no auto-demotion as per intentional design)
+            apps = apps
+                .Where(app =>
+                {
+                    if (!IsAzureHostedUrl(app.Url))
+                        return true;
+
+                    var idMatched = !string.IsNullOrWhiteSpace(app.Id) && discoveredIds.Contains(app.Id);
+                    if (idMatched)
+                        return true;
+
+                    return TryGetHost(app.Url, out var appHost) && discoveredHosts.Contains(appHost);
+                })
+                .ToList();
         }
 
         // ── Sort and write back ───────────────────────────────────────────────
@@ -195,6 +203,27 @@ private static string Slugify(string name)
         return s.Trim('-');
     }
 
+    private static bool TryGetHost(string? url, out string host)
+    {
+        host = string.Empty;
+        if (string.IsNullOrWhiteSpace(url))
+            return false;
+        if (!Uri.TryCreate(url, UriKind.Absolute, out var uri) || string.IsNullOrWhiteSpace(uri.Host))
+            return false;
+        host = uri.Host;
+        return true;
+    }
+
+    private static bool IsAzureHostedUrl(string? url)
+    {
+        if (!TryGetHost(url, out var host))
+            return false;
+
+        return host.EndsWith(".azurewebsites.net", StringComparison.OrdinalIgnoreCase)
+            || host.EndsWith(".azurestaticapps.net", StringComparison.OrdinalIgnoreCase)
+            || host.EndsWith(".azurecontainerapps.io", StringComparison.OrdinalIgnoreCase);
+    }
+
     // ── Local model types (only used for apps.json serialisation) ─────────────
 
     private class AppsWrapper

tests/PoPunkouterSoftware.Tests.Unit/AppsJsonSyncerTests.cs

@@ -261,6 +261,31 @@ public async Task SyncAsync_EmptyReport_PreservesExistingApps()
         doc.RootElement.GetProperty("apps").GetArrayLength().Should().Be(1);
     }
 
+    [Fact]
+    public async Task SyncAsync_RemovedAzureHostedApp_IsPruned_WhenDiscoverySucceeds()
+    {
+        var path = TempPath("apps.json");
+        var initial =
+            "{" +
+            "\"apps\":[" +
+            "{\"id\":\"poappidea\",\"name\":\"PoAppIdea\",\"description\":\"desc\",\"status\":\"inactive\",\"url\":\"https://poappidea-web.azurewebsites.net\",\"technologies\":[\"Azure\"]}," +
+            "{\"id\":\"keep-app\",\"name\":\"KeepApp\",\"description\":\"desc\",\"status\":\"active\",\"url\":\"https://keep-app.azurewebsites.net\",\"technologies\":[\"Azure\"]}" +
+            "]}";
+        await File.WriteAllTextAsync(path, initial);
+
+        var report = BuildReport(("keep-app", "https://keep-app.azurewebsites.net", "microsoft.web/sites"));
+        await AppsJsonSyncer.SyncAsync(report, path);
+
+        var json = await File.ReadAllTextAsync(path);
+        var doc  = JsonDocument.Parse(json);
+        var ids = doc.RootElement.GetProperty("apps")
+            .EnumerateArray()
+            .Select(a => a.GetProperty("id").GetString())
+            .ToList();
+
+        ids.Should().ContainSingle().Which.Should().Be("keep-app");
+    }
+
     [Fact]
     public async Task SyncAsync_MissingAppsJson_CreatesNewFile()
     {