@lovenyberg/ove 0.5.1 → 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/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) {
@@ -236,23 +238,42 @@ export async function runSetup(opts?: { fixOnly?: string[] }): Promise<void> {
       }
     }
 
-    // Collect repos
+    // 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");
       }
     }
 
@@ -260,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
@@ -276,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")) {
@@ -378,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 } : {}),
       };
@@ -446,6 +468,18 @@ async function installSystemdService(rl: ReturnType<typeof createInterface>): Pr
   }
 
   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
@@ -458,6 +492,7 @@ ExecStart=${bunPath} run src/index.ts
 Restart=always
 RestartSec=5
 EnvironmentFile=${envPath}
+Environment=PATH=${pathLine}
 
 [Install]
 WantedBy=multi-user.target