Skip to content

fix(actions): remove explicit any type annotations, fix type safety i… - #7

Closed
AlrafuruNotFound wants to merge 4 commits into
mainfrom
dev
Closed

AlrafuruNotFound wants to merge 4 commits into
mainfrom
dev

Conversation

@AlrafuruNotFound

Copy link
Copy Markdown
Collaborator

…n server actions, and resync rules

📋 Deskripsi Perubahan

🌿 Jenis Branch

  • be/* — Backend (Server Actions, Prisma, lib)
  • fe/* — Frontend (components, pages, styles)
  • dev/* — Integration (merge be + fe)
  • hotfix/* — Hotfix kritis
  • docs/* — Dokumentasi saja

✅ ANF Docs-Sync Law Checklist

Wajib untuk SEMUA PR:

  • .docs/CHANGELOG.md sudah diupdate dengan entri baru
  • Tidak ada commit langsung ke main

Jika ada file baru dibuat:

  • File terdaftar di project-manifest.md → Active Inventory

Jika ada route baru:

  • Route tercantum di architecture.md → Section 7 Routing

Jika ada perubahan arsitektur/pattern:

  • architecture.md diupdate di section yang relevan

🔧 Backend (be/*) Checklist

  • 'use server' ada di baris pertama setiap action file
  • Zod schema didefinisikan untuk semua input
  • Return type konsisten: { success, data?, message? }
  • revalidatePath() dipanggil setelah setiap operasi mutasi
  • Tidak ada query prisma.* langsung di luar actions/ atau lib/
  • Panggilan verifySession() atau verifySuperAdmin() dari lib/safe-action.ts di awal Server Action administratif untuk autentikasi sesi JWT terpusat

🎨 Frontend (fe/*) Checklist

  • export const dynamic = "force-dynamic" di setiap page yang mengambil data
  • Tidak ada fetch() langsung ke database dari Client Component
  • Props interface TypeScript didefinisikan ("Hole Pattern")
  • Responsif: sudah ditest di lebar 375px, 768px, 1024px
  • Tidak ada console.log yang tertinggal

🔐 Security Checklist

  • Lolos verifikasi scripts/security-validator.js via Git pre-commit hook secara lokal (tidak ada kebocoran kredensial)
  • Tidak ada secret/credential hardcoded di kode
  • Semua input divalidasi dengan Zod
  • Tidak ada NEXT_PUBLIC_* env var yang berisi data sensitif

📸 Screenshot (wajib jika ada perubahan UI)

Sebelum Sesudah
(tempel screenshot) (tempel screenshot)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant