refactor: remove the FieldKind::Bytes variant and use FieldKind::String instead.

Author: plusvic

lib/src/modules/mod.rs

@@ -401,7 +401,7 @@ pub mod mods {
                         }
                         RuntimeType::Bool => FieldKind::Bool,
                         RuntimeType::String => FieldKind::String,
-                        RuntimeType::VecU8 => FieldKind::Bytes,
+                        RuntimeType::VecU8 => FieldKind::String,
                         RuntimeType::Enum(_) => FieldKind::Integer,
                         RuntimeType::Message(m) => {
                             FieldKind::Struct(Struct::new(m))
@@ -433,8 +433,6 @@ pub mod mods {
             Bool,
             /// A string.
             String,
-            /// A bytes sequence.
-            Bytes,
             /// A structure.
             Struct(Struct),
             /// An array.

lib/src/modules/tests.rs

@@ -371,19 +371,19 @@ fn test_reflect() {
 
     let field = fields.next().unwrap();
     assert_eq!(field.name(), "bytes_foo");
-    assert_eq!(field.kind(), FieldKind::Bytes);
+    assert_eq!(field.kind(), FieldKind::String);
 
     let field = fields.next().unwrap();
     assert_eq!(field.name(), "bytes_bar");
-    assert_eq!(field.kind(), FieldKind::Bytes);
+    assert_eq!(field.kind(), FieldKind::String);
 
     let field = fields.next().unwrap();
     assert_eq!(field.name(), "bytes_raw");
-    assert_eq!(field.kind(), FieldKind::Bytes);
+    assert_eq!(field.kind(), FieldKind::String);
 
     let field = fields.next().unwrap();
     assert_eq!(field.name(), "bytes_undef");
-    assert_eq!(field.kind(), FieldKind::Bytes);
+    assert_eq!(field.kind(), FieldKind::String);
 
     let field = fields.next().unwrap();
     assert_eq!(field.name(), "enumeration");

ls/src/features/completion.rs

@@ -437,7 +437,6 @@ fn kind_to_string(k: &FieldKind) -> String {
         FieldKind::Float => "float".to_string(),
         FieldKind::Bool => "bool".to_string(),
         FieldKind::String => "string".to_string(),
-        FieldKind::Bytes => "bytes".to_string(),
         FieldKind::Struct(_) => "struct".to_string(),
         FieldKind::Array(inner) => format!("array<{}>", kind_to_string(inner)),
         FieldKind::Map(key, value) => {

ls/src/tests/testdata/completion8.response.json

@@ -338,7 +338,7 @@
   {
     "label": "pdb_path",
     "labelDetails": {
-      "description": "bytes"
+      "description": "string"
     },
     "kind": 5
   },