fix(files): keep replacement content literal

Author: icecrasher321

apps/sim/lib/copilot/tools/server/files/edit-content.test.ts

@@ -154,7 +154,7 @@ describe('edit_content', () => {
     )
   })
 
-  it('honors replaceAll for exact patch intent', async () => {
+  it('honors replaceAll with literal replacement content for exact patch intent', async () => {
     waitForLatestFileIntentMock.mockResolvedValue({
       operation: 'patch',
       fileId: 'text-1',
@@ -168,13 +168,11 @@ describe('edit_content', () => {
       createdAt: Date.now(),
     })
 
-    await expect(editContentServerTool.execute({ content: 'new' }, context)).resolves.toMatchObject(
-      {
-        success: true,
-      }
-    )
+    await expect(editContentServerTool.execute({ content: '$&' }, context)).resolves.toMatchObject({
+      success: true,
+    })
     expect(compileDocForWriteMock).toHaveBeenCalledWith(
-      expect.objectContaining({ source: 'new new' })
+      expect.objectContaining({ source: '$& $&' })
     )
   })
 })

apps/sim/lib/copilot/tools/server/files/file-preview.test.ts

@@ -99,4 +99,19 @@ describe('buildFilePreviewText', () => {
       })
     ).toBe('hello sim')
   })
+
+  it('treats replaceAll preview content as literal text', () => {
+    expect(
+      buildFilePreviewText({
+        operation: 'patch',
+        existingContent: 'old old',
+        streamedContent: '$&',
+        edit: {
+          strategy: 'search_replace',
+          search: 'old',
+          replaceAll: true,
+        },
+      })
+    ).toBe('$& $&')
+  })
 })

apps/sim/lib/copilot/tools/server/files/file-preview.ts

@@ -53,7 +53,7 @@ function extractPatchPreview(
     const search = typeof edit?.search === 'string' ? edit.search : ''
     if (!search) return undefined
     if (edit?.replaceAll === true) {
-      return existingContent.replaceAll(search, streamedContent)
+      return existingContent.replaceAll(search, () => streamedContent)
     }
     const firstIdx = existingContent.indexOf(search)
     if (firstIdx === -1) return undefined

apps/sim/lib/workspace-files/edit-content.test.ts

@@ -70,6 +70,10 @@ describe('applyStringReplacement', () => {
     expect(applyStringReplacement('a a a', 'a', 'b', true)).toBe('b b b')
   })
 
+  it('treats replacement substitution tokens as literal content', () => {
+    expect(applyStringReplacement('a a', 'a', '$&', true)).toBe('$& $&')
+  })
+
   it('rejects an oversized replaceAll result before constructing it', () => {
     expect(() =>
       applyStringReplacement('aaaa', 'a', '0123456789', true, { maxOutputBytes: 20 })

apps/sim/lib/workspace-files/edit-content.ts

@@ -134,7 +134,7 @@ export function applyStringReplacement(
     options?.maxOutputBytes
   )
 
-  if (replaceAll) return text.replaceAll(search, content)
+  if (replaceAll) return text.replaceAll(search, () => content)
 
   const index = text.indexOf(search)
   return text.slice(0, index) + content + text.slice(index + search.length)