Revert "fix(integrations): label sites with the console name Site Manager shows"

This reverts commit 12db2c0677.

packages/integrations/src/unifi-site-manager/test/unifi-site-manager-types.spec.ts

@@ -162,14 +162,14 @@ describe("mapSites", () => {
     },
   };
 
-  const host = { id: "host-1", reportedState: { name: "Console A", deviceState: "updateAvailable" } };
+  const host = { id: "host-1", reportedState: { deviceState: "updateAvailable" } };
 
   test("maps a complete site", () => {
     const [mapped] = mapSites([site], [host], [], new Map(), NOW);
 
     expect(mapped).toMatchObject({
       siteId: "site-1",
-      name: "Console A",
+      name: "Example site",
       wanUptimePct: 99.5,
       ispName: "Example ISP",
       externalIps: ["203.0.113.10", "203.0.113.11"],
@@ -180,51 +180,12 @@ describe("mapSites", () => {
     });
   });
 
-  /**
-   * Le défaut relevé sur le compte réel : une console gardait la description de
-   * site par défaut alors que Site Manager l'affiche sous son propre nom. C'est
-   * le nom de console que le propriétaire reconnaît.
-   */
-  test("labels a site with its console name rather than a differing site description", () => {
-    const [mapped] = mapSites([{ ...site, meta: { desc: "Default", name: "default" } }], [host], [], new Map(), NOW);
-
-    expect(mapped!.name).toBe("Console A");
-  });
-
-  /** Le nom de console seul ne distinguerait plus les deux lignes. */
-  test("composes the console name with the site description when one console carries several sites", () => {
-    const mapped = mapSites(
-      [
-        { ...site, siteId: "site-1", meta: { desc: "Warehouse", name: "default" } },
-        { ...site, siteId: "site-2", meta: { desc: "Loading dock", name: "dock" } },
-      ],
-      [host],
-      [],
-      new Map(),
-      NOW,
-    );
-
-    expect(mapped.map((entry) => entry.name)).toEqual(["Console A · Warehouse", "Console A · Loading dock"]);
-  });
-
-  test("falls back to the site description when no host names the console", () => {
-    const [mapped] = mapSites([site], [], [], new Map(), NOW);
-
-    expect(mapped!.name).toBe("Example site");
-  });
-
   test("falls back to meta.name when there is no description", () => {
     const [mapped] = mapSites([{ ...site, meta: { name: "default" } }], [], [], new Map(), NOW);
 
     expect(mapped!.name).toBe("default");
   });
 
-  test("falls back to the site identifier when nothing is named", () => {
-    const [mapped] = mapSites([{ siteId: "site-1", meta: {} }], [], [], new Map(), NOW);
-
-    expect(mapped!.name).toBe("site-1");
-  });
-
   /**
    * `null` veut dire « aucun host ne correspond », ce qui n'est pas « la console
    * est à jour ». Le widget doit pouvoir les distinguer.

packages/integrations/src/unifi-site-manager/unifi-site-manager-types.ts

@@ -52,8 +52,6 @@ export const siteManagerHostSchema = z.looseObject({
   id: z.string(),
   reportedState: z
     .looseObject({
-      /** Le nom de la console, celui que Site Manager affiche. Voir `resolveSiteName`. */
-      name: z.string().optional(),
       deviceState: z.string().optional(),
       /**
        * `.nullable().optional()` et non `.nullish()` : « présent et explicitement
@@ -294,52 +292,14 @@ const resolveConsoleUpdate = (host: SiteManagerHost | undefined): boolean | null
   return null;
 };
 
-/**
- * Le libellé d'une ligne est le nom de la CONSOLE, pas la description du site.
- *
- * Site Manager nomme ses lignes d'après `host.reportedState.name` : c'est ce que
- * le propriétaire du compte lit chez Ubiquiti, donc ce qu'il reconnaît dans le
- * widget. `meta.desc` est une autre chose — la description du site *à
- * l'intérieur* de la console — qui ne coïncide que lorsqu'on a nommé les deux
- * pareil. Sur le compte réel, une console garde le `meta.desc` par défaut
- * « Default » alors que Site Manager l'affiche sous son propre nom : le widget
- * étiquetait donc « Default » un site que son propriétaire identifie autrement.
- * `meta.name` est pire encore — le slug interne, « default » sur les deux
- * consoles.
- *
- * Une console peut porter plusieurs sites. Son nom ne distingue alors plus les
- * lignes entre elles, et la description les départage.
- */
-const resolveSiteName = (site: SiteManagerSite, host: SiteManagerHost | undefined, sitesOnConsole: number): string => {
-  const consoleName = host?.reportedState?.name?.trim();
-  if (consoleName === undefined || consoleName === "") return site.meta.desc ?? site.meta.name ?? site.siteId;
-
-  const description = site.meta.desc?.trim();
-  if (sitesOnConsole > 1 && description !== undefined && description !== "") return `${consoleName} · ${description}`;
-
-  return consoleName;
-};
-
-/** Compté une fois sur tout le compte, pas par ligne. */
-const countSitesPerHost = (sites: SiteManagerSite[]): Map<string, number> => {
-  const counts = new Map<string, number>();
-  for (const site of sites) {
-    if (site.hostId === undefined) continue;
-    counts.set(site.hostId, (counts.get(site.hostId) ?? 0) + 1);
-  }
-  return counts;
-};
-
 export const mapSites = (
   sites: SiteManagerSite[],
   hosts: SiteManagerHost[],
   metrics: IspMetricsEntry[],
   sdWanBySite: Map<string, UnifiSiteSdWanStatus>,
   now: Date,
-): UnifiSiteManagerSite[] => {
-  const sitesPerHost = countSitesPerHost(sites);
-
-  return sites.map((site) => {
+): UnifiSiteManagerSite[] =>
+  sites.map((site) => {
     const statistics = site.statistics;
     const counts = statistics?.counts;
     const host = site.hostId === undefined ? undefined : hosts.find((candidate) => candidate.id === site.hostId);
@@ -350,7 +310,7 @@ export const mapSites = (
 
     return {
       siteId: site.siteId,
-      name: resolveSiteName(site, host, site.hostId === undefined ? 1 : (sitesPerHost.get(site.hostId) ?? 1)),
+      name: site.meta.desc ?? site.meta.name ?? site.siteId,
       wanUptimePct: statistics?.percentages?.wanUptime ?? null,
       ispName: statistics?.ispInfo?.name ?? null,
       externalIps: Object.values(statistics?.wans ?? {})
@@ -367,4 +327,3 @@ export const mapSites = (
       sdWan: sdWanBySite.get(site.siteId) ?? null,
     };
   });
-};