@uxland/primary-shell 8.0.0-rc.0 → 8.0.0-rc.1

package/src/api/region-manager/remove-view-idempotency.test.ts

@@ -0,0 +1,98 @@
+// @vitest-environment happy-dom
+import { createRegionManager } from "@uxland/harmonix";
+import { type IRegionAdapter, Region } from "@uxland/regions";
+import { beforeEach, describe, expect, it, vi } from "vitest";
+
+/**
+ * Aquest test NO fa servir dobles del regionManager: munta una `Region` de `@uxland/regions` i un
+ * `RegionManager` de `@uxland/harmonix` de debo.
+ *
+ * El motiu: des que el shell desmunta totes les vistes al tancar (`_removeAllViews`), l'
+ * `unregisterViews` de cada plugin s'executa DESPRES, sobre vistes que ja no hi son. Tot el
+ * disseny es sosté en que aquesta segona passada sigui inofensiva, i aixo depen del comportament
+ * intern d'aquestes dues llibreries, no del nostre codi. Amb dobles no es provaria res.
+ *
+ * Si una actualitzacio d'aquests paquets fa que la segona crida arribi a l'adaptador, el
+ * component es desmuntaria dues vegades i ara mateix nomes ho sabriem en produccio.
+ */
+const REGION = "test-region";
+const VIEW_KEY = "some-plugin::some-view";
+
+let adapter: IRegionAdapter;
+let regionManager: ReturnType<typeof createRegionManager>;
+
+const createSpyAdapter = (): IRegionAdapter => ({
+  activateView: vi.fn().mockResolvedValue(undefined),
+  deactivateView: vi.fn().mockResolvedValue(undefined),
+  removeView: vi.fn(),
+  viewAdded: vi.fn().mockResolvedValue(undefined),
+  behaviors: [],
+});
+
+const mountRegion = () => {
+  const host = document.createElement("div") as HTMLElement & { uxlRegion?: unknown };
+  const region = new Region(REGION, regionManager as never, host as never, adapter, {
+    name: REGION,
+    targetId: REGION,
+  });
+  regionManager.add(REGION, region);
+  return region;
+};
+
+const addAndActivateView = async (region: Region) => {
+  await regionManager.registerViewWithRegion(REGION, VIEW_KEY, {
+    factory: () => Promise.resolve(document.createElement("span")),
+  });
+  // Nomes les vistes actives tenen component, i `removeView` nomes crida l'adaptador si n'hi ha.
+  await region.activate(VIEW_KEY);
+};
+
+beforeEach(() => {
+  adapter = createSpyAdapter();
+  regionManager = createRegionManager("test");
+});
+
+describe("treure una vista dues vegades (shell + plugin)", () => {
+  it("la primera passada desmunta la vista de debo", async () => {
+    const region = mountRegion();
+    await addAndActivateView(region);
+
+    await regionManager.unregisterViewFromRegion(REGION, VIEW_KEY);
+
+    expect(adapter.deactivateView).toHaveBeenCalledOnce();
+    expect(adapter.removeView).toHaveBeenCalledOnce();
+  });
+
+  it("la segona no torna a tocar l'adaptador", async () => {
+    const region = mountRegion();
+    await addAndActivateView(region);
+
+    await regionManager.unregisterViewFromRegion(REGION, VIEW_KEY);
+    vi.mocked(adapter.deactivateView).mockClear();
+    vi.mocked(adapter.removeView).mockClear();
+
+    // El que fa `unregisterViews` del plugin despres que el shell ja ho hagi tret tot.
+    await regionManager.unregisterViewFromRegion(REGION, VIEW_KEY);
+
+    expect(adapter.deactivateView).not.toHaveBeenCalled();
+    expect(adapter.removeView).not.toHaveBeenCalled();
+  });
+
+  it("la segona passada no llanca ni deixa la vista registrada", async () => {
+    const region = mountRegion();
+    await addAndActivateView(region);
+
+    await regionManager.unregisterViewFromRegion(REGION, VIEW_KEY);
+
+    await expect(regionManager.unregisterViewFromRegion(REGION, VIEW_KEY)).resolves.toBeDefined();
+    expect(region.containsView(VIEW_KEY)).toBe(false);
+    expect(regionManager.getRegisteredViews(REGION)).toEqual([]);
+  });
+
+  it("treure una vista que no s'ha registrat mai tampoc fa res", async () => {
+    mountRegion();
+
+    await expect(regionManager.unregisterViewFromRegion(REGION, "mai::registrada")).resolves.toBeDefined();
+    expect(adapter.removeView).not.toHaveBeenCalled();
+  });
+});

package/src/api/token-manager/token-manager.test.ts

@@ -1,3 +1,4 @@
+// @vitest-environment happy-dom
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import axios from "axios";
 import { BROKER_EVENTS } from "../broker/broker-events";
@@ -68,17 +69,13 @@ describe("TokenManager", () => {
     it("should throw error if tokens are already initialized", () => {
       const tokenManager = new TokenManagerImpl(mockBroker);
       tokenManager.setInitialTokens("manual_access", "manual_refresh");
-      expect(() => tokenManager.setInitialTokens("another_access", "another_refresh")).toThrow(
-        "Token already initialized",
-      );
+      expect(() => tokenManager.setInitialTokens("another_access", "another_refresh")).toThrow("Token already initialized");
     });
 
     it("should throw error if tokens were already initialized from URL", () => {
       const tokenManager = new TokenManagerImpl(mockBroker);
       tokenManager.getToken();
-      expect(() => tokenManager.setInitialTokens("manual_access", "manual_refresh")).toThrow(
-        "Token already initialized",
-      );
+      expect(() => tokenManager.setInitialTokens("manual_access", "manual_refresh")).toThrow("Token already initialized");
     });
   });
 

package/src/disposer.ts

@@ -1,8 +1,8 @@
 import { activityMonitor, sessionRefreshTimer, shellApi } from "./api/api";
 import { disposeFeatures } from "./features/bootstrapper";
 import { ExitShellPayload } from "./features/exit/request";
-export const disposeShell = () => {
-  disposeFeatures(shellApi);
+export const disposeShell = async () => {
+  await disposeFeatures(shellApi);
   shellApi.interactionService.dispose();
   sessionRefreshTimer.stop();
   activityMonitor.stop();

package/src/features/bootstrapper.ts

@@ -12,10 +12,10 @@ export const useFeatures = (api: PrimariaApi) => {
   api.broker.send(new GetUserInfo());
 };
 
-export const disposeFeatures = (api: PrimariaApi) => {
+export const disposeFeatures = async (api: PrimariaApi) => {
   disposeGetUserInfo();
   disposeExitShell();
   container.unbindAll();
-  api.regionManager._destroy();
+  await api.regionManager._destroy();
   api.broker._clear();
 };

package/src/features/exit/handler.test.ts

@@ -23,6 +23,9 @@ const createMockApi = (): PrimariaApi =>
     notificationService: {
       error: vi.fn(),
     },
+    regionManager: {
+      _removeAllViews: vi.fn().mockResolvedValue(undefined),
+    },
   }) as any;
 
 describe("ExitShellHandler", () => {
@@ -120,4 +123,61 @@ describe("ExitShellHandler", () => {
     expect(raiseCloseEvent).toHaveBeenCalled();
     expect(raiseCustomCloseEvent).not.toHaveBeenCalled();
   });
+
+  describe("ordre de tancament: vistes abans que plugins", () => {
+    /**
+     * Els cleanups dels `useEffect` s'executen en desmuntar-se el component i poden enviar
+     * comandes pel broker (p. ex. `ClearTransportDetail` a petitioner). Si el `dispose` dels
+     * plugins corre primer, el handler ja no hi es i el `send` rebutja. Cada plugin d'aquest
+     * repositori ja ho ordena be, pero els de tercers no els controlem: aquest ordre els cobreix.
+     */
+    it("desmunta totes les vistes abans de disposar els plugins", async () => {
+      const order: string[] = [];
+      (mockApi.regionManager._removeAllViews as any).mockImplementation(async () => {
+        order.push("removeAllViews");
+      });
+      (disposePlugins as any).mockImplementation(async () => {
+        order.push("disposePlugins");
+      });
+
+      await handler.handle({} as ExitShell);
+
+      expect(order).toEqual(["removeAllViews", "disposePlugins"]);
+    });
+
+    it("no comenca a disposar fins que el desmuntatge ha acabat", async () => {
+      let finishUnmount: () => void = () => {};
+      (mockApi.regionManager._removeAllViews as any).mockImplementation(
+        () =>
+          new Promise<void>((resolve) => {
+            finishUnmount = resolve;
+          }),
+      );
+      (disposePlugins as any).mockResolvedValue(undefined);
+
+      const handled = handler.handle({} as ExitShell);
+      await Promise.resolve();
+      await Promise.resolve();
+      expect(disposePlugins).not.toHaveBeenCalled();
+
+      finishUnmount();
+      await handled;
+
+      expect(disposePlugins).toHaveBeenCalled();
+    });
+
+    it("el limit de temps del tancament segueix cobrint tot el proces", async () => {
+      vi.useFakeTimers();
+      (mockApi.regionManager._removeAllViews as any).mockImplementation(() => new Promise<void>(() => {}));
+
+      const handled = handler.handle({} as ExitShell);
+      await vi.advanceTimersByTimeAsync(10000);
+      await handled;
+
+      // Encara que el desmuntatge no acabi mai, el tancament continua.
+      expect(disposeShell).toHaveBeenCalled();
+      expect(raiseCloseEvent).toHaveBeenCalled();
+      vi.useRealTimers();
+    });
+  });
 });

package/src/features/exit/handler.ts

@@ -17,10 +17,10 @@ export class ExitShellHandler {
 
       // Per si un plugin tarda molt en fer dispose, màxim deixarem 5 segons, per no interrompre el tancar infinitament
       await Promise.race([
-        disposePlugins(), // S'intenta executar un dispose normal
+        this.unmountViewsAndDisposePlugins(), // S'intenta executar un dispose normal
         this.timeout(10000), // Si passen 5s, es segueix amb l'execució
       ]);
-      disposeShell();
+      await disposeShell();
       this.emitClose(evt);
     } catch (error) {
       this.api.notificationService.error(translate("errors.exit"));
@@ -28,6 +28,22 @@ export class ExitShellHandler {
     }
   }
 
+  /**
+   * Desmunta les vistes de tots els plugins abans de fer-los el `dispose`.
+   *
+   * Els cleanups dels `useEffect` s'executen en desmuntar-se el component i poden enviar comandes
+   * pel broker (p. ex. `ClearTransportDetail` a petitioner). Si el `dispose` s'executa primer, el
+   * handler ja no hi és i el `send` rebutja. Cada plugin d'aquest repositori ja desmunta les seves
+   * vistes primer, però els de tercers no els controlem: fer-ho aquí els cobreix tots.
+   *
+   * Va dins del mateix `Promise.race` que el `dispose`, així que el límit de temps del tancament
+   * no canvia.
+   */
+  private async unmountViewsAndDisposePlugins(): Promise<void> {
+    await this.api.regionManager._removeAllViews();
+    await disposePlugins();
+  }
+
   private timeout(ms: number) {
     return new Promise<void>((resolve) => setTimeout(resolve, ms));
   }

package/src/features/get-user-info/backend-adapter.ts

@@ -1,8 +1,12 @@
 import { inject, injectable } from "inversify";
-import { IUserInfo } from "./model";
 import { jwtDecode } from "jwt-decode";
-import { TYPES } from "../../infrastructure/ioc/types";
 import { shellApi } from "../../api/api";
+import { TYPES } from "../../infrastructure/ioc/types";
+import { IUserInfo } from "./model";
+import { PdsAccessInfo, PdsToken } from "./pds-token";
+
+/** Ordre de preferència dels identificadors del professional dins del token. */
+const USER_ID_TYPES = ["DNI", "NIE", "PASS"];
 
 export abstract class UserInfoBackendAdapter {
   abstract getUserInfo(): Promise<IUserInfo>;
@@ -11,26 +15,34 @@ export abstract class UserInfoBackendAdapter {
 export class GetUserInfoBackendAdapter implements UserInfoBackendAdapter {
   constructor(@inject(TYPES.primaryApi) private api: typeof shellApi) {}
   getUserInfo(): Promise<IUserInfo> {
-    const decodedToken = jwtDecode(this.api.tokenManager.getToken());
+    const decodedToken = jwtDecode<PdsToken>(this.api.tokenManager.getToken());
+    const accessInfo = decodedToken.access_info;
+
+    // `IUserInfo` declara `string` a tots els camps, però el token no en garanteix cap. Abans
+    // el token era `any` i els absents arribaven com a `undefined` amb tipus `string`; ara es
+    // normalitzen a cadena buida perquè el tipus digui la veritat.
     const userInfo: IUserInfo = {
-      userId: this.getUserId(decodedToken.access_info),
-      firstName: decodedToken.access_info.trace_user_given_name,
-      familyName: decodedToken.access_info.trace_user_family_name,
-      lastName: decodedToken.access_info.trace_user_last_name,
-      registrationNumber: decodedToken.access_info.identifier.find((id: { type: string; value: string }) => id.type === "NUMCOL")
-        ?.value,
-      workCenter: decodedToken.access_info.center_code,
-      role: decodedToken.access_info.role_type,
-      speciality: decodedToken.access_info.professional_category,
+      userId: this.getUserId(accessInfo) ?? "",
+      firstName: accessInfo?.trace_user_given_name ?? "",
+      familyName: accessInfo?.trace_user_family_name ?? "",
+      lastName: accessInfo?.trace_user_last_name ?? "",
+      registrationNumber: this.findIdentifier(accessInfo, "NUMCOL") ?? "",
+      workCenter: accessInfo?.center_code ?? "",
+      role: accessInfo?.role_type ?? "",
+      speciality: accessInfo?.professional_category ?? "",
     };
     return Promise.resolve(userInfo);
   }
 
-  getUserId(access_info) {
-    return (
-      access_info.identifier?.find((identifierItem: { type: string; value: string }) => identifierItem.type === "DNI")?.value ||
-      access_info.identifier?.find((identifierItem: { type: string; value: string }) => identifierItem.type === "NIE")?.value ||
-      access_info.identifier?.find((identifierItem: { type: string; value: string }) => identifierItem.type === "PASS")?.value
-    );
+  private findIdentifier(accessInfo: PdsAccessInfo | undefined, type: string) {
+    return accessInfo?.identifier?.find((identifier) => identifier.type === type)?.value;
+  }
+
+  getUserId(accessInfo: PdsAccessInfo | undefined) {
+    for (const type of USER_ID_TYPES) {
+      const value = this.findIdentifier(accessInfo, type);
+      if (value) return value;
+    }
+    return undefined;
   }
 }

package/src/features/get-user-info/handler.test.ts

@@ -27,8 +27,14 @@ describe("GetUserInfoHandler", () => {
 
   const message = {} as GetUserInfo;
   const mockUserInfo: IUserInfo = {
-    name: "John Doe",
-    email: "john@example.com",
+    userId: "12345678Z",
+    firstName: "John",
+    familyName: "Doe",
+    lastName: "Smith",
+    registrationNumber: "081234",
+    workCenter: "CAP",
+    role: "assist",
+    speciality: "MF",
   };
 
   beforeEach(() => {

package/src/features/get-user-info/pds-token.ts

@@ -0,0 +1,28 @@
+/**
+ * Els camps del token del PDS que aquest adaptador llegeix.
+ *
+ * Es declara aquí, i no s'importa `PDSToken` de `@pds/core`, per dues raons: el shell no depèn
+ * d'aquell package, i el seu `AccessInfo` no inclou `trace_user_last_name`, que sí que es fa
+ * servir. Si algun dia s'unifiquen els dos contractes, aquest fitxer hauria de desaparèixer.
+ *
+ * Tot és opcional perquè el token no en garanteix cap: fins ara el tipus era `any` i
+ * `IUserInfo` prometia `string` a tot arreu, cosa que no era certa en runtime.
+ */
+export interface PdsAccessInfo {
+  trace_user_given_name?: string;
+  trace_user_family_name?: string;
+  trace_user_last_name?: string;
+  center_code?: string;
+  role_type?: string;
+  professional_category?: string;
+  identifier?: PdsIdentifier[];
+}
+
+export interface PdsIdentifier {
+  type: string;
+  value: string;
+}
+
+export interface PdsToken {
+  access_info?: PdsAccessInfo;
+}

package/src/features/get-visi-id/action.ts

@@ -0,0 +1,11 @@
+import { PrimariaApi } from "../../api/api";
+
+export const getVisiId = async (api: PrimariaApi): Promise<string | undefined> => {
+  const context = api.contextManager.getContext();
+  if (context.visi_id) return context.visi_id as string;
+  try {
+    return (await api.broker.send("get_visi_id")) as string;
+  } catch {
+    return undefined;
+  }
+};

package/src/types/custom.d.ts

@@ -0,0 +1,30 @@
+import "react";
+
+/**
+ * Els components web del Design System de Salut (`dss-*`) no porten tipus, així que s'obre
+ * `JSX.IntrinsicElements` per acceptar qualsevol element personalitzat.
+ *
+ * S'amplien els dos especificadors: amb `jsx: "react-jsx"`, la cerca del namespace JSX passa per
+ * `react/jsx-runtime`, que hereta de `React.JSX`.
+ *
+ * Aquest fitxer és idèntic a tots els plugins i no es pot compartir des de l'arrel: sota pnpm,
+ * l'arrel no té `react` a `node_modules`, així que l'especificador no resoldria i el
+ * `declare module` no ampliaria res.
+ */
+declare module "react" {
+  namespace JSX {
+    interface IntrinsicElements {
+      // biome-ignore lint/suspicious/noExplicitAny: els components del DSS no estan tipats
+      [key: string]: any;
+    }
+  }
+}
+
+declare module "react/jsx-runtime" {
+  namespace JSX {
+    interface IntrinsicElements {
+      // biome-ignore lint/suspicious/noExplicitAny: els components del DSS no estan tipats
+      [key: string]: any;
+    }
+  }
+}