@posiwise/common-services 0.1.91 → 0.1.92

package/fesm2022/posiwise-common-services.mjs

@@ -1353,7 +1353,7 @@ class PermissionService {
             // or Pages.Beta or Pages.Alpha
             if (permissionName?.includes('||')) {
                 const parts = permissionName.split('||').map(p => p.trim());
-                if (this.hasAnyGrantedPermission(parts, user, productSlug)) {
+                if (this.hasAnyGrantedPermission(parts, user)) {
                     return true;
                 }
             }
@@ -1370,12 +1370,8 @@ class PermissionService {
         // user not logged-in
         return false;
     }
-    hasAnyGrantedPermission(parts, user, productSlug = null) {
-        return parts.some(part => {
-            // Format the permission part with the current subscription slug if needed
-            const formattedPart = this.getFormattedPermissionName(part, productSlug);
-            return user['auth']?.['granted'][formattedPart];
-        });
+    hasAnyGrantedPermission(parts, user) {
+        return parts.some(part => user['auth']?.['granted'][part]);
     }
     handleProductKey(productKey, user, permission_key, permissionName) {
         if (productKey) {
@@ -1407,15 +1403,11 @@ class PermissionService {
         else {
             slugToCheck = productSlug;
         }
-        // Only format if we have a valid slug to check
-        if (slugToCheck) {
-            if (!permissionName.includes(slugToCheck) &&
-                permissionName.includes('Pages.Product.')) {
-                permissionName = permissionName.replace('Pages.Product.', slugToCheck);
-            }
-            if (permissionName.includes('Pages.Role.') && !permissionName.includes(slugToCheck)) {
-                permissionName = permissionName.replace('Pages.Role.', slugToCheck);
-            }
+        if (!permissionName.includes(slugToCheck) && permissionName.includes('Pages.Product.')) {
+            permissionName = permissionName.replace('Pages.Product.', slugToCheck);
+        }
+        if (permissionName.includes('Pages.Role.') && !permissionName.includes(slugToCheck)) {
+            permissionName = permissionName.replace('Pages.Role.', slugToCheck);
         }
         return permissionName;
     }
@@ -1500,8 +1492,7 @@ class PermissionService {
                         evaluated = this.isGranted(raw, productKey, permission_key, productSlug);
                     }
                     else {
-                        // Pass productSlug to isGranted to ensure slug-aware permission checking
-                        evaluated = this.isGranted(raw, null, null, productSlug);
+                        evaluated = this.isGranted(raw);
                     }
                     expr += ` ${evaluated} `;
                 }
@@ -2806,6 +2797,7 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "19.2.18", ngImpo
 
 class SentryErrorHandler {
     static { this.sentryInitialized = false; }
+    static { this.unhandledRejectionListenerAttached = false; }
     constructor(appConfigService) {
         this.appConfigService = appConfigService;
     }
@@ -2820,16 +2812,15 @@ class SentryErrorHandler {
             if (env?.integrations?.sentry_config) {
                 init({ ...config, ...env.integrations.sentry_config });
                 SentryErrorHandler.sentryInitialized = true;
-                // Add a global listener for unhandled promise rejections
-                window.addEventListener('unhandledrejection', (event) => {
-                    this.handlePromiseRejection(event);
-                });
+                // Add a global listener for unhandled promise rejections (once)
+                if (!SentryErrorHandler.unhandledRejectionListenerAttached) {
+                    window.addEventListener('unhandledrejection', (event) => {
+                        this.handlePromiseRejection(event);
+                    });
+                    SentryErrorHandler.unhandledRejectionListenerAttached = true;
+                }
             }
         });
-        // Add a global listener for unhandled promise rejections
-        window.addEventListener('unhandledrejection', (event) => {
-            this.handlePromiseRejection(event);
-        });
     }
     async getSentryConfigAsync() {
         return new Promise(resolve => {
@@ -2842,18 +2833,23 @@ class SentryErrorHandler {
                 // The previous integration (`BrowserTracing` + `routingInstrumentation`) was part of
                 // the legacy `@sentry/angular-ivy` API and isn't available in `@sentry/angular` v10.
                 beforeSend(event, hint) {
+                    const initialExceptionValuesLength = Array.isArray(event?.exception?.values)
+                        ? event.exception.values.length
+                        : undefined;
+                    const safeIncludes = (val, needle) => typeof val === 'string' && val.includes(needle);
                     // Check if the event is of type CloseEvent and ignore it
                     if (event?.exception?.values) {
                         event.exception.values = event.exception.values.filter(value => {
                             // Update the condition based on the actual structure of the event
-                            return !(value.type === 'CloseEvent' || value.value.includes('CloseEvent'));
+                            return !(value.type === 'CloseEvent' ||
+                                safeIncludes(value.value, 'CloseEvent'));
                         });
                     }
                     if (event?.exception?.values) {
                         event.exception.values = event.exception.values.filter(value => {
                             // Check for the specific error message and ignore it
                             return !(value.type === 'Error' &&
-                                value.value.includes('ResizeObserver loop completed'));
+                                safeIncludes(value.value, 'ResizeObserver loop completed'));
                         });
                     }
                     // originates from vendor.js file
@@ -2870,21 +2866,21 @@ class SentryErrorHandler {
                     if (event?.exception?.values) {
                         event.exception.values = event.exception.values.filter(value => {
                             return !(value.type === 'ChunkLoadError' ||
-                                value.value.includes('ChunkLoadError: Loading chunk'));
+                                safeIncludes(value.value, 'ChunkLoadError: Loading chunk'));
                         });
                     }
                     // Check if the event is a script loading error and ignore it
                     if (event?.exception?.values) {
                         event.exception.values = event.exception.values.filter(value => {
                             return !(value.type === 'Error' &&
-                                value.value.includes('Could not load "util"'));
+                                safeIncludes(value.value, 'Could not load "util"'));
                         });
                     }
                     // Check if the event is a timeout error and ignore it
                     if (event?.exception?.values) {
                         event.exception.values = event.exception.values.filter(value => {
                             return !(value.type === 'Error' &&
-                                value.value.includes('Error loading plotly.js library from'));
+                                safeIncludes(value.value, 'Error loading plotly.js library from'));
                         });
                     }
                     // Check if the event is related to cross-origin frame access
@@ -2898,16 +2894,6 @@ class SentryErrorHandler {
                             return null;
                         }
                     }
-                    // Check if the event has an exception and values array
-                    if (event?.exception?.values) {
-                        // Filter out events generated by TryCatch and GlobalHandlers
-                        event.exception.values = event.exception.values.filter(value => {
-                            const mechanism = value.mechanism;
-                            return !(mechanism &&
-                                (mechanism.type === 'TryCatch' ||
-                                    mechanism.type === 'GlobalHandlers'));
-                        });
-                    }
                     // Check if it's a non-error exception
                     const isNonErrorException = event?.exception?.values?.[0]?.value?.startsWith('Non-Error exception captured') ??
                         hint?.originalException?.message?.startsWith('Non-Error exception captured');
@@ -2915,6 +2901,14 @@ class SentryErrorHandler {
                         // We want to ignore those kinds of errors
                         return null;
                     }
+                    // If we started with exception values but filtered them all out, drop the event.
+                    // Otherwise Sentry will group it as an "unlabeled event" and spam the issue stream.
+                    if (typeof initialExceptionValuesLength === 'number' &&
+                        initialExceptionValuesLength > 0 &&
+                        Array.isArray(event?.exception?.values) &&
+                        event.exception.values.length === 0) {
+                        return null;
+                    }
                     return event;
                 },
                 denyUrls: [/drift.*\.js/i, /analytics.*\.js/i, /polyfills.*\.js/i, /vendor.*\.js/i],