@uxland/primary-shell 7.46.13 → 7.46.15

package/dist/primary/shell/src/UI/shared-components/primaria-text-editor/primaria-rich-text-editor.d.ts

@@ -1,5 +1,10 @@
 import { LitElement } from 'lit';
 import { default as Quill } from 'uxl-quill/dist/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>)[];
 export declare class PrimariaRichTextEditor extends LitElement {
     private textChangeTimeout;
     render(): import('lit').TemplateResult<1>;
@@ -19,7 +24,7 @@ export declare class PrimariaRichTextEditor extends LitElement {
     _getOptions(): {
         modules: {
             toolbar: {
-                container: never[];
+                container: ToolbarOption[];
                 handlers: {
                     redo(): void;
                     undo(): void;
@@ -57,3 +62,4 @@ export declare class PrimariaRichTextEditor extends LitElement {
     };
     getCursorPosition(): any;
 }
+export {};

package/dist/primary/shell/src/api/import-data-manager/component/component.d.ts

@@ -5,6 +5,7 @@ export declare class ImportDataManagerModal extends LitElement implements Confir
     setResult: (result: undefined) => void;
     setIsValid: (isValid: boolean) => void;
     confirm: () => void;
+    confirmResult: (result: undefined) => void;
     cancel: () => void;
     render(): import('lit').TemplateResult<1>;
     static styles: import('lit').CSSResult;

package/dist/primary/shell/src/features/get-user-info/backend-adapter.d.ts

@@ -1,5 +1,6 @@
-import { IUserInfo } from './model';
 import { shellApi } from '../../api/api';
+import { IUserInfo } from './model';
+import { PdsAccessInfo } from './pds-token';
 export declare abstract class UserInfoBackendAdapter {
     abstract getUserInfo(): Promise<IUserInfo>;
 }
@@ -7,5 +8,6 @@ export declare class GetUserInfoBackendAdapter implements UserInfoBackendAdapter
     private api;
     constructor(api: typeof shellApi);
     getUserInfo(): Promise<IUserInfo>;
-    getUserId(access_info: any): any;
+    private findIdentifier;
+    getUserId(accessInfo: PdsAccessInfo | undefined): string | undefined;
 }

package/dist/primary/shell/src/features/get-user-info/pds-token.d.ts

@@ -0,0 +1,26 @@
+/**
+ * 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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxland/primary-shell",
-  "version": "7.46.13",
+  "version": "7.46.15",
   "description": "Primaria Shell",
   "author": "UXLand <dev@uxland.es>",
   "homepage": "https://github.com/uxland/harmonix/tree/app#readme",
@@ -47,7 +47,7 @@
     "tslib": "^2.3.0",
     "uxl-quill": "^1.0.1",
     "vite": "^7.0.0",
-    "@primaria/plugins-core": "1.0.21"
+    "@primaria/plugins-core": "1.0.22"
   },
   "scripts": {
     "serve": "vite",

package/src/UI/components/quick-actions-menu/quick-actions-menu.ts

@@ -35,11 +35,11 @@ export class QuickActionsMenu extends LitElement {
 
     // Obtener los elementos asignados al otro slot
     const assignedElements = contentSlot?.assignedElements();
-    if (assignedElements[0])
-      this._addMenuCustomStyle(contentSlot as HTMLElement, actionMenu as HTMLElement);
+    if (assignedElements[0]) this._addMenuCustomStyle(contentSlot as HTMLElement, actionMenu as HTMLElement);
 
     // Hacer target al elemento que deseas (por ejemplo, un <p>)
-    const targetElement = assignedElements.find((el) => el.id === "content");
+    // Els components del DSS exposen `show()`, que no és a la interfície `Element`.
+    const targetElement = assignedElements.find((el): el is Element & { show(): void } => el.id === "content");
     if (targetElement) {
       // Modificar el contenido o estilo del elemento
       targetElement.show();

package/src/UI/shared-components/bootstrapper.ts

@@ -12,5 +12,7 @@ export const useSharedUI = () => {
   customElement("primaria-rich-text-editor")(PrimariaRichTextEditor);
   customElement("primaria-nav-tree-menu")(PrimariaNavTreeMenu);
   customElement("primaria-character-limit-badge")(PrimariaCharacterLimitBadge);
-   customElement("primaria-region")(PrimariaRegion);
+  // @ts-ignore: `PrimariaRegion` estén el mixin `PrimariaRegionHost`, que està tipat com a
+  // `any`, així que TypeScript no el reconeix com a `CustomElementClass`.
+  customElement("primaria-region")(PrimariaRegion);
 };

package/src/UI/shared-components/primaria-nav-item/primaria-nav-item.ts

@@ -48,7 +48,9 @@ export class PrimariaNavItem extends regionView(LitElement) {
 
   _subscribeEvents() {
     const subscription = shellApi.broker.subscribe(BROKER_EVENTS.shell.mainViewChanged, (payload: { viewId: string }) => {
-      this.isActive = payload.viewId === this.view?.id;
+      // `ViewDefinition` de @uxland/regions no declara `id`, però el shell només hi registra
+      // vistes que en porten.
+      this.isActive = payload.viewId === (this.view as { id?: string } | undefined)?.id;
     });
     this.subscriptions.push(subscription);
   }

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/ecap-event-manager/ecap-event-manager.test.ts

@@ -8,6 +8,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 +38,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 +50,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 +70,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 +80,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/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

@@ -1,14 +1,9 @@
-import {
-  ConfirmationOptions,
-  ConfirmationResult,
-  ConfirmComponentUI,
-  PrimariaInteractionService,
-} from "./interaction-service";
+import { ConfirmationOptions, ConfirmationResult, ConfirmComponentUI, PrimariaInteractionService } from "./interaction-service";
 import * as React from "react";
 import { useEffect, useState, useRef } from "react";
 import { createRoot } from "react-dom/client";
 import { ConfirmationMessage } from "./confirmation-message.tsx";
-import modalStyles from './modal-styles.css?raw'; 
+import modalStyles from "./modal-styles.css?raw";
 
 const defaultOptions: ConfirmationOptions = {
   title: "Confirmació",
@@ -19,16 +14,24 @@ const defaultOptions: ConfirmationOptions = {
   showCloseButton: true,
   fullCustomization: false,
   closeOnOutsideClick: false,
-  hideIcon: false
+  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 ParimariaInteractionServiceImpl extends PrimariaInteractionService {
   private readonly openDialogCancellers = new Set<() => void>();
 
   confirm<TData = undefined, TResult = undefined>(
     data: TData | undefined,
     componentUI: ConfirmComponentUI<TData, TResult>,
-    options?: ConfirmationOptions | undefined
+    options?: ConfirmationOptions | undefined,
   ): Promise<ConfirmationResult<TResult | undefined>> {
     const finalOptions: ConfirmationOptions = { ...defaultOptions, ...(options || {}) };
     const { openDialogCancellers } = this;
@@ -42,26 +45,25 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
       const DialogComponent = componentUI.component;
       const DialogComponentStyles = componentUI.styles;
       let componentHeadStyles;
-      if(DialogComponentStyles) {
+      if (DialogComponentStyles) {
         componentHeadStyles = document.createElement("style");
         componentHeadStyles.textContent = componentUI?.styles;
         document.head.appendChild(componentHeadStyles);
       }
 
       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);
+          window.addEventListener("keydown", handleKeydown);
 
-          return () => window.removeEventListener('keydown', handleKeydown);
+          return () => window.removeEventListener("keydown", handleKeydown);
         }, []);
 
         useEffect(() => {
@@ -75,11 +77,11 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
         }, []);
 
         const handleKeydown = (event: KeyboardEvent) => {
-          if (event.key === 'Escape') {
+          if (event.key === "Escape") {
             handleClose(false);
           }
         };
-        const handleOutsideClick = (event: Event) => {
+        const handleOutsideClick = (event: React.MouseEvent<HTMLDivElement>) => {
           if (dialogRef.current) {
             if (event.target === modalRef.current && finalOptions.closeOnOutsideClick) {
               handleClose(false);
@@ -87,6 +89,9 @@ export class ParimariaInteractionServiceImpl 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;
@@ -119,7 +124,7 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
           resolvePromise(finalResult, true);
         };
 
-        const resolvePromise = (finalResult: TResult, confirmed: boolean) => {
+        const resolvePromise = (finalResult: TResult | undefined, confirmed: boolean) => {
           if (isClosing) return;
 
           isClosing = true;
@@ -132,8 +137,7 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
             root.unmount();
             document.body.removeChild(div);
             document.head.removeChild(wrapperHeadStyles);
-            if (DialogComponentStyles)
-              document.head.removeChild(componentHeadStyles);
+            if (DialogComponentStyles) document.head.removeChild(componentHeadStyles);
           }, 300);
         };
 
@@ -160,7 +164,7 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
         };
 
         const _renderContent = () => {
-          return isWebComponent ? (
+          return isWebComponentConstructor(DialogComponent) ? (
             <div ref={dialogComponentRef} />
           ) : (
             <DialogComponent
@@ -176,7 +180,7 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
           );
         };
         return (
-          <div className="modal" ref={modalRef} onClick={handleOutsideClick} >
+          <div className="modal" ref={modalRef} onClick={handleOutsideClick}>
             <div className="dialog" ref={dialogRef}>
               {!finalOptions.fullCustomization && (
                 <div className="dss-dialog-header">
@@ -188,30 +192,27 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
                           fill
                           class={`dss-dialog-header-state--${getHeadIconVariantClass()}`}
                           icon={getHeadIcon()}
-                        ></dss-icon>
+                        />
                       )}
                       {finalOptions.title}
                     </div>
                   </div>
                   {finalOptions.showCloseButton && (
                     <dss-icon-button
-                      class="dss-dialog-header-close" 
-                      size="md" 
-                      variant="neutral" 
-                      icon="close" 
+                      class="dss-dialog-header-close"
+                      size="md"
+                      variant="neutral"
+                      icon="close"
                       onClick={() => handleClose(false)}
-                    ></dss-icon-button>
+                    />
                   )}
                 </div>
               )}
-              <div className={`modal-content ${finalOptions.fullCustomization ? 'full-customization' : ''}`}>
+              <div className={`modal-content ${finalOptions.fullCustomization ? "full-customization" : ""}`}>
                 {_renderContent()}
               </div>
               {!finalOptions.fullCustomization && (
-                <div
-                  className="dss-modal-footer"
-                  hidden={!finalOptions.showCancelButton && !finalOptions.showConfirmButton}
-                >
+                <div className="dss-modal-footer" hidden={!finalOptions.showCancelButton && !finalOptions.showConfirmButton}>
                   {finalOptions.showCancelButton && (
                     <dss-button
                       label={finalOptions.cancelButtonText}
@@ -238,17 +239,13 @@ export class ParimariaInteractionServiceImpl extends PrimariaInteractionService
         );
       };
 
-
       const root = createRoot(div);
       root.render(<DialogWrapper />);
     });
   }
 
-  confirmMessage(
-    message: string,
-    options?: ConfirmationOptions | undefined,
-  ): Promise<ConfirmationResult> {
-    return this.confirm(message, {component: ConfirmationMessage}, options);
+  confirmMessage(message: string, options?: ConfirmationOptions | undefined): Promise<ConfirmationResult> {
+    return this.confirm(message, { component: ConfirmationMessage }, options);
   }
 
   dispose(): void {

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/region-manager/region-manager.test.ts

@@ -3,7 +3,7 @@ import { createRegionManagerProxy } from "./region-manager";
 import { BROKER_EVENTS } from "../broker/broker-events";
 
 const pluginId = "test-plugin";
-const mockView = { id: "view1" };
+const mockView = { id: "view1", factory: () => Promise.resolve(document.createElement("div")) };
 
 const createMockRegion = () => ({
   activate: vi.fn(),

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

@@ -109,14 +109,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);