Fix unicode handling in PLE2515

Previously, we used byte indices when we should have used char indices, causing crashes when the were non-ascii characters before our replaces

Fixes #3716

crates/ruff/src/rules/pylint/rules/invalid_string_characters.rs

@@ -184,7 +184,9 @@ pub fn invalid_string_characters(
     let text = locator.slice(Range::new(start, end));
 
     for (row, line) in UniversalNewlineIterator::from(text).enumerate() {
-        for (column, match_) in line.match_indices(&['\x08', '\x1A', '\x1B', '\0', '\u{200b}']) {
+        for (column_bytes, match_) in
+            line.match_indices(&['\x08', '\x1A', '\x1B', '\0', '\u{200b}'])
+        {
             let (replacement, rule): (&str, DiagnosticKind) = match match_.chars().next().unwrap() {
                 '\x08' => ("\\b", InvalidCharacterBackspace.into()),
                 '\x1A' => ("\\x1A", InvalidCharacterSub.into()),
@@ -193,7 +195,8 @@ pub fn invalid_string_characters(
                 '\u{200b}' => ("\\u200b", InvalidCharacterZeroWidthSpace.into()),
                 _ => unreachable!(),
             };
-            let location = helpers::to_absolute(Location::new(row + 1, column), start);
+            let column_chars = line[..column_bytes].chars().count();
+            let location = helpers::to_absolute(Location::new(row + 1, column_chars), start);
             let end_location = Location::new(location.row(), location.column() + 1);
             let mut diagnostic = Diagnostic::new(rule, Range::new(location, end_location));
             if autofix {

crates/ruff/src/rules/pylint/snapshots/ruff__rules__pylint__tests__PLE2515_invalid_characters.py.snap

@@ -23,4 +23,67 @@ expression: diagnostics
           row: 34
           column: 13
   parent: ~
+- kind:
+    name: InvalidCharacterZeroWidthSpace
+    body: "Invalid unescaped character zero-width-space, use \"\\u200B\" instead"
+    suggestion: Replace with escape sequence
+    fixable: true
+  location:
+    row: 38
+    column: 35
+  end_location:
+    row: 38
+    column: 36
+  fix:
+    edits:
+      - content: "\\u200b"
+        location:
+          row: 38
+          column: 35
+        end_location:
+          row: 38
+          column: 36
+  parent: ~
+- kind:
+    name: InvalidCharacterZeroWidthSpace
+    body: "Invalid unescaped character zero-width-space, use \"\\u200B\" instead"
+    suggestion: Replace with escape sequence
+    fixable: true
+  location:
+    row: 39
+    column: 59
+  end_location:
+    row: 39
+    column: 60
+  fix:
+    edits:
+      - content: "\\u200b"
+        location:
+          row: 39
+          column: 59
+        end_location:
+          row: 39
+          column: 60
+  parent: ~
+- kind:
+    name: InvalidCharacterZeroWidthSpace
+    body: "Invalid unescaped character zero-width-space, use \"\\u200B\" instead"
+    suggestion: Replace with escape sequence
+    fixable: true
+  location:
+    row: 39
+    column: 60
+  end_location:
+    row: 39
+    column: 61
+  fix:
+    edits:
+      - content: "\\u200b"
+        location:
+          row: 39
+          column: 60
+        end_location:
+          row: 39
+          column: 61
+  parent: ~