@sifxprime/krouter 0.5.9 → 0.5.11

package/cli.js

@@ -55,10 +55,9 @@ try { ensureSqliteRuntime({ silent: true }); } catch {}
 // Self-heal tray runtime (systray for macOS/Linux only). Windows skipped.
 try { ensureTrayRuntime({ silent: true }); } catch {}
 
-// Upgrade safety: existing v0.5.6 users who had autostart enabled may have a
-// LaunchAgent / Startup .vbs / .desktop file pointing at the old `9router`
-// global binary that's been uninstalled. Sweep dangling entries so launchd
-// doesn't error every boot — user re-enables from the new tray when ready.
+// Sweep autostart entries whose binary path no longer exists on disk (e.g. the
+// user uninstalled the global kRouter then reinstalled to a different path).
+// Cheap startup check; no-op when nothing's wrong.
 try {
   const { sweepDanglingAutostartEntries } = require("./src/cli/tray/autostart");
   sweepDanglingAutostartEntries();
@@ -71,11 +70,7 @@ const INSTALL_CMD_LATEST = `npm i -g ${APP_NAME}@latest --prefer-online`;
 const DEFAULT_PORT = 20128;
 const DEFAULT_HOST = "0.0.0.0";
 const MAX_PORT_ATTEMPTS = 10;
-// Identifiers for killAllAppProcesses - only kill our own app specifically (new + legacy names)
-const PROCESS_IDENTIFIERS = [
-  'krouter',
-  '9router'  // legacy — kept so an existing install still cleans up properly
-];
+const PROCESS_IDENTIFIERS = ['krouter'];
 
 // Parse arguments
 let port = DEFAULT_PORT;
@@ -142,36 +137,12 @@ function compareVersions(a, b) {
   return 0;
 }
 
-// Get app data dir (must match src/mitm/paths.js — same one-time auto-migration).
+// Get app data dir — kept in sync with src/mitm/paths.js
 const DATA_DIR_NAME = "krouter";
-const LEGACY_DATA_DIR_NAME = "9router";
-function dataDirByName(name) {
-  return process.platform === "win32"
-    ? path.join(process.env.APPDATA || "", name)
-    : path.join(os.homedir(), `.${name}`);
-}
-let legacyCoexistWarned = false;
 function getAppDataDir() {
-  const target = dataDirByName(DATA_DIR_NAME);
-  const legacy = dataDirByName(LEGACY_DATA_DIR_NAME);
-  try {
-    if (!fs.existsSync(target) && fs.existsSync(legacy)) {
-      fs.renameSync(legacy, target);
-      console.log(`[cli] Migrated data dir: ${legacy} → ${target}`);
-    } else if (fs.existsSync(target) && fs.existsSync(legacy) && !legacyCoexistWarned) {
-      // Both dirs exist — auto-migration won't fire. Surface so user can reconcile.
-      // Don't auto-delete (user data); they decide how to merge / remove the legacy.
-      legacyCoexistWarned = true;
-      console.warn(
-        `[cli] Legacy ${legacy} still exists alongside ${target}. ` +
-        `New writes go to ${target}; the legacy is not read. ` +
-        `If unmerged settings exist there, back it up, merge manually, then: rm -rf ${legacy}`
-      );
-    }
-  } catch (e) {
-    console.warn(`[cli] Data-dir auto-migration failed (${e.code || e.message}); continuing with ${target}`);
-  }
-  return target;
+  return process.platform === "win32"
+    ? path.join(process.env.APPDATA || "", DATA_DIR_NAME)
+    : path.join(os.homedir(), `.${DATA_DIR_NAME}`);
 }
 
 // Kill PID from file (best-effort, removes file after)
@@ -228,7 +199,7 @@ function killCloudflaredByAppPort(appPort) {
   return pids;
 }
 
-// Kill all krouter (or legacy 9router) processes
+// Kill all kRouter processes
 function killAllAppProcesses(appPort) {
   return new Promise((resolve) => {
     try {
@@ -254,11 +225,11 @@ function killAllAppProcesses(appPort) {
           });
           const lines = output.split("\n").slice(1).filter(l => l.trim());
           lines.forEach(line => {
-            // Whitelist: real node process running krouter (or legacy 9router) cli.js, or next-server.
+            // Whitelist: real node process running krouter cli.js, or next-server.
             // Avoids killing editors/grep/strace/cursor that just incidentally match the name.
             const cmd = line.toLowerCase();
-            const hasAppName = cmd.includes("krouter") || cmd.includes("9router");
-            const hasAppPath = cmd.includes("cli.js") || cmd.includes("\\krouter") || cmd.includes("/krouter") || cmd.includes("\\9router") || cmd.includes("/9router");
+            const hasAppName = cmd.includes("krouter");
+            const hasAppPath = cmd.includes("cli.js") || cmd.includes("\\krouter") || cmd.includes("/krouter");
             const isAppProcess =
               (cmd.includes("node") && hasAppName && hasAppPath)
               || cmd.includes("next-server");
@@ -282,11 +253,11 @@ function killAllAppProcesses(appPort) {
           const lines = output.split('\n');
 
           lines.forEach(line => {
-            // Whitelist: real node process running krouter (or legacy 9router) cli.js, or next-server.
+            // Whitelist: real node process running krouter cli.js, or next-server.
             // Avoids killing grep/strace/editors/cursor that incidentally match the name.
             const cmd = line.toLowerCase();
-            const hasAppName = cmd.includes("krouter") || cmd.includes("9router");
-            const hasAppPath = cmd.includes("cli.js") || cmd.includes("/krouter") || cmd.includes("/9router");
+            const hasAppName = cmd.includes("krouter");
+            const hasAppPath = cmd.includes("cli.js") || cmd.includes("/krouter");
             const isAppProcess =
               (cmd.includes("node") && hasAppName && hasAppPath)
               || cmd.includes("next-server");
@@ -859,7 +830,7 @@ function startServer(latestVersion) {
 
     // EADDRINUSE recovery path: another process is holding the port. Most
     // commonly a stale next-server from a previous run that didn't exit
-    // cleanly, or an old `9router` global from before the rebrand. Sweep all
+    // cleanly. Sweep all
     // PIDs on the port (including parent+child pairs); if even after that the
     // port is still occupied, exit with a clear actionable error instead of
     // looping forever (which is what 0.5.7 did).

package/hooks/sqliteRuntime.js

@@ -8,23 +8,13 @@ const path = require("path");
 
 const BETTER_SQLITE3_VERSION = "12.6.2";
 
-// Must match src/mitm/paths.js / src/lib/dataDir.js — one-time auto-migration
-// of ~/.9router → ~/.krouter. Idempotent.
-function appNameDir(name) {
-  return process.platform === "win32"
-    ? path.join(process.env.APPDATA || os.homedir(), name)
-    : path.join(os.homedir(), `.${name}`);
-}
+// Kept in sync with src/mitm/paths.js / src/lib/dataDir.js
 function getDataDir() {
   if (process.env.DATA_DIR) return process.env.DATA_DIR;
-  const target = appNameDir("krouter");
-  const legacy = appNameDir("9router");
-  try {
-    if (!fs.existsSync(target) && fs.existsSync(legacy)) {
-      fs.renameSync(legacy, target);
-    }
-  } catch { /* best effort */ }
-  return target;
+  if (process.platform === "win32") {
+    return path.join(process.env.APPDATA || os.homedir(), "krouter");
+  }
+  return path.join(os.homedir(), ".krouter");
 }
 
 function getRuntimeDir() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sifxprime/krouter",
-  "version": "0.5.9",
+  "version": "0.5.11",
   "description": "kRouter (Kodelyth Router) CLI — Kodelyth AI Infrastructure",
   "homepage": "https://github.com/sifxprime/krouter",
   "repository": {
@@ -9,8 +9,7 @@
   },
   "bugs": "https://github.com/sifxprime/krouter/issues",
   "bin": {
-    "krouter": "./cli.js",
-    "9router": "./cli.js"
+    "krouter": "./cli.js"
   },
   "files": [
     "cli.js",

package/src/cli/api/client.js

@@ -15,28 +15,13 @@ const DEFAULT_CONFIG = {
 
 const CLI_TOKEN_HEADER = "x-9r-cli-token";
 const CLI_TOKEN_SALT = "9r-cli-auth";
-// Must match src/mitm/paths.js / src/lib/dataDir.js — one-time auto-migration
-// of ~/.9router → ~/.krouter. Idempotent.
-const APP_NAME = "krouter";
-const LEGACY_APP_NAME = "9router";
-
-function appNameDir(name) {
-  if (process.platform === "win32") {
-    return path.join(process.env.APPDATA || path.join(os.homedir(), "AppData", "Roaming"), name);
-  }
-  return path.join(os.homedir(), `.${name}`);
-}
-
+// Kept in sync with src/mitm/paths.js / src/lib/dataDir.js
 function getDataDir() {
   if (process.env.DATA_DIR) return process.env.DATA_DIR;
-  const target = appNameDir(APP_NAME);
-  const legacy = appNameDir(LEGACY_APP_NAME);
-  try {
-    if (!fs.existsSync(target) && fs.existsSync(legacy)) {
-      fs.renameSync(legacy, target);
-    }
-  } catch { /* best effort */ }
-  return target;
+  if (process.platform === "win32") {
+    return path.join(process.env.APPDATA || path.join(os.homedir(), "AppData", "Roaming"), "krouter");
+  }
+  return path.join(os.homedir(), ".krouter");
 }
 
 const MACHINE_ID_FILE = path.join(getDataDir(), "machine-id");
@@ -378,7 +363,7 @@ async function deleteCombo(id) {
 /**
  * Get CLI tool settings
  * @param {string} tool - Tool name: claude | codex | droid | openclaw
- * @returns {Promise<Object>} { success, data: { installed, has9Router, ... } }
+ * @returns {Promise<Object>} { success, data: { installed, hasKRouter, ... } }
  */
 async function getCliToolSettings(tool) {
   return makeRequest("GET", `/api/cli-tools/${tool}-settings`);

package/src/cli/menus/cliTools.js

@@ -179,10 +179,10 @@ async function buildCodexHeader() {
   const result = await api.getCliToolSettings("codex");
   if (!result.success) return `  ${COLORS.red}Failed to load settings${COLORS.reset}`;
 
-  const { installed, has9Router, config } = result.data;
+  const { installed, hasKRouter, config } = result.data;
   if (!installed) return `Status:   ${COLORS.red}✗ Codex CLI not installed${COLORS.reset}`;
 
-  if (!has9Router) {
+  if (!hasKRouter) {
     return [
       `Status:   ${COLORS.red}✗ Not configured${COLORS.reset}`,
       `${COLORS.dim}Run "Quick Setup" to configure${COLORS.reset}`
@@ -267,10 +267,10 @@ async function buildDroidHeader() {
   const result = await api.getCliToolSettings("droid");
   if (!result.success) return `  ${COLORS.red}Failed to load settings${COLORS.reset}`;
 
-  const { installed, has9Router, settings } = result.data;
+  const { installed, hasKRouter, settings } = result.data;
   if (!installed) return `Status:   ${COLORS.red}✗ Factory Droid not installed${COLORS.reset}`;
 
-  if (!has9Router) {
+  if (!hasKRouter) {
     return [
       `Status:   ${COLORS.red}✗ Not configured${COLORS.reset}`,
       `${COLORS.dim}Run "Quick Setup" to configure${COLORS.reset}`
@@ -279,7 +279,7 @@ async function buildDroidHeader() {
 
   // Extract kRouter custom model config (read either new or legacy id for compat)
   const custom = settings?.customModels?.find(
-    m => m.id === "custom:kRouter-0" || m.id === "custom:9Router-0"
+    m => m.id === "custom:kRouter-0"
   );
   const lines = [`Status:   ${COLORS.green}✓ Configured${COLORS.reset}`];
   if (custom?.baseUrl) lines.push(`Endpoint: ${COLORS.cyan}${custom.baseUrl}${COLORS.reset}`);
@@ -352,10 +352,10 @@ async function buildOpenClawHeader() {
   const result = await api.getCliToolSettings("openclaw");
   if (!result.success) return `  ${COLORS.red}Failed to load settings${COLORS.reset}`;
 
-  const { installed, has9Router, settings } = result.data;
+  const { installed, hasKRouter, settings } = result.data;
   if (!installed) return `Status:   ${COLORS.red}✗ Open Claw not installed${COLORS.reset}`;
 
-  if (!has9Router) {
+  if (!hasKRouter) {
     return [
       `Status:   ${COLORS.red}✗ Not configured${COLORS.reset}`,
       `${COLORS.dim}Run "Quick Setup" to configure${COLORS.reset}`
@@ -363,10 +363,10 @@ async function buildOpenClawHeader() {
   }
 
   // Extract kRouter provider config (read either new or legacy key for compat)
-  const provider = settings?.models?.providers?.["krouter"] ?? settings?.models?.providers?.["9router"];
+  const provider = settings?.models?.providers?.["krouter"];
   const primary = settings?.agents?.defaults?.model?.primary || "";
-  const model = /^(?:krouter|9router)\//.test(primary)
-    ? primary.replace(/^(?:krouter|9router)\//, "")
+  const model = /^krouter\//.test(primary)
+    ? primary.replace(/^krouter\//, "")
     : (provider?.models?.[0]?.id || "");
   const lines = [`Status:   ${COLORS.green}✓ Configured${COLORS.reset}`];
   if (provider?.baseUrl) lines.push(`Endpoint: ${COLORS.cyan}${provider.baseUrl}${COLORS.reset}`);
@@ -435,10 +435,10 @@ async function buildOpenCodeHeader() {
   const result = await api.getCliToolSettings("opencode");
   if (!result.success) return `  ${COLORS.red}Failed to load settings${COLORS.reset}`;
 
-  const { installed, has9Router, opencode } = result.data;
+  const { installed, hasKRouter, opencode } = result.data;
   if (!installed) return `Status:   ${COLORS.red}✗ OpenCode CLI not installed${COLORS.reset}`;
 
-  if (!has9Router) {
+  if (!hasKRouter) {
     return [
       `Status:   ${COLORS.red}✗ Not configured${COLORS.reset}`,
       `${COLORS.dim}Run "Quick Setup" to configure${COLORS.reset}`
@@ -523,10 +523,10 @@ async function buildHermesHeader() {
   const result = await api.getCliToolSettings("hermes");
   if (!result.success) return `  ${COLORS.red}Failed to load settings${COLORS.reset}`;
 
-  const { installed, has9Router, settings } = result.data;
+  const { installed, hasKRouter, settings } = result.data;
   if (!installed) return `Status:   ${COLORS.red}✗ Hermes Agent not installed${COLORS.reset}`;
 
-  if (!has9Router) {
+  if (!hasKRouter) {
     return [
       `Status:   ${COLORS.red}✗ Not configured${COLORS.reset}`,
       `${COLORS.dim}Run "Quick Setup" to configure${COLORS.reset}`