More improvements to error messages to better guide the LLM

Author: elifarley

src/cedarscript_editor/cedarscript_editor.py

@@ -220,10 +220,11 @@ def _apply_action(
 
             case MoveClause(insert_position=insert_position, to_other_file=other_file, relative_indentation=relindent):
                 # TODO Move from 'lines' to the same file or to 'other_file'
-                if range_spec <= range_spec_to_delete:
+
+                if range_spec < range_spec_to_delete:
                     range_spec_to_delete.delete(lines)
                     range_spec.write(content, lines)
-                else:
+                elif range_spec > range_spec_to_delete:
                     range_spec.write(content, lines)
                     range_spec_to_delete.delete(lines)
 

src/text_manipulation/range_spec.py

@@ -156,7 +156,8 @@ def from_line_marker(
             if search_term.offset is None and match_type_count > 1:
                 raise ValueError(
                     f"There are {match_type_count} lines matching `{search_term.value}`. "
-                    f"Suggestion: Try using a different line as marker (a couple lines before or after the current one)."
+                    f"Suggestions: 1) Try using a *different line* as marker (a couple lines before or after the one you tried); "
+                    f"2) If you wanted to *REPLACE* line, try instead to replace a *SEGMENT* of a couple of lines."
                     # f"Add an `OFFSET` (after the line marker) and a number between 0 and {match_type_count - 1} to determine how many to skip. "
                     # f"Example to reference the *last* one of those: `LINE '{search_term.value.strip()}' OFFSET {match_type_count - 1}`"
                     # ' (See `offset_clause` in `<grammar.js>` for details on OFFSET)'

src/text_manipulation/text_editor_kit.py

@@ -58,7 +58,7 @@ def marker_or_segment_to_search_range_impl(
         case Marker(type=MarkerType.LINE):
             result = RangeSpec.from_line_marker(lines, self, search_range)
             assert result is not None, (
-                f"Unable to find `{self}`; Try: 1) Double-checking the marker "
+                f"Unable to find {self}; Try: 1) Double-checking the marker "
                 f"(maybe you specified the the wrong one); or 2) using *exactly* the same characters from source; "
                 f"or 3) using another marker"
             )
@@ -83,9 +83,9 @@ def segment_to_search_range(
 
     start_match_result = RangeSpec.from_line_marker(lines, start_relpos, search_range)
     assert start_match_result, (
-        f"Unable to find segment start `{start_relpos}`; Try: "
+        f"Unable to find segment start: {start_relpos}; Try: "
         f"1) Double-checking the marker (maybe you specified the the wrong one); or "
-        f"2) using *exactly* the same characters from source; or 3) using a marker from above"
+        f"2) Using *exactly* the same characters from source; or 3) using a marker from above"
     )
 
     start_index_for_end_marker = start_match_result.as_index
@@ -96,8 +96,9 @@ def segment_to_search_range(
         start_index_for_end_marker, search_range.end, start_match_result.indent
     ))
     assert end_match_result, (
-        f"Unable to find segment end `{end_relpos}` - Try: "
-        f"1) using *exactly* the same characters from source; or 2) using a marker from below"
+        f"Unable to find segment end: {end_relpos}; Try: "
+        f"1) Using *exactly* the same characters from source; or "
+        f"2) using a marker from below"
     )
     if end_match_result.as_index > -1:
         one_after_end = end_match_result.as_index + 1