Skip to content

S8: perbaikan audit akhir UI + dokumen - #108

Merged
coderbuzz merged 1 commit into
mainfrom
feat/ui-s8-audit
Oct 15, 2026
Merged

coderbuzz merged 1 commit into
mainfrom
feat/ui-s8-audit

Conversation

@coderbuzz

Copy link
Copy Markdown
Owner

Ringkasan

S8: perbaikan temuan audit akhir (A1..A14) + pembaruan dokumen.

  • A1 id="app-form" dan getElementById di apps_add (Save as Draft, Save & Redeploy, Move app tidak lagi men-submit form Sign out); fallback requestSubmit. Langkah permanen di tests/e2e/ui.mjs (auth aktif): Save as Draft, Save & Redeploy, Move app; semua berakhir di /apps/<id>.
  • A2 error.html: ui_alert danger (role=alert) + Back (history.back(), fallback ke .) + Go Home.
  • A3/A4 SPEC.md (struktur file, §UI Style Guide ringkas → docs/DESIGN-SYSTEM.md), AGENTS.md, README (Appearance 3 palet, tab bar, lazy-load; klaim lama dihapus). README tidak punya screenshot UI lama.
  • A5 hasil prune dirender di servers_detail (toast role=status atau ui_alert danger "Prune failed"); flashMsg di app handler memang sudah url.QueryEscape (tidak diubah).
  • A6 sheet Prune dan Move app berbahasa Inggris.
  • A7 servers: bar jadi latar sel (.bar-cell, --w) + sub-teks data-sub (core/MB/GB) tanpa node tambahan: 10 node/baris (sebelum 14); chip Error; apps: chip Deploying, kolom Domain bisa di-sort; tanpa <progress>.
  • A8 static/console.js (satu fabrik konsol untuk server dan app, enc di luar onopen), dimuat di dua halaman detail; semua <script src> dan xterm dinamis diberi ?v=.
  • A9 Copy export env dan passphrase memakai data-copy (fallback execCommand + "Copied"); 4 alert() → showToast (grep alert( = 0).
  • A10 /export dan /import aktif di tab Settings; breadcrumb → Settings. A11 dockifyApplyTheme memanggil syncAppearance. A12 tanpa "⚠"; "Reset filter".
  • A13 ui.mjs menolak BASE non-lokal dan menghapus data ZZ Uji di finally. A14 CSS mati/duplikat dihapus (.live-badge dasar, blink-pulse, progress.bar, .secret-row ganda, .stats-header/.card h3 ganda, radius 6 dasar tombol); content-visibility:auto pada baris daftar.

Bukti

  • build/vet/test hijau; grep -rn "alert(" templates = 0; <progress = 0; app.css 871 baris.
  • UI_USER=admin UI_PASS=uji123 node tests/e2e/ui.mjs (auth aktif): exit 0 termasuk 3 langkah A1; overflow 0; kontras 19/19 ×6; a11y bersih. Navigasi (CPU 4x, +80 ms): / 302, /servers 300, /apps 303, /apps/1 330 ms DCL. Transfer dingin: / 126 KB.
  • Hasil prune: ?msg=Prune+error… → "Prune failed" (alert), ?msg=Worker disk prune completed → toast.
  • Remove server dengan app (409) menampilkan alert + Back + Go Home (screenshot di laporan).

Penyimpangan (diterima leader)

Roll secret / Redeploy All / Update now di luar Danger zone (halaman tanpa zona bahaya; outline merah + konfirmasi); animasi sheet-pop tanpa animasi keluar; webhook secret dan AuthPass di atribut value (ada sebelum revamp; tugas terpisah). Tambahan saya: bar di sel servers memakai latar sel (bukan div.bar>span) demi ≤12 node/baris; console.js memuat kedua konsol (server dan app) dalam satu berkas.

🤖 Generated with Claude Code

…lists, docs)

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderbuzz
coderbuzz merged commit 0de66cf into main Oct 15, 2026
2 checks passed
@coderbuzz
coderbuzz deleted the feat/ui-s8-audit branch October 15, 2026 14:08
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