Lewati ke konten utama

Code Review Checklist

Checklist yang wajib digunakan reviewer saat melakukan code review.

General​

  • Code mengikuti naming convention yang sudah ditetapkan
  • Tidak ada commented-out code yang tertinggal
  • Tidak ada hardcoded string/value (gunakan constants)
  • Tidak ada TODO tanpa ticket reference
  • Tidak ada println/debugPrint/console.log yang tertinggal

Architecture​

  • Mengikuti Clean Architecture (separation of layers)
  • ViewModel/Cubit tidak import framework UI
  • Business logic tidak ada di View/Screen
  • Dependency injection digunakan (bukan direct instantiation)
  • Repository menggunakan interface/protocol

Error Handling​

  • Semua API call memiliki error handling
  • Error ditampilkan ke user dengan pesan yang jelas
  • Tidak ada empty catch block
  • Loading state ditangani dengan benar

Performance​

  • Tidak ada unnecessary recomposition/rebuild
  • Image di-cache dan di-resize sesuai kebutuhan
  • List menggunakan lazy loading / pagination
  • Tidak ada network call di main thread

Security​

  • Sensitive data tidak di-log
  • Token/credential disimpan di secure storage
  • API key tidak hardcoded di source code
  • Input divalidasi sebelum diproses

Testing​

  • Unit test ditambahkan untuk logic baru
  • Edge cases ditangani dan ditest
  • Test coverage tidak menurun

UI/UX​

  • Responsive di berbagai ukuran layar
  • Loading state & empty state ditangani
  • Accessible (content description, semantic labels)
  • Tidak ada hardcoded color/size (gunakan theme)