@lovenyberg/ove 0.5.2 → 0.6.0

package/README.md

@@ -15,6 +15,18 @@ Talk to Ove from Slack, WhatsApp, Telegram, Discord, GitHub issues, a Web UI, or
 
 **Just chat.** You don't need to memorize commands. Talk to Ove like you'd talk to a colleague — ask questions, describe what you need, paste error messages, think out loud. He understands natural language. The commands below are shortcuts, not requirements.
 
+<p align="center">
+  <img src="screenshot-telegram.png" width="320" alt="Ove on Telegram" />
+</p>
+
+### Web UI
+
+![Ove Web UI](screenshot-chat.png)
+
+### Trace Viewer
+
+![Ove Trace Viewer](screenshot-trace.png)
+
 ## Quick Start
 
 ```bash

package/docs/index.html

@@ -5,6 +5,20 @@
   <meta name="viewport" content="width=device-width, initial-scale=1.0">
   <link rel="icon" href="favicon.ico">
   <title>Ove — Your Grumpy Dev Companion</title>
+  <meta name="description" content="AI coding agent for Slack, WhatsApp, Telegram, Discord, GitHub and CLI. Routes tasks to Claude Code or Codex in isolated worktrees.">
+
+  <!-- Open Graph -->
+  <meta property="og:type" content="website">
+  <meta property="og:url" content="https://jacksoncage.github.io/ove/">
+  <meta property="og:title" content="Ove — Your Grumpy Dev Companion">
+  <meta property="og:description" content="AI coding agent for Slack, WhatsApp, Telegram, Discord, GitHub and CLI. Routes tasks to Claude Code or Codex in isolated worktrees.">
+  <meta property="og:image" content="https://jacksoncage.github.io/ove/logo.png">
+
+  <!-- Twitter Card -->
+  <meta name="twitter:card" content="summary">
+  <meta name="twitter:title" content="Ove — Your Grumpy Dev Companion">
+  <meta name="twitter:description" content="AI coding agent for Slack, WhatsApp, Telegram, Discord, GitHub and CLI.">
+  <meta name="twitter:image" content="https://jacksoncage.github.io/ove/logo.png">
   <link rel="preconnect" href="https://fonts.googleapis.com">
   <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
   <link href="https://fonts.googleapis.com/css2?family=Inter:wght@400;500&family=JetBrains+Mono:wght@400;500&display=swap" rel="stylesheet">
@@ -671,6 +685,24 @@
       "People today don't even know how a worktree works. In my day we used patches. On paper."
       <span class="attr">— Ove, probably</span>
     </div>
+
+    <h3>Screenshots</h3>
+    <div style="display: flex; flex-direction: column; gap: 16px; margin-top: 16px;">
+      <div style="display: flex; justify-content: center;">
+        <div style="max-width: 360px;">
+          <div class="feature-label">Telegram</div>
+          <img src="screenshot-telegram.png" alt="Ove on Telegram" style="width: 100%; border: 1px solid var(--border); border-radius: 3px;">
+        </div>
+      </div>
+      <div>
+        <div class="feature-label">Web UI — Chat</div>
+        <img src="screenshot-chat.png" alt="Ove Web UI chat interface" style="width: 100%; border: 1px solid var(--border); border-radius: 3px;">
+      </div>
+      <div>
+        <div class="feature-label">Trace Viewer</div>
+        <img src="screenshot-trace.png" alt="Ove Trace Viewer" style="width: 100%; border: 1px solid var(--border); border-radius: 3px;">
+      </div>
+    </div>
   </section>
 
   <section class="section" id="getting-started">

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lovenyberg/ove",
-  "version": "0.5.2",
+  "version": "0.6.0",
   "description": "Your grumpy but meticulous dev companion. AI coding agent for Slack, WhatsApp, Telegram, Discord, GitHub, HTTP API, and CLI.",
   "type": "module",
   "bin": {

package/src/config.test.ts

@@ -60,6 +60,12 @@ describe("getUserRepos", () => {
     config.users["slack:U123"] = { name: "test", repos: ["a", "b"] };
     expect(getUserRepos(config, "slack:U123")).toEqual(["a", "b"]);
   });
+
+  it("returns wildcard for known user with empty repos", () => {
+    const config = loadConfig();
+    config.users["slack:U123"] = { name: "test", repos: [] };
+    expect(getUserRepos(config, "slack:U123")).toEqual(["*"]);
+  });
 });
 
 describe("saveConfig / addRepo / addUser", () => {

package/src/config.ts

@@ -79,6 +79,9 @@ export function loadConfig(): Config {
 export function getUserRepos(config: Config, platformUserId: string): string[] {
   const user = config.users[platformUserId];
   if (!user) return [];
+  // Known users with empty repos get wildcard access — avoids
+  // silently falling back to discuss-only mode with no tracing.
+  if (user.repos.length === 0) return ["*"];
   return user.repos;
 }
 
@@ -86,7 +89,8 @@ export function isAuthorized(config: Config, platformUserId: string, repo?: stri
   const user = config.users[platformUserId];
   if (!user) return false;
   if (!repo) return true;
-  return user.repos.includes("*") || user.repos.includes(repo);
+  // Empty repos = wildcard (same as getUserRepos)
+  return user.repos.length === 0 || user.repos.includes("*") || user.repos.includes(repo);
 }
 
 export function saveConfig(config: Config): void {

package/src/flows.test.ts

@@ -426,7 +426,7 @@ describe("Full follow-up conversation flow", () => {
     const historyContext = history.length > 1
       ? "Recent conversation:\n" + history.slice(0, -1).map(m => `${m.role}: ${m.content}`).join("\n") + "\n\n"
       : "";
-    const resolvePrompt = `You are a repo-name resolver. ${historyContext}The user's latest message:\n"what about tomorrow"\n\nAvailable repos: iris, docs, my-app\n\nRespond with ONLY the repo name that best matches their request. Consider the conversation context if the current message doesn't mention a specific repo. Nothing else — just the exact repo name from the list. If you cannot determine which repo, respond with "UNKNOWN".`;
+    const resolvePrompt = `You are a repo-name resolver. ${historyContext}The user's latest message:\n"what about tomorrow"\n\nAvailable repos: iris, docs, my-app\n\nRespond with ONLY the repo name that best matches their request. Consider the conversation context if the current message doesn't mention a specific repo. Nothing else — just the exact repo name from the list. If the question doesn't need a specific repo (e.g. "list my open PRs", "what should I work on today", cross-repo queries, general questions about the user's GitHub activity), respond with "NONE". If you cannot determine which specific repo, respond with "UNKNOWN".`;
 
     expect(resolvePrompt).toContain("Recent conversation:");
     expect(resolvePrompt).toContain("check the roadmap on iris");

package/src/handlers.ts

@@ -76,20 +76,68 @@ export function splitAndReply(text: string, platform: string): string[] {
   return parts;
 }
 
+function getUserRepoNames(userId: string, deps: HandlerDeps): string[] {
+  const userRepos = getUserRepos(deps.config, userId);
+  if (userRepos.includes("*")) return deps.repoRegistry.getAllNames();
+  return userRepos;
+}
+
 function resolveRepo(userId: string, hint: string | undefined, deps: HandlerDeps): string | null {
   if (hint && deps.getRepoInfo(hint)) return hint;
+  const repoNames = getUserRepoNames(userId, deps);
+  if (repoNames.length === 1) return repoNames[0];
+  return null;
+}
 
-  const userRepos = getUserRepos(deps.config, userId);
-  const hasWildcard = userRepos.includes("*");
+type RepoResolution =
+  | { kind: "resolved"; repo: string }
+  | { kind: "none" }
+  | { kind: "unknown"; repoNames: string[] }
+  | { kind: "error"; message: string };
+
+async function resolveRepoWithLLM(
+  userId: string,
+  rawText: string,
+  hint: string | undefined,
+  deps: HandlerDeps,
+  onStatus?: (text: string) => void,
+): Promise<RepoResolution> {
+  if (hint && deps.getRepoInfo(hint)) return { kind: "resolved", repo: hint };
+
+  const repoNames = getUserRepoNames(userId, deps);
+  if (repoNames.length === 0) return { kind: "error", message: "No repos discovered yet. Set one up with `init repo <name> <git-url>` or configure GitHub sync." };
+  if (repoNames.length === 1) return { kind: "resolved", repo: repoNames[0] };
+
+  const history = deps.sessions.getHistory(userId, 6);
+  const recentTasks = deps.queue.listByUser(userId, 5);
+  const lastRepo = recentTasks.find(t => t.status === "completed" || t.status === "failed")?.repo;
+  const historyContext = history.length > 1
+    ? "Recent conversation:\n" + history.slice(0, -1).map(m => `${m.role}: ${m.content}`).join("\n") + "\n\n"
+    : "";
+  const lastRepoHint = lastRepo && repoNames.includes(lastRepo)
+    ? `The user's most recent task was on repo "${lastRepo}", but only use this if the conversation context supports it.\n\n`
+    : "";
+  const resolvePrompt = `You are a repo-name resolver. ${historyContext}${lastRepoHint}The user's latest message:\n"${rawText}"\n\nAvailable repos: ${repoNames.join(", ")}\n\nRespond with ONLY the repo name that best matches their request. Consider the conversation context if the current message doesn't mention a specific repo. Nothing else — just the exact repo name from the list. If the question doesn't need a specific repo (e.g. "list my open PRs", "what should I work on today", cross-repo queries, general questions about the user's GitHub activity), respond with "NONE". If you cannot determine which specific repo, respond with "UNKNOWN".`;
+
+  onStatus?.("Figuring out which repo...");
+  try {
+    const runner = deps.getRunner(deps.config.runner?.name);
+    const result = await runner.run(resolvePrompt, deps.config.reposDir, { maxTurns: 1 });
+    const resolved = result.output.trim().replace(/[`"']/g, "");
 
-  if (!hasWildcard && userRepos.length === 1) return userRepos[0];
+    if (resolved === "NONE") {
+      logger.info("repo resolver returned NONE — falling back to discuss", { userText: rawText.slice(0, 80) });
+      return { kind: "none" };
+    }
+    if (resolved === "UNKNOWN" || !repoNames.includes(resolved)) {
+      return { kind: "unknown", repoNames };
+    }
 
-  if (hasWildcard || userRepos.length > 1) {
-    const repoNames = hasWildcard ? deps.repoRegistry.getAllNames() : userRepos;
-    if (repoNames.length === 1) return repoNames[0];
+    logger.info("repo resolved via LLM", { resolved, userText: rawText.slice(0, 80) });
+    return { kind: "resolved", repo: resolved };
+  } catch (err) {
+    return { kind: "error", message: `Couldn't figure out the repo: ${String(err).slice(0, 300)}` };
   }
-
-  return null;
 }
 
 // --- Individual command handlers ---
@@ -284,8 +332,7 @@ async function handleTrace(msg: IncomingMessage, args: Record<string, any>, deps
 
 async function handleSchedule(msg: IncomingMessage, parsedRepo: string | undefined, deps: HandlerDeps) {
   await msg.updateStatus("Parsing your schedule...");
-  const rawRepos = getUserRepos(deps.config, msg.userId);
-  const userRepos = rawRepos.includes("*") ? deps.repoRegistry.getAllNames() : rawRepos;
+  const userRepos = getUserRepoNames(msg.userId, deps);
 
   if (userRepos.length === 0) {
     await msg.reply("You don't have access to any repos. Set one up first with `init repo <name> <git-url>`.");
@@ -396,77 +443,35 @@ async function handleInitRepo(msg: IncomingMessage, args: Record<string, any>, d
 }
 
 async function handleTaskMessage(msg: IncomingMessage, parsed: ParsedMessage, deps: HandlerDeps) {
-  // If the repo hint doesn't match a known repo, clear it and let auto-resolution handle it
-  if (parsed.repo && !deps.getRepoInfo(parsed.repo)) {
-    parsed.repo = undefined;
-  }
+  const hint = parsed.repo && deps.getRepoInfo(parsed.repo) ? parsed.repo : undefined;
+  const resolution = await resolveRepoWithLLM(msg.userId, parsed.rawText, hint, deps, (text) => msg.updateStatus(text));
 
-  if (!parsed.repo) {
-    const userRepos = getUserRepos(deps.config, msg.userId);
-    const hasWildcard = userRepos.includes("*");
-
-    if (!hasWildcard && userRepos.length === 1) {
-      parsed.repo = userRepos[0];
-    } else if (hasWildcard || userRepos.length > 1) {
-      const repoNames = hasWildcard ? deps.repoRegistry.getAllNames() : userRepos;
-
-      if (repoNames.length === 1) {
-        parsed.repo = repoNames[0];
-      } else if (repoNames.length === 0) {
-        const reply = "No repos discovered yet. Set one up with `init repo <name> <git-url>` or configure GitHub sync.";
-        await msg.reply(reply);
-        return;
-      } else {
-        // Try last completed task's repo first (cheap)
-        const recentTasks = deps.queue.listByUser(msg.userId, 5);
-        const lastRepo = recentTasks.find(t => t.status === "completed" || t.status === "failed")?.repo;
-        if (lastRepo && repoNames.includes(lastRepo)) {
-          parsed.repo = lastRepo;
-          logger.info("repo resolved from recent task", { resolved: lastRepo, userText: parsed.rawText.slice(0, 80) });
-        } else {
-          // Resolve repo via a quick LLM call, then enqueue through the normal path
-          const repoList = repoNames.join(", ");
-          const history = deps.sessions.getHistory(msg.userId, 6);
-          const historyContext = history.length > 1
-            ? "Recent conversation:\n" + history.slice(0, -1).map(m => `${m.role}: ${m.content}`).join("\n") + "\n\n"
-            : "";
-          const resolvePrompt = `You are a repo-name resolver. ${historyContext}The user's latest message:\n"${parsed.rawText}"\n\nAvailable repos: ${repoList}\n\nRespond with ONLY the repo name that best matches their request. Consider the conversation context if the current message doesn't mention a specific repo. Nothing else — just the exact repo name from the list. If you cannot determine which repo, respond with "UNKNOWN".`;
-
-          await msg.updateStatus("Figuring out which repo...");
-          try {
-            const runner = deps.getRunner(deps.config.runner?.name);
-            const result = await runner.run(resolvePrompt, deps.config.reposDir, { maxTurns: 1 });
-            const resolved = result.output.trim().replace(/[`"']/g, "");
-
-            if (resolved === "UNKNOWN" || !repoNames.includes(resolved)) {
-              const reply = `Which repo? I see ${repoNames.length} repos. Some matches: ${repoNames.slice(0, 10).join(", ")}${repoNames.length > 10 ? "..." : ""}.\nSay it again with 'on <repo>'.`;
-              await msg.reply(reply);
-              deps.sessions.addMessage(msg.userId, "assistant", reply);
-              return;
-            }
-
-            parsed.repo = resolved;
-            logger.info("repo resolved via LLM", { resolved, userText: parsed.rawText.slice(0, 80) });
-          } catch (err) {
-            await msg.reply(`Couldn't figure out the repo: ${String(err).slice(0, 300)}`);
-            return;
-          }
-        }
-      }
-    } else {
-      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\``;
+  switch (resolution.kind) {
+    case "none": {
+      const history = deps.sessions.getHistory(msg.userId, 6);
+      return handleDiscuss(msg, { ...parsed, type: "free-form" }, history, deps);
+    }
+    case "unknown": {
+      const { repoNames } = resolution;
+      const reply = `Which repo? I see ${repoNames.length} repos. Some matches: ${repoNames.slice(0, 10).join(", ")}${repoNames.length > 10 ? "..." : ""}.\nSay it again with 'on <repo>'.`;
       await msg.reply(reply);
+      deps.sessions.addMessage(msg.userId, "assistant", reply);
+      return;
+    }
+    case "error": {
+      await msg.reply(resolution.message);
       return;
     }
   }
 
+  parsed.repo = resolution.repo;
+
   if (!isAuthorized(deps.config, msg.userId, parsed.repo)) {
     await msg.reply(`Not authorized for ${parsed.repo}.`);
     return;
   }
 
-  const repoInfo = deps.getRepoInfo(parsed.repo);
-  if (!repoInfo) {
+  if (!deps.getRepoInfo(parsed.repo)) {
     await msg.reply(`Unknown repo: ${parsed.repo}`);
     return;
   }