Derive domain status from subscription status and add backfill

Reviewed-on: http://gitea.lan:3000/robbond/pleskSaas/pulls/2

components/dashboard/dashboard-controls.tsx

@@ -49,6 +49,26 @@ type Props = {
   >;
 };
 
+function getDomainStatusBadge(status: string | null) {
+  const normalized = (status ?? "unknown").toLowerCase();
+
+  if (
+    normalized.includes("active") ||
+    normalized.includes("ok") ||
+    normalized.includes("enabled")
+  ) {
+    return "bg-emerald-100 text-emerald-700";
+  }
+
+  if (normalized.includes("suspended") || normalized.includes("disabled")) {
+    return "bg-rose-100 text-rose-700";
+  }
+
+  if (normalized.includes("inactive")) return "bg-amber-100 text-amber-700";
+
+  return "bg-slate-100 text-slate-600";
+}
+
 export function DashboardControls({
   instances,
   subscriptions,
@@ -351,14 +371,28 @@ export function DashboardControls({
                         <span className="font-medium">{instance.status}</span>
                       </p>
                       <p className="text-xs text-slate-500">
-                        Last sync: {formatDateTime(instance.last_sync_at)}
-                        {instance.last_sync_status
-                          ? ` (${instance.last_sync_status})`
-                          : ""}
+                        Subscriptions: {instance.last_sync_subscriptions ?? 0}
                       </p>
                       <p className="text-xs text-slate-500">
-                        Last counts: {instance.last_sync_subscriptions ?? 0}{" "}
-                        subscriptions, {instance.last_sync_domains ?? 0} domains
+                        Domains: {instance.last_sync_domains ?? 0}
+                      </p>
+                      <p className="text-xs text-slate-500">
+                        Last sync status:{" "}
+                        <span
+                          className={`inline-flex rounded px-2 py-0.5 text-[11px] font-medium ${
+                            instance.last_sync_status === "success"
+                              ? "bg-emerald-100 text-emerald-700"
+                              : instance.last_sync_status === "failed" ||
+                                  instance.last_sync_status === "error"
+                                ? "bg-rose-100 text-rose-700"
+                                : "bg-slate-100 text-slate-600"
+                          }`}
+                        >
+                          {instance.last_sync_status ?? "unknown"}
+                        </span>
+                      </p>
+                      <p className="text-xs text-slate-500">
+                        Last sync time: {formatDateTime(instance.last_sync_at)}
                       </p>
                       <p className="text-xs text-slate-500">
                         Last connected:{" "}
@@ -521,11 +555,15 @@ export function DashboardControls({
                 ) : (
                   filteredDomains.map((domain) => (
                     <tr key={domain.id} className="border-b border-slate-100">
-                      <td className="px-2 py-2 font-medium text-slate-900">
+                      <td className="font-medium text-slate-900">
                         {domain.domain_name}
                       </td>
                       <td className="px-2 py-2">
+                        <span
+                          className={`inline-flex rounded px-2 py-0.5 text-xs font-medium ${getDomainStatusBadge(domain.status)}`}
+                        >
                           {domain.status ?? "unknown"}
+                        </span>
                       </td>
                       <td className="px-2 py-2">
                         {domain.hosting_type ?? "unknown"}

lib/plesk/sync.ts

@@ -113,12 +113,44 @@ export async function syncPleskInstance(
       if (error) throw new Error(error.message);
     }
 
+    const { data: localSubscriptions, error: localSubscriptionsError } =
+      await supabase
+        .from("plesk_subscriptions")
+        .select("id, plesk_subscription_id, status")
+        .eq("agency_id", instance.agency_id)
+        .eq("plesk_instance_id", instance.id);
+
+    if (localSubscriptionsError) {
+      throw new Error(localSubscriptionsError.message);
+    }
+
+    const subscriptionByExternalId = new Map<
+      string,
+      { id: string; status: string | null }
+    >(
+      (localSubscriptions ?? [])
+        .filter((row: any) => row?.plesk_subscription_id && row?.id)
+        .map((row: any) => [
+          String(row.plesk_subscription_id),
+          {
+            id: String(row.id),
+            status: row?.status ? String(row.status) : null,
+          },
+        ]),
+    );
+
     const domainsForUpsert = domainsRaw
       .map((item: any) => {
         const domainId = item?.id ?? item?.guid ?? item?.name;
         const domainName = item?.name ?? item?.domain_name ?? item?.domain;
-        const externalSubscriptionId =
+        const subExternalId =
           item?.subscription?.id ?? item?.subscription_id ?? item?.webspace_id;
+        const externalSubscriptionId = subExternalId
+          ? String(subExternalId)
+          : null;
+        const localSubscription = externalSubscriptionId
+          ? subscriptionByExternalId.get(externalSubscriptionId)
+          : undefined;
         const rawCreated = item?.created_at ?? item?.created;
         const sourceCreatedAt = rawCreated
           ? new Date(String(rawCreated)).toISOString()
@@ -135,18 +167,12 @@ export async function syncPleskInstance(
         return {
           agency_id: instance.agency_id,
           plesk_instance_id: instance.id,
-          subscription_id: null,
-          external_subscription_id: externalSubscriptionId
-            ? String(externalSubscriptionId)
-            : null,
+          subscription_id: localSubscription?.id ?? null,
+          external_subscription_id: externalSubscriptionId,
           plesk_domain_id: String(domainId),
           domain_name: String(domainName),
-          status: item?.status ? String(item.status) : null,
-          hosting_type: item?.hosting_type
-            ? String(item.hosting_type)
-            : item?.hosting?.type
-              ? String(item.hosting.type)
-              : null,
+          status: localSubscription?.status ?? "unknown",
+          hosting_type: item?.hosting_type ? String(item.hosting_type) : null,
           source_created_at: sourceCreatedAt,
           aliases_count: aliasesCount,
         };

supabase/migrations/CL37_domain_status_backfill_from_subscription.sql

@@ -0,0 +1,6 @@
+-- One-time backfill: align existing domain status with linked subscription status
+update public.plesk_domains d
+set status = s.status
+from public.plesk_subscriptions s
+where d.subscription_id = s.id
+  and d.agency_id = s.agency_id;