fix(chat): distinguish a line-less file-selection label from the whole file

Cursor Bugbot: a rich-markdown selection labelled itself with the bare file
name, which is exactly the whole-file chip's label. Menu-driven inserts reject
any context whose label is already taken (isContextAlreadySelected closes the
menu silently), so once a markdown selection was attached, mentioning that same
file was quietly dropped. One-way: the reverse order works because programmatic
inserts ordinalize through prepareContextForInsert.

Fallen out of dropping the fabricated line range from this editor — with no
range left, the label collapsed onto the file name. Fixed at the single source:
buildFileSelectionLabel now returns 'notes.md (selection)' when there is no line
range, so it stays honest about location while remaining distinguishable.

Author: waleedlatif1

apps/sim/lib/copilot/chat/selection-context.test.ts

@@ -22,8 +22,11 @@ describe('buildFileSelectionLabel', () => {
     expect(buildFileSelectionLabel('notes.md', 12)).toBe('notes.md:12')
   })
 
-  it('falls back to just the file name when the source has no line numbers', () => {
-    expect(buildFileSelectionLabel('notes.md')).toBe('notes.md')
+  it('marks a line-less selection so it cannot collide with the whole-file chip', () => {
+    // A bare 'notes.md' would equal the whole-file chip's label, and menu inserts
+    // silently reject an already-taken label — blocking the file's own mention.
+    expect(buildFileSelectionLabel('notes.md')).toBe('notes.md (selection)')
+    expect(buildFileSelectionLabel('notes.md')).not.toBe('notes.md')
   })
 })
 

apps/sim/lib/copilot/chat/selection-context.ts

@@ -44,9 +44,14 @@ export function truncateSelectionText(text: string): string {
 }
 
 /**
- * Builds the IDE-style chip label for a file selection, e.g. `notes.md:12-40`,
- * `notes.md:12`, or just `notes.md` when the source has no line numbers (the
- * rich-markdown editor, whose document model has no source lines).
+ * Builds the IDE-style chip label for a file selection, e.g. `notes.md:12-40` or
+ * `notes.md:12`. Without a line range — the rich-markdown editor, whose document
+ * model has no source lines — it falls back to `notes.md (selection)`.
+ *
+ * That suffix is load-bearing, not decoration: a bare file name would be
+ * identical to the whole-file `@notes.md` chip's label, and menu-driven inserts
+ * reject any context whose label is already taken (`isContextAlreadySelected`).
+ * A markdown selection would then silently block mentioning its own file.
  *
  * Kept ASCII so the label survives being inserted as an inline mention token in
  * the chat input. Labels must be unique across a message's chips — the caller
@@ -57,7 +62,7 @@ export function buildFileSelectionLabel(
   startLine?: number,
   endLine?: number
 ): string {
-  if (!startLine) return fileName
+  if (!startLine) return `${fileName} (selection)`
   const range = endLine && endLine !== startLine ? `${startLine}-${endLine}` : `${startLine}`
   return `${fileName}:${range}`
 }