Fix connectors overwriting link titles on comment updates

Author: KrisBraun

connectors/AGENTS.md

@@ -903,7 +903,8 @@ After creating a new connector, add it to `pnpm-workspace.yaml` if not already c
 12. **❌ Forgetting to clean up on disable** — Delete callbacks, webhooks, and stored state
 13. **❌ Two-way sync without metadata correlation** — Embed Plot ID in external item metadata to prevent duplicates from race conditions (see SYNC_STRATEGIES.md §6)
 14. **❌ Stripping HTML tags locally** — Pass raw HTML with `contentType: "html"` for server-side conversion. Local regex stripping breaks encoding and loses links
-15. **❌ Not setting `created` on notes from external data** — Always pass the external system's timestamp (e.g., `internalDate` from Gmail, `created_at` from an API) as the note's `created` field. Omitting it defaults to sync time, making all notes appear to have been created "just now"
+15. **❌ Using placeholder titles in comment/update webhooks** — `title` always overwrites on upsert. Always use the real entity title (fetch from API if not in the webhook payload). Never use IDs or keys as placeholder titles
+16. **❌ Not setting `created` on notes from external data** — Always pass the external system's timestamp (e.g., `internalDate` from Gmail, `created_at` from an API) as the note's `created` field. Omitting it defaults to sync time, making all notes appear to have been created "just now"
 
 ## Study These Examples
 

connectors/jira/src/jira.ts

@@ -764,7 +764,7 @@ export class Jira extends Connector<Jira> {
     const link: NewLinkWithNotes = {
       ...(source ? { source } : {}),
       type: "issue",
-      title: issue.key, // Placeholder; upsert by source will preserve existing title
+      title: issue.fields?.summary || issue.key,
       notes: [
         {
           key: `comment-${comment.id}`,

connectors/linear/src/linear.ts

@@ -783,6 +783,10 @@ export class Linear extends Connector<Linear> {
       return;
     }
 
+    // Fetch issue title from Linear API (title always overwrites on upsert)
+    const client = await this.getClient(projectId);
+    const issue = await client.issue(issueId);
+
     // Extract comment author from webhook payload
     const commentAuthor = this.resolveAuthorContact(comment.user);
 
@@ -792,7 +796,7 @@ export class Linear extends Connector<Linear> {
     const newLink: NewLinkWithNotes = {
       source: threadSource,
       type: "issue",
-      title: issueId, // Placeholder; upsert by source will preserve existing title
+      title: issue.title,
       notes: [
         {
           key: `comment-${comment.id}`,

twister/src/plot.ts

@@ -904,7 +904,12 @@ export type NewLink = (
  * Creates a thread+link pair, with notes attached to the thread.
  */
 export type NewLinkWithNotes = NewLink & {
-  /** Title for the link and its thread container */
+  /**
+   * Title for the link and its thread container.
+   * Must be the real entity title (e.g. issue title, message subject),
+   * never a placeholder or ID. This value overwrites the existing title on upsert.
+   * If the title is not available in the webhook payload, fetch it from the API.
+   */
   title: string;
   /** Notes to attach to the thread */
   notes?: Omit<NewNote, "thread">[];