@uxland/primary-shell 7.46.2 → 7.46.4

package/dist/primary/shell/src/UI/components/clinical-pathways-action-menu/clinical-pathways-action-menu.d.ts

@@ -1,5 +1,5 @@
-import { PropertyValues } from 'lit';
 import { IRegion } from '@uxland/regions';
+import { PropertyValues } from 'lit';
 declare const ClinicalPathwaysActionMenu_base: any;
 export declare class ClinicalPathwaysActionMenu extends ClinicalPathwaysActionMenu_base {
     constructor(icon: string, label: string);

package/dist/primary/shell/src/UI/components/communication-action-menu/communication-action-menu.d.ts

@@ -1,5 +1,5 @@
-import { PropertyValues } from 'lit';
 import { IRegion } from '@uxland/regions';
+import { PropertyValues } from 'lit';
 declare const CommunicationActionMenu_base: any;
 export declare class CommunicationActionMenu extends CommunicationActionMenu_base {
     constructor(icon: string, label: string);

package/dist/primary/shell/src/UI/components/petitioner-action-menu/petitioner-action-menu.d.ts

@@ -1,5 +1,5 @@
-import { PropertyValues } from 'lit';
 import { IRegion } from '@uxland/regions';
+import { PropertyValues } from 'lit';
 declare const PetitionerActionMenu_base: any;
 export declare class PetitionerActionMenu extends PetitionerActionMenu_base {
     constructor(icon: string, label: string);

package/dist/primary/shell/src/UI/components/primaria-shell/primaria-shell.d.ts

@@ -24,6 +24,5 @@ export declare class PrimariaShell extends PrimariaShell_base {
         message: string;
     }): void;
     _unsubscribeEvents(): void;
-    _scrollToNavItem(navItemMenuKey: string): Promise<void>;
 }
 export {};

package/dist/primary/shell/src/api/broker/broker-events.d.ts

@@ -21,8 +21,6 @@ export declare const BROKER_EVENTS: {
         readonly refreshTokenFailed: "refreshTokenFailed";
         readonly mpidHeaderInvalid: "mpidHeaderInvalid";
         readonly quickActionBusyChanged: "quickActionBusyChanged";
-        readonly scrollToNavItemRequested: "scrollToNavItemRequested";
-        readonly scrollToNavItemCompleted: "scrollToNavItemCompleted";
     };
     readonly administrativeManagement: {
         readonly tabSelected: "administrative_management_tab_selected";

package/dist/primary/shell/src/api/token-manager/token-manager.d.ts

@@ -1,10 +1,23 @@
 import { PrimariaBroker } from '../broker/primaria-broker';
+/**
+ * API pública de tokens disponible per als plugins via `api.tokenManager`.
+ * Els plugins poden llegir el token actual o forçar-ne el refresc; NO han de
+ * fixar els tokens inicials (això és una responsabilitat del host de bootstrap
+ * — vegeu {@link InternalTokenManager}).
+ */
 export interface TokenManager {
-    setInitialTokens: (access_token: string, refresh_token: string) => void;
     getToken: () => string;
     refreshToken: () => Promise<string>;
 }
-export declare class TokenManagerImpl implements TokenManager {
+/**
+ * Extensió host-only de {@link TokenManager}. Només el host del shell
+ * (`primary/app`) usa `setInitialTokens` durant el bootstrap; és deliberadament
+ * absent del tipus `TokenManager` que veuen els plugins.
+ */
+export interface InternalTokenManager extends TokenManager {
+    setInitialTokens: (access_token: string, refresh_token: string) => void;
+}
+export declare class TokenManagerImpl implements InternalTokenManager {
     private readonly broker;
     constructor(broker: PrimariaBroker);
     getUrlParams: () => URLSearchParams;

package/dist/primary/shell/src/index.d.ts

@@ -2,7 +2,9 @@ export * from './bootstrapper';
 export * from './disposer';
 export * from './api/region-manager/regions';
 export * from './handle-plugins';
-export * from './api/api';
+export type { PrimariaApi } from './api/api';
+export { PrimariaRegionHost, shellApi } from './api/api';
+export type { TokenManager, InternalTokenManager } from './api/token-manager/token-manager';
 export * from './api/broker/primaria-broker';
 export type { PrimariaContextManager, EcapContext } from './api/context-manager/context-manager';
 export * from './UI/index';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxland/primary-shell",
-  "version": "7.46.2",
+  "version": "7.46.4",
   "description": "Primaria Shell",
   "author": "UXLand <dev@uxland.es>",
   "homepage": "https://github.com/uxland/harmonix/tree/app#readme",

package/src/UI/components/clinical-pathways-action-menu/clinical-pathways-action-menu.ts

@@ -1,9 +1,10 @@
-import { LitElement, html, css, unsafeCSS, PropertyValues } from "lit";
-import { template } from "./template";
-import { property, state } from "lit/decorators.js";
 import { IRegion, region } from "@uxland/regions";
-import { PrimariaRegionHost, shellApi } from "../../../api/api";
+import { LitElement, PropertyValues, css, html, unsafeCSS } from "lit";
+import { property, state } from "lit/decorators.js";
+import { PrimariaRegionHost } from "../../../api/api";
+import { shellRegions } from "../../../api/region-manager/regions";
 import styles from "./styles.css?inline";
+import { template } from "./template";
 
 export class ClinicalPathwaysActionMenu extends PrimariaRegionHost(LitElement) {
   constructor(icon: string, label: string) {
@@ -14,7 +15,7 @@ export class ClinicalPathwaysActionMenu extends PrimariaRegionHost(LitElement) {
 
   @state() showText = false;
 
-  @region({ targetId: "clinical-pathways-sidenav-region-container", name: shellApi.regionManager.regions.shell.clinicalPathwaysSidenav })
+  @region({ targetId: "clinical-pathways-sidenav-region-container", name: shellRegions.clinicalPathwaysSidenav })
   clinicalPathwaysSidenavRegion: IRegion | undefined;
 
   static styles = css`

package/src/UI/components/communication-action-menu/communication-action-menu.ts

@@ -1,9 +1,10 @@
-import { LitElement, html, css, unsafeCSS, PropertyValues } from "lit";
-import { template } from "./template";
-import { property, state } from "lit/decorators.js";
 import { IRegion, region } from "@uxland/regions";
-import { PrimariaRegionHost, shellApi } from "../../../api/api";
+import { LitElement, PropertyValues, css, html, unsafeCSS } from "lit";
+import { property, state } from "lit/decorators.js";
+import { PrimariaRegionHost } from "../../../api/api";
+import { shellRegions } from "../../../api/region-manager/regions";
 import styles from "./styles.css?inline";
+import { template } from "./template";
 
 export class CommunicationActionMenu extends PrimariaRegionHost(LitElement) {
   constructor(icon: string, label: string) {
@@ -14,7 +15,7 @@ export class CommunicationActionMenu extends PrimariaRegionHost(LitElement) {
 
   @state() showText = false;
 
-  @region({ targetId: "communication-sidenav-region-container", name: shellApi.regionManager.regions.shell.communicationSidenav })
+  @region({ targetId: "communication-sidenav-region-container", name: shellRegions.communicationSidenav })
   comminucationSidenavRegion: IRegion | undefined;
 
   static styles = css`

package/src/UI/components/petitioner-action-menu/petitioner-action-menu.ts

@@ -1,9 +1,10 @@
-import { LitElement, html, css, unsafeCSS, PropertyValues } from "lit";
-import { template } from "./template";
-import { property, state } from "lit/decorators.js";
 import { IRegion, region } from "@uxland/regions";
-import { PrimariaRegionHost, shellApi } from "../../../api/api";
+import { LitElement, PropertyValues, css, html, unsafeCSS } from "lit";
+import { property, state } from "lit/decorators.js";
+import { PrimariaRegionHost } from "../../../api/api";
+import { shellRegions } from "../../../api/region-manager/regions";
 import styles from "./styles.css?inline";
+import { template } from "./template";
 
 export class PetitionerActionMenu extends PrimariaRegionHost(LitElement) {
   constructor(icon: string, label: string) {
@@ -14,7 +15,7 @@ export class PetitionerActionMenu extends PrimariaRegionHost(LitElement) {
 
   @state() showText = false;
 
-  @region({ targetId: "petitioner-sidenav-region-container", name: shellApi.regionManager.regions.shell.petitionerSidenav })
+  @region({ targetId: "petitioner-sidenav-region-container", name: shellRegions.petitionerSidenav })
   petitionerSidenavRegion: IRegion | undefined;
 
   static styles = css`

package/src/UI/components/primaria-shell/primaria-shell.ts

@@ -1,14 +1,14 @@
+import { BrokerDisposableHandler } from "@uxland/harmonix";
 import { IRegion, region } from "@uxland/regions";
 import { LitElement, css, html, unsafeCSS } from "lit";
 import { state } from "lit/decorators.js";
 import { PrimariaRegionHost, shellApi } from "../../../api/api";
-import { disposeShell } from "../../../disposer";
 import { BROKER_EVENTS } from "../../../api/broker/broker-events";
+import { disposeShell } from "../../../disposer";
 import { translate } from "../../../locales";
 import { shellViews } from "./constants";
 import styles from "./styles.css?inline";
 import { template } from "./template";
-import { BrokerDisposableHandler } from "@uxland/harmonix";
 
 export class PrimariaShell extends PrimariaRegionHost(LitElement) {
   render() {
@@ -89,12 +89,6 @@ export class PrimariaShell extends PrimariaRegionHost(LitElement) {
         this.quickActionBusy = detail.busy;
       }),
     );
-
-    this.subscriptions.push(
-      shellApi.broker.subscribe(BROKER_EVENTS.shell.scrollToNavItemRequested, (navItemMenuKey: string) => {
-        this._scrollToNavItem(navItemMenuKey);
-      }),
-    );
   }
 
   _handleError(error: { message: string }) {
@@ -106,88 +100,4 @@ export class PrimariaShell extends PrimariaRegionHost(LitElement) {
   _unsubscribeEvents() {
     this.subscriptions.forEach((s) => s.dispose());
   }
-
-  async _scrollToNavItem(navItemMenuKey: string) {
-    const region = await shellApi.regionManager.getRegion(shellApi.regionManager.regions.shell.navigationMenu);
-
-    const allViews = region.currentActiveViews;
-
-    // Extract the actual view id from the key (remove plugin prefix)
-    // navItemMenuKey comes as "primaria-shell::pdf-viewer", we need just "pdf-viewer"
-    const viewId = navItemMenuKey.includes("::") ? navItemMenuKey.split("::")[1] : navItemMenuKey;
-
-    const targetView = allViews.find((view: any) => view.id === viewId);
-
-    if (!targetView) {
-      shellApi.broker.publish(BROKER_EVENTS.shell.scrollToNavItemCompleted, {
-        scrollTop: 0,
-        containerTop: 0,
-        itemIndex: -1,
-      });
-      return;
-    }
-
-    // Sort views by sortHint to find the correct position
-    const sortedViews = [...allViews].sort((a: any, b: any) => {
-      const sortHintA = a.sortHint || "999";
-      const sortHintB = b.sortHint || "999";
-      return sortHintA.localeCompare(sortHintB);
-    });
-
-    const targetIndex = sortedViews.findIndex((view: any) => view.id === viewId);
-
-    if (targetIndex === -1) {
-      shellApi.broker.publish(BROKER_EVENTS.shell.scrollToNavItemCompleted, {
-        scrollTop: 0,
-        containerTop: 0,
-        itemIndex: -1,
-      });
-      return;
-    }
-
-    const menuContainer = this.shadowRoot?.querySelector("#menu-region-container") as HTMLElement;
-    if (!menuContainer) {
-      shellApi.broker.publish(BROKER_EVENTS.shell.scrollToNavItemCompleted, {
-        scrollTop: 0,
-        containerTop: 0,
-        itemIndex: targetIndex,
-      });
-      return;
-    }
-
-    const itemHeight = 51;
-    const targetPosition = itemHeight * targetIndex;
-    const containerHeight = menuContainer.clientHeight;
-
-    // Scroll to position the item in the middle of the container
-    const scrollPosition = targetPosition - containerHeight / 2 + itemHeight / 2;
-    menuContainer.scrollTo({
-      top: Math.max(0, scrollPosition),
-      behavior: "smooth",
-    });
-
-    // Wait for scroll to complete and then publish the completed event
-    setTimeout(() => {
-      const containerRect = menuContainer.getBoundingClientRect();
-
-      // Get all nav items from the container
-      const navItems = Array.from(menuContainer.children) as HTMLElement[];
-
-      // Find the actual DOM element for the target item
-      let itemAbsoluteY = window.innerHeight / 2; // default fallback
-
-      if (navItems[targetIndex]) {
-        const itemRect = navItems[targetIndex].getBoundingClientRect();
-        itemAbsoluteY = itemRect.top;
-      }
-
-      const data = {
-        scrollTop: menuContainer.scrollTop,
-        containerTop: containerRect.top,
-        itemIndex: targetIndex,
-        itemAbsoluteY: itemAbsoluteY,
-      };
-      shellApi.broker.publish(BROKER_EVENTS.shell.scrollToNavItemCompleted, data);
-    }, 300);
-  }
 }

package/src/api/broker/broker-events.ts

@@ -21,8 +21,6 @@ export const BROKER_EVENTS = {
     refreshTokenFailed: "refreshTokenFailed",
     mpidHeaderInvalid: "mpidHeaderInvalid",
     quickActionBusyChanged: "quickActionBusyChanged",
-    scrollToNavItemRequested: "scrollToNavItemRequested",
-    scrollToNavItemCompleted: "scrollToNavItemCompleted",
   },
   administrativeManagement: {
     tabSelected: "administrative_management_tab_selected",

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

@@ -2,17 +2,31 @@ import axios from "axios";
 import { BROKER_EVENTS } from "../broker/broker-events";
 import { PrimariaBroker } from "../broker/primaria-broker";
 
+/**
+ * API pública de tokens disponible per als plugins via `api.tokenManager`.
+ * Els plugins poden llegir el token actual o forçar-ne el refresc; NO han de
+ * fixar els tokens inicials (això és una responsabilitat del host de bootstrap
+ * — vegeu {@link InternalTokenManager}).
+ */
 export interface TokenManager {
-  setInitialTokens: (access_token: string, refresh_token: string) => void;
   getToken: () => string;
   refreshToken: () => Promise<string>;
 }
 
+/**
+ * Extensió host-only de {@link TokenManager}. Només el host del shell
+ * (`primary/app`) usa `setInitialTokens` durant el bootstrap; és deliberadament
+ * absent del tipus `TokenManager` que veuen els plugins.
+ */
+export interface InternalTokenManager extends TokenManager {
+  setInitialTokens: (access_token: string, refresh_token: string) => void;
+}
+
 let token: string;
 let refreshToken: string;
 let tokenInitialized = false;
 
-export class TokenManagerImpl implements TokenManager {
+export class TokenManagerImpl implements InternalTokenManager {
   constructor(private readonly broker: PrimariaBroker) {}
 
   getUrlParams = (): URLSearchParams => {

package/src/features/utils.ts

@@ -3,10 +3,13 @@ import type { interfaces } from "inversify";
 import { IRequest, IRequestClass } from "..";
 import { PrimariaApi } from "../api/api";
 
-//TODO aixo ho fan ja les Broker extensions de plugins/core. Veure si s'ha de canviar
+// Helper intern del shell: registra una request al broker i binda el seu handler
+// al container d'Inversify. És el bessó del `registerRequest` de
+// `@primaria/plugins-core` (brokerExtensions), però es manté aquí a propòsit:
+// `plugins-core` depèn del shell (no al revés), i exposar aquest helper a l'API
+// pública del shell la bloataria innecessàriament.
 export const registerRequest =
-  (api: PrimariaApi, container: Container) =>
-  (command: IRequestClass<any>, handler: interfaces.Newable<any>) => {
+  (api: PrimariaApi, container: Container) => (command: IRequestClass<any>, handler: interfaces.Newable<any>) => {
     container.bind(handler).to(handler).inTransientScope();
     injectable()(handler);
     const disposable = api.broker.registerRequest(command, (cmd: IRequest<any>) => {

package/src/handle-plugins.ts

@@ -1,9 +1,5 @@
 import type { BootstrappedPlugin, PluginDefinition, Plugin as PluginType } from "@uxland/harmonix";
 import { bootstrapPlugins as pluginBootstrapper } from "@uxland/harmonix";
-import {
-  dispose as activityHistoryDispose,
-  initialize as activityHistoryInitialize,
-} from "../../../plugins/activity-history/src/plugin";
 import { PrimariaApi, primariaApiFactory } from "./api/api";
 export type { PluginDefinition, PluginInfo } from "@uxland/harmonix";
 
@@ -29,13 +25,13 @@ const CORE_FIRST_ID = "activity-history";
 // from the plugin store (versioned/uploadable) like any other plugin; this
 // bundled copy is the fallback used when the caller does not provide it — e.g.
 // third-party sandboxes that install primary-shell without a plugin store.
+//
+// The import stays inside the importer on purpose: importing it statically would
+// evaluate the plugin's modules on shell load and register its custom elements,
+// so the copy coming from the store would fail to define them again.
 const bundledActivityHistory: PluginDefinition = {
   pluginId: CORE_FIRST_ID,
-  importer: () =>
-    Promise.resolve({
-      initialize: activityHistoryInitialize,
-      dispose: activityHistoryDispose,
-    }),
+  importer: () => import("../../../plugins/activity-history/src/plugin"),
 };
 
 export const bootstrapPlugins = async (plugins: PluginDefinition[], startup?: PrimariaStartupPlugin) => {

package/src/index.ts

@@ -12,7 +12,15 @@ export * from "./bootstrapper";
 export * from "./disposer";
 export * from "./api/region-manager/regions";
 export * from "./handle-plugins";
-export * from "./api/api";
+// Superfície pública explícita: `primariaApiFactory`, `activityMonitor` i
+// `sessionRefreshTimer` són singletons interns del shell i no s'exposen.
+// `shellApi` es manté només perquè els hosts (app/sandbox) l'usen per arrencar;
+// els plugins han d'usar l'`api` que reben a `initialize(api)`.
+export type { PrimariaApi } from "./api/api";
+export { PrimariaRegionHost, shellApi } from "./api/api";
+// `TokenManager` és el tipus públic per a plugins; `InternalTokenManager`
+// (amb `setInitialTokens`) és host-only, per al bootstrap de `primary/app`.
+export type { TokenManager, InternalTokenManager } from "./api/token-manager/token-manager";
 export * from "./api/broker/primaria-broker";
 export type { PrimariaContextManager, EcapContext } from "./api/context-manager/context-manager";
 export * from "./UI/index";