Refactor symbol replacement spacing and trimming logic for fragments and pasted text

- do we call trim() on text between then replaced symbols? we should

├── apps
│   └── editor
│       └── src
│           ├── utils
│           │   └── prompt-builder.ts
│           └── views
│               └── prompt
│                   ├── backend
│                   │   ├── message-handlers
│                   │   │   ├── handle-copy-prompt.ts
│                   │   │   ├── handle-preview-prompt.ts
│                   │   │   ├── handle-save-prompt-pasted-text.ts
│                   │   │   └── utils
│                   │   │       ├── build-cli-prompt.ts
│                   │   │       └── build-prompt-payload.ts
│                   │   ├── prompt-view-provider.ts
│                   │   └── utils
│                   │       ├── persistent-text-storage.ts
│                   │       └── symbols
│                   │           ├── fragment
│                   │           │   └── replace-fragment-symbol.ts
│                   │           ├── pasted-text
│                   │           │   └── replace-pasted-text-symbol.ts
│                   │           └── replace-symbols.ts
│                   ├── frontend
│                   │   └── hooks
│                   │       └── use-prompt
│                   │           └── use-prompt.ts
│                   └── types
│                       └── messages.ts
└── packages
    └── ui
        └── src
            └── components
                └── editor
                    └── common
                        └── prompts
                            └── PromptField
                                └── hooks
                                    └── use-handlers
                                        └── handle-paste.ts

Author: robertpiosik

apps/editor/src/views/prompt/backend/utils/symbols/fragment/replace-fragment-symbol.ts

@@ -14,7 +14,8 @@ export const replace_fragment_symbol = async (params: {
       const matches = [...instruction.matchAll(regex)]
       if (matches.length === 0) return instruction
 
-      let result = instruction
+      let result_string = ''
+      let last_index = 0
 
       for (const match of matches) {
         const full_match = match[0]
@@ -76,13 +77,26 @@ export const replace_fragment_symbol = async (params: {
           content = `// Error reading file ${file_path}`
         }
 
+        const text_between = instruction.slice(last_index, match.index)
+        const trimmed_text = text_between.trim()
+
+        if (trimmed_text) {
+          result_string += (result_string ? '\n\n' : '') + trimmed_text
+        }
+
         const range_str = ` (${start_line}-${end_line})`
-        const replacement = `\n\n\`${file_path}\`${range_str}\n\n\`\`\`\n${content}\n\`\`\`\n\n`
+        const replacement = `\`${file_path}\`${range_str}\n\n\`\`\`\n${content}\n\`\`\``
+
+        result_string += (result_string ? '\n\n' : '') + replacement
+        last_index = match.index + full_match.length
+      }
 
-        result = result.replace(full_match, replacement)
+      const tail = instruction.slice(last_index).trim()
+      if (tail) {
+        result_string += (result_string ? '\n\n' : '') + tail
       }
 
-      return result
+      return result_string
     })
   )
 }

apps/editor/src/views/prompt/backend/utils/symbols/pasted-text/replace-pasted-text-symbol.ts

@@ -41,19 +41,28 @@ export const replace_pasted_text_symbol = async (params: {
         const match = matches[i]
         const replacement = replacements[i]
 
-        result_string += instruction.slice(last_index, match.index)
+        const text_between = instruction.slice(last_index, match.index)
+        const trimmed_text = text_between.trim()
+
+        if (trimmed_text) {
+          result_string += (result_string ? '\n\n' : '') + trimmed_text
+        }
 
         if (replacement.success && replacement.content) {
           const backticks = replacement.content.includes('```') ? '````' : '```'
-          result_string += `\n${backticks}\n${replacement.content}\n${backticks}\n`
+          const block = `${backticks}\n${replacement.content}\n${backticks}`
+          result_string += (result_string ? '\n\n' : '') + block
         } else {
-          result_string += match[0]
+          result_string += (result_string ? '\n\n' : '') + match[0]
         }
 
         last_index = match.index + match[0].length
       }
 
-      result_string += instruction.slice(last_index).trim()
+      const tail = instruction.slice(last_index).trim()
+      if (tail) {
+        result_string += (result_string ? '\n\n' : '') + tail
+      }
 
       return result_string
     })