fix(chat): include the line range in file-selection equality

Cursor Bugbot: areContextsEqual compared only fileId and text for
file_selection, while the comment directly above claimed equality was the
selected range. A line that occurs twice in a file — a repeated import, a
closing brace — highlighted at both places produced identical text, so
prepareContextForInsert called the second a duplicate and dropped its chip,
even though the labels (notes.md:12 vs notes.md:50) were plainly different.

Comparing startLine/endLine as well makes the code match what the comment
promised. The rich-markdown editor omits the range, so both sides are undefined
there and identical text still dedupes — correct, since two identical passages
are genuinely indistinguishable without line numbers.

Tests cover both: distinct ranges stay distinct, an exact repeat still dedupes,
and the no-line-number path still dedupes. Verified the first fails against
text-only equality.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/utils.test.ts

@@ -67,6 +67,22 @@ describe('prepareContextForInsert', () => {
     expect(prepareContextForInsert(other, [fileSelection()])).not.toBeNull()
   })
 
+  it('distinguishes identical text highlighted at two places in one file', () => {
+    // A repeated line — an import, a closing brace — selected twice. Comparing
+    // text alone would call the second a duplicate and silently drop its chip.
+    const first = fileSelection({ label: 'notes.md:12', startLine: 12, endLine: 12 })
+    const second = fileSelection({ label: 'notes.md:50', startLine: 50, endLine: 50 })
+
+    expect(prepareContextForInsert(second, [first])).not.toBeNull()
+    expect(prepareContextForInsert(first, [first])).toBeNull()
+  })
+
+  it('still dedupes identical text when the source has no line numbers', () => {
+    // The rich-markdown editor omits the range, so two identical passages are
+    // indistinguishable in the data model and deduping is the honest outcome.
+    expect(prepareContextForInsert(fileSelection(), [fileSelection()])).toBeNull()
+  })
+
   it('treats the same rows picked in a different order as one selection', () => {
     // Row ids iterate in click order, so re-picking the same rows differently
     // must no-op rather than add a second chip over rows already referenced.

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/copilot/components/user-input/utils.ts

@@ -254,7 +254,18 @@ export function areContextsEqual(c: ChatContext, context: ChatContext): boolean
     // already-referenced file would be swallowed as a duplicate.
     case 'file_selection': {
       const ctx = context as FileSelectionContext
-      return c.fileId === ctx.fileId && c.text === ctx.text
+      // Location too, not just the text: the same line can occur twice in a file
+      // (a repeated import, a closing brace), and comparing text alone would
+      // treat the second highlight as a duplicate and drop its chip. Where the
+      // source has no line numbers — the rich-markdown editor — both are
+      // undefined and identical text is genuinely indistinguishable, so it
+      // correctly still dedupes.
+      return (
+        c.fileId === ctx.fileId &&
+        c.text === ctx.text &&
+        c.startLine === ctx.startLine &&
+        c.endLine === ctx.endLine
+      )
     }
     case 'table_selection': {
       const ctx = context as TableSelectionContext