refactor(chat): share generic resource-title set across route and persistence

Dedupes the placeholder-title set that the chat-resource route and the
server-side persistence merge each defined separately. They had already
drifted ('Log' was missing from persistence); a single shared
GENERIC_RESOURCE_TITLES keeps title-upgrade behavior consistent, including
the new 'Scheduled Task' placeholder.

Author: waleedlatif1

apps/sim/app/api/copilot/chat/resources/route.ts

@@ -17,6 +17,7 @@ import {
   createUnauthorizedResponse,
 } from '@/lib/copilot/request/http'
 import type { ChatResource, ResourceType } from '@/lib/copilot/resources/persistence'
+import { GENERIC_RESOURCE_TITLES } from '@/lib/copilot/resources/types'
 import { withRouteHandler } from '@/lib/core/utils/with-route-handler'
 
 const logger = createLogger('CopilotChatResourcesAPI')
@@ -31,15 +32,6 @@ const VALID_RESOURCE_TYPES = new Set<ResourceType>([
   'log',
   'integration',
 ])
-const GENERIC_TITLES = new Set([
-  'Table',
-  'File',
-  'Workflow',
-  'Knowledge Base',
-  'Folder',
-  'Scheduled Task',
-  'Log',
-])
 
 export const POST = withRouteHandler(async (req: NextRequest) => {
   try {
@@ -85,7 +77,7 @@ export const POST = withRouteHandler(async (req: NextRequest) => {
 
     let merged: ChatResource[]
     if (prev) {
-      if (GENERIC_TITLES.has(prev.title) && !GENERIC_TITLES.has(resource.title)) {
+      if (GENERIC_RESOURCE_TITLES.has(prev.title) && !GENERIC_RESOURCE_TITLES.has(resource.title)) {
         merged = existing.map((r) =>
           `${r.type}:${r.id}` === key ? { ...r, title: resource.title } : r
         )

apps/sim/lib/copilot/resources/persistence.ts

@@ -3,7 +3,7 @@ import { copilotChats } from '@sim/db/schema'
 import { createLogger } from '@sim/logger'
 import { toError } from '@sim/utils/errors'
 import { eq, sql } from 'drizzle-orm'
-import type { MothershipResource } from './types'
+import { GENERIC_RESOURCE_TITLES, type MothershipResource } from './types'
 
 export {
   extractDeletedResourcesFromToolResult,
@@ -42,7 +42,6 @@ export async function persistChatResources(
 
     const existing = Array.isArray(chat.resources) ? (chat.resources as ChatResource[]) : []
     const map = new Map<string, ChatResource>()
-    const GENERIC = new Set(['Table', 'File', 'Workflow', 'Knowledge Base', 'Folder', 'Log'])
 
     for (const r of existing) {
       map.set(`${r.type}:${r.id}`, r)
@@ -51,7 +50,10 @@ export async function persistChatResources(
     for (const r of toMerge) {
       const key = `${r.type}:${r.id}`
       const prev = map.get(key)
-      if (!prev || (GENERIC.has(prev.title) && !GENERIC.has(r.title))) {
+      if (
+        !prev ||
+        (GENERIC_RESOURCE_TITLES.has(prev.title) && !GENERIC_RESOURCE_TITLES.has(r.title))
+      ) {
         map.set(key, r)
       }
     }

apps/sim/lib/copilot/resources/types.ts

@@ -25,6 +25,22 @@ export function isEphemeralResource(resource: MothershipResource): boolean {
   return resource.type === 'generic' || resource.id === 'streaming-file'
 }
 
+/**
+ * Placeholder resource titles emitted before a specific name is known. A more
+ * specific title may overwrite one of these during dedup; a specific title is
+ * never downgraded back to a placeholder. Shared by the chat-resource route and
+ * the server-side persistence merge so the two stay in lockstep.
+ */
+export const GENERIC_RESOURCE_TITLES = new Set<string>([
+  'Table',
+  'File',
+  'Workflow',
+  'Knowledge Base',
+  'Folder',
+  'Scheduled Task',
+  'Log',
+])
+
 export const VFS_DIR_TO_RESOURCE: Record<string, MothershipResourceType> = {
   tables: 'table',
   files: 'file',