JavaScript ES Modules: Import, Export & Tree Shaking

// ====== 1. SELALU gunakan file extension (.js) ======
// Beberapa bundler membutuhkan eksplicit extension
import { helper } from './utils/helper.js';  // ✅
import { helper } from './utils/helper';      // ❌ bisa error di Deno/native ESM

// ====== 2. Hindari circular dependencies ======
// a.js mengimpor b.js, b.js mengimpor a.js → masalah!
// Solusi: pindahkan shared code ke modul ketiga (c.js)

// ====== 3. Satu ekspor per file (Single Responsibility) ======
// ✅ Bagus: file fokus pada satu hal
// UserService.js — hanya tentang User Service
export class UserService {
  async getUser(id) { /* ... */ }
  async updateUser(id, data) { /* ... */ }
  async deleteUser(id) { /* ... */ }
}

// ❌ Kurang baik: file menangani banyak hal berbeda
// utils.js — terlalu banyak ekspor berbeda
export function formatRupiah() { /* ... */ }
export function validateEmail() { /* ... */ }
export function sendNotification() { /* ... */ }
export class DatabaseService { /* ... */ }
export const API_URL = '...';

// ====== 4. Gunakan constants terpisah ======
// constants/routes.js
export const ROUTES = {
  HOME: '/',
  LOGIN: '/login',
  DASHBOARD: '/dashboard',
  PROFILE: '/profile',
  SETTINGS: '/settings',
};

export const API_ENDPOINTS = {
  USERS: '/api/users',
  PRODUCTS: '/api/products',
  ORDERS: '/api/orders',
};

// ====== 5. Type exports (untuk TypeScript/JSDoc) ======
/** @typedef {{ id: number, nama: string, email: string }} User */

// Export type sebagai value (untuk JSDoc runtime checking)
export const UserType = Object.freeze({
  id: 'number',
  nama: 'string',
  email: 'string',
});

// ❌ 1. Mutable exports — bisa menyebabkan bug tak terduga
export let counter = 0;
export function increment() { counter++; }
// Importer mendapat binding langsung, bukan copy!

// ✅ Solusi: Gunakan getter pattern
let _counter = 0;
export function getCounter() { return _counter; }
export function increment() { _counter++; }

// ❌ 2. Import dengan side effects yang tidak disadari
import './analytics'; // Apa yang dilakukan modul ini?

// ✅ Solusi: Jelaskan mengapa Anda mengimpor side-effect module
// Load analytics untuk tracking page views
import './analytics';

// ❌ 3. Deep import dari internal package
import helper from 'some-package/lib/internal/utils/helper.js';

// ✅ Solusi: Gunakan public API
import { helper } from 'some-package';

// ❌ 4. Barrel file yang terlalu besar
// components/index.js — re-export 100+ komponen!
export { Button } from './Button';
export { Modal } from './Modal';
// ... 98 lagi

// ✅ Solusi: Sub-barrel files
// components/forms/index.js — hanya form components
// components/layout/index.js — hanya layout components

// ❌ 5. Mix CommonJS dan ESM secara acak
const fs = require('fs');         // CommonJS
import path from 'path';          // ESM
// Di Node.js, ini bisa menyebabkan error!

// ✅ Solusi: Pilih satu, konsisten
import fs from 'node:fs/promises'; // ESM sepanjang jalan
import path from 'node:path';