Skip to content

fix: address high-priority Nexora audit findings - #21

Merged
fabrianivan merged 1 commit into
mainfrom
feat/high-priority-audit-remediation
Oct 20, 2026
Merged

fabrianivan merged 1 commit into
mainfrom
feat/high-priority-audit-remediation

Conversation

@fabrianivan

Copy link
Copy Markdown
Owner

Ringkasan

  • Hubungkan state perusahaan UI ke /api/workspace agar dapat dimuat dan disimpan ke MySQL; pertahankan data workspace lama yang memakai schema ternormalisasi.
  • Tambahkan sesi server bertanda tangan dengan cookie HttpOnly/SameSite, akses API terlindungi, validasi payload, batas permintaan dasar, dan security headers.
  • Redaksi field identitas demo dan ubah skor/status dokumen, benchmark, scanner, billing, persona RBAC, serta jawaban fallback Copilot agar tidak mengklaim verifikasi yang tidak dilakukan.
  • Tambahkan navigasi utama mobile, perbarui tes E2E, dan koreksi README/.env.example agar sesuai dengan perilaku saat ini.

Verifikasi

  • npm run lint — lulus
  • npx tsc --noEmit — lulus
  • npm test — 10 tes lulus
  • npm run build — lulus
  • npm run test:e2e — 4 tes lulus dengan token sementara khusus tes
  • git diff --check — lulus

Batasan yang masih ada

  • Auth sekarang memakai satu token operator bersama. Belum ada akun per pengguna, peran server individual, atau isolasi tenant.
  • Persistensi MySQL sudah terhubung pada UI, tetapi koneksi dan round-trip data belum diuji terhadap server MySQL nyata di lingkungan ini.
  • Scanner tetap simulasi. Keluaran Gemini, bila dikonfigurasi, tidak memiliki retrieval/sitasi dan tetap ditandai belum terverifikasi.
  • Tarif dan klaim pajak contoh perlu tinjauan ahli pajak. PERLU VERIFIKASI AHLI PAJAK.
  • Lighthouse mobile/desktop belum dijalankan karena CLI tidak tersedia di lingkungan ini.

@fabrianivan
fabrianivan merged commit 64a8254 into main Oct 20, 2026
1 check passed
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