Switch to Terser library for minification.

.github/workflows/ci.yml

@@ -19,6 +19,10 @@ jobs:
         run: npm ci --timing
       - name: Lint javascript
         run: npm run lint
+      - name: Verify minification uses script
+        run: |
+          npm run minify
+          git diff --exit-code Universal-Federated-Analytics-Min.js Universal-Federated-Analytics-Min.js.map
       - name: Run tests
         # aa-exec applies Ubuntu's AppArmor profile for stable Chrome binaries to the Chrome for Testing binary downloaded by Puppeteer
         # Without it, Puppeteer fails with error "No usable sandbox!"

Dockerfile

@@ -1,6 +1,6 @@
 FROM nginx
 ARG DAP_ENV='dev'
 ENV DAP_ENV=${DAP_ENV}
-COPY test_site Universal-Federated-Analytics-Min.js Universal-Federated-Analytics.js Federated.js.map /usr/share/nginx/html/
+COPY test_site Universal-Federated-Analytics.js Universal-Federated-Analytics-Min.js Universal-Federated-Analytics-Min.js.map /usr/share/nginx/html/
 COPY nginx-test.conf.template /etc/nginx/conf.d/
 RUN envsubst '${DAP_ENV}' < /etc/nginx/conf.d/nginx-test.conf.template > /etc/nginx/conf.d/default.conf

README.md

@@ -14,7 +14,7 @@ The latest version 8.6 of DAP GA code contains GA4 tracking only. DAP UA data co
 
 * [`Universal-Federated-Analytics.js`](Universal-Federated-Analytics.js) (full)
 * [`Universal-Federated-Analytics-Min.js`](Universal-Federated-Analytics-Min.js) (minified)
-* [`Federated.js.map`](Federated.js.map) (source map)
+* [`Universal-Federated-Analytics-Min.js.map`](Universal-Federated-Analytics-Min.js.map) (source map)
 
 ### DAP Code Implementation
 
@@ -129,6 +129,14 @@ This repo uses Eslint for code static analysis. Run the linter with:
 npm run lint
 ```
 
+#### Minification
+
+This repo uses Terser for minification. After making changes to Universal-Federated-Analytics.js, regenerate the minified file by running:
+
+```bash
+npm run minify
+```
+
 #### Run integration tests
 
 Start up the test site at http://localhost:8080/ with one of the following:

package.json

@@ -7,6 +7,7 @@
   "scripts": {
     "clean": "rm ./Universal-Federated-Analytics-Min.js",
     "lint": "eslint Universal-Federated-Analytics.js",
+    "minify": "terser Universal-Federated-Analytics.js --output Universal-Federated-Analytics-Min.js --config-file terser.config.json",
     "cucumber": "cucumber-js",
     "cucumber:debug": "VERBOSE=true node --inspect-brk node_modules/@cucumber/cucumber/bin/cucumber-js",
     "cucumber:report": "cucumber-js --format \"html\":\"output/test-results.html\"",
@@ -29,7 +30,8 @@
     "eslint": "^9.8.0",
     "eslint-plugin-compat": "^6.0.0",
     "eslint-plugin-jsdoc": "^50.2.2",
-    "puppeteer": "^23.3.0"
+    "puppeteer": "^23.3.0",
+    "terser": "^5.46.1"
   },
   "dependencies": {
     "web-vitals": "^4.2.4"

terser.config.json

@@ -0,0 +1,9 @@
+{
+  "sourceMap": {
+    "filename": "Universal-Federated-Analytics-Min.js",
+    "url": "Universal-Federated-Analytics-Min.js.map"
+  },
+  "format": {
+    "max_line_len": 500
+  }
+}