Chore/ev5 0 cleanup tech debt

[hanchan-dev]

Summary

Melakukan cleanup dan perbaikan fondasi pada EV5-0, khususnya menghapus fitur revisi surat oleh warga yang sudah tidak menjadi bagian dari scope sistem serta memperbaiki konfigurasi assigned_role pada letter_types agar sesuai dengan TDD.

Perubahan dibuat tetap terbatas pada scope EV5-0 dan tidak mencakup rewrite besar pada approval flow atau migration Category+Flow.

Jira Ticket

Ticket: EV5-0

Type of Change

• Bug fix
• New feature
• Refactor
• Documentation update
• Configuration change

Changes Made

• Menghapus fitur revisi surat oleh warga:

  • Menghapus WaitingRevisionWarga dan RejectedRevision dari LetterStatus.
  • Menghapus logic revision_count dan branch needs_revision dari KasiApprovalService.
  • Menghapus endpoint resubmit() dari Api\LetterController.

• Memperbaiki AssignedRole:

  • Mengganti role rw menjadi kasi_pelayanan dan kaur_tu_umum.
  • Memperbarui ENUM letter_types.assigned_role pada migration.
  • Memperbarui LetterTypeSeeder agar tidak lagi menggunakan rw.

• Menambahkan dan memperbaiki unit/feature test untuk memastikan perubahan enum dan penghapusan fitur revisi berjalan sesuai scope.

• Tidak mengubah approval flow granular yang masih digunakan oleh sistem saat ini.

• Tidak memperbaiki bug assigned_role == 'rw' pada KasiApprovalService::getPendingLetters() karena perubahan tersebut merupakan scope EV5-4-S6.

• Tidak melakukan migration terpisah untuk menghapus revision_count, karena tabel letters akan di-fresh-migrate pada EV5-2.

How to Test

1. Install/update dependencies jika diperlukan:

   composer install

2. Jalankan test yang terkait:

   php artisan test tests/Unit/LetterStatusTest.php
   php artisan test tests/Unit/AssignedRoleEnumTest.php
   php artisan test tests/Unit/LetterControllerResubmitRemovedTest.php
   php artisan test tests/Feature/LetterTypeAssignedRoleTest.php

3. Jalankan seluruh test suite:

   php artisan test

4. Untuk memverifikasi migration dan seeder:

   php artisan migrate:fresh --seed

5. Pastikan proses migrate:fresh --seed berhasil dan tidak ada lagi seed assigned_role = 'rw'.

Screenshots (if applicable)

Tidak ada perubahan UI.

Checklist

• Kode sudah di-test secara lokal
• Tidak ada console.log / dd() / dump() yang tertinggal
• Tidak ada conflict dengan branch target
• Nama branch mengikuti konvensi
• PR ditujukan ke branch dev (bukan langsung ke main)