9router-imagefix 0.5.112 → 0.5.113

package/app/package.json

@@ -1,6 +1,6 @@
 {
   "name": "9router-app",
-  "version": "0.5.112",
+  "version": "0.5.113",
   "description": "9Router web dashboard",
   "private": true,
   "scripts": {

package/cli.js

@@ -571,6 +571,11 @@ if (skipKill) {
     .then(() => startServer(updatePromise));
 }
 
+// Logika menu tinggal di modul terpisah supaya bisa diuji tanpa menjalankan
+// CLI: cli.js ini mengeksekusi dirinya di module scope (startServer, bahkan
+// process.exit kalau standalone build tidak ada).
+const { buildInterfaceMenuItems, resolveMenuAction } = require("./src/cli/utils/interfaceMenu");
+
 // Show interface selection menu
 async function showInterfaceMenu(latestVersion) {
   const { selectMenu } = require("./src/cli/utils/input");
@@ -592,40 +597,10 @@ async function showInterfaceMenu(latestVersion) {
 
   const subtitle = `🚀 Server: \x1b[32m${serverUrl}\x1b[0m`;
 
-  const menuItems = [];
-
-  if (latestVersion) {
-    menuItems.push({ label: `Update to v${latestVersion} (current: v${pkg.version})`, icon: "⬆" });
-  }
-
-  menuItems.push(
-    { label: "Web UI (Open in Browser)", icon: "🌐" },
-    { label: "Terminal UI (Interactive CLI)", icon: "💻" },
-    { label: "Hide to Tray (Background)", icon: "🔔" },
-    { label: "Exit", icon: "🚪" }
-  );
-
+  const menuItems = buildInterfaceMenuItems(latestVersion, pkg.version);
   const selected = await selectMenu(`Choose Interface (v${pkg.version})`, menuItems, 0, subtitle);
 
-  // `selectMenu` mengembalikan -1 untuk dua hal yang SAMA SEKALI BUKAN "exit":
-  //   - ESC ditekan (input.js: resolve(-1))
-  //   - stdin bukan TTY, mis. CLI dijalankan dari skrip / terminal tanpa TTY
-  //
-  // Dulu keduanya jatuh ke `return "exit"` di bawah, dan pemanggilnya
-  // menjalankan cleanup() yang MEMBUNUH server. Akibatnya menekan ESC sekali
-  // saja sudah cukup untuk mematikan gateway — inilah "tiba-tiba berhenti".
-  // Sekarang: batalkan saja, jangan sentuh server.
-  if (selected === -1) return "back";
-
-  const offset = latestVersion ? 1 : 0;
-
-  if (latestVersion && selected === 0) return "update";
-  if (selected === offset) return "web";
-  if (selected === offset + 1) return "terminal";
-  if (selected === offset + 2) return "hide";
-  // Indeks tak dikenal: jangan pernah menafsirkannya sebagai perintah mematikan
-  // server. Lebih baik tampilkan menu lagi.
-  return "back";
+  return resolveMenuAction(selected, latestVersion, pkg.version);
 }
 
 const MAX_RESTARTS = 2;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "9router-imagefix",
-  "version": "0.5.112",
+  "version": "0.5.113",
   "description": "9Router CLI - Start and manage 9Router server (with image fix)",
   "bin": {
     "9router-imagefix": "./cli.js"

package/src/cli/utils/interfaceMenu.js

@@ -0,0 +1,73 @@
+/**
+ * Logika menu "Choose Interface" — dipisah dari cli.js supaya bisa diuji.
+ *
+ * cli.js menjalankan dirinya di module scope (startServer, bahkan
+ * process.exit kalau standalone build tidak ada), jadi mengimpornya dari test
+ * akan menjalankan CLI sungguhan. Fungsi-fungsi di sini murni: tidak menyentuh
+ * proses, jaringan, atau berkas.
+ */
+
+/**
+ * Susun daftar item menu.
+ *
+ * Setiap item membawa `action`-nya sendiri, dan aksi dibaca dari array ini —
+ * bukan dari konstanta offset yang dihitung manual.
+ *
+ * Bug yang dihilangkan (v0.5.113): dulu pembacaannya memakai `offset + N`
+ * hardcoded dan TIDAK ADA cabang yang menangani indeks "Exit". Memilih Exit
+ * mengembalikan "back" — menu muncul lagi, server tetap hidup, dan opsi itu
+ * tampak tidak berfungsi. Bug yang sama mengintai setiap kali item ditambah
+ * atau diurutkan ulang, karena nomornya harus diperbarui manual di dua tempat.
+ *
+ * @param {string|null} latestVersion
+ * @param {string} currentVersion
+ * @returns {Array<{label:string, icon:string, action:string}>}
+ */
+function buildInterfaceMenuItems(latestVersion, currentVersion) {
+  const items = [];
+
+  if (latestVersion) {
+    items.push({
+      label: `Update to v${latestVersion} (current: v${currentVersion})`,
+      icon: "⬆",
+      action: "update",
+    });
+  }
+
+  items.push(
+    { label: "Web UI (Open in Browser)", icon: "🌐", action: "web" },
+    { label: "Terminal UI (Interactive CLI)", icon: "💻", action: "terminal" },
+    { label: "Hide to Tray (Background)", icon: "🔔", action: "hide" },
+    { label: "Exit", icon: "🚪", action: "exit" }
+  );
+
+  return items;
+}
+
+/**
+ * Terjemahkan indeks pilihan menu menjadi aksi.
+ *
+ * @param {number} selected  indeks dari selectMenu; -1 = ESC / bukan TTY
+ * @param {string|null} latestVersion
+ * @param {string} currentVersion
+ * @returns {"update"|"web"|"terminal"|"hide"|"exit"|"back"}
+ */
+function resolveMenuAction(selected, latestVersion, currentVersion) {
+  // `selectMenu` mengembalikan -1 untuk dua hal yang SAMA SEKALI BUKAN "exit":
+  //   - ESC ditekan (input.js: resolve(-1))
+  //   - stdin bukan TTY, mis. CLI dijalankan dari skrip / terminal tanpa TTY
+  //
+  // Dulu keduanya jatuh ke `return "exit"`, dan pemanggilnya menjalankan
+  // cleanup() yang MEMBUNUH server. Akibatnya menekan ESC sekali saja sudah
+  // cukup untuk mematikan gateway — inilah keluhan "tiba-tiba berhenti".
+  // Sekarang: batalkan saja, jangan sentuh server.
+  if (selected === -1) return "back";
+
+  const items = buildInterfaceMenuItems(latestVersion, currentVersion);
+  const item = items[selected];
+  // Indeks tak dikenal: jangan pernah menafsirkannya sebagai perintah mematikan
+  // server. Lebih baik tampilkan menu lagi.
+  return item?.action || "back";
+}
+
+module.exports = { buildInterfaceMenuItems, resolveMenuAction };