@inceptionstack/roundhouse 0.3.10 → 0.3.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inceptionstack/roundhouse",
-  "version": "0.3.10",
+  "version": "0.3.12",
   "type": "module",
   "description": "Multi-platform chat gateway that routes messages through a configured AI agent",
   "license": "MIT",

package/src/cli/doctor/checks/agent.ts

@@ -12,18 +12,28 @@ export const agentChecks: DoctorCheck[] = [
   {
     id: "pi-sdk", category: "agent", name: "Pi SDK",
     async run() {
+      const PI_PKG = join("@mariozechner", "pi-coding-agent", "package.json");
+      const searchPaths = [
+        join(process.cwd(), "node_modules", PI_PKG),
+      ];
+      // Also check global npm root
       try {
-        const pkgPath = join(process.cwd(), "node_modules", "@mariozechner", "pi-coding-agent", "package.json");
-        const raw = await readFile(pkgPath, "utf8");
-        const ver = JSON.parse(raw).version;
-        return { id: "pi-sdk", category: "agent", name: "Pi SDK", status: "pass", summary: `v${ver}` };
-      } catch {
-        return {
-          id: "pi-sdk", category: "agent", name: "Pi SDK", status: "fail", summary: "not found",
+        const { execFileSync } = await import("node:child_process");
+        const globalRoot = execFileSync("npm", ["root", "-g"], { encoding: "utf8" }).trim();
+        searchPaths.push(join(globalRoot, PI_PKG));
+      } catch {}
+      for (const pkgPath of searchPaths) {
+        try {
+          const raw = await readFile(pkgPath, "utf8");
+          const ver = JSON.parse(raw).version;
+          return { id: "pi-sdk", category: "agent", name: "Pi SDK", status: "pass" as const, summary: `v${ver}` };
+        } catch {}
+      }
+      return {
+          id: "pi-sdk", category: "agent", name: "Pi SDK", status: "fail" as const, summary: "not found",
           details: ["@mariozechner/pi-coding-agent not installed"],
           fix: { description: "Install pi SDK", command: "npm install @mariozechner/pi-coding-agent" },
         };
-      }
     },
   },
 

package/src/cli/doctor/checks/credentials.ts

@@ -22,7 +22,7 @@ export const credentialChecks: DoctorCheck[] = [
         return {
           id: "telegram-token", category: "credentials", name: "Telegram bot token",
           status: "fail", summary: "TELEGRAM_BOT_TOKEN not set",
-          details: ["Set TELEGRAM_BOT_TOKEN in your environment or ~/.roundhouse/env"],
+          details: ["Set TELEGRAM_BOT_TOKEN in your environment or ~/.roundhouse/.env"],
         };
       }
       if (!/^\d+:[A-Za-z0-9_-]+$/.test(token)) {

package/src/cli/setup.ts

@@ -181,7 +181,7 @@ export function parseSetupArgs(argv: string[]): SetupOptions {
   }
 
   // Validate
-  if (!opts.botToken) {
+  if (!opts.botToken && !opts.dryRun) {
     throw new Error(
       "Bot token required. Provide via:\n" +
       "  TELEGRAM_BOT_TOKEN=... roundhouse setup --user USERNAME\n" +
@@ -451,7 +451,11 @@ async function stepInstallPackages(opts: SetupOptions): Promise<void> {
 }
 
 async function stepStoreSecrets(opts: SetupOptions, botInfo: BotInfo): Promise<void> {
-  if (!opts.psst) return;
+  if (!opts.psst) {
+    step("⑥", "Storing secrets...");
+    ok("Skipped (--no-psst)");
+    return;
+  }
 
   step("⑥", "Storing secrets in psst...");
 
@@ -626,7 +630,7 @@ async function stepConfigure(
   }
 
   await atomicWriteText(ENV_PATH, envLines.join("\n") + "\n");
-  ok(`~/.roundhouse/env${opts.psst ? " (non-secret config only)" : ""}`);
+  ok(`~/.roundhouse/.env${opts.psst ? " (non-secret config only)" : ""}`);
 }
 
 async function stepPair(opts: SetupOptions, botInfo: BotInfo): Promise<PairResult | null> {
@@ -869,7 +873,7 @@ export async function cmdSetup(argv: string[]): Promise<void> {
     }
     log(`   Bot: @${botInfo.username}`);
     log(`   Memory: ${opts.extensions.some((e) => e.includes("pi-memory")) ? "agent-managed" : "roundhouse-managed"}`);
-    log(`   Secrets: ${opts.psst ? "psst vault (encrypted)" : "~/.roundhouse/env (plaintext)"}`);
+    log(`   Secrets: ${opts.psst ? "psst vault (encrypted)" : "~/.roundhouse/.env (plaintext)"}`);
     log(`   Send /status to @${botInfo.username} on Telegram.\n`);
   } catch (err: any) {
     log("\n━━━━━━━━━━━━━━━━━━━");
@@ -989,7 +993,7 @@ function printDryRun(opts: SetupOptions): void {
   log(`  Set defaultProvider: ${opts.provider}`);
   log(`  Set defaultModel: ${opts.model}`);
   log(`Would write: ~/.roundhouse/gateway.config.json`);
-  log(`Would write: ~/.roundhouse/env${opts.psst ? " (non-secret config only)" : ""}`);
+  log(`Would write: ~/.roundhouse/.env${opts.psst ? " (non-secret config only)" : ""}`);
   log(`Would register ${BOT_COMMANDS.length} bot commands`);
   if (opts.systemd) log(`Would install systemd service`);
   log("\nNo changes made.\n");

package/src/config.ts

@@ -21,7 +21,10 @@ export const LEGACY_CONFIG_DIR = resolve(homedir(), ".config", "roundhouse");
 /** Active config directory — use ROUNDHOUSE_DIR */
 export const CONFIG_DIR = ROUNDHOUSE_DIR;
 export const CONFIG_PATH = resolve(ROUNDHOUSE_DIR, "gateway.config.json");
-export const ENV_FILE_PATH = resolve(ROUNDHOUSE_DIR, "env");
+export const ENV_FILE_PATH = resolve(ROUNDHOUSE_DIR, ".env");
+
+/** Legacy env file name (deprecated) */
+export const LEGACY_ENV_FILE_PATH = resolve(ROUNDHOUSE_DIR, "env");
 
 /** Cron directories */
 export const CRON_JOBS_DIR = resolve(ROUNDHOUSE_DIR, "crons");
@@ -109,15 +112,26 @@ export async function resolveConfigPath(): Promise<{ path: string; legacy: boole
 let envFileWarned = false;
 
 export async function resolveEnvFilePath(): Promise<string> {
+  // Canonical: ~/.roundhouse/.env
   if (await fileExists(ENV_FILE_PATH)) return ENV_FILE_PATH;
-  const legacyEnv = resolve(LEGACY_CONFIG_DIR, "env");
-  if (await fileExists(legacyEnv)) {
+  // Legacy: ~/.roundhouse/env (old name, same directory)
+  if (await fileExists(LEGACY_ENV_FILE_PATH)) {
+    if (!envFileWarned) {
+      envFileWarned = true;
+      console.warn(`[roundhouse] \u26a0\ufe0f  Env file found at legacy path: ${LEGACY_ENV_FILE_PATH}`);
+      console.warn(`[roundhouse]    Rename it to ${ENV_FILE_PATH} \u2014 legacy name will be removed in a future version.`);
+    }
+    return LEGACY_ENV_FILE_PATH;
+  }
+  // Legacy: ~/.config/roundhouse/env (old directory)
+  const legacyDirEnv = resolve(LEGACY_CONFIG_DIR, "env");
+  if (await fileExists(legacyDirEnv)) {
     if (!envFileWarned) {
       envFileWarned = true;
-      console.warn(`[roundhouse] \u26a0\ufe0f  Env file found at legacy path: ${legacyEnv}`);
+      console.warn(`[roundhouse] \u26a0\ufe0f  Env file found at legacy path: ${legacyDirEnv}`);
       console.warn(`[roundhouse]    Move it to ${ENV_FILE_PATH} \u2014 legacy path will be removed in a future version.`);
     }
-    return legacyEnv;
+    return legacyDirEnv;
   }
   return ENV_FILE_PATH;
 }