use terser minifier

Author: fadi-george

package-lock.json

@@ -65,6 +65,7 @@
     "prettier": "3.6.2",
     "prettylint": "^2.0.0",
     "sass-embedded": "^1.89.2",
+    "terser": "^5.43.1",
     "type-fest": "^4.41.0",
     "typescript": "^5.8.3",
     "vite": "^7.0.6",
@@ -76,17 +77,17 @@
   "size-limit": [
     {
       "path": "./build/releases/OneSignalSDK.page.js",
-      "limit": "490 B",
+      "limit": "450 B",
       "gzip": true
     },
     {
       "path": "./build/releases/OneSignalSDK.page.es6.js",
-      "limit": "54.812 kB",
+      "limit": "52.245 kB",
       "gzip": true
     },
     {
       "path": "./build/releases/OneSignalSDK.sw.js",
-      "limit": "15.3 kB",
+      "limit": "14.563 kB",
       "gzip": true
     },
     {

package.json

@@ -1,7 +1,7 @@
 import { IS_SERVICE_WORKER, LOGGING } from '../utils/EnvVariables';
 
 export default class Log {
-  private static shouldLog(): boolean {
+  private static _shouldLog(): boolean {
     if (IS_SERVICE_WORKER)
       return !!(self as unknown as ServiceWorkerGlobalScope).shouldLog;
     try {
@@ -30,7 +30,7 @@ export default class Log {
 
   private static createLogMethod(consoleMethod: keyof Console) {
     return (...args: unknown[]): void => {
-      if (LOGGING || this.shouldLog() || consoleMethod === 'error') {
+      if (LOGGING || this._shouldLog() || consoleMethod === 'error') {
         (console[consoleMethod] as (...args: unknown[]) => void)(...args);
       }
     };

src/shared/libraries/Log.ts

@@ -1,8 +1,6 @@
 export const SLIDEDOWN_CSS_CLASSES = {
-  allowButton: 'onesignal-slidedown-allow-button',
   body: 'slidedown-body',
   buttonIndicatorHolder: 'onesignal-button-indicator-holder',
-  cancelButton: 'onesignal-slidedown-cancel-button',
   container: 'onesignal-slidedown-container',
   dialog: 'onesignal-slidedown-dialog',
   footer: 'slidedown-footer',

src/shared/models/PermissionPromptType.ts

@@ -2,9 +2,9 @@ import { Browser } from './constants';
 import type { BrowserValue } from './types';
 
 interface BrowserConfig {
-  name: string;
-  pattern: RegExp;
-  versionPattern: RegExp;
+  _name: string;
+  _pattern: RegExp;
+  _versionPattern: RegExp;
 }
 
 interface IBrowserResult {
@@ -15,67 +15,67 @@ interface IBrowserResult {
 // Top popular browsers set
 const BROWSER_CONFIGS: BrowserConfig[] = [
   {
-    name: 'Opera',
-    pattern: /(?:opera|opr|opios)/i,
-    versionPattern: /(?:opera|opr|opios)[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Opera',
+    _pattern: /(?:opera|opr|opios)/i,
+    _versionPattern: /(?:opera|opr|opios)[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Facebook',
-    pattern: /FBAN\//i,
-    versionPattern: /FBAV\/(\d+(?:\.\d+)?)/i,
+    _name: 'Facebook',
+    _pattern: /FBAN\//i,
+    _versionPattern: /FBAV\/(\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Samsung Browser',
-    pattern: /samsungbrowser/i,
-    versionPattern: /samsungbrowser[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Samsung Browser',
+    _pattern: /samsungbrowser/i,
+    _versionPattern: /samsungbrowser[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Yandex Browser',
-    pattern: /yabrowser/i,
-    versionPattern: /yabrowser[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Yandex Browser',
+    _pattern: /yabrowser/i,
+    _versionPattern: /yabrowser[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Vivaldi',
-    pattern: /vivaldi/i,
-    versionPattern: /vivaldi[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Vivaldi',
+    _pattern: /vivaldi/i,
+    _versionPattern: /vivaldi[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'UC Browser',
-    pattern: /ucbrowser/i,
-    versionPattern: /ucbrowser[ /](\d+(?:\.\d+)?)/i,
+    _name: 'UC Browser',
+    _pattern: /ucbrowser/i,
+    _versionPattern: /ucbrowser[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Microsoft Edge',
-    pattern: /edg/i,
-    versionPattern: /edg[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Microsoft Edge',
+    _pattern: /edg/i,
+    _versionPattern: /edg[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Firefox',
-    pattern: /firefox|iceweasel|fxios/i,
-    versionPattern: /(?:firefox|iceweasel|fxios)[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Firefox',
+    _pattern: /firefox|iceweasel|fxios/i,
+    _versionPattern: /(?:firefox|iceweasel|fxios)[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Chromium',
-    pattern: /chromium/i,
-    versionPattern: /chromium[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Chromium',
+    _pattern: /chromium/i,
+    _versionPattern: /chromium[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Chrome',
-    pattern: /chrome|crios|crmo/i,
-    versionPattern: /(?:chrome|crios|crmo)[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Chrome',
+    _pattern: /chrome|crios|crmo/i,
+    _versionPattern: /(?:chrome|crios|crmo)[ /](\d+(?:\.\d+)?)/i,
   },
   {
-    name: 'Safari',
-    pattern: /safari|applewebkit/i,
-    versionPattern: /version[ /](\d+(?:\.\d+)?)/i,
+    _name: 'Safari',
+    _pattern: /safari|applewebkit/i,
+    _versionPattern: /version[ /](\d+(?:\.\d+)?)/i,
   },
 ];
 
 export function getBrowser(userAgent: string): IBrowserResult {
   for (const config of BROWSER_CONFIGS) {
-    if (config.pattern.test(userAgent)) {
-      const version = userAgent.match(config.versionPattern)?.[1] ?? '';
-      return { name: config.name, version };
+    if (config._pattern.test(userAgent)) {
+      const version = userAgent.match(config._versionPattern)?.[1] ?? '';
+      return { name: config._name, version };
     }
   }
   return { name: 'Unknown', version: '' };

src/shared/models/WindowEnvironmentKind.ts

@@ -60,7 +60,16 @@ export default defineConfig(({ mode }) => {
        * `var It=Object.defineProperty;` above the IIFE.
        */
       target: 'es2022',
-      minify: isProdEnv,
+      minify: isProdEnv ? 'terser' : false,
+      terserOptions: {
+        mangle: {
+          properties: {
+            // Enable property name mangling/minification for private-style properties (starting with _)
+            regex: /^_/,
+          },
+        },
+      },
+
       lib: {
         ...libConfig[lib],
         name: 'OneSignal',