[OPTIMIZATION] Build Config - Blocking Issues & CI/CD - vite.config.ts & package.json #65

New issue

Closed

opened 2026-06-06 20:48:23 +00:00 by Mark · 0 comments

Mark commented

2026-06-06 20:48:23 +00:00

Owner

Critical Issues (1)

Build fails: Mock/default export mismatch for dynamic imports
- vite.config.ts aliases @capacitor-community/call-log → ./src/test/mocks/call-log.ts
- call-log.ts exports named CallLog, but communication.ts expects default
- Fix: Add export default CallLog to mock files (also sms, email in capacitor.ts)

High Issues (1)

Missing sideEffects: false in package.json
- Fix: Add "sideEffects": false for aggressive tree-shaking
- Impact: 10-30% smaller vendor chunks

Medium Issues (2)

No compression plugin - production bundles served uncompressed
- Fix: Add vite-plugin-compression (gzip + brotli)
- Impact: 60-80% smaller transfer size
No CI/CD pipeline with caching - every build runs npm ci from scratch
- Fix: GitHub Actions with actions/cache for node_modules + Docker layers
- Impact: 50-80% faster CI

Low Issues (2)

console.error in production code (11 locations across 6 files)
- Fix: Wrap in if (!import.meta.env.PROD) or use esbuild.pure: ["console.error"]
Browserslist database 6 months outdated
- Fix: Run npx update-browserslist-db@latest

## Critical Issues (1) 1. **Build fails**: Mock/default export mismatch for dynamic imports - `vite.config.ts` aliases `@capacitor-community/call-log` → `./src/test/mocks/call-log.ts` - `call-log.ts` exports named `CallLog`, but `communication.ts` expects `default` - Fix: Add `export default CallLog` to mock files (also `sms`, `email` in `capacitor.ts`) ## High Issues (1) 2. **Missing `sideEffects: false`** in package.json - Fix: Add `"sideEffects": false` for aggressive tree-shaking - Impact: 10-30% smaller vendor chunks ## Medium Issues (2) 3. **No compression plugin** - production bundles served uncompressed - Fix: Add `vite-plugin-compression` (gzip + brotli) - Impact: 60-80% smaller transfer size 4. **No CI/CD pipeline with caching** - every build runs `npm ci` from scratch - Fix: GitHub Actions with `actions/cache` for node_modules + Docker layers - Impact: 50-80% faster CI ## Low Issues (2) 5. **`console.error` in production code** (11 locations across 6 files) - Fix: Wrap in `if (!import.meta.env.PROD)` or use `esbuild.pure: ["console.error"]` 6. **Browserslist database 6 months outdated** - Fix: Run `npx update-browserslist-db@latest`