fix: normalize cursor source to cursor-sync to prevent duplicate dropdown entries

Author: waynesutton

.gitignore

@@ -26,4 +26,7 @@ dist/
 # Logs
 *.log
 
+# Documentation (internal)
+docs/
+
 

changelog.md

@@ -6,6 +6,17 @@ Format based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ## [Unreleased]
 
+### Fixed
+
+- Fixed duplicate "cursor" and "Cursor" entries in source filter dropdown
+  - Normalized source values: "cursor" is now converted to "cursor-sync" on sync
+  - Updated sessions.upsert and batchUpsert to normalize source on insert/update
+  - Updated messages.upsert and batchUpsert to normalize source for auto-created sessions
+  - Updated users.me query to normalize enabledAgents (converts "cursor" to "cursor-sync", deduplicates)
+  - Updated users.updateEnabledAgents to normalize input before saving
+  - Added "cursor" as alias in source.ts getSourceLabel and getSourceColorClass
+  - Added "cursor" mapping in Dashboard AI_AGENTS_MAP for legacy data display
+
 ### Added
 
 - Sessions pagination: loads 40 sessions initially with "Load more" button for 20 at a time

convex/messages.ts

@@ -60,6 +60,9 @@ export const upsert = internalMutation({
 
     // Auto-create session if it doesn't exist (handles out-of-order sync)
     if (!session) {
+      // Normalize source: "cursor" -> "cursor-sync" for consistency
+      const rawSource = args.source || "opencode";
+      const normalizedSource = rawSource === "cursor" ? "cursor-sync" : rawSource;
       sessionId = await ctx.db.insert("sessions", {
         userId: args.userId,
         externalId: args.sessionExternalId,
@@ -68,7 +71,7 @@ export const upsert = internalMutation({
         projectName: undefined,
         model: args.model,
         provider: undefined,
-        source: args.source || "opencode",
+        source: normalizedSource,
         promptTokens: 0,
         completionTokens: 0,
         totalTokens: 0,
@@ -258,6 +261,9 @@ export const batchUpsert = internalMutation({
       if (!session) {
         // Create session for out-of-order messages
         const firstMsg = messages[0];
+        // Normalize source: "cursor" -> "cursor-sync" for consistency
+        const rawSource = firstMsg.source || "opencode";
+        const normalizedSource = rawSource === "cursor" ? "cursor-sync" : rawSource;
         sessionId = await ctx.db.insert("sessions", {
           userId: args.userId,
           externalId: sessionExternalId,
@@ -266,7 +272,7 @@ export const batchUpsert = internalMutation({
           projectName: undefined,
           model: firstMsg.model,
           provider: undefined,
-          source: firstMsg.source || "opencode",
+          source: normalizedSource,
           promptTokens: 0,
           completionTokens: 0,
           totalTokens: 0,

convex/sessions.ts

@@ -322,7 +322,9 @@ export const upsert = internalMutation({
 
     const promptTokens = args.promptTokens ?? 0;
     const completionTokens = args.completionTokens ?? 0;
-    const source = args.source || "opencode";
+    // Normalize source: "cursor" -> "cursor-sync" for consistency
+    const rawSource = args.source || "opencode";
+    const source = rawSource === "cursor" ? "cursor-sync" : rawSource;
 
     if (existing) {
       // Idempotency check: skip if recently updated with same key values
@@ -593,7 +595,9 @@ export const batchUpsert = internalMutation({
 
         const promptTokens = session.promptTokens ?? 0;
         const completionTokens = session.completionTokens ?? 0;
-        const source = session.source || "opencode";
+        // Normalize source: "cursor" -> "cursor-sync" for consistency
+        const rawSource = session.source || "opencode";
+        const source = rawSource === "cursor" ? "cursor-sync" : rawSource;
 
         if (existing) {
           // Idempotency check

convex/users.ts

@@ -29,13 +29,20 @@ export const me = query({
 
     if (!user) return null;
 
+    // Normalize enabledAgents: convert "cursor" -> "cursor-sync" and deduplicate
+    let normalizedAgents = user.enabledAgents;
+    if (normalizedAgents) {
+      normalizedAgents = normalizedAgents.map((a) => (a === "cursor" ? "cursor-sync" : a));
+      normalizedAgents = [...new Set(normalizedAgents)];
+    }
+
     return {
       _id: user._id,
       email: user.email,
       name: user.name,
       avatarUrl: user.avatarUrl,
       hasApiKey: !!user.apiKey,
-      enabledAgents: user.enabledAgents,
+      enabledAgents: normalizedAgents,
       createdAt: user.createdAt,
     };
   },
@@ -146,9 +153,14 @@ export const updateEnabledAgents = mutation({
 
     if (!user) throw new Error("User not found");
 
+    // Normalize: convert "cursor" -> "cursor-sync" and deduplicate
+    const normalizedAgents = [...new Set(
+      enabledAgents.map((a) => (a === "cursor" ? "cursor-sync" : a))
+    )];
+
     // Patch directly for idempotency
     await ctx.db.patch(user._id, {
-      enabledAgents,
+      enabledAgents: normalizedAgents,
       updatedAt: Date.now(),
     });