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

package/src/UI/shared-components/primaria-text-editor/primaria-rich-text-editor.ts

@@ -7,6 +7,31 @@ import { fixSpellCheckerIssue } from "./utils";
 
 fixSpellCheckerIssue(Quill);
 
+/**
+ * Una opció de la barra d'eines de Quill: el nom solt (`"bold"`), un objecte de configuració
+ * (`{ header: 1 }`) o un grup d'opcions, que és com Quill separa els blocs de la barra.
+ */
+type ToolbarOption = string | Record<string, unknown> | (string | Record<string, unknown>)[];
+
+/**
+ * Opcions de la barra d'eines que Quill espera com a objecte de configuració en lloc del nom
+ * solt. Abans era una cadena d'`else if`.
+ */
+const TOOLBAR_OPTION_ALIASES: Record<string, ToolbarOption> = {
+  color: { color: [] },
+  ol: { list: "ordered" },
+  ul: { list: "bullet" },
+  subindex: { script: "sub" },
+  superindex: { script: "super" },
+  // outdent: { indent: "-1" },
+  // indent: { indent: "+1" },
+  size: { size: ["small", false, "large", "huge"] },
+  header: { header: [1, 2, 3, 4, 5, 6, false] },
+  background: { background: [] },
+  font: { font: [] },
+  align: { align: [] },
+};
+
 export class PrimariaRichTextEditor extends LitElement {
   private textChangeTimeout: number | null = null;
 
@@ -124,7 +149,8 @@ export class PrimariaRichTextEditor extends LitElement {
   formats: string;
 
   _getOptions() {
-    let toolbarOptions = [];
+    // Sense anotar, `[]` s'infereix com a `never[]` i totes les assignacions posteriors fallen.
+    let toolbarOptions: ToolbarOption[] = [];
     const availableOpts = this.availableOptions;
     if (this.options === undefined || this.options.length === 0) {
       toolbarOptions = [
@@ -145,26 +171,13 @@ export class PrimariaRichTextEditor extends LitElement {
         // ["clean"],
       ];
     } else {
-      toolbarOptions = this.options.split(",");
-      toolbarOptions.forEach((option) => {
-        if (availableOpts.indexOf(option) <= -1) toolbarOptions.splice(toolbarOptions.indexOf(option), 1);
-      });
-      toolbarOptions = toolbarOptions.map(function (option) {
-        if (option == "color") return { color: [] };
-        else if (option == "ol") return { list: "ordered" };
-        else if (option == "ul") return { list: "bullet" };
-        else if (option == "subindex") return { script: "sub" };
-        else if (option == "superindex") return { script: "super" };
-        // else if (option == 'outdent') return {indent: '-1'};
-        // else if (option == 'indent') return {indent: '+1'};
-        else if (option == "size") return { size: ["small", false, "large", "huge"] };
-        else if (option == "header") return { header: [1, 2, 3, 4, 5, 6, false] };
-        else if (option == "background") return { background: [] };
-        else if (option == "font") return { font: [] };
-        else if (option == "align") return { align: [] };
-        /*         else if (option == 'undo') return {undo: undo_icon};
-        else if (option == 'redo') return {redo: redo_icon}; */ else return option;
-      });
+      // Abans això filtrava amb un `splice` dins d'un `forEach`, que salta elements quan n'hi ha
+      // dos d'invàlids seguits. Amb `filter` el resultat és el que la intenció demanava.
+      toolbarOptions = this.options
+        .split(",")
+        .filter((option) => availableOpts.indexOf(option) > -1)
+        // Les opcions amb configuració es tradueixen; la resta passen tal com són.
+        .map((option) => TOOLBAR_OPTION_ALIASES[option] ?? option);
       // if (toolbarOptions.indexOf("clean") < 0) {
       // toolbarOptions.push("clean");
       // }

package/src/api/activity-monitor/activity-monitor.test.ts

@@ -1,3 +1,4 @@
+// @vitest-environment happy-dom
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { ActivityMonitorImpl } from "./activity-monitor";
 
@@ -63,5 +64,4 @@ describe("ActivityMonitor", () => {
 
     expect(monitor.getLastActivityTimestamp()).toBe(timestampBeforeStop);
   });
-
 });

package/src/api/ecap-event-manager/ecap-event-manager.test.ts

@@ -1,3 +1,4 @@
+// @vitest-environment happy-dom
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { createEcapEventManager, LAST_WORKED_DIAGNOSTICS_KEY } from "./ecap-event-manager";
 
@@ -8,6 +9,13 @@ const createMockGlobalStateManager = (data: Record<string, unknown> = {}) =>
     clearData: vi.fn(),
   }) as any;
 
+/** `mock.calls[0][0]` és `unknown | undefined` amb `noUncheckedIndexedAccess`. */
+const firstPostedMessage = (spy: { mock: { calls: unknown[][] } }): string => {
+  const payload = spy.mock.calls[0]?.[0];
+  if (typeof payload !== "string") throw new Error("s'esperava un missatge publicat");
+  return payload;
+};
+
 describe("EcapEventManagerImpl", () => {
   const mockPostMessage = vi.fn();
   const mockConsoleLog = vi.fn();
@@ -31,7 +39,7 @@ describe("EcapEventManagerImpl", () => {
 
     eventManager.publish("click", "navigate", payload);
 
-    const postedEvent = JSON.parse(mockPostMessage.mock.calls[0][0]);
+    const postedEvent = JSON.parse(firstPostedMessage(mockPostMessage));
     expect(postedEvent.DIAGNOSTICS).toBeUndefined();
     expect(globalStateManager.getData).not.toHaveBeenCalled();
   });
@@ -43,7 +51,7 @@ describe("EcapEventManagerImpl", () => {
 
     eventManager.publish("PRESCRIPCIO_ACTIVA", "NO_TANCAR", {});
 
-    const postedEvent = JSON.parse(mockPostMessage.mock.calls[0][0]);
+    const postedEvent = JSON.parse(firstPostedMessage(mockPostMessage));
     expect(postedEvent.DIAGNOSTICS).toEqual(diagnostics);
     expect(globalStateManager.getData).toHaveBeenCalledWith(LAST_WORKED_DIAGNOSTICS_KEY);
   });
@@ -63,7 +71,7 @@ describe("EcapEventManagerImpl", () => {
 
     eventManager.publish(eventType, "NO_TANCAR", {});
 
-    const postedEvent = JSON.parse(mockPostMessage.mock.calls[0][0]);
+    const postedEvent = JSON.parse(firstPostedMessage(mockPostMessage));
     expect(postedEvent.DIAGNOSTICS).toEqual(diagnostics);
   });
 
@@ -73,7 +81,7 @@ describe("EcapEventManagerImpl", () => {
 
     eventManager.publish("ORDRE_TRACTAMENT", "NO_TANCAR", {});
 
-    const postedEvent = JSON.parse(mockPostMessage.mock.calls[0][0]);
+    const postedEvent = JSON.parse(firstPostedMessage(mockPostMessage));
     expect(postedEvent.DIAGNOSTICS).toEqual([]);
   });
 

package/src/api/http-client/http-client.test.ts

@@ -1,3 +1,4 @@
+// @vitest-environment happy-dom
 import axios, { AxiosInstance } from "axios";
 import AxiosMockAdapter from "axios-mock-adapter";
 import { beforeEach, describe, expect, it, vi } from "vitest";
@@ -14,7 +15,7 @@ const newAccessToken =
   "eyJhbGciOiJSUzI1NiIsInR5cCIgOiAiSldUIiwia2lkIiA6ICJidFZxNnRMWGpmcXdzbm5MR2FXMXdhdU9McDNiTmY4bWZ3Rm1SZ0lBS2VJIn0.eyJleHAiOjE3NDI5MTAzNzgsImlhdCI6MTc0MjkwOTQ3OCwianRpIjoiMTg4OTE3ZDYtNmM0OS00YWY0LTlhYjItZjEzYTM4MGY5OTgyIiwiaXNzIjoiaHR0cHM6Ly9wcmVwcm9kdWNjaW8ucGRzLmhlcy5jYXRzYWx1dC5nZW5jYXQuY2F0L2F1dGgvcmVhbG1zL0hFUyIsInN1YiI6Ijk1OGUyZWQ5LTJkNmUtNDZjOC1hOTE5LTU5MDQ0ZTYwMzVjMiIsInR5cCI6IkJlYXJlciIsImF6cCI6ImV0Yy1jY2YtcHJlIiwic2Vzc2lvbl9zdGF0ZSI6IjY5ODdhMzExLWM1NzQtNDc1Ni1hMzY0LWY1ZjQzZmViZjNhYiIsInJlYWxtX2FjY2VzcyI6eyJyb2xlcyI6WyJkZWZhdWx0LXJvbGVzLWhlcyIsIlJPTEVfSEVTX0VUQyIsIlJPTEVfSEVTX0xBTkRJTkciXX0sInNjb3BlIjoiIiwic2lkIjoiNjk4N2EzMTEtYzU3NC00NzU2LWEzNjQtZjVmNDNmZWJmM2FiIiwiY2xpZW50SG9zdCI6IjEwLjUzLjI1NC4xNTAiLCJhY2Nlc3NfcnVsZXMiOnt9LCJhY2Nlc3NfaW5mbyI6eyJtb2R1bGVfY29kZSI6IkEwMDEiLCJyb2xlX3R5cGUiOiJOT1JNIiwidHJhY2VfdXNlcl9pZCI6IlVzZXJfSUQiLCJjZW50ZXJfdHlwZSI6IkUiLCJtcGlfcGF0aWVudF9pZCI6IjVlYzc0YzdiLTVhYTUtNGE4NC1iM2I0LWYwMmVkZGZkZjZkYyIsInVwX2NvZGUiOiIwNzczMyIsInRyYWNlX3VzZXJfZ2l2ZW5fbmFtZSI6IkdpdmVuIE5hbWUiLCJ0cmFjZV91c2VyX2ZhbWlseV9uYW1lIjoiRmFtaWx5IE5hbWUiLCJ1c2VyX3R5cGUiOiJBRE0iLCJjZW50ZXJfY29kZSI6IkUwODU4Njk2MyIsInByb2Zlc3Npb25hbF9jYXRlZ29yeSI6IjMwOTM0MzAwNiIsInNlcnZpY2VfY29kZSI6IjVTMDg5IiwiZXBfY29kZSI6IjAyMDgiLCJpZGVudGlmaWVyIjpbeyJ0eXBlIjoiRE5JIiwidmFsdWUiOiI3MzI4ODIxOUEifSx7InR5cGUiOiJOVU1DT0wiLCJ2YWx1ZSI6IjIifV0sImFsdF9pZGVudGlmaWVyIjpbeyJ0eXBlIjoiTVBJIiwidmFsdWUiOiIwNjIxY2Y3ZC03ZDYzLTQ5ZWMtODAwNi04YzA1NjkyZWQzNzcifV19LCJjbGllbnRBZGRyZXNzIjoiMTAuNTMuMjU0LjE1MCIsImNsaWVudF9pZCI6ImV0Yy1jY2YtcHJlIn0.zPxoN5d8_uxdbxeH7ly9sQdlnDxNp8_eWDzNSzjzkUPsRVkLxgjQw_WcjZfns-LNgGr0FToS6uSDIC7uj2hPpKqD7HPWbI6UPa7g-43Jf9rh-KSOXEHJWhlASi2w0n3xQHFSxHDYBwhyXlvWQ_Tr1Vh37MroWmEL84fdwqhcfZHmD9Ek52EuRRrB-XkymCmN1ZvO2GPU04LbBvQxLVzdAaKRzr1cUqHTnsEdgRduyKShPcyth01L_X7m128SIwjJsa1lYxFE_DbSDKz8dPTsxoS1DgwRxeArrkMlIdTHyxmDRezZUGhamyLuWpXLDXJCFfrTUB8-j_JnVlILHeR6O";
 
 describe("HTTP Client", () => {
-  let broker = createBroker();
+  const broker = createBroker();
   let tokenManager: TokenManager;
   let axiosInstance: AxiosInstance;
   let axiosMockInstance: AxiosMockAdapter;
@@ -43,9 +44,7 @@ describe("HTTP Client", () => {
       expect(response.data).toBe("success");
       expect(axiosMockInstance.history.get?.length).toBe(1);
       // @ts-ignore
-      expect(axiosMockInstance.history.get?.[0]?.headers.Authorization).toBe(
-        `Bearer ${access_token}`,
-      );
+      expect(axiosMockInstance.history.get?.[0]?.headers.Authorization).toBe(`Bearer ${access_token}`);
     });
 
     it("should retry request with  new token if failed with 401 after token refresh, and requests after that have new token", async () => {
@@ -71,12 +70,8 @@ describe("HTTP Client", () => {
       expect(response.status).toBe(200);
       expect(response.data).toBe("success");
       expect(axiosMockInstance.history.get?.length).toBe(3); // 3 Calls 2 for the first request (request and retry), 1 for the second request
-      expect(axiosMockInstance.history.get?.[1]?.headers?.Authorization).toBe(
-        "Bearer " + newAccessToken,
-      );
-      expect(axiosMockInstance.history.get?.[2]?.headers?.Authorization).toBe(
-        "Bearer " + newAccessToken,
-      );
+      expect(axiosMockInstance.history.get?.[1]?.headers?.Authorization).toBe(`Bearer ${newAccessToken}`);
+      expect(axiosMockInstance.history.get?.[2]?.headers?.Authorization).toBe(`Bearer ${newAccessToken}`);
     });
 
     it("should fail request if token refresh fails and publish an event", async () => {
@@ -86,9 +81,7 @@ describe("HTTP Client", () => {
       const brokerSpy = vi.spyOn(broker, "publish");
       axiosMockInstance.onGet("/api/clinical-course").replyOnce(401, "Unauthorized");
 
-      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow(
-        "Request failed with status code 401",
-      );
+      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow("Request failed with status code 401");
 
       expect(axiosMockInstance.history.get?.length).toBe(1); // No retry happened
       expect(brokerSpy).toHaveBeenCalledWith(BROKER_EVENTS.shell.refreshTokenFailed, {});
@@ -121,9 +114,7 @@ describe("HTTP Client", () => {
       const brokerSpy = vi.spyOn(broker, "publish");
 
       // Perform the request
-      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow(
-        "Mpid header is missing",
-      );
+      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow("Mpid header is missing");
       expect(brokerSpy).not.toHaveBeenCalled();
     });
 
@@ -133,9 +124,7 @@ describe("HTTP Client", () => {
       });
       const brokerSpy = vi.spyOn(broker, "publish");
       // Perform the request
-      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow(
-        "Mpid header value is invalid",
-      );
+      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow("Mpid header value is invalid");
       expect(brokerSpy).toHaveBeenCalledWith(BROKER_EVENTS.shell.mpidHeaderInvalid, expect.any(Object));
     });
 
@@ -148,9 +137,7 @@ describe("HTTP Client", () => {
       expect(response.data).toBe("success");
       expect(axiosMockInstance.history.get?.length).toBe(1);
       // @ts-ignore
-      expect(axiosMockInstance.history.get?.[0]?.headers.Authorization).toBe(
-        `Bearer ${access_token}`,
-      );
+      expect(axiosMockInstance.history.get?.[0]?.headers.Authorization).toBe(`Bearer ${access_token}`);
     });
 
     it("should retry request with  new token if failed with 401 after token refresh, and requests after that have new token", async () => {
@@ -177,12 +164,8 @@ describe("HTTP Client", () => {
       expect(response.status).toBe(200);
       expect(response.data).toBe("success");
       expect(axiosMockInstance.history.get?.length).toBe(3); // 3 Calls 2 for the first request (request and retry), 1 for the second request
-      expect(axiosMockInstance.history.get?.[1]?.headers?.Authorization).toBe(
-        "Bearer " + newAccessToken,
-      );
-      expect(axiosMockInstance.history.get?.[2]?.headers?.Authorization).toBe(
-        "Bearer " + newAccessToken,
-      );
+      expect(axiosMockInstance.history.get?.[1]?.headers?.Authorization).toBe(`Bearer ${newAccessToken}`);
+      expect(axiosMockInstance.history.get?.[2]?.headers?.Authorization).toBe(`Bearer ${newAccessToken}`);
     });
 
     it("should fail request if token refresh fails and publish an event", async () => {
@@ -192,9 +175,7 @@ describe("HTTP Client", () => {
       const brokerSpy = vi.spyOn(broker, "publish");
       axiosMockInstance.onGet("/api/clinical-course").replyOnce(401, "Unauthorized");
 
-      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow(
-        "Request failed with status code 401",
-      );
+      await expect(axiosInstance.get("/api/clinical-course")).rejects.toThrow("Request failed with status code 401");
 
       expect(axiosMockInstance.history.get?.length).toBe(1); // No retry happened
       expect(brokerSpy).toHaveBeenCalledWith(BROKER_EVENTS.shell.refreshTokenFailed, {});

package/src/api/import-data-manager/component/component.ts

@@ -10,6 +10,9 @@ export class ImportDataManagerModal extends LitElement implements ConfirmationCo
   setResult: (result: undefined) => void;
   setIsValid: (isValid: boolean) => void;
   confirm: () => void;
+  // Injectat pel servei d'interacció igual que la resta; faltava des que es va afegir a la
+  // interfície.
+  confirmResult: (result: undefined) => void;
   cancel: () => void;
 
   render() {

package/src/api/import-data-manager/import-data-manager-impl.test.ts

@@ -11,6 +11,8 @@ import {
 class MockInteractionService extends PrimariaInteractionService {
   private mockConfirmResult: { confirmed: boolean; result?: unknown } = { confirmed: true };
 
+  dispose(): void {}
+
   // Mètode per configurar el resultat del mock
   setMockConfirmResult(result: { confirmed: boolean; result?: unknown }) {
     this.mockConfirmResult = result;

package/src/api/interaction-service/interaction-service-impl.tsx

@@ -17,6 +17,14 @@ const defaultOptions: ConfirmationOptions = {
   hideIcon: false,
 };
 
+/**
+ * Un diàleg pot ser un component de React o un web component. Només el primer es pot renderitzar
+ * com a JSX, i cal un predicat de tipus perquè TypeScript ho estrenyi al punt d'ús: guardar el
+ * resultat en un booleà no li serveix.
+ */
+const isWebComponentConstructor = (component: unknown): component is new () => HTMLElement =>
+  typeof component === "function" && component.prototype instanceof HTMLElement;
+
 export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
   private readonly openDialogCancellers = new Set<() => void>();
 
@@ -44,13 +52,13 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
       }
 
       const DialogWrapper = () => {
-        const [result, setResult] = useState<ConfirmationResult<TResult | undefined>>();
+        const [result, setResult] = useState<TResult>();
         const [isValid, setIsValid] = useState(true);
-        const dialogComponentRef = useRef(null);
+        const dialogComponentRef = useRef<HTMLDivElement>(null);
         const dialogRef = useRef<HTMLDivElement>(null);
         const modalRef = useRef<HTMLDivElement>(null);
 
-        const isWebComponent = typeof DialogComponent === "function" && DialogComponent.prototype instanceof HTMLElement;
+        const isWebComponent = isWebComponentConstructor(DialogComponent);
 
         useEffect(() => {
           window.addEventListener("keydown", handleKeydown);
@@ -73,7 +81,7 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
             handleClose(false);
           }
         };
-        const handleOutsideClick = (event: Event) => {
+        const handleOutsideClick = (event: React.MouseEvent<HTMLDivElement>) => {
           if (dialogRef.current) {
             if (event.target === modalRef.current && finalOptions.closeOnOutsideClick) {
               handleClose(false);
@@ -81,6 +89,9 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
           }
         };
 
+        // L'efecte només ha de crear el web component quan canvia el component o les dades.
+        // Afegir-hi els handlers i `isValid` el faria recrear a cada render, perdent l'estat.
+        // biome-ignore lint/correctness/useExhaustiveDependencies: recrear-lo perdria l'estat
         useEffect(() => {
           if (isWebComponent && dialogComponentRef.current) {
             const parent = dialogComponentRef.current;
@@ -113,7 +124,7 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
           resolvePromise(finalResult, true);
         };
 
-        const resolvePromise = (finalResult: TResult, confirmed: boolean) => {
+        const resolvePromise = (finalResult: TResult | undefined, confirmed: boolean) => {
           if (isClosing) return;
 
           isClosing = true;
@@ -153,7 +164,7 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
         };
 
         const _renderContent = () => {
-          return isWebComponent ? (
+          return isWebComponentConstructor(DialogComponent) ? (
             <div ref={dialogComponentRef} />
           ) : (
             <DialogComponent
@@ -181,7 +192,7 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
                           fill
                           class={`dss-dialog-header-state--${getHeadIconVariantClass()}`}
                           icon={getHeadIcon()}
-                        ></dss-icon>
+                        />
                       )}
                       {finalOptions.title}
                     </div>
@@ -193,7 +204,7 @@ export class PrimariaInteractionServiceImpl extends PrimariaInteractionService {
                       variant="neutral"
                       icon="close"
                       onClick={() => handleClose(false)}
-                    ></dss-icon-button>
+                    />
                   )}
                 </div>
               )}

package/src/api/localization/localization.ts

@@ -11,44 +11,48 @@ import { getLanguage, getLocales, localizerFactory, setLocales } from "@uxland/l
  *   getCurrentLanguage: () => string,
  * }>} A promise that resolves to an object with methods for translating strings and getting translations.
  */
-export const createLocaleManager =
-  (pluginId: string) => (translations: Record<string, Record<string, string>>) => {
-    const localizer = localizerFactory(getLanguage(), translations, "", true);
-    setLocales(translations);
-    return Promise.resolve({
-      /**
-       * Translates the given path using the provided variables and the plugin's ID.
-       *
-       * @param {string} path - The path to be translated.
-       * @param {Record<string, string>} [variables] - Optional variables to be used in the translation.
-       * @return {string} The translated string, or the original path if an error occurs.
-       */
-      translate: (path: string, variables?: Record<string, string>) => {
-        try {
-          return localizer(`${pluginId}.${path}`, variables);
-        } catch (e) {
-          console.error(e);
-          return path;
-        }
-      },
-      /**
-       * Retrieves the translations for the current language and plugin.
-       *
-       * @return {Record<string, string>} The translations for the current language and plugin.
-       */
-      getTranslations: () => {
-        const translations = getLocales();
-        const language = getLanguage();
-        return translations[language][pluginId] || {};
-      },
+/** Arbre de traduccions: claus imbricades fins a arribar al text. */
+export interface LocalizationMessages {
+  [key: string]: string | LocalizationMessages;
+}
 
-      /**
-       * Retrieves the current language.
-       *
-       * @return {string} The current language.
-       */
-      getCurrentLanguage: () => {
-        return getLanguage();
-      },
-    });
-  };
+export const createLocaleManager = (pluginId: string) => (translations: Record<string, LocalizationMessages>) => {
+  const localizer = localizerFactory(getLanguage(), translations, "", true);
+  setLocales(translations);
+  return Promise.resolve({
+    /**
+     * Translates the given path using the provided variables and the plugin's ID.
+     *
+     * @param {string} path - The path to be translated.
+     * @param {Record<string, string>} [variables] - Optional variables to be used in the translation.
+     * @return {string} The translated string, or the original path if an error occurs.
+     */
+    translate: (path: string, variables?: Record<string, string>) => {
+      try {
+        return localizer(`${pluginId}.${path}`, variables);
+      } catch (e) {
+        console.error(e);
+        return path;
+      }
+    },
+    /**
+     * Retrieves the translations for the current language and plugin.
+     *
+     * @return {Record<string, string>} The translations for the current language and plugin.
+     */
+    getTranslations: () => {
+      const translations = getLocales();
+      const language = getLanguage();
+      return translations[language][pluginId] || {};
+    },
+
+    /**
+     * Retrieves the current language.
+     *
+     * @return {string} The current language.
+     */
+    getCurrentLanguage: () => {
+      return getLanguage();
+    },
+  });
+};

package/src/api/notification-service/notification-service-impl.test.ts

@@ -1,3 +1,4 @@
+// @vitest-environment happy-dom
 import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 import { PrimariaNotificationServiceImpl } from "./notification.service-impl";
 
@@ -15,7 +16,7 @@ describe("PrimariaNotificationServiceImpl", () => {
     vi.useRealTimers();
   });
 
-  const assertToast = (expectedState: string, message: string, duration: number = 5000) => {
+  const assertToast = (expectedState: string, message: string, duration = 5000) => {
     const toast = document.querySelector("dss-toast") as HTMLElement;
     expect(toast).not.toBeNull();
     expect(toast.getAttribute("isshow")).toBe("true");

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

@@ -1,9 +1,10 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import { createRegionManagerProxy } from "./region-manager";
+// @vitest-environment happy-dom
+import { beforeEach, describe, expect, it, vi } from "vitest";
 import { BROKER_EVENTS } from "../broker/broker-events";
+import { createRegionManagerProxy } from "./region-manager";
 
 const pluginId = "test-plugin";
-const mockView = { id: "view1" };
+const mockView = { id: "view1", factory: () => Promise.resolve(document.createElement("div")) };
 
 const createMockRegion = () => ({
   activate: vi.fn(),
@@ -26,6 +27,7 @@ describe("RegionManagerProxy", () => {
       registerViewWithRegion: vi.fn(),
       unregisterViewFromRegion: vi.fn(),
       getRegion: vi.fn().mockReturnValue(regionMock),
+      getRegisteredViews: vi.fn().mockReturnValue([]),
       destroy: vi.fn(),
     };
 
@@ -127,4 +129,58 @@ describe("RegionManagerProxy", () => {
     proxy._destroy();
     expect(regionManager.destroy).toHaveBeenCalled();
   });
+
+  describe("_removeAllViews", () => {
+    /**
+     * Es crida en tancar, abans del `dispose` dels plugins, perque els cleanups dels `useEffect`
+     * trobin els handlers del broker encara registrats. Ha d'esperar cada desmuntatge: el
+     * `clear()`/`destroy()` del regionManager fan `forEach` amb crides asincrones sense
+     * esperar-les, i per aixo no serveixen.
+     */
+    it("desregistra totes les vistes de totes les regions que tenen", async () => {
+      regionManager.getRegisteredViews = vi.fn((regionName: string) =>
+        regionName === "main-region"
+          ? [
+              { key: "a::v1", view: mockView },
+              { key: "b::v2", view: mockView },
+            ]
+          : [],
+      );
+
+      await proxy._removeAllViews();
+
+      expect(regionManager.unregisterViewFromRegion).toHaveBeenCalledWith("main-region", "a::v1");
+      expect(regionManager.unregisterViewFromRegion).toHaveBeenCalledWith("main-region", "b::v2");
+      expect(regionManager.unregisterViewFromRegion).toHaveBeenCalledTimes(2);
+    });
+
+    it("espera cada desmuntatge abans de continuar amb el seguent", async () => {
+      const order: string[] = [];
+      regionManager.getRegisteredViews = vi.fn((regionName: string) =>
+        regionName === "main-region"
+          ? [
+              { key: "a::v1", view: mockView },
+              { key: "b::v2", view: mockView },
+            ]
+          : [],
+      );
+      regionManager.unregisterViewFromRegion = vi.fn(async (_region: string, key: string) => {
+        order.push(`start:${key}`);
+        await new Promise((resolve) => setTimeout(resolve, 0));
+        order.push(`done:${key}`);
+      });
+
+      await proxy._removeAllViews();
+
+      expect(order).toEqual(["start:a::v1", "done:a::v1", "start:b::v2", "done:b::v2"]);
+    });
+
+    it("no fa res si cap regio te vistes", async () => {
+      regionManager.getRegisteredViews = vi.fn().mockReturnValue([]);
+
+      await proxy._removeAllViews();
+
+      expect(regionManager.unregisterViewFromRegion).not.toHaveBeenCalled();
+    });
+  });
 });

package/src/api/region-manager/region-manager.ts

@@ -1,9 +1,9 @@
-import { HarmonixRegionManager, IRegionManager, HarmonixViewDefinition } from "@uxland/harmonix";
-import { PluginInfo } from "../../handle-plugins";
+import { HarmonixRegionManager, HarmonixViewDefinition, IRegionManager } from "@uxland/harmonix";
 import { IRegion } from "@uxland/regions";
-import { shellRegions, clinicalMonitoringRegions, activityHistoryRegions, administrativeManagementRegions } from "./regions";
-import { PrimariaBroker } from "../broker/primaria-broker";
+import { PluginInfo } from "../../handle-plugins";
 import { BROKER_EVENTS } from "../broker/broker-events";
+import { PrimariaBroker } from "../broker/primaria-broker";
+import { activityHistoryRegions, administrativeManagementRegions, clinicalMonitoringRegions, shellRegions } from "./regions";
 
 export interface PrimariaRegionManager extends HarmonixRegionManager {
   regions: {
@@ -18,9 +18,18 @@ export interface PrimariaRegionManager extends HarmonixRegionManager {
   activateMainView(viewId: string): Promise<void>;
   activatePluginView(regionName: string, pluginId: string, viewId: string): Promise<void>;
   getCurrentMainViewActive(): string | undefined;
-  _destroy(): void;
+  _removeAllViews(): Promise<void>;
+  _destroy(): Promise<void>;
 }
 
+/** Totes les regions que coneix el shell, per poder-les recórrer al tancar. */
+const ALL_REGION_NAMES: string[] = [
+  ...Object.values(shellRegions),
+  ...Object.values(clinicalMonitoringRegions),
+  ...Object.values(activityHistoryRegions),
+  ...Object.values(administrativeManagementRegions),
+];
+
 class RegionManagerProxy implements PrimariaRegionManager {
   regions = {
     shell: shellRegions,
@@ -109,14 +118,24 @@ class RegionManagerProxy implements PrimariaRegionManager {
    */
   isViewActive(regionName: string, viewId: string) {
     const region = this.regionManager.getRegion(regionName);
-    const isViewActive = region.currentActiveViews.some((view) => view.id === viewId);
+    // `ViewDefinition` de @uxland/regions no declara `id`, però el shell només hi registra
+    // `HarmonixViewDefinition`, que sí que en porta.
+    const isViewActive = region.currentActiveViews.some((view) => (view as HarmonixViewDefinition).id === viewId);
     return Promise.resolve(isViewActive);
   }
   async registerQuickAction(view: HarmonixViewDefinition): Promise<void> {
-    await this.regionManager.registerViewWithRegion(this.regions.shell.quickActions, `${this.pluginInfo.pluginId}::${view.id}`, view);
+    await this.regionManager.registerViewWithRegion(
+      this.regions.shell.quickActions,
+      `${this.pluginInfo.pluginId}::${view.id}`,
+      view,
+    );
   }
   async registerNavigationMenu(view: HarmonixViewDefinition): Promise<void> {
-    await this.regionManager.registerViewWithRegion(this.regions.shell.navigationMenu, `${this.pluginInfo.pluginId}::${view.id}`, view);
+    await this.regionManager.registerViewWithRegion(
+      this.regions.shell.navigationMenu,
+      `${this.pluginInfo.pluginId}::${view.id}`,
+      view,
+    );
   }
   async registerMainView(view: HarmonixViewDefinition): Promise<void> {
     await this.regionManager.registerViewWithRegion(this.regions.shell.main, `${this.pluginInfo.pluginId}::${view.id}`, view);
@@ -173,8 +192,31 @@ class RegionManagerProxy implements PrimariaRegionManager {
     this.broker.publish(BROKER_EVENTS.shell.mainViewChanged, { viewId });
   }
 
-  _destroy() {
-    this.regionManager.destroy();
+  /**
+   * Desmunta totes les vistes de totes les regions, i espera que acabin.
+   *
+   * Es crida en tancar, ABANS de fer el `dispose` dels plugins: els cleanups dels `useEffect` que
+   * s'executen en desmuntar-se poden enviar comandes pel broker, i han de trobar el handler encara
+   * registrat. Cada plugin ja ho fa pel seu compte al `dispose`, però això només cobreix els
+   * plugins d'aquest repositori; els de tercers (econsulta, allergies…) no els controlem.
+   *
+   * Fa servir `unregisterViewFromRegion` i no `destroy()`/`clear()` del regionManager perquè
+   * aquests fan `forEach` amb crides asíncrones sense esperar-les, així que retornen abans que les
+   * vistes s'hagin desmuntat de debò.
+   *
+   * És idempotent: desregistrar una vista que ja no hi és no fa res.
+   */
+  async _removeAllViews(): Promise<void> {
+    for (const regionName of ALL_REGION_NAMES) {
+      const registered = this.regionManager.getRegisteredViews(regionName) ?? [];
+      for (const { key } of [...registered]) {
+        await this.regionManager.unregisterViewFromRegion(regionName, key);
+      }
+    }
+  }
+
+  async _destroy() {
+    await this.regionManager.destroy();
   }
 }