@lovenyberg/ove 0.5.0 → 0.5.2

package/src/handlers.ts

@@ -1,3 +1,4 @@
+import { mkdirSync } from "node:fs";
 import { parseMessage, buildContextualPrompt } from "./router";
 import type { ParsedMessage } from "./router";
 import { isAuthorized, getUserRepos, addRepo, addUser } from "./config";
@@ -332,6 +333,7 @@ async function handleDiscuss(msg: IncomingMessage, parsed: ParsedMessage, histor
   await msg.updateStatus("Thinking...");
 
   try {
+    mkdirSync(deps.config.reposDir, { recursive: true });
     const discussRunner = deps.getRunner(deps.config.runner?.name);
     const result = await discussRunner.run(
       prompt,
@@ -452,7 +454,7 @@ async function handleTaskMessage(msg: IncomingMessage, parsed: ParsedMessage, de
         }
       }
     } else {
-      const reply = "You don't have access to any repos yet. Set one up:\n`init repo <name> <git-url> [branch]`\nExample: `init repo my-app git@github.com:user/my-app.git`";
+      const reply = `No repos configured yet. Add one with:\n\`init repo <name> <git-url> [branch]\`\nExample: \`init repo my-app git@github.com:user/my-app.git\``;
       await msg.reply(reply);
       return;
     }
@@ -521,6 +523,13 @@ export function createMessageHandler(deps: HandlerDeps): (msg: IncomingMessage)
       return;
     }
 
+    // If user has no repos, fall back to discuss mode so they can still chat
+    const userRepos = getUserRepos(deps.config, msg.userId);
+    if (userRepos.length === 0) {
+      const history = deps.sessions.getHistory(msg.userId, 6);
+      return handleDiscuss(msg, { ...parsed, type: "discuss", args: { topic: parsed.rawText } }, history, deps);
+    }
+
     // For all other types (free-form, review-pr, fix-issue, simplify, validate) — task dispatch
     await handleTaskMessage(msg, parsed, deps);
   };

package/src/index.ts

@@ -140,7 +140,8 @@ if (process.env.HTTP_API_PORT) {
     process.env.HTTP_API_KEY || crypto.randomUUID(),
     trace,
     queue,
-    sessions
+    sessions,
+    process.env.HTTP_API_HOST
   );
   eventAdapters.push(httpAdapter);
 }

package/src/router.ts

@@ -116,6 +116,46 @@ export function parseMessage(text: string): ParsedMessage {
     };
   }
 
+  // Natural language repo setup: "clone org/repo", "setup org/repo", "add org/repo"
+  const naturalRepoMatch = trimmed.match(/^(?:clone|setup|add|init|use)\s+(?:(?:the|repo)\s+)?([a-zA-Z0-9_.-]+\/[a-zA-Z0-9_.-]+)(?:\s+(?:repo(?:sitory)?))?\s*$/i);
+  if (naturalRepoMatch) {
+    const slug = naturalRepoMatch[1];
+    const name = slug.split("/").pop()!;
+    const url = `git@github.com:${slug}.git`;
+    return {
+      type: "init-repo",
+      args: { name, url, branch: "main" },
+      rawText: trimmed,
+    };
+  }
+
+  // Detect org/repo or GitHub URLs anywhere in a message that looks like a setup request
+  const setupIntent = /(?:clone|setup|add|init|use|start\s+(?:with|on)|work\s+(?:on|with))/i.test(lower);
+  if (setupIntent) {
+    const ghUrl = trimmed.match(/((?:git@github\.com:|https:\/\/github\.com\/)([a-zA-Z0-9_.-]+\/[a-zA-Z0-9_.-]+?)(?:\.git)?)\b/);
+    if (ghUrl) {
+      const slug = ghUrl[2];
+      const name = slug.split("/").pop()!;
+      const url = ghUrl[1].endsWith(".git") ? ghUrl[1] : ghUrl[1] + ".git";
+      return {
+        type: "init-repo",
+        args: { name, url: url.startsWith("git@") ? url : `git@github.com:${slug}.git`, branch: "main" },
+        rawText: trimmed,
+      };
+    }
+    const slugMatch = trimmed.match(/\b([a-zA-Z0-9_.-]+\/[a-zA-Z0-9_.-]+)\b/);
+    if (slugMatch && slugMatch[1].indexOf("/") > 0) {
+      const slug = slugMatch[1];
+      const name = slug.split("/").pop()!;
+      const url = `git@github.com:${slug}.git`;
+      return {
+        type: "init-repo",
+        args: { name, url, branch: "main" },
+        rawText: trimmed,
+      };
+    }
+  }
+
   const repoHint = trimmed.match(/(?:in|on)\s+(\S+?)[?.!,]*\s*$/i);
   return { type: "free-form", repo: repoHint?.[1], args: {}, rawText: trimmed };
 }

package/src/setup.ts

@@ -1,6 +1,6 @@
 import { existsSync, readFileSync, writeFileSync } from "node:fs";
 import { execFileSync } from "node:child_process";
-import { resolve } from "node:path";
+import { resolve, dirname } from "node:path";
 import { userInfo } from "node:os";
 import { createInterface } from "node:readline/promises";
 import type { Config } from "./config";
@@ -59,7 +59,9 @@ export function validateConfig(opts?: { configPath?: string; envPath?: string })
   if (existsSync(configPath)) {
     try {
       const config = JSON.parse(readFileSync(configPath, "utf-8")) as Partial<Config>;
-      if (!config.repos || Object.keys(config.repos).length === 0) {
+      const hasGithubSync = config.github?.orgs && config.github.orgs.length > 0;
+      const hasWildcardUser = Object.values(config.users || {}).some(u => u.repos?.includes("*"));
+      if (!hasGithubSync && !hasWildcardUser && (!config.repos || Object.keys(config.repos).length === 0)) {
         issues.push("No repos configured");
       }
       if (!config.users || Object.keys(config.users).length === 0) {
@@ -209,6 +211,8 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
       process.stdout.write("\n");
       const port = (await ask(rl, "HTTP API port [3000]")) || "3000";
       envValues.HTTP_API_PORT = port;
+      const host = await ask(rl, "Bind address [127.0.0.1] (0.0.0.0 for all interfaces)");
+      envValues.HTTP_API_HOST = host || "127.0.0.1";
       const key = await ask(rl, "API key (leave empty to generate)");
       envValues.HTTP_API_KEY = key || crypto.randomUUID();
     }
@@ -225,23 +229,51 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
       envValues.CLI_MODE = "true";
     }
 
-    // Collect repos
+    // Tracing
+    if (!fixing) {
+      process.stdout.write("\n");
+      const enableTrace = await ask(rl, "Enable task tracing? (y/n)");
+      if (enableTrace.toLowerCase() === "y") {
+        envValues.OVE_TRACE = "true";
+      }
+    }
+
+    // Collect repos — GitHub sync or manual
     const repos: Record<string, { url: string; defaultBranch: string }> = existingConfig.repos
       ? { ...existingConfig.repos }
       : {};
+    let githubOrgs: string[] = existingConfig.github?.orgs || [];
+    let useWildcard = false;
 
     if (needsRepos || needsConfigFile) {
-      let addMore = true;
-      while (addMore) {
-        process.stdout.write("\n  Add a repo:\n");
-        const name = await ask(rl, "Name");
-        if (!name) break;
-        const url = await ask(rl, "Git URL");
-        const branch = (await ask(rl, "Default branch [main]")) || "main";
-        repos[name] = { url, defaultBranch: branch };
-
-        const again = await ask(rl, "Add another repo? (y/n)");
-        addMore = again.toLowerCase() === "y";
+      const repoMode = await choose(rl, "How do you want to manage repos?", [
+        "Auto-discover from GitHub (uses gh CLI)",
+        "Add repos manually",
+        "Skip — add repos later via chat",
+      ]);
+
+      if (repoMode === 0) {
+        // GitHub auto-discovery
+        const orgs = await ask(rl, "GitHub orgs/users to sync (comma-separated, or empty for your repos)");
+        githubOrgs = orgs ? orgs.split(",").map(s => s.trim()).filter(Boolean) : [];
+        useWildcard = true;
+        process.stdout.write("  Repos will sync from GitHub on startup.\n");
+      } else if (repoMode === 1) {
+        let addMore = true;
+        while (addMore) {
+          process.stdout.write("\n  Add a repo:\n");
+          const name = await ask(rl, "Name");
+          if (!name) break;
+          const url = await ask(rl, "Git URL");
+          const branch = (await ask(rl, "Default branch [main]")) || "main";
+          repos[name] = { url, defaultBranch: branch };
+
+          const again = await ask(rl, "Add another repo? (y/n)");
+          addMore = again.toLowerCase() === "y";
+        }
+      } else {
+        useWildcard = true;
+        process.stdout.write("  You can add repos later: 'clone org/repo' in chat.\n");
       }
     }
 
@@ -249,7 +281,7 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
     const users: Record<string, { name: string; repos: string[] }> = existingConfig.users
       ? { ...existingConfig.users }
       : {};
-    const repoNames = Object.keys(repos);
+    const repoNames = useWildcard ? ["*"] : Object.keys(repos);
 
     if (needsUsers || needsConfigFile) {
       // Ask for user name once
@@ -265,7 +297,7 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
         if (userId) users[`slack:${userId}`] = { name: userName || "user", repos: repoNames };
       }
       if (has("Telegram")) {
-        const userId = await ask(rl, "Your Telegram user ID");
+        const userId = await ask(rl, "Your Telegram user ID (send /start to @userinfobot to find it)");
         if (userId) users[`telegram:${userId}`] = { name: userName || "user", repos: repoNames };
       }
       if (has("Discord")) {
@@ -325,6 +357,7 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
       if (has("HTTP API")) {
         envLines.push("# HTTP API + Web UI");
         envLines.push(`HTTP_API_PORT=${envValues.HTTP_API_PORT || "3000"}`);
+        envLines.push(`HTTP_API_HOST=${envValues.HTTP_API_HOST || "127.0.0.1"}`);
         envLines.push(`HTTP_API_KEY=${envValues.HTTP_API_KEY || ""}`);
         envLines.push("");
       }
@@ -344,6 +377,13 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
         envLines.push("");
       }
 
+      // Tracing
+      if (envValues.OVE_TRACE) {
+        envLines.push("# Tracing");
+        envLines.push(`OVE_TRACE=${envValues.OVE_TRACE}`);
+        envLines.push("");
+      }
+
       // Always include repos dir
       envLines.push("# Repos directory");
       envLines.push(`REPOS_DIR=${envValues.REPOS_DIR || "./repos"}`);
@@ -359,6 +399,7 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
         repos,
         users,
         claude: existingConfig.claude || { maxTurns: 25 },
+        ...(githubOrgs.length > 0 ? { github: { orgs: githubOrgs } } : existingConfig.github ? { github: existingConfig.github } : {}),
         ...(existingConfig.mcpServers ? { mcpServers: existingConfig.mcpServers } : {}),
         ...(existingConfig.cron ? { cron: existingConfig.cron } : {}),
       };
@@ -367,15 +408,38 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
     }
 
     // Systemd service setup
+    let installedSystemd = false;
     if (!fixing) {
       const installService = await ask(rl, "Install as systemd service? (y/n)");
       if (installService.toLowerCase() === "y") {
-        await installSystemdService(rl);
+        installedSystemd = await installSystemdService(rl);
       }
     }
 
     if (!fixing) {
-      process.stdout.write("\n  Nåväl. Run 'ove start' when you're ready.\n\n");
+      process.stdout.write("\n  Nåväl.\n");
+      if (installedSystemd) {
+        process.stdout.write("\n  Useful commands:\n");
+        process.stdout.write("    sudo systemctl status ove     # check status\n");
+        process.stdout.write("    sudo journalctl -u ove -f     # follow logs\n");
+        process.stdout.write("    sudo systemctl restart ove    # restart\n");
+        process.stdout.write("    sudo systemctl stop ove       # stop\n");
+        if (has("HTTP API")) {
+          const port = envValues.HTTP_API_PORT || "3000";
+          const host = envValues.HTTP_API_HOST || "127.0.0.1";
+          const displayHost = host === "0.0.0.0" ? "<your-ip>" : host;
+          process.stdout.write(`\n  Web UI: http://${displayHost}:${port}\n`);
+          if (envValues.OVE_TRACE === "true") {
+            process.stdout.write(`  Traces: http://${displayHost}:${port}/trace\n`);
+          }
+          if (host === "0.0.0.0") {
+            process.stdout.write("  (bound to all interfaces)\n");
+          }
+        }
+        process.stdout.write("\n");
+      } else {
+        process.stdout.write("  Run 'ove start' when you're ready.\n\n");
+      }
     } else {
       process.stdout.write("\n");
     }
@@ -384,7 +448,7 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
   }
 }
 
-async function installSystemdService(rl: ReturnType<typeof createInterface>): Promise<void> {
+async function installSystemdService(rl: ReturnType<typeof createInterface>): Promise<boolean> {
   const detectedUser = userInfo().username;
   const detectedDir = resolve(".");
   let detectedBun = "";
@@ -400,10 +464,22 @@ async function installSystemdService(rl: ReturnType<typeof createInterface>): Pr
 
   if (!bunPath) {
     process.stdout.write("  Could not find bun. Skipping service install.\n");
-    return;
+    return false;
   }
 
   const envPath = resolve(workDir, ".env");
+
+  // Build PATH that includes dirs for bun, claude, and gh
+  const pathDirs = new Set<string>();
+  for (const bin of [bunPath, "claude", "gh"]) {
+    let resolved = "";
+    try { resolved = execFileSync("which", [bin]).toString().trim(); } catch {}
+    if (resolved) pathDirs.add(dirname(resolved));
+  }
+  // Always include standard dirs
+  for (const d of ["/usr/local/bin", "/usr/bin", "/bin"]) pathDirs.add(d);
+  const pathLine = Array.from(pathDirs).join(":");
+
   const service = `[Unit]
 Description=Ove - Personal AI coding assistant
 After=network.target
@@ -416,6 +492,7 @@ ExecStart=${bunPath} run src/index.ts
 Restart=always
 RestartSec=5
 EnvironmentFile=${envPath}
+Environment=PATH=${pathLine}
 
 [Install]
 WantedBy=multi-user.target
@@ -438,8 +515,10 @@ WantedBy=multi-user.target
         execFileSync("sudo", ["systemctl", "start", "ove"]);
         process.stdout.write("  Service started.\n");
       }
+      return true;
     } catch (err) {
       process.stdout.write(`  Failed to install service: ${err}\n`);
     }
   }
+  return false;
 }