@uxland/primary-shell 7.46.12 → 7.46.14

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/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;
+    }
+  }
+}