review fixes

Author: charlesvien

apps/twig/src/renderer/sagas/task/task-creation.ts

@@ -40,7 +40,7 @@ Convert the task description into a concise task title.
 - Avoid using jargon or overly technical terms unless absolutely necessary.
 - The title should be easy to understand for anyone reading it.
 - Use sentence case (capitalize only first word and proper nouns)
--Remove: the, this, my, a, an
+- Remove: the, this, my, a, an
 - If possible, start with action verbs (Fix, Implement, Analyze, Debug, Update, Research, Review)
 - Keep exact: technical terms, numbers, filenames, HTTP codes, PR numbers
 - Never assume tech stack
@@ -54,19 +54,23 @@ Examples:
 - "Review pull request #123" → Review pull request #123
 - "debug 500 errors in production" → Debug production 500 errors
 - "why is the payment flow failing" → Analyze payment flow failure
-- "So how about that weather huh" → "Weather chat"
-- "dsfkj sdkfj help me code" → "Coding help request"
-- "👋😊" → "Friendly greeting"
-- "aaaaaaaaaa" → "Repeated letters"
-- "   " → "Empty message"
-- "What's the best restaurant in NYC?" → "NYC restaurant recommendations"`;
+- "So how about that weather huh" → Weather chat
+- "dsfkj sdkfj help me code" → Coding help request
+- "👋😊" → Friendly greeting
+- "aaaaaaaaaa" → Repeated letters
+- "   " → Empty message
+- "What's the best restaurant in NYC?" → NYC restaurant recommendations
+
+Never wrap the title in quotes.`;
 
 async function generateTaskTitle(
   taskId: string,
   description: string,
   posthogClient: PostHogAPIClient,
 ): Promise<void> {
   try {
+    if (!description.trim()) return;
+
     const authState = useAuthStore.getState();
     const apiKey = authState.oauthAccessToken;
     const cloudRegion = authState.cloudRegion;
@@ -80,7 +84,7 @@ async function generateTaskTitle(
       messages: [{ role: "user", content: description }],
     });
 
-    const title = result.content.trim();
+    const title = result.content.trim().replace(/^["']|["']$/g, "");
     if (!title) return;
 
     await posthogClient.updateTask(taskId, { title });

packages/agent/src/adapters/claude/claude-agent.ts

@@ -170,7 +170,9 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
 
     // Fire off MCP metadata fetch early — it populates a module-level cache
     // used later during permission checks, not needed by buildSessionOptions or query()
-    fetchMcpToolMetadata(mcpServers, this.logger);
+    fetchMcpToolMetadata(mcpServers, this.logger).catch((err) => {
+      this.logger.warn("Failed to fetch MCP tool metadata", { err });
+    });
 
     const options = timeSync("buildSessionOptions", () =>
       buildSessionOptions({
@@ -269,7 +271,9 @@ export class ClaudeAcpAgent extends BaseAcpAgent {
     const mcpServers = parseMcpServers(params);
 
     // Fire off MCP metadata fetch early — populates cache for permission checks
-    fetchMcpToolMetadata(mcpServers, this.logger);
+    fetchMcpToolMetadata(mcpServers, this.logger).catch((err) => {
+      this.logger.warn("Failed to fetch MCP tool metadata on resume", { err });
+    });
 
     const permissionMode: TwigExecutionMode =
       meta?.permissionMode &&