@clawmem-ai/clawmem 0.1.7 → 0.1.8

package/openclaw.plugin.json

@@ -1,7 +1,7 @@
 {
   "id": "clawmem",
   "name": "ClawMem",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Mirror OpenClaw sessions into GitHub-compatible issues and comments.",
   "kind": "memory",
   "configSchema": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@clawmem-ai/clawmem",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Mirror OpenClaw sessions into GitHub-compatible issues and comments.",
   "type": "module",
   "license": "MIT",

package/src/conversation.test.ts

@@ -0,0 +1,76 @@
+// Tests for conversation title derivation logic.
+import { deriveInitialTitle } from "./conversation.js";
+import type { NormalizedMessage } from "./types.js";
+
+function msg(role: string, text: string): NormalizedMessage {
+  return { role, text };
+}
+
+const tests: Array<{ name: string; messages: NormalizedMessage[]; sessionId: string; expected: string | RegExp }> = [
+  {
+    name: "uses first user message",
+    messages: [msg("user", "How do I configure Redis rate limiting?")],
+    sessionId: "abc123",
+    expected: "How do I configure Redis rate limiting?",
+  },
+  {
+    name: "truncates long messages to 50 chars",
+    messages: [msg("user", "I need help with configuring the distributed rate limiting system for our production Redis cluster")],
+    sessionId: "abc123",
+    expected: /^I need help with configuring the distributed rate…$/,
+  },
+  {
+    name: "strips markdown formatting",
+    messages: [msg("user", "## How do I **configure** `Redis` rate limiting?")],
+    sessionId: "abc123",
+    expected: "How do I configure Redis rate limiting?",
+  },
+  {
+    name: "falls back to session ID for short messages",
+    messages: [msg("user", "hi")],
+    sessionId: "abc-def-123",
+    expected: "Session: abc-def-123",
+  },
+  {
+    name: "falls back to session ID when no user messages",
+    messages: [msg("assistant", "Hello!")],
+    sessionId: "xyz-789",
+    expected: "Session: xyz-789",
+  },
+  {
+    name: "falls back to session ID for empty messages",
+    messages: [],
+    sessionId: "empty-sess",
+    expected: "Session: empty-sess",
+  },
+  {
+    name: "collapses whitespace",
+    messages: [msg("user", "How do   I    configure\n\nRedis?")],
+    sessionId: "abc",
+    expected: "How do I configure Redis?",
+  },
+  {
+    name: "skips assistant messages, uses first user message",
+    messages: [msg("assistant", "Welcome!"), msg("user", "Fix the login bug please")],
+    sessionId: "abc",
+    expected: "Fix the login bug please",
+  },
+];
+
+let passed = 0;
+let failed = 0;
+
+for (const t of tests) {
+  const got = deriveInitialTitle(t.messages, t.sessionId);
+  const ok = t.expected instanceof RegExp ? t.expected.test(got) : got === t.expected;
+  if (!ok) {
+    console.error(`FAIL: ${t.name}\n  got:      ${JSON.stringify(got)}\n  expected: ${t.expected instanceof RegExp ? t.expected.toString() : JSON.stringify(t.expected)}`);
+    failed++;
+  } else {
+    console.log(`PASS: ${t.name}`);
+    passed++;
+  }
+}
+
+console.log(`\n${passed} passed, ${failed} failed`);
+if (failed > 0) process.exit(1);

package/src/conversation.ts

@@ -47,7 +47,8 @@ export class ConversationMirror {
       );
       this.resetIssueBinding(session);
     }
-    const title = `${SESSION_TITLE_PREFIX}${session.sessionId}`;
+    // Use first user message as title (truncated), falling back to session ID.
+    const title = deriveInitialTitle(snapshot.messages, session.sessionId);
     const labels = this.buildLabels(session, snapshot, false);
     const body = this.renderBody(session, snapshot, "pending", false);
     await this.client.ensureLabels(labels);
@@ -59,9 +60,10 @@ export class ConversationMirror {
     session.updatedAt = session.createdAt;
   }
 
-  async syncBody(session: SessionMirrorState, snapshot: TranscriptSnapshot, summary: string, closed: boolean): Promise<void> {
+  async syncBody(session: SessionMirrorState, snapshot: TranscriptSnapshot, summary: string, closed: boolean, titleOverride?: string): Promise<void> {
     if (!session.issueNumber) return;
-    const title = `${SESSION_TITLE_PREFIX}${session.sessionId}`;
+    // Prefer explicit override (LLM-generated title), then keep existing title, then fall back to session ID.
+    const title = titleOverride?.trim() || session.issueTitle || `${SESSION_TITLE_PREFIX}${session.sessionId}`;
     const body = this.renderBody(session, snapshot, summary, closed);
     const hash = sha256(`${title}\n${body}\n${closed ? "closed" : "open"}`);
     if (hash === session.lastSummaryHash) return;
@@ -86,22 +88,28 @@ export class ConversationMirror {
     return count;
   }
 
-  async generateSummary(session: SessionMirrorState, snapshot: TranscriptSnapshot): Promise<string> {
+  async generateSummaryAndTitle(session: SessionMirrorState, snapshot: TranscriptSnapshot): Promise<{ summary: string; title?: string }> {
     if (snapshot.messages.length === 0) throw new Error("no conversation messages to summarize");
     const subagent = this.api.runtime.subagent;
     const sessionKey = subKey(session, "summary");
     const message = [
-      "Summarize the following conversation.",
-      'Return valid JSON only in the form {"summary":"..."}',
+      "Summarize the following conversation and generate a short title.",
+      'Return valid JSON only in the form {"summary":"...","title":"..."}',
       "The summary should be concise, factual, and written in 2-4 sentences.",
       "Do not include markdown, bullet points, or analysis.",
+      "",
+      "Title rules:",
+      "- Under 50 characters, evocative like a good article headline.",
+      "- Should make someone curious to read the conversation.",
+      "- Must be in the same language as the majority of the conversation content.",
+      "- Good: creative, captures the spirit. Bad: dry meeting-minutes style.",
       "", "<conversation>", fmtTranscript(snapshot.messages), "</conversation>",
     ].join("\n");
     try {
       const run = await subagent.run({
         sessionKey, message, deliver: false, lane: "clawmem-summary",
-        idempotencyKey: sha256(`${session.sessionId}:${snapshot.messages.length}:summary`),
-        extraSystemPrompt: "You summarize OpenClaw conversations. Output JSON only with one string field named summary.",
+        idempotencyKey: sha256(`${session.sessionId}:${snapshot.messages.length}:summary-v2`),
+        extraSystemPrompt: "You summarize OpenClaw conversations and generate evocative titles. Output JSON only with string fields summary and title.",
       });
       const wait = await subagent.waitForRun({ runId: run.runId, timeoutMs: this.config.summaryWaitTimeoutMs });
       if (wait.status === "timeout") throw new Error("summary subagent timed out");
@@ -109,7 +117,7 @@ export class ConversationMirror {
       const msgs = normalizeMessages((await subagent.getSessionMessages({ sessionKey, limit: 50 })).messages);
       const text = [...msgs].reverse().find((e) => e.role === "assistant" && e.text.trim())?.text;
       if (!text) throw new Error("summary subagent returned no assistant text");
-      return parseSummary(text);
+      return parseSummaryAndTitle(text);
     } finally { subagent.deleteSession({ sessionKey, deleteTranscript: true }).catch(() => {}); }
   }
 
@@ -184,11 +192,36 @@ function isNotFoundError(error: unknown): boolean {
   const text = String(error);
   return text.includes("HTTP 404");
 }
-function parseSummary(raw: string): string {
-  const tryParse = (s: string): string | null => {
-    try { const p = JSON.parse(s) as { summary?: unknown }; return typeof p?.summary === "string" && p.summary.trim() ? p.summary.trim() : null; }
-    catch { const i = s.indexOf("{"), j = s.lastIndexOf("}");
-      if (i >= 0 && j > i) { try { const p = JSON.parse(s.slice(i, j + 1)) as { summary?: unknown }; return typeof p?.summary === "string" && p.summary.trim() ? p.summary.trim() : null; } catch { return null; } }
+/** Derive a conversation title from the first user message, truncated to 50 chars. */
+export function deriveInitialTitle(messages: NormalizedMessage[], sessionId: string): string {
+  const firstUserMsg = messages.find((m) => m.role === "user")?.text?.trim() ?? "";
+  // Strip markdown, collapse whitespace.
+  const clean = firstUserMsg.replace(/[#*`~>|]/g, "").replace(/\s+/g, " ").trim();
+  if (clean.length >= 5) {
+    return clean.length <= 50 ? clean : clean.slice(0, 49).trimEnd() + "…";
+  }
+  return `${SESSION_TITLE_PREFIX}${sessionId}`;
+}
+
+function parseSummaryAndTitle(raw: string): { summary: string; title?: string } {
+  const tryParse = (s: string): { summary: string; title?: string } | null => {
+    try {
+      const p = JSON.parse(s) as { summary?: unknown; title?: unknown };
+      const summary = typeof p?.summary === "string" && p.summary.trim() ? p.summary.trim() : null;
+      if (!summary) return null;
+      const title = typeof p?.title === "string" && p.title.trim() ? p.title.trim() : undefined;
+      return { summary, title };
+    } catch {
+      const i = s.indexOf("{"), j = s.lastIndexOf("}");
+      if (i >= 0 && j > i) {
+        try {
+          const p = JSON.parse(s.slice(i, j + 1)) as { summary?: unknown; title?: unknown };
+          const summary = typeof p?.summary === "string" && p.summary.trim() ? p.summary.trim() : null;
+          if (!summary) return null;
+          const title = typeof p?.title === "string" && p.title.trim() ? p.title.trim() : undefined;
+          return { summary, title };
+        } catch { return null; }
+      }
       return null;
     }
   };
@@ -196,5 +229,5 @@ function parseSummary(raw: string): string {
   const direct = tryParse(t); if (direct) return direct;
   const f = /^```(?:json)?\s*([\s\S]*?)```$/i.exec(t);
   if (f?.[1]) { const nested = tryParse(f[1].trim()); if (nested) return nested; }
-  return t;
+  return { summary: t };
 }

package/src/github-client.ts

@@ -38,8 +38,15 @@ export class GitHubIssueClient {
     const unmanaged = extractLabelNames(issue.labels).filter((l) => !isManagedLabel(l));
     await this.updateIssue(issueNumber, { labels: [...new Set([...unmanaged, ...desired])] });
   }
-  async createAnonymousSession(): Promise<AnonymousSessionResponse> {
-    return this.req<AnonymousSessionResponse>("anonymous/session", { method: "POST" }, { omitAuth: true });
+  async getRepoInfo(): Promise<{ description?: string; name?: string }> {
+    return this.req<{ description?: string; name?: string }>(this.repoPath("").replace(/\/$/, ""), { method: "GET" });
+  }
+  async updateRepoDescription(description: string): Promise<void> {
+    await this.req(this.repoPath("").replace(/\/$/, ""), { method: "PATCH", body: JSON.stringify({ description }) });
+  }
+  async createAnonymousSession(locale?: string): Promise<AnonymousSessionResponse> {
+    const body = locale ? JSON.stringify({ locale }) : undefined;
+    return this.req<AnonymousSessionResponse>("anonymous/session", { method: "POST", ...(body ? { body } : {}) }, { omitAuth: true });
   }
 
   private repoPath(suffix: string): string {

package/src/service.ts

@@ -151,12 +151,20 @@ class ClawMemService {
     let allOk = true;
     if (next.length > 0) { const n = await conv.appendComments(s.issueNumber!, next); s.lastMirroredCount += n; s.turnCount += n; allOk = n === next.length; }
     let summary = "pending";
-    try { summary = await conv.generateSummary(s, snap); } catch (e) { summary = `failed: ${String(e)}`; }
+    let generatedTitle: string | undefined;
+    try {
+      const result = await conv.generateSummaryAndTitle(s, snap);
+      summary = result.summary;
+      generatedTitle = result.title;
+    } catch (e) { summary = `failed: ${String(e)}`; }
     await conv.syncLabels(s, snap, true);
-    await conv.syncBody(s, snap, summary, true);
+    await conv.syncBody(s, snap, summary, true, generatedTitle);
     await mem.syncFromConversation(s, snap);
     if (allOk) s.finalizedAt = new Date().toISOString();
     await this.persistState();
+
+    // Auto-name the repo if it still has no description (first few conversations).
+    this.maybeAutoNameRepo(agentId, summary, generatedTitle);
   }
 
   // --- Infrastructure ---
@@ -226,7 +234,8 @@ class ClawMemService {
     if (!route.baseUrl) { this.api.logger.warn(`clawmem: cannot provision Git credentials for ${agentId} without a baseUrl`); return false; }
     try {
       const client = new GitHubIssueClient(route, this.api.logger);
-      const sess = await client.createAnonymousSession();
+      const locale = Intl?.DateTimeFormat?.()?.resolvedOptions?.()?.locale ?? "";
+      const sess = await client.createAnonymousSession(locale);
       await this.persistAgentConfig(agentId, { baseUrl: route.baseUrl, authScheme: "token", token: sess.token, repo: sess.repo_full_name });
       this.config.agents[agentId] = { ...(this.config.agents[agentId] ?? {}), baseUrl: route.baseUrl, authScheme: "token", token: sess.token, repo: sess.repo_full_name };
       this.api.logger.info?.(`clawmem: provisioned Git credentials for agent ${agentId} -> ${sess.repo_full_name} via ${route.baseUrl}`);
@@ -267,6 +276,28 @@ class ClawMemService {
       mem: new MemoryStore(client, this.api, this.config),
     };
   }
+  /**
+   * After finalization, check if the repo still has an empty/default description.
+   * If so, use the conversation summary to suggest a meaningful name and update
+   * the repo description automatically. Best-effort, fire-and-forget.
+   */
+  private maybeAutoNameRepo(agentId: string, summary: string, title?: string): void {
+    if (!summary || summary.startsWith("failed:") || summary === "pending") return;
+    const snippet = title || summary.slice(0, 100);
+    void (async () => {
+      try {
+        const client = new GitHubIssueClient(resolveAgentRoute(this.config, agentId), this.api.logger);
+        const repo = await client.getRepoInfo();
+        // Only auto-name if description is still empty or a default placeholder.
+        if (repo.description && repo.description !== "My Memory Space" && repo.description !== "我的记忆空间" && repo.description !== "マイメモリースペース") return;
+        // Use the conversation title or summary as a lightweight description.
+        await client.updateRepoDescription(snippet);
+        this.api.logger.info?.(`clawmem: auto-named repo to "${snippet}"`);
+      } catch (e) {
+        this.api.logger.warn(`clawmem: auto-name repo failed: ${String(e)}`);
+      }
+    })();
+  }
   private warn(scope: string, error: unknown): void { this.api.logger.warn(`clawmem: ${scope} failed: ${String(error)}`); }
 }